Fixed spacing

- Removed space after casts
- Removed spaces in array index
- Added spaces around string concat
- Added space after words: switch, foreach
- else if -> elseif
- Removed parentheses around require_once, because it is not a function
- Added newline at end of file
- Removed double spaces
- Added spaces around operations
- Removed repeated newlines

Bug: T102609
Change-Id: Ib860222b24f8ad8e9062cd4dc42ec88dc63fb49e

Make a line shorter for phpcs

phpcs failure reported at
I30a6af657a11bf822736d667a82b21e813f0db4e

Change-Id: Ia5941684300670e0f232b82393c9af28ce4b972e

Merge "Fix ChangeTags functions after split of definedTags from tagUsageStatistics"

Fix ChangeTags functions after split of definedTags from tagUsageStatistics

This provides fixes to the can*Tag functions of ChangeTags following the
removal of listDefinedTags from the tagUsageStatistics function in
I410e9a935bd202faac92f430c0b4dae1a48e2d21.

Change-Id: I087bb7107ae0c83946f8e2f417a6121bb2da18b4

Fix braces code style

Bug: T102805
Change-Id: I7ce4c71dd605f9be79a630602391271bb269b962

Remove multiple empty lines in functions

Functions must not contain multiple empty lines in a row
(Squiz.WhiteSpace.SuperfluousWhitespace sniff).

Bug: T102774
Change-Id: Ib706e4fc34b95c0d7c887e54af0ea61227767d8f

Merge "Clarify that gender-unknown option is gender neutral"

Merge "ResourceLoaderImageModule: Fix incorrect documentation"

ResourceLoaderImageModule: Fix incorrect documentation

The docs claimed that you had to do something like
'images' => array( 'filename.svg' => array( 'name' => 'foo' ) )
but what you actually need to do is
'images' => array( 'foo' => array( 'file' => 'filename.svg' ) )

Change-Id: Idca2d5c58e4d9fcf008ef5cfe7f8ce77b996c203

Update OOjs UI to v0.11.5

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

Change-Id: I98bcf928a922e03f7024edaacc9beac95f2b74cf

Merge "Don't include never-applied defined tags in tagUsageStatistics function"

Don't include never-applied defined tags in tagUsageStatistics function

This removes the call to ChangeTags::listDefinedTags() in the function
tagUsageStatistics of ChangeTags. So only those in the change_tag table,
i.e. with a hitcount greater than zero, are returned.
Instead at SpecialTags, those tags in the list of defined tags, already
retrieved for other purposes, are appended with zero hitcount, when not
already inserted.
This incidentally makes it easier to get a list of tags applied at least
once, as needed for T27909 (where we don't want never applied tags).

Bug: T91535
Change-Id: I410e9a935bd202faac92f430c0b4dae1a48e2d21

Merge "Made User generally use DB_SLAVE by default"

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

Merge "API: Add some BCarray into ApiQuerySiteinfo"

Localisation updates from https://translatewiki.net.

Change-Id: I8c1f493e82e411960f6743e77151fbee23c7342c

Merge "Fix whitespace issues around parentheses"

Move dependency on user.tokens from mediawiki.api.edit to mediawiki.api

The postWithToken() method used to be specific to the edit module.
However that logic was moved and refactored into getToken() and
postWithToken() a while back.

While waiting for user.tokens instead of making api requests for tokens
is worth it. The logic in mediawiki.api.js inspects mw.user.tokens
only at initial run time. So it must be on 'mediawiki.api'.

Anyhow, in practice it doesn't matter since the user.tokens is
embedded in the page. So there's no actual waiting of any kind.

Change-Id: I10fae93c8f1500b4aa752101256fbba94c2ae40d

Fix whitespace issues around parentheses

Fix issues found by MediaWiki.WhiteSpace.SpaceyParenthesis sniff.

Bug: T102617
Change-Id: Iec7f71e64081659fba373ec20d9d2006306a98f4

Made MessageCache use the WAN cache

* This makes sure edits to MediaWiki: pages update
  the cache in all DCs

Bug: T99208
Change-Id: I177608729063b800fb97374f31f316779effce15

API: Add some BCarray into ApiQuerySiteinfo

The main change is that siprop=namespaces will output an array instead
of a numerically-keyed object with formatversion=2.

Various other properties also got the same treatment, but in those cases
the intention was always array and possible non-array output was due to
array_unique or odd indexes in config variables.

Bug: T102645
Change-Id: I486131ac0d6763c3fe40fa45bcee472fe1b46fd6

Merge "Moved patrol code in commitRollback() down to reduce contention"

Made User generally use DB_SLAVE by default

* By default, users will load from the slave unless
  the thread did a recent DB write. This is to handle
  changes within a request.
* ChronologyProtector should avoid staleness in common
  cases, and the CAS check on user_touched is a final
  barrier to block stale user object updates.
* Note that passwords are not cached, so they hit the
  DB when ever needed. Passwords now load from slaves
  when possible, instead of the master.
* This should get the code closer to handling user
  login and logged in users when the master is down.
* Fixed loadFromId() when READ_LOCKING is used.
* Also addressed TODO comment in load().

Bug: T92357
Change-Id: I0a8bdab720c19fe3fc2381799ae2e90ff09bb4cf

Ensure that files end with a newline

Bug: T102619
Change-Id: Iae6e722151581d15c9421d41c4d14b100bb6e437

Merge "Correct quotes in Polish translation"

Correct quotes in Polish translation

The correct closing quotation mark in Polish is ” rather than “.

This changes a bunch of messages, so I think it's easier to fix here
and then import in Translatewiki?

Change-Id: I77153964af15368b7f83512b17d3cc9e9066b675

Merge "HTTPS-ify links to Wikimedia's sites in MW core source"

Merge "database: Don't treat $defaultSchemas as containing all types/drivers"

Merge "resourceloader: Fix broken getRequest/getDirection in derived context"

Merge "resourceloader: Use -1 instead of null in DerivativeResourceLoaderContext"

Merge "Reduce RC table contention for file deletion"

Reduce RC table contention for file deletion

Bug: T98706
Change-Id: I13f12b93abccb348c7093c77c363a54a65b59142

Merge "Fix InstantCommons - HTTP access to Commons does not work anymore"

Merge "Simplify canUpdateTags function in ChangeTags"

Fix InstantCommons - HTTP access to Commons does not work anymore

Bug: T102562
Change-Id: Ie0b4ec179b9ccd13015c9bcbfa5c9917ff26de36

Localisation updates from https://translatewiki.net.

Change-Id: I85d0a4d444cf082e3e3fa51be190ee3248597d32

Merge "Add a SpecialSearchResultsAppend hook"

Add a SpecialSearchResultsAppend hook

This allows additional HTML to be included below search results.  This
will be used to optionally include a feedback link fter search results.

Bug: T101783
Change-Id: I5c4bab12ed0b022c84aa6b50ab72635e9dd0bd0c

build: Remove obsolete jshint exclusion for jscompat

Upstream https://github.com/jshint/jshint/issues/729 was fixed.

Change-Id: Ia8f3e53ec0117b57156208033c0a49835a18702c

Merge "jobqueue: use more sensible metric key names"

database: Don't treat $defaultSchemas as containing all types/drivers

The Database::factory() method treats $dbType as of one of
predefined "canonicalDBTypes", and defaults to using it as driver.
Which is then used for the name of the Database subclass.

This allows extensions and wiki farms to have custom subclasses
to override certain methods, or even provide new drivers.

The $defaultSchemas array added in f7174057a4 was given all canonical
keys (with null values) to allow unconditionally access. This
doesn't scale very well and is error-prone. Reduce it to the
override only and fallback make the fallback to null explicitly.

See T102285 for where this would help prevent a PHP Notice.

Change-Id: I3f1e1f59c300d34de30f6480ff4e54f159d51b16

Merge "Update mediawiki/mediawiki-codesniffer dependency to 0.2.0"

Update mediawiki/mediawiki-codesniffer dependency to 0.2.0

Change-Id: If6887ce9d445fd6b7dc036f68e803f4c5d86dce7

jobqueue: use more sensible metric key names

* Since JobQueue metrics are qualified with 'jobqueue.', don't add a 'job-'
  prefix to each metric.
* Separate the key from the job type with a dot rather than a dash.
* To avoid having a Graphite node that is both a "directory" and a metric, use
  '.all' as a suffix for aggregates.

Change-Id: I2ac604d3c042dbfb0b3a27759800f435ec22041e

Merge "Disable Preferences save button before setting change"

Merge "Avoid muliple cache calls to explicitly defined tags"

composer.json: Set classmap-authoritative: true

For developers with older versions of composer the setting will
just be ignored.

Bug: T85182
Change-Id: I66d4eace301eb4319e4c0137c8f3ee6f35fe4e51

Simplify canUpdateTags function in ChangeTags

This simplifies the function canUpdateTags by checking that none of the
tags are defined by an extension instead of checking that all of them
are either manually defined or not defined at all.
Doing so avoids a cache call and makes it easy to propose only relevant
tags for addition/removal to users at Special:EditTags.
This also makes the respective checks for adding/removing tags only when
necessary.

Change-Id: Iddee4d4efb109b0fccf1ece400d166147e1700fc

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

Localisation updates from https://translatewiki.net.

Change-Id: Iaddc25ac27e8fa35a69b8751ae9890adf31fb101

Avoid muliple cache calls to explicitly defined tags

This avoids muliple cache calls to explicitly defined tags by
calling the showTagEditUI of ChangeTags only once in logs and
histories.

Change-Id: I2e36dbd96d3fcca06de0bf418bc6dc294d8d18d3

Merge "Add Goan Konkani Language"

Merge "specials: Simplify return logic of various SpecialUserlogin methods"

specials: Simplify return logic of various SpecialUserlogin methods

* Handle exceptional case before common case in makeLanguageSelector
  by using early returns. Better reflects the intended effect
  of the exception by making it harder to accidentally run code
  after the 'else' statement.

Change-Id: I710a94adf22bc4e6dc539e12c69e4ba96bf1068c

Update jQuery Client to v2.0.0

Project

* https://github.com/wikimedia/jquery-client

Release

* https://github.com/wikimedia/jquery-client/releases/tag/v2.0.0
* https://github.com/wikimedia/jquery-client/blob/v2.0.0/jquery.client.js

Breaking changes:

* Microsoft's new Edge browser is now identified as "edge" instead of "msie".

Change-Id: Id3df3e637cd9751a293ca655c51d39492d04d7f6

Localisation updates from https://translatewiki.net.

Change-Id: I47268e9b17dfaa6b274ef28577b3a3e717fa486b

Moved patrol code in commitRollback() down to reduce contention

* It still happens if the edit fails as before
* Also, women...they exist...

Change-Id: If8cc4c55d4edcfb083b7018b2645167c8220ec81

HTTPS-ify links to Wikimedia's sites in MW core source

See https://blog.wikimedia.org/2015/06/12/securing-wikimedia-sites-with-https/

Ignoring SVN stuff because svn.wikimedia.org's SSL certificate has expired, and
ignoring wgSharedUploadPath because T25122 wants to get rid of it.

Also a couple of BZ -> phab changes

Change-Id: I02fd23ac6c30a4a4c718e57d4dedbc693dd653c0

Merge "Tablesorter: Replace wgContentLanguage by wgPageContentLanguage"

Tablesorter: Replace wgContentLanguage by wgPageContentLanguage

wgContentLanguage contains the language code of whole project.
wgPageContentLanguage contains the language code of the content of the page.

For example https://translatewiki.net/wiki/Project:About/de?uselang=eo has
* wgContentLanguage en
* wgPageContentLanguage de
* wgUserLanguage eo

The tablesorter works on the content of the page.

Change-Id: I3e51d96c838a3a4f4562c1f27c360d515e4d8ca6

Merge "Use XmlSelect to simplify Xml::monthSelector"

resourceloader: Fix broken getRequest/getDirection in derived context

getDirection() isn't a simple getter value like the others. It actually
is tightly coupled with getLanguage() and lazy-initialised.

When calling setLanguage(), we shouldn't reset direction back to the
parent class but make sure getDirection() will recompute it based
on the local value.

Added regression test (which fails without this patch).

The parent getDirection() looks for $this->request, but the subclass
doesn't assign that member in the constructor. getRequest() forwards
it accordingly, so make sure getRequest() is also used internally.

Change-Id: Ifec703647368c3bb58748288ed754aaaf3730e19

resourceloader: Use -1 instead of null in DerivativeResourceLoaderContext

The ResourceLoaderContext class used null to determine absence of
an overridde in the derivative object.

However three of the members in question allow null as legitimate value.
(Namely 'only', 'user', and 'version').

This makes is impossible for a derivative context to remove one
of those values if the parent context has them set.

Use case: I782df43c needs to create a derivative context of
          load.php?only=scripts&modules=startup without 'only'.

Use -1 instead as internal placeholder value.

Also:

* ResourceLoaderContext::getSkin() was documented as returning 'string|null' when in
  fact it always has a default value. Never returns null.

* DerivativeResourceLoaderContext::setOnly() and setVersion() were missing
  type hint for 'null' (as it was incompatible with their getter). Adding 'false'.

* Swap if/else statements to handle the special case first (inheriting).
  Allowing the rest of the function body to handle the local value.
  In preparation for further development.

Change-Id: I058884525237effe8aef35469ed7693bb7cea591

Merge "Rename testBug25702() to testPrefixNormalizationSearchBug()"

Merge "Add "rel=discussion" attribute to talk page tabs in skin template"

Rename testBug25702() to testPrefixNormalizationSearchBug()

Change-Id: If17a2cd2c1481cc564c61be73af64938824d9a9c

Merge "Omit 'external' changes from ChangesFeed"

Add "rel=discussion" attribute to talk page tabs in skin template

Bug: T65198
Change-Id: Ifd1a7333c7b657d74860d206ea5b82d29bf97851

Merge "ApiQueryRevisions: Remove bogus query optimization"

Merge "Consistency tweaks: xml -> XML, Href -> href"

Merge "Fix rel="copyright" for ApiHelp"

Fix rel="copyright" for ApiHelp

Bug: T94038
Bug: T93995
Change-Id: I06e9bb6797c4e4ee00e5d72eda1e273800b2c7f4

Merge "Allow 'uselang', 'useskin', 'debug' as query parameters in RedirectSpecialPages"

Merge "StashEdit: Use postWithToken"

Merge "Check for link-local addresses in isPublic"

Merge "Add image/pjpeg as an alias for image/jpeg"

Merge "Do not support new installations via .php5 entrypoint any longer"

Localisation updates from https://translatewiki.net.

Change-Id: I5cc62b447fa6116ba6bab21556e4a0abc0d8738c

mediawiki.widgets: Add missing dependency on mediawiki.Title, mediawiki.api

Used by mw.widgets.TitleInputWidget.

Change-Id: Ic47fb914c62c227f37c7eadb7bb6fbb846ebf3ad

Merge "Simplify SpecialBlockList::preprocessResults"

Disable Preferences save button before setting change

Disable the Special:Preferences 'Save' button if no
settings have been changed (uses keydown and mousedown
events as an alternative to change event, so is not perfect)
This prevents unnecessary saving when you can't remember
if you saved the settings or not.

Bug: T89457
Change-Id: I7c2e11302099280c561e435425b23afb9fb760b5

Job queue: prefix metric keys with 'jobqueue'; segment with '.'

Currently names are segmented with a dash (for legacy reasons), which
makes them clutter the top-level MediaWiki.* metric namespace.

Change-Id: I785819723808368fbf9c3f5ea167f50f40733a3a

Merge "Add login error browser test"

Merge "Instrument edit failures"

Instrument edit failures

Users on enwiki's VPT are reporting a spike in the frequency of rejected edits
due to loss of session data. It is difficult to corroborate these reports when
we don't have any instrumentation for them in the code, and thus no visibility
into how frequently they are occurring in the wild.

Bug: T102199
Change-Id: I0c99f292002209cf2bf4376f97b8c70a6b1ec021

Add release notes for f6e5079

Change-Id: I4ccfebe6a59148c9825521f98b0f3654e87f3e11

Merge "Live preview: Don't show empty summary preview"

Add Goan Konkani Language

Bug: T96468
Change-Id: I12857c36baa2b931f2db86f6be9a50e5e3057967

Merge "Use mediawiki/at-ease library for suppressing warnings"

Use mediawiki/at-ease library for suppressing warnings

wfSuppressWarnings() and wfRestoreWarnings() were split out into a
separate library. All usages in core were replaced with the new
functions, and the wf* global functions are marked as deprecated.

Additionally, some uses of @ were replaced due to composer's autoloader
being loaded even earlier.

Ie1234f8c12693408de9b94bf6f84480a90bd4f8e adds the library to
mediawiki/vendor.

Bug: T100923
Change-Id: I5c35079a0a656180852be0ae6b1262d40f6534c4

Localisation updates from https://translatewiki.net.

Change-Id: Ifee61df93c4c0ceaf8c410e11429ec41d953dd35

Add LCStore implementation that uses static arrays in PHP files

Implementation written by Fred Emmott of Facebook. Quoting Fred:

As well as array access being faster, the main advantage is actually
that this significantly reduces the use of unserialize(), which does a
lot of memcpys when making the strings.

Benchmarks compared to LCStoreCDB:
* HHVM (no repo-auth): ~7% improvement
* HHVM (with repo-auth): ~12% improvement
* PHP7: ~1% improvement

My (Legoktm) brief testing noted that the generated PHP files were
noticiably larger than the CDB ones:
* 1.5M en.l10n.php
* 932K l10n_cache-en.cdb

Bug: T99740
Change-Id: Ib2c5856d40cd928cab4a79cb935b3ce08c598300

Add image/pjpeg as an alias for image/jpeg

image/pjpeg is a non-standard mime type for progressive jpegs
that is sometimes used. Not considering this an alias of normal
jpeg images was causing content type validation failures in
gwtoolset.

Bug: T102136
Change-Id: I505f16c4e8e4ee2e2a4675f4f07c4b8c12f9d187

Merge "Made MessageCache::replace() ignore messages with content language suffix"

AjaxResponse: Fix broken logic for extracting HTTP status codes

Follows-up 6584cef207 (r16266; 2006; MediaWiki 1.8.0).
This regex was just trimming whitespace from the start of the string. It
matched whitespace, then a match group for a first sequence of digits. The
replacement is just the digits match. This is essentially `ltrim()`. It didn't
account for the content after the match.

> echo preg_replace( '/^ *(\d+)/', '\1', '200 OK' );
> "200 OK"

This was causing:
> Warning: Unknown HTTP status code 200 OK in libs/HttpStatus.php:100
> Warning: Unknown HTTP status code 304 Not Modified OK in libs/HttpStatus.php:100

It defaults to HTTP 200. So presumbaly the impact was low. Though it may've caused
304 responses to have been broken (content body missing).

Bug: T102028
Change-Id: Iafff9982bbbee893c13f891901dde88f998db7a6

Merge "Allow users with 'viewsuppressed' to view when limitTitle()d"

Allow users with 'viewsuppressed' to view when limitTitle()d

This was probably missed when 'suppressrevision' was split into
'suppressrevision' and 'viewsuppressed'.

It was properly separated in limitPerformer() but not in limitTitle().

Bug: T101982
Change-Id: I9c62599124cbb74d2004b416f6e490c231dbd7b2

Merge "Fix the autonym of Northern Luri and change Central to Northern"

Merge "Update namespaces and special pages for Northern Luri (lrc) from translatewiki"

Fix the autonym of Northern Luri and change Central to Northern

Change-Id: Ie60524ec5735fce03f0b8add635cf7c17798b0ab