Merge "Include AuthManager API endpoints in authmanager channel"

Merge "Add dependency to 'jquery.textSelection' for 'mediawiki.toolbar'"

Merge "Show wgPageParseReport on page previews too"

Merge "Fix increment() statsd call in JobRunner"

Fix increment() statsd call in JobRunner

Change-Id: I17e04db59a44a491aae99c4542216316361010a0

Fix for "Invalid key type: integer"

MWCryptHash::hmac requires a string, but mt_rand() returns an integer.
This issue resulted in an uncaught exception in a fresh installation
of mediawiki.

Imported from https://github.com/wikimedia/mediawiki/pull/55

Change-Id: Idccf5f230bfc2de30357b03c78c51cdad4839515

Fix inconsistent RevDelFileItem visibilities

Bug: T142228
Change-Id: Ib5a0a1adbea2daf2b035cf6be367841362de03cc

Merge "Keep $user->mEditCount up to date"

Merge "SpecialProtectedpages: Use LinkRenderer instead of Linker::link()"

Merge "Unify Special:Contribs/DeletedContribs subtitle implementation"

Show wgPageParseReport on page previews too

Also make EditPage::getPreviewLimitReport return the same
output instead of using copy-pasted code from the old hook.

Bug: T142210
Bug: T142214
Change-Id: I16c3c2da067fd24bf7ae564cb1d61beaa3be58c1

Merge "SpecialContributions: Use LinkRenderer instead of Linker::link()"

Merge "Provide context in ContributionsToolLinks hook"

Merge "Change explicit class to self in AutoloadGenerator"

Localisation updates from https://translatewiki.net.

Change-Id: I049258472a2111a94733403ee8eb8a36bd0280cc

Change explicit class to self in AutoloadGenerator

Follow-up: I3a426b92892f4c00cab33a13f6a717751120367c

Change-Id: I9d91ca42dbf41604639fff2ca00efa6fb262cb3d

build: Fix stylelint devDependencies

Change-Id: I26cee89c27e3cd8f3c1fc48088b2a8e8b3927e4a

Keep $user->mEditCount up to date

Whenever User::incEditCount() is called,
this tries to keep the user object up
to date so hooks can read the edit count
without reloading the user from the db.

Another option would be invalidate the
instance cache and let the read
repopulate it. It would add a db access
on each edit.

Bug: T128249
Change-Id: I79194c41d6b2fd84ad658909a2941d9d3d28d94e

Add dependency to 'jquery.textSelection' for 'mediawiki.toolbar'

Change-Id: Ie3b001551802364b488ba84588a028c44c73e755

Include AuthManager API endpoints in authmanager channel

Change-Id: I0fa6e9687d02a67c5d36e16b3827e7cc3beb8259

Merge "Balancer style tweaks"

Merge "Balancer: remove redundant assignment"

Merge "Balancer: consistent single-line comment style"

Merge "resourceloader: Add missing @private and @protected to mw.loader methods"

resourceloader: Add missing @private and @protected to mw.loader methods

Several local functions were indexed by JSDuck as mw.loader methods whereas
they are in fact they don't even exist on that object. Mark them as @private.

Also add @protected on the internal methods 'work()' and 'require()',
which are public but not for general use (and may change without warning).

Change-Id: If521cc5ae4a565a46161890cf32e95a8c7f4ca50

SpecialProtectedpages: Use LinkRenderer instead of Linker::link()

And remove an unused variable while we're at it.

Change-Id: I494caccfff4fb0510f26554395150e59c728c5a1

Unify Special:Contribs/DeletedContribs subtitle implementation

Basically all of the code was duplicated, except that Contribs had a
link to DeletedContribs, and DeletedContribs had a link to Contribs.

Now DeletedContribs will call the function in Contribs, and remove the
DeletedContribs link, and add its own link to Contribs instead of
needing to duplicate the entire list of links.

Change-Id: Ia004a4a69427fd5b554131b19b0e263523b7da67

Balancer style tweaks

* Use for loops where appropriate, instead of while
* De-indent a large block which was unnecessarily indented
* Use camel case for variable names, per the style guide

Change-Id: I0b2c37fdcab7f7238db0393085c43297e7a03ab2

Balancer: remove redundant assignment

Change-Id: I6c22d6227e43a2c5be454955eff6b053a94a1657

resourceloader: Disable filesystem cache for Less

Follows-up ea862efa7, which enables use of Less_Cache as part
of the switch from leafo/lessphp to oyejorge/less.php.

Since then 12afb3607 introduced a better caching method in
FileModule::compileLessFile(), which:

* Uses APC where possible, with fallback to main objectcache.
* Verifies both the input file and the indirectly used files have
  not changed before using the cache. The built-in cache system
  only seems to check the root file.

As such, we're incurring the overhead of generating them and storing
on-disk, but raraly ever making use of them. (And even if we do
use it when the APC object first expires, it's risky due to the
lack of indirect file validation).

Bug: T121519
Change-Id: I6e3c66f86a536463488697efc166fb704be0715c

SpecialContributions: Use LinkRenderer instead of Linker::link()

Change-Id: I9d6b81d5c71ab56fef2fe4bd95cfa1a6c38cbc32

Provide context in ContributionsToolLinks hook

Most of the hook functions need context to see what the current user's
permissions are, to generate messages, or the LinkRenderer service to
generate the tool links.

Change-Id: I19fa27c8115ee39dded6cb98f29c35b66b934f8a

Localisation updates from https://translatewiki.net.

Change-Id: I7f4659b50dcebe65487d1cc01ab9b1b6aeefbcec

Merge "Add search-ignored-headings string, copied from cirrus-search-ignored-headings."

Balancer: consistent single-line comment style

Also break a line that was over 100 bytes

Change-Id: I875d572d4147f2438526a49ca6cb5b73907bdc9b

Merge "Allow closures as HTMLInfoField values"

Merge "resourceloader: Implement 'deprecated' option for FileModule"

Add search-ignored-headings string, copied from cirrus-search-ignored-headings.

Change-Id: I4178f872996800379843301f7119840f4a4551df

resourceloader: Implement 'deprecated' option for FileModule

ResourceLoader modules can now carry a 'deprecated' option which can
be a boolean or an object with message key. This message or a default
deprecation message will be show whenever that module is used in production.

Note: This will not work in debug mode for ResourceLoaderFile modules
and this is deemed acceptable for the time being. We can revisit later.

Bug: T137772
Change-Id: Ib9ebd2d39a59fd41d8537e06884699f77b03580c

Autodiscover extension unittests

Instead of requiring every extension that wants to add unit tests to
copy the exact same boilerplate over and over, let's just automatically
discover them. We now have an extension registry, so we know exactly
which extensions are loaded (this won't work for extensions not being
loaded through extension.json).

For each extension, we check to see if the directory "tests/phpunit/"
exists, and if it does, add those unit tests. If there is a
`UnitTestsList` subscriber already set, PHPUnit will automatically
de-duplicate test cases so we won't be running anything twice.

Change-Id: I6ec654ef2d8ee3630b121b1277b4ee21ba0b6cd4

Merge "Make content handlers assemble content for search"

Merge "MediaHandlerFactory: Don't use any global state"

Merge "Move MediaHandler defaults out of global scope"

Merge "Override MediaHandlers in tests using MediaWikiServices"

Merge "Introduce MediaHandlerFactory to create MediaHandler objects"

Merge "OutputPage: Add addHeadItems() method"

Merge "Pass $archivedRevisionCount to the ArticleDeleteComplete hook"

Pass $archivedRevisionCount to the ArticleDeleteComplete hook

Bug: T134502
Change-Id: I1df89beeadbec2a3592e51fbb525ed6db2424619

Merge "phpdoc File::makeTransformTmpFile can return null"

MediaHandlerFactory: Don't use any global state

Instead inject the configured MediaHandlers in the service constructor.

Change-Id: I039c01ef531389c74524cb7adcb8cf1229d9a95d

Move MediaHandler defaults out of global scope

The defaults that were in $wgMediaHandlers are now listed in
MediaHandlerFactory.

The main advantage of doing this is we get O(1) replacement when
extensions set a media handler in their extension.json.

Bug: T141305
Change-Id: I05771a673837ab8d6331eedc24eb707be7f3a250

Override MediaHandlers in tests using MediaWikiServices

Change-Id: Ie39fd9243e7817191b87179f792d932f7b96de0c

Introduce MediaHandlerFactory to create MediaHandler objects

This will allow further refactoring of override logic in parser tests.

Ideally the factory class would not use $wgMediaHandlers directly, but
that ends up breaking too many tests for now.

Change-Id: I34a63ee7089ff26f86f3dd6f3cd1a37928bc4005

Merge "Wrap revision info and nav in a div for easy formatting in MobileFrontend"

Localisation updates from https://translatewiki.net.

Change-Id: I0ac53363139c522e4b35b8fbe49a7ac8d5ccbf45

Merge "Pingback: Fix phpdoc"

Merge "auth: Fix case of function call"

phpdoc File::makeTransformTmpFile can return null

Change-Id: Ibeefb87919dfa2e5f850570942cbeab01e84e8a7

Update OOjs UI to v0.17.7

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.17.7

Change-Id: I0f36a3485a75ad0346eef9ae98aa47eddb099faf

Structure test: Add autoload.php order check

If a class is added manually to the autoload.php another run of
the maintenance script will probably add unnecessary noise to
another change. The added structure test checks, if the output of
the maintenance script equals to the contents of the commited
autoload.php.

Bug: T121921
Change-Id: I3a426b92892f4c00cab33a13f6a717751120367c

Pingback: Fix phpdoc

Change-Id: Ib4fd932a9b65e46e9b43e594a6e1ab5a4dac76d9

auth: Fix case of function call

Change-Id: I415a7f7ba2ea97515a110cd963ed96bb77ec7b97

Wrap revision info and nav in a div for easy formatting in MobileFrontend

Revision info and nav used to be combined with breadcrumbs which made
styling the revision info and nav complicated. The patch wraps these
two divs into an outer div so that MF can style them together easily.

Also make sure rev info and nav appear on different lines as the
`br` tag between them has been removed because of wrapping.

Bug: T139930
Change-Id: Ic7ecf3be7eac6500c20cfc3b3b1fa49d4b4e0d19

Merge "UploadBase: Formally deprecate old methods for stashing files"

UploadBase: Formally deprecate old methods for stashing files

Change-Id: I4c60083a4e773e401aae8c90b99a051cbca4c961

Merge "Add 'targets' => [ 'desktop', 'mobile' ] to all UploadWizard dependencies"

Merge "Skin: Remove long-deprecated aliases for Linker methods"

Merge "Canonicalise '@deprecated since' doc comments"

Canonicalise '@deprecated since' doc comments

Change-Id: I440041512feb47a998efb45e76e3fd00e0655752

resourceloader: Separate 'user.styles' module from 'user'

Fixes addModuleStyles() violation from T92459 (a464d1d41).

Similar to 93ed259cf and Id2342454b for the 'site' module.
Doesn't need to be in separate commits since per-user HTML isn't
cached the same way.

Bug: T92459
Bug: T108590
Change-Id: I195f67d061de1306c97413aada7919e9f1b1d12c

Skin: Remove long-deprecated aliases for Linker methods

They don't seem to be used anymore in any skins in our version control.
I searched for:

  (->|(?<!Linker)::)(commentBlock|generateRollback|link|linkKnown|userLink|userToolLinks)

Change-Id: Id558c2d307dc6e9fbf0e642d5d638f7c2c9544b2

Allow closures as HTMLInfoField values

This makes it possible to parametrize info fields so they can be
easily altered.

Change-Id: I17328cf3f1a710096c64b17dde0864cb9be3892d

Merge "Remove unused conds.mustache test file"

Localisation updates from https://translatewiki.net.

Change-Id: If70dbb59179e6529b838f963aca55c4dab68f663

Merge "Provide missing system message corresponding to "right-purge""

Merge "Move HTMLFormField subclasses to a separate directory"

Merge "Avoid stack overflow in LoadBalancer with CACHE_DB WAN/server cache"

Avoid stack overflow in LoadBalancer with CACHE_DB WAN/server cache

* Add the ability to expose key BagOStuff attributes like whether the
  emulation SQL cache is being used. Several callers use hacks to detect
  this and can be updated later.
* Fallback to a safe empty WAN cache in the CACHE_DB case. This fixes
  a regression from f4bf52e8438.
* Also add this protection to the server cache, which could break
  similarly before too.
* Also fix CACHE_ANYTHING by avoid the recursion risk from
  fc1d4d796024 by just checking the disabled service map.

Bug: T123829
Bug: T141804
Change-Id: I17ee26138f69e01ec1aaddb55ab27caa4d542193

Merge "MessagesGom_deva: Correct syntax in namespace alias"

Merge "resourceloader: Remove styles from 'site', depend on 'site.styles'"

Merge "Unification of Czech terminology in Special:Listgrants"

Provide missing system message corresponding to "right-purge"

Change-Id: I2f98da7e1537da9d9b9a02f881465be11c31d698

Unification of Czech terminology in Special:Listgrants

The title of Special:Listgrants was "Svolení" in Czech
and the name of the special page was "Seznam grantů".
None of this has the exact meaning as the original title/name.
The Czech word "Grant" (another form of "grantů")
usualy means a financial gift from somebody limited by its purpose.

The Czech word "Svolení" has similar meaning as "agreement" in English.

"Skupina oprávnění" can be translated as "Group of permissions".

I've changed related pages in translatewiki.net to update
the description, displayed title and the title of the first
column in table which is generated by this special page.

It has been suggested by Dvorapa at my talk page in cswiki, see
https://cs.wikipedia.org/?oldid=13980091#Granty.3F_Svolen.C3.AD.3F
(in Czech).

Change-Id: Ic3148fc689de32bcba6344ad1f98491c5f540973

Remove unused conds.mustache test file

Follows-up 2b7d281 which added the file but no use of it.

Change-Id: I59faab7f50abedeb1d57663128872032b6ec28a2

Merge "phpdoc File::transform can return ThumbnailImage"

phpdoc File::transform can return ThumbnailImage

Change-Id: I463176d06c8d519a52c5713eacd44bff87e72f56

Localisation updates from https://translatewiki.net.

Change-Id: If2dbb58d65cef8c68255275a70d3ecc96bc47488

Merge "Allow SVGs encoded as WINDOWS-125[0-8]."

Merge "$namespaceNames for tcy language"

Allow SVGs encoded as WINDOWS-125[0-8].

The check is meant to prevent weird encodings like UTF-7 or HZ.
Encodings like the WINDOWS-125X family which are extensions of ascii
are safe. Additionally people still use windows-1252 on rare occasion.

Bug: T72937
Change-Id: I6cd63274cc04a7fca3afd244b4122ea64042dced

Move HTMLFormField subclasses to a separate directory

It's getting more difficult to navigate the files in includes/htmlform/
with every new field and every new helper class that is being added.

Change-Id: I92ce2356baf6151f17b2440970d5abdf86503820

$namespaceNames for tcy language

Bug: T140898
See-also: I3ddfc12efff0911b547ac93ab31c92d49447b224
Change-Id: Ie8a22d4d679d368aa5777a485ab5e18282c55a4c

Merge "Make recordUpload2() deferred updates PRESEND again"

Make recordUpload2() deferred updates PRESEND again

Bug: T141687
Change-Id: Ica1a09a6e580b9da659d22183f396d88f694f256

Merge "Revert "README: Changed link from bugs.mediawiki.org...""

Revert "README: Changed link from bugs.mediawiki.org..."

The link is intentional as that is the canonical location
of where bugs should be filed, and is independent of any
future software change.

Finally, changes to the README should be drafted on-wiki first
at [[mw:README]].

This reverts commit af9258794cfd59d1209b97080c365060cbbd2e89.

Change-Id: Ie453cee5218261cba4f6e18d13c4089c23ff2166

Merge "Remove DifferenceEngine::generateDiffBody()"

Merge "Localisation, minior cleanup, minor changes"

Localisation, minior cleanup, minor changes

Bug: T141743
Change-Id: I6eb7a4ae2bba0708ec96568d90b43170dbde7f6b

Remove DifferenceEngine::generateDiffBody()

Bug: T122754
Change-Id: Iedd319fa5e7b7218ee5341f18f31a8f1101b5eee