Merge "Remove "dash" case in preprocessToObj"

Merge "Move phpunit @group from file comment to class comment"

Localisation updates from https://translatewiki.net.

Change-Id: Iea4877827a12d5117578b8d1a6f1fefc53e1cdeb

Localisation updates from https://translatewiki.net.

Change-Id: I556a81048fbf024a585f7f586b91762eea568cb3

Move phpunit @group from file comment to class comment

Remove @group from non tests

Change-Id: Iae9ee3bc5f539a9b4ded8374006ab2993234450e

Merge "Fix typo in word compatibility"

Cleanup some comments related to DB replication

Change-Id: Icaaa5144beaedfebb88530480733a1e916c935fb

Remove "dash" case in preprocessToObj

This was introduced in 2877402 and removed in 186a182

Change-Id: Ibfa1ae1597bfc50ae6ea49402c7966ca042f12e5

Fix typo in word compatibility

Change-Id: Ic6ad14f5e156311cee3021cd427561dfcd68633a

Localisation updates from https://translatewiki.net.

Change-Id: Ib13656ed8c5c45facc4166963cd4a7f32ade30bf

Merge "Add ExternalUserNames::getLocal() to get local part of username"

Merge "API: Update prefixsearch/opensearch docs"

Add ExternalUserNames::getLocal() to get local part of username

Bug: T189320
Change-Id: I99908ced218c45ecbb23f390f065170292f153b4

API: Update prefixsearch/opensearch docs

The psnamespace parameter to action=query&list=prefixsearch is ignored
if pssearch has a valid namespace prefix. This should be documented.

Same for the similar parameters to action=opensearch.

Bug: T189139
Change-Id: I4914e5c915e6f8e5c0d5116f034ca671fe45a3f8

Remove useless use

A use declaration for a non-namespaced class in a non-namespaced context
causes a PHP warning.

Bug: T189302
Change-Id: I023e64c8194dd03cc3a1098e2d60c73f99bb02e3

Remove deprecated function wfBaseConvert()

Bug: T189300
Change-Id: I5f05beb105dca8e8079e770df562c004a689c2ed

Merge "Replace wfGetLBFactory"

Merge "purgeExpiredUserrights: Show number of rows purged"

Merge "Fix spelling mistake inside API i18n message"

Fix spelling mistake inside API i18n message

Change-Id: Ief2814e2fdd94b68eb6a52261b3aa4f47009bfc6

Merge "rdbms: small cleanups to session loss handling"

Merge "Update OOUI to v0.25.3"

Typo fix

Bug: T189251
Change-Id: I5e7af2629d566181f3280049b6847c0126850ff2

rdbms: small cleanups to session loss handling

Split the two callback runner calls in handleSessionLoss() into two
separate try/catch loops. Return the first exception, if any.

Also make the $recoverable check for connection loss in Database::query
slightly more readable by checking the positive rather than the negative.

Change-Id: I75935fa69e40450ac3983f0d3451ab3001650b6f

Localisation updates from https://translatewiki.net.

Change-Id: I34c71561c72518697570e7f35bbf3e5ca6ea8e3d

Merge "Job::insert: Hard deprecate (soft-deprecated in 1.21!)"

Drop OpenSearch::getOpenSearchTemplate(), deprecated in 1.25

Change-Id: Ib76b96cf392b7f9fa38d28173dd2cd170e08a881

rdbms: Restore test for Database::setFlag()

Follows-up b4eb1feed05b5, which inadvertendly replaced the setFlag()
test with the clearFlag() test.

Also move the test to the DatabaseTest.php file given it's only
executing and covering base class.

Change-Id: I2f4ed6c4eeba845eb67013e1ab7d2b2bde863119

Merge "Self links should not be marked as legacy css"

Merge "maintainance: Re-order OOUI script to be clearer, rename"

Merge "tables.sql: Correct lie about ct_params being unused"

tables.sql: Correct lie about ct_params being unused

It is used by ContentTranslation to store things like
{"from":"es","to":"en"}

Change-Id: I880ecc74ab46dde2fa00ddb7baea08edeabcf676

Update wikimedia/remex-html to 1.0.3

Depends-On: I537768aa34f0bfa8da3aa401d9a7c4b8ba380508
Change-Id: Ifedb963c4ddf941c46bea9527fd8860b273e2d71

Localisation updates from https://translatewiki.net.

Change-Id: Ifae4500f5cfbeb69cf8a055b7d0274898ea68fee

Remove FakeDatabaseMysqlBase test class

Improve some flag tests that did not seem to test much

Change-Id: Iecfc877645cc66cb696beb2b314a6d149745a530

maintainance: Re-order OOUI script to be clearer, rename

Change-Id: I6fa9b4840a4b01a71aeb618973f4b858fa1a6c9c

Update OOUI to v0.25.3

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

Depends-On: I4b847a3e253def12ba1fa15607e1700a4a2c236a
Change-Id: Id10209ea2e36beaac36a0d3509edaa83442290b4

Merge "Restore index forcing in ContribsPager"

Merge "Introduce IDatabase::buildIntegerCast"

Merge "Introduce IDatabase::buildSubstring"

Article.php: Use BCP 47 language code for HTML attribute lang

Change-Id: If2194fbb98cbab284aba74e127cc3953a5714b9b

Introduce IDatabase::buildIntegerCast

Change-Id: Ib24856d1ebe017ff07ae497972c764b4a3f3c7df

Introduce IDatabase::buildSubstring

Change-Id: I96f3e0c4920d52f63175cb6767c149f20a8a8cde

Merge "Add support for 'hu-formal'"

Localisation updates from https://translatewiki.net.

Change-Id: I8d388605f385e8be40f22459be2cda7105ce6600

Merge "Use `class="free external"` only on unbracketed URLs"

Use `class="free external"` only on unbracketed URLs

The ability for URLs to be marked free even if they use bracketed syntax
but "sorta look free" (aka unbracketed) was added 13 years ago in
2d71cb30802e5cb78a423a1699ad87e23bfe9f4e (r7074).

It seemed like a reasonable idea at the time: make printed output a little
prettier by marking "sorta free" URLs as free.  But this complicates the
semantics of wikitext, and introduces all sorts of strange corner cases,
for example:

  [http://example.com/& http://example.com/&]

isn't marked as free, even though the parser output is:

  <a rel="nofollow" class="external text" href="http://example.com/&amp;">http://example.com/&amp;</a>

This functionality isn't actually needed: if you want the pretty printed
output of an unbracketed URL, then actually use an unbracketed URL.

In recent years we're more concerned with simplifying the semantics of
wikitext and eliminating corner cases, such that the content of our wikis
can be effectively archived.  The "effectively free" URLs are low-hanging
fruit in this quest.

Change-Id: I339e8698786c60c96a37a73443cb9a04362662c4

Merge "Allow querying contributions of interwiki usernames"

Sync up with Parsoid parserTests.txt

This now aligns with Parsoid commit 7d2a92f81ebbc0941e8fba2a136f5929406ea5e6

Change-Id: I0b57b1bd3b0802ce08249dd0bf376b931d8c7698

Merge "Use RemexHtml as the tidy implementation for parser tests"

Localisation updates from https://translatewiki.net.

Change-Id: Ic9ff9f542c9e630bcac1d4037ca3e8faf2dbcb03

Merge "Change Recent Changes "Number of edits..." preference message"

Allow querying contributions of interwiki usernames

Both for Special:Contributions and ApiQueryUserContributions.

Bug: T187079
Change-Id: I4398e473b90172851e875b3f219f7076dceda66c

Self links should not be marked as legacy css

Bug: T181472
Change-Id: Ia19a223a2393d5920ae044c396dc25e80354b3fe

Restore index forcing in ContribsPager

For now at least this will avoid a filesort for some cases. But it might
start misbehaving again when $wgActorTableSchemaMigrationStage is set to
WRITE_BOTH or WRITE_NEW.

Bug: T189026
Change-Id: Idd987181b17b824fdf1094f5c3b1c689b1792eb0

Don't target log_user_text for registered users

Apparently when log_user_text was added no one bothered to populate it
for existing rows. Sigh, #Technical-Debt.

Bug: T188826
Change-Id: Ice1e6454e439a4abdf9594f6e964478dd484bc20

Merge "mediawiki.special.apisandbox: Reorder functions"

Merge "Introduce DatabaseLogEntry::newFromId"

Merge "Add test for WikiPage post-edit stats update"

Merge "rdbms: cleanup some type checks and documentation"

Add test for WikiPage post-edit stats update

Bug: T187585
Change-Id: I734b7f42799a9bfe77a1fb1269bb3d48b3852ffd

Introduce DatabaseLogEntry::newFromId

It's a helper function for when you need just one log entry.

Change-Id: Ic5e9db0def857d9dcecbd06bf081c8c83712c1ea

Merge "resourceloader: Add test for non-empty user module in scripts-only queue"

Merge "Update docs for AllowUserCssPrefs to not mention performance"

Merge "DifferenceEngine: Use CSS classes instead of inline styles"

resourceloader: Add test for non-empty user module in scripts-only queue

The getData() tests did have a test case for a 'user' group module with
isKnownEmpty in the script-only queue, but not yet a non-empty one.

The makeLoad() tests did have both already. Arguably the makeLoad()
tests are higher level and observe its behaviour already (outputting
a script tag means 'loading', outputting nothing means 'ready'),
but adding it to the getData() tests for consistency.

Bug: T188689
Change-Id: I842febf996ba8bb0ea25b5ca3b5ac4503db87376

SECURITY: Fix revdel checks in LogPager

Follow-up to ce881e02e8d6 where the check for performer
restrictions and action restrictions was reversed.

Bug: T188145
Change-Id: I85a44f925212929ac87fb7a7e494023258f2d148

Update docs for AllowUserCssPrefs to not mention performance

* Before 2011, this was implemented as an inline stylesheet in OutputPage.
* In 2011, this was re-implemented through the 'user.cssprefs' module
  (class: ResourceLoaderUserCSSPrefsModule) embedded in OutputPage.
* Between 2013 and 2016, all user-css preferences have either been
  removed or migrarated to not need this module, eventually resulting
  in the removal of that module in 2c688cfb11ea6c.

More details on T127328, but suffice it to say, it has no significant
performance concerns currently.

Change-Id: Ia89c4f91ecb67e3116989f2ea97c7ca69275a2fb

Localisation updates from https://translatewiki.net.

Change-Id: I6aab8000374adfaa44c7f4ec3490969f58419f06

[JobQueueSecondTestQueue] Support read-only mode.

In order to switch non-idempotent jobs without losing the
backlog in redis we should support read-only mode in the queue,
where the messages only gets written into the new queue.

Change-Id: I2e9cb2d9cbbd2d657d042b55d4ea0819d21cdd6f

Merge "Revert "Log the reason why revision->getContent() returns null""

Merge "EditPage: In the list of "templates used", highlight redirect templates"

EditPage: In the list of "templates used", highlight redirect templates

Bug: T188737
Change-Id: I16af53c5048a95fd166f8e79ea26f164c1d3a49d

DifferenceEngine: Use CSS classes instead of inline styles

Change-Id: Iee0e5e7eb73023a6f57c1f462e71cbb0e771d1f2

Merge "StripState testing and cleanup"

Merge "Limit total expansion size in StripState and improve limit handling"

Revert "Log the reason why revision->getContent() returns null"

This reverts commit 8eb00f158422e5e2a9198f9d4318a1570b0f47af.

Change-Id: Id14fd26845349f8aca110ac1bad0598770122f25

Merge "Fix master/replica race condition when creating null revisions"

StripState testing and cleanup

* Added StripState unit tests
* Deprecated unmaintained "half-parsed" serialization experiment
* Renamed some variables for brevity and removed unused "prefix"

Change-Id: I838d7ac7f9a2189e13d39c6939dba5d70e74a6b7

Limit total expansion size in StripState and improve limit handling

* Add a new limit to the parser which limits the size of the output
  generated by StripState. The relevant bug shows exponential blowup in
  output size.
* Remove the $prefix parameter from the StripState constructor. Used by
  no Gerrit-hosted extensions, hard-deprecated since 1.26.
* Convert the existing unstrip recursion depth limit to a normal parser
  limit with limit report row, warning and tracking category. Provide
  the same features in the new limit.
* Add an optional $parser parameter to the StripState constructor so
  that warnings and tracking categories can be added.

Bug: T187833
Change-Id: Ie5f6081177610dc7830de4a0a40705c0c8cb82f1

mediawiki.special.apisandbox: Reorder functions

This avoids forward references and
  /* eslint-disable no-use-before-define */
can removed.

Change-Id: If26a9c275665d20864687cd99d3679b362291fa9

ApiQueryContributors: Use correct variable

Bug: T188813
Change-Id: Ibc705d61d57cfe8867d1bde35781515c25b777c1

rdbms: cleanup some type checks and documentation

This clears up some IDEA warnings for database classes

Change-Id: Ic6f3812b012c1791986d24723b757daf4e151e45

Merge "Drop 'comma' value for wgArticleCountMethod"

Localisation updates from https://translatewiki.net.

Change-Id: I12d8afdf599386fa27fd3f9ff96f77e16ef9a560

purgeExpiredUserrights: Show number of rows purged

Show the number of rows purged when running
maintenance/purgeExpiredUserrights.php

Bug: T176754
Change-Id: I091042baef8982c108b17126cde4998fc9a0cd6d

Fix master/replica race condition when creating null revisions

RevisionStore::newFromNullRevision() carefully reads the latest revision
from the master, but then forgets to instruct emulateMainSlot_1_29() to
do the same, so the lookup query for rev_text_id in the text table is
done against a replica.

This is not usually a problem for null revisions, but it is a problem
when editing a page and then immediately moving it: this causes a null
revision to be created based on a freshly created revision, which
doesn't yet exist on the replica.

Bug: T184670
Change-Id: Ied4db5ee770716ff3edb918d2be722934c79d0c7

Use RemexHtml as the tidy implementation for parser tests

* RemexHtml is the future of "tidy" in MediaWiki,
  so run our parser tests using it.

* This is a necessary step before we can make it
  the default in MediaWiki (T185753).

* Cleaned up a bunch of tests:
  (a) where html/php+tidy and html/parsoid match up,
      retained a html+tidy section and removed the others.
  (b) where html/php and html/php+tidy match up,
      retained the html/php section and removed the
      html/php+tidy section.

* Annotating tests with explanations where Parsoid & Remex
  output differ. This is usually because of two reasons:
  (a) Parsoid has Tidy-emulation code in some cases (which
      we can consider stripping away separately).
  (b) Parsoid does a bunch of cleanup on the DOM (which was
      probably done to emulate Tidy output, but which could
      probably be retained). Since Parsoid (in some form)
      will be default parser in the future, no reason to try
      to port this cleanup (in broken markup scenarios) into
      Remex.

* Left a bunch of FIXMEs for later followup.

Unrelated cleanup:
* Renamed a few tests since the functionality in Parsoid
  was fixed up. There is no more "implicit <td>" support.
  Those all now lead to fostered content.
* Fixed some clearly broken output in html/parsoid sections
  for some tests.

Co-Authored-by: Kunal Mehta <legoktm@member.fsf.org>
Co-Authored-by: Subramanya Sastry <ssastry@wikimedia.org>
Bug: T188167
Depends-On: I646dbabb3c2ed28c1ea72c5bd8f7f92d03f57c75
Change-Id: Ic7c34d57a300dbd36a37f03fbfe33391b2950b44

Localisation updates from https://translatewiki.net.

Change-Id: I29741ea3f2e69c74456584857b144f2cdb7465f9

Merge "Introduce hasSlot in RevisionRecord and RevisionSlots."

[MCR] NameTableStore

General purpose cached store for things like:
 - content_models (id,name)
 - slot_roles (id,name)
And in the future possibly namespaces & content_formats
as mentioned at:
https://www.mediawiki.org/wiki/Multi-Content_Revisions/Schema_Migration#Name_tables

Bug: T188518
Change-Id: Ia550ef7fe30af25ac3fee5ac8a89d032544563bf

Merge "rdbms: make LoadBalancer::getConnection() ignore CONN_TRX_AUTO when unusable"

Merge "rdbms: avoid strange uses of empty()"

Merge "rdbms: add missing hint check DatabaseMysqlBase::isInsertSelectSafe"

Merge "Ensure abort link parsing on xmlish tag in link title position"

rdbms: make LoadBalancer::getConnection() ignore CONN_TRX_AUTO when unusable

Change-Id: I1fd13171c3cfbe071e8e398d561281188d998767

Merge "Fix undefined $ok in DatabaseUpdater::migrateComments"

Fix undefined $ok in DatabaseUpdater::migrateComments

Change-Id: I608a693e688039526975ef5d3fe4c40c7679ed7b

Merge "User: Remove ::setCookie/etc., deprecated in 1.27"

Merge "EditPage: Remove methods from OOUI transition, deprecated in 1.30"