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 "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"

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

Allow editors to exclude navigation items from search indices

When you perform a particular search, the results can be polluted by
navigation elements that are not supposed to be displayed. This gives
editors the ability to mark sections of the document that should not
be indexed.

Bug: T162905
Change-Id: Iab2b83c3778cd5f7f44736c0da569fd938ae2968

Localisation updates from https://translatewiki.net.

Change-Id: I6c1534c40494dff1a9866fa5ec478c4646477688

UploadBase::getTitle can return null

Change-Id: I5bd94f6233476bda43a01155f6e7d6df420412e2

Remove links list from the 'rclinks' message

In some languages, the 'rclinks' message ends
with '...<br />$3.' where $3 is a list of links.
When $3 is empty, we end up with a lonely period
on a new line.

This change removes $3 and adds it on the page
on its own. Unfortunately, it requires fixing all
the messages. Not sure if it's the proper way to
do this.

Bug: T162844
Change-Id: If17f056c135b2e51747a5a1c8aeaba785cf18ff2

Merge "EditPage: Restore IE 6 compatibility for OOUI-style buttons"

EditPage: Restore IE 6 compatibility for OOUI-style buttons

Follow-up to 97d7de0b84ef65094108fbcf62ffcc1c55d75345.

Bug: T163219
Change-Id: Ifb5145c631ce1a3aa7e51956fab3ec7da8b994d2

EditPage: Restore ability to provide edit summary for OOUI-style input

The 'name' attribute was missing, so the field was not being submitted
with the form.

Change-Id: Ic7238b9bfe9ca254f844fb828cf09ecf44171461

Do an HTMLCacheUpdate of templatelinks on page creation

When a page is created, anything that's trying to transclude that page
needs to have an HTMLCacheUpdate run on it. But that's not currently
happening.

Normal template transclusion happens to get updates only because
transclusion of a nonexistent template is rendered as a link to that
template, which registers a pagelink, and pagelinks *are* being updated
on page creation to cause redlinks to turn blue.

Change-Id: I18b8cfdeedc53f8fdb1e758a6fca1cd23d9574f8

Merge "Use more specific string[] type hint for language variants"

Merge "Avoid overflowing of unbreakable words in interwiki search results"

Set default to 1 recent contributor instead of -1

Bug: T56888
Change-Id: I06bd0e5959d1ff561d71163bd5ce26f28f73981c

Fix EditPage button accessors on PageObject

This will change the accessors to the EditPage in the PageObject to
fit the new DOM when using OOUI.

Bug: T163193
Change-Id: Ic6858d458e2f781c20a101105eb03bd483d6af73

Merge "Use .toc instead of #toc as selector for TOC"

Use .toc instead of #toc as selector for TOC

Since 1bf5a652 multiple TOCs are supported.

In CSS the selector #toc is kept for backward compatibility for uses in the
content.

Change-Id: I07036c0d5782420fc127781a15c675fc7e6fafaf

Merge "RCFilters UI: Don't steal focus when clicking highlight button"

Merge "RCFilters UI: Prevent label from stealing focus on click"

Merge "RCFilters UI: Display 'no results found' if... no results were found!"

Merge "RCFilters UI: Fix scrolling to item on tag click"

RCFilters UI: Display 'no results found' if... no results were found!

The "normal" operation of an OO.ui.MenuSelectWidget is to hide itself
if no results are found in the filtering process. However, we want
the menu to remain visible but display a 'no filters found' message
like it did before the migration to a MenuTagMultiselectWidget.

Bug: T163170
Change-Id: Iee7b07961b90ac514a4aa1f7c56019ecd7b4c3cc

Merge "Split up LoadBalancer::getReaderIndex() and simplify the code a bit"

Split up LoadBalancer::getReaderIndex() and simplify the code a bit

Change-Id: I4e0b5318ad2c987b2a059c4ef2bda3de14486687

RCFilters UI: Fix scrolling to item on tag click

Bug: T163138
Change-Id: I1f3a5b29eb3d9125faf1d4d27c84b8711fd4fdda

RCFilters UI: Don't steal focus when clicking highlight button

Change-Id: I6c41803221fc09ae2676bd09dd734d36f284cc09

RCFilters UI: Fix input direction in RTL again

The style rule for this was in a block for &-search,
which doesn't exist any more.

Change-Id: I1d60cd42df173f7f71085c2a3e5d46bd4bf23ef1

Merge "Move DB errors to Rdbms namespace"

RCFilters UI: Prevent label from stealing focus on click

The filter options' labels are set as "label for" the checkbox
which means that by default they "give" the focus away to the
checkbox when clicked. However, we don't want that behavior so
that we can continue arrow movement through the filter option
items rather than the DOM elements. Also, the checkbox prevents
defaults on click as well as it is.

Change-Id: I9758b0dacd8cd647ec5e3e623edc66e1d7f7420a

Merge "Clarify getLaggedReplicaMode()/laggedReplicaUsed() in LoadBalancer"

Use more specific string[] type hint for language variants

This patch only touches PHPDoc documentation, nothing else.

Change-Id: Ia79d06425a3b8629c171cd68ae435c64dac86f46

Localisation updates from https://translatewiki.net.

Change-Id: I6518eb7aa39e3933714d2df77f881a0929196d71

Merge "RCFilters: Remove 'overflow:hidden' from filter header"

Merge "Support GENDER in the message "sp-contributions-userrights""

Changed "modified" to "edited"

There were two messages that said "This page was last modified etc.".
They are now "This page was last edited". The word "modify" is used
elsewhere in MediaWiki messages for things other than editing a page,
like uploaded files, databases, and configuration, whereas actual
pages are edited and not modified.

Change-Id: I79b68bbec35f7747cd2a27c1a29e154139e3ab76

Localisation updates from https://translatewiki.net.

Change-Id: Id03227c4e4d8963db6cd44ee0bde953dbe2128c6

Merge "EditPage: Make input and button widgets infusable"

Merge "Convert various FormActions to OOUI"

Localisation updates from https://translatewiki.net.

Change-Id: Ib5b863d0778647034f75e65c45e4a10986ec1a3d

EditPage: Make input and button widgets infusable

This will allow gadget developers greater control, e.g. disabling.

Change-Id: I9a21cd2dd3fc7695fc09632b844cc1afd0db0355