Merge "Handle no pageids response field in MessagePoster factory"

Merge "APIQueryImageInfo: Set META_BC_BOOLS for 'value' in 'metadata' block"

Merge "Do not encode '~' as %7E. Fixes redirect loop in chrome."

Do not encode '~' as %7E. Fixes redirect loop in chrome.

As of 155d555b83eca640, we now redirect variations on hex escapes
into their canonical form. This was causing '~' to be redirected
to %7E. However google chrome seems to canonicalize %7E back
into ~, causing a redirect loop.

RFC 3986 says ~ is unreserved, so not hex encoding it should be
fine. To quote: "For consistency, percent-encoded octets in the
ranges of...tilde (%7E) should not be created by URI producers"

Bug: T105265
Change-Id: I01556eee496e2fb540de8ff09c082c1fedddb5f7

Merge "Allow detected external links to linebreak at any place"

Allow detected external links to linebreak at any place

These so called 'bare urls', which are not wrapped in [] can be very
long at times and because they are not normal text usually don't wrap
to a new line when needed, causing annoying overflows,
especially on smaller screens and often on talk pages where they are a
bit more common. This is CSS3.

Change-Id: Ied3f5fb52d79318c40d541f2c47ae36ea2dfe504

Merge "Fix inaccurate documentation re $wgEnotifMinorEdits"

Cleanup in doTableStuff

Change-Id: I75c0a943b24f96a30c6ee1efc3f0b11388f892b7

Merge "Sync up with Parsoid parserTests."

Fix inaccurate documentation re $wgEnotifMinorEdits

In actuallyNotifyOnPageChange, the $wgEnotifUserTalk is inside
a check involving $wgEnotifMinorEdits, so it won't fire if
it's a minor edit and $wgEnotifMinorEdits is false.

Change-Id: I6576cb1735db5d9288257e6877b8755450d0dacd

APIQueryImageInfo: Set META_BC_BOOLS for 'value' in 'metadata' block

The 'value' property can be a boolean, which was previously output as a
raw boolean.

Bug: T103781
Change-Id: I788f13f33885b7c8f694470e44478907f28cfbfd

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 34bd76a1747816ed96952ce81ce2fafa751dbb21

Change-Id: Iabf427db488251e26d55a0eb68b282eccd9eb8df

Unbreak missing flags in enhanced RC

$this->recentChangesFlags() was being called on the data twice, so the
second time it was processing the string, causing the flags go missing.

Follows-up 94f153db6a0.

Bug: T105237
Change-Id: I04465d0317eecc1c12e47bc9a74f11986eba99a4

Localisation updates from https://translatewiki.net.

Change-Id: I5277704a2e9af694cf5b5cd70e3fb291238deade

Merge "Allow using createAndPromote.php with custom groups"

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

Allow using createAndPromote.php with custom groups

This lets any group(s) be added using a comma-separated argument,
in addition to the --sysop, --bureaucrat, and --bot that already
existed.

Bug: T105079
Change-Id: I1274b065e3dad917e545f9278b996da014d87ae9

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