Merge "Use LoggerInterface::class in MediaWikiTestCaseTest"

Merge "Add @since for Message::newFromSpecifier"

Merge "Make sure OtherBlockLogLink's second parameter is a string"

Add @since for Message::newFromSpecifier

Adds doctag missing from I2e6195b.

Change-Id: I3a0918c48b49a85498f856896575d6c69e5547e1

Merge "Point skins/extension READMEs to Phabricator"

Use LoggerInterface::class in MediaWikiTestCaseTest

Change-Id: Ic9c24b813786bc4dc2d5aa48a17a958eef0a9ddb

Merge "ApiOptions: set form field parent earlier"

If $line is empty in EnhancedChangesList, don't try to access parts of array

In getLineData, if the EnhancedChangesListModifyLineData hook returns false,
then $line is an empty array and recentChangesFlagsRaw is not set.

If $line is empty then presumably we don't want to render it and
can skip the line here.

Bug: T133296
Change-Id: I92fded07274a06a0dd2929b97815bfe56f1847ea

ApiOptions: set form field parent earlier

Bug: T134351
Change-Id: I44b9bbd1663c876cf0c6160f10badfd8f380656b

Fix HTMLFormField calling Message::setContext with null

This is a hotfix. If you think it's better to revert I2e6195b instead
please do so.

Bug: T134351
Change-Id: Ifcc832a731b18933bdf6edfd6eb7a5cd6046c3ba

Update OOjs UI to v0.17.1

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

Change-Id: I6175a5d788f71f5d07efeabe54aca6335fce4766

Follow-up f9b4bbff: Fix typo'ed variable name

Change-Id: Ied52bbaa8d9b8e15470a99cd45ebef0c43c5dd69

Merge "registration: Allow string value for Hooks"

Merge "Correct error message for missing permission to get patrol information"

registration: Allow string value for Hooks

Instead of forcing an object even for single-hook-listeners, allow
string values, too (one hook listener for one hook, only). Also:
use it as default for the conversion script, if only one listener
is added to a hook (which is usually the case). This leads into a
much cleaner output of the Hooks section of extension.json.

Bug: T133628
Change-Id: Ie9e54f0931c41706eb8d82d00256698992ec41cc

Merge "Localisation updates from https://translatewiki.net."

Localisation updates from https://translatewiki.net.

Change-Id: Ibee58256489a534f8adf5cd4f3ffc14c85ea197a

Fix Undefined variable: namespaces in includes/search/SearchEngineConfig.php on line 109

Bug: T134305
Change-Id: I220886e12a6d083ac34a8a75bc77871e89dbf747

Merge "Support hash fragments in wfAppendQuery()"

Merge "SessionManager: Add provision for encrypting session data"

Merge "Fix HTMLForm noData logic in trySubmit"

Check if user is blocked during upload process

Further down, this was already somewhat being checked.
On L112, `verifyTitlePermissions` is called, which will fail
if the user is blocked. However:

* This was not being checked during stashed uploads
* Block just "happens" to be part of that check: the intent is
  actually to verify the file title. The error is treated as
  recoverable (the title can be changed), but it isn't (the
  user can't unblock himself in this process)

Bug: T111228
Change-Id: I9cbf250a0b92c3daa3a0843f2257cc049abd3923

Merge "Clean up PoolWorkArticleView type hints and fix IDEA errors"

Merge "Document mInterwikis in LinksUpdate"

Clean up PoolWorkArticleView type hints and fix IDEA errors

All callers pass a WikiPage here already.

Change-Id: I6a17bf52fb2547729c6a1fa40704f1c9efe28b12

Support hash fragments in wfAppendQuery()

Change-Id: Icb99d5479836fea25a47451b5a758dd71f642f71

Merge "ParserCache: Don't try to save to nothing if disabled"

ParserCache: Don't try to save to nothing if disabled

If the ParserCache is disabled via $wgParserCacheType = CACHE_NONE,
don't bother trying to save anything because it won't do anything.

This also removes the misleading "Saved in parser cache" HTML comment
from the rendered output since it isn't being saved at all.

Change-Id: I5603d2b2866f0079cdcd837b1f1a4efc00bc8ed9

Document mInterwikis in LinksUpdate

Change-Id: If10840c5df849455027b5a497097e583fb982a65

Fix HTMLForm noData logic in trySubmit

This was missed in If4e0dfb and causes missing array key warnings
for fields using skipLoadData.

Change-Id: Ib52ee2bc9af278f03b48730acc1edb30f5ff1f88

Correct error message for missing permission to get patrol information

Patrolmarks permission is enough to request patrol information, as it's coded
in useRCPatrol() and useNPPatrol().

Change-Id: I4115bdbbcaf6764d71784efe2d8ac63cc77ce770

Merge "Quote column name aliases"

Merge "Update Microsoft SQL Server schema"

Merge "Pass through errors creating the main page to the installer."

Quote column name aliases

Sometimes, a column name alias is chosen that is a reserved word or is
otherwise invalid (such as an alias named "user" on postgres or mssql).
Let's unconditionally quote the alias so that no matter what the user
specifies, it will not cause the query to error out.

Bug: T118973
Change-Id: If3ea7324980355e78afcdb3bf2596d23bb96412d

Merge "Quick-fail refreshLinksJob if the triggering revision isn't the latest"

Merge "Unify HTMLForm message handling"

Merge "Make refreshLinksJob explicitly check the cache rev ID"

Quick-fail refreshLinksJob if the triggering revision isn't the latest

If we already know that the triggeringRevisionId is outdated, fail early
instead of doing all the work of re-parsing that old revision and
preparing all the updates only to fail later at the lockAndGetLatest()
call.

Change-Id: Ic70c659899d5d47e74fa17c88ed26b436732ca8a

Unify HTMLForm message handling

Improves Ida647973a which unified message handling for form fields
but did not make the functionality available to HTMLForm itself.

Change-Id: I2e6195ba13afbd8b993acb47409fab1be91c547e

Make refreshLinksJob explicitly check the cache rev ID

This is needed if the $useOutdated behavior of ParserCache
is modified per Ibd111bed203dd.

Bug: T133659
Change-Id: I70806dffba8af255d7cdad7663132b58479f63e3

Merge "Log when resyncFiles() fails in FileBackendMultiWrite"

Merge "Allow specifying targets for ResourceLoaderWikiModule"

Allow specifying targets for ResourceLoaderWikiModule

Change-Id: Ic8f29aecfbd1e6bfaef44405190043055ce6215a

Update Microsoft SQL Server schema

The MSSQL schema is now brought in-line with the MySQL schema.
Additionally, various issues that prevented successful installation or
updates via MSSQL were fixed, notably with respect to creating bits of
the database should other bits already exist as well as issues with
previous updater patches not working correctly.

Additional MSSQL bugfixes will come in separate patches, as they are less
related to the schema/install/upgrade process.

Change-Id: If3eea625499d3cb14abba40f528208173067a53a

Merge "Set explicit direction to ApiHelp headers with module names"

Merge "Give redirects a sort index in title widget"

Merge "jquery.makeCollapsible: Simplify code"

Merge "EditPage: Style anonymous editor preview warning as a warning box"

Merge "Removing unicode-bidi: -webkit-isolate"

jquery.makeCollapsible: Simplify code

Instead of using if checks, use a piping system (like in the first function).

Change-Id: Ib96f9db9c56965c336ac85d9bdcf21bdbb4e9cce

Merge "In ParserCache, respect $useOutdated"

EditPage: Style anonymous editor preview warning as a warning box

Change-Id: Idd4c50dd9539b95350818a5a6eaebbaf085105ba
Follow-Up: I22c51a46a249247abccda709d74356e8e31a19a8
Bug: T127403

Localisation updates from https://translatewiki.net.

Change-Id: Ife995ee37ad1b23b10b802d7f765750892558609

Removing unicode-bidi: -webkit-isolate

Chrome supports non-prefixed "isolate" since v48,
and apparently the prefixed version causes issues
in old versions of Safari, so it's time to remove it.

Bug: T134067
Change-Id: Ic30d144e5b1aad3159e61fda31a7c31db4029b11

Merge "Style warnings from live preview on Special:Upload"

Give redirects a sort index in title widget

Otherwise the sort compare function returns NaN
and the order is completely broken.

Bug: T134020
Change-Id: Ie3bbc153e6941d35cfab971690c8934123185d48

Fix and standardize Doxygen tags

* Use "@param datatype $paramname description" format

* String → string, Integer → int etc.

* @return $string → @return string

Change-Id: I860d222382cb4c5699d313b0600bd22503c8c385

Merge "Add class to div wrapper around category lists"

Avoid lock error exceptions during upgradeRow() contention

Bug: T132921
Change-Id: I229031c3d4ae5b700fcc4d4dd3f5208a853823dc

Add class to div wrapper around category lists

Allows VE to target non-editable content to keep on the page.

Change-Id: If687b84d9ad33a0315a8ba4260bc3532691f05ed

Localisation updates from https://translatewiki.net.

Change-Id: I75205fc07e65c44bd543a2fa890c6cca94a21b8e

PrefixSearchTest, SearchEnginePrefixTest: ignore ext. special pages

It was attempting to do this before, but it didn't handle special pages
registered by hook, and was not clearing the list properly (so it was
too late when wgSpecialPages was cleared).

This would cause the test to fail if certain extensions were
installed (e.g. AbuseFilter).

Change-Id: I9affac5ad00e97326ebf9961bbf709ac22aef498

Merge "Avoid master queries in getAutoDeleteReason()"

Point skins/extension READMEs to Phabricator

These are actually far more useful listings than the mediawiki
group on Gitblit, which is everything MediaWiki and provides
no actual listing beyond "recently active"

Change-Id: I741cb53ca417bddfa176868fb78ce67b1475fc70

Log when resyncFiles() fails in FileBackendMultiWrite

This distinguishes cases that recover from sync errors
from those that do not.

Change-Id: Ibcdea18041e7344077c8308359a1a6087139f2e0

Merge "Remove vague GPL headers from browser tests with dead links"

Make sure OtherBlockLogLink's second parameter is a string

Bug: T132935
Change-Id: I0583d166d8bfc50e6d10a0ac46c18aeadefad068

oojs/oojs-ui: Swap to using Phabricator for the links to release notes

Change-Id: I6bc1caa02673615f6fab539cec0c8de36958dd63

Remove vague GPL headers from browser tests with dead links

They don't actually state what license they're under, they just link
to a license file that no longer exists. Prior to deletion, it was:

https://phabricator.wikimedia.org/diffusion/GQAB/browse/master/LICENSE;0135f0c57627db28bdf451b5b8d21a1a60ec0b06

That's GPL! It's kind of weird to have a non-standard GPL header like this
so just remove them. Rest of the browser test files don't contain any sort
of header at all

Change-Id: I7b49b806588f029fc6017915a19994a0d52b7177

SessionManager: Add provision for encrypting session data

This follows the model Chris Steipp implemented for OATHAuth.

At the moment, this avoids the need to require a crypto PHP extension by
adding a configuration variable to enable plaintext storage. Someday
when there's time for the necessary code review, we should probably
import a pure-PHP implementation of AES to fall back to when the crypto
extensions are unavailable.

Change-Id: Ie9cae1526d3b8bf3f517f3226ddd888893f65656

Set explicit direction to ApiHelp headers with module names

Bug: T112364
Change-Id: I28e8e0cbbeb0ff0a41a06cb28fbda26d72725e59

Merge "Add 'api-error-ratelimited' message for UploadWizard"

Assert jobs called from WIS::resetNotificationTimestamp are correct

This also adds tests for some previously untested paths.

Change-Id: Id95144c0bad90cfc123a4444a03aad91531d2332

Fix timestamp check in ApiStashEdit::checkCache

* The wrong time method was used, and it also was not set.
* Bumped the threshold a bit while at it, which was basically
  0-5 minutes before.

Bug: T133332
Change-Id: Ide3e66f551aa6e50410c562e5c917141d59b7f64

Merge "Remove unused imports from MediaWikiServices"

Merge "Avoid DBPerformance log warnings in saveNamespaces()"

Remove unused imports from MediaWikiServices

Change-Id: Ib9935e69b3e7c30a90b3f96e56e31b93b81f1f11

Avoid DBPerformance log warnings in saveNamespaces()

Bug: T92357
Change-Id: Ia1165d8da55cf506ff101987e0b0326fc1b6719a

Avoid master queries in getAutoDeleteReason()

This largely used the slaves anyway, but not for certain bits.

Bug: T92357
Change-Id: I909a14b25a959ee28960aa7aa3d799c5f10988d1

Merge "Use typehinted methods for search stuff in ServiceWiring"

Localisation updates from https://translatewiki.net.

Change-Id: I36e4ef817d0736a3f44dfa94df5c4b0e26ae6a34

Use typehinted methods for search stuff in ServiceWiring

Change-Id: Ibc0a218e40cc24daaf310d028d71dc27e510a140

Merge "Move wgContLang from config to injectable"

Merge "Enable 4:2:0 chroma subsampling for JPEG thumbnails"

Enable 4:2:0 chroma subsampling for JPEG thumbnails

* Add $wgJpegPixelFormat, default to 'yuv420'
* Implemented for ImageMagick via CLI and extension
* Currently ignored for other scaler backends
* Added test case to run when using ImageMagick

4:2:0 subsampling can save an average of 17% bandwidth
over 4:4:4 subsampling, at the cost of some artifacting
at sharp red or blue edges. This is usually not noticeable
in photographic images.

To restore the previous behavior, set to false:

  $wgJpegPixelFormat = false;

which will maintain the original file's pixel subsampling
settings in the thumbnail.

Can set explicitly to one of:

  'yuv444' - never subsample
  'yuv422' - subsample 2x horizontally, not vert
  'yuv420' - subsample 2x in both dimensions

Bug: T129128
Change-Id: Ib9cb36c3a7e6a69d66c11150ef4a1d02dbac2df5

LinkCache: Clarify in doc comments what format $title is

Sometimes $title is a Title object, sometimes it is the prefixed DB key
as a string.

Change-Id: I0e574f35f0c3193bc933e77250c42765626d3cbf

Add 'api-error-ratelimited' message for UploadWizard

One day, I'm going to move the UploadWizard-specific messages there.

Bug: T132920
Change-Id: Ieb218c2a2d7bf7a703369772018a246bcf639f0b

LinkCache: Remove unncessary & from singleton() function

Change-Id: I9e9e530a59d7b9062d0214ba82146b313e6b10fd

Title: Update conversion functions for LinkTarget interwiki support

Title::newFromLinkTarget() will now take the interwiki component from
the LinkTarget object, and Title::getTitleValue() will pass on the
interwiki part if there is one.

Follows-up 9b1f8b4ca331e45.

Change-Id: If1fecc9d71e58c476d6cebe6164cdc95f183048d

Localisation updates from https://translatewiki.net.

Change-Id: I8e0a4883bd78e81a72bfafb8b7813dc45799e0f3

Merge "Enforce calling HTMLForm::prepareForm before displayForm"

Style warnings from live preview on Special:Upload

The live preview about warnings on Special:Upload does not style
warnings added to the form. Added styles to the container to style the
warnings.
Also reuse the style to give the warning box on the non-js from the same
style. The visual effect here is an expand box to the full width.

Change-Id: I36e0bf0332eac476002569f46ab116f271f3c468

Enforce calling HTMLForm::prepareForm before displayForm

Bug: T133163
Change-Id: Idd5d117cb0dd65c195019dcd321cd4bf9024b426

Move wgContLang from config to injectable

Change-Id: Iffdc39f2de7d38ee9ef882bb796e8969e95e75c6

Merge "Add interwiki support to LinkTarget and TitleValue"

Merge "Add missing imports to MediaWikiServices"

Add a type hint to RevDelList::setVisibility

Docblock mentions an array already. All usages in core I've managed
to track also pass in arrays.

Change-Id: I830f446a2b63d09c7aa81205d66ac56d8bdab733

Add getSkinFactory to MediaWikiServices

Change-Id: Ife79aca542e0e85405976de7be7acebfa3359569

Add missing imports to MediaWikiServices

Change-Id: I8d4da075e18a87c5a5084f874c2cdce9eb08237a