Cleanup postEdit and match styles to mw.notification

* Remove HTML template
* Add mw-notification class and dependency
* Fix variable scoping issues so firing two notifications
  doesn't cause one to become stuck

Bug: T58313
Change-Id: I9a702d23a8a9b5cc098cbb413213d73813bec097

Merge "Expire the cache of a User object when a group membership is set to expire"

Localisation updates from https://translatewiki.net.

Change-Id: Ia2686ac474c045e91fd29daef68ec0925b73823e

Merge "Release prior row locks beforehand in LinksUpdate::updateCategoryCounts"

Merge "SpecialJavaScriptTest: Do not accidentally pass parameters to QUnit.start()"

Merge "SpecialJavaScriptTest: Format long JavaScript code fragments"

Merge "Prepare for 1.30 branch"

SpecialJavaScriptTest: Do not accidentally pass parameters to QUnit.start()

If the mw.loader.using() call's promise is rejected, we end up
passing an Error object as the 'count' parameter to QUnit.start().
This seems to be harmless, but is very confusing when debugging.

Change-Id: I44caca5285dbced5a5876d9d7ff6236dbd3efc35

SpecialJavaScriptTest: Format long JavaScript code fragments

Change-Id: Iaa6369467e9fa846a8d5eb8f43cf0b523ef1a64e

Merge "Block: Remove old IP addresses from default autoblock_whitelist message"

Merge "mw.widgets.DateInputWidget: Highlight widget and open calendar on focus"

Merge "Correct the "Help", "Media" and talk namespace names for Dinka"

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

Correct the "Help", "Media" and talk namespace names for Dinka

The corrections come from users Dinkawiki and Mawanmuortat,
who did most of the translation into this language.

No aliases are needed, because the file was only created recently,
and not yet used anywhere.

Change-Id: Ic1ddfaca403e05e5971c516d6e7a454311794f3c

Release prior row locks beforehand in LinksUpdate::updateCategoryCounts

Bug: T163801
Change-Id: I6a026ee9a249da5a8fd9ceb216416340a9188d3f

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

Prepare for 1.30 branch

Change-Id: If9842b19ec995d03f93f96bfad4d61ca96c37f3c

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

Expire the cache of a User object when a group membership is set to expire

To avoid user permissions persisting for up to 1 hour beyond their expiry
time.

Bug: T163691
Change-Id: I85b578e6b7816639c3d0101d66efb1efedb17af6

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

mw.widgets.DateInputWidget: Highlight widget and open calendar on focus

Bug: T159987
Change-Id: I24327e1fe4b23de1b35833d3f23634094b5a2664

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