Wait for Slave Lag to catch up in wrapOldPasswords.php

Change-Id: Ib02be41c8166a3567d2a1c61352cb383fbb7aee9

Add missing doc param

Change-Id: I674b0bdaab33f858a63afb1dfa04618311086ab5

Gorontalo (gor) to Names.php and RELEASE-NOTES-1.28

Bug: T136083
Change-Id: Ib3e46962de9cc7abc2052cfdf06e95f898e36142

Localisation updates from https://translatewiki.net.

Change-Id: I1de41d1ef19a714081976ddd1d0845952db6a323

Merge "Karelian (krl) to Names.php and RELEASE-NOTES-1.28"

Localisation updates from https://translatewiki.net.

Change-Id: Iaa6a4d7d2d13b59fe16ac650465c3e3b5d9d814b

Karelian (krl) to Names.php and RELEASE-NOTES-1.28

* Added Karelian (krl) to Names.php.
* Added mention to RELEASE-NOTES-1.28
* Created MessagesKrl.php and $fallback = 'fi';

Bug: T137415
Change-Id: Iee2d03d042c8b514ee2575450a37158b26c732a3

Merge "MediaWiki namespace (and talk) back to English in MessagesCdo.php"

Merge "Add !!html/php parser test for HTML comment after template name"

Localisation updates from https://translatewiki.net.

Change-Id: Ibc4d14ebcf6e6cd78fe2cd4b9c8344fd5a181a78

Merge "More robust, cleaned up MessageTest"

Merge "Get StatsdDataFactory from MediaWikiServices"

MediaWiki namespace (and talk) back to English in MessagesCdo.php

Bug: T122428
Change-Id: I740addd0fce913ca74394c663c6858849aa4e2d5

Add !!html/php parser test for HTML comment after template name

I came looking for a test with comments inside transclusions
because of bug T89615. I could only find a test for Parsoid
and not for PHP, so I added it.

Bug: T89615
Change-Id: I4ee3bc75ec86cc6fde6449790b81d287f0ca8345

Get StatsdDataFactory from MediaWikiServices

Change-Id: Ia4a32e7932299168649b6472f503249630fd7c1e

More robust, cleaned up MessageTest

This is a direct follow up to my comments in Ia6ec10d.

Relevant for T146416 are the two messages I added. They explain why these
assertions are duplicated.

assertEquals behaves bogus when used with strings, especially when used
with formatters and parsers, which is the case here. For example,
assertEquals( '9a', 9 ) succeeds. assertEquals is useful when comparing
objects.

Bug: T146416
Change-Id: Ie8df41b3ce6678f3add55bab6701b56b66447a2e

Merge "resourceloader: Use cached Revision::newKnownCurrent for WikiModule"

Merge "tests: stop updating the search engine"

Localisation updates from https://translatewiki.net.

Change-Id: Ide14b88775d42539f39da5854090541c599c2213

Merge "API: Pretty-printed responses should always use HTTP status 200"

Merge "Depedency inject remappedTableName() map via MWLBFactory"

Merge "Validate getExamplesMessages() query strings"

Merge "Fix ApiSandbox"

Merge "CryptRand: remove newlines from logging message"

Merge "Convert CREDITS to list of all known git contributors"

Merge "Convert CREDITS to single list of contributors"

Merge "Deprecate Message::$format (mostly)"

Merge "Add Message test for implicit formatting"

Fix ApiSandbox

I2b8ef74e broke it by removing the isValid() method from
TextInputWidget.

Bug: T150454
Change-Id: Iae48c6e038ce66d294a1bf5f6852e29540ac208c

Validate getExamplesMessages() query strings

Assert that they don't begin with 'api.php?'.

Change-Id: Idf5e8e08863a379a37a427a11936e5f9ce567396
Depends-On: I8a82eecefbb0ba327d8e8bc24ec535bcf40e6429
Depends-On: I476aad09655a2822381a2c61690b4b0ad423151e

CryptRand: remove newlines from logging message

5bd6de6 converted CryptRand to use the logger command but kept the
newlines in the message which is needed for wfDebug() but not with the
logger interface.

Patch also removed a call to wfGetAllCallers() from the first debug
message of generate(), leaving an uneeded debugging message.

Hence we had output such as:
---
[CryptRand] Generating cryptographic random bytes for

[CryptRand] mcrypt_create_iv generated 20 bytes of randomness.

[CryptRand] 0 bytes of randomness leftover in the buffer.

---

Drop the newlines from all debug message.
Drop the lonely debug message from generate().

Result:
---
[CryptRand] mcrypt_create_iv generated 20 bytes of randomness.
[CryptRand] 0 bytes of randomness leftover in the buffer.
---

Change-Id: I572206296fc3a0a498febb18925672a67480eb18

Merge "TransformationalImageHandler: Fix use of __METHOD__ in closure"

TransformationalImageHandler: Fix use of __METHOD__ in closure

Change-Id: Idff6303fac1f494ea59b88fd73245c63f2cf4a76

Do not pass parameter as reference

See comment in https://gerrit.wikimedia.org/r/#/c/318292/7/includes/EditPage.php

Change-Id: I98df04f5445dcc87a96cde157df13786414b8c3a

Merge "Fix parameter name"

Fix parameter name

See comment in https://gerrit.wikimedia.org/r/#/c/318535/5/includes/EditPage.php

Change-Id: I4c3da3872997c20a756405099110c8acc37fa188

Merge "un -> in"

Deprecate Message::$format (mostly)

Message::__toString() used the same formatting mode that the last
explicit transformation used:

    $msg = new Message( 'foo' );
    echo $msg; // escaped
    echo $msg->plain();
    echo $msg; // not escaped

This is not particularly useful and makes code review hard, so let's
get rid of it.

The same behavior with $msg->toString() is left intact (and logged)
for now.

Bug: T146416
Change-Id: Ia9b2a1dcf09d52348b2c6d8299fd849b809f6e74

Merge "Revert "Don't construct SpecialPages twice""

Revert "Don't construct SpecialPages twice"

This causes issues with transcluded special pages.

This reverts commit ba40a63c0ee23daec4886a3e7da62b061639f19c.

Bug: T132545
Change-Id: I14d5300d6b90766532924f9d1e52c4230e2d9772

Depedency inject remappedTableName() map via MWLBFactory

Change-Id: I71d3481fc624f665bac81725026647bbb84ae608

un -> in

Change-Id: If8aecd28bcf7d6bf7c44141f83ac3c47ee4419e4

Merge "Document removal of Database::getSearchEngine()"

Document removal of Database::getSearchEngine()

Bug: T147683
Change-Id: I55dcad7932e77e30f9211421489ec801810223be

Merge "Alphasort lists"

Alphasort lists

Change-Id: Ibcc3541d5bd97eeda3b43a3f9acc2e90644b36b9

Localisation updates from https://translatewiki.net.

Change-Id: I145d58981452d76628b9894faa4b6bee717adf43

Merge "Connect OOjs UI's deprecation warnings to MediaWiki's logging system"

Merge "Update OOjs UI to v0.18.0"

Merge "Extract methods while creating the textbox"

Merge "Extract addition of conflict header explanation"

API: Pretty-printed responses should always use HTTP status 200

To accomplish this, the responsibility for setting the HTTP status code
in the response is moved to ApiFormatBase.

This also adds a line to the pretty-printed response and to ApiSandbox's
output to indicate the status that would be used.

Bug: T150344
Change-Id: Iaf0698ee1b93565d9b02b5a9aa8f93ceb135658b

Fix missing globals

Apperently something went wrong when manual rebasing
If5909f7a16c98504e78388a5ea6c26196f82cd12 and went unnoticed.

Change-Id: Icf8964b0fa0857a44814df3bb11eaf2907e9af56

Extract methods while creating the textbox

This is also mainly done so they can be better utilized when
extending the EditPage class.

Bug: T143823
Change-Id: Id592c9ffa60dc6a904da7ce514e19954125c8ee5

Extract addition of conflict header explanation

This is done so it can be altered when using an alternative EditPage.

Bug: T143823
Change-Id: I31f5f2b7d35806a11925fb00e3272cac47c783ec

Merge "Even more out-factoring to methods in the header"

Merge "Move condition to not show form one level up"

Even more out-factoring to methods in the header

Change-Id: If5909f7a16c98504e78388a5ea6c26196f82cd12

Update OOjs UI to v0.18.0

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

Change-Id: I2b8ef74e090c10a0abc0938e2f3e51ebe3553a6b
Depends-On: I791df74d96b82d53530a365281407704ad1bd122

Add Message test for implicit formatting

Change-Id: Ia6ec10d0d6da8c2b323054642e206c722d17f0b0

Release notes for Id88c8066, Id1ace959, I3c29131e

* Id88c8066fae3f369e8977b4b7488f67071bdeeb7
* Id1ace9599642a36b333c63eaeebab0537466e7bd
* I3c29131ea2041fa7429eae5fab5fae04e39f3da0

Change-Id: Ibd21c5bdd0601ec302cd774488822064ced6b906

Merge "Revert "MediaWiki.php: Redirect non-standard title urls to canonical""

Merge "Revert "Use display name in category page subheadings if provided""

Revert "Use display name in category page subheadings if provided"

This reverts commit 8ccde8984913896d59a3c2b529768cfe74100afd.

For the reason, see T43720#2531092

Bug: T149510
Change-Id: Id1ace9599642a36b333c63eaeebab0537466e7bd

Revert "Move NewPP limit report HTML comments to JS variables" and followups

This change resulted in unreasonable feature loss (human-readable
limit report was gone). Three months and multiple followups later,
the functionality is still not completely restored. Given lack
of response from the original author, I think it is time to revert
and reconsider, especially since the 1.28 release is soon.

A machine-readable limit report would be a very useful feature,
but not at the cost of losing human-readable limit report.

This reverts the following commits:

* Move NewPP limit report HTML comments to JS variables
  b7c4c8717f964d1890d185ec3e6e9481fcb734e4
* Only pretty-print the parser report JS vars
  28adc4d7eef2d7d8e5696a4f9849538a769daa00
* Show wgPageParseReport on page previews too
  1255654ed5a89ed57491bda38f544ed87e3bc601
* Re-add human readable parser limit report
  0051f108b954b52b9981d5d85862ac1f292db80c
* Restore hooks.txt for ParserLimitReportFormat
  4663e7a7371fabb96ed9c909e5b93042c5f08438

Resolved minor merge conflicts in OutputPage (with 80e5b160)
and release notes.

Bug: T110763
Bug: T142210
Change-Id: Id88c8066fae3f369e8977b4b7488f67071bdeeb7

Merge "Update weblinks in comments from HTTP to HTTPS"

Merge "Clarify "while blocked" where something else could be "blocked" too"

Localisation updates from https://translatewiki.net.

Change-Id: Ifac228e77e7eb1d8e74aeac5a13bbe815ba80ca2

Merge "[search] Don't show the create link twice on results page"

Merge "[search] Remove more dead code"

Merge "[search] Remove unused SpecialSearch functions"

Merge "Convert action=markpatrolled fallback interface to HTTP POST"

UC Mini should be blacklisted

The UC Mini browser should be blacklisted. The experience
when we try to run JavaScript is suboptimal - very similar
to how it was for Opera Mini

Bug: T147369
Change-Id: I60b40bcb8c684aa836a4a65ed1deecf06b28e51c

Merge "Lazy load PasswordReset on SpecialPasswordReset"

Merge "Let findHooks.php find UserCreateForm/UserLoginForm"

Merge "Restore hooks.txt for ParserLimitReportFormat"

Merge "Fix docs for OutputPage::addLanguageLinks and  OutputPage::setLanguageLinks"

Merge "Remove ParserTestParser hook from hooks.txt"

Merge "Rebuilt PHPVersionCheck to be an own class"

resourceloader: Use cached Revision::newKnownCurrent for WikiModule

Change-Id: I0c68c649783042a959dc20d9675daae790d82cb1

[search] Remove more dead code

This code in SpecialSearch looks to still be around from the days where
there was a feature that go would send you to directly to an edit page,
rather than showing search results. That feature no longer exists, so
remove the related code.

Change-Id: If5b064b08a6258bfab31ab8f8b62c1d7283c8912

[search] Remove unused SpecialSearch functions

AFAICT SpecialSearch is never extended into anything else. These
functions are protected, an unused within this class. There are a couple
other public functions that look unused, but i suppose a hook consumer
could be using them so leaving in for now.

Change-Id: I214c6144f81dade14a0a8c616743191a295b716d

Add $wgCSPFalsePositiveUrls to release notes

Follow up d84479c4.

Change-Id: I604bdd601b9b1e36964342eed3858336e683d769

[search] Don't show the create link twice on results page

Due to how things flow through the rendering if an offset is provided
and there are no results, SpecialSearch::showCreateLink gets run twice
and the user gets an odd output.

Bug: T149269
Change-Id: Ifed921207f82cc0d1c1cb621a81127486d4dd03e

Localisation updates from https://translatewiki.net.

Change-Id: I0655e33ea35f6b96ea4a404ad858ab668c689733

Merge "LoginSignupSpecialPage: Don't output empty 'uselang'"

Merge "IcuCollation: Do not split $tailoringFirstLetters into verified/not verified"

Replace abandoned color from early palette iteration

Replacing abandoned color `#9aa0a7` from early iteration of the
overhauled color palette as it was too close to `#a2a9b1`.
Follow-up to I94a54b0b0cbc95f95cb8a53a02c199f6f28178ab

Bug: T148267
Change-Id: I50f5b2688683f2048a4f0ae037027b22d29f1962

MWExceptionHandler: Do not use 'exception' for custom log data

When we create an exception-like set of data for logging in
MWExceptionHandler::handleFatalError, add it to the logging context as
a 'fatal_exception' member. The 'exception' value in a log context
should only be populated with a real Exception object.

Bug: T150106
Change-Id: I253943849f19ed5480dbda7bfbc0bf607f69c47d

LoginSignupSpecialPage: Don't output empty 'uselang'

It confuses things because the empty string is accepted as a language,
which falls back to English.

Bug: T146147
Change-Id: Ie56eb95a85a24c5701901abdfb722cebc108bde4

Merge "Remove empty lines at end of functions"

Fix docs for OutputPage::addLanguageLinks and  OutputPage::setLanguageLinks

Per what has been described and fixed in
Ie9c42ac2b4ff143e36d07642f57cca769e8c00e7.

Change-Id: I2af28ae97805f3259ca038942a84b43f89b55150

Merge "Fix docs for LanguageLinks hook"

Update weblinks in comments from HTTP to HTTPS

Use HTTPS instead of HTTP where the HTTP link is a redirect to the HTTPS link.

Also update some defect links.

Change-Id: Ic3a5eac910d098ed5c2a21e9f47c9b6ee06b2643

Merge "Remove redundant "below" from authmanager-create-from-login"

Fix docs for LanguageLinks hook

Also updated on mw.org.
This array never has language keys (at least not
unless other extensions add them) but core will
always just have numerical keys.

Change-Id: Ie9c42ac2b4ff143e36d07642f57cca769e8c00e7

Merge "Rewording BotPasswords message to reflect what "grant" really does"

Rewording BotPasswords message to reflect what "grant" really does

Bug: T150080
Change-Id: Ie2ba9cc0638ab44966cf31f83dbf90036234c0e7

Localisation updates from https://translatewiki.net.

Change-Id: I6dba125eef74a035a20055c88e8578e5f7d2f219

Remove redundant "below" from authmanager-create-from-login

System messages should not hardcode references to specific positions
of the interface elements, especially when such specifications do not
help understanding.

Bug: T137690
Change-Id: I306efc768bf20f6b4aa03cbca42f0cdf084b9a31