Merge "registration: Allow specifying extension dependencies"

Merge "registration: Generalize CoreVersionChecker to VersionChecker"

Merge "Fix PhanTypeInvalidLeftOperand in Language.php"

Merge "Fix undefined $param"

Protect language converter markup in the preprocessor.

This ensures that `{{echo|-{R|foo}-}}` is parsed correctly as
a template invocation with a single argument, not as two separate
arguments split by the `|`.

Bug: T146304
Change-Id: I709d007c70a3fd19264790055042c615999b2f67

Fix undefined $param

Change-Id: I2b6862284b80c27f1f6189a261428324295699ae

Merge "MessageCache invalidation improvements"

Localisation updates from https://translatewiki.net.

Change-Id: Ia6cd0fab7698eda791b80f21ee7b15a259738bd7

Fix two instances of calling function with too few args

At one point SearchIndexFieldDefinition was updated to require the
engine to be passed in, but it seems that update was missed here.

BackupDumper::loadPlugin() requires the second argument, set it to
the empty string to keep current behaviour.

Change-Id: Ifbd8fc4870ff63b2d338f8bb4d251d7a3477b989

Merge "Correct DatabaseSqlite::unionQueries annotation"

Merge "Enable supplying inline interwiki results"

Correct DatabaseSqlite::unionQueries annotation

The annotation was claiming string, but it's very obvious from
the usage the it should be string[].

Change-Id: I99849a338971590ca68dfb175b2458ad5ebc0df2

Merge "Set $linktrail for MessagesKu_latn.php"

Merge "Add CollationFa"

Add CollationFa

Bug: T139110
Change-Id: Ie15a2ee1c22ff4a1d2b721ed137227fe83dd12ea

Set $linktrail for MessagesKu_latn.php

Bug: T153233
Change-Id: I488129a002a92481412667706dbd2c534aa8d45b

Merge "SpecialUserrights: Use session data instead of URL parameter for success"

Merge "Fix linking Special:ChangeContentModel from action=info"

Merge "SpecialRunJobs: Use wfHttpError to generate error messages"

Merge "Add maintenance/view.php for viewing page contents"

SpecialRunJobs: Use wfHttpError to generate error messages

wfHttpError generates HTML so the content fits to the Content-Type.

Bug: T146546
Change-Id: Ia90f22564ecb3807aff3b787242bc74fe1678def

RELEASE-NOTES: Fixup for dfb50ab9f3

Change-Id: Ibe2efcd11c3f97ca343f43168bde278e44537be9

Merge "Back off from job types longer for DB read-only errors"

Merge "Escape return path extra params to php mail()"

Merge "A full word instead of an acronym for the revid message"

Merge "Always show exceptions of type 'error' in LTR"

Make DifferenceEngine's markPatrolledLink and getRevisionHeader public so that hooked functions can use them where needed

The said hooks were added to core in
https://gerrit.wikimedia.org/r/#/c/298026/ . One of the many intended use
cases is https://gerrit.wikimedia.org/r/#/c/326074/ which currently fatals
due to these two being protected, and obviously we don't want to lose
functionality when hooking into one (or more) of the aforementioned hooks.

Change-Id: I260c8b57c0bb2af3a6982bd7142b112a4a023391
Depends-On: I02de9069854532faec4c0c1798a10f862e6dfd7c

A few more hooks in DifferenceEngine

For mucking with the class member variable mNewContent and optionally
allowing the suppression of the "missing revision" message when revision
data is not found for a requested revision.

Originally implemented as the "GetUserMessagesDiffCurrent" hook (yes,
these three separate hooks in three separate places were essentially the
same) by Wikia for their SiteWideMessages extension.

Change-Id: Ie0c175af2af418d4ed3de28c94df918115312da3

Localisation updates from https://translatewiki.net.

Change-Id: I1b8437a0df98dccdd524ef3a0ff8493a98bf59f1

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 73798df0632e10313b82987d0b99e93c73407ca7

Change-Id: Ia0e511311eb05276617cc7bdff72b07347591ca3

Merge "Fix parameter type docs"

Merge "Document type of LogEntry::isPatrollable property"

Merge "mediawiki.special.apisandbox: Avoid bitwise tricks when checking for integers"

registration: Allow specifying extension dependencies

There are some extensoins that depend upon another extension or skin,
usually in different ways:
* A constant that is added in the dependency extension, and the
existence of is checked for. This is problematic because it requires a
specific load order.
* Checking whether a specific class exists. This is problematic because
it is extremely fragile, and breaks whenever the class is renamed.
* Checking ExtensionRegistry::isLoaded(). This is mostly there, but it
only checks at runtime, and doesn't provide any machine readable data.

Furthermore, developers implement each one differently, with very little
standardization.

With this, extensions may now specify what other extensions they depend
on. This is for explicit *hard* dependencies that must be installed.

For example:
"requires": {
"MediaWiki": ">= 1.25.0",
"extensions": {
"FakeExtension": "*"
},
"skins": {
"FakeSkin": "*"
}
}

This would add a minimum requirement on MediaWiki 1.25.0+ (already
implemented), as well as the requirement that the FakeExtension extension
needs to be installed, as well as the FakeSkin skin. A wildcard (*) is
used instead of an explicit version requirement as many extensions do
not actually version themselves, and there is no consistent versioning
scheme yet.

Bug: T117277
Change-Id: If1cccee1a16a867a71bb0285691c400443d8a30a

registration: Generalize CoreVersionChecker to VersionChecker

This allows us to put other requirements more easily into extension
registration, such as skins and/or extensions.

Bug: T117277
Change-Id: I3ec1b28b6af380621585cd61b38e5ebb8be9f9c7

Merge "Fix small typos in includes/lib/lockManager/LockManager.php"

Merge "Fix assigning void return value to a variable"

Merge "API: Expose wfIsBadImage() in prop=imageinfo"

Fix parameter type docs

Changes:
 - uses int instead of number as param and return value type,
 - uses stdClass instead of stdObject
 - fixes ResourceLoaderClientHtml constructor's $target param type:
   it is string|null, not an array (previously misspelled as "aray")
 - changes the type of references to XML parser in XMP lib to resource
   instead of not existing XMLParser

Change-Id: I98c363ebc6658d1f4dcabad97a9a92f3fcd7ea8c

Document type of LogEntry::isPatrollable property

Change-Id: I557b92f25f15f57123af329c79ef1752025aedf3

mediawiki.special.apisandbox: Avoid bitwise tricks when checking for integers

In addition to being more obvious than `n | 0`, `Math.floor( n )` also
works correctly for numbers outside of 32-bit int range.

Change-Id: I254457500e141c0dd4e28c707fb4a00c1b04c272

Merge "mediawiki.page.watch: Avoid dynamic message keys"

mediawiki.page.watch: Avoid dynamic message keys

Restructure the code to avoid constructing message names by adding
strings together. Makes it easier to find the usages.

Also, simplify a check for odd/even namespace numbers.

Change-Id: I0b6794cc07ed2a600499632d240b71898715c5b0

Fix assigning void return value to a variable

Change-Id: I1e675d121938c5b6b7f8165681c645154e3a70d0

Merge "Add externallinks.el_index_60 column and indexes"

Merge "Export: Use BCP 47 language code for attribute xml:lang"

Merge "Message: Fix buggy parameter handling in Message::params()"

Merge "The automatic test in MediaWikiTestCase was marked as risky"

Merge ""Image size limit" text should always read left-to-right"

Fix small typos in includes/lib/lockManager/LockManager.php

Change-Id: Ib3e3c6e01dc1ad4604b1cf6c95467f25444f38a2

Merge "Convert legacy bugs ID to Phabricator tasks ID for Parser class"

Merge "Add script to support per-line @suppress annotations in Phan"

API: Expose wfIsBadImage() in prop=imageinfo

If iiprop=badfile is set, expose the wfIsBadImage() status as a boolean.
This is mainly intended for use by Parsoid, but could be useful for
other API users too. The context title can be set by using the
badfilecontextitle parameter.

Also be a little more paranoid and make sure wfIsBadImage() always
returns a boolean.

Bug: T148343
Change-Id: I3432ead92b8d2a17de5243285f2760fc39b501ae

Apply color palette change also to wikitable cell border

In e7a4daf the color of the wikitable border was changed but not the color of
the cells. This leads to a different border color for empty cells and for
non-empty cells.

Change-Id: Ifb691b7f7c388b068e26ccecb94f822432552466

Merge "Add googleweblight to JS blacklist"

Add googleweblight to JS blacklist

Google runs JS on their proxy servers and outputs a page which may
be broken in creative ways (e.g. clicking anywhere triggers the
fundraising banner). Treat them as a non-JS browser to minimize
bugginess.

Currently broken with CentralNotice banner:
https://googleweblight.com/?lite_url=https://en.wikipedia.org

Images below the fold never load:
https://googleweblight.com/?lite_url=https://en.wikipedia.org/wiki/Barack_Obama

Bug: T152602
Change-Id: Ia277ab51634b770a950616a842bedb29fcbf1f12

Localisation updates from https://translatewiki.net.

Change-Id: I37f811dd07e6704c47cc159715f2e24ddaf50ab0

Fix PhanTypeInvalidLeftOperand in Language.php

Not entirely sure what's going on here. Best guess is phan isn't able
to figure out that array + mixed will result in an array, and then
adding validNamespaces (another array) is ok. Could make things a little
more explicit with array_merge, but this seems to work to remove the
issue without changing the meaning of the code.

Change-Id: I7031ae4e68878ec3198e47c55ab5de4d52a6d922

Add script to support per-line @suppress annotations in Phan

Adds the functionality of per-line @suppress annotations. Phan already
supports per-class or per-method annotations, but does not have any
per-line support due to the PHP AST only returning comments that are
class/property/method level docblocks.

This is a bit of a hack, but get's the job done. Removes the
PhanTypeInvalidLeftOperand issue from blacklist and suppresses it
to demonstrate the supression works as expected.

Change-Id: I5066b3b431fb69175a711ee366e95f31c7c47639

Export: Use BCP 47 language code for attribute xml:lang

The patch changes for example
 https://crh.wikipedia.org/wiki/Mahsus:Export/Ba%C5%9F_Saife
from
 xml:lang="crh-latn"
to
 xml:lang="crh-Latn"

Change-Id: I2fb218fe026c5ffee081fb8aaee7b154a8732bdc

Adjust phan to allow for making progress strengthening analysis

With phan set to only report critical errors it will be quite hard
to cleanup the code base to the point where we are running stronger
analysis. Instead of setting minimimum severity to critical, set
minimum severity to low and suppress all existing issues.

This will allow developers to choose an issue type from the list, fix
it, and update the config.php to report on the issue in the future.

Change-Id: I2771a4ff37f43cca215b939ed7d82f137d72cea6

Merge "Block API: Allow blocking/unblocking by user's ID"

Do not lose message parameters in UploadFromChunks::verifyChunk()

This code is gross, and my changes do not really make it better,
but it works more correctly more often.

Bug: T147720
Change-Id: If9ce05045ada1e3f55e031639e4c4ebc2a216de8

Block API: Allow blocking/unblocking by user's ID

Add feature to block/unblock users by their ID. For this,a new
parameter `userid` is added to block & unblock API request.

Bug: T34496
Change-Id: I084a4e275cd937053c505cd388a365b316990ece

Remove "Generated by PHPUnit" comments in 3 files

Change-Id: I8624f8cd0f658a37384a948076e8f6d68f484ce1

Convert legacy bugs ID to Phabricator tasks ID for Parser class

Side edits to comments:
  - update a @fixme as T10068 has been declined
  - fix spelling

Change-Id: I7f9f191ff68bb56de72563dde957ccf4731267e4

Set $wgUserEmailUseReplyTo = true by default

Mainly to work around restrictive DMARC policies (e.g., Yahoo!).

Bug: T66795
Change-Id: I3b34f6820915cc68d46ff6fd38feacd06efd2b69

Merge "Mark pages on watchlist as visited using JS without reload"

Merge "Move Special:Watchlist auto-reload check from PHP to JS"

Always show exceptions of type 'error' in LTR

Bug: T153027
Change-Id: Iad5b9a01f5b8cdaa2ed94b3ece937acc1f6faa60

Localisation updates from https://translatewiki.net.

Change-Id: I1b174ddfd90298e39e68a7495439e8c3b08feb1c

Mark pages on watchlist as visited using JS without reload

When the "mark pages as visited" is clicked, a dialog appears,
asking for confirmation. On confirmation, an API request is sent
to mark all pages as visited, and all unvisited watchlist entries
are changed to appear visited.

Based on a userscript by User:NQ (from English Wikipedia)
https://en.wikipedia.org/wiki/User:NQ/WatchlistResetConfirm.js

Bug: T150045
Change-Id: I45fb02a1edc1b0331925e9a244a2455f86ad3886

Article::fetchContent() was removed

Bug: T145728
Change-Id: Ie8bc12e391f2983452bf472333ea3cfaa72e4918

Merge "WikiPage::getText() was removed"

WikiPage::getText() was removed

Bug: T145728
Change-Id: I9e5ac149ad1780719b5c8f4e27048168cf33c0b9

Merge "ImageHistoryPseudoPager: Don't ignore limit from URL"

Merge "WikiPage::prepareTextForEdit() was removed"

WikiPage::prepareTextForEdit() was removed

Bug: T145728
Change-Id: If15777c04a52bd398092884fab592cfb0930255b

"Image size limit" text should always read left-to-right

Bug: T144386
Change-Id: I10ab5ed71c114bbfefcc841d8a3d4b280bf1acf2

Merge "installer: Restore PHP 5.3 compatibility in index.php"

Localisation updates from https://translatewiki.net.

Change-Id: I204f2ce7bacf1bcf382dc51b2a574e405220d5d3

installer: Restore PHP 5.3 compatibility in index.php

This file should remain compatible with PHP 5.3, so that we can
actually display the error message about old PHP version, rather than
a syntax error.

Change-Id: If12f79f5d375e925291728aac7c9db18ca226d26

The automatic test in MediaWikiTestCase was marked as risky

MediaWikiTestCase::testMediaWikiTestCaseParentSetupCalled is
marked as risky when strict code coverage is activated in
PHPUnit (--coverage-html ../../docs/code-coverage
--strict-coverage --debug -v).

This patch adds the annotation @coversNothing to disable
code coverage on this test. As an automatically-added test,
it must adapt to a number of different environments, so it
is not feasible to add the exhaustive list of @uses classes.

Bug: T152923
Change-Id: Ic670ff2d89f1c5df500c0d10b153d4efd013d7f4

ImageHistoryPseudoPager: Don't ignore limit from URL

I thought the parent class would handle this for us, but no: our setLimit()
call has been overriding the limit from URL set in parent constructor.

Follow-up to f94d5239b528c406c85e090152805a1306e13ded.

Bug: T152813
Change-Id: I93762021f70613bfde72da1eb737e9fa32d7d97c

Move Special:Watchlist auto-reload check from PHP to JS

This makes the watchlist js load all the time, instead of only
if the auto-reload preference is set, so we can put other things
in the same js module.

Bug: T150045
Change-Id: Ib8acca39593fe3d2369dc3187a9a55413553843d

Localisation updates from https://translatewiki.net.

Change-Id: I73bab410a14742c1c93b79bceb7de73feed5d752

Escape return path extra params to php mail()

PHP only escapes some dangerous shell characters. This is a hardening
measure, as MW's sanitizeEmail routines should also have prevented
evil characters from being in mail addresses in the first place.

Bug: T152717
Change-Id: I3736d612ed40d257ee3dde8e98eb30ccf432670a

Back off from job types longer for DB read-only errors

Such error are likely to persist longer than other random
exceptions. In that case, it is better to avoid burning
through the job retry count.

Change-Id: I6785bd608856f98d21e0b0b05d3899a7081c38e2

Merge "Fix links problem on Special:Search"

Include numerical namespace in slow-parse.log

This makes it easier to sort by namespace rather than trying to parse it
out of the title.

Change-Id: I946cb00548bcb69bd2be98c15a9f1e02e546fa24

Merge "Special:RC 'hidemajor' filter"

Merge "Make refreshImageMetadata not fail completely if it doesn't like a single file"

Localisation updates from https://translatewiki.net.

Change-Id: Ic25484ba66bfe1291c39d7e12ed08fa7d8b633c6

Fix links problem on Special:Search

Fix bug that most links on Special:Search did not have text rendered
Also restore some changes from commit f736c3709a54be33f808e8d12f85707b90763165
where Linker::link() usage was replaced by LinkRenderer

This reverts commit 791d5319bff0be23c3ea474a84d42ed788355688

Change-Id: Id25d1b1acd0dcafc47511b2db3782e5d441f3731

Follow-up e8dd1f43fb169: fix JS error when $wgTranslateNumerals is false

Bug: T152800
Change-Id: I99c341168dada063876a66689d68e0b9adf7f09a

Make refreshImageMetadata not fail completely if it doesn't like a single file

Change-Id: I1170cf91ba6dea08d35457c9859b8c18482d4aa6

Merge "ApiQueryUsers: Add ability to search by user ID"

Merge "Revert "Replace Linker::link() usage with LinkRenderer""

Revert "Replace Linker::link() usage with LinkRenderer"

Most links on Special:Search no longer have links with text rendered.
Please revert this revert and fix the issue before re-merging.

This reverts commit f736c3709a54be33f808e8d12f85707b90763165.

Change-Id: Id92c484adc1d3339f9412bfb60ffe213cc1315ee

Merge "Remove empty lines from PHP and JavaScript comment blocks"