Parser: Avoid producing <span></span> in the TOC

If someone renames a section but wants old targeted links to still work,
<span id="old-anchor"></span> is the usual solution. And sometimes
people put it inside the section header markup, like

 == <span id="old-anchor"></span>New name ==

since putting it before makes it be considered part of the previous section
while putting it after causes the browser to scroll the section header
off the screen.

But this has the unfortunate side effect that the TOC text for that
section will be "<span></span>New name". We should strip that useless
empty span.

Bug: T96153
Change-Id: I47a33ceb79d48f6d0c38fa3b3814a378feb5e31e

Update environments.yml file according to the documentation

Bug: T105174
Change-Id: Ib049d12f24444029f5aeb8ef3149c731c2b9c810

mediawiki.api: Include 'mobile' target in mediawiki.api.parse module

Bug: T104940
Change-Id: Ic654f0e38b27b41df13c0423460c1c12c9aec9c3

mediawiki.api: Refactor to use server.respondImmediately

* Simplifies code a lot and removes the need to explicitly flush
  and handle the request queue with respond().
  This was especially annoying when a request spawned others as
  they wouldn't be in the queue yet.

* Make tests more explicit and resilient by specifying what they
  respond to instead of assigning to requests[i] directly.
  This also makes the failure better if one request isn't made,
  instead of throwing for accessing properties on undefined objects.

* Avoid relying on test order for badToken().
  Follows-up 7b05096bcae0. Tests should be atomic and not rely on
  order. This is already important as QUnit re-orders failed tests.
  And in the future they may run concurrently.

  Resolve using a unique name (like the other tests).

  Also, the previous test wasn't asserting that badToken() works,
  it was merely asserting that getToken() works and that it wasn't
  yet cached. The new test will fetch and purge its own token.

Change-Id: I26d22ace6c5df19d7144779be1a625aede79749f

Merge "HTMLTextAreaField: Honor 'rows' setting in OOUI format"

Merge "OutputPage: Only set <link hreflang=x-default> on wiki/pages with variants"

Merge "Improved fail-over in ReplicatedBagOStuff for redis"

Merge "Move `$.extend( OO.ui, mw.widgets )` hack to 'mediawiki.widgets' module"

Merge "Add Content-Length header to thumb.php error responses"

Merge "qunit: Restore sinon.sandbox after teardown instead of before"

Merge "Update Sinon.JS to 1.15.4"

Merge "Update QUnit to v1.18.0"

Update OOjs UI to v0.11.8

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

Change-Id: I701c504acf0d527916f1925bbf49177cc9c0d883

HTMLTextAreaField: Honor 'rows' setting in OOUI format

And throw an exception if 'cols' is set, since it won't work.

Requires eed0f5294b0080 in OOUI.

Bug: T104682
Change-Id: I8e09402a01cecac8a90497d31b3b1ca15ff2d949

Merge "Explain API action=parse&disablepp option more clearly"

Explain API action=parse&disablepp option more clearly

Brought up by Purodha on translatewiki.net Support page.

Also change a few ungrammatical "Disable"s to "Omit"s.

Change-Id: I9a14fa16cc166471d66c53489e8186ec09297399

Localisation updates from https://translatewiki.net.

Change-Id: I3d2c6254a862d635419018c2c4fe3f824ae43a93

Merge "resourceloader: Set a TTL for minification cache entries"

backupTextPassTest: Disable flaky test 'testCheckpointPlain'

Follows-up 9107fb539111de.

Ref T70653

Change-Id: I493c0a97bd9b5213a3f1f6ed0b2a6c591fd67d3b

API: Move stats update for API module execution to when the module is actually executed

The place it was before could easily be reached without actually
executing (or even constructing) the module.

Change-Id: I6f807adc9cbf71c5d7b83c7eec43965dce1d2a16

Add alias for Special:ChangeContentModel

Follow up to fa31bf1675af725.

Change-Id: I2589b8a48c82484f8ef561d9562b28f613fa15b1

resourceloader: Set a TTL for minification cache entries

Follows-up 458e7cabbbafd. HHVM's APC cache doesn't have any
limitation or eviction logic. Thus entries without a TTL will
linger indefinitely.

At time of writing over 99% of APC entries were from "resourceloader:filter"
because there are far more unpopular resources than popular ones.

A fixed TTL is suboptimal as it will also cause popular resources
(e.g. startup module, or jquery) to be re-minified every day.

Beware this isn't one single minification. It's one minification,
for every JS/CSS resource, on every app server, for many different
lang/skin/config variations.

Bug: T104769
Change-Id: Ib62c320da0eeede68aad3f2294006ec7dcf1df55

qunit: Restore sinon.sandbox after teardown instead of before

This was causing pending (mocked) XHR requests to sometimes be
cleaned up prematurely.

As a general rule, setup always calls its parent first. And teardown
always calls its parent last. The inner callback runs within the
scope of the outer one and should still have acces to the sinon mocks.

Change-Id: Ic4d9243c04af73e529b689e650d35ddabaa7f4b9

Update Sinon.JS to 1.15.4

* Code
  http://sinonjs.org/releases/sinon-1.15.4.js
  http://sinonjs.org/releases/sinon-ie-1.15.4.js

* Change log
  http://sinonjs.org/Changelog.txt
  https://github.com/cjohansen/Sinon.JS/commits/v1.15.4

Mainly:
* Support XHR.responseType 'json'.
* Remove deprecation warnings from sinon.mock().

Change-Id: I4e0d8f044eb2841ab361b1efc76fb36797f15e42

Update QUnit to v1.18.0

* Code
  https://code.jquery.com/qunit/qunit-1.18.0.css
  https://code.jquery.com/qunit/qunit-1.18.0.js

* Change log
  https://github.com/jquery/qunit/blob/1.18.0/History.md

Change-Id: I9da13eedb2f9b5ee936aaa658c3da6ce222cfa0a

Merge "API: Use ApiBase::PARAM_HELP_MSG_PER_VALUE for 'prop' in query+watchlist"

Merge "mediawiki.searchSuggest: Allow multiple suggestion fields"

Move `$.extend( OO.ui, mw.widgets )` hack to 'mediawiki.widgets' module

OO.ui.infuse assumes all widgets are in the OO.ui. namespace.
We previously made that happen on document-ready in
'mediawiki.page.ready' module, but that's too late: document-ready
handlers defined by other modules' can run earlier.
Let's just do it in 'mediawiki.widgets' after all widgets are defined.

Change-Id: Ib2184bf87d72db64d1565d742ef9f1be37f24b6f

OutputPage: Only set <link hreflang=x-default> on wiki/pages with variants

Follows-up 91d8a51825.

Having the default variant advertised seems useful, however for
wikis and/or pages that have no variants or translated versions,
outputting this header doesn't seem useful.

https://support.google.com/webmasters/answer/189077 doesn't
justify sending it unconditionally.

To try out, setting $wgLanguageCode = 'zh'; will result in all
pages having hreflang=x-default and all variants. And setting to
'en' or 'de' will result in none of those link tags.

Change-Id: I21cd072534ae1df960209e657b19c96889ece27c

Merge "mediawiki.ui: Add @activeColor argument to the .button-colors() mixin"

mediawiki.ui: Add @activeColor argument to the .button-colors() mixin

Have kept the colors same for now except for .mw-ui-button:active and
.mw-ui-button.mw-ui-checked . Can be changed later once the design
team has decided.

Change-Id: I7c418b970c5e5fa95f740ecf3d90622bf7f02364

Update composer-merge-plugin to v1.2.1

Fixes bug in optimized autoloader generation on initial plugin install.

Change-Id: If637d2b5583b32895fe8839478e7e553927f1d68
Requires: I22d1fd4044320613c6ba61ac5e84263e5af4ff27
Bug: T104037

Merge "TitleInputWidget: Add 'relative' option"

Add Content-Length header to thumb.php error responses

Without the Content-Length header, the response is sent with Transfer-Encoding:
Chunked, which is somehow mangled by mod_fastcgi. Varnish then claims the
response is malformed and declines to process it, sending the client a 503
instead.

Change-Id: I977387bf3e039926df58e3e5a14d0d0bd55201c4

Merge "Allow hooks to abort lines in EnhancedRC"

Merge "API: Avoid caching when prop=info&intestactions=... is used"

API: Use ApiBase::PARAM_HELP_MSG_PER_VALUE for 'prop' in query+watchlist

Change-Id: I5e956f4b4dccb2b109d514332e4347252ce846c4

Merge "Add hook EnhancedChangesListModifyBlockLineData"

Merge "Add tests for Special:BlankPage"

Merge "Localisation updates from https://translatewiki.net."

Merge "Add SpecialPageTestBase to simplify testing of special pages"

Add tests for Special:BlankPage

This uses SpecialPageTestBase and demonstrates how that class
should be used.

Change-Id: Ibbf3fdcb172d7c1c71491d27b534faa65fd9be63

Add SpecialPageTestBase to simplify testing of special pages

This is copied from Wikibase. A follow up change adds a test
for Special:BlankPage to demonstrate how it works.

Change-Id: I3c34fd17ceb0049b160ec4f821474de457533983

Localisation updates from https://translatewiki.net.

Change-Id: Ifcf5396c7f9a8a6abb43dd9233dc6c55b6ca00c1

Merge "Add {{bidi:}} syntax for directionality-safe arguments"

API: Avoid caching when prop=info&intestactions=... is used

Whether the actions pass or not depends on the current user, including
the results of several hooks and whether the current user is blocked.

Also, we can clean up the test for public inprops slightly.

Change-Id: I7e2610dd609c52711355b640778f795520a4ff8c

Merge "mediawiki.jqueryMsg: Declare dependency on user.options"

Merge "editWarning: Declare dependency on user.options"

Merge "Count API module instantiations and Hook runs"

TitleInputWidget: Add 'relative' option

Currently the JavaScript widget will return a value that is relative to
the namespace, so for "Category:Foo", it will return "Foo". This is
problematic for server-side forms that want a full title returned, so
make this configurable.

Change-Id: I605df2ca41831cae1c8f0a3331600d4487e7798f

editWarning: Declare dependency on user.options

Bug: T102564
Change-Id: Ia4bc7860e6783c1ff9d94a9097c4e19cd34d57f8

mediawiki.jqueryMsg: Declare dependency on user.options

Change-Id: I8219a78fdecf6cdc33a677811fd4160cd56a46b7

Localisation updates from https://translatewiki.net.

Change-Id: Ic760a22b691004d0b1f8bae63aacf33ad1c4fc99

i18nize [[Special:RandomInCategory]] submit button

This commit fixes a regression caused by an earlier refactor of the
Special:RandomInCategory page, which introduced message reuse. This
commit splits the message again.

Bug: T101717
Change-Id: Iabfe172b2e1d8464262b244337789bca388460f8

Merge "Use localization for server default timezone on Special:Preferences"

Merge "Removing obsolete `-moz-inline-block` property"

Localisation updates from https://translatewiki.net.

Change-Id: Iaf200f28c06c3dddc582581c5c69bbb260e8351e

mediawiki.searchSuggest: Allow multiple suggestion fields

Store form data in context instead of a single private variable.

Bug: T104763
Change-Id: I9c8827eab1e074cf2b5a573942cf983dbda07149

Use localization for server default timezone on Special:Preferences

The timezone list on Special:Preferences contains all time zones with a
localized region and also an option to use the system default value.
The message for this option gets the current server timezone as
parameter, but that parameter is not localized.
Refactor the timezone list processing into a new function and call it at
the beginning to do a lookup for the localized name and use it, if it
exists in the array.

Also changed from procedural style to object oriented style for the
DateTime processing.

Bug: T33516
Change-Id: I00e81324d0d16fbe6c9811480210ab6513461823

Localisation updates from https://translatewiki.net.

Change-Id: If1a80bd1be0256e5ba3d60b15454f194515f1d47

Merge "Add missing dependency to mediawiki.widgets"

Removing obsolete `-moz-inline-block` property

Bug: T103605
Change-Id: Id689cfd629d94a82999020121d4275c5daac4e58

Allow hooks to abort lines in EnhancedRC

Bug: T104564
Change-Id: I4a2f97d83f38071984d571773a6b09b6b6643d6d

Add hook EnhancedChangesListModifyBlockLineData

Introduce a new hook to allow (single) block-level entries.
Very similar to EnhancedChangesListModifyLineData.

Bug: T104399
Change-Id: I6b4715277d44e5f09d7a230b33e956676aeab1c2

Merge "RawAction: Clean up max-age/s-maxage computation"

Merge "API: Use message-per-value for apihelp-query+revisions+base-param-prop"

API: Remove some unused i18n messages

apihelp-query+deletedrevisions-param-limit and
apihelp-query+deletedrevisions-param-prop aren't actually used. Instead
these are documented with apihelp-query+revisions+base-param-limit and
apihelp-query+revisions+base-param-prop, respectively.

Change-Id: I4a337e21695c34ab5fe44411978fcce78dce4ac7

API: Use message-per-value for apihelp-query+revisions+base-param-prop

Use message-per-value for message
apihelp-query+revisions+base-param-prop to allow smaller strings
for translation and to add a parameter for the contentmodel for
prop=parsetree, as already done for action=parse&prop=parsetree.
Each prop in a message also will show up a new parameter on the help
page without the adjust of the translation (but than in english instead
of fully skip it).

Change-Id: I8aaa061cb59ec788f8f8e09e8088ba1064e3aa1e

FauxRequest: Add @since for new setHeaders and setCookie methods

Follows-up 53b0df09ab69.

Change-Id: Ia854aef848062624eccc6a5b25b7d1f1f8fbc444

RawAction: Clean up max-age/s-maxage computation

No behavioural change, but makes the code easier to understand.
It was somewhat all scattered.

* Remove outdated comment about 24 hours.
  - ForcedRawSMaxage defaults to 5 minutes.
  - SquidMaxage defaults to 5 hours (wmf-config: 31 days).

Change-Id: I7f3b67780ba9e8c024dcbd68772495b91abb2d01

Count API module instantiations and Hook runs

We previously had counts for hooks and modules as a side-effect of having them
profiled. We removed the profiling for performance reasons, which left us
without counts also. But the performance of counters is not a concern, and
their signal value not insubstantial. So introduce them here.

Fix getModulePath() to not crash while we're at it.

Change-Id: Ic04daf475b936b942833362c7a979dde671b3ef4

Merge "Log http error in ForeignAPIRepo::httpGet"

Merge "Add cookies and improve headers in FauxRequest"

Localisation updates from https://translatewiki.net.

Change-Id: If712259c32498a8c9ce1242e828bebe9c04f8805

Add cookies and improve headers in FauxRequest

For testing, it can be useful for a FauxRequest to actually contain
cookies. And for convenience, it's useful to allow setting headers by
passing an array rather than by making multiple calls to ->setHeader().

Change-Id: I3b84246e954aae594490927d2b0a57336af2e3a5

Merge "Revision: Interpret a NULL rev_content_model as the default model"

Merge "Correct mw-ui-icon size"

Merge "Made SqlBagOStuff set() wrap setMulti()"

Merge "Declare and document $wgStatsdServer and $wgStatsdMetricPrefix"

Merge "Add class to items in changeslist legend"

Merge "Make an apihelp message less repetitive"

Merge "SpecialDoubleRedirects: Remove redirect=no for action=edit"

Merge "Fix reference to nonexistent ApiBase::getModulePathString in inline docs"

Declare and document $wgStatsdServer and $wgStatsdMetricPrefix

Follows-up 87dfc20.

Change-Id: Ie1b799a8f2a73b974ba8c930cfabc00e377d6eee

Revision: Interpret a NULL rev_content_model as the default model

If a revision using the default content model is saved, the
rev_content_model field will be set to NULL in order to save space.
However, Revision::getContentModel() was using Title::getContentModel()
when it found a NULL, which will return the current content model rather
than the default. So change that to use the default content model, as
returned by ContentHandler::getDefaultModelFor() rather than the current
one.

Change-Id: I7011812b6b131170b3c593917ad263bcba83898d

Made WANObjectCache::getCheckKey() automatically init the key

* This still allows If-Modified-Since logic but does not
  need to broadcast initialization values just because a
  key fell out of cache. The value can differ between DCs
  anyway via skew, this just lets them drift more. Actual
  purge events are still broadcasted, which is what matters.
* The User class has now been simplified given this change.
* Added more general comments to getCheckKeyTime().

Change-Id: Ic1f4bbb1947e0d1dd47499c9e9dc86991c30580c

Merge "Improved job pickup time stats for delayed jobs"

Change "? :" to "?:"

Anomie pointed out that I am the only person who writes it as "? :", and
that all other instances in core use no space.

Change-Id: I01b68eed5f866fa7582fb4257f795c4c2d1244d1

Fix reference to nonexistent ApiBase::getModulePathString in inline docs

Change-Id: Iad6420ea88c4cf94c234be7f01a2ed2854e02919

Ensure DerivativeContext has a $stats property

Should have been a part of Ie10db1c15 but wasn't.

Change-Id: I4552390e1476766ae2d0946df3836a8a14182b34

Merge "Account for unset visiting watchers"

Correct mw-ui-icon size

Icons should be 24px by 24px
1em = 16px,
1.5em = 24px

Change-Id: I98564ad2b02668efa616a6f3c536c78440edcac9

Don't try looking for phpunit if it is already loaded

If phpunit is installed via composer, it will already be present in the
autoloader, so including other paths won't do anything.

Also add some output to make it easier to figure out where phpunit is
being loaded from.

Change-Id: Id6d23626b158779f6ff56e6d0f20d519f2f4ad18

Improved job pickup time stats for delayed jobs

* The delay time should not count

Bug: T102743
Change-Id: I9e8b1f33b65681fd9f4f667233bce280bf6f227d

Add explicit GENDER to logentry-contentmodel-change

Change-Id: Iad544e1b61661acd3e54777859e72a59349fabb5

Merge "Localisation updates from https://translatewiki.net."

Localisation updates from https://translatewiki.net.

Change-Id: I730d53aaf52bb6000ff33d06f325e5ec30280f87

Merge "Update OOjs UI to v0.11.7"

Merge "mw.Title: Handle empty this.title"

mw.Title: Handle empty this.title

Otherwise we break code like this:
mw.Title.newFromText( '.foo' ).getPrefixedText()
because this.title[0] won't be set and you can't uppercase it

Bug: T104377
Change-Id: I1dae1e63e474757da725f76bced442ebd5b76c25