Merge "Use MediaWiki\SuppressWarnings around trigger_error('') instead @"

Merge "Include bot password app ID in audit data"

Merge "Fix regular expression"

Clean ups of dropping tag_summary table

Change-Id: I92dc906d0dfd870e6b77671e5d7cfd3f56090325

Merge "travis: Add PHP 7.3 (allowed failure)"

Merge "Special:AllMessages: Improve zebra striping on hover"

Localisation updates from https://translatewiki.net.

Change-Id: Idfb2f9b39fe0eacd31663f9c769be3e88c46353e

Drop tag_summary and valid_tag tables

Bug: T209525
Change-Id: I153607239145a772f5ea47eefd3340096a494b4b

ChangesList separates content elements from presentation

Special:RecentChanges and Special:Watchlist benefit from this change

Bug: T205581
Change-Id: If46d9ed904fdc6f01decbfde12ade0c47e282171

Fix copy-paste error

It's actually adding the column on the wrong table

Change-Id: I2fd8ea50f3eb4b5da04fce2ea0348a2dc6329965

Option to load restrictions from DB_MASTER

Adds a $readLatest parameter to Title::loadRestrictions that skips
the replica DBs and goes straight to the master DB to get the latest.

TODO: should I add $readLatest to all the getRestrictions* functions?

Bug: T210983
Change-Id: I834de033fb79870de7d081701693663fb6ee93d3

Merge "HTMLForm OOUI Flatlist: Reduce `margin` slightly"

Merge "MagicWordFactory: Follow-up on I5d2166f155e31900cb40c"

MagicWordFactory: Follow-up on I5d2166f155e31900cb40c

Applied comments from I5d2166f155e31900cb40c to improve on
the unit test. makeMagicWordFactory() makes passing "null"
optional (which is default, hence some cleanup for calls).

Change-Id: Ib9a096c519f0de7286aa244bf3bb842cc11740f9

Merge "ApiComparePages: Don't error with no prev/next rev"

Merge "tests: Add PHPUnit tests for methods in MagicWordFactory::class"

tests: Add PHPUnit tests for methods in MagicWordFactory::class

* testGetContentLanguage() - covers the getContentLanguage() method.
* testGet() - covers the get() method in the MagicWordFactory class.
* testGetVariableIDs() - covers the getVariableIDs method.
* testGetSubstIDs() - covers the getSubstIDs() method.
* testGetCacheTTL() - covers the getCacheTTL() method. This covers
  both paths when there is a valid and an invalid caching hint.
* makeMagicWordFactory() - a helper function for creating an object
  of the MagicWordFactory class.
* testGetDoubleUnderscoreArray() - covers getDoubleUnderscoreArray()
  method and also calls newArray() under the hood so that is also
  covered.

Change-Id: I5d2166f155e31900cb40c22fb976e81d0d545627

HTMLForm OOUI Flatlist: Reduce `margin` slightly

When provided without label, flatlists are in general modificators
logically connected to the input above as seen on Special:BlockList
for example. Reducing the `margin` slightly helps to visually
emphasize the connection.

Bug: T177683
Change-Id: Ice666e54e440f97d177580ac258dfbed1577da0a

Upgrade wikimedia/ip-set to 2.0.0

Bug: T209756
Depends-On: I21a6032defb15b77519631b251b5e577207902a8
Change-Id: I424cf8871daf1e19b9d03511ace5f09d77746145

travis: Add PHP 7.3 (allowed failure)

Change-Id: I2205ae6b3543ec8fc094c043723c73ab1d1476b5

Merge "Message: Don't include Title objects in the serialization (part 2)"

Merge "ProfilerExcimer: allow early start"

Localisation updates from https://translatewiki.net.

Change-Id: I14d75d0d131a158d36055d612ab9139586258978

Merge "DateTimeInputWidget: Fix UI/UX glitches"

Merge "Follow-up I2266ec0: Mark new method as an internal"

Merge "Set default of sleep time in populateChangeTagDef to zero"

Set API limits for page restrictions to 10

Bug: T212142
Change-Id: Ide0f2f86df2101194155be53a4b9012ddefbbcaf

Merge "Fix sql.php --json behavior"

Merge "DateTimeInputWidget: Use 'clear' icon to clear input"

Merge "Stop updating tag_summary table"

Special:AllMessages: Improve zebra striping on hover

Previously, the full row (spanning two sub-rows) would not
get the hover effect when hovering over the second sub-row.

Inspired by <https://stackoverflow.com/a/15465002>.

Change-Id: Id698516da4fbe01f90cbe1499fb81721642a4dfd

Merge "Special:AllMessages: Remove al_title and allmessages-customised classes"

Special:AllMessages: Remove al_title and allmessages-customised classes

* The al_title class was unused, and thus removed.

* The allmessages-customised class was used in CSS to target
  elements with 'al_default' and 'al_actual' classses within it,
  which are in turn only used for that purpose.
  Change the HTML to match the needs of the CSS by only setting
  al_default/al_actual classes on customised messages and remove
  need for the parent element class.

On a plain install this reduced output by ~2K from 75.8K to 74.0K,
and allocate 100 (2x50) fewer attribute nodes.

Change-Id: If2dd4cf5c937935f03f30b8410a32b6821b0bf34

Fix sql.php --json behavior

* return empty array when the query had no result (instead of
  falling back into non-JSON mode)
* return JSON for write queries as well
* retain legacy behavior in non-JSON mode for empty SELECTs, just in
  case somebody relied on it

Change-Id: Iaefbb443650a395278d1cc9ab6aa668b13b217c9

ProfilerExcimer: allow early start

Allow the profiler to be started elsewhere and passed into
ProfilerExcimer via the configuration array. This allows the profiler to
be started in the auto_prepend_file. XHProf doesn't need this because it
has a single global profiler.

Change-Id: I348499a15d9cc42de0ba1a20afc2283b794931a3

DateTimeInputWidget: Fix UI/UX glitches

- Limiting focus state of inputs to datetime handle boundaries
- Showing clear button only when there's value to be cleared
- Distance clear button from input
- Removing focus outline from day and timezone values

Bug: T212047
Change-Id: I9b815f38abeacacf921873529117c51c187d7fb9

Merge "Add ProfilerExcimer"

Localisation updates from https://translatewiki.net.

Change-Id: I005fa8d24c284741e5b139e1228b6a3251407a65

Set default of sleep time in populateChangeTagDef to zero

The script already has waitForReplication(), that's good for non-WMF setups

Bug: T206568
Change-Id: I020a4472796eeff2561da6c17a817091d4e9af5a

DateTimeInputWidget: Use 'clear' icon to clear input

Also removing unnecessary indicator overrides.

Bug: T212022
Change-Id: Ie962138811aff03d72ea29eb260131c37067456b

Fix exception on certain http failures

Task T202110 included a change to recognize an HTTP status code
of 0 (zero) as an error, but it failed to set a status message,
resulting in an exception.  Changed to set a status message of
'Error' so that required value is not empty.

Bug: T212005
Change-Id: I5fb78555bfcaeccdd726432f4dfc70924a385c41

Localisation updates from https://translatewiki.net.

Change-Id: I20f67d42ea1cd74c732c87a50d4db04e16a5abc5

Merge "PostgreSQL: Drop three unused columns"

Merge "Use SpecialPageFactory in ApiQueryQueryPage"

PostgreSQL: Drop three unused columns

The columns dropped in this commit are:

* category.cat_hidden
  (for MySQL, dropped in 1.20 / 0cbc85494d47cdeb)
* site_stats.ss_admins
  (for MySQL, dropped in 1.21 / bd602d20833f435f)
* recentchanges.rc_cur_time
  (for MySQL, dropped in 1.24 / 2b441eba406f2c49)

Bug: T64103
Change-Id: I7a7e9a2ac5066a9c9ab2583419d507e14fdd8c2c

Use SpecialPageFactory in ApiQueryQueryPage

Instead of assuming that all query page classes can be instantiated with
no constructor parameters, use a SpecialPageFactory.

This would not have prevented T208924 (that would just have resulted in
a later, different error), but it’s just a good idea in general.

Change-Id: I2473a53acd9fefdeffb1fa7ecb88873f3ee345cc

Merge "eslint: Turn on reportUnusedDisableDirectives"

Merge "Introduce TagMultiselectWidget.php"

Introduce TagMultiselectWidget.php

TitlesMultiselectWidget and UsersMultiselectWidget share
a lot of functionality, so implement a common base class.

This also adds some things to UsersMultiselectWidget:
* shows a pending element to users with JavaScript
* makes the input configurable

Change-Id: Ie6649b476c64e795254f457e3863fa7f14aa05ac

eslint: Turn on reportUnusedDisableDirectives

Change-Id: I3fb88324ba5a46e44041c13eb6447c8e33f4e79b

Message: Don't include Title objects in the serialization (part 2)

Stringify the title instead. This will help avoid running into the
PHP/HHVM serialization incompatibility described in T210528.

This is part 2: Once all servers have part 1 (and rollback is unlikely)
we can stop storing the Title objects.

Bug: T210528
Change-Id: If3acfeb42788bd675c3022cc1b49dccc9ac1ecbe

Merge "Increase default minimum password length on privileged groups and bots"

Increase default minimum password length on privileged groups and bots

Minimum password length is now 10 by default on privileged
groups (bureaucrat, sysop, interface-admin), and bots.

Bug: T208246
Change-Id: I373c5c6865b90cdc5c4848266c5996dd190f6001

Merge "Use username from last successful login in Special:PasswordReset"

Merge "Use singular they to refer to IP address"

Follow-up I2266ec0: Mark new method as an internal

Per Krinkle.

Change-Id: I9720aea6a55a414d861b74edba7d7b421e06db2b

Merge "Soft deprecate Title::getUserCaseDBKey()"

Soft deprecate Title::getUserCaseDBKey()

Identifiers of groups are encouraged to be in lowercases, so we'll
soft deprecate this rarely-used method now and remove it later.

Bug: T202094
Change-Id: Idee1b7f491dbe1114b4489fda3c16be70175f2d8

Merge "Temporarily remove guzzle as default for HttpRequestFactory"

Merge "Revert "ChangesListSpecialPage: Make maximum limit consistent (1000)""

Revert "ChangesListSpecialPage: Make maximum limit consistent (1000)"

This reverts commit ac2837c9b3ed8a7f242b371d072b75880f29f4e7.

Reason for revert: See T210897#4821559

Bug: T210897
Change-Id: Ifb6debe3eaddb4a2cb62ace5cfea59c06e77d141

Include bot password app ID in audit data

Authentication audit logs should indicate whether a login is via the
normal password or a bot password (and which one). For failed logins
it could be included in the error message, and it usually is, but
for successful ones there is no message, so we'll send the app ID as
a new AuthManagerLoginAuthenticateAudit parameter.

Bug: T194338
Change-Id: I8aab48177b81a8e80dae118e6476a8f6a32089f1
Depends-On: Id482d2e2205960a0facd334e456d3a23bcad0ece

Temporarily remove guzzle as default for HttpRequestFactory

Temporarily switching back to curl/php as the default for
HttpRequest objects, to resolve production issues until a
long-term fix for the out-of-memory issue from T211886 is implemented.

Bug: T211886
Change-Id: I6201ac1292b41f82d920a431376d791a666355e8

Use singular they to refer to IP address

The previous “him/her” does not cover all pronouns people might use, and
is also at odds with how MediaWiki usually addresses users whose gender
is not known.

Change-Id: I468fbd25ab80f21eb591c349117774fe8ce2d9dc

Merge "ApiPageSet::initFromPageIds: Default $filterIds to true"

Merge "objectcache: Fix WinCache keys longer than 150 chars"

Merge "Remove loading of module 'mediawiki.toc' in special pages"

Fix guzzle InvalidArgumentException when body is passed as an array

The postBody option to GuzzleHttpRequest can be passed as an array
or as a string.  We were previously handling the array case incorrectly.

Bug: T211806
Change-Id: I8f40b9de9d40a9361eb45103608bf3aaa943bf73

Merge "Improve readability of SpecialBlock::checkUnblockSelf"

Merge "Migrate SpecialUndelete and Diff from tag_summary to change_tag"

Merge "Add TransactionProfiler::redefineExpectations and migrate appropriate callers"

Merge "filebackend: simplify code in FileBackendStore::getFileStat slightly"

Localisation updates from https://translatewiki.net.

Change-Id: Iea258666b6ad7373ca01812dd23931e00c72f833

Stop updating tag_summary table

This table will be dropped in favor of change_tag/change_tag_def

Bug: T209525
Change-Id: I9e305dc0f71a126cff57cade37180ad438838030

objectcache: Fix WinCache keys longer than 150 chars

Adds MD5 hashing to the generation of keys for WinCache. This
fixes the issue of keys (e.g. sub-headings) longer than 150
characters.

Bug: T210794
Change-Id: Ieabf6b014a0660198077a23760a05564bf268535

ApiPageSet::initFromPageIds: Default $filterIds to true

I have no idea why that was false in Ic1975220. Particularly when, of
the three calls, only one was changed to specify the parameter and that
was changed to pass false explicitly.

If it hadn't been false, it would have prevented T211804.

Bug: T211804
Change-Id: I8eea8936e9f57bf3e336f81b62bb11b9e2668788

Merge "mw.Title.newFromUserInput: Rename vars, improve docs for double signature"

Merge "Replace $.type with typeof"

mw.Title.newFromUserInput: Rename vars, improve docs for double signature

Also add tests to cover this usage.

Change-Id: I61c7e7377ac7d75073dac025f5f7ff4e599756cb

Replace $.type with typeof

Change-Id: I75857eb205ce2fc3ba8f836f3f2ee66987b45748

Merge "Remove unused /* global console */"

Merge "Link log entries to Special:Log on ApiFeedWatchlist"

ImageListPager: Don't query by oi_user

For some reason we have indexes for `image` on `(img_user_text,img_timestamp)` and
`(img_user,img_timestamp)`, but for `oldimage` we only have
`(oi_user_text,oi_timestamp)`. Thus, when building the query in
ImageListPager, we have to be sure to avoid trying to use `oi_user`
rather than `oi_user_text` in the WHERE part.

Bug: T211774
Change-Id: Ibea058031f1cb3421e92e09f0a705ea00fb22008

Link log entries to Special:Log on ApiFeedWatchlist

This prevents RSS readers from breaking, because there aren't
entries which contain a link to a zero diff anymore.

Bug: T210920
Change-Id: Ia5f57b4ab60873bf5d7b2fcf2a2b4f6f90c135b3

Remove unused /* global console */

Change-Id: I330208689281cae251c1da9c147611a79b9b153a

ApiBase: Pass empty string, not null, for $conds to ->select

Bug: T211769
Change-Id: I4cf224c19b340fac5fc895bbee7507d77fd3fdfa

API: Use parenthesized join in ApiQueryBase::showHiddenUsersAddBlockInfo

The query is LEFT JOINing ipblocks, so we need to properly scope the new
JOINs with the comment and actor tables to get the block reason and
by-actor.

Bug: T210937
Change-Id: I230a4ab78e3ba6e83008b0466eeca8d40013ba3d

Merge "Have BotPassword::login() call AuthManagerLoginAuthenticateAudit"

Merge "Add PHPUnit tests for methods in ViewAction::class"

Merge "Support message parameters in JavaScript messages with uselang=qqx"

Support message parameters in JavaScript messages with uselang=qqx

Change-Id: I2266ec0770f08e826892a28f52f3201dfd10191d

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

Localisation updates from https://translatewiki.net.

Change-Id: I78c4d9c910da554460b91b6871b5e268a3a32c27

Merge "Improve layout of ExpiryInputWidget on mobile"

Have BotPassword::login() call AuthManagerLoginAuthenticateAudit

To facilitate logging all authentications, even ones not via
AuthManager, BotPassword::login() should call the
AuthManagerLoginAuthenticateAudit hook.

Bug: T194338
Change-Id: I27b75c56e060fcb8f71d84ddc2eb8e1533fef33a

Improve layout of ExpiryInputWidget on mobile

Bug: T208271
Change-Id: If3f9c417cd2fad98165b8a2eaef9a05673ce5b4d

filebackend: simplify code in FileBackendStore::getFileStat slightly

Change-Id: Ic888192b592a62f316fa076678a9fcef9bb8553b

Merge "rdbms: reduce LoadBalancer replication log spam"

Fix typo in message documentation

Change-Id: I373c973f35bb8897a00c96c51d83371b2d9eca4a

Merge "Add new rules when user is blocked for UTP"