Merge "Do not add limit to ApiQueryPagePropNames when database type is mysql"

Merge "Add a space between "Go to page" dropdown and button for multipage images"

Merge "Revert, Follow-up: -1 recent contributors"

Revert, Follow-up: -1 recent contributors

This reverts commit b67f40e5b01af569794eba26da79497329590d36.

There was an issue that the web installer, when inserting initial
values into the db table site_stats, would not insert any value for
ss_active_users. That lead to some places, most notably
Special:CreateAccount, to show off -1 recent contributors.

The commit to revert solved this by setting a default value of 1 for
the ss_active_users field in the database schema. However, while the
default schema has been changed, there hasn't been any schema update
procedure implemented. Also, the default of 1 is inconsistent with
other defaults in that table.

Revert that schema change completely. Instead, make the web installer
insert 0 to the field ss_active_users (just like it does with all other
fields in that table).

Bug: T56888
Change-Id: I4298754c2238f6e5447a5df77a4ada722ba302b2

Merge "mediawiki.toolbar: Emit deprecation warnings"

mediawiki.toolbar: Emit deprecation warnings

Callers to the EditPageBeforeEditToolbar hook can now return false to
signal that the toolbar is over-written, in which case this old code
won't be called (so no deprecation warnings for users of WikiEditor,
CodeEditor, etc.).

Bug: T30856
Depends-On: I5e755ef5dffb843368563511044d3123f14dc4bc
Change-Id: I3bed70a20e4b94fe3d04a00925b8012312202574

Merge "exception: Add newline at the end of a debugging line for CLI users"

exception: Add newline at the end of a debugging line for CLI users

Bug: T162315
Change-Id: Ia2963fa298ab0e4782ef823e7f53322cd6a9c99b

Localisation updates from https://translatewiki.net.

Change-Id: Ice5f1ec383e94de1b6e43c581a821e415edfe767

Merge "Add percent and permille to special character list"

Extend grammar rules for Ukraininan language

Change-Id: I2a18acee6a1c4a7462953477da50d57239994f37

Add percent and permille to special character list

Bug: T163894
Change-Id: I82a926412fb0f7c886e1e17ed4cf05cf2d4f00d9

Merge "SpecialMediaStatistics: Remove WMF-specific hack, no longer needed"

SpecialMediaStatistics: Remove WMF-specific hack, no longer needed

We deleted the broken row from the databases per T155769.
This reverts 6e02a95227c4ef194ff2299a823aaa51aa9a223c.

Bug: T74535
Bug: T155769
Change-Id: Iac0873eb7112502dd7fc6b34ca956d9a2dffcb44

Merge "EditPage: Factor out getSaveButtonLabel method"

Merge "Interwiki: Don't override interwiki map order"

Update OOjs UI to v0.21.2

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

Change-Id: Id7cae80a4f3d84ea5d50003f2409537b307c0c58

Merge "RCFilters UI: Create base classes for shared objects"

RCFilters UI: Create base classes for shared objects

Preparing for adding other types of items than filters (namespaces,
users, tags, etc) this commit creates base classes for the model
and relevant widgets and extends Filter* from them.

Bug: T159942
Bug: T163521
Change-Id: I61c88a1f14a3ca9d91aa831187eda156468a6591

Merge "Special:Block: Fix FOUC of bold 'mw-block-hideuser'"

Interwiki: Don't override interwiki map order

The ksort() here was causing the order to be enforced as
alphabetical instead of preserving the original order.

The order usually doesn't matter, except with regards to handling
of duplicates. Due to Parsoid normalising external links to interwiki
links, it has to do a reverse lookup. In doing so it has to decide
which one to prefer. It currently picks the first match from the
API request for meta=siteinfo&siprop=interwikimap, which didn't
match the defined order in the actual Interwiki map due to ksort()
being called in getAllPrefixes().

Sort in this function was originally introduced in 2010 with
commit 844e7c83e4 (2011; r92528; T21838), which is otherwise unrelated
and left no rationale.

The existing unit tests needed to be adjusted slightly as they
assumed alphabetical order. While it appeared they were also defined
in alphabetical order, this was merely the order of the variable
creation. The effective order is preserved within locals and globals,
but overall globals come before locals.

Also removed the duplicate test for Hash and CDB in InterwikiTest
that belongs in ClassicInterwikiLookupTest instead.

Bug: T145337
Change-Id: I7348748801cbdf16c6ceea5b0654fc174b79707e

Special:Block: Fix FOUC of bold 'mw-block-hideuser'

Loading Special:Block/<your own name> shows "Confirm block", but
this then becomes bold a second later. This is because the styles
are part of 'mediawiki.special.block' which also contains JavaScript
and is loaded with addModules().

Move the styles to mediawiki.special.css instead.

Change-Id: I1c47256227603a6cde36238fdd7e11205d9494ec

Localisation updates from https://translatewiki.net.

Change-Id: I53453d3129920681a8eb8ece0c9c2cba400cab77

Add a space between "Go to page" dropdown and button for multipage images

Change-Id: I174da5597563be36cc6e67a485f8556b9bcd84f6

Merge "Keep headers from jumping when expire interface is shown"

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

Localisation updates from https://translatewiki.net.

Change-Id: I4ccfc62901bfa044c19b63fa30b450af8e9aa0a5

Merge "Remove usages of RequestContext::getStats()"

Remove usages of RequestContext::getStats()

Bug: T156810
Change-Id: Ic07896a68389e20ce0a35a06049b8c51b6c3f699

Keep headers from jumping when expire interface is shown

Change-Id: If2cb7c8e2e30c5e40f32113353f571a1fba63d29

Add Dinka (din) to Names.php and languages/messages/

Bug: T163600
Change-Id: I2bf03c97680c17cddf48d05adee88581640268ec

Merge "New maintenance script to clean up rows with invalid DB keys"

Do not add limit to ApiQueryPagePropNames when database type is mysql

It's a known bug in some versions of mysql (including WMF dbs) that it
won't do a loose index scan when a limit is applied. Given that number
of possible types is still very low, this shouldn't be a problem.
More info: https://bugs.mysql.com/bug.php?id=61517

Bug: T115825
Change-Id: I4c3b885ac05b793088a92e054a38a36b9d07c0d4

Merge "mw.widgets: Add SelectWithInputWidget and its PHP implementation"

mw.widgets: Add SelectWithInputWidget and its PHP implementation

Bug: T106999
Change-Id: Ic158bec3c463fba5099b05f41c9686f833e1c313

Localisation updates from https://translatewiki.net.

Change-Id: Iba16aa2179c535c6d48a3745cb3b861ebd39b067

Localisation updates from https://translatewiki.net.

Change-Id: I4a646e2126a03104fbb2a23eeec923cdcea5d4a8

Merge "Set a persistent connection ID for RedisConnectionPool"

Merge "Add benchmarkTidy.php, to benchmark tidy drivers"

Merge "Send 500 http status code, instead of 200, for DBConnectionErrors"

Set a persistent connection ID for RedisConnectionPool

This re-uses the options hash as the ID so that re-used connections
do not clobber each others settings.

Change-Id: I17e5993ecdab1770259803e06956b85b9ebb2ba6

Merge "RCFilters UI: Fix FilterTagItemWidget styles"

Localisation updates from https://translatewiki.net.

Change-Id: Ifec3cc97ff0bf69a31843968228c9bd9c1618068

RCFilters UI: Fix FilterTagItemWidget styles

The styles got a bit messed up when we switched to using
TagMultiselectWidget. Fixes:

* Switch to using the methods of flaggable element (which TagItemWidget
  already is using) instead of custom css classes
* Use flag:invalid for conflicts, so the styles are consistent with
  OOUI styling for invalid tags, but keep the label red color, so it
  is still evident when the tag is selected.
* Bonus: Actually apply filterTagMultiselectWidget base class, because
  it wasn't applied before (oops)

Bug: T163522
Change-Id: I1cc3bd743d8b930bee23386516ca123ee3c36983

Merge "Rephrase enotif_lastdiff and enotif_lastvisited"

Merge "Set default to 1 recent contributor instead of -1"

HTMLForm: add errors to html in ooui variants

HTMLForm when using OOUI mode was not adding errors to the resulting
HTML, if depending on message arrays, as previously used before
Status.

This exposed additional problems. Aligned the function a bit closer to
HTMLForm's getErrorsOrWarnings()

Bug: T158492
Change-Id: I8765a025dd441676e35a7c183c67b37036643c1e

Send 500 http status code, instead of 200, for DBConnectionErrors

Bug: T160875
Change-Id: I4c909457938a19358dfc4b2aec216b1a148df46c

Add benchmarkTidy.php, to benchmark tidy drivers

Plus representative input file

Change-Id: I254793fc55c57a98c07ae1e4c27e6005965c9a20

Merge "Fix preference tab switching so it doesn't fight prevTab v url hash"

Merge "RCFilters UI: Unify li bullets with and without highlights"

Merge "RCFilters UI: Change text for edit authorship group"

Fix preference tab switching so it doesn't fight prevTab v url hash

There was a bug where, when loading the Preferences page, the tab
was initially set by the URL hash (correct behavior) but then later
the system asked for the 'previous tab' in session storage, and changed
it to that -- which overrode the URL hash placement. In most cases,
the previous tab and url hash seemed to have been correct because
the user would go to the Preferences section and expect whatever
was the most recent tab they saw to open - but in the case of
specifically hitting "Beta" link (or clicking a direct link with
a specific hash to one of the preferences tabs) the previous tab
and hash had a fight, where the url-hash was the loser.

This commit fixes that by deleting the 'previousTab' from session
when the hash is set. Also, as a bonus, made sure that when the
previousTab removes itself from session, it actually removes the
correct key (and not tries to remove the value, which it did up
until now, and would've resulted in the previousTab value never
actually being removed)

Bug: T162938
Change-Id: I95c809f41dec7d7120c73dffa4a107346d049f70

Merge "Fix bogus variable use in RefreshLinksJob::run()"

Merge "Use makeKey() in BacklinkCache"

Localisation updates from https://translatewiki.net.

Change-Id: Ia8218a33495bb93bad4f4970047dde834bb61f8d

Use makeKey() in BacklinkCache

Change-Id: I23c000c8c2c77aa538fe258ea34eb764987eb52d

Merge "Hold number of search results in a data attribute"

Fix bogus variable use in RefreshLinksJob::run()

Also removed two unused loop variables.

Change-Id: I9a9e0a83bdaa13c031857bc20f977161cf85baff

Merge "Switch to librarized version of TestingAccessWrapper"

Merge "Fix EditPage button/input accessors on PageObject"

Switch to librarized version of TestingAccessWrapper

Replaces \TestingAccessWrapper (defined in core) with
\Wikimedia\TestingAccessWrapper (defined in the composer package
wikimedia/testing-access-wrapper).

See https://gerrit.wikimedia.org/r/#/q/topic:librarize-testing-access-wrapper
for downstream patches.

The core version of the class is kept around for a while to avoid
circular dependency problems.

Bug: T163434
Change-Id: I52cc257e593da3d6c3b01a909e554a950225aec8

Fix EditPage button/input accessors on PageObject

The DOM was changed again in https://gerrit.wikimedia.org/r/#/c/348765/
and input elements where used for the buttons.

Bug: T163193
Change-Id: I910a31a30d5569aab69dc2e5f81b377b6d867f64

benchmarks: Report more metrics (min/max/median)

Add minimum, maximum, median to the report in addition to the mean (average)
which was already there. Based on benchmarkTidy.php from I254793fc5.

Example output:

> Delete
>    times: 10
>    total:   7.47ms
>      min:   0.53ms
>   median:   0.74ms
>     mean:   0.75ms
>      max:   1.21ms
>
> Truncate
>    times: 10
>    total:  72.38ms
>      min:   1.37ms
>   median:   8.32ms
>     mean:   7.24ms
>      max:  15.73ms

Change-Id: Ifd3064a3621e07f55505490403189cb47022c6c7

benchmarks: Add setup, bench naming, and custom count default

* bench(): Add support for setup function.
  Demonstrated by converting bench_delete_truncate.php to use Benchmarker.

* bench(): Allow benchmarks to be named. Default remains (fn + args).
  Useful for closures.

* Benchmarker: Support overriding the default count of 100.
  Demonstrated in bench_delete_truncate.php to run 10x instead of
  100x (previous: 1x).

Change-Id: Iac182eaf3053f5bf0e811cd23082f530629d8a4e

benchmarks: Minor clean up

Change-Id: I446ae1a9d9cdb6b26a6bb62367a432cea082f343

Fix bogus field reference in Category::getCountMessage() callback

Follows-up 922e68f739f143. (T162121)

Bug: T162941
Change-Id: I40623203e97f7155c2af171a37b1128a59415315

Add different log-in tooltip for private wikis

Bug: T148006
Change-Id: I14e9a554c72222e6f67bc120941199b999740886

Merge "Document which EtcdConfig parameters are optional"

Merge "Remove load array indexes from LoadBalancer errors"

Merge "UploadBase::getTitle can return null"

Merge "Lower the amount of jobs pushed into redis at once"

Document which EtcdConfig parameters are optional

Change-Id: Ic6b56ed2f8789ae7920cdfc12f34b00db4a76c3a

ReadOnlyMode: Add a few doc blocks

Follows-up 820f46964f7968a.

Change-Id: I7866eb7c8bb9c45a24a3c567a7befe3505821873

Lower the amount of jobs pushed into redis at once

This further limits how long the server can be tied up by push().

Change-Id: I02d242578dadc19912c9fccfdcf5e15c5eb78e9e

Merge "A service for read-only mode"

RCFilters UI: Unify li bullets with and without highlights

Make sure that the placement of the texts under the bullets is
the same whether the highlight is on or off. Otherwise, the list
is shifting a bit to the right to make room for highlight 'bullets'
when we press the highlight key.

Bonus: Make sure the highlight container is only appended to each
bullet once, even if the process is called for more than once on
the same content.

Bug: T163275
Bug: T163387
Change-Id: I4c2cf6176d5129dd2bc37d2f58ed84e85aca8560

RCFilters UI: Change text for edit authorship group

Bug: T149385
Change-Id: I96831b2a650eb0232013762675fb4c5bd325b6a9

Remove load array indexes from LoadBalancer errors

This are not very useful and where not using SPI interpolation either.

Change-Id: Ia3a33da3a4593fbcba59b21f5b5028860752ce09

Localisation updates from https://translatewiki.net.

Change-Id: I6e67503e539268e66f495a594f29231e3ee4b6dd

Merge "media: Remove unused private function in SVGMetadataExtractor"

media: Remove unused private function in SVGMetadataExtractor

Change-Id: I3cd00bb4f460a141966ad01e155a97b8a087a8e8

A service for read-only mode

Introduce a service to represent wfReadOnly() and friends.

It's necessary to have two service instances, one for wfReadOnly() and
one for wfConfiguredReadOnlyReason(), to avoid a circular dependency,
since LoadBalancer needs the configured reason during construction, but
wfReadOnly() needs to query the currently active load balancer.

Not having a cache of the configuration makes it possible to dynamically
change the configuration. Ideally things would not change the
configuration, and I removed such instances in core, but to support
extensions, I added a test ensuring that the configuration can be changed.

Change-Id: I9bbee946c10742526d3423208efd68cb3cc5a7ee

Merge "Remove links list from the 'rclinks' message"

Hold number of search results in a data attribute

Current way of counting returned search results (e.g. for
Event Logging) relies on counting the appropriate elements
in the search results page's DOM, up to the limit the user
requested (e.g. 20 by default). This allows us to record
the total number and the offset, useful for event logging.
For example, if we wanted to know whether the user viewed
2nd or 3rd set of search results.

Change-Id: Ic8601e9eeddac84ba8e0d7dc6f127bf360b6f90f

HACK: Temporarily patch OOjs UI locally to work around upstream problem

Applied upstream in Ic11b981e70e3.

Bug: T163358
Change-Id: If8c26ffe5095e4e8c2f206c9dda1981bccba2b62

Merge "Allow editors to exclude navigation items from search indices"

Merge "RC Filters: Detect filters conflicts to by-pass db query"

RC Filters: Detect filters conflicts to by-pass db query

Filters are in conflict when their combination is guaranteed
to return no results. For instance: minor and log entries
is a conflict because major/minor does not apply to
log entries and the field is set to major by default.

Letting conflicts go through result in some very slow
database queries.

Bug: T160220
Change-Id: Ia6b0125c675c4a3cc4e4be4f83d1bd10d23059ba

Merge "qunit: Also clear $.timers when stopping unfinished animations"

EditPage: Factor out getSaveButtonLabel method

Change-Id: I89a2e3b6b8d23b2fafa5eeec59d79e37f61edefd

Merge "resourceloader: Move mwNow() to after isCompatible()"

Merge "Update OOjs UI to v0.21.1"

qunit: Also clear $.timers when stopping unfinished animations

Stopping the animation is essentially the same as pausing, it remains in the
animation registry and will continue to fail all subsequent tests until one
of those later tests starts the animation queue again and also happens to
wait long enough for this unrelated animation to finish.

Fix the testrunner to actually fully stop the pre-existing animations, which
requires clearing $.timers as well. This matches the logic we have for
pending ajax requests.

Bug: T163211
Change-Id: Ic7d848187bc3c800e8347e0650093b2ffce6dddc

Update OOjs UI to v0.21.1

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

Depends-On: I5edcf21effee760d9951965a152481f2c603b59b
Change-Id: I5edcf21effee760d9951965a152481f2c603b5b9

build: Sync qunitjs version for karma with version of Special page

Special:JavaScriptTest loads qunitjs from /resources/lib/qunitjs
which was upgraded in 99aad9397e to 1.23.1.

However, when running QUnit tests from the command-line, QUnit is
loaded by karma-qunit using the npm qunitjs dependency, which was
still at v1.22.0.

Change-Id: Ie24b2b061735fa03a8ed1527d09f65c103506c67

Merge "Do an HTMLCacheUpdate of templatelinks on page creation"

Merge "RCFilters UI: Select tag when filter item is selected"

RCFilters UI: Select tag when filter item is selected

Retain the selection behavior from previous versions, and synchronize
the selection of the filter item in the menu with the tag items
in the MenuTagMultiselectWidget.

Bug: T163138
Change-Id: I548a1fca65372c697e24b77d928d56b18f4df1c5

resourceloader: Move mwNow() to after isCompatible()

Follows-up f2fb4a21af. This is logically still the same point in time.
Only 1 function before it now, isCompatible(), which is fine since it
is small and the variable is not used anyway if it returns false.

Change-Id: I34bbe8edf6e9625f8d80f829707adafcb1b91980