Merge "Use ParserOutput stateless transforms"

Merge "ParserOutput: Add stateless transforms to getText()"

Merge "Title: Use a more proper way of detecting whether interwikis are local"

Merge "objectcache: Make WANObjectCache interim caching not interfere with ChronologyProtector"

objectcache: Make WANObjectCache interim caching not interfere with ChronologyProtector

Also removed useless line from testLockTSE(). That would have needed
to be using $this->internalCache and those locks are freed immediately.

Bug: T180035
Change-Id: Ida1a923f779aaf8410da76643457d2200da6cb20

Merge "Remove @defgroup from DefaultSettings.php"

Remove @defgroup from DefaultSettings.php

This renders in Doxygen as a blank page in the "Modules" sidebar,
doesn't seem to serve any purpose.

No mentions of "Globalsettings" or "Global settings" elsewhere
in the repository.

Change-Id: I2c03b4190c899f2acff400d63bbd3df09f8d49d1

Remove some dumb headers with my name in it

Change-Id: Ib34b623b47fb92f1383bc5c55c50a16dd6d49abd

Merge "resourceloader: Remove deprecated minifier config vars"

Title: Use a more proper way of detecting whether interwikis are local

Bug: T181575
Change-Id: Ifcfe609ce01b174120b81b8c0bf82c4ed6077641

Merge "Make ChronologyProtector actually use cpPosTime cookies"

resourceloader: Remove deprecated minifier config vars

Unused, their values have been hard-coded since 1.27.
No uses anywhere in Wikimedia Git, besides their definition
in this file.

Change-Id: I6689274d0ad028e2c468560816ab753b345048e9

Merge "Fix phpdoc for Category::newFromName"

Merge "WatchedItemStoreUnitTest return strings in db mocks"

Merge "Add missing @throw tags to IDatabase/Database"

Merge "Use parent function in mysql DB class in getWeightScale()"

Make ChronologyProtector actually use cpPosTime cookies

Setup.php was only injected the value from $_GET. LBFactory used
to check both before 0e5cd18b74d83c50, with the cookie check left
in LBFactoryMW. After d175b391ae2a7, LBFactoryMW class switched to
the default LBFactory method for making a ChronologyProtector
instance, so the cookie check was lost there too.

This means that an HTTP GET request due to a redirect from a POST
request will still make sure the DB position store is up-to-date.
This only matters is that store uses replication (e.g. cross DC).

Also refactor LBFactory handling of the cpPosTime URL parameter.

Change-Id: Ie5744577609f4c193cacd71334f92b30f02e3846

Add missing @throw tags to IDatabase/Database

Change-Id: I235c5beacce38e9121fb1db94ce410d010356fcf

Use parent function in mysql DB class in getWeightScale()

May as well keep this case in sync with the base class

Change-Id: I4c62c3247ec3372c6a274b889fac1b7a883f5cf5

Merge "jobs: Remove ClearUserWatchlistJob 'batchSize' option"

Localisation updates from https://translatewiki.net.

Change-Id: Idd9b2d5c55e485a3d30473323ee224b628c0e83b

jobs: Remove ClearUserWatchlistJob 'batchSize' option

Use $wgUpdateRowsPerQuery instead, and use its value at run-time
instead of inside the job parameters. Also helps with deduplication
if batchSize is changed.

Change-Id: Ifef25a3ae5ae2418359a41eba05778613fbb548f

Merge "RCFilters: Followup I75cfb2b56a: Fix function declaration"

Use ParserOutput stateless transforms

We still set the state in many cases for benefit of extensions, but all
calls within core should no longer be using non-default state.

Change-Id: I78b62ec33fcb8273acb9b3b4e9012215442be94c
Depends-On: I140ff32373430b61b92226689ef9b58cca317450

ParserOutput: Add stateless transforms to getText()

The stateful transforms are deprecated.

Inspired by Krinkle's If2fb32fc.

Bug: T171797
Change-Id: Ied5fe1a6159c2d4fa48170042b44d735ce7b6f9b

RCFilters: Followup I75cfb2b56a: Fix function declaration

Bug: T181671
Change-Id: Ie69e9f0f77b2c463bc2e4d41cd9a1b8e87f7efef

Merge "Combine changes limit and time period selectors"

Merge "Add new core tags"

Add new core tags

Add tags to types of edits that get automatic edit summaries:
- Making a page a redirect
- Changing redirect target
- Changing an existing redirect into a non-redirect
- Blanking of the page
- Removing nearly all (more than 90%) content
- Rolling back an edit

Bug: T167656
Bug: T73236
Change-Id: Ie7f637fcec5ee659c1086e28e8ba21f470c45160

Fix phpdoc for Category::newFromName

Change-Id: I78e18b41acda9aa143360a49fd161b8af1d218be

HTMLMultiSelectField: Allow formatting in section headings in OOUI mode

Follow-up to 5a113417e5af9d0d0dbed63429649a9780784d45.
It is allowed in non-OOUI modes.

Bug: T181698
Change-Id: Ib2ebcd1a92f1b02a528787fe3fd1f5452b9808c4

WatchedItemStoreUnitTest return strings in db mocks

Change-Id: Ib74acd0e2e3a9fd94ca1b9bc06529852f770d0d1

Fix timestamps in ImportTest

They need to be passed through $db->timestamp().

Bug: T75174
Change-Id: I94ff961ae27220bcf15eda201677159f6c2d38ef

Convert timestamp from DB in RevisionDbTestBase::testInsertOn

Bug: T75174
Change-Id: Ia45eec0e072c794931ea7a415259602d03f4c31e

Merge "Add wfWaitForSlaves() to maintenance/migrateComments.php"

Merge "Database: Fix degenerate parenthesized joins"

Avoid DB rows with usable names but ID = 0 by introducing "interwiki" usernames

Importing revisions in MediaWiki has long been weird: if the username on
the imported revision exists locally it's automatically attributed to
the local user, while if the name does not exist locally we wind up with
revision table rows with rev_user = 0 and rev_user_text being a valid
name that someone might later create. "Global" blocks too create rows
with ipb_by = 0 an ipb_by_text being a valid name.

The upcoming actor table change, as things currently stand, would
regularize that a bit by automatically attributing those imported
revisions to the newly-created user. But that's not necessarily what we
actually want to happen. And it would certainly confuse CentralAuth's
attempt to detect its own global blocks.

Thus, this patch introduces "interwiki" usernames that aren't valid for
local use, of the format "iw>Example".[1] Linker will interpret these
names and generate an appropriate interwiki link in history pages and
the like, as if from wikitext like `[[iw:User:Example]]`.

Imports for non-existant local users (and optionally for existing local
users too) will credit the edit to such an interwiki name. There is also
a new hook, 'ImportHandleUnknownUser', to allow extension such as
CentralAuth to create local users as their edits are imported.

Block will no longer accept usable-but-nonexistent names for 'byText' or
->setBlocker(). CentralAuth's global blocks will be submitted with an
interwiki username (see Ieae5d24f9).

Wikis that have imported edits or CentralAuth global blocks should run
the new maintenance/cleanupUsersWithNoId.php maintenance script. This
isn't done by update.php because (1) it needs an interwiki prefix to use
and (2) the updater can't know whether to pass the `--assign` flag.

[1]: '>' was used instead of the more usual ':' because WMF wikis have
many existing usernames containing colons.

Bug: T9240
Bug: T20209
Bug: T111605
Change-Id: I5401941c06102e8faa813910519d55482dff36cb
Depends-On: Ieae5d24f9098c1977447c50a8d4e2cab58a24d9f

Added new success message when CLI Installer completes its work succesfuly.

Bug: T181468
Change-Id: Iddce5d0e1244ff1e4740fdaf1b07023355c62f27

Merge "Add wfDeprecated() call to MimeMagic::singleton()"

Merge "WebStart: Remove use of realpath() for $IP"

Merge "s/Latn/latn/"

Merge "Shell: skip null parameters"

Merge "Disallow setting DBO_IGNORE in Database for sanity"

s/Latn/latn/

Follows-up I7f89569c6ee6640d368af1378e84c5a9e725dad0

Change-Id: I7600ebe27cfc8c8d892c8169a6f59e934e5eec08

WebStart: Remove use of realpath() for $IP

When installing MediaWiki in a sub directory of document root,
and including it from an /index.php file in the document root,
MediaWiki succesfully includes WebStart from index.php, but
WebStart.php fails to include Setup.php.

For example, MediaWiki installation at /var/www/mediawiki with the
following file at /var/www/index.php.

```
<?php
require __DIR__ . '/mediawiki/index.php';
```

Failure:

> Fatal error:
> require_once(): Failed opening required '/var/www/includes/Setup.php'
> (include_path='.:/usr/local/lib/php') in
> /var/www/mediawiki/includes/WebStart.php on line 97
>
> Stack trace:
>   1. {main}() /var/www/index.php:0
>   2. require() /var/www/index.php:3
>   3. require() /var/www/mediawiki/index.php:40

Bug: T153882
Change-Id: Icd8cfa580ce1c22bc3bf177570a9f4a940d2427c

Merge "objectcache: use INTERIM_KEY_TTL constant in WANObjectCache for readability"

Disallow setting DBO_IGNORE in Database for sanity

In the off chance something called this, it would break all
sorts of code that expects that either query result functions
either succeed or throw an error.

Callers are not expected to have to check if the result of
a query is meaningful or false due to an error.

Change-Id: I0b4fe1403f55a399ffd40817ed12f857087d6f83

Merge "Add ip_changes to postgres/tables.sql"

Add ip_changes to postgres/tables.sql

Follow-up Ic11c64813ee04e8520771bfa156f8e51404273e7

Bug: T177258
Change-Id: I69a278c1928ed91ef5cbe5b66e590eef629a2981

Localisation updates from https://translatewiki.net.

Change-Id: I71dd8b5115cb4c4db79542b25e00f0cbea8a1275

Merge "Remove DEFAULT '' from postgres ip_changes.ipc_rev_timestamp"

Merge "Increase Opera minimum for Grades A and C to 15"

Merge "Remove moment.js bump from 1.31 release notes"

Database: Fix degenerate parenthesized joins

The SQL standard supports parenthesized joins like

    a JOIN (b JOIN c ON (...)) ON (...)

But it doesn't support parenthesizing a single table name, i.e. a
one-table "join", like

    a JOIN (b) ON (...)

Detect the degenerate single-table case and omit the parentheses.

Bug: T181674
Change-Id: I82cacd80465092aa67ff19bdcfd6682001bf12ab

Shell: skip null parameters

Right now they're treated as empty strings, however
this doesn't allow skipping parameters in the middle like
 $params = [
     'foo',
     $x ? '--bar' : null,
     '--baz',
 ];

In some cases this matters, e.g. `ls` works while `ls ''` doesn't.

Also, fix spacing problems the new tests uncovered:
* Extra space when using params()
* Missing space when combining params() and unsafeParams()

Change-Id: Icb29d4c48ae7f92fb5635e3865346c98f47abb01

Merge "mw.notifiction: Bring in some rules from Vector"

Remove moment.js bump from 1.31 release notes

Going to be backported to 1.30

Follows-up I7f89569c6ee6640d368af1378e84c5a9e725dad0

Change-Id: I691f5e0cf1f4685f8d1ff55a7ff76a1ad2587d89

Merge "resources: Bump moment.js from 2.15.0 to 2.19.3"

Remove DEFAULT '' from postgres ip_changes.ipc_rev_timestamp

Follows-up Ic11c64813ee04e8520771bfa156f8e51404273e7

Bug: T181672
Change-Id: I58fe5edb1e7656773271634f702ed044c6036497

resources: Bump moment.js from 2.15.0 to 2.19.3

Languages and locales now with support:
* bm
* de-ch
* gom-latn
* gu
* kn
* sd
* tet
* ur
* uz-latn
* yo

Skipped languages because we don't support them in MW:
* ar-dz
* ar-kw
* ar-ly
* en-ie
* en-nz
* es-do
* es-us
* fr-ch
* me
* nl-be
* tlh
* tzl

Also drop 'x-pseudo' which was included but not used.

Bug: T161732
Bug: T181547
Change-Id: I7f89569c6ee6640d368af1378e84c5a9e725dad0

Add wfWaitForSlaves() to maintenance/migrateComments.php

Change-Id: Idb8d21116a52c473cc4658883fe13855bad92222

Merge "Align other review tools and saved filters buttons"

Merge "Allow disabling exact input query in results in mw.widget.TitleWidget"

Merge "Small change to "Other review tools" link"

Allow disabling exact input query in results in mw.widget.TitleWidget

* Add configuration option to mw.widget.TitleWidget, to make
addition of exact input query to the results optional.

Change-Id: Ic7a3aaf38a3827917e4d349cc6b828f37cc64b67

Merge "mediawiki.special.apisandbox.js: Apply JavaScript coding conventions"

mw.debug: Avoid overlap between debug toolbar and OOUI dropdowns

Bug: T106095
Change-Id: Ia519205852324899430ac7a769ee7e81ed80c37f

mw.notifiction: Bring in some rules from Vector

The larger padding makes sense as a default. Also increase
the specificity of the error/warn rules so they are overridden
by changes to the default state.

Change-Id: I157e41631c4a09842b869215b0d6dbb99c1a4d36

Merge "Revert "Hide empty OOUI FieldsetLayout headers""

Merge "Swap german alias order for img_thumbnail and img_manualthumb"

objectcache: use INTERIM_KEY_TTL constant in WANObjectCache for readability

Change-Id: I13936397def11a13849fe14491b752dd797f6759

Increase Opera minimum for Grades A and C to 15

This drops into Grade X the Opera versions 12.10+ from Grade A,
and versions 10.x from Grade C.

Bug: T121517
Change-Id: Ie4df9c751bd0250581cd2e25cf1b38e69e67e82f

Add SQL for postgres, and fail gracefully in populateIpChanges

If the ip_changes table doesn't exist, the populateIpChanges maintenance
script will fail gracefully, throwing a descriptive error.

The postgres SQL is untested.

Bug: T177258
Change-Id: Ic11c64813ee04e8520771bfa156f8e51404273e7

Merge "Fix undefined $db"

Fix undefined $db

Bug: T181565
Change-Id: I286374d4ca0ab9435d7d54c4f8b9660a60e5f1e3

Revert "Hide empty OOUI FieldsetLayout headers"

No longer needed after I79fe95e32272f013a1ce4e48a811fa9fbb53f4e6
(released in OOUI 0.24.3).

This reverts commit 382e366196b2b9aa7eb08c92776e6f716f8f5d65.

Depends-On: Ifdd44ead7d782288aaef0979322f0943f7304835
Change-Id: I37747b4b68801caf442b77e747f486053f279853

Update OOjs UI to v0.24.3

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.24.3

Depends-On: Ifdd44ead7d782288aaef0979322f0943f7304853
Change-Id: Ifdd44ead7d782288aaef0979322f0943f7304835

Merge "objectcache: only give current format keys getWithSetCallback() callbacks"

Merge "Clean up formatting of SwiftFileBackend::sanitizeHdrsStrict()"

objectcache: only give current format keys getWithSetCallback() callbacks

Callback code that happens to make use of $oldValue might not be able
to handle missing, new, or changed fields due to key version changes.

Overhaul testGetWithSetCallback_versions() to be cleaner and cover the
case of unversioned => versioned keys.

Change-Id: If108a73078c530c985d30bdadcbfa9ddd53dc2be

Combine changes limit and time period selectors

- Combine number of results and the time period panels.
- Add gear/settings icon.
- Make the "View newest changes" link more prominent, by moving
the indicator into the same row as "Live updates" button and
the combined number of results and time period selector.
- Add reload icon to "View newest changes" link.

Bug: T177926
Change-Id: I793bfaa91a7823bdec7af5816d9398d2a993262a

objectcache: add more comments to WANObjectCache

Change-Id: I567f45fa7b33226b0ccf7d80440802eab71ff753

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

Localisation updates from https://translatewiki.net.

Change-Id: I8d6691b507556430230e527732387e7955f2c02c

Merge "Use time forcing methods to avoid WANObjectCacheTest flakeiness"

Small change to "Other review tools" link

- Reduce the separation between the label and the expand/collapse icon.

Bug: T179539
Change-Id: I0bfc8a72650325a815be930a64e472ae66f536f6

Merge "Introduce ClearUserWatchlistJob"

Introduce ClearUserWatchlistJob

Change-Id: Icea573a10078ea3f09dc2e4e9fdc737bf639935d

Clean up formatting of SwiftFileBackend::sanitizeHdrsStrict()

Change-Id: I5e411f4bf50836d22db01220853776494a9e69d5

mw.special.preferences: Widen the dropdown of the "Time zone" field

Same styles as for the "Reason" field on Special:Block
(b2a0e3aac8f70802a5f45cc7e7fbb0f9c517fb82).

Bug: T181433
Change-Id: I957226ce3764e1bcc14ea088cadf4531a1dd1886

Do not strip Content-Type header for POST requests to swift

libcurl adds 'Content-Type: application/x-www-form-urlencoded'
to a POST request if the 'Content-Type' header is not set
manually. Because data in swift is updated via POST, the
Content-Type header must be set explicitly to stop a run of
refreshFileHeaders.php from changing the Content-Type of all
files in swift to application/x-www-form-urlencoded

Bug: T178849
Change-Id: I43c21bc1b73e37104cf07cd5f1c1557f472b9898

Merge "Document how to run just one Selenium test"

Merge "ESLint ecmaVersion setting is not needed if env is es6"

Document how to run just one Selenium test

Bug: T139740
Change-Id: Ieb5f27285046c33d77b615159dc2000d80ef644a

Align other review tools and saved filters buttons

Bug: T179825
Change-Id: I654d417072bb58a6518a1094061208788b64c1f8

Merge "Allow file headers to be refreshed from database"

Use time forcing methods to avoid WANObjectCacheTest flakeiness

Use of microtime() is now just for baselines, and it is no longer
assumed to be increasing with each call. Such an assumption is
particuliarly bad on Windows.

I've done 100X run rounds with now failures on Windows.

Change-Id: Ica2a47982495bc95b10ca507414972744ea9507e

Merge "logging: Abstract LogPager enforcing of action and performer restrictions"

logging: Abstract LogPager enforcing of action and performer restrictions

This avoids duplicating the code in the future, such as proposed for
I3ea2c050b6dd6c (T16711).

Change-Id: Ic53b074f542014f156b006864d91a138ba5fb22b

Merge "objectcache: add "graceTTL" option to WANObjectCache::getWithSetCallback()"