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

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"

Merge "Avoid fatal when accessing method of non-objects on Special:Undelete"

Merge "Don't add a link to Special:RecentChanges when tag filter is disabled"

Don't add a link to Special:RecentChanges when tag filter is disabled

When wgUseTagFilter = false there is no need to link to
Special:RecentChanges because the special page cannot display the tag
filter and will show all changes, which is wrong.

Bug: T105650
Change-Id: I389aba9cb2862df3ba6980333c5cc94bb30dc202

Merge "Import BatchRowUpdate classes from Echo"

Import BatchRowUpdate classes from Echo

This is a set of classes written for Echo to simplify writing
maintenance scripts that iterate over an entire table and update
some of those rows.

This has shown to be reusable elsewhere, especially the BatchRowIterator
class and will be useful to have generally avilable in core. The Echo
classes are all prefixed with the Echo name so there wont be any
conflict is both are installed.

Change-Id: I64c1751106caf34f41af799dbaf8794115537f06

Update OOjs UI to v0.12.4

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.12.4/History.md

Change-Id: I3d5a027e06a6d81289988a5199e56a7b1b25d35c

Added statsd metrics for ObjectCacheSessionHandler

Change-Id: I86e967a027e2010f962831b55a9fceab02d8a19e

Merge "Change "1 title" to "A single title" in watchlistedit-normal-done"

Localisation updates from https://translatewiki.net.

Change-Id: I50410890d654c1a81684392ef1e66c81b181ccee

Merge "Don't send email notifs to blocked users if $wgBlockDisablesLogin is true"

Enable users to watch category membership changes

Bug: T9148
Change-Id: I5a89d8f19804b1120f4c755d834e2da6ca12ceae

Change "1 title" to "A single title" in watchlistedit-normal-done

It was customized this way in en.wikipedia.org,
and it indeed looks better in English.

Change-Id: If43e788f996e2f99fdeda746296129b2c7212f52

Expose ICU version on Special:Version

ICU (International Components for Unicode) <http://site.icu-project.org/>
is used to implement ordering of category members (category collation)
when $wgCategoryCollation is set to one of the supported "uca-xx" collations.
Its version determines whether the ordering will be correct for a number
of languages MediaWiki supports.

Change-Id: Id938a9510074f1743a57a840590cabcf3b90a103

Adding documentation for new RandomPageQuery hook to hooks.txt

Hook added in I91cd231d1

Change-Id: I448e0227b00e0e5b9390b144b9de360b8453f9ff

Adding a new hook to allow modification of Special:Random query

See: Ia84c54dcdadf54e9475c1dc7d0ec7c2c48afeaae
Bug: T9937
Change-Id: I91cd231d1cbf610f20134fd0271af46842a9a17d

Try to make POSTs as transactional as possible

* WMF has has ignore_user_abort() for *all* request in configuration
  for many years. This brings this to default MediaWiki for all POST
  requests, which are likely to do writes. Of course, some do not,
  notably parse requests and previews, since GET cannot carry the
  payload. Avoiding data corruption is more important though.

Bug: T102890
Change-Id: I11c8b0d99583a682f756cef9747ec5ba9751c5e3

Merge "Optimized notification timestamp clearing"

Merge "Preferences: Don't load all default options for every default preference"

Optimized notification timestamp clearing

* Hint that mariadb can avoid scanning already NULL rows since
  we want to set them all to NULL. This lets the index with the
  timestamp be used, which avoids logs of scanning for users with
  many rows. For non-parallel replication, this also avoids slave
  lag in particular.

Bug: T107923
Change-Id: I8cded5476ef6adb2f8fea57239afd6e9b2a49708

Preferences: Don't load all default options for every default preference

Change-Id: Iaaebea56674dfaa48dbfedf6c7e5ad9eaa05759e

Added wfTransactionalTimeLimit() method and applied it

* Potentially long running POST requests often use multiple transactions,
  talk to multiple services, or defer updates. Try to make sure they have
  a chance to complete all of the work. WMF already sets ignore_user_abort()
  across the board in config, but this applies it to key spots for all
  installs, in addition to bumping the time limit.
* Eventually this can lower the need for high overall time limits.

Bug: T102890
Change-Id: I893ddd773064dcd63b5b24c84c6391974f4b5aee

Merge "Made JobRunner bail more smoothly on near OOM"

Localisation updates from https://translatewiki.net.

Change-Id: I02ea462b65c3d406bfc552dd2f6f09d3984bbeb1

Output stylesheet links before other link elements in <head>

Since external CSS is render-blocking, it should be discoverable to the browser
as soon as possible. So move it up to the top of <head>.

Change-Id: I68a6cce2f33ceb7f6461fcbb7323ae13c413aa89

Merge "Changed "archived (revisions)" to "deleted (revisions)""

Revert "mediawiki.page.ready: When infusing, infuse by id, not by element"

Infusing by element is better, since it avoids the
document.getElementById() lookup and behaves better in case
of duplicate ids (which is invalid HTML, but happens).

This reverts commit 803d89173fc3a56871767663907785cb3eaf7aa5.
Depends on I11fd1c7f2434f904b9ddc94b0903f8278e06f3a1 in OOjs UI.

Bug: T105828
Change-Id: I8c1f5015229db7b8a391f324cfd96be4ac158041

Update OOjs UI to v0.12.3

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.12.3/History.md

Change-Id: Idd3a4e41154837e84d1dcb4622d04b5765e6cbf9

Merge "wikibits.js: Support multiple parameter in document.write"

wikibits.js: Support multiple parameter in document.write

document.write and document.writeln supports multiple arguments:
http://www.w3.org/html/wg/drafts/html/CR/webappapis.html#document.write%28%29

Support this in the replacement code.

Change-Id: I0ffc51352278d434d8347502c778b16b34c7d73f

Restored I4afaecd8: "Avoiding writing sessions for no reason"

* After 203d2c9c11643, the refresh updates should actually trigger

This reverts commit e48fec5a8ab5916735849d420f2dfeec7eb0fced.

Change-Id: I8427ed5b3a5bb80033cbdf071a33f8a3999ecb97

Merge "Fixed @var doc in MultiWriteBagOStuff"

Fixed @var doc in MultiWriteBagOStuff

Change-Id: I53384460b7a6ccda50a23e3ab1fc614198e82d1a