Merge "Don't rely on $wgTitle in WebRequest"

Merge "Set content language for ContentHandlerTest::testGetAutosummary"

Merge "Make submit of Special:NewFiles progressive"

Merge "Fix styling of deletion page when $wgUseMediaWikiUIEverywhere enabled"

Make submit of Special:NewFiles progressive

Bug: 71445
Change-Id: Iba2a8e7f37c2c1af3a018740d5a9b48edc84e238

Merge "Change mw-ui anchors to require mw-ui-anchor base class, and introduce mw-ui-text base class"

Change mw-ui anchors to require mw-ui-anchor base class, and introduce mw-ui-text base class

Introduced mw-ui-text base class for plain-text, non-interactable elements which require context colors (eg. icons)
Also added mw-ui-anchor for interactable elements (such as anchors)

Note: This is not supported by IE6 at current time.

Bug: 69212
Bug: 70801
Change-Id: I4d017d0a22cb4f3ca52b6228e45c0463c110ae64

Merge "API: Fix ApiQueryBacklinks logic and use *_from_namespace"

Loosened the CLI restriction in RequestContext::importScopedSession()

* Jobs can be run via HTTP runners rather than just CLI ones.
  The old check breaks jobs run on HHVM fcgi runners for example.
* Always check the validity of the IP for sanity.

Change-Id: I1d03c2ab6628b2d124bc7b9f6936788dea22e3e4

Merge "Fix SpecialPageFactory list handling"

Merge "Remove $wgGoToEdit functionality"

Merge "Avoid stupid mariadb filesort in findMissingFiles.php"

Merge "Workaround for thumbnail pre rendering with PHP < 5.4.7"

Avoid stupid mariadb filesort in findMissingFiles.php

Change-Id: I6267cfaf375fcc0eea8291fd7a2a0f7daa05a941

Workaround for thumbnail pre rendering with PHP < 5.4.7

Since this code is dealing with scheme-less URLs, parse_url()
can cause issues in older versions of PHP.

Change-Id: Ic1276feed9ed51772a05c4a73c787a315b848066
Mingle: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/301

SECURITY: OutputPage: Remove separation of css and js module allowance

* No longer segment module origin allowance by an "only=" content
type. Both can be sensitive security-wise and there's no valid
use case for allowing CSS anywhere you want to disallow JS. Both
can significantly impact the user interface and cause unintended
actions to be taken on the user's behalf, or desired actions to
be made practically impossible.

* While at it, also remove the ability to set the module allowance
directly. The reduceAllowedModuleOrigin method is all we need.
I couldn't find usage or mention of setAllowedModules() in
mediawiki-core nor in any other Wikimedia-hosted repository.

Bug: 70672
Change-Id: I308e794daca27a9380c67be350f8ab51f9c2de34

API: Fix ApiQueryBacklinks logic and use *_from_namespace

The original intent of this patch was to have ApiQueryBacklinks use the
*_from_namespace fields added in Icca99b6a. It does that, but in the
process several other bugs were found in the module:

* Continuation could skip pages when blredirect was used.
* The result object would be populated incorrectly if
  $wgAPIMaxResultSize was hit and blredirect was used.
* Continuation could (probably) skip or (maybe) repeat pages when
  blredirect was used and $wgAPIMaxResultSize was hit.

In the process of analyzing and fixing these problems, the code was
heavily refactored.

Change-Id: I32381c0f082d2f8e063af99ee353ae003c163c23

Localisation updates from https://translatewiki.net.

Change-Id: Id353e6c005c2cd58d7b1aa18dbd5638d8221cf1b

Set content language for ContentHandlerTest::testGetAutosummary

This test depends on content language when comparing to a english
message text.

Change-Id: Ic50c0b6a821fb1d51d70119fbf712a18b73ce4c8

Special:BookSources: Correct validation of ISBNs containing X

PHP's "equal" (==) operator considers the integer 0 to be equal to
the string 'X', and when 'X' is converted to a number, it becomes 0.
Neither is desired here.

* Fail when an X is encountered while calculating the check digit.
  (X can only occur as the check digit of an ISBN-10.)
* Fixed the check digit comparisons by adding an explicit string cast.
* Used the "identical" operator to make it more obvious that no type
  juggling should take place during the comparisons.
* Added some test cases.
* Removed an outdated TODO.

Bug: 67021
Change-Id: I85f53c41f403a60340e9441757fe66b9764e623c

Preserve log comment text field value across import attempts

This was first attempted way back in r45340, but was reverted in r45489.

Bug: 16793
Change-Id: Ieb9fb7e7e918b2e7e2b8289b3fdf82a49ea75a17

Fix SpecialPageFactory list handling

* Since Ic917c7d8/I7420b9ec, SpecialPageFactory doesn't properly cache
  SpecialPageFactory::$list.
* SpecialPageFactory::resetList() has never really worked right, it
  loses all the core special pages.
* SpecialPageFactory::getAliasListObject() could be called recursively
  from a SpecialPage_initList hook. There's no particular reason to fail
  it, just allow the original call to override the result of the
  recursive one and hope it works.

Change-Id: I7adb346eab00d5849d087ddff75230a35be3ee8f

Merge "Use ems instead of pixels for checkbox"

Merge "Clarify documentation for icons"

Merge "Fix input regression affecting HTMLForms that use VForm"

Use ems instead of pixels for checkbox

Mixing pixels and ems results in a negative height, which is invalid.
Use ems instead of pixels for @focusBottomBorderSize.

Follow up: Ia6adc4bda000508f14acf4214e778118efa35be0

Bug: 71506
Change-Id: Ia619916a172a501365e0104e9927c7fae9de8a3f

Merge "User rights API: Abstract out some stuff about core's form into separate methods"

User rights API: Abstract out some stuff about core's form into separate methods

To we can make a CentralAuth API module for global user rights.

Bug: 71495
Change-Id: I139c43e5eaa1fb63b75baddbafb7caa3c964d6b5

Merge "mediawiki.jqueryMsg: Support arbitrary expressions in plural forms"

Merge "Use skinStyles for mediawiki.ui.icon"

Fix input regression affecting HTMLForms that use VForm

Although the classes (e.g. mw-ui-input) are still there,
the style module was not being loaded.

This affects forms that deliberately use VForm even without
wgUseMediaWikiUIEverywhere (since they pre-date it).

This applies at least to PasswordReset, which was added in
d32891d99f185e5cf2b0e25bf947de221d4c3295 (last October).

Bug: 71448
Change-Id: Ieca36c0036d4c93244c533237eddd678564dd3c0

Use skinStyles for mediawiki.ui.icon

This allows skin customization.  This was missed since it was under
code review when the others went in.

Change-Id: I2c640fda4d2c6e40be0add52f6fc948a8e07c010

Clarify documentation for icons

Change-Id: I3452366505bde7a75a6a922138daa6a19a235d20

mediawiki.jqueryMsg: Support arbitrary expressions in plural forms

In I15c167f245 mw.jqueryMsg.htmlEmitter#plural was modified to
convert the plural params to flat html. This caused extlink params
as plural forms expanded as text node and escaped.

This patch improves the plural support by:
* Allowing valid parser expressions as plural form parameters. They wont
  be converted to html, parsed nodes will be preserved in plural
  calculation
* For explicit plural forms like n=expression, the expression can be any
  valid parser expression. They also will not be coverted to html,
  nodes will be preserved
* Move the splitting by equal sign logic for explicit plural forms to
  jqueryMsg, avoiding mediawiki.language#convertPlural to do
  parsing logc. Explicit plural forms, if any, are passed as a parameter
  to mediawiki.language#convertPlural

Includes test cases for all combinations of the above cases.

mediawiki.jqueryMsg.peg can be improved to support digit=expression as an
additional form for templateParam to better handle this. But it looks
it can potentially disturb other parser functions. Not attempted.

Bug: 70617
Change-Id: I3c3cbb6d3676903941c40b658032b4c543556e06

Update OOjs UI to v0.1.0-pre (30b0407428)

New changes:
3386c39 Make this.$.$iframe actually point to an iframe again
bb7bbf5 ClippableElement: Use 'overflow-[xy]: scroll' rather than 'auto'
71176eb TextInputWidget: Don't mix inner and outer heights in calculations
f1dd60e TextInputWidget: Work around Blink height miscalculation
954a369 WindowManager: Emit 'resize' event when a window is resized
3232d6f Try to stop user from tabbing outside of open dialog box
6c4baa3 Tool: Use FlaggedElement mixin to let tools be flagged

Change-Id: Ia0a0b45e5f210314592733114a05583973b011d6

Merge "Skip MailAddressTest::testNewFromUser on windows"

Merge "CSSMin: Don't generate double rules for IE < 8 when embedding SVG files"

Merge "CSSMin: Do not base64-encode non-binary files when embedding"

Merge "Update OOjs UI to v0.1.0-pre (afa3241e8e)"

Update OOjs UI to v0.1.0-pre (afa3241e8e)

New changes:
8daef61 TextInputWidget: Add setValidation function to change pattern used to validate text
953c46c [BREAKING CHANGE] Create a MediaWiki theme
e2a3bae PopupWidget: Position body/anchor using 'margin-left' instead of 'left'
4f50c34 Correct the left positioning of a popup wrapper
afa3241 Move references to pending.gif to the themes that provide it

Update changes:
* Update links to OOjs UI distribution files

Change-Id: Ic59650b56de44807348db4df46c538756ed9329e

CSSMin: Don't generate double rules for IE < 8 when embedding SVG files

Bug: 71003
Change-Id: Ic232e8d62b164940003afdfe7cce9f964d7e9cbc

CSSMin: Do not base64-encode non-binary files when embedding

Do not base64-encode non-binary files (containing only whitespace and
printable ASCII characters, which matches sane SVG files).

For SVG files the percent-encoded URIs are actually slightly longer
than the base64-encoded ones (~10%), but compress a lot better
resulting on 15-20% less data to transfer after gzip compression.

(The effect is best seen on the 'oojs-ui' module, which consists
mostly of SVG icons – especially after commenting out everything other
than 'oojs-ui.svg.css'.)

I tried this for binary files too, just in case; but as expected, they
suffer from a noticeable size increase even with compression (~15%).

Bug: 67341
Change-Id: Iddaf863b6be98570a2bb8e606f13946a96345f65

Fix GitInfoTest::testValidJsonData on windows

GitInfo.php is replacing DIRECTORY_SEPARATOR by '-'.
On windows this is \, which was not set in the tests.

1) GitInfoTest::testValidJsonData
Failed asserting that false is true.

Change-Id: I1871c342b61a335b59a98755d8b824cc58c1543e

Localisation updates from https://translatewiki.net.

Change-Id: I8d868812d775114923187d2db27201080017d452

Merge "Introduce Message::plaintextParam"

Introduce Message::plaintextParam

I have run into numerous issues trying to utilize unsafe user
provided content as an argument to a Message instance.  Specific
cases are enumerated in MessageTest.php

Typically the solution to using user provided text is to use
Message::rawParam, but this pushes escaping of the parameter to
the caller.  This patch introduces Message::plaintextParams which
handles escaping of the string parameter to match the requested
output format.

The functionality is:
* plain and text: exactly like rawParams()
* escaped, parse and parseAsBlock: escape it but don't do brace expansion

Additionaly, similar to Message::rawParam, plaintext parameters are not
valid parser function arguments.

Change-Id: I320645cd23c98fea4bfc32ab22b7ef8d320957cb

Disable "linkPrefixExtension" on Central Kurdish

Disable prefix on Ckb and format it like Fa, it currently links
"«[[Link]]»" to "<a>«Link</a>»" on Ckb but on Fa will "«<a>Link</a>»"
which per a community member is much better and expected

Change-Id: I1d17fc6ea511f13bde4fef7538117c9c2eebc6db

Merge "Restore ability for getAttributes to set HTMLTextField type"

Restore ability for getAttributes to set HTMLTextField type

Change I47db5eab broke SecurePoll, which was relying on this.

Change-Id: Ib11bf9d34eb12f14b9636498c00ff4040b35f130

Fix version comments in thumbnail prerendering code

Change-Id: I74b52d2ecc382dfc0b55615ce512e8c64d02e9cb
Mingle: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/301

Merge "Added missing JsonFormat::parse() RELEASE NOTES, fixed docs"

Added missing JsonFormat::parse() RELEASE NOTES, fixed docs

Constant values were changed to be above 0xFF - this way
we can easily decide to allow depth-parsing-limit to be OR-able:

  FormatJson::parse( $value, 30 | FormatJson::FORCE_ASSOC )

Follows-up Ic0eb0a7 and I1c4f37a.

Change-Id: I9bfd67a5ca4ea1d399821549c7e63ffdecd56ad1

Implement autocompletion for Performer field on Special:Log

Bug: 48241
Change-Id: I5e6af5aba56de613f10f862aaf064aaa32b5b176

Add test cases for wfAppendQuery

Change-Id: I51e0e1764ad70d922466d62f1ddf3f8e3fb4568f

Remove deprecated "jquery.json" module

Announced in https://www.mail-archive.com/wikitech-l%40lists.wikimedia.org/msg78108.html

Change-Id: Icc9dec0146cd374787124b7e1ed295c77737d11b

mediawiki.api: Remove deprecated function parameters

* 'ok' and 'err' parameters to various functions.
  Tracked with mw.track( 'mw.deprecate' ) as 'api.cbParam'.
  Deprecated since MW 1.20, printing console warnings since MW 1.23.
* 'async' parameter to mediawiki.api.category #getCategories.
  Tracked with mw.track( 'mw.deprecate' ) as 'api.async'.
  Deprecated and printing console warnings since MW 1.23.

Also removed is mw.Api#normalizeAjaxOptions, which is no longer needed
and which has never been used by anything outside mediawiki.api module.

Announced in https://www.mail-archive.com/wikitech-l%40lists.wikimedia.org/msg78108.html

Change-Id: I0a650fdb4affd394ae419e21d54baf790116f6f7

mediawiki.user: Remove deprecated name() and anonymous() methods

Removed mw.user#name and mw.user#anonymous which have been deprecated
in favor of mw.user#getName and mw.user#isAnon since 1.20.

Announced in https://www.mail-archive.com/wikitech-l%40lists.wikimedia.org/msg78108.html

Change-Id: I3be1cfecb79faee5701708136ede133870b9940d

PasswordFactory::newFromPlaintext( null ) needs to work

Various code passes null around to mean "an invalid password". It
shouldn't all have to test for null and specially handle that.

This also fixes a codepath where User::$mNewpassword could get set to an
empty string rather than a password object, which would cause problems
later when anything else tries to use it.

Bug: 71421
Change-Id: Ib5f94b52c07e7dba89328b98fb43c86db95ee09f

Merge "HistoryAction: Use correct classes for buttons"

Localisation updates from https://translatewiki.net.

Change-Id: I46d7b8b70d02685d9b5c20bd14a0da56dffbcf2b

Fix styling of deletion page when $wgUseMediaWikiUIEverywhere enabled

* Stop using tables use divs instead. It's the future!
* Update MediaWiki UI so that inline inputs have a minimum width (as pointed
  out by Bartosz, the summary field would've gotten smaller than it was
  before to the point of being near unusable).
* Change submit button to be mw-ui-destructive (it deletes a page after all).
* Switch to using Html rather than Xml where possible.

Bug: 70134
Change-Id: Ic308a43cb0ec26ea9fdb8c9c8aa0de481a922435

HistoryAction: Use correct classes for buttons

Bug: 71141
Change-Id: I95ac411e7c06d3b7aec966527f196ef347f1c710

Merge "API: Rewrite queries for list=allusers"

Merge "Special:Contributions: Use mediawiki.ui"

Merge "Bidi-isolate and nowrap parser tags on Special:Version"

Merge "Do not read metadata from non-existing djvu image"

Merge "Fixed spacing"

Merge "Register missing messages for mediawiki.special.upload"

Register missing messages for mediawiki.special.upload

Follows-up I43bf828654d342893a19656ddd4151c2558d2f63.
The confirm close needs messages and jQueryMsg for parsing "int:".

Change-Id: If9a69749dcff2196dfe51a4484c6faeb98674ffa

Merge "Introduce ContentAlterParserOutput hook"

Merge "Lowered ActiveUsers query period to reduce timeouts"

Do not read metadata from non-existing djvu image

Tests are printing:
*** [1-11711] Failed to open 'some-nonexistent-file': No such file or
directory.
*** (ByteStream.cpp:693)
*** 'DJVU::GUTF8String DJVU::ByteStream::Stdio::init(const DJVU::GURL&,
const char*)'
djvutxt: [1-11711] Failed to open 'some-nonexistent-file': No such file
or directory.
djvutxt: 'ByteStream.cpp:693'

That just spamming the logs, so adding a valid check before do anything.

Change-Id: Ia77377e632587ede76af4c722a1b8ac34fa50e1c

Fixed spacing

- Added newline at end of file
- Removed double spaces/newlines
- Added space after if/function and parentheses/brackets
- Removed space before comma/cast
- Fixed indent of some lines

Change-Id: I29867ffdffdfb7d2b56997e9393497c7dc12f7d3

Introduce ContentAlterParserOutput hook

This hook is the counterpart of ContentGetParserOutput: it gives
high-level access to the parsing of a content object (as opposed
to lower-level parser hooks like ParserBefore*/ParserAfter* which
are called for all kinds of non-content input like interface
messages and don't have the necessary context to know what kind of
input is being parsed), but it is called when the parsing has
already finished.

The intention is to provide a convenient location for ParserOutput
modifications which depend on some global property of the parsed
content, but need to happen before LinksUpdate. Currently there are
no hooks between parsing (initiated by WikiPage::doEditUpdates()
calling WikiPage::prepareContentForEdit()) and updating the link
tables (initiated by the same method calling
Content::getSecondaryDataUpdates()), so any hook an extension would
try to use would face one of the following problems:
* the parsing output is not available yet
* LinksUpdate has run already, modifying list properties is unsafe
* there is not enough context to tell what's being parsed

A typical use of this hook would be to add tracking categories when
something is missing from the text (e.g. there are no references).
For the concrete use case, see I43ed79b6a54cd31820ecae8139e29c5880f5dd1b

Alternative approaches that have been suggested and do not require
a new hook but are not robust / do not rely on a clear contract:
* use ParserAfterTidy or a similar hook, assume that the input
  being parsed is the main page content iff limit reporting is
  enabled
* use ParserAfterTidy or a similar hook, assume that the input
  being parsed is the main page content iff
  ParserOptions::getInterfaceMessage() is false (this would yield
  some false positives, but in those cases adding a tracking
  category would probably have no effect)

Change-Id: I685b285fcc772382993116f7822a832eeecc0681

Lowered ActiveUsers query period to reduce timeouts

Change-Id: If34266cbac79d319fe33e1f362791fb438577fd7

Special:Contributions: Use mediawiki.ui

 * Use 'mw-ui-input-inline' for tagfilter input
 * Use 'mw-ui-input-inline' for IP/Username
 * Use 'mw-ui-progressive' for Search button
 * Vertically align the select dropdown

Bug: 71137
Change-Id: I20b0ebeb194ce0505abede6fb3bdd69b28350f04

Add "returntoquery" URL parameter support in Special:ChangeEmail

For consistency with other special pages, e.g. Special:ChangePassword.

Change-Id: I97eac516c8a366624413bfcd0f4bd9f1185b6419

Merge "Revert "Replace wfMsgReplaceArgs by RawMessage""

Merge "Article: Handle double redirects in data for mediawiki.action.view.redirect"

Revert "Replace wfMsgReplaceArgs by RawMessage"

This was merged too quickly, has several issues.

This reverts commit 6abefb274ff9cd86f75d5d13f713e1971e1b27ce.

Change-Id: Ibf3e464fa9eb534d599274dd9262a30d5a6a6948

Article: Handle double redirects in data for mediawiki.action.view.redirect

Bug: 71002
Change-Id: Id7dd37a4865635d3b68aac70179890be6ab89210

Replace wfMsgReplaceArgs by RawMessage

The replace of $n params can also be done using the Message class. In
case there are no message keys, the RawMessage class is used.

Deprecated wfMsgReplaceArgs, but added no warning, because it is used in
some extensions and from deprecated functions

Change-Id: I62091b09e4490e59ed7258566e0ddf2f8ee799d2

Merge "Move addTrackingCategory from Parser to ParserOutput"

API: Rewrite queries for list=allusers

The existing query only works with a single value for augroup, or mostly
if it's combined with auprop=groups or auprop=rights (since most users
don't have every possible group).

When used with multiple values for augroup, it will raise an error if it
happens to encounter enough users who have more than one of the
specified groups. And further, it will lead to repeated groups for these
users if combined with auprop=groups or auprop=rights.

An attempt to use EXISTS instead of a JOIN to query user_groups failed
because it made the planner scan the user table where a smaller scan of
user_groups plus a filesort was faster. So instead, let's just fix it
directly by acknowledging that supplying multiple groups for
augroup is going to give us duplicate rows.

At the same time, let's try using a subquery (with GROUP_CONCAT) to
fetch the actual groups the user belongs to, both to avoid processing so
many duplicate rows and to simplify the row-processing code. And let's
kill the forced index, it's probably not needed anymore.

Bug: 70496
Change-Id: Ic87dbc3b2d933775ca71a72932e0658e2f082bb6

Merge "Don't clobber revision size of 0"

Don't clobber revision size of 0

When a revision's size is 0, don't clobber it by changing it to null. This
causes, among other things, moves of empty pages to not show a revision
size. If we get to where the else used to be, either mSize is 0 (and doing
nothing is correct), or mSize is already null, in which case this didn't
do anything anyway. We know this because, above, we initialized it with
$this->mSize = isset( $row['len'] ) ? intval( $row['len'] ) : null;

Change-Id: I3d9b9163368c57b9a7906f6569119081b945d57e

Merge "Watchlist: changed "all" for the max number of days available"

Merge "Set rup/roa-rup autonym to "armãneashti""

Move addTrackingCategory from Parser to ParserOutput

addTrackingCategory is more in line with ParserOutput's functionality
(addLink, addCategory etc), and tracking categories are useful even for
content types which do not use the parser at all. There is no reason to
require the caller to obtain a Parser object just to be able to add
tracking categories.

Change-Id: I89d9ea1db3a4e6486e77eee940bd438f7753b776

Merge "Add window close warning to Special:Upload"

Merge "Implement mediawiki.confirmCloseWindow module"

Add window close warning to Special:Upload

Change-Id: I43bf828654d342893a19656ddd4151c2558d2f63

Localisation updates from https://translatewiki.net.

Change-Id: Icb08b905fced272db07a100be7f15ebed1ce2ccd

mediawiki.action.view.redirect: Remove deployment hack

Follow-up to c49bd9cb.

Change-Id: Ic3904d40e89533483434e074b2021d4ec3db6831

Fix case of SpecialAllPages/SpecialAllMessages in SpecialPageFactory

[autoloader] Class SpecialAllpages was loaded using incorrect case
[autoloader] Class SpecialAllmessages was loaded using incorrect case

Follow-Up: Ic137bc7adbc4a9ed96448ba0fee4807b67b3112d
Follow-Up: I21a164af0b6ec123bf654cd1e4e7085b1192f067
Change-Id: Idae65141de0de41bf4244408218a91c4b87d5b7d

Fix case of SpecialShortPages in SpecialPageFactory

[autoloader] Class ShortpagesPage was loaded using incorrect case

Just for information: This was not visible on Special:Specialpages.
In SpecialPageFactory::$list the class LongPagesPage is before
ShortPagesPage and loads that class due to "extends", when
ShortPagesPage should be loaded, the class is already there and no
warning is generated. The warning is shown, when reording the $list.

Change-Id: Ie0ac708b40ff51372545278bb694865650fa8b1f

Update CREDITS

Add myself as a developer, and sort the whole thing alphabetically.

Change-Id: Ib089c09fa0c52d8adba2058251c626d9161b048b

Merge "Remove dead code after removal of cancel button on Special:ChangeEmail"

Merge "FormatJson::parse( TRY_FIXING ) - remove trailing commas"

Merge "Bidi-isolate extension version on Special:Version"