Merge "Added a tombstone comment to WANObjectCache"

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit e2aa4e4159b9c7b5350567f26f5f14780843a368

Change-Id: I64b4e901ece0e66011b5110312885fab9c8b9cb8

Merge "Migrate move protect log to new log system"

T106578: Update Sanitizer to match legal HTML5 character entities.

Invalid HTML5 character entities become instances of UTF8_REPLACEMENT,
so we also ensure that checkCSS notices this and emits the proper
human-friendly sanitization notice.

Change-Id: I76cef7c772b1e3eba0af8dab6403e9100beab03a

Support IPv6 URLs in bracketed and auto links.

The corresponding patch for Parsoid is
Ibb33188cdfe2004e469c3f6ee6f30d34d1923283.

Task: T23261
Change-Id: Iff077bf31168b431febb243e2e62f2c6502616bc

Migrate move protect log to new log system

Migrate the move protect log as first sub type of the protection log,
because it does not have complex log parameter, which needs some way of
handling/migration.
It also keeps the gerrit change smaller and hopefully makes review
easier.
The other sub types of the protection log will be migrated in a later
patch set.

This allows use of gender on Special:Log. Old message is kept for use
in IRC. A test was added to ensure an unchanged IRC message.

Bug: T47988
Change-Id: I57b3bd8a7dc823acdbb56520d2364f5542283373

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

Previous changes in includes/api/i18n/qqq.json are overwritten now

This reverts commit 70a8ddb78ce354b780828e2a6c70aa3dcee791bd.

Change-Id: I783d4333843d02359af623cd0f18362963f03d0c

Localisation updates from https://translatewiki.net.

Change-Id: I1efd504c98c539d26753e01d501fb1d8a88a7a11

Merge "Add American sign language (ase)"

Merge "Prevent the jumbling up of autocomments in RTL"

mediawiki.Upload, mediawiki.Upload.Dialog: Tweak documentation

Mostly whitespace changes, since my editor insists on highlighting
mixed tabs and spaces as errors.

Change-Id: I17ccff4ea80c15015fb156547abd0db87c1ab8a1

Prevent the jumbling up of autocomments in RTL

Change-Id: I75f66dcd839ead0759f4c6e90ab0ddbc905a07d9

Merge "Revert "resourceloader: Add must-revalidate to Cache-Control""

Merge "Avoid some possible deadlocks on account creation"

Avoid some possible deadlocks on account creation

* This uses a non-blocking $wgMemc lock to reserve the user
  name in question. This should prevent two threads from
  reaching LOCK IN SHARE MODE and getting stuck on INSERT.
  The lock is global to better cover auth plugins.
* This adds a BagOStuff::getScopedLock() convenience method.
  It uses less queries than LockManager by being EX only.
* Avoid extra lock attempt in lock() in non-blocking mode.
* Removed (un)lock() HashBagOStuff overrides that made it
  behave differently than other caches (wrt to re-entrance).

Bug: T106850
Change-Id: Iecf95206d712367f5d202f76ab0eaa9d7bdabf2b

Merge "API: Use message-per-value for apihelp-query+protectedtitles-param-prop"

Merge "API: Use message-per-value for apihelp-query+usercontribs-param-prop"

Merge "API: Use message-per-value for apihelp-query+recentchanges-param-prop"

Merge "API: Use message-per-value for apihelp-query+alllinks-param-prop"

Merge "API: Use message-per-value for apihelp-query+allusers-param-prop"

Merge "API: Use message-per-value for apihelp-query+blocks-param-prop"

Merge "API: Use message-per-value for apihelp-query+categorymembers-param-prop"

Merge "API: Use message-per-value for apihelp-query+exturlusage-param-prop"

Merge "API: Use message-per-value for apihelp-query+filearchive-param-prop"

Merge "API: Use message-per-value for apihelp-query+iwbacklinks-param-prop"

Merge "API: Use message-per-value for apihelp-query+iwlinks-param-prop"

Merge "API: Use message-per-value for apihelp-query+langbacklinks-param-prop"

Merge "API: Use message-per-value for apihelp-query+langlinks-param-prop"

Merge "API: Use message-per-value for apihelp-query+linkshere-param-prop"

Merge "API: Use message-per-value for apihelp-query+logevents-param-prop"

Merge "API: Use message-per-value for apihelp-query+pageswithprop-param-prop"

Merge "API: Use message-per-value for apihelp-query+search-param-prop"

Merge "API: Use message-per-value for apihelp-query+siteinfo-param-prop"

Merge "API: Use message-per-value for apihelp-query+tags-param-prop"

Merge "API: Use message-per-value for apihelp-query+users-param-prop"

Merge "API: Use message-per-value for apihelp-query+userinfo-param-prop"

Merge "API: Use message-per-value for apihelp-query+watchlistraw-param-prop"

Merge "API: Use message-per-value for apihelp-query+allcategories-param-prop"

Merge "API: Use message-per-value for apihelp-query+categories-param-prop"

Merge "Pass user to WikiPage::doEditContent from EditPage"

Merge "Expose ICU version on Special:Version"

Merge "Cleanup after extraction of utfnormal library"

Merge "mediawiki.api: Small code quality and documentation tweaks"

Merge "CSSMin: version URLs based on content, not mtime"

Merge "Fixed IDE warning in doIsRootJobOldDuplicate()"

Merge "Include job UUID in doAck() error message"

Merge "Made WANObjectCache::get() comments more accurate"

CSSMin: version URLs based on content, not mtime

The content of these files is more stable than their mtimes, which change
every time we roll out a new branch. Because MD5 avalanches well[0], using
the first five hexadecimal digits is sufficient to ensure that the chance
of two successive versions colliding is improbably small (roughly one in a
million).

[0]: https://en.wikipedia.org/wiki/Avalanche_effect

Change-Id: I1bdf94c58786d2545311b238476b48217a5a60af

mediawiki.feedback: Use .parent instead of .super

'super' is a reserved keyword and using it like this causes a syntax
error on IE 8.

Change-Id: I1ec22a7d0d61b5731ecba7653c9189eb702f968c

Add American sign language (ase)

* Was added in 5fac9aa71974db71e161fb233bde42d0990167b4
* Quite a non-standard locale, but apparently there are clients
  that can make sense out of it.

Change-Id: I0a7acc185d8c035fdb3dd649eb74f7e70bc2304e

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 48617c40c2eb9c7a421bb9c5b3ba0907fdec9c21

Change-Id: I565886079e64985844fc0a7c67ab74340b42f84a

Merge "Only convert boolean true/false to !0/!1"

Localisation updates from https://translatewiki.net.

Change-Id: Idea3aef13a224782cf99204aad0426aefe29fde4

mediawiki.api: Small code quality and documentation tweaks

Change-Id: I70380e8f821c66ad08bfe12e242b204ffed9071d

Merge "Fix return value comment on wfMemoryLimit()"

Fix return value comment on wfMemoryLimit()

It was changed incorrectly in I893ddd77.

Change-Id: I1f1e3c53a2cefe69b42ef338aa2a6b126e3d5526

Rename variable to avoid collisions

The overwriting of $status in one switch case looks like a likely cause
of T109193, so let's rename one of the two.

Bug: T109193
Change-Id: I373f0e01d349883c7735c8e6acf9d258900c166e

Merge "Remove unnecessary ZWNJ character [azb]"

i18n: Make noarticletext direction aware

Currently https://fa.wikipedia.org/wiki/MediaWiki:TEST.js?uselang=fa
is displaying "noarticletext" in LTR which is wrong.

Bug: T60685
Change-Id: I4dc76dc1c5134c0dbe0988d6d0a3286a0e52242f

SpecialAllPages: Fix a few subtle "Previous page" link bugs

* When finding the start of the previous page, exclude redirects if
  "Hide redirects" is checked. Otherwise, if some of the previous 345
  wiki pages were redirects, an equal number of results would be
  repeated when going back.
* Use the DB key form of the title (not the unprefixed text form)
  when building the query.
* Don't select unnecessary rows beyond the start of the previous page.
  At the same time, fix a bug that could cause results close to the
  beginning to be skipped when going back.
* When finding the start of the first page, don't omit ORDER BY for
  MySQL and SQLite.
* Don't suppress the link when the unprefixed title is "0" or similar
  or is numerically equal to the "from" parameter value.
* Suppress the link when the title would come after the "from"
  parameter value, not just when it would be equal to that value.

Change-Id: I2cc6958b894afa824a5345871b5a27b01f9fcc73

Cleanup after extraction of utfnormal library

* Delete includes/compat/normal/.gitignore, which excluded files
  now located in various directories within the utfnormal repo.
* Remove "For NFC see includes/compat/normal" comments from
  maintenance/language, replacing them with a note that "This
  data file is used after normalizing to NFC."

Change-Id: I0ac990ec1a547b4c454ed738ce1dfffce0f53ba4

Localisation updates from https://translatewiki.net.

Change-Id: I8752df5f72fd7ba63aeb440cd8203c46a1318c80

Rename mediawiki.legacy.feeds to mediawiki.feedlink

Follow-up to d51317ccc24940b77f4f7226d9455c801c09b457. There's nothing
"legacy" about this newly added module.

Also updated comments and marked the copy in shared.css for removal
when cached page renders expire.

Change-Id: Iabaebe9bf1e11a06bb951769693b1be4404e5b00

Merge "mediawiki.legacy.common: don't embed legacy images"

Merge "Add mediawiki.legacy.feeds"

Only convert boolean true/false to !0/!1

Fix-up for I5ab29b686b8. If we encounter stupid code like
`a.true = 1;` or `a = { true: 1 }`, we should not convert that to !0/!1.
Because JSMin barfs on such input, it is necessary to add another parameter to
the test method which specifies whether or not the minified JavaScript is
supposed to be valid JavaScript by the standards of JSMin.

Change-Id: Ib78c628147fdb95982d6e33e0ab298584fb63d0b

Always call setValidationHash() in MessageCache::saveToCaches()

* This avoids constant churn when $wgUseLocalMessageCache is set to false
* Follow up to db464b8a847a

Bug: T109183
Change-Id: I8da324c53527da32d09964be6c3a92176af4ee7b

Made WANObjectCache::get() comments more accurate

Change-Id: I531c2dca6b552a6b3732d2506e7373c7d17b4b4c

Localisation updates from https://translatewiki.net.

Change-Id: I4bb5aeb31faebf7981341ce5ba49df8401739a7e

mediawiki.legacy.common: don't embed legacy images

This drops the /* @embed */ directive for the images referenced by the
mediawiki.legacy.common stylesheet. The images themselves and the styles which
apply them remain. They will simply not be pre-loaded as data URIs within the
stylesheet, which is fantastic because mediawiki.legacy.common is top-loaded on
all pages and embedded images make up nearly half of its weight. With this
change, the gzipped size of mediawiki.legacy.common drops from 27.6k to 15.6k.

Affected classes:

* `.mw-icon-question`: Used for tooltips generated by Preferences::getOptionFromUser.
* `.mw-help-field-hint`: Not in use since I8e092e9d.
* `.mw-small-spinner`: Not in use since I2a2556a0.
* `.mw-ajax-loader`: Not in use.

Change-Id: I6c7e592a502ae14b9f81165f10a1d4240e592649

Add mediawiki.legacy.feeds

Instead of embedding feed icons in mediawiki.legacy.shared, which is loaded on
every page, embed them only in mediawiki.legacy.feeds, which is loaded only on
pages which have feedlinks.

Change-Id: Ife5d84fb95944cb24939936785f3d8c4d9c2c187

Merge "mediawiki.legacy.config: use own help-field styling"

Merge "Make PPFrame::RECOVER_COMMENTS actually work"

Fix some space-related phpcs warnings in includes/

Change-Id: I7cf7206696a5e77bc02e3630d1d88d4c176ea844

Remove unused globals

* $wgLogExceptionBacktrace from MWExceptionHandler
  unused since 77a397125fd1b901c3141cbfe7084a7534ff881b

* $wgPasswordPolicy from WebInstallerName
  unused since 6a69a4eb733bd6129533c93b7a1ca4564b16ac32

Change-Id: Ibaac1bad05f4326a2602958a93d4d46135bdca63

Make PPFrame::RECOVER_COMMENTS actually work

Because of a missing condition, it generally only had an effect on
output type Parser::OT_WIKI, and thus {{msgnw:}} would strip comments
except when substituted during a pre-save transform.

Bug: T98841
Change-Id: I1e47696434fe87475f9902e6bfb8990566456e2f

mediawiki.legacy.config: use own help-field styling

Instead of using .mw-help-field-container from mediawiki.legacy to style help
tooltips, use a class that is specific to the WebInstaller and which is styled
in its stylesheet. This allows us to stop embedding it in mediawiki.legacy
(which is loaded on every page).

The CSS is copied directly from the styles for mw-help-field-container et al.
in mediawiki.legacy.

Change-Id: I8e092e9d2c91b5f70499d93c1134f43f2e96348c

mediawiki.special.upload: use spinner from jquery.spinner

mediawiki.special.upload already depends on jquery.spinner, which supplies its
own spinner CSS classes which utilize the same image (spinner.gif) that
.mw-small-spinner uses. Both are /* @embed */-ed, which causes unnecessary
bloat. By using the class from jquery.spinner, we can stop embedding the one in
mediawiki.legacy. (This will happen in a follow-up patch.)

Change-Id: I2a2556a0a46b134a3e83f63733d988bbd8890323

Include job UUID in doAck() error message

Change-Id: Ib84a55bd92802684b98cc1cf06e5414bd1bd9fd6

Fixed IDE warning in doIsRootJobOldDuplicate()

Change-Id: I95110dff0755336d9bb245503e4ae856171064fc

Add pear/mail_mime-decode to composer suggests

Change-Id: Ieb8f2c9d007a9d97956f42354180745e4698a8c4

Merge "Adding release note for category changes in watchlists"

Add simplified storage API

Provide a standard mechanism for accessing localStorage.

It may seems simplistic right now, but to give an idea of the why:
* We already have jquery.jStorage.js which is a much more heavyweight
approach to storing non-essential values.
* We are repeating ourselves a lot in extensions by having
to do localStorage detection and then deal with full localStorage.
In MobileFrontend we have a settings module. This is one of the reasons Gather
depends on MobileFrontend and I'm keen to remove that dependency.
* We might want to move to indexdb in future. Having a single API makes moving
this easier - we don't have to update everywhere that uses localStorage
* Saving non-string support would be useful. The API could be adjusted to take a
mixed second parameter that stringifys JSON objects.
* Cookie fallbacks are a possible alternative when localStorage is not supported. This allows
us to be agnostic of the storage mechanism going forward.

Note:
This doesn't reuse the handling in mediawiki.js as at this point
I am not sure there is value. mw.loader.store.enabled is false when
$wgResourceLoaderStorageEnabled is not true and this should work even
without that case. We can review this at a latter point.

See:
Id5c32bb7a662dda8d153490f7c47e972cabc1efd
I3fd44b0ae6633a7053aee247bc3c4704ba987bc8

Bug: T96155
Change-Id: Idb37352acecd745beb53aa8d77ea050851448e0d

MessageCache: derive the hash from the cache contents

Follow-up for I020617d, where I got this wrong. It is probable that multiple
application servers will try to build a local cache at the same time, in which
case they will cause the cache key to thrash. It is necessary that two
application servers building a local cache from the database generate the same
hash for a given set of messages.

Change-Id: Ieeefc2094a83be9401c466bec859c1588ddfbcdf

Merge "ResourceLoader: apply minify-js filter to config scripts"

Localisation updates from https://translatewiki.net.

Change-Id: Ib58e7609fd477b4f763c9a7d2a982a56535a026f

Merge "Optimize the order of styles and scripts in <head>"

Optimize the order of styles and scripts in <head>

Emit styles and scripts in the following order:
* Inline <script>...</script> tags.
* External stylesheets.
* External script tags.

I have attempted this before in I98d383a6, and subsequently reverted (in
I151f74a41) when it failed to show a performance improvement. Our ability
to measure performance were substantially poorer then, so I would like to
give this another try and see if it makes a dent in first render times.

Change-Id: I5e6c79c7041aa77bcdc6130e91e77aa538334c42

ResourceLoader: apply minify-js filter to config scripts

Minify the JavaScript generated in ResourceLoader::makeConfigSetScript() unless
debug mode is set.

Change-Id: I45b5839803546c8e35e74b72c679777815294315

Adding release note for category changes in watchlists

Bug: T9148
Change-Id: If49bc9dc7385b1bb87933d47643f21444824e3b1

Add descriptions for composer.json suggested packages

The value for a suggested package is intended to be a description of why
the library might be desired.
<https://getcomposer.org/doc/04-schema.md#suggest>

Change-Id: Ic9e7cc884baf82c493b72ef405cfd5ded810dded

Merge "JavaScriptMinifier: minify booleans"

JavaScriptMinifier: minify booleans

Minify true to !0 and false to !1, like Google Closure Compiler and UglifyJS2.

Change-Id: I5ab29b686b812d64a2913611be091084c06d630b

UserMailer comment cleanup

Use `//` line comment markers.

Change-Id: I327bf2de17bd5996f3bf1b8e74c55f7067df3336

Add PEAR's Mail and Mail_Mime to composer.json's suggestions

Bug: T97454
Change-Id: Ib9f0f5329249b504814d08638f5b7fbde091344c

More multi-DC tweaks to MessageCache::load()

* Use hash key volatility to propagate invalidations over DCs
  in addition to memcached->APC instances.
* Make use of stale APC cache due to hash mismatch instead
  of waiting around in some cases. This is similar to the
  TTL expiry and volatility cases.
* Renamed $hashExpired and added some comments for clarity.

Change-Id: I8890fb174cae72c4ce8872df64f52f5cbd55183b

Fix some newlines in en.json of API

The api modules are grouped with newlines, but some newlines where
misplaced.

Change-Id: I8ba1fc9cffe0466721b9b96ddd958a23aa65cf1b

Pass user to WikiPage::doEditContent from EditPage

This avoids the use of $wgUser inside of WikiPage::doEditContent.
It does not avoid the use of $wgUser in general, because EditPage does
not know about context, but it will make the indirect use visible and a
rewrite of EditPage can address the issue better.

Change-Id: Ifbcf52f2f076dc5c177bea22f90eb3b9fb8e3224

Ignore tagfilter url parameter on Special:EditTags for log items

There is no need to pre-filter the seleteced log items for a tag.
This avoids use of $wgRequest in ChangeTags::modifyDisplayQuery

Similar to I5899a66ad8c500f058ce006c623d3e20c5bf8508

Change-Id: I36c2e2c638dc2418b9fb5b07c46a24235746d99f

Merge "Enable users to watch category membership changes"