ResourceLoaderImage: Add basic tests

Also remove some wrappers from ResourceLoaderImageModule tests that
are no longer necessary, since the files they were mocking now exist.

Bug: T86334
Change-Id: If02e58716ce8e6c8327c8939c6c6425bd48bb560

ResourceLoaderImageModule: Rename 'image' in descriptors to 'file'

Like in OOjs UI. Less confusing. No one is using this yet.

Bug: T93669
Change-Id: Ib1c9af96b46ba660b3697d272129e071da1fdf18

ResourceLoaderImageModule: Remove 'type' stuff

Provides no value and makes the definitions uglier. Also, the
implementation sucks.

This is a breaking change, but ResourceLoaderImageModule was not in
any public release yet.

Submitted patches to fix two usages in extensions:
* Gather: I371209afe7b48e7c215ea9912826d4eb2cacf4e5
* MobileFrontend: I1963f5fe759c3a031220157d3a6f0f3b42bc5426

Bug: T94073
Change-Id: I36cfdb09bb203b8d9958e6016447e446dd6ff78b

ResourceLoaderImageModule: Implement CSS selector templates

Instead of a 'prefix', a 'selector' can be specified, with a string
containing a simple template to use for CSS selector. For example:

    'selector' => '.mw-ui-icon-{name}'

'prefix' continues to work as before.

When using variants, one might want to provide separate
'selectorWithoutVariant' and 'selectorWithVariant' options.

Available variables are {prefix}, {type}, {name}, and {variant}.

Bug: T78215
Change-Id: I99ccaf25e8d24fed5afd0c4b770d2f389789ce4b

ResourceLoaderImageModule: Add basic tests

Bug: T86334
Change-Id: Ie98bf5dec2d4d4c309ed5075b3191da3017f5ad4

ResourceLoaderImageModule: Improve PHP 5.3 compatibility

In PHP 5.3.x and older, the isset() check in code example below will
yield true. In PHP 5.4 and later, and HHVM, it will correctly yield
false. http://3v4l.org/8p3hm

  $options = 'foo';
  isset( $options['bar'] );

Let's not depend on this behavior here.

Change-Id: I67e83af8afe85b3ddfb5db0009759b8ac5bb7d67

mediawiki.action.edit.preview: Disable if there is no #wpTextbox1

ProofreadPage has custom editforms for Index pages, which are not
supported by the API. These editforms have no #wpTextbox1, so disable
live preview in that case.

Make this check on click avoid T94089.

Bug: T78709
Change-Id: I628d3b226a4294f2878fc4175080a975fd86e507

Localisation updates from https://translatewiki.net.

Change-Id: I8b8619a0715efcec1117bf8d086ff9496be48ac2

Merge "Add release-notes for message escaping"

Simplify TitleTest::testWgWhitelistReadRegexp()

* Use setMwGlobals instead of doing it manually
* Give user no rights to avoid a database query

Change-Id: I3ab20972314e39bb81bc97c89fce7a4c0d331fe6

Remove unused 'section-symbol' message

Change-Id: I3aed335d5dc57f0a6d87828bace0dc55c89c6d68

build: Update grunt-banana-checker to v0.2.1

Change-Id: I7b60b856d8cf21640c382647c6570e32091b5cde

Merge "Fix dependency of ResourceLoaderSpecialCharacterDataModule"

Merge "Change default of $wgResourceLoaderMaxQueryLength to 2000"

Change default of $wgResourceLoaderMaxQueryLength to 2000

The lowest commonly supported length is a bit over 2000.  That is the limit of
Internet Explorer.  For discussion of the IE limit see
http://blogs.msdn.com/b/ieinternals/archive/2014/08/13/url-length-limits-in-internet-explorer.aspx .
Some servers only support 4k.

Having it unlimited does not work when running qunit jenkins test jobs of the
Wikibase extension or when running its qunit tests in vagrant, because it hits
the nginx limit.

This also adds a mw.track call for when the request split happens.

Bug: T90453
Change-Id: Ic416def846f361425c46f7bd1022ed85fa8ac85e

Split out internal parseAndStash from the edit stashing API

Bug: T90040
Change-Id: Ifa49fb9955dafff0c82ae635a5e1b3fb16d5e783

Fix dependency of ResourceLoaderSpecialCharacterDataModule

I think this is probably what's causing the test failure in I3fafc561 on PS7

Follows-up If9c5cdbe

Change-Id: I8fafd734db49fdfaf088c68098c41920a2e6011b

Merge "Changed user login logic to use less expensive touch() method"

Report uncaught errors via mw.track

Adds a global error handler that sends errors to mw.track as
an errorLogging.windowOnerror event.

Bug: T88874
Change-Id: Ic091c9f93c59bda47bda2cfd609c64cd1d014b39

Remove MWLoggerMonologSamplingHandler

The Monolog\Handler\SamlingHandler class available since Monolog v1.12.0
is an upstreamed equivalent of MWLoggerMonologSamplingHandler.

Requires: I8790da95fd658234e35b2d846af35993ebcd80e9
Change-Id: I3841cbab95382a66098d90f5570fa0bf3521578a

Merge "Correct typo in MediaWiki:Apihelp-paraminfo-param-formatmodules"

Merge "Move WikiEditor's special character data and messages into core for use in other extensions"

Add release-notes for message escaping

Some of this messages where used with raw html on wmf wikis, that makes
it possible that this message also will be used with raw html on third
party wikis, therefore give a hint in the release notes.

This includes release notes for:
Id55295a0406659242bbc12a59af7e23fb742237c: editsection
Ic4fcb40de844ac51c88fb51d752a4c286313ecf9: redirectto
I1e90962c3c94c8b72aadfc4c0d041d52e2cde3f1: version-poweredby-others
I7a5dfc8ecd2f84da0dce3e1357af462a3c0da23f: thisisdeleted, viewdeleted
Ifb58bd3823aa633135a701ae3e2ae9c955b422c0: protect-summary-cascade
Id16d8c8dff73fdacad6c9a4ff7f2919945b7e893: editlink, viewsourcelink
Ibcc1b49946bc91e12756eb3866448159493c61f4: lastmodifiedat,
laggedslavemode
Ifd696ecd93c76e56cb21b3f2645367beeeeb5ba0: retrievedfrom

Bug: T85864
Change-Id: I1b144242ccb8780120f1a8ad9e3b6fde0da760f5

Avoid using DB_MASTER in makeLikeArray

Bug: T92357
Change-Id: Ifd134f8572a247d6d977fdb36fb2814bc51e7fbe

Replace Html::... with self::... in the Html class

How cool is that, I can call a patch "SelfHTML". ;-)

Change-Id: I17d36bc45a349c92715b88004aaae046d4f7be1c

Move WikiEditor's special character data and messages into core for use in other extensions

Using a ResourceLoader module to send the data to the client

For I3fafc561

Message changes:
wikieditor-toolbar-characters-page-.* -> special-characters-group-
wikieditor-toolbar-characters-(endash|emdash|minus) -> special-characters-title-

Bug: T91608
Change-Id: If9c5cdbe077e8796e1ebae99f759331251b5c4b1

Merge "Add mobile target for mediawiki.confirmCloseWindow"

Merge "Update MWLoggerLegacyLogger phpdoc"

Merge "Allow MWLoggerMonologHandler to take prefix from channel"

Add mobile target for mediawiki.confirmCloseWindow

Bug: T88949
Change-Id: I1e06668cbd149c23f7b961f7603be07262a021a9

Merge "Made ProfilerStub return null instead of a dummy ScopedCallback"

Merge "Removed view event from $wgAutopromoteOnce"

Merge "Set parameter default to array() in WebResponse::setcookie()"

Correct typo in MediaWiki:Apihelp-paraminfo-param-formatmodules

Change-Id: I961b4a6cd7b8f9ea99fb93a2a624d45fe99501a7

Merge "Fix TestingAccessWrapper::__call"

Merge "ApiParse: Fix parse of new section title"

Set parameter default to array() in WebResponse::setcookie()

This is a direct follow-up for the question raised in Ifab16c2.
Null is not a meaningful value in this case. As far as I can see the
only reason it was added was to make the parameter optional. Optional
array parameters are better marked with "= array()".

Change-Id: I86965d390fdb718de7fb81a9f4c48b2261c16aa9

Merge "Remove unneeded variable assignment in Usercreate.php"

Made User actually use the mQuickTouched process cache

Change-Id: I158eae2dac16b5fdacd095fff7fb031b42804a1e

Don't require database access for ContentHandlerTest

We can just mark all the titles as non-existent in the LinkCache

Change-Id: Ifd6fdfd0f8318dac8af7f7509705cf30d44c8a4e

Merge "Revert "mediawiki.action.edit.preview: Disable if there is no #wpTextbox1""

Removed view event from $wgAutopromoteOnce

* This avoids master queries on view. It could use local jobs, but nothing
  was using this by default anyway.

Bug: T92357
Change-Id: Id6353942215a3c704848d3bcc31c2b76225c78be

Merge "Made addAutopromoteOnceGroups check wfReadOnly"

Merge "Introduced User::touch() method to bump the getTouched() value using memcached"

Made addAutopromoteOnceGroups check wfReadOnly

Change-Id: I2fecf6616dd00cd34647f67298583ea8ec2977c0

Merge "Made showJobs.php include abandoned jobs in --list"

Merge "refreshLinks.php: Limit rows scanned per query in deleteLinksFromNonexistent()"

Revert "mediawiki.action.edit.preview: Disable if there is no #wpTextbox1"

This caused T94089

This reverts commit c890a2fdefaae2551897ce2f1b9f39d82b435dc2.

Change-Id: Ia5e151d12f0ad03ce50802502e661d7499749526

Introduced User::touch() method to bump the getTouched() value using memcached

* This lets some callers avoid the heavyweight invalidateCache() method

Bug: T91279
Bug: T92357
Change-Id: I8c1c7ff9c5574f0eca23e7effde199ab13a19231

Localisation updates from https://translatewiki.net.

Change-Id: I2c29611aa564f96e33980784eb64907598230542

refreshLinks.php: Limit rows scanned per query in deleteLinksFromNonexistent()

This also makes the -e and [start] options apply to --dfn-only, making it
possible to restart the script other than at the beginning if necessary.

Bug: T38195
Change-Id: I828b522039290b5700faa89fb8ad4075e50332e7

Made showJobs.php include abandoned jobs in --list

Change-Id: I7ae8dd2470d5e15fd66c6c06f3feb6d70527daa3

ApiParse: Fix parse of new section title

Mirror how EditPage transforms a new sectiontitle to be gray, have
the arrow etc...

Bug: T84877
Change-Id: If4d8fd11386c87a5b6d51b6afbeae737b3a829a5

Remove unneeded variable assignment in Usercreate.php

Change-Id: I8fe2e6bc3a6c63dacee8d6c8f314d9b4161b7144

Fix inline documentation in WebResponse

Main reason to touch this file are the missing "null" types in the
documentation.

Change-Id: Ifab16c276efb3d8c4aa0a27d8d5106d4e3303794

Merge "Update OOjs UI to v0.9.4"

Changed user login logic to use less expensive touch() method

Change-Id: I5143c381e870d65a1c5bf4cf28ec4abe1bbf3bfe

Merge "Fixed the usage of $flags in loadLastEdit()"

Merge "ApiParse: Forbid section=new when using page, pageid, or oldid"

Merge "LivePreview: Update the correct mw-editfooter-list"

Merge "Fix typos in qunit tests"

Fix typos in qunit tests

seperator -> separator

Change-Id: I67ff04d11f0600dd528fa09894b48d1d2edd248f

Merge "Fix misleading $class = false default in Html::infoBox"

Merge "More specific types in doc tags in the Html class"

LivePreview: Update the correct mw-editfooter-list

mw-editfooter-list is used by both templatesUsed and hiddencats, so
make sure we replace correct one.

Bug: T78834
Change-Id: I00be75ac6d68f96fb366b9561ec1ee0e1998f656

Make wfWarn name the caller in Http::get

The warning is useless as of now, unless you have stracktraces.

Bug: T91764
Change-Id: I8fcae49f3943ab2f6d13519c5f8d370ed147b185

Fix misleading $class = false default in Html::infoBox

I found this because my PHPStorm complains about the type mismatch.
I could have changed the @param tag to "string|bool", but when looking
at the code, the $class variable is casted to a string anyway and
never used as a bool.

Change-Id: I3450fa8a898923bbae26830ed3be0017685020d3

More specific types in doc tags in the Html class

This is a pure inline-documentation patch. It fixes a few actual
mistakes in documentation tags and makes some generic "array" types
more specific, if that's possible.

Change-Id: Id02e1e936624b845316b8ce99f8b8d2a1f829e97

Fix TestingAccessWrapper::__call

We were only passing the first parameter to the wrapped object's methods.

Change-Id: I27a69d1cc1b2d048e44514af8b4ac79d7ee1fb85

Merge "LivePreview: Let's not potentially mangle #wikiPreview"

Made User::loadFromRow protected for sanity

* The loading logic is complex enough without more possible
  entry points to cover
* No outside callers found

Change-Id: I7e9bace35afb0a540993423cf1da98381a516ac5

Deprecate wfDiff()

This function is backwards:
* The stated purpose is to generate a list of differences for callers to parse,
  however we already have DiffEngine that returns a list of differences that you
  don't need to parse manually shooting your feet off in process.
* With improvements in PHP performance in the last 10 years, shelling out instead of using
  a pure-PHP diff has a doubtful performance benefit.
* Even in configurations where it's faster on predominant diffs (which have to be huge),
  shelling out has its own security implications.

This function is not used in core. The only extensons using it are Echo (which just parses
its output so would be better off with DiffEngine) and AbuseFilter, let's not add more users
by tolerating this function.

Also, add error checking while I'm at it.

Change-Id: Ia67debce39de8252312fd887ebfbe6fb89f9edc9

Made ProfilerStub return null instead of a dummy ScopedCallback

* This can happen well over 10k times per request due to the message
  hooks. Avoiding the object construction seems prudent.

Change-Id: I45ff7b8c10851f15a25cbea9a2df3669ec21dbd3

Fixed the usage of $flags in loadLastEdit()

Bug: T93976
Change-Id: I57eb41b463116ea27c0fc8dac8fff07db593499a

Merge "Replace mb_strtolower with strtolower"

Merge "mediawiki.action.edit.preview: Replace only title in firstHeading"

ApiParse: Forbid section=new when using page, pageid, or oldid

Change-Id: I45ce0fe2083587e429143aa3d8d832aa884e1ad7

LivePreview: Let's not potentially mangle #wikiPreview

Sometimes you look at what you have written and realize it can be better...

Bug: T90490
Change-Id: If620e5c672e4632997028ddc9f802a82a83e6924

Merge "LivePreview: Notifiy user about errors"

LivePreview: Notifiy user about errors

Bug: T90490
Change-Id: Ic52311839c175a213065eec442e50b4de0463822

Update OOjs UI to v0.9.4

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

Change-Id: I0ab9ef6d67e16379785e143cb20ae8de7fe78a7e

Localisation updates from https://translatewiki.net.

Change-Id: I79e13cf2d6d3bc8c447b75a49ac0bdca2ae536d9

Merge "Remove never thrown UploadStashNotAvailableException"

LivePreview: Support section=new for preview

Reproduce the new section behavior for the full submit preview.
Diffing will require a bit more additional work in Api.

Bug: T84877
Change-Id: Iec71fdc7ba09f91683214f3d65472ea2f72a06b1

Merge "PrefixSearchTest: Move insertPage() from setUp() to addDBData()"

Remove never thrown UploadStashNotAvailableException

Also move creation of UploadStash to main entry point of the special
page to avoid use of context before it is set (by called getUser).

Change-Id: Ibcb17b6ee1b853d807f91104ba428b307e9a5208

PrefixSearchTest: Move insertPage() from setUp() to addDBData()

Doesn't change the semantics much since run() calls addDBData()
right before setUp(), but these belong addDBData() per the
documentation of setUp().

Change-Id: I304a0aff1fc6b2f0541b3dc3c9e3527772c8f91c

MediaWikiTestCase: Consistently use UTSysop user in setup/teardown

Change-Id: I42d108ca11246725d936e505508b73a9ae7553c6

Fallback to read from master in DifferenceEngine::loadRevisionData

In DifferenceEngine we need to show a diff immediately after
an edit happened if someone did a rollback. To make that work
again, try loading a revision from master if we have more than
one database servers and we failed loading the revision from
a slave.

Change-Id: I08a16faa9e637c614c6d2fc9a10536c7b41685a8
Follows: 3c2bc32ae1775f5e875958b62d7181b4b5cfe24e
Bug: T93866

Replace mb_strtolower with strtolower

Function mb_strtolower is not guaranteed to be present. In this case
we are dealing with language codes, which are ASCII, for which there
is no need to use multibyte version in the first place.

Bug: T55912
Change-Id: I009bab2ff03b22142d35c8be60711266ca3683c1

mediawiki.action.edit.preview: Replace only title in firstHeading

Bug: T85784
Change-Id: I71098997ae4e1172381350443637e51a03260d1a

Revert "Optimize order of styles and scripts"

The patch did not improve performance. I'd like to think that the increased
control over when inline scripts are executed makes the patch worthwhile
regardless, but that is post hoc justification and possibly a bit of personal
ego. Krinkle agrees that we may use some of the ideas in this patch in the
future but he thinks we're better off not heading down this path before we
have a better sense of where we're going, and I trust his judgment.

This reverts commit e86e5f8460b922cadac231142a495f3259c67b43.

Change-Id: I151f74a41dd664b5a0aa5cfd99fcc95e2686a1e6

Update HTML title with displaytitle on live preview

Set the new global JavaScript variable 'wgEditMessage' with the name of the message.

Document this global JavaScript variable in
https://www.mediawiki.org/wiki/Manual:Interface/JavaScript#mw.config
after merge.

Bug: T88117
Change-Id: Ida5c1d470908cefd16925bd5455758d25db02113

Localisation updates from https://translatewiki.net.

Change-Id: Id37e5c4448f3e9b003626009b9c6b821aad4d6cc

Merge "Allow callers to load user objects from slaves if desired"

ResourceLoaderImageModule: Synchronize CSS with .background-image-svg LESS mixin

Bug: T87504
Change-Id: I790bd953fabbd4ff03f15c94fe101e2f65891520

Allow callers to load user objects from slaves if desired

Bug: T92357
Change-Id: Ib9d21b8ee988c84e7b3d1676ad6cbbbf07f795a4

Merge "Use ksort() in getDeduplicationInfo for sanity"

Merge "RELEASE-NOTES: Increase minimum MySQL version to 5.0.3"

RELEASE-NOTES: Increase minimum MySQL version to 5.0.3

Follows-up 5d9b67e09ed1.

Also updated the list of supported database servers in INSTALL
to match the list from RELEASE-NOTES-1.25.

Change-Id: I6881b1163455d834b058f4b4a7839a6aea44cdab