Merge "Change .tocnumber to use grey text for section numbers in Table of Contents boxes"

Merge "Refactor rev_delete handling on deletion"

Localisation updates from https://translatewiki.net.

Change-Id: I83ef5fa88f65ec941b87888943273892857d5561

Remove tracking category stuff that accidentally slipped into 61adc1e14

Bug: T149310
Change-Id: I0a3725a72b1467c57280ae1880935dd5fa54ae9e

Refactor rev_delete handling on deletion

Refactoring handling of the rev_delete database field, which is not
needed to select, when the suppress option is set.

Also added a new const, to avoid a comment about that things should be
15 ...

Change-Id: Ia5bcb835172bf6c10e2e872555fc67e91806ebe1

Fix reference in docs of HashConfig::set

Change-Id: I82473e42a746ed4280ea22372d8d17d797070334

Merge "UploadBase: Permit SVG files with broken namespace definition (Inkscape bug)"

UploadBase: Permit SVG files with broken namespace definition (Inkscape bug)

Inkscape mangles namespace definitions created by Adobe Illustrator
(apparently it can't parse custom entities or something, maybe just
in 'xmlns' attributes). These files are still valid SVG, and not
a security issue (although Illustrator probably won't like them),
so it's okay to allow them.

Added tests with some example files.

* buggynamespace-original.svg
  File generated by Illustrator (edited by hand to reduce filesize).
  Based on <https://commons.wikimedia.org/w/?curid=16495597>.

* buggynamespace-okay.svg
  The original file, opened and saved in Inkscape (no other changes).

* buggynamespace-okay2.svg
  The original file, opened and saved in Inkscape twice.

* buggynamespace-bad.svg
  The original file, edited by hand to remove custom entities.
  This is not valid XML and should be rejected (although it's valid
  when parsed as HTML, and some image viewers might display it).

* buggynamespace-evilhtml.svg
  An SVG file using an entity declared namespace for a namespace
  we want to ban. Based on buggynamespace-original.svg.

Bug: T144827
Change-Id: I0eb9766cab86a58d729f10033c64f57d2076d917

Merge "New HTMLForm size filter field; add size filter to Special:Newpages"

Merge "Add short-circuit to DatabasePostgres::schemaExists()"

Merge "Factor out some methods in the header generation"

New HTMLForm size filter field; add size filter to Special:Newpages

The conversion of SpecialNewpages to HTMLForm seems to be half-finished.
It's not using HTMLForm to read in the request query, which means we have
to roll our own logic. Kind of defeats the purpose of using HTMLForm in
the first place.

When ProtectedPages is converted to HTMLForm (T117722), it can use this new
field type.

Bug: T12817
Change-Id: I069609fbb37b18c3df25156779ad7ac7cd5d6813

Add short-circuit to DatabasePostgres::schemaExists()

Change-Id: I5221f7d937be1e87689df5a21fd64e244dbd4c2a

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

Conflicts with https://gerrit.wikimedia.org/r/#/c/318151/

This reverts commit bc2fe61db26ef99f14b17be36476fc20a3bb4e0f.

Change-Id: Ib17efe29d4f78eb3508ffe57c3fa365b64aa1341

Localisation updates from https://translatewiki.net.

Change-Id: I62a31a3a58174c1c4481bcfb51ab94cc17cac7d9

Add a grant for viewing restricted log entries

In core this holds the 'suppressionlog' right. SpamBlacklist and
TitleBlacklist would add their permissions here as well.

Bug: T149235
Change-Id: I8e2304e7b6f2af0c3d21d7d7390e4979b8eaee39

Factor out some methods in the header generation

As preparation for some more minor refactoring there.

Change-Id: I31f585e5c4fcf724d9aaf670a21f4f0af42e98ce

Bump version number to 1.29.0-alpha for 1.29 development cycle

Branch point was dc0f9b3a3a75e80a0c5f09dd76b4df1fcc05080d

The following commits missed the branch point and should probably
be backported because master reports them as 1.28
 - 4290f686c07265d40718fc3358f196de41bbde57
 - 81698d4c1605f491709e636acd5ca0e6857e3821
 - 40da8bf039019a2845405854a2d636be4bd3b98d
 - 95db9833dd8d75ff3dc44d587fdc3ad80332a500
 - 7bd97758f7e1496c7d4a1fa4e7275c9d6c01c5c3

Change-Id: I51562ba357b5533500ef9dd1e29107dd05cc9e1e

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