Merge "Add additional tracking information to mediawiki.searchSuggest"

Add additional tracking information to mediawiki.searchSuggest

Adds a few pieces of information to improve tracking of autocomplete
usage.

* When using Special:Search 'go' feature forward wprov parameter to redirect
* Include a data attribute indicating autocomplete location to
  differentiate usage of the header and Special:Search content autocompletes
* Report exact query string that was used for impression-results
* Add handling to allow searchSuggest subscribers to append tracking
  information to generated article links
* Add a new hook, SpecialSearchGoResult, that can either change the url
  redirected to in the 'go' feature or cancel it entirely.

Bug: T125915
Change-Id: Iec7171fcf301f1659d852afa87ce271f468177c1

Merge "Follow-up e33d6f45a. Fix some mistakes w/ limit on QueryPage"

Merge "Revert "Allow auto suggestion for subpages of Special:BotPasswords""

Revert "Allow auto suggestion for subpages of Special:BotPasswords"

This is completely wrong. If you're going to auto-complete anything here (which is probably not really necessary), you need to auto-complete the "bot names" for the current user rather than every user name on the wiki.

This reverts commit aa8823ac4b58be611caf2595ad3c7b0ecacbc8ac.

Change-Id: I5d477198827c107f947d25dddc2513f12d2bfd62

Merge "Revert "Templatize Special:Contributions lines""

Revert "Templatize Special:Contributions lines"

Breaks RevisionDelete.

This reverts commit 1bce6db10e99840cfbd10fa074c93d8d96d88412.

Change-Id: I8f63e002e34a9d13e2597a27e3dd918a687d1e7c

Fix parserTests.php script

 * Follow up to 60e4f3fd

Change-Id: If1370a720da21c3662fda4100c96b1758ddc1dc0

Follow-up e33d6f45a. Fix some mistakes w/ limit on QueryPage

Fix some rather embarrassing mistakes. See comments on e33d6f45a.

Also change the behaviour when someone sets an offset higher than
max allowed. Returning a page with 0 results and broken prev/next
links is poor UI imo.

Change-Id: Ibfc983675ae0c600eeccd5e361550e9b5f96f5fd

Merge "mediawiki.ui: Button group active buttons unchanged on user interaction"

Merge "Upstream `isElementInViewport` from MobileFrontend"

Merge "Templatize Special:Contributions lines"

Merge "MediaWiki Widgets: Add new SearchInputWidget"

mediawiki.ui: Button group active buttons unchanged on user interaction

Applying default cursor and making active button's hover state behave same
as normal state for improved user experience.

Bug: T77916
Depends-On: Ib89500d70b3e79edcf9ba989f05a311e0edec82b
Change-Id: I8fd74c28ccb89df1e257aa3d63d362697c4542ed

Templatize Special:Contributions lines

Bug: T122537
Change-Id: I11aac43de495881e10e393d075a231bd346ea547

Integrate Less helper `.mixin-screen-reader-text()`

Adding Less parametric helper mixin for screen reader users.
This will help with changes such as
I7db2d7e5387f1a4392d587764598dbd82dd0a5cd

Bug: T119444
Change-Id: Ie9bce512804385578361eb77579106c799d5ff57

Merge "Prevent "Failed to create..." warnings when session loading fails"

MediaWiki Widgets: Add new SearchInputWidget

SearchInputWidget is similar to a TitleInputWidget, but doesn't has
a visible loading indication, doesn't highlight the first result and
uses the opensearch api endpoint for suggestions, instead of
prefixsearch.

Extra points:
 * Improve documentation of mw.widgets.TitleInputWidget's configuration
   option validateTitle

Bug: T118443
Change-Id: I8b8098041fe2971389fa908d007d2e77255829ec

Merge "Add tags support to patrol, protect, unblock, and undelete"

Merge "installer: Update main RequestContext when updating $wgLang"

Merge "tests: Ensure main RequestContext reflects custom $wgLang"

Merge "maintenance: Remove odd use of $wgLang"

installer: Update main RequestContext when updating $wgLang

Change-Id: I1e3b76b2358a90a551616dc84eb6f2be6bb41c0a

tests: Ensure main RequestContext reflects custom $wgLang

Call RequestContext::resetMain() so that any subsequent call
to getMain() and getLanguage() is aware of the latest state
as set up by the setUp() and setupGlobals() methods.

The MediaWikiTestCase class for PHPUnit did this already, but
the parserTest suite and UploadFromUrlTestSuite didn't yet.

Change-Id: I6481176228944004091078704d0346c8f3fc0cf1

Merge "filebackend: Clean up TempFSFile and fix IDEA errors"

maintenance: Remove odd use of $wgLang

* testCompression.php: Called $wgLang, but didn't set it.
  It just assumed it was English. Use a local variable instead.

* update.php: Set $wgLang to English. Set main RequestContext as well.

* transstat.php: Rename $wgLanguages to $languages for less confusion.
  It's just a local instance of language.inc's Languages class.

Change-Id: I67865f4d8710946001f7586ab0a17e125366b184

filebackend: Clean up TempFSFile and fix IDEA errors

Change-Id: I4e25e3bf906fa3a918f4462fac1a6be5e85696aa

CSSMin: Remove file-existance filter in getLocalFileReferences()

Follows-up 8f5cd11d82fa.

The old getLocalFileReferences() method is no longer used anywhere.
Remove it and rename getAllLocalFileReferences back to it.

Change-Id: I864258aad128ba9b54464c7bc854543f2937f977

resourceloader: Don't cache CSSMin::remap() calls

The output of this function is not deterministic. This function
is what tranforms urls to include content hashes and also embeds
content as data URI. The calls to this function are expected to be
up to date and matching the version hash as constructed by the
current web request.

Previously, the content hash as constructed based on the collective
file hash of all references images was accurate but the generated
css associated with it could be stale.

Unless we abuse MemoizedCallable to take a separate cache buster
(with $localFileRefs?), it's best to remove this as it isn't
a deterministic function and shouldn't be memoized.

Bug: T128668
Change-Id: Icb87ddc585d7320ac48619446bb8b9cbe5f4780e

Prevent "Failed to create..." warnings when session loading fails

If there is an existing session for a given ID but loading it fails,
there is no point in trying to create a new empty session with that
ID. Just fail silently (the reason for not loading the session
should be logged elsewhere), don't spam the logs and don't slow
down execution by throwing and catching an exception.

Change-Id: I8299872cc29c32cb245686ed0bca6b9a5902cdc1

Linker: Avoid passing false to Title::newFromText

This is needed because the call to substr() returns false when the string
is only 1 character long, which occurs when parsing degenerate links like
[[:]] and [[::|foo]].

The seemingly unnecessary test for $match[1] !== '' is for forwards
compatibility with PHP 7.

Bug: T121706
Change-Id: Icc19ee990d01958d64b938d298e9a7e1df7181b5

Upstream `isElementInViewport` from MobileFrontend

As a useful utility function, we've copied this method several times
across multiple extensions, which is a pretty good sign it should
actually live in core.

Changes:
 * Add `mediawiki.viewport` module
 * Rewrite method to be more robust and accept any viewport
 * Add `mw.viewport` to jsduck categories file
 * Add method for checking if an element is close to the viewport
 * Add unit tests

Bug: T124317
Change-Id: I38eec4f1e568f51e7e212b2b3f10b8da8d36f316

api: Fix "PHP Notice: Undefined variable: config"

Follows-up 362c220aa1, which split the method up but didn't
re-create a local $config variable.

Change-Id: Ic32ac3c600d0b4d69e8129fa50ea47d9a6a54b20

Localisation updates from https://translatewiki.net.

Change-Id: Ib781f4c698a70de2afd5db1c3a0c58bf8f4285f0

Merge "Add ContentHandler::supportsCategories method"

Merge "AutoloadGenerator: Account for PHP 5.5 syntax like "self::static""

Fix trivial PHPCS violations in */skins/* directories

Due to T127238, files in various */skins/* directories are not checked
by PHPCS. Temporarily removed the exclude rule from phpcs.xml and ran:

  composer fix includes/skins/* tests/phpunit/includes/skins/* tests/phpunit/skins/*

Change-Id: I9240c1cee825920b6634903282be6252cce55686

Merge "Revert "Deprecating: Consolidating `progressive` & `constructive` buttons""

Revert "Deprecating: Consolidating `progressive` & `constructive` buttons"

I merged it too early, sorry. Deployment is planned for a week later.

This reverts commit cb8fa5fd173f4cfdb9945019fa9ac894eafa3ffe.

Bug: T110555
Change-Id: I369fa6d2b4f6531f7ec475c08cba283d323a0ffb

Merge "Add help link for Special:ChangeContentModel"

AutoloadGenerator: Account for PHP 5.5 syntax like "self::static"

Bug: T128625
Change-Id: I905b549a738b76c07c56ec92f500553c1d48f558

Merge "Allow FSFile objects for src in FileBackend::do*Operations"

Allow FSFile objects for src in FileBackend::do*Operations

Convenience aside, this lets multiwrite backends do async replication for
"store" operations safely, buy keeping a handle to the source file that
prevents it from getting prematurely deleted before the post-send writes
to the secondary backends can happen.

Bug: T91869
Change-Id: I1254de527c47835c35fed6e526b42953c1b2b2ca

Add help link for Special:ChangeContentModel

Bug: T128556
Change-Id: Id55a463cf6eac4f20e800ae49448e6007f68a3c4

Fix use of RawMessage in Status::getMessage()

RawMessage::__construct() takes a single array of params, it's not
varargs-style like wfMessage().

Change-Id: I63950ee16e359aa2627228350e27d9e94bb6e8ce

Merge "Deprecating: Consolidating `progressive` & `constructive` buttons"

Add ContentHandler::supportsCategories method

and check for this in WikiPage::doEditUpdates before
inserting a new CategoryMembershipChangeJob.

Some content models like the Wikibase ones do not
have categories and it's wasteful to add these jobs
for all Wikibase edits.

Bug: T126977
Change-Id: I2c54a4ba1546445dc41101e15cb83a2c6cc2b1c9

Merge "Fix comment in SearchEngine.php"

Merge "objectcache: Fix MultiWriteBagOStuff docs ('asyncHandler', not 'asyncCallback')"

objectcache: Fix MultiWriteBagOStuff docs ('asyncHandler', not 'asyncCallback')

Follows-up 1171cc00cd.

Change-Id: I97189e916d0aba507273216e3e9ab28ce0d5ffec

Merge "Convert remaining array() to [] in API"

Merge "mw.Upload.BookletLayout/Dialog: Add determinate progress bar"

Merge "findHooks.php: Use recursive directory iterator"

Merge "WebRequestUpload: Use canonical header name in getHeader()"

findHooks.php: Use recursive directory iterator

The following dirs were not previously included:
* includes/compat/
* includes/compat/normal/
* includes/composer/
* includes/config/
* includes/db/loadbalancer/
* includes/debug/logger/
* includes/filebackend/filejournal/
* includes/filebackend/lockmanager/
* includes/jobqueue/aggregator/
* includes/jobqueue/jobs/
* includes/jobqueue/utils/
* includes/libs/
* includes/objectcache/
* includes/pager/
* includes/poolcounter/
* includes/profiler/output/
* includes/profiler/
* includes/registration/
* includes/templates/
* includes/tidy/
* includes/title/
* includes/utils/iterators/
* includes/widget/
* maintenance/ subdirectories

Change-Id: I6c01b8768f9f70152e320d254f8025a0ffde8c62

user: Remove obsolete note in docs/memcached.txt

The User object cache hasn't been that since b3acd4f.

Change-Id: Ia7ab78ef22c24a7421ea25db1440e7267f0a725d

Update OOjs UI to v0.16.1

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.16.1/History.md

Change-Id: I4bd51d4c105a09d71a911e04538b51f58c3ea721

Localisation updates from https://translatewiki.net.

Change-Id: I7c15b259b0f88dbc05ce1520ecb319a66eed6f73

Merge "Add mobile target to mediawiki.special"

Merge "Add access modifiers to MagicWordArray.php"

Merge "Add access modifiers to MagicWord.php"

Merge "Move MagicWordArray class to own file"

Merge "Rm unused $config from SpecialRecentChanges::getDefaultOptions"

Rm unused $config from SpecialRecentChanges::getDefaultOptions

Change-Id: I5dc8f091c06b0d32d5d84af885608a57ca9c054a

Add mobile target to mediawiki.special

Bug: T126935
Change-Id: I1135397d150b5ad978da88bfd9d85c1d9e294eb7

Merge "Reset scoped session for upload jobs after deferred updates"

Merge "Allow API action=logout with BotPasswords"

Merge "Unpersist the session on session load failure"

Add access modifiers to MagicWordArray.php

Change-Id: Ib5fe0fd22a6f24142f1e87338298c1bdef45cc4a

Add access modifiers to MagicWord.php

Change-Id: I3588d9d2c17203ec3ff11736cd6ce84f687677a5

Move MagicWordArray class to own file

Change-Id: I9ba602da77f1c8d4270d8ae10d03e82a3197e5b1

Fix comment in SearchEngine.php

Change-Id: Ib00b42d2210be5bc1125fa7ba74c27a5d7fbf36c

Merge "Unpersist the session on logout"

Allow API action=logout with BotPasswords

Since login is handled specially, do the same for logout.

Bug: T128335
Change-Id: Ib3b12d7045449b5c44f5ff7d1ecfce14416e8400

Merge "Use static::class instead of get_called_class()"

Cache user data in memory

Stores user data looked up in WAN cache in memory so that lookups in the
same request do not result in new memcached lookups.

Quick and dirty solution, but nicer ones are more difficult:
* no idea what replacing WANObjectCache::get/set with getWithSetCallback
  (which has its own in-process cache) would do, the code is complex
  and completely different
* would be nice to wrap the logic into a proxy object (like CachedBagOStuff)
  but WANObjectCache calls set() internally (and marks it final), so
  inheriting is not safe; the interface and implementation should be
  separated, and that means updating all external callers which do a type
  check.
* ObjectCache::getInstance('hash') cannot be used because it has no
  item limit and this could eat up the memory with a script that iterates
  through lots of users

The patch does not attempt to replicate tombstoning for
User::clearSharedCache('refresh').

Based on Iec1504700a and Idef9a9d3.

Change-Id: I419f356b0c306d16711b433da95dccdb44645154
Co-Authored-By: Ori Livneh <ori@wikimedia.org>
Bug: T128157

Merge "Revert "Add type hint against LinkTarget""

Merge "Add CategoryMembershipChangeJobTest"

Merge "CategoryViewer: Do opportunistic category recounts for mismatched file counts, too"

Merge "mail: Turn UserMailer::quotedPrintableCallback into an inline closure"

Merge "media: Fix casing of IPTC::parse() in tests"

Revert "Add type hint against LinkTarget"

Can cause fatal if your database has invalid titles
stored in it (Title::makeTitleSafe is fed to $lb->addObj()
without checking that the result is non-null)

This reverts commit 0e5a1f8879b211b44416e3762bf2d76c28f3f569.

Change-Id: I4174b2fdcc3d8e24ecf0bcc39cd0f0f2e8bfd6b2

Add CategoryMembershipChangeJobTest

Change-Id: Ia04bb85260be7d2aaaf171a2c886633b66ee6e77

mail: Turn UserMailer::quotedPrintableCallback into an inline closure

Change-Id: I1b11a95d47caeaf02291a5ab522b57d7570ca738

media: Fix casing of IPTC::parse() in tests

Change-Id: Id7271fe635c92f2c82c6e45c64a8b23b69bc3c03

mail: Change $wgNoReplyAddress default to $wgPasswordSender

Stop using a TLD in the EmailNotification class that can now
potentially become a valid domainname.

Bug: T97711
Change-Id: If69d5f32b97320986b14260c851d4aca0d28aa69

Convert remaining array() to [] in API

Change-Id: Ib44d59ff0cf0d6d8efc462671cac716dbead8ffc

Use static::class instead of get_called_class()

Available as of PHP 5.5 and more idomatic. Foo::class (explicit),
self::class (defined), and static::class (late bound).

Change-Id: I66937f32095a4e4ecde94ca20a935a3c3efc9cee

Unpersist the session on session load failure

There's no point in keeping broken cookies around, it just means all
future requests will continue to flood the logs.

Change-Id: Ib10c9ed9049b71ed434950fc731ea77960ceca0c

CategoryViewer: Do opportunistic category recounts for mismatched file counts, too

There are some categories on Commons that have no files, but are
recorded to have some, e.g. [1] appears to have 15 files [2]. We have
logic to fix these, but it wasn't being checked for file counts (only
pages and subcats).

[1] https://commons.wikimedia.org/wiki/Category:Police_of_Mexico,_D._F._vehicles
[2] https://commons.wikimedia.org/w/index.php?title=Category:Police_of_Mexico,_D._F._vehicles&action=info

Change-Id: If88fc15347eb592e957452884c2179d7c237bd2c

Add tags support to patrol, protect, unblock, and undelete

- Add 'tags' parameters to appropriate API modules
- Add tag-adding logic to appropriate functions that carry out
  relevant functions
- ManualLogEntry::{set,get}Tags to handle adding tags to log
  entries in a cleaner fashion
- Use ManualLogEntry::setTags in LocalFile::recordUpload2

Bug: T97720
Change-Id: I98c52da7985623bfdafda2dc2dae937b39b72419

Localisation updates from https://translatewiki.net.

Change-Id: I5da9bd040f15c0ded7c5b507c36f6d31a1dd049a

Unpersist the session on logout

Clearing the cookies in this case is probably a good idea.

This also clears cookies when a non-persisted session's metadata is
dirty, for parallelism with what happens to persisted sessions.

Bug: T127436
Change-Id: I76897eaac063e5e3c3563398d0f4cb36cf93783b

Revert "Prevent duplicate memcached lookups for user record"

This reverts commit 7d67b4d9195346e1.
Caused T128246, T128263, T128275.

Change-Id: I639ea59e2d42ffbbf700451074174720f2f45cf4

Merge "MimeMagic: Recognise font files"

Merge "resourceloader: Remove PHP4-compat code"

Merge "Add request error state to ApiBase::logRequest"

Merge "Update ApiAction logging channel values"

Merge "Remove fix for a 5.3 problem"

Merge "Remove useless PHP version comparison from wfDebugBacktrace()"