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 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"

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

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

Follow-Up: I4da8ecb457a77473e32d745ba48ab8505b35e45f
Change-Id: I30ebf2a21bb09d992140baa2bd8707a674da6d51

SpecialStatsAddExtra: Format column label with msg

This allows the user of the SpecialStatsAddExtra hook to provide
formatting for the row label using an i18n message key.  If given, the
message is given the row key as a parameter.  To maintain backward
compatibility, the key is used as-is as was done previously if a message
key is not provided.

Bug: T97623
Change-Id: I43c522b24372e115ed78adf69848bf50cbab8295

Localisation updates from https://translatewiki.net.

Change-Id: I49999070014b45d52c59ec38c4bd84202e1172c6

Made RecentChange defer the save() method via DeferredUpdates

* Also made it handle calling PatrolLog::record() itself

Bug: T100042
Change-Id: I58ef060e02b89a5f9dadc0dbc4edba667932beda

Localisation updates from https://translatewiki.net.

Change-Id: I0b708cf7c6301f57aa618284f960b9701157cf15

Defer some updates in doEditUpdates()

* These updates add to editing time and can be done
  after sending the HTTP response for performance
* Also improved the active users job insertion logic

Change-Id: I5b25217c4f08db7fa9a05eac046283f02d45865e

ProfilerOutputStats: replace single ':' with '.', too

This maps 'hook: ' and 'query: ' to a metric name segment, instead of just a
metric name prefix.

Change-Id: I41fb71becbc862597b998c73ccfcaca5a3510ffc

Improve ProfilerXhprof's blacklist/whitelist capabilities

* Apply the blacklist / whitelist to profiled sections, not just function names.
* Allow shell-style wildcard patterns in blacklist / whitelist.
* Prefix all profiled section names with 'section.', to distinguish them from
  functions.

Note that shell-style wildcard patterns are not supported by xhprof natively,
but it won't barf on them either, nor will they match against actual function
names (since shell wildcard characters are not valid for PHP function names),
and the filtering will still be enforced in ProfilerXhprof.

This has the side-effect of working around https://github.com/facebook/hhvm/issues/4385

Bug: T99829
Change-Id: I8354ed922fa7b42857eda03be8f62b89ac78d0d6

Merge "API: Add 'wrfromtitle' and 'wrtotitle' to list=watchlistraw"

Merge "Avoid calling load() in LocalFile::invalidateCache; not needed"

Merge "ProfilerOutputStats: allow a key prefix to be specified"

Avoid calling load() in LocalFile::invalidateCache; not needed

Change-Id: I1e2d61722a18e4046bdf7c375ea9c1dbbea4a157

ProfilerOutputStats: allow a key prefix to be specified

If one wants to nest all metrics emitted by the profiler under a metric
namespace, one can now set the 'prefix' param.

Task: T66301
Change-Id: I6c52f20e39017f4c818ca6623bb7f48683fc8abc

Merge "Search: Allow searchContainedSyntax to be defined by constructor"

Merge "registration: Don't ignore empty array config settings when converting"

Merge "registration: Add test case to demonstrate T98347 is invalid"

registration: Don't ignore empty array config settings when converting

Remember which settings we set to an empty array and then only skip
those settings.
The weird variable name "$__settings" is used to avoid any conflicts
similar to what eval.php does.

Bug: T98739
Change-Id: Iefbf8a98fd433cfbe0087aca6821d90c2786b0cb

API: Add 'wrfromtitle' and 'wrtotitle' to list=watchlistraw

Naming is "fromtitle" rather than just "from" because "from" parameters
in API query modules generally mean a title without namespace, and this
should make use of the client-supplied namespace.

Bug: T98985
Change-Id: I8370f47f6eed71ea97a44a66eb6be2a8f89fea53

Merge "Avoid pointless getNativeData() call in isCountable()"

Avoid pointless getNativeData() call in isCountable()

Change-Id: I40ed1972a8815de645d11217b35c1060d5007e5a

Make sure $dbw is an object in DeferredUpdates::addUpdate()

Change-Id: Ib52574bc7f2e13bf06f595e349a7b3ccea425056

Merge "registration: Notify of problems before filemtime, not after"

registration: Notify of problems before filemtime, not after

Without this several lines of warning are emitted before the final error indicating the cause.

Bug: T99846
Change-Id: Ic1b9d9a80ed995273c56c447c8b044285a8843a7

Flush lazy jobs in load.php and api.php too

* A more proper solution will have one method for all this stuff

Bug: T99775
Bug: T99849
Change-Id: I16a32b260386a85504eb85ea58f1156b4e581a83

Merge "Bumping lightncandy version from 0.18 to 0.21"

Merge "Fix Special:Preferences onBeforeUnload handler"

Merge "Pass a message key to MalformedTitleException constructor"

Fix Special:Preferences onBeforeUnload handler

Bug: T94194
Change-Id: If9e3836c6061cf99dc0859dafb99b9456e6efd0e

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

Pass a message key to MalformedTitleException constructor

'badtitletext' seems to be equivalent to the situation before I4da8ecb4,
so let's go with that.

Bug: T99818
Change-Id: Ifdf66d2806df3ae7d2d378343491dd8976c31981

Localisation updates from https://translatewiki.net.

Change-Id: I606ac1f2237b561daf4bc88426e3b9f5e65a8fc9

Improve documentation of maintenance/sql.php

I found this to be lacking earlier this week...

Change-Id: I409c7451baa71b88cf554af0d82d1bf5904a2c4c

Keep jenkins happy by limiting line length.

Just w/s changes to break up long lines.

Change-Id: I69bf30f6c05813cfa32c8cd194f121cedcc4cb8a

Merge "Message text clarified."

Merge "Make JobRunner flush DeferredUpdates after each job"

Merge "Simplified doEditContent exception handling"

Merge "Fixed wrong EnqueueJob comment"

Merge "Do not allow setting deprecated $wgSpecialPageGroups over extension.json"

Fixed wrong EnqueueJob comment

Change-Id: I8a84bae9cd4206eb042e5a29c91d8281e296ecd1

User::clearSharedCache: don't load the full user record for just the ID

Rather than call $this->load() unconditionally, get just the ID via
$this->getId(), potentially saving us a database lookup.

Change-Id: Ifa3b96956a1d66d3e877573ff01970758fad8790

resourceloader: Replace timestamp system with version hashing

Modules now track their version via getVersionHash() instead of getModifiedTime().

== Background ==

While some resources have observeable timestamps (e.g. files stored on disk),
many other resources do not. E.g. config variables, and module definitions.

For static file modules, one can e.g. revert one of more files in a module to a
previous version and not affect the max timestamp.

Wiki modules include pages only if they exist. The user module supports common.js
and skin.js. By default neither exists. If a user has both, and then the
less-recently modified one is deleted, the max-timestamp remains unchanged.

For client-side caching, batch requests use "Math.max" on the relevant timestamps.
Again, if a module changes but another module is more recent (e.g. out-of-order
deployment, or out-of-order discovery), the change would not result in a cache miss.

More scenarios can be found in the associated Phabricator tasks.

== Version hash ==

Previously we virtually mapped these variables to a timestamp by storing the current
time alongside a hash of the value in ObjectCache. Considering the number of
possible request contexts (wikis * modules * users * skins * languages) this doesn't
work well. It results in needless cache invalidation when the first time observation
is purged due to LRU algorithms. It also has other minor bugs leading to fewer
cache hits.

All modules automatically get the benefits of version hashing with this change.
The old getDefinitionMtime() and getHashMtime() have been replaced with dummies
that return 1. These functions are often called from getModifiedTime() in subclasses.

For backward-compatibility, their respective values (definition summary and hash)
are now included in getVersionHash directly.

As examples, the following modules have been updated to use getVersionHash directly.
Other modules still work fine and can be updated later.

* ResourceLoaderFileModule
* ResourceLoaderEditToolbarModule
* ResourceLoaderStartUpModule
* ResourceLoaderWikiModule

The presence of hashes in place of timestamps increases the startup module size on
a default MediaWiki install from 4.4k to 5.8k (after gzip and minification).

== ETag ==

Since timestamps are no longer tracked, we need a different way to implement caching
for cache proxies (e.g. Varnish) and web browsers. Previously we used the
Last-Modified header (in combination with Cache-Control and Expires).

Instead of Last-Modified (and If-Modified-Since), we use ETag (and If-None-Match).

Entity tags (new in HTTP/1.1) are much stricter than Last-Modified by default.
They instruct browsers to allow usage of partial Range requests. Since our responses
are dynamically generated, we need to use the Weak version of ETag.

While this sounds bad, it's no different than Last-Modified. As reassured by
RFC 2616 <http://www.w3.org/Protocols/rfc2616/rfc2616-sec13.html#sec13.3.3> the
specified behaviour behind Last-Modified follows the same "Weak" caching logic as
Entity tags. It's just that entity tags are capable of a stricter mode (whereas
Last-Modified is inherently weak).

== File cache ==

If $wgUseFileCache is enabled, ResourceLoader uses ResourceFileCache to cache
load.php responses. While the blind TTL handling (during the allowed expiry period)
is still maxage/timestamp based, tryRespondNotModified() now requires the caller to
know the expected ETag.

For this to work, the FileCache handling had to be moved from the top of
ResoureLoader::respond() to after the expected ETag is computed.

This also allows us to remove the duplicate tryRespondNotModified() handling since
that's is already handled by ResourceLoader::respond() meanwhile.

== Misc ==

* Remove redundant modifiedTime cache in ResourceLoaderFileModule.

* Change bugzilla references to Phabricator.

* Centralised inclusion of wgCacheEpoch using getDefinitionSummary. Previously this
  logic was duplicated in each place the modified timestamp was used.

* It's easy to forget calling the parent class in getDefinitionSummary().
  Previously this method only tracked 'class' by default. As such, various
  extensions hardcoded that one value instead of calling the parent and extending
  the array. To better prevent this in the future, getVersionHash() now asserts
  that the '_cacheEpoch' property made it through.

* tests: Don't use getDefinitionSummary() as an API.
  Fix ResourceLoaderWikiModuleTest to call getPages properly.

* In tests, the default timestamp used to be 1388534400000 (which is the unix time
  of 20140101000000; the unit tests' CacheEpoch). The new version hash of these
  modules is "XyCC+PSK", which is the base64 encoded prefix of the SHA1 digest of:
  '{"_class":"ResourceLoaderTestModule","_cacheEpoch":"20140101000000"}'

* Add sha1.js library for client-side hash generation.
  Compared various different implementations for code size (after minfication/gzip),
  and speed (when used for short hexidecimal strings).
  https://jsperf.com/sha1-implementations
  - CryptoJS <https://code.google.com/p/crypto-js/#SHA-1> (min+gzip: 2.5k)
    http://crypto-js.googlecode.com/svn/tags/3.1.2/build/rollups/sha1.js
    Chrome: 45k, Firefox: 89k, Safari: 92k
  - jsSHA <https://github.com/Caligatio/jsSHA>
    https://github.com/Caligatio/jsSHA/blob/3c1d4f2e/src/sha1.js (min+gzip: 1.8k)
    Chrome: 65k, Firefox: 53k, Safari: 69k
  - phpjs-sha1 <https://github.com/kvz/phpjs> (RL min+gzip: 0.8k)
    https://github.com/kvz/phpjs/blob/1eaab15d/functions/strings/sha1.js
    Chrome: 200k, Firefox: 280k, Safari: 78k

  Modern browsers implement the HTML5 Crypto API. However, this API is asynchronous,
  only enabled when on HTTPS in Chromium, and is quite low-level. It requires boilerplate
  code to actually use with TextEncoder, ArrayBuffer and Uint32Array. Due this being
  needed in the module loader, we'd have to load the fallback regardless. Considering
  this is not used in a critical path for performance, it's not worth shipping two
  implementations for this optimisation.

May also resolve:
* T44094
* T90411
* T94810

Bug: T94074
Change-Id: Ibb292d2416839327d1807a66c78fd96dac0637d0

Simplified doEditContent exception handling

* Callers should not catch errors and fail to rollback
  changes as much here as anywhere else

Change-Id: I65cbeb8d0895223b7ad60c9081d703d14197cb4a

Localisation updates from https://translatewiki.net.

Change-Id: Idff54a370b7401517d4f2313576773bc765ca3e4

Merge "registration: Have cache entries expire after 24 hours"

Merge "registration: Remove implementation of custom processors for now"

Merge "registration: Use a static whitelist of keys that are not attributes"

Merge "Added documentation for HTMLFormField's "hide-if""

Message text clarified.

See also:
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Title-invalid-interwiki/en

Change-Id: Id6f4573527415ec5f82fd5a4599480e5c17c84d3

mediawiki.language: Respect $wgTranslateNumerals in convertNumber()

This commit exposes the $wgTranslateNumerals variable via mw.config.

Bug: T72109
Change-Id: Id84ecc0a23aa83041f44294f249af28b92a1e629

Merge "Added ObjectCache::getMainStashInstance() and $wgMainStash"

Merge "Revert "Remove invalid language code "bh" (alias of "bho")""

registration: Don't array_unique() over the queue before loading it

array_unique works over values, not keys (you can't have an array with
duplicate keys in PHP) so it would end up removing extensions that
happened to have an identical timestamp with another one.

Bug: T98956
Change-Id: I66e202ea882ae4d5cb1bcafb7882cbd9cd52a1b3

registration: Have cache entries expire after 24 hours

Since our cache invalidation strategy is to just change the key, don't
have old entries stick around forever.

Change-Id: Ib62b69adfc2855a2ecd08558d4054d9b5d3ed571

Revert "Remove invalid language code "bh" (alias of "bho")"

Still being heavily used for interwiki links.
See eg. https://www.wikidata.org/wiki/Wikidata:Contact_the_development_team#Link_for_bh.wiki

This reverts commit 34a3de0f65609c7ffdaafbf3770001e531a616cf.

Bug: T99059
Change-Id: I6db0c436dc2c7556bbc060584e0a725402238adf

Made pushLazyJobs() handle all queue groups

* This avoids __destruct() warnings
* Also remove push() code in __destruct() that just made
  more warnings since it's too late

Change-Id: I32a3cfabc94e36b9d2808b45d55209f3df46e47d

Make JobRunner flush DeferredUpdates after each job

Change-Id: Iff6625ddc04a15751d2bb07dc6558145e7ceb14a

Search: Allow searchContainedSyntax to be defined by constructor

The SearchResultSet is returned where no query is performed because it
is known ahead of time that no result exists. In cases where that is
known due to the syntax of the query (e.g. the syntax requires
searching a category that does not exist) this searchContainedSyntax
method returns the wrong value. This patch makes it possible for the
instantiating code to properly set the return value of this method.

Change-Id: I7b7be24f5630a48d2a561e4fda9cec696a8cacf9

Added documentation for HTMLFormField's "hide-if"

Change-Id: I410c58dc0586cfe97e39668c9608ce0d9e10a08b

Reapply f45be1f78b8462a62d77893fd7ac15757b049381

Accidentally reverted in 8a157f34c83a69c0b93be1a009f814179f4ff5ab.

Change-Id: I027f71961672936dc8c7d51b5e6313215748c21b

Merge "Use getActionName() static"

Use getActionName() static

Change-Id: I3225ecd2b8e4294c28ea0aa664f0246d0d028b11

Better information about downloading skins via Git

Rather than futilely try to explain Git in a single sentence, just link to
<https://www.mediawiki.org/wiki/Download_from_Git#Using_Git_to_download_MediaWiki_skins>.

Change-Id: Ib4323fa04309b80e3197834bdde48d4022458488

Localisation updates from https://translatewiki.net.

Change-Id: If429c59c6cc6733a9fc4fa92bb3f18f5f045fd7e

Merge "Support changing icon variants on hover"

Added ObjectCache::getMainStashInstance() and $wgMainStash

* This provides factory/config more appropriate for stashes
* Cleaned up some neighboring config comments while at it

Bug: T88493
Bug: T97620
Change-Id: Id370ee50be6e493a4700c858df863a183abc05fd

Merge "Better message wording."

Use lazyPush() instead of push() in a few places for jobs

Change-Id: I56661ea29988e4ee217a63f4c3ffcb78333cc454

Added JobQueueGroup::lazyPush method

* Made use of this in triggerOpportunisticLinksUpdate()
* This will defer and better batch job insertion
* Lazy job insertion and other deferred updates
  make use of register_postsend_function if present
* Also cleaned up some return types and exceptions
  in JobQueueGroup

Bug: T99302
Change-Id: I3a3968d75cb37563f970be08e63f31a090e0e037

Merge "API Import: Die when namespace and rootpage is set"

Merge "registration: Remove duplicate "ParserTestFiles" key from schema"

Merge "Set correct default date format for ce"

Merge "Converted FileBackendStore to using the WAN cache"

Chinese Conversion Table Update 2015-3

Update the Chinese conversion table routinely to fix bugs reported from
https://zh.wikipedia.org/wiki/Wikipedia:%E5%AD%97%E8%AF%8D%E8%BD%AC%E6%8D%A2/%E4%BF%AE%E5%A4%8D%E8%AF%B7%E6%B1%82.

It is only data changes and only works for Chinese WikiProjects.

Change-Id: I8c9a52a3c03031330b8be771f3c74e6007efab73

Merge "Converted User object cache to the WAN cache"

registration: Use a static whitelist of keys that are not attributes

Rather than continuously creating an array of keys that were used, just
hardcode a list of keys that aren't attributes. This also handles keys
like "AutoloadClasses" that is handled by the registry.

Bug: T98977
Change-Id: I35088a76ff4c58db71e8c9bc139fa0ccca738e3b

Merge "Cleanups to WANObjectCache::getWithSetCallback code"

Merge "Reduced the rate of RecentChangesUpdateJob insertion"