Merge "Add List-Unsubscribe header to emails"

Merge "mediawiki.ui: checkbox: Render "on top of" the label"

registration: Remove "additionalitems" property from array (authors)

It doesn't make sense for arrays which can't have items to begin with.

Follows up bfe4ddd8108efe

Change-Id: I851b70782fa8210802fe576fee771ddf1bdd7fec

Merge "composer.json: Downgrade PHPUnit to 3.7.37 (temporarily)"

Merge "Follow-up 727454f: Add mediawiki.widgets to jsduck"

Merge "Add help link for 8 more special pages"

Follow-up 727454f: Add mediawiki.widgets to jsduck

Change-Id: I1715c5bbca8735de8c391dee4e2cec8269414acf

Merge "Commit all connections after each job"

Commit all connections after each job

If you don't commit the slave connections then they keep their old snapshots.
This clears the snapshots so they don't get out of date views of the world.

Bug: T100838
Change-Id: I1f6f910d88324beb589b2ad9466d8786376eda55

Localisation updates from https://translatewiki.net.

Change-Id: I491182176e3357ed36de4856e20af71d735dcd27

Merge "Moved addAutopromoteOnceGroups() call to a deferred update"

Merge "Add tests for HttpError"

Add tests for HttpError

Still missing tests for HttpError::report, but would
have caught the issue that Ie216d19 fixed.

Change-Id: Ic35d70af52c3a2d2a25fc3b9952383198db31fb1

Remove legacy vertical-align from select fields

They shouldn't be aligned at the top by default.

Bug: T98493
Change-Id: I17b0825079ff28dfc9ea2c1ec24018073aec3794

Fix undefined variable $header in HttpError

Change-Id: Ie216d1959f2ea903fdbc124b718b4988e08229ae
Follows-up: I8b61d7e9ea4101e3a9ef5f9a59a97db45aeef68c

Merge "Add 'mediawiki.skinning.logo' module"

HttpError: Consistently escape document title

Wasn't actually a vulnerability because HttpStatus::getMessage
can only return one of a fixed set of values which are all plain
text without any special characters. However the return value
there is meant to plain text and not html, so just like
Html::element and other interfaces, things should be consitently
escaped.

Also renamed variables for clarity.

Change-Id: I8b61d7e9ea4101e3a9ef5f9a59a97db45aeef68c

Merge "Add Vivek Ghaisas (polybuildr) to the CREDITS file"

Merge "Made isAllowed(), showHistory() and formatRevisionRow() protected to allow for easier subclassing"

Merge "SpecialMediaStatistics: Protect against invalid indexes"

Merge "Fix some RecentChange phpdoc return types"

Merge "Tweak 'position' of 'mediawiki.skinning.*' modules"

Fix some RecentChange phpdoc return types

Change-Id: I3e75a97f7fc76e2dc052463511e32f330cfca9ce

Merge "Use a fixed marker prefix string in the Parser and MWTidy"

Merge "Move MWTimestamp::getHumanTimestamp() to Language"

Use a fixed marker prefix string in the Parser and MWTidy

Generating one-time, unique strip markers hurts us in multiple ways:

* The strip marker regexes don't benefit from JIT compilation, so they are
  slower to execute than they could be.
* Although the regexes don't benefit from JIT compilation, they are still
  compiled, because HHVM bets on regexes getting reused. This extra work is
  fairly costly (1-2% of CPU usage on the app servers) and doesn't pay off.
* The size of the PCRE JIT cache is finite, and the caching of one-off regexes
  displaces from the cache regexes which are in fact reused.

Tim's preferred solution (per his review comment on
https://gerrit.wikimedia.org/r/167530/) is to use fixed strip markers.
So:

* Replace usage of $parser->mUniqPrefix with Parser::MARKER_PREFIX, which
  complements the existing Parser::MARKER_SUFFIX.
* Deprecate Parser::mUniqPrefix and its accessor, Parser::uniqPrefix().
* Deprecate Parser::getRandomString(), since it is no longer useful.
* In Preprocessor_*:preprocessToObj() and Parser::fetchTemplateAndTitle,
  replace any occurences of \x7f with '?', to prevent strip marker forgery.
  \x7f is not valid input anyway.
* Deprecate the $prefix parameter for StripState::__construct, since a custom
  prefix may no longer be specified.

Change-Id: I31d4556bbb07acb72c33fda335fa5a230379a03f

Remove deprecated MWFunction::newObj()

Change-Id: I180e9e1e0bcf17c9e72b607c69cae00f47de6579

Whitespace fix for RELEASE-NOTES-1.26

Change-Id: I23a02f465b20117a1855e2a0881fc3bdf1e9ab14

Merge "Remove mw-ui-icon*:after"

Merge "Create mediawiki.widgets and mw.widgets.TitleInputWidget in it"

Merge "Implement MalformedTitleException for JS and use in constructor"

mailmap: Add an additional e-mail for Bartosz per request

Change-Id: Ia3c9f1c83a8d15cb5758288d435b31c3bf1e8b19

Implement MalformedTitleException for JS and use in constructor

Implements MalformedTitleExceptions to provide information on
  invalid titles.  Throw these exceptions from Title constructor
  instead of generic Errors.

Bug: T89648
Change-Id: I9ed16a5a30b01df785ad736df4b84111c5939bbf

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

Localisation updates from https://translatewiki.net.

Change-Id: I42c49984b15a57196744b95c393615fe7f4f797f

resourceloader: Fix broken 'version' param in mw.loader requests

All requests were using version=da39a3ee5e6b, where da39a3ee5e6b
is the sha1() of the empty string.

Follows-up f37cee996e which incorrectly assummed the local
'modules' variable to hold a list of objects, when in fact it
holds a list of strings.

Change-Id: I717b8657300263606832ff1ec67a34a7163de842

Create mediawiki.widgets and mw.widgets.TitleInputWidget in it

Change-Id: I43fb412ff65271fd411b14352784cb92f1f5487b

Merge "list=categorymembers: Do not log feature usage when using default value"

Remove mw-ui-icon*:after

Not being used anywhere from what I can see.

Change-Id: I83cff1444a12fdd390f78dd1c09b5b09b0c84704

Fix to digits grouping rule

Avoid grouping in negative numbers too.
Following the example of LanguagePl.php

Bug: T94300
Change-Id: Ib96616497507c9347cc72f894243b583619c2b11

Move MWTimestamp::getHumanTimestamp() to Language

We're trying to remove MediaWiki dependencies from MWTimestamp so it can
be split out into a separate library. In addition to getting rid of a
dependency on RequestContext, Language, and User, it makes more logical
sense there anyways.

Bug: T100924
Change-Id: If46eaea42d8a5a808c10f0dc353e181714295a44

Localisation updates from https://translatewiki.net.

Change-Id: I4d6988a77f0414b4906f123d888f930b9cdfe253

list=categorymembers: Do not log feature usage when using default value

list=categorymembers&cmstartsortkey was logged even the default value
was set

Bug: T100878
Change-Id: Idc52fe0c208216bc3e475d0e4b2e398cb73a6b80

DeferredUpdates doc comment tweak

Change-Id: Ib58bcb065ef7fac58c3c579eb0ca45e51ccdae11

Add 'mediawiki.skinning.logo' module

Provides ability to display wiki logo with ResourceLoader without loading
the big 'mediawiki.skinning.interface' module.

Used by GreyStuff skin: Id79688a4eb4bf7f813e0c5af1d89556c488b0bde

Change-Id: I7cf51214d9fa378a321c90fc85b0c6799e3b3931

Tweak 'position' of 'mediawiki.skinning.*' modules

mediawiki.skinning.elements:
mediawiki.skinning.content:
  Add 'position' => 'top', like mediawiki.skinning.interface.
  Follow-up to d6b4d3c537fa5944c2f234213b20037fe3192d62.

mediawiki.skinning.content.externallinks:
  Set 'position' => 'bottom', as this module only provides icons
  (and provides a ton of them), which can be loaded after initial load.

Also tweaked some comments.

Change-Id: Ib26afd6cbb2b5d4a7557c5c8243d2f7482a6f064

QA: Upgrade mediawiki_selenium to 1.2 for Raita logging

Change-Id: I4c6bddca1a7c964f3e8543ff7853565505485c51

Fixed Job constructor IDE notices about variable types

Change-Id: I4b4e4e38e8d416c3445c52ced311f5fbfcde868a

Moved addAutopromoteOnceGroups() call to a deferred update

Change-Id: If7941465889c1848ccf1be4ff5505c5eb53a14f1

registration: Don't hardcode list of extension types in schema

Extension types can be added via a hook, so don't try and validate
against a hardcoded list.

Change-Id: I0bc6fbcedc0b582b81c7037724ad01441ae415f6

registration: Improve ResourceModules schema definition

* "-" is a valid character in module names
* debugScripts, loaderScripts, dependencies, and messages can all be
strings in addition to arrays.

Change-Id: I76570ebf1cb02ca51c5185d4b0c48a553a0e7d2f

Cleaned up various revisiondelete IDE warnings

Change-Id: I4750eabf9985785520aaac8eca12d488ca746966

Refactored entry points to have uniform shutdown handling

* Added doPreOutputCommit() and doPostOutputShutdown(),
  which most entry points just using the later
* Also fixed problem where text profiling did not show up
* Avoid calling triggerJobs() in the file streaming
  entry points

Bug: T100127
Bug: T100085
Change-Id: Ibc7e768fd483389a01847f08cdeba4058c853d3f

Merge "Make ResourceLoaderWikiModule support custom position"

Make ResourceLoaderWikiModule support custom position

Bug: T97410
Change-Id: Ib1180c5b2126f4ba8ac5b54578fcce18a3f35d85

ApiOpenSearch: Ignore any other unexpected keys in json output

Iad295d2f fixed warnings to not break the opensearch json output. But
other stuff like $wgDebugToolbar can do the same thing, so let's strip
all unrecognized keys to avoid that problem.

Bug: T100635
Change-Id: I20b6502482af537ccb703f400508960038213a91

Add Vivek Ghaisas (polybuildr) to the CREDITS file

Change-Id: I5bc31ba9f727123a374f0d14c8da22e0c38a0f3f

Localisation updates from https://translatewiki.net.

Change-Id: I202e3eff5b911eaf11f960c66e3d2d5bc95e1d0f

API: Add more parameter types and improve info

New types 'text' and 'password' for where a <textarea> or
<input type="password"> would be preferred over <input type="text">.

Some timestamp parameters get actually tagged as 'timestamp'.

'submodule' types change the 'submodules' output property from a boolean
to an object indicating the mapping from values to module paths. And
they get an indication of the submodule parameter prefix (e.g.
generator's "g"), if applicable. "generator" actually gets reported as a
submodule type, using this new mechanism.

action=paraminfo will now indicate ApiBase::PARAM_RANGE_ENFORCE status,
and return better-formatted defaults for timestamps and booleans.

Change-Id: Ic862d6f8fe13f7eb6b4298683514d33af5823e47

Merge "Cleaned up JobQueueRedis exceptions"

SpecialMediaStatistics: Protect against invalid indexes

Avoids Undefined index: \d in
includes/specials/SpecialMediaStatistics.php

Bug: T89534
Follow-Up: Icd06d2b22b1fcb57a8849ef6dc8659b424f27fdc
Change-Id: I21b0bd4329c9b6fa06526326342edddba3f250f6

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

composer.json: Downgrade PHPUnit to 3.7.37 (temporarily)

In preparation of fetching development dependencies (such as PHPUnit!)
for unit tests via composer in jenkins runs, we should initially use the
same version of PHPUnit to avoid any issues that might come during a
version bump. Once that is stable, we can upgrade PHPUnit again.

Bug: T90303
Change-Id: I907e6d061cab72d1647fcbb2eea35ff144febd7e

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