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"

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

Since the removal of the cancel button, the "wpCancel" parameter
cannot be present anymore, thus this check is now unneccessary.

Change-Id: Id72b29484554989ce8a639f2e0d3ecc07622edad

Merge "Remove executable bit from non-executable files"

Remove executable bit from non-executable files

Change-Id: I6def7588b970a15c4e6bd49dc4f7a7138687a99e

Localisation updates from https://translatewiki.net.

Change-Id: I2932704fa919944fc2ddf8c0578e4947e010974d

FormatJson::parse( TRY_FIXING ) - remove trailing commas

Removes trailing commas from json text when parsing
Solves very common cases like [1,2,3,]

Resulting status will be set to OK but not Good to warn caller

Change-Id: Ic0eb0a711da3ae578d6bb58d7474279d6845a4a7

Remove explict autoload from phpunit tests

The phpunit tests are loading all classes, so there is no need for an
explicit load.

Change-Id: Ifb26a5082e3366c44f6a1e65baf64c6ff9bf8449

Merge "Add release notes for skins/common/ removal"

Update OOjs UI to v0.1.0-pre (98e770ce46)

New changes:
888f1c6 Fix for endless loop in SelectWidget
28f6b94 PopupWidget: Make $element not be a 0x0 box

Change-Id: Id9c9e79e4a1fb4d3a42f7fec65c1360b3b192bfd

Merge "Remove Preferences::trySetUserEmail (deprecated 1.20)"

Merge "Remove LogEventsList::getDisplayTitle (deprecated 1.20)"

Merge "Remove XmlDumpWriter::schemaVersion (deprecated 1.20)"

Merge "Remove ApiQueryUsers::getAutoGroups (deprecated 1.20)"

Remove Preferences::trySetUserEmail (deprecated 1.20)

Change-Id: I31b008c93eae54b2aecc18154d0c2e3df3fdb712

Remove LogEventsList::getDisplayTitle (deprecated 1.20)

Change-Id: If8306adec1cd64064280f8ffa1163237ee427cf2

Remove XmlDumpWriter::schemaVersion (deprecated 1.20)

Change-Id: I725d19e46d4728d46d58bfbd3d5a9ad54c4df24a

Remove ApiQueryUsers::getAutoGroups (deprecated 1.20)

Change-Id: Idb43eca139c3e63f8a53ef076b2dfa7c5e3941b5

Merge "buttons: Update focus for quiet buttons"

Merge "buttons: Update focus state"

buttons: Update focus for quiet buttons

Bug: 62924
Change-Id: Ide6174c5da3a04b8e6c4370fbc191783e976f6ee

buttons: Update focus state

Currently the hover and focus state are exactly the same. The hover
state is subtle and making it hard to navigate using a keyboard.

Now the buttons have a border when focused.

Bug: 62924
Change-Id: Ibd75f7caaf5f34d8bcc53381335a048cede26950

Set site logo in mediawiki.skinning.interface module instead of inline styles

Requires skins use the 'mw-wiki-logo' class on the logo element for the styles
to apply and to load the 'mediawiki.skinning.interface' module (or implement
their own skin module that uses the ResourceLoaderSkinModule class).

This is backwards compatible with skins using inline styles as the class only
applies when used.

Bug: 56257
Change-Id: I4e6db89a688843ac24539f8fd1b408c0d04881b3

Localisation updates from https://translatewiki.net.

Change-Id: I0117cee6fead0262213b250e7844183362b03562

i18n: Use native digits on 'difference-missing-revision'

Change-Id: I19cd68562690c4ab8b077681d0f3d18d4286a5cd

Merge "QA: Do not mention Wikimedia Foundation wikis in browser tests"

Merge "API: Add ApiPageSet accessors for just good and missing titles"

COPYING: Update the list of CC licenses icons used

Change-Id: If2e28e63be25c0e2284c1469a466954f497db457

Merge "Added FormatJson::parse( $value, $options = 0 ) returning Status"

Merge "Avoid [GlobalTitleFail] spam"

Added FormatJson::parse( $value, $options = 0 ) returning Status

* Returns Status object that will contain decoded value on success
* Adds i18n messages for all available PHP JSON errors

ATTN Translation team: please copy these messages:

gwtoolset-json-error-depth => json-error-depth
gwtoolset-json-error-state-mismatch => json-error-state-mismatch
gwtoolset-json-error-ctrl-char => json-error-ctrl-char
gwtoolset-json-error-syntax => json-error-syntax
gwtoolset-json-error-utf8 => json-error-utf8

Change-Id: I1c4f37aaabad369b75a1fbd223fad27ebcfe1c3c

Avoid [GlobalTitleFail] spam

MediaWiki::getTitle is checking, if a title is set on the context, but
that needs to call RequestContext::getTitle, which gives a warning, when
no title is set:

RequestContext::getTitle called by MediaWiki::getTitle with no title
set.

Adding a RequestContext::hasTitle function to check the title and avoid
the warning. Was added in I72c5c3fb75c9ceff02390c91b3735f2da90845de.

Change-Id: I4d8f8ef799cd91f0d2affe881019583ac77a31b7

Merge "Test what happens with Konqueror 3.5 against the blacklist"

QA: Do not mention Wikimedia Foundation wikis in browser tests

Change-Id: Iab47a1deff9f7ecda7edf3de18afdfcc00815819
Paired-With: Chris McMahon
Bug: 67616

API: Add ApiPageSet accessors for just good and missing titles

ApiPageSet::getAllTitlesByNamespace is fine when that's what you want.
But in all existing uses in core, it's not what's actually wanted.
* ApiQueryBacklinksprop can generate wrong ORDER BY because it considers
  redirect and invalid titles that it doesn't acutally query.
* ApiQueryCategoryInfo raises a fatal error with category redirects.
* ApiQueryDuplicateFiles and ApiQueryImageInfo will add information for
  redirects, leaving them missing the standard data and missing from
  indexpageids.

What all these need is really something along the lines of
"ApiPageSet::getGoodAndMissingTitlesByNamespace". So let's add that,
plus a few similar methods for good measure.

Bug: 71337
Change-Id: I3a205057edcbc303393718e1c8bfb1f1b1f95323

Add release notes for skins/common/ removal

Bug: 69277
Change-Id: I87566d4adcdcfba7a3fec2a5992ee61f95ec512c

Language: Correct getSpecialPageAliases() documentation

Learned the hard way.

Change-Id: I72044c71f064f78a0d2d50034bdffae1e5985cde

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

Bug 26022

Continues the change Iffcbc837, which was abandoned for old.

Change-Id: I349c801d74dade093a64f17afa8f1d18e0249c3e

Test what happens with Konqueror 3.5 against the blacklist

Konqueror 3.5 is not mentioned in recent compatibility decisions
https://lists.wikimedia.org/pipermail/wikitech-l/2014-August/077958.html
and should not be affected by the recent changes, but one 3.5.4
user reports problems on en.wikipedia.org since few days ago.

Konqueror 3.5 was released in 2006 and is more a contemporary of
Firefox 2 than of Firefox 3.5, though last in 3.5 series was released
on August 2008. http://www.kde.org/announcements/announce-3.5.10.php
Maybe it should be declared as grade C if it's really not currently
functioning.

Some vulnerabilities are known to CVE for 3.5.5, 3.5.7 and prior;
Secunia reports a moderately critical unpatched vulnerability.
https://secunia.com/community/advisories/product/3166

Change-Id: Ia42a9e5028f1aae4837b204eab1e50c4121061c5

Remove executable bit from non-executable files

Follow-up:
- Ibd1c0617c4 (d30edce)
- I1a906f533f (42583e9)

Change-Id: I8081ed2f3fbb08f1d2c0c839a697029234e4d12d

Use CSSJanus from upstream (v1.1.0)

Remove the local copy and include upstream instead.

Remove local unit tests as it's now a proper upstream lib that
is tested by uptream.

Release:

  https://github.com/cssjanus/php-cssjanus/releases/tag/v1.1.0

Changes:

  Features
  * Support "/*!" syntax for @noflip.
  * Flip border-style.

  Bug fixes
  * Improve flipping of background-position.
  * Restrict four_notation_quantity to margin, padding and border-width.
  * Flip two values in border-radius rule (in addition to four values).

Change-Id: Ieb6e179d6163f99a9c98fd99c2277d25135871fe

Merge "Simplified getMaxLag() to use getLagTimes()"

Merge "Special:Categories: Use mediawiki.ui classes and fix layout"

Simplified getMaxLag() to use getLagTimes()

* This method now benefits from more cache sharing and de-duplicated
  lag time querying to reduce connection stampedes.

Change-Id: I2f3b9a22e4adabea703fbae1f96e65fb65125e2b

Special:Categories: Use mediawiki.ui classes and fix layout

Bug: 71146
Change-Id: I96c684f6f7b352afd368624d814761f359e7b551

Merge "buttons: Set min-width of button groups and icon buttons"

Merge "Implement FauxRequest::getRequestURL() - needed for MF tests"

buttons: Set min-width of button groups and icon buttons

This was initially changed to 80px in
I8e6beaf12d7cf1f1af20f4ff144eb0a7c2667a47
and then to 70px because 80px was too large for mobile in
I00b44508e7783957a8c24c4890cc1ba4dfe265da

Buttons inside button groups and icon buttons have a requirement
of being narrower than that but not narrower than their height.

When pseudo element css is not supported icons need a min-width

Bug: 71248
Change-Id: Icf45b9c8800067d3e30f23bbda4a6de21b1289d1

Merge "Remove duplicate "languages" from Grunt config"

Merge "Add parser callback to get a page's current revision"

Remove duplicate "languages" from Grunt config

Change-Id: I9cbb1211cf5e43cf119bd953a21ea6f0eebd6fa0

Implement FauxRequest::getRequestURL() - needed for MF tests

Change-Id: Ib780582cdc68cb770c5fd12d805f4561bc0e2a13

Revert "API: Fix list=allusers with multiple values for augroup"

Causing a large load spike on enwiki api db slaves

This reverts commit aa5800519f301cbafa835ef3ee95c7b3f24050b4.

Change-Id: Ied559c4e701bb644b354af4e158b84537dde94e9

Merge "Revert "Set site logo url in ResourceLoaderSiteModule instead of inline styles""

Revert "Set site logo url in ResourceLoaderSiteModule instead of inline styles"

This caused bug 71334. The logo should be split out
into its own ResourceLoader module rather than
being in the site module.

This reverts commit efe0169c591d4987872143a3760228a694af33c4.

Bug: 71334
Change-Id: I0234e78f93c2eb6574bae4ad48941498709ae528

Localisation updates from https://translatewiki.net.

Change-Id: I4eb4f41d8495ffc44d0d60563a4fb5652abd5cdc

Merge "Created DatabaseBase::trxTimestamp method and added usage in ActiveUsers query"

Merge "Move editoptions styling to mediawiki.skinning/interface.css"

Move editoptions styling to mediawiki.skinning/interface.css

Adding the editoptions box styling to mediawiki.skinning.interface instead
of the skin-agnostic mediawiki.action.* means it will only be added to skins
where it's likely to be appropriate, as this is also where the similar styles
for toc, categories, etc are defined.

This way skins in which the editoptions box would be inappropriate need never
worry about it anyway because they won't be using mediawiki.skinning.interface
for the same reason.

Bug: 58254
Change-Id: Ibd1c0617c426e34e7213ca0ce89a126340b012a7

ResourceLoaderFileModule: Support fallback in 'languageScripts'

Bug: 58139
Change-Id: Idd7657aa48a9eb2b075a85f77e066b425f330890

Merge "Make mediawiki.action.view.dblClickEdit recheck preference"

Make mediawiki.action.view.dblClickEdit recheck preference

There's a preference which makes the browser go to action=edit when double
clicking on the content. In VE we want to disable this while the editor is
open. With this check in place, we can turn it off by falsifying the value
of the preference (and then restoring the original value when VE closes).

Bug: 53058
Change-Id: I0f6225b25ad279755548361f82b855727758da2b

Merge "Fix string callbacks for special page registration"

Fix string callbacks for special page registration

Currently registering a special page with a factory by passing the
callback string does not work because the is_string check comes
before the is_callback check. If we change the order of this if-
clauses we can also support callbacks like 'Factory::createPage'.

Change-Id: I751f97026ed8c580732c74d66b2804dd180ed16f