Merge "Add a bunch of @since tags to Linker"

Merge "Add @since tag to SpecialPage::msg"

Merge "Use LoggerInterface::class in MediaWikiTestCaseTest"

Remove upgradeRow() call from ?action=purge

This is overkill and can cause excess lock contention
on larger files. Purges are typically used to fix
thumbnails or CDN, not metadata. Scripts can always
be run to fix any real breakage.

Bug: T132921
Change-Id: Ieecf1c015735b7219ca51c87951b59c453cf588f

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

Allow reset of global services (redux).

(This is part of I6ec374ac9 wich was a re-submit of Ie98bf5af5
which got reverted by Ide7ab563)

This change provides a mechanism to reset global service instances
in an orderly manner. There are three use cases for this:

* the installation process
* integration tests (which most of the existing phpunit tests are)

In contrast to I6ec374ac9, this change does not cause singeltons
of legacy services to be reset. It is assumed that legacy services
use global state to access services and configuration, so any
change in confuguration would affect them immediately.

NOTE: the original I6ec374ac9 would cause session information to
get lost if the user session was creatsed before initialization
was complete. This was apparently triggered by the MobileFrontend
extension under some circumstances. Check with Addshore and Catrope.

Change-Id: Ie06782ffb96e675c0aa55dc26fb8f22037e8517d

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

Do not show useless form at Special:ChangeContentModel

If the choosen title cannot be converted to any of the available
models, show an error page instead of an empty and confusing dropdown
field asking for the new content model.

Change-Id: Idc7372a8870f98aa8148592385f3a837f16886f0

ApiOptions: set form field parent earlier

Bug: T134351
Change-Id: I44b9bbd1663c876cf0c6160f10badfd8f380656b

Add missing 'public' keywords to some more Language methods

All of them are already being used outside the class:
* getMonthAbbreviation
* getMonthAbbreviationsArray
* getWeekdayName
* sprintfDate
* userAdjust
* date
* time
* timeanddate
* getMessage
* iconv
* ucfirst
* uc

Change-Id: I63ec93858cebc02cdf3b9b042eddf4ef620cc110

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

Add a bunch of @since tags to Linker

Change-Id: I8bab046c33f370d63a29020154e5691b47881dec

Add @since tag to SpecialPage::msg

Introduced in 5f6862c89e5832c117613cf6232f08ab57436fdb

Change-Id: Ie36b37c5c28b67a090cbe69a0d8b37f830104a3f

Print styles: Wrap CSS-generated URLs

Chrome does not actually break CSS-generated URL content with word-wrap, which
causes some infoboxes to expand to the entire page.

This patch adds the word-break CSS property, which allows Chrome to break
those URLs at any position. This fixes the rendering of infoboxes on pages
like https://en.wikipedia.org/wiki/San_Francisco.

With basically all PDF generators (including browsers) including clickable
links & many PDFs consumed on-screen it might be worth considering making
print URLs optional at some point, and defaulting to "screen optimized" PDFs.

Change-Id: I8d03032a553643198da6a1dd8b78201a9255fcbb

Print CSS: Avoid page breaks inside <figure>

Change-Id: Iee0932286135ea25a7f47fb2e6ddb6e6d669fa7a

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

Convert Special:Lockdb and Special:Unlockdb to OOUI

Bug: T134308
Change-Id: Ie5bdba62fa8e39f44522c39cc6b6d3649aec8141

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

Bug: T134305
Change-Id: I220886e12a6d083ac34a8a75bc77871e89dbf747

Merge "Support hash fragments in wfAppendQuery()"

Upgrade mismatched begin() warnings to exceptions

No WMF logs for these events in the last 7 days.

Change-Id: I79f2f2c8117ffa887a550a793a9e490c2a26f721

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

Tweak RefreshLinksJob cache logic

* Make this actually use the cache beyond edge cases
  by making the page_touched check less strict. The
  final check on the cache timestamp is good enough.
* Log metrics to statsd to give visibility.

Change-Id: I14c14846a7b68d079e1a29c6d50e354a3c1926d6

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"

mediawiki.api: Make the result promise of #postWithToken abortable

Calling .abort() will abort whichever request is ongoing at the moment.

Change-Id: I76acf0f7f270c5d2bd651595a8b9407d37fbb095

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

Fix executeTiming statsd metrics

Nothing was being sent out before.

Change-Id: I5d3c681337fc49bd3026781c1bcddf0f2d8e6b8a

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

Avoid spamming the duplicate key fetch log in BagOStuff::merge()

Change-Id: I68b10208f222f81f173f6e282c9ecf0bfcdde1fc

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"