Remove unused minor and major in WikiReference

Given that the class is not constructed anywhere outside
of core the constructor can be altered without having to
worry about b/c.

Change-Id: Ie49c43a1724f05cb95a296dc0fad0f1c587d80ef

Merge "API: mention how to get page property names"

UploadBase: Remove UPLOAD_VERIFICATION_ERROR

It was replaced by HOOK_ABORTED five years ago and isn't being used
anywhere now.

Change-Id: I20feb33c108ae56f25a0cd01da1a326b290106c2

API: mention how to get page property names

Mention action=query&list=pagepropnames to get a list of properties in
use.
Also rephrase apihelp-main-param-uselang to use active voice.

To test, visit
  api.php?action=help
  api.php?action=help&modules=query%2Bpagepropnames
  api.php?action=help&modules=query%2Bpageswithprop

Change-Id: Ic13f6c00396aed1a1c016bfcf6a39d621a4ebc06

Merge "Added $wgDataCenterId/$wgDataCenterRoles"

Reword "passwordreset-emailsent"

Remove potential confusion.

Bug: T99533
Change-Id: Ifb28f4e1fb6fb692d6f052d5a3c1a67f881948bc

Added $wgDataCenterId/$wgDataCenterRoles

* This is used to set sticky DC cookies to avoid
  session replication lag (which also makes sure
  ChronologyProtector works)

Bug: T91816
Change-Id: I7bc2f8185a3c05cb3ca5ccc42d300eccffae48e1

Merge "Fixed remaning IDEA errors in DatabaseBase"

Made LBFactorySimple set "master" flags like LBFactoryMulti

* wfGetDB( DB_MASTER )->getLBInfo() now shows the master flag
* A corresponding slave flag was also added
* Added a few badly needed LBFactory/LoadBalancer tests

Change-Id: I9254b12cff63af7d754a3a14c5db44276f58d280

Merge "Do not split parser cache if limitation is reached."

Fixed remaning IDEA errors in DatabaseBase

Change-Id: Ife8d7ecd4f224f1e2ef6f729765c303ec69a3f5f

Refactor hashing utility functions from MWCryptRand and make public

MWCryptRand already has some useful utility functions wrapping PHP's
hash() and hash_hmac(). Let's make them public so we can use them from
other code.

But since "MWCryptRand" isn't really a good place for hashing functions,
let's move them to "MWCryptHash" instead.

Change-Id: I7542c719ac72beba7b0f6aa170bdb4c69fa6beab

Merge "build: Don't run Travis CI for wmf/* branches"

Merge "mediawiki.page.watch.ajax: Add dependency on mediawiki.page.startup"

build: Don't run Travis CI for wmf/* branches

These test runs don't seem very valuable and take up a lot of
slots that could be used for other wikimedia/* repos.

Change-Id: I47ff3a31883d02ec6b535e411d92ada92a84aa2f

Merge "Retry getting CatChange RC from master"

Retry getting CatChange RC from master

This adds optional flags to Revision::getRecentChange
And uses them in CategoryMembershipChange

Bug: T109700
Change-Id: I197ebccf1f62cdcb03ce4daa2527b973e495236c

resourceloader: Deprecate two pointless minification config vars

When minifying JavaScript, never put each statement on a separate line, and
always set a target maximum line length of 1000. These behaviors were
previously configurable via $wgResourceLoaderMinifierStatementsOnOwnLine and
$wgResourceLoaderMinifierMaxLineLength, respectively.

Change-Id: I0b0eb632875b5e16f728fd0aa62f7f5ecd79ef62

Made User::loadFromId() skip cache with READ_LATEST

* That flag is used for anti-dependencies and thus
  should never rely on cache

Bug: T95839
Change-Id: I4ffc8325e55588ef649b96e7b90bc95282f765a9

Merge "API: Remove warning about continuation change"

Merge "build: Enable jscs jsDoc rule 'checkParamNames' and make pass"

Merge "mediawiki.messagePoster: Update comments after mw.Api file renames"

mediawiki.messagePoster: Update comments after mw.Api file renames

* Use class name instead of file name
* Expand abbreviation

Follow-up 0bfdd927

Change-Id: I33ad2b10ffb4b4e5e3a38a5c6f644eb183134a36

Merge "Fix LESS file dependency tracking in ResourceLoader"

Merge "mw.Upload.BookletLayout: Go back to upload page on upload error"

Fix LESS file dependency tracking in ResourceLoader

As Timo noted in a review of I1ff6115, using '+=' to combine file dependencies
is incorrect here. I fixed it in one place but not the other, causing the file
dependencies to vary based on whether a .lesscache file was used.

Change-Id: Ie3f48048febc32c5c2d45acd87e10f946f6dfc94

mw.Upload.BookletLayout: Go back to upload page on upload error

Bug: T114130
Change-Id: Ibe50ce845a42bb5e66f842c68a35541747c9a570

mw.Upload.BookletLayout: Make all errors unrecoverable

Any error apart from 503's can't be recovered just by hitting 'Try
Again'. The user will _have_ to make some changes before the process
can continue.

Bug: T114130
Change-Id: I8e2d619be5b7e47bc903155c353bc1da36a62bb6

Merge "mw.Upload.BookletLayout: Move error checking for uploadToStash to uploadFile"

mw.Upload.BookletLayout: Move error checking for uploadToStash to uploadFile

We were checking for uploadToStash errors in saveFile, instead of in
uploadFile itself. Apart from being just wrong it was causing two
issues:

1. We would show an upload error until after you've tried to save the
   file.

2. If there was an error in saving, and you try again, the first thing
   it would do is check mw.Upload.State to hopefully check for
   uploadToStash errors. Instead of running the finishStashUpload again
   it would fail because of the earlier error, thus making all errors
   unrecoverable.

Bug: T114130
Change-Id: I56f4ef6e6536840a63402688eb2a1875a6876fcc

API: Remove warning about continuation change

Change-Id: I2a69e8ae7dc7df915ceaaeed740b74d55f9207ce

Localisation updates from https://translatewiki.net.

Change-Id: I5fc58474ab4bc404a9a74d734e2fe4e61bfaa2ad

mediawiki.page.watch.ajax: Add dependency on mediawiki.page.startup

Follows-up 77cdf19 which accidentally removed it.
It is used to host the mw.page.watch method.

Bug: T114288
Change-Id: I1aad9c3c46c3765b6b241cf3fd7b2f1956b8cfa0

Make lines short to pass phpcs in 2 files under tests/phpunit/includes/

Bug: T102614
Change-Id: I21a7cbc40a3015c3d6cbea1b9d7df58a358beb6a

Merge "mw.widgets.CategorySelector: Link to category page and display existence status"

Make lines short to pass phpcs in BagOStuffTest.php

Bug: T102614
Change-Id: I125cf7ec294818ae4c7741acf24a40980448daa9

Merge "mediawiki.action.edit.preview: Remove spurious 'return'"

Merge "Pass real boolean or array params to mw.Api#get and #post when convenient"

Merge "Rename RESTBase1/Parsoid3 `bodyOnly` parameter to `body_only`"

Make lines short to pass phpcs in ResourcesTest.php

Bug: T102614
Change-Id: I3c5df376dbfc6cbb114b59202ee9d68d4d850746

Merge "Chinese Conversion Table Update 2015-8"

Merge "Rename message 'move-new-title' back to 'newtitle'"

Rename RESTBase1/Parsoid3 `bodyOnly` parameter to `body_only`

The old `bodyOnly` name was deprecated on 2015-09-09.

Bug: T114185
Change-Id: If1fe919672dec4ca281c7f72bfd769a4bcaa5047

mediawiki.action.edit.preview: Remove spurious 'return'

Retuning another Promise from a .done() callback does nothing, it's
only meaningful for .then(). Looking at other code, the .done() rather
than .then() was intended, the 'return' wasn't.

Change-Id: I1022ba45b8f422a626246fa38c4fe81428acb3bc

Merge "Cleaned up WANObjectCache toy example code blocks"

Pass real boolean or array params to mw.Api#get and #post when convenient

We have this wonderful feature, let's use is more often!

Change-Id: I3cc48d11780c0f83df2f13acc9885bb0b4124a92

Merge "Converted Interwiki::load to using getWithSetCallback()"

Merge "Converted checkRedirect() to using getWithSetCallback()"

Rename message 'move-new-title' back to 'newtitle'

Per post-merge feedback on 795922d59c4ab57115d68547c57ffd3c7daa91c5,
the changes to message contents are not substantial enough to
invalidate all existing translations.

Change-Id: Ifbbfd27c8ca6a4abe71e91e6fb7d61155a7be955

Merge "Introduce CategoryMembershipChange"

Merge "Added DeleteLinksJob to support purging backlinks via job runners"

Converted checkRedirect() to using getWithSetCallback()

* Also merged in the code for LocalRepo::getArticleID()

Change-Id: I80d46d2358a48a39e251be83cdb336f663dbfaff

Converted Interwiki::load to using getWithSetCallback()

Change-Id: I4299715ce4c996fed8add773aa03441324eb0dfc

Cleaned up WANObjectCache toy example code blocks

The list order logic was backwards

Change-Id: Id26e13b69d9e1e6ef132792bdcc693836757cbd8

Merge "Improve brevity of Special:MovePage form and its description"

Merge "Added testStaleSet() WAN cache test"

Make lines short to pass phpcs in 2 files under includes/

Bug: T102614
Change-Id: Iae4d4fbdba88ebcd9bcb018a2a0015d0c0ac607c

Merge "Improved getWithSetCallback() docs to have more use cases"

Merge "Add option to expose original sha1 in thumb url"

Added testStaleSet() WAN cache test

Change-Id: Ib8565c48af09af0df1809e877fde9d4d6c7e7038

Merge "resourceloader: Vary module_deps on language (in addition to skin)"

Merge "Fix Memcached key decode"

Merge "resourceloader: Store relative instead of absolute paths in module_deps"

resourceloader: Vary module_deps on language (in addition to skin)

Re-use the md_skin database field for now as to not need a
schema change, beause this table is going away soon (T113916).

Bug: T113868
Change-Id: I7c7546ec58fd9be0447604989b908dd2084b0fe3

Added DeleteLinksJob to support purging backlinks via job runners

* This jobs should only be constructed via relevant Content object,
  e.g. the result of enqueueUpdate() being called on a DataUpdate
  returned by Content::getSecondaryUpdates().
* Also modified LinksDeletionUpdate to support a $pageId parameter.
* LinksDeletionUpdate can now be enqueued to a DeleteLinksJob.

Change-Id: I650dcf0bd172ede0d61357ec158a4704ae1f2033

Improved getWithSetCallback() docs to have more use cases

Change-Id: I428617c159009f22fa78129c16a8964ac2e830b9

resourceloader: Store relative instead of absolute paths in module_deps

Make paths stored in the module_deps table relative to $IP. This ensures that when
the MediaWiki install path changes and/or if the location of the extension or skins
directory changes, that ResourceLoader's internal model is still accurate.

Previously when these paths change, ResourceLoader would exhibit various bugs.

1. Unable to detect changes in the module (if the directory no longer exists).
2. Point #1 is usually preceeded by one last cache invalidation as the content hash
   of the file path changes (from a valid hash to an empty string).
3. Unnecessary cache invalidation (if both old and new directories exist). This
   happens when a file is both an entry point (in the 'scripts' or 'styles' array)
   and also a file dependency. At first they are de-duplicated by array_unique.
   But after the disk path changes, the next check will result in the old path
   being fetched from module_deps, and the new path from the live configuration.
   This causes two changes that result in needless cache invalidation:
   - The hash list contains one more item (T111481).
   - The hash list contains both the old and new version of a file.
     (or even alternate versions, e.g. when a change is backported to the old
     wmf branch; it also affects wikis on the new branch due to the stale
     file path still in the database).

It seems unusual to move a MediaWiki install, and usually we recommend third
parties to run update.php if site administrators do move their wiki. However
Wikimedia's deployment system implicitly moves the MediaWiki install continously
from e.g. "/srv/mediawiki/php-1.26wmf5" to "/srv/mediawiki/php-1.26wmf6".

This caused virtually all ResourceLoader caching layers to get invalidated every
week when another wmf-branch is deployed, thus breaking these file paths, which
changes the version hash, which then invalidates the cache.

Bug: T111481
Change-Id: I173a9820b3067c4a6598a4c8d77e239797d2659c

Remove b/c dependency of mediawiki.widgets upon the DateInputWidget

The only uses in gerrit were MediaWiki's ForeignStructuredUpload,
updated in 6318b4af, and the UploadWizard extension, updated in
f4632c01.

Change-Id: Ic404d9fb07008cdc2393e20cf92e7ec173e06473

Merge "ForeignStructuredUpload: Depend on mediawiki.widgets.DateInputWidget"

Merge "mediawiki.api.upload: Fix jsduck indentation error"

ForeignStructuredUpload: Depend on mediawiki.widgets.DateInputWidget

Change-Id: I1ba1b5eb415edb7de8d41429199658e9df21cb5c

Merge "FauxResponse: Honor cookie options, and other cleanup"

Merge "Bump oyejorge/less.php to 1.7.0.9"

Introduce CategoryMembershipChange

This is split from:
I03516bb34144d95e5f25c46ae98ab70ce699b31b

Change-Id: I27539d25ef3e81cf991657dffc0a62b9719d21d8

mediawiki.api.upload: Fix jsduck indentation error

Follows-up 4738b66e754.

The incorrect indentation caused the second line of this description
to be rendered as a <pre> block.

Change-Id: I99b9718234073ed4e7ae02396ff8eb54f87fe1aa

FauxResponse: Honor cookie options, and other cleanup

We have this nice class for unit testing cookie-setting, but the
cookie-setting method ignores all the parameters! Fix that. Also provide
accessors to check the entire set of cookie data, and the set cookies as
a whole.

While this does change the semantics of FauxRequest::getcookie() in that
the name now needs to be prefixed, no extension in Gerrit uses this
method so we should be fine.

Also clean up the case of the setCookie and getCookie methods while
we're at it. Since PHP method names are case-insensitive, this doesn't
even break compatibility with extensions.

Change-Id: Ib44a074bf9796bc0b470d557e39465792f399d30

Merge "Split DateInputWidget & CalendarWidget into a separate ResourceLoader module"

Bump oyejorge/less.php to 1.7.0.9

Fixes leading whitespace in oyejorge/less.php/lib/Less/Version.php

Change-Id: Ifbc353b583fc182c0dc297ccacc8102ed9279c59

Split DateInputWidget & CalendarWidget into a separate ResourceLoader module

...as the first step to splitting up the 'mediawiki.widgets' module.

For backwards-compatibility, it is a dependency of 'mediawiki.widgets'
until extensions that use the widget can be updated.

Also create the `mediaWiki.widgets` object in mediawiki.js instead of
having an entire file just for that one line.

Bug: T108733
Change-Id: Ife4cfba278dad063a3609929ff87eda7c877d589

Merge "Refactored global state out of EnqueueableDataUpdate"

Merge "Removed old fake slave hacks used for debug log testing"

Refactored global state out of EnqueueableDataUpdate

* Follow-up 25a44aa3e4fa
* The static DataUpdate::enqueueUpdates() method will
  deal with JobQueueGroup construction instead

Change-Id: Id7741a770cee81fdc3f61804b7574ccf737bc338

Add option to expose original sha1 in thumb url

Bug: T112546
Change-Id: I545586ff8d0020d00d9faa1dff2d0d9721134452

Update OOjs UI to v0.12.10

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

Change-Id: I048794c2c3e66e11762598ed97db31afc00cba0c

Merge "More namespace translations for Extremeñu (ext)"

Merge "Actually declare the base LoadMonior::clearCaches"

Merge "Removed $options from RecentChange::newFromConds"

Localisation updates from https://translatewiki.net.

Change-Id: I91a42f708cd83f7dd52a91641088148add7f9a83

Avoid paranoid IDEA error on $conn->getSlavePos() call

Change-Id: I07ca17ad3a434496a3cc14112ad361457c096b4b

Merge "Work around IE9's broken handling of .styleSheet.cssText"

Work around IE9's broken handling of .styleSheet.cssText

IE9 drops @media queries from .styleSheet.cssText when that property
is read. Therefore we can't trust it to contain what we put there
previously.

Bug: T108727
Change-Id: Ic4f553966e48e3fc608a47ef193df923227ceb96

mw.widgets.CategorySelector: Link to category page and display existence status

This contains a silly amount of code to avoid firing separate requests
for each CategoryCapsuleItemWidget that is created, which really
should be a separate class shared with TitleInputWidget but isn't now.
The real widget code here is quite simple.

Change-Id: I66e603fd45b8d3fd61618f9115cd031f6fa01e9d

User::incEditCountImmediate(): Add explicit IS NOT NULL check

Within the UPDATE query, if user_editcount IS NULL, user_editcount+1 will
also be NULL, and the change becomes a no-op. So in MySQL, the number of
affected rows would be zero, which is the number of rows that actually
changed.

However, other DBMSs (e.g. PostgreSQL, SQLite) do count no-op changes,
meaning the code would not initialize user_editcount if it were NULL.
Explicitly checking for NULL should ensure consistent behavior across
database types.

Also, if the CLIENT_FOUND_ROWS flag is set when connecting to MySQL, the
server returns "Rows matched:" instead of "Changed:" as the affected row
count. This change would be necessary if MediaWiki, like Drupal, is
changed to use that flag <https://www.drupal.org/node/805858>.

Change-Id: Idac160bae56adc5c5d17f8558c55d87000019741

Merge "Remove $wgUseLinkNamespaceDBFields"

More namespace translations for Extremeñu (ext)

Change-Id: I1bdadbbeb7a416e1047d5961a3579a367aae1be8

Remove $wgUseLinkNamespaceDBFields

Marked as may removed with 1.25, 1.27 should be okay than.

Change-Id: I4ccd7dfe79fa555946cab64dd1eb179a3fc8291d

Removed $options from RecentChange::newFromConds

* The FORCE INDEX statements are not needed

Change-Id: Ia6f386140e88ad01fc35a10b00dbcb0d1eb934ea

Make it more obvious that users can remove their email address

* Add help message to new email address field indicating that the field
  should be left blank if the user want to remove their email address.
  and the consequences of no email address it.
* Update changeemail related messages.

Bug: T98799
Change-Id: Ie92e393207ee30811fd30e25e72b9e523d95fd62

Merge "Add mw.ForeignStructuredUpload.BookletLayout"

Merge "mediawiki.jqueryMsg: Allow more than one thing inside a wikilink"