Merge "Add documentation for wfClientAcceptsGzip()"

Add documentation for wfClientAcceptsGzip()

Change-Id: I18b9311d71278ca4c843908a11f77270f03cf534

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

Merge "Fix DatabaseSqlite IDEA warnings"

Localisation updates from https://translatewiki.net.

Change-Id: I832a9f7f9ac7c4afd2c358fb506ac3a11f713414

Add class to EditPage text area when editing an old revision

This adds a class 'mw-textarea-oldrev' to the editing area when an old
page revision is edited. It seems that the warning MediaWiki:Editingold
is not sufficient for some users and some css tweaks might be
appreciated, as is already done for protected pages.

Bug: T124226
Change-Id: Ifc53af0f827a5cebfd9cc220b0aa247210e8dcfc

Fix DatabaseSqlite IDEA warnings

Change-Id: Idd52a27c853653e9ba64d765f4f526f5e8a13bcd

Parser functions now format numbers according to page language

Also fixed incorrect return value documentation.

Bug: T62604
Change-Id: I1e16f20ca79436afe17eba711981b2ae43fed9e1

resourceloader: Deprecate mw.Map

This has never been used outside mediawiki.js for things like mw.config,
and mw.messages.

The public API of those objects will remain, but the constructor will
no longer be supported in the future in order to allow these to be
backed by native Map in the future (with a minimal polyfill rather than
a complete one).

* Remove 'mw.Map(valuesObject)'. (follows-up 63f8d7b9e5).
  This is incompatible with native Map and isn't actually used anywhere.

* Deprecate mw.Map#values. Undocumented and considered private.
  A few uses have appeared in the wild so keep a short deprecation period.

* Update the MediaWiki QUnit runner to construct new maps in setup/teardown
  instead of swapping the internal values object directly.
  Also update the runner to update mediawiki.jqueryMsg's parser default since
  it kept a reference to the first mw.messages object.

* Add 'logName' parameter to mw.log.deprecate(). Default logName
  of property key made sense for its original use case (deprecation warnings
  for global variables, which are a property of the window object). But
  for keys of any other object, we'll want to some context for the name.

* Remove redundant test expect numbers. (Follows-up 7c36375, c4c7007d)

* Ensure mw.Map.prototype.constructor points to mw.Map.
  This was broken by 5819c37b3, which accidentally made it point to Object.

Bug: T146432
Change-Id: I7ff0b230608f5ca582fc6752b59c4bb998c78ba2

mediawiki.storage: Catch exceptions on window.localStorage property access

Follows-up 430a0d3984, which created a private property that stored a
reference to the localStorage object outside the get and set methods.

Where previously it was used directly, and as such, covered by
the try/catch around getItem and setItem calls.

Bug: T148998
Change-Id: Ibe75408913d4ffbf1791e1a6470296b29f809185

mw.loader: Fix off-by-one error in splitModuleKey()

Follows-up 45bec76. This caused most localStorage cache entries from
mw.loader.store to be ignored because it'd store tje source code with
a version string that wrongly started with an "@".

As such, on the next page view, when comparing against the version
string from the startup manifest, it would mismatch and eventually
be pruned by mw.loader.store.

Change-Id: If14c80989c0bb28b7b209abe89f83d585c563987

Merge "Swapping defaultbranch for track"

Swapping defaultbranch for track

The former is a maintenance nightmare when branching.

Bug:T146293
Change-Id: Ic892c2f5778153f6e99107fe24dafc2c8797f86d

Merge "Clean up newWANCacheFromParams() getInstance() calls"

Merge "Small DB classes IDEA warning fixes and cleanups"

Merge "Cleanup unused LBFactory $domain arguments"

Cleanup unused LBFactory $domain arguments

Change-Id: I17b409a71e263ff5e0175f6a35fb9c5288f00c11

Merge "Remove redundant LoadBalancerSingle doc block"

Merge "API: Update ApiOptions docs for I45f69997"

Localisation updates from https://translatewiki.net.

Change-Id: I0f0c8c41d403de7df04aa49aebf3c21c9e5d3957

Comments: Added phabricator IDs to bz bug #s

Change-Id: Ia8d4e570e78c62e5c26bda0264fbf1efc74e58a2

Added array_replace_recursive merge strategy

For extension registry, add array_replace_recursive merge strategy,
as some extensions/configuration may prefer that to array_merge_recursive.

In some cases, configuration is merged from multiple extensions,
such as JsonConfig's $wgJsonConfigs configuration: ZeroBanner defines

"JsonZeroConfig": {
"namespace": 480,
"nsName": "Zero",
"isLocal": false,
"cacheKey": "1"
}

and mobile.php overrides it with
$wgJsonConfigs['JsonZeroConfig']['isLocal'] = false;
$wgJsonConfigs['JsonZeroConfig']['remote'] = [
'url' => 'https://zero.wikimedia.org/w/api.php',
'username' => $wmgZeroPortalApiUserName,
'password' => $wmgZeroPortalApiPassword,
];

Having identical value 'isLocal' would be converted into an array
if array_merge_recursive is used, but the replace strategy fixes it.

Change-Id: Ica6ddd0ae76f23e60de9b6235c6e2a3f2754a95d

API: Update ApiOptions docs for I45f69997

Now that we have an alternative multiple-value separator, the
documentation for action=options's change parameter should recommend
that instead of stating that values cannot contain pipes.

Change-Id: Ia0ff2c16be9cab3e1e303a5c7c618455167db31c

Merge "Add a @throws doc line to getGrammarTransformations() in Language.php"

Remove redundant LoadBalancerSingle doc block

Change-Id: Ibbf952bb1c6ded5e47a325de3c2ecdc5e171e4e7

Merge "Make LBFactorySingle call initLoadBalancer() as the others do"

Merge "Improve Status/StatusValue::splitByErrorType type hint"

Small DB classes IDEA warning fixes and cleanups

Change-Id: I65b5f452a364b329bb671540f4115dbc4f40eebd

Force boolean type for $ok in StatusValue::setResult

Change-Id: Ie9e933fba3445872e39d70ab3a8230f204106996

Clean up newWANCacheFromParams() getInstance() calls

Change-Id: I0daa2531708d0a110e8028c4a5641153a23c0a97

Improve Status/StatusValue::splitByErrorType type hint

Change-Id: I6deb76f8906aa86481b84ce1aaebaddf3a52e31d

Localisation updates from https://translatewiki.net.

Change-Id: I6e7f0a9500f0c67789b8dcfb09fef5a381c0928f

Localisation updates from https://translatewiki.net.

Change-Id: I3552487ae6111a7725ae64f4cea51b768a261b66

Add a @throws doc line to getGrammarTransformations() in Language.php

Follow up to
I6b9b29b7017f958d62611671be017f97cee73415

Change-Id: I65b4a7b482238d2df97639f41f05af8566686414

Merge "Use adaptive CDN TTLs for page views"

Remove unreachable code after an exception

Follow up to
I6b9b29b7017f958d62611671be017f97cee73415

Change-Id: Ib3ccbb103097a50f04b798a4ff33eedc64e62aef

Merge "Make grammar data loadable as an RL module and usable in JS"

Localisation updates from https://translatewiki.net.

Change-Id: Iefabb99a1acb00c45a585eabf082c1c2b7a83df3

Make grammar data loadable as an RL module and usable in JS

* Load the data of this variable from a JSON file to the same
  data structure that ResourceLoader uses for digitTransformTable,
  pluralRules, etc.
* Change the JSON structure to ensure the order of the rules.
  Otherwise JavaScript processes the keys in a random order.
* Delete the grammar code from JS and replace it with
  the same logic that is used in PHP for processing the data.

For now this is done only for Russian.

The next step will be to make the PHP and JS
data processing logic reusable.

Bug: T115217
Change-Id: I6b9b29b7017f958d62611671be017f97cee73415

Make LBFactorySingle call initLoadBalancer() as the others do

Change-Id: I0ef09d7f4ffbba1debb43def785f0052139d7d86

Merge "objectcache: avoid using process cache in nested callbacks"

Merge "Show user ID on action=info"

Use adaptive CDN TTLs for page views

This makes frequently changed pages be less likely to be
seen in stale forms if purges are delayed or lost. Pages
that have not been edited for somewhat longer than the
nominal CDN cache TTL will retain the nominal TTL.

Category pages will adapt based on page_touched rather
than the last revision, given the possibility of constaintly
changing membership. With their lesser overall usage,
this is less risky than for Article, and also more useful.

Change-Id: If621aca2fb68e9f87a50c891dac8dc6ec7641f5c

Show user ID on action=info

Change-Id: I43293e319e02dd388c4ee3e952962ad725ab2bab

Merge "OutputPage: Reduce getTitleInfo() calls (improve preloading)"

objectcache: avoid using process cache in nested callbacks

Because the process cache can be lagged by virtue of blind TTL,
the HOLDOFF_TTL might not be enough to account for it, so avoid
using it when already inside a callback.

Also split of the tests from the MediaWiki test class, so this
does not require DB access anymore.

Change-Id: I743a1233a5efc7f036fad140a9ff8a30b32f8f27

OutputPage: Reduce getTitleInfo() calls (improve preloading)

* Remove filter between $moduleStyles and preloadTitleInfo().
  Previously, this was filtering out wiki modules created by
  gadgets and extensions. Still causing a couple of direct queries
  from getTitleInfo().

* Store __METHOD__ in $fname outside getWithSetCallback.
  This way queries are logged as either getTitleInfo or preloadTitleInfo.
  This regressed in 0852a000a5, after which the latter was logged
  as "::{closure}".

Change-Id: I454e43f43f8ad3270b86f4cfdbd68192c305cc07

Merge "Add caching to ResourceLoaderWikiModule::preloadTitleInfo()"

Clarify phpdoc of DatabaseUpdater::addExtensionUpdate

Change-Id: Ibc9951e70b8388f5c23c3e05e31e20bd037beacb

Merge "Relax phpdoc of PermissionError to match actual usage"

Merge "FormSpecialPage: Only add redirectparams for POST forms"

Add caching to ResourceLoaderWikiModule::preloadTitleInfo()

This is one of the top three DB queries showing up in xenon
reverse flamegraph profiling.

It works via a per-wiki check key that is bumped whenever
someone changes a .js or .css page on that wiki.

Change-Id: I73f419558864ba3403b4601a098f6aaf84a3e7c1

Merge "Use a more specific rule for setting cursor:pointer on JS links"

Merge "Fixing numeric sorting for numbers with leading zeros"

Localisation updates from https://translatewiki.net.

Change-Id: Id5d711a76c60c866e751c009d310fc26e3b54ee3

Use a more specific rule for setting cursor:pointer on JS links

This rule may not work in old versions of IE, but neither
does JS.

This avoids cusor:pointer being applied to ContentEditable links
which use cursor:text.

Change-Id: I0d87d67f7d4aa1c1118f3b3613c755535748c799

Fixing numeric sorting for numbers with leading zeros

Bug: T148774
Change-Id: I34aa330645d9d82b6c4e57542e891dd2b36e42ad

add RELEASE notes for 28417328 (CSS3 attr() is banned)

Change-Id: Ie42cb63072520f347393e7dbaaec381c6e106c1e

resourceloader: Make cache-eval in mw.loader.work asynchronous

This is an amended version of reverted commit 482ad8d9fb2e.

The difference is that this change is smaller.

We keep the eval as one large chunk. Not split up in <=50ms chunks and
spread out based on how fast code executes on the device. This means we keep
the issue of an irresponsive UI thread during execution, and trade it for
finishing quicker. This doesn't feel right, but is something we should change
separately, later because it has side-effects (T146510).

Doing it in one chunk is the status quo as we've had it for the last two
years. This commit merely defers the big eval chunk to one tick later so
that higher priority work may happen first (e.g. rendering).

Bug: T142129
Change-Id: I2b31b27554cd29b48d986ed6a6f698a77e59c0e5

Merge "Fix FileBackendStore IDEA warnings due to var reuse"

Merge "JpegMetadataExtractor: Don't fail when garbage bytes are present between JPEG sections"

Localisation updates from https://translatewiki.net.

Change-Id: I8c578090b46c06951b88c6893f97f16b79e85fc9

Special:BlockList: Chain methods of HTMLForm

Much easier to read. Also, changed '' as a value for displayForm
to false.

Change-Id: Iab932d34952a1f7aab9da02d5ddf137edd53ad43

Merge "Add 'noflip' option on RL modules to disable CSSJanus"

JpegMetadataExtractor: Don't fail when garbage bytes are present between JPEG sections

In theory JPEG files are not allowed to contain anything between the sections,
but in practice they sometimes do. It's customary to ignore the garbage data.

Bug: T148606
Change-Id: I98f2609644bcd8bfd7c1679afc6e7af83e228685

Fix FileBackendStore IDEA warnings due to var reuse

Change-Id: I1b667fb7e891053c7898a0c247745b4376061590

Merge "MessageCache: Use checkKeys for large messages"

Merge "Remove dire config-dbsupport-postgres warning"

Merge "ApiPurge: Remove setTriggeringUser call."

ApiPurge: Remove setTriggeringUser call.

This is a revert of 84ffa5d1242. At that point we've been implementing
page-properties-change event for the EventBus extension and thought
adding this call would be a good idea. However, it was discovered
afterwards that there's much more extensions that don't set the
triggering user. Since that call is deprecated and plans are to
remove it - revert the change.

Change-Id: I9891da628f968970247edc37cccd4a666950273b

Merge "Make MessageCache::load() require a language code"

Remove dire config-dbsupport-postgres warning

The other DBs (even oracle and mssql) do not have this. Postgres
support is far better than some of those and tends to break most
when using master (and fixes are backported if bug slip into
releases). This is one of the three decently supported DB types.

Change-Id: I5da74d5f9f0ea055529456eddb47ca0be380cbe4

Merge "Clean up postgres connection handling"

Clean up postgres connection handling

* Remove non-connection magic case when no DB $user
  is given. This was removed from the base class.
* Use PGSQL_CONNECT_FORCE_NEW to let LoadBalancer
  handle connection reuse. This makes it work like
  the mysql classes.
* Make postgres connection error messages actually
  be useful by using the PHP error when possible.
  This makes it clear if the problem is authentication
  or something else and so on.

Change-Id: I3fd76c1e2db8d6008074f5347b201554579b549a

Make MessageCache::load() require a language code

Also make it protected; no outside callers exist.

Change-Id: I9f35d05a5e031d1c536a44b19b108803db068677

Merge "Postgres installation fixes"

Merge "Force User::__toString() return value to be string"

Force User::__toString() return value to be string

Bug: T148486
Change-Id: I19f483b16c99d05f4d1626ee89ba1cfb7a78d52b

Merge "Move FSFile classes to /fsfile"

Merge "Deprecate Database::getProperty()"

Move FSFile classes to /fsfile

Change-Id: I21f3a9ac52b1f953ca09da0705cac868785193e1

FormSpecialPage: Only add redirectparams for POST forms

If the form is submitted via GET, using redirectparams doesn't make
sense, and you end up with redirect params being included in the GET
query string. And then if the form is submitted again, the
redirectparams include the previous redirectparams, and so on.

Change-Id: I9bc930e5dca557571b4658444fea6aec59c5797a

Relax phpdoc of PermissionError to match actual usage

Also fix OutputPage::showPermissionsErrorPage to handle everything
it claims to handle.

Change-Id: I8ed4a6f2f038fe70084ab673300e24af97e935db

Merge "HTMLForm: Add OOUI support for 'multiselect' with 'dropdown' => true"

Add 'noflip' option on RL modules to disable CSSJanus

Should be useful for cases where we pull in an external library
that already manages RTL flipping in its styles, and CSSJanus
does the wrong thing without extra markup.

Example:

  'ext.tmh.video-js' => $baseExtensionResource + [
      'scripts' => 'resources/videojs/video.js',
      'styles' => 'resources/videojs/video-js.css',
      'noflip' => true,
      ...
  ],

Bug: T148572
Bug: T148565
Change-Id: Icbad20d8a6e9a0d354ad159f5816f4fb67cc2775

Merge "Allow BatchRowIterator to also set query options."

Merge "Gallery: Use intrinsic width for gallery to center caption"

Localisation updates from https://translatewiki.net.

Change-Id: Ib8545d516989d75a398f098146c1a93df81360a0

Allow BatchRowIterator to also set query options.

Change-Id: I15927c5b7f1edcfb40edd3cda9257b40d0276073

Merge "MWServices load new ServiceWiringFiles after ExtRegistry load"

Deprecate Database::getProperty()

Change-Id: If0399dded731b139ffbaab405eceeeafcdf9aed9

MWServices load new ServiceWiringFiles after ExtRegistry load

Extensions that include ServiceWiringFiles in their
extension.json file do not currently get loaded.
This is due to the resetGlobalInstance call after
the extension.json files are loaded not reloading
service wiring.

Adding the 'load' param resolves this.

The current sequence of events is as follows:
 - Setup.php
 - ExtensionRegistry::loadFromQueue
 - ExtensionRegistry::exportExtractedData
 - MediawikiServices::resetGlobalInstance

This can be tested using the Cognate extension
(linked in the ticket) which may be the first extension
to use both extension registration while including
service wiring files.

Bug: T148419
Change-Id: Ic42ac8e28c66b79193bcafe332b1ef2ebc29742d

Merge "Fix ExtensionRegistry cache doc that is out of date"

Fix ExtensionRegistry cache doc that is out of date

Change-Id: If019599baec4e8c566362744c2f565ca169b66b6

Merge "registration: Fix ignoring 'merge_strategy' key in manifest_version 2"

registration: Fix ignoring 'merge_strategy' key in manifest_version 2

Bug: T148493
Change-Id: Ibd88c4dc98f36d7d8336691884bbaa9a4864da83

MessageCache: Use checkKeys for large messages

Also make use of the cache set options and use
Revision::newKnownCurrent() to avoid excessive
revision table queries during miss periods.

Bug: T144952
Change-Id: Ic1c649478b0f87420052d8c99b2962920f8b5c96

Merge "Use namespaced ScopedCallback"

Use namespaced ScopedCallback

The un-namespaced \ScopedCallback is deprecated.

Change-Id: Ie014d5a775ead66335a24acac9d339915884d1a4

Merge "Move most of MWCryptHKDF into libs"