Merge "Cleaned up JobQueueRedis exceptions"

Merge "registration: Remove unused variable"

Merge "Update context stats in SiteStatsUpdate"

Update context stats in SiteStatsUpdate

In addition to updating the database / memcached, make SiteStatsUpdate also
update the context stats object with the deltas it is processing.

Change-Id: Icc12c07ab4233e1105a402b80d290bdb64df6ea7

registration: Fix types of ResourceModules properties

styles and scripts can both be strings, and that is treated as if that was
the first value in the array.

style can be used as an object, the filename is the key, and value can be an
object that states what media type the stylesheet is for. An example of this
can be seen in the MonoBook skin.

Change-Id: If7d3a220cdccbb1ce096d776592535fe5782a954

registration: Version the extension.json schema

Versioning the extension.json schema will allow us to make breaking changes
to the schema in a non-breaking manner.

Extensiosn and skins will set a 'manifest_version' value, stating which
version of the schema the file is written for. Processor::extractInfo() will
be  passed the version number, and can switch behavior depending upon it.

For backwards-compatability, a version number of 1 is assumed if none is set.
The validateRegistrationFile.php script will emit a warning if this is the
case.

Bug: T99344
Change-Id: I2086a1465ceaeedd1ccc6804fda2c304ad16ffa0

registration: Remove unused variable

It is immediately overwritten. Follows up c573fc5a47204d29314.

Change-Id: I901cfebe088ca0020323db03378fe23117bb3985

Localisation updates from https://translatewiki.net.

Change-Id: I9c9efe2b5b83d4971ef8ebddb7c4ce57006365a2

Merge "Don't run phpcs on node_modules folder"

Warn when duplicate arguments are found

Currently, duplicate arguments result in a categorization but not a
warning, and it's often difficult to find where in the template hierarchy
the problem lies. This causes a warning to be provided containing the
calling page's name, the called template's name, and the parameter's name.

Bug: T85352
Change-Id: I26b9a7ed5a2f246d00a49a5f6effe40b4443a9d0

Don't run phpcs on node_modules folder

This is an artifact created by npm in certain modules. These modules
may contain PHP e.g. grunt-phpdocumentor

Change-Id: Ie46bceb5acbe6141d1e0dc4d3397d04f41d01485

OOUIHTMLForm: Access context after parent constructor is called

This avoids "ContextSource::getContext (OOUIHTMLForm): called and $context
is null. Using RequestContext::getMain() for sanity"

Change-Id: I20c2bb58b3a0e5e259241a3715ee8fdf4b57e148

HTMLForm::factory() doesn't throw MWExceptions

Change-Id: I0104bf1dd9ebb813aa37f92b21515a964ca2a709

Merge "Bump composer-merge-plugin to v1.1.0"

Bump composer-merge-plugin to v1.1.0

Bug: T97560
Change-Id: Idc3ae196f7fc9483f5c474b32d92e30be8325d62

Merge "Revert "Convert mediawiki.toc and mediawiki.user to using mw.cookie""

Revert "Convert mediawiki.toc and mediawiki.user to using mw.cookie"

Caused a meltdown in production.

DO NOT MERGE COOKIE CHANGES WITHOUT AN OK FROM OPS

This reverts commit a4d3d3b4277137b765ceab8752f5562dd27928d3.

Change-Id: I40f0c2141e25ad37af0babfa95421915adce496b

Localisation updates from https://translatewiki.net.

Change-Id: Ibfc0b3b833ef6389f5ca73a0f276da2e58929152

Merge "Log what job types were never inserted"

Merge "Bypass TransformTooBigImageAreaError for ForeignApiFile"

Log what job types were never inserted

I am seeing a constant stream of

Notice: JobQueueGroup::__destruct: 1 buffered job(s) never inserted. in
/srv/mediawiki/php-1.26wmf6/includes/jobqueue/JobQueueGroup.php on line 419

In the production logfiles.  Log what type of jobs these are so we can figure out
how to fix them.

Change-Id: I7beca910d8c4b20aa20bdec4883e399fd94fbcfd

Merge "Do not load mw-ui-icon-ok in icons pack"

Localisation updates from https://translatewiki.net.

Change-Id: I2ec18fd0b189c9fdd0590d89a740e28883091f48

Do not load mw-ui-icon-ok in icons pack

The library should not provide any icons. This is an unused css rule
that is being shipped to all our users.

Change-Id: I6f237ffc1ed7dccbab68666d5c219f05178a1e1b

Merge "Process cache tagUsageStatistics() to improve performance"

Merge "Removed partly obsolete and redundant BagOStuff comment"

Merge "Remove mediawiki.action.history group"

Explicitly define filepage module position

Style modules currently added through addModuleStyles default
to being in the head ("top" position). This is an unhealthy default,
since only critical styles that are needed at pageload should be
in the head. In order to be able to switch the default to "bottom",
existing module positions have to be defined explicitly.

Bug: T97410
Change-Id: Id98b9eb46ef56a43bae99830f7db554c90073407

Bypass TransformTooBigImageAreaError for ForeignApiFile

The check if the image area is smaller than $wgMaxImageArea can be
bypassed for ForeignApiFile, because the transform of the file is done
on another physical server and therefore has it own check for
$wgMaxImageArea.
A possible TransformTooBigImageAreaError from the foreign server will be
shown when needed (Prepended by message 'thumbnail_error_remote').

Bug: T34387
Change-Id: Iccb25c00e132a4fe744840b7b8db7d9af620bce2

Removed partly obsolete and redundant BagOStuff comment

Change-Id: I265912276af8f2bcdba4e49b4cc522df0a39a5db

Fixed regression from 97b2a1dfd with URL specified edit tags

Bug: T100248
Change-Id: I2688356cd5f628dca395d1caaa82b9a5b21c025e

Make the output of UnifiedDiffFormatter match diff -u

GNU 'diff -u' prefixes each line with either a space, a '+', or a '-'.
UnifiedDiffFormatter does the same, but it also adds an additional column of
whitespace between the prefix and the line. GNU diff only does that in
non-unified mode.

Fix this by implementing lines() in UnifiedDiffFormatter, overriding the parent
class implementation.

Bug: T100069
Change-Id: I1bf1b8e6d1d5aceb2c3836548f492f7edebe5a12

Remove mediawiki.action.history group

Per task: "This group was added as a misguided patch for T29418. The proper
solution to the problem reported there is to use the top queue (which didn't
even exist at the time; and still isn't being used for these modules!).  A
separate group is harmful, because it causes an extra RL request for things
that depend on diff styles (like VisualEditor)."

Bug: T90842
Change-Id: Ie008316791558749ee3adca45a941ccfd99914b9

Add ability to load plain CSS files at the bottom

Currently all styles modules added to the page using
addModuleStyles are put into the head, regardless
of their "position" value.

Bug: T97420
Change-Id: Ie4287e17d6f298cc63f42f257b1f67ee36961b77

SVG upload with specific error (warning) message when blocking

This patch is to generate specific error (warning) message when
blocking an svg file.

The checkSvgScriptCallback function has been updated, and it's
return type is changed from boolean to array.

A new variable is added to XmlTypeCheck class that contains the
type of error when svg file is uploaded, which is used to generate
concrete error messages later on.

I have added concrete error messages to i18n/en.json and their description
to qqq.json file. Please review the error messages and their description.

Bug: T85924
Change-Id: I3f687bf5b86ce66b703591b85fd03f073aacff4f

Localisation updates from https://translatewiki.net.

Change-Id: I51170c4c3f422d8d4dc322be978ce0901d96ed1d

Merge "Provide a .mailmap file to make git log/etc. cleaner"

Provide a .mailmap file to make git log/etc. cleaner

Produced via `git shortlog REL1_24..master --no-merges --oneline --stat -sne`,
and then manual pruning and merging.

Change-Id: Ic2dc9c9117e790f4b9e2b745bec46d6306a43a87

Introduce ParserCacheSaveComplete hook.

Rationale: give extensions a way to track which "renderings"
of a page exist in the cache. This is particularly relevant
for multi-lingual wikis that splpit the parser cache by user
language on some pages. In that case, hooking into
ParserAfterParse or LinksUpdateComplete is insufficient to
track all language specific renderings.

Bug: T99511
Change-Id: Iebf526098ca837a7df637c650097119495000c81

Move 1.25 Release Notes to HISTORY

Change-Id: I28d2fee8e287a62a6597fa3ddcec893bd5b7d159

Merge "Don't localize parentheses in version number in parserTests.php"

Merge "registration: Fix 'ResourceModules' in schema"

Merge "add link description to wantedpages-summary"

add link description to wantedpages-summary

https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Wantedpages-summary/en

Change-Id: I0ee6fb312bf57be0a775e68553351b6094738d64

registration: Fix 'ResourceModules' in schema

Change-Id: I6fb833133a6d6bde42cb40eb8fd7e02f7ca9797a

Merge "jquery.tipsy: Improve accessibility slightly"

Merge "Introduce wikipage.editform hook"

Merge "Follow-up a43af3bc: Add Import integration test"

Introduce wikipage.editform hook

* New wikipage.editform hook (Bug T53583)
* Make editpage module emit wikipage.editform
* Make collapsibleFooter setup the .hiddencats .limitreport and
  .templatesUsed elements from the wikipage.editform hook

Bug: T55918
Change-Id: I515f7cdf4a6bbff2da3670946a12dc8027507226

Follow-up a43af3bc: Add Import integration test

Previously failing test until fixed by a43af3bc.

Bug: T89307
Change-Id: I2be12fa7d439ba4ad7e00fdd0f73495322c870a6

Localisation updates from https://translatewiki.net.

Change-Id: Ief31d3f05bd25c74872dcb8c2f61471242a880e0

Merge "Reset Title cache when importing titles."

Reset Title cache when importing titles.

WikiImporter now uses NaiveImportTitleFactory, which in turn uses Title::makeTitleSafe,
bypassing the internal title cache. To avoid (potentially cached) Title objects obtained
via Title::newFromText getting out of sync, WikiImporter now clears the title
cache in addition to clearing the LinkCache.

NOTE: a test for this is provided by I2be12fa7d439b.

Bug: T89307
Change-Id: Ib50c48d4797fc21c62090c0be69e87f7e7d07428

Merge "HISTORY: Note new $wgThumbnailBuckets as of 1.24.0"

HISTORY: Note new $wgThumbnailBuckets as of 1.24.0

Follow-up of 02ca7bff9b100f9def06dad4b28fe6fdc2c26b44

Bug: T78053
Change-Id: I625fee0c4ba85f80396ea396f3fc0019c612ff76

Installer: fix images in CSS

Paths are tricky and installer code is old and ugly. Somebody
should really clean this up. Prep the paths always, not just
on the environment checks page.

Co-Authored-By: Timo Tijhof <krinklemail@gmail.com>
Bug: T100145
Change-Id: I4e55c59c2bed0dc94e6bc3a8c59709ec80822235

Merge "Implement OOUI display format for HTMLForm"

Merge "resourceloader: Rename ResourceLoaderImageModule::ensureStuffLoaded to loadFromDefinition"

Merge "installer: Export wgResourceBasePath in LocalSettingsGenerator"

resourceloader: Rename ResourceLoaderImageModule::ensureStuffLoaded to loadFromDefinition

The name was a result of my having a bad day when I wrote that code.

Change-Id: I6b729764a963563f0820b8063c3d04f6e676c61e

Installer: use proto-relative URLs for license picker

While encouraging HTTPS is nice, you pass styling to the CC picker
and when installing on a non-HTTPS host you end up with mixed
content warnings.

Bug: T100146
Change-Id: Iad706072b1b14b0ce2fb1c1597aae46e8ec15136

Merge "API: Ignore META_BC_SUBELEMENTS elements that aren't actually set"

Merge "On missing php-xml, don't mention Mandrake explicitly in the installer"

installer: Export wgResourceBasePath in LocalSettingsGenerator

wgRightsIcon uses wgResourceBasePath in its value, therefore it
must be exported first. Otherwise $wgResourceBasePath will still
be null (as from DefaultSettings.php) and cause an incorrect
value to be assigned.

Also fix $wgLogo to use wgResourceBasePath instead of wgScriptPath,
to match what Setup.php does already.

Bug: T75031
Change-Id: I6f18895f3caa50871327eec108fc7d70c1077f1e

Merge "Export allowed tags from Sanitizer to mediawiki.jqueryMsg"

Export allowed tags from Sanitizer to mediawiki.jqueryMsg

Specifically, we export only those that are allowed to be paired,
because its parser doesn't handle self-closing tags.

Bug: T66740
Change-Id: I9944f9af915715c57a7d9ce3c62c3e61e54a75ba

Process cache tagUsageStatistics() to improve performance

Change-Id: I4a95c00d84c216ad08f30f0971877dfb728f86ff

On missing php-xml, don't mention Mandrake explicitly in the installer

"Mandrake" would be "Mageia" now, but the package advice may apply (or
not) to other distributions as well as the package exists in multiple.

https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Config-xml-bad/es

Change-Id: I6910e0400f2ad9f97f54cb970165d1146d6bf0a2

XMP: Use structured logging instead of wfDebugLog

Co-Authored-By: Brian Wolff <bawolff+wn@gmail.com>
Change-Id: I486192a718576a4d1e585ffb390e297b14dde087

Merge "XMP: Throw generic exceptions instead of MWException"

XMP: Throw generic exceptions instead of MWException

Change-Id: I8bd81cfccedaa6f474e541b71c93c66150b6f7b3

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

Localisation updates from https://translatewiki.net.

Change-Id: I5b90c8fd056231ded8e22cff284d76a60539fbe3

Merge "Remove unused 'XMPGetInfo' and 'XMPGetResults' hooks"

Remove unused 'XMPGetInfo' and 'XMPGetResults' hooks

These hooks are unused in all extensions in Gerrit. We need to remove
them so we can move these classes into a separate library.

Change-Id: I66406c642168adc703361b75deb95c830c1ddab1

Merge "Special:PageLanguage: Show log extract only when title known"

Merge "Update namespace aliases for Luri (lrc) from translatewiki"

Merge "Remove MWLogger and non-namespaced MWLogger* shim classes"

Remove MWLogger and non-namespaced MWLogger* shim classes

These classes were introduced and deprecated during the 1.25 development
cycle.

Bug: 100098
Change-Id: I3a479d194bdf5137f99a7e27fc71c7b438b1e136

Merge "Add gender in message 'editinguser'"

Fix false Namespaceselector value for "all" on Special:LinkSearch

Added the ability to define the value in HTMLSelectNamespace.

Bug: T98496
Follow-Up: I8503c391a40f1654f8570578a9de9015d86c9845
Change-Id: Ic6a871507a027d28b3aeb1efeb9abbceb5734a9e

Replaced spaces with underscores in NS_ consts

* All $namespaceNames and similar messages
that reference NS_* constants seem to use '_' for spaces,
except a few cases. I suspect its a mistake, thus replacing.

Regexes used:
 ([^'"\n]*['"][^'"\n]*(['"][^'"\n]*['"])*[^'"\n]*[^A-Za-z0-9]NS_)
contains a space in front, replaced with _\1

[^A-Za-z0-9]NS_([^'"\n]*['"][^'"\n]*['"])*[^'"\n]*['"][^'"\n]*
contains a space at the very end, replaced with \1_

Change-Id: Ibbc201678ee91db2b5bf3de597c1598b86558d77

Special:PageLanguage: Show log extract only when title known

When open Special:PageLanguage without a page, the whole pagelang log is
shown and the notice:
Title::newFromText: $text must be a string. This will throw an
InvalidArgumentException in future. [Called from LogPager::limitTitle]
The whole log is useless at this position, so showing it only when
looking at a specific page.

Change-Id: I2a0b7ccc71789506554617e1d12797e72835a12a

Cleaned up JobQueueRedis exceptions

Change-Id: Ic6908a759a66ef578593d50c622aca6cb92fb01f

Cleaned up CAS check in RevDelLogItem::setBits for recentchanges

Change-Id: I6c29653361a0937489c27891739a9dc626c20167

Cleaned up some Job doc types

Change-Id: I205345819bc07575995ab9aff54f40d547bd1d6f

Actually set the $res field via doQuery() in setVisibility

* Otherwise, it is thrown away and redone on a slave

Change-Id: I1252fa5bc98d908cf289f4d93cbdd8d6d33828ea

Merge "resources: Add OOjs UI's Apex theme's files"

resources: Add OOjs UI's Apex theme's files

Currently not used, but there are big plans for them!

Change-Id: Iaa0e82a3ebda4665c5eafac5167c8912e82199d4

Add gender in message 'editinguser'

Bug: T99223
Change-Id: Ie8b2a569f9f213c9d2683907c6d17dc637083359

Merge "Check for dependencies in entry point and Maintenance.php"

Merge "mediawiki.language: Respect $wgTranslateNumerals in convertNumber()"

Check for dependencies in entry point and Maintenance.php

This way, if someone tries to install MediaWiki via
either web installer or install.php maintenance script,
after obtaining from git, they get some useful information
on what to do.

Put the checks alongside the php version check, as
stuff installed via composer (mustache / lightncandy)
is used for NoLocalSettings.php and install.php
attempts to use logging stuff.

Also tried to make PHPVersionError look slightly nicer,
with some <h2> elements and more padding for the <p>
elements.

and centralized this code in one place, as much as possible,
for improved ease of maintenance.

Bug: T90438
Bug: T88951
Change-Id: Iae4eb42c4266dbe9213c5de8a96fccfbeaa9acb0

Merge "Fix edit link for messages in $wgForceUIMsgAsContentMsg"

Merge "Format number for message 'title-invalid-too-long'"

Merge "Add explicit plural for number of bytes"

Implement OOUI display format for HTMLForm

Bug: T85291
Change-Id: I6ffe93c16d6b209a0ab08d714ad8ddaefb6acd52

Add explicit plural for number of bytes

Reported by Purodha at
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Title-invalid-too-long/ksh

Change-Id: I5555826ce3f71741696cbb2ef2521fe1ff7e6253

Merge "SpecialStatsAddExtra: Format column label with msg"

Added $res var docs for RevisionListBase

Change-Id: Ib71b9c662e68b64f1c6ad07c1902ed8ed432f175