mw.feedback: Use standard OOjs UI dialog error handling

Previously, we'd close the original dialog on failure and display a
new one, which had the unfortunate effect of losing the user's input
(which presumably was not saved due to the failure).

Bug: T147056
Change-Id: I8893658807f6a584d6db0a84bc508d87ef508c7e

Merge "Use MAX_LOCK_TTL in RedisLockManager to avoid premature lock expiration"

Merge "Cleanup MemcLockManager and move it to /libs"

Pass interwiki option to SearchEngine

Currently interwiki result are always computed even if the client does
not request them.
Pass the option via featureData to allow SearchEngine not to run
interwiki searches when not needed.

Bug: T147003
Change-Id: I55067bc7e5d926a3e37f4d24fb8e085bd252c376

Merge "Move ChronologyProtector up a directory instead of being by itself"

Use MAX_LOCK_TTL in RedisLockManager to avoid premature lock expiration

App servers should not lower the redis structure TTL of locks set
from CLI scripts, which is what would typically happen before if
they touched the same keys. Instead, keep a high lock record TTL
and rely on the logical TTLs for expiry semantics.

Change-Id: I2948369845c47d2682a85dd0564673f3b813be83

Cleanup MemcLockManager and move it to /libs

* Remove wf* function and ObjectCache dependencies.
* Use the base class session field.
* Lower physical lock structure TTL and move it to the base
  class as a constant.
* Resolve TODO about acquiring mixed lock types in one pass.
* Only mark servers down for 30 seconds in case of long-running
  scripts.

Change-Id: Icd4be407e599524cc620975d27e85666d2532b95

Merge "Remove test for <math> which fails when Extension:Math is installed"

Merge "Don't skip tests which lack non-tidy output"

Merge "JpegMetadataExtractor: Allow empty segments"

Move ChronologyProtector up a directory instead of being by itself

Change-Id: Ie4859a24fb22e5816718a7d1d9444316b1f444c4

MovePage: Check ContentHandler::canBeUsedOn()

If the old content model cannot be used on the new page title, the page
move should be disallowed.

Change-Id: I6078a4cde5a76a050dddedf59634d800765cbabf

JpegMetadataExtractor: Allow empty segments

A segment which consists only of the marker and length value
(equal to 2, the length of the value itself) appears to be
perfectly valid, and is ignored by every image viewer I tested.

Bug: T147015
Change-Id: I3124c587ccb1c457df25fd5bf7a47feab9312a38

Merge "Make multi-write backend "template" config work as expected"

Localisation updates from https://translatewiki.net.

Change-Id: Id78eb74f69b0450cdf09a39a41c057b888c143fa

Make multi-write backend "template" config work as expected

* FileBackendGroup now applies the usual config for these sub-backends,
  the same config get() applies.
* Also make sure FileBackendMultiWrite::concatenate() uses the status wrapper.

Bug: T146904
Change-Id: I1e9b5027dbac11ea9484cd16851e5db998574429

Merge "Do not run exact db match if offset is > 0"

RELEASE-NOTES for wikimedia/scoped-callback

Change-Id: I197028956f2dc7f0b1edcbcf4ba02687440309a7

Use wikimedia/scoped-callback

The ScopedCallback class was moved into a separate library. This updates
all callers to use the namespaced version, and provides a
backwards-compatibility class wrapper under the old name.

Bug: T146258
Change-Id: I2dd0a66fe2f510f26bdfef6b0a975c1beb3fd93c
Depends-On: Iea0c40bdd7776372ccf72db8f088a2abaa4d3721

Merge "Move DatabaseBase to a class alias for type-hints"

Make WikiPage::getAutosummary emit deprecation warnings.

Bug: T146164
Change-Id: Ibe76af8af3686c028e54923b9ce01b93af3032ab

Revert "Add the main stash, WAN, and server caches to MediaWikiServices"

 $ php maintenance/eval.php
 > $wgMessageCacheType = CACHE_ACCEL;

 > wfGetMessageCacheStorage();
 PHP Warning:  Illegal offset type in isset or empty in includes/objectcache/ObjectCache.php on line 93
 PHP Warning:  Illegal offset type in isset or empty in includes/objectcache/ObjectCache.php on line 125
 PHP Notice:  Array to string conversion in includes/objectcache/ObjectCache.php on line 133
 Caught exception InvalidArgumentException: Invalid object cache type "Array" requested. It is not present in $wgObjectCaches.
 #0 includes/objectcache/ObjectCache.php(94): ObjectCache::newFromId(Array)
 #1 includes/objectcache/ObjectCache.php(287): ObjectCache::getInstance(Array)
 #2 [internal function]: ObjectCache::getLocalServerInstance(Array)
 #3 includes/objectcache/ObjectCache.php(181): call_user_func('ObjectCache::ge...', Array)
 #4 includes/objectcache/ObjectCache.php(137): ObjectCache::newFromParams(Array)
 #5 includes/objectcache/ObjectCache.php(94): ObjectCache::newFromId(3)
 #6 includes/GlobalFunctions.php(3430): ObjectCache::getInstance(3)
 #7 maintenance/eval.php(78) : eval()'d code(1): wfGetMessageCacheStorage()
 #8 maintenance/eval.php(78): eval()
 #9 {main}

This reverts commit 6d99fa58247a1de7eaf9a79a061f0b4207154072.

Change-Id: I4d756eb6f66318a12d8cbbc1ba546397be8a29bf

Merge "Emit deprecation warnings for Article::doEdit and WikiPage::doEdit"

Move DatabaseBase to a class alias for type-hints

Change-Id: Ib6bbbd751358cbc2f2d2767c80721d3d8f142c0d

Merge "registration: Improve license-name validation"

Merge "Replace DatabaseBase => Database in more places"

Merge "Make LBFactory return type docs more specific to avoid IDEA warnings"

Remove test for <math> which fails when Extension:Math is installed

And add a test for <mw:toc>

C. Scott asked for this test to be added in CR on
Ib48b91238c3eddd6a86b62, but it was never actually executed prior to
Ic840678b851d6fc472b834 due to a TestFileReader bug. The nuisance it
causes now becomes apparent.

As was noted at the time, it is unnecessary because Tidy is not a
sanitization pass, <math> is not whitelisted in the Sanitizer. C. Scott
said "I was just concerned that the particular means used to hide the
mathml from tidy might also hide it from the sanitizer." I don't think
that's plausible, but we can test for it anyway by trying to pass
<mw:toc> elements through the Sanitizer.

Change-Id: I5928e6f31d8b80ef384a1c0e6bacfa789cac9770

Don't skip tests which lack non-tidy output

Previously some 60 tests which only had tidy output were silently
removed from the test list and were not executed. Perhaps miraculously,
they all pass (if Math is disabled).

Also, throw an error if a test lacks any kind of result section. Fix one
test which accidentally lacked a result section.

Change-Id: Ic840678b851d6fc472b834cde3d8c83359f1c272

Merge "Add the main stash, WAN, and server caches to MediaWikiServices"

Merge "Disallow css attr() with url type"

Merge "API: Add generator non-continuation data"

Replace DatabaseBase => Database in more places

Change-Id: If37a7909056bf2c31a8228cbc84f0fbbf5f1c517

Make LBFactory return type docs more specific to avoid IDEA warnings

Change-Id: I5fa6ed87e35f3b8ee17e576faf5d00763229dd4b

Merge "resourceloader: Disable ChronologyProtector to speed up wfGetLB()"

Add the main stash, WAN, and server caches to MediaWikiServices

Also removed unused getLocalServerInstance() b/c $fallback logic.

Change-Id: Ifa5f798de10783741a7b079f22d283bb9cb7f4c0

Merge "rdbms: Lazy-init DatabaseDomain::getId()"

rdbms: Lazy-init DatabaseDomain::getId()

Follows-up 847b91b.

Noticed that in Xenon profiles, about 25% of Database::__construct
time is spent in array_map, via DatabaseDomain::convertToString.

* Database::__construct
  * DatabaseDomain::__construct/convertToString/array_map
  * DatabaseDomain::newUnspecified
    * DatabaseDomain::__construct/convertToString/array_map
  * DatabaseMysqlBase::open
    * mysqli::*

Change-Id: Ib5ae7d1d03b27df504a7e5f552e36a0ca6fcf97e

Merge "Parser: Allow `<s>` and `<strike>` in table of contents"

Merge "Fix use of bogus $params var in FileBackend::__construct()"

Merge "resourceloader: Avoid needless GPC normalisation"

Parser: Allow `<s>` and `<strike>` in table of contents

Bug: T35715
Change-Id: Iec6a05e3e6bb622f477e6ebeb57e9f65da5f22bd

Fix use of bogus $params var in FileBackend::__construct()

Follows-up 81a0e79, 24aa72de, dc522cf0.

Also set missing section name parameter in scopedProfileSection().

Bug: T146904
Change-Id: I897c21b766bfea4e4cb83c63f0e40e945bd73747

registration: Improve license-name validation

Our hardcoded enum list in the extension.json schema for license-name
values was incomplete and did not cover the full SPDX license identifier
specification, which includes things like "AND" for specifying multiple
licenses.

Composer already has solid code in a library to do this validation, so
let's use it! This updates both the validateRegistrationFile.php and
ExtensionJsonValidationTest.php to use the composer/spdx-licenses
library (a new development dependency) to ensure the license-name field
is a valid SPDX identifier.

Also fix a silly typo in the validateRegistrationFile script which
prevented it from running, and use ::class so it will be easier to
detect typos like that in the future.

Bug: T146862
Change-Id: Ibb8973ed7950ae81c90558f9630f73746b2aff2c

resourceloader: Disable ChronologyProtector to speed up wfGetLB()

For load.php requests, Xenon in Wikimedia production measures that half
the time for wfGetLB() is spent in ChronologyProtector::initPositions,
and its query to Memcached.

Change-Id: I24f90897e56fd679c6bd1acf764722ed58e51c67

Merge "Use classes to apply the 'editfont' preference"

Localisation updates from https://translatewiki.net.

Change-Id: Ia03a6db5b325c673887b1fc30ecc72e2312af23d

mediawiki.UI: Align buttons to design guidelines and OOjs UI

Aligning buttons to design guidelines in https://phabricator.wikimedia.org/M101
and to OOjs UI, similar to https://phabricator.wikimedia.org/rGOJUed9fef533bea
Additionally:
- aligning CSS code to MediaWiki Coding Guidelines,
- adding `transition`s equivalent to OOjs UI,
- replacing deprecated `@colorWhite` variable,
- replacing counter-intuitive `.button-colors()` Less mixin guard, given
our updated widget guidelines has only two primary button types
(`progressive` & `destructive`) left, with separate mixins and
- removing deprecated `constructive` CSS standalone selector and
corresponding KSS styleguide comments.

Bug: T146823
Change-Id: I6b9faefabc62689eabf6f4bccaa3d3756b5b662a

Merge "Use HTTPS instead of HTTP for ietf.org URLs"

Merge "Clean up use of UTSysop in TemporaryPasswordPrimaryAuthenticationProviderTest"

Merge "Avoid DBPerformance warnings in ApiClearHasMsg"

resourceloader: Avoid needless GPC normalisation

Utility methods like getFuzzyBool already use getRawVal instead
of getVal (see 83df6d8). But there was still direct use of getVal
which meant we still pay for WebRequest normalisation in load.php.

ResourceLoaderContext::__construct's use of WebRequest::getVal
accounts for 0.12% of execution time according to a Xenon daily log
(2016-09-27.all.reversed.svgz). The total use of WebRequest::getVal
is 0.26% of execution time. So almost half of that comes from RL.

Change-Id: Id37619ceadb5c990abf83f7910104cbc95523d28

Use HTTPS instead of HTTP for ietf.org URLs

http://tools.ietf.org/ redirects to https://tools.ietf.org/

Change-Id: Ia45d2ce80399e1c9172e0924abd6bf794a53122f

Clean up use of UTSysop in TemporaryPasswordPrimaryAuthenticationProviderTest

No idea what's causing T146498, but maybe this cleanup will fix it.

Note we can't clean up every use because we can't create test users from
data providers.

Bug: T146498
Change-Id: I5c7ea7f49ed42cb2ed36460afcf6aec30e9359de

Fix display of categories

Fixes regression from 1885a1ac462c08, which accidentally removed
page_namespace and page_title as select fields.

Bug: T146873
Change-Id: I19cff0cf5bc015da7311196749a9201a8e458b49

Merge "Avoid DBPerformance log warnings in SpecialEditWatchlist"

Merge "Fix "Incorrect watchlist token provided" unit test errors"

Merge "resourceloader: Fix undefined getPrefixedText()"

Avoid DBPerformance log warnings in SpecialEditWatchlist

Change-Id: I97b31f5e805e9e3e4192d15195ff26ec416ad6b2

resourceloader: Fix undefined getPrefixedText()

Follows-up dbe592df. Used to go in one-way only and silenced by
addObj(). Now that we're iterating directly as well, need a
conditional.

Bug: T146728
Change-Id: Ief9d44cc5cb8f5d687560f59cc696c5671546163

Fix "Incorrect watchlist token provided" unit test errors

Change-Id: Ib133ffe9b9ef438ff5716ad25a897e2c5a4f7d58

Merge "Add @deprecated annotations to NS_IMAGE and NS_IMAGE_TALK"

Add @deprecated annotations to NS_IMAGE and NS_IMAGE_TALK

Hopefully this will discourage people from using them in new code.

Change-Id: I165a19406dfc0bcc471252e416480fa3af92d969

Merge "Set ignore_user_abort( true ) during DB commit for sanity"

Merge "Use getConnectionRef in WatchedItemStore"

Use getConnectionRef in WatchedItemStore

Change-Id: Ica6fce8aeec08ace34cd23d3bb0390a859175e71

Merge "Remove full form of ChangeTags::buildTagFilterSelector"

Merge "Create ChangeTagsUpdate hook"

Create ChangeTagsUpdate hook

This creates a hook triggered when updating change tags, so that
extensions can take actions when tags are updated.
When tagging accompanies the action, the recent change is passed
while when tagging is subsequent to the action, the user who
performs the tagging is passed.

Bug: T118698
Change-Id: Ifb0cdc43252c4185e4f216d23b8a21fb31595a37

Lower $wgTrxProfilerLimits for JobRunner a bit

Jobs should always use query batching.

Change-Id: Iddbcc32b44e6c71feb66e2f52afb20e824e6eb72

mediawiki.UI: Align further greys to improved color palette

Aligning more of the base grey shades to improved color palette
https://phabricator.wikimedia.org/M82

Bug: T146799
Change-Id: I40d69e5fdb5456511be4884f947635179aaf1020

Make RecentChange::addTags() accept a string

Most planned callers of RecentChange::addTags() only want to add a
single tag, so forcing them to create an array seems unfriendly. Plus
this lets us avoid a array_merge call.

Change-Id: If001ba9ff01a33be158e7edd5c5e4de9825fa180

Use classes to apply the 'editfont' preference

This class can be re-used on non textarea inputs
(e.g. ContentEditable or <input>).

The 'default' preference can't be achieved in CSS alone, so
for non-textarea cases, use 'monospace' as is used in the majority of
browsers/OSes. In future a client script could compute the default
font of textarea and add it to the -default rule.

Bug: T145015
Change-Id: I5f0433e9cae6f692396537412c11b79b61db062f

Emit deprecation warnings for Article::doEdit and WikiPage::doEdit

Bug: T145736
Change-Id: I637daea7412d8acb8561f9a821af1882e49ddd9b

Localisation updates from https://translatewiki.net.

Change-Id: Ieda6cba6bfc651c153cf87c9e89c1d3a8ef7f7d0

Merge "Directly use Language::factory instead of wfGetLangObj"

Merge "Avoid multiple writes to changetags table in recentchanges_save hook"

Avoid multiple writes to changetags table in recentchanges_save hook

Currently, extensions that add tags within the recentchanges_save
hook directly call ChangeTags::addTags. This often results in
consecutive writes to the changetags table.
This provides a addTags method to RecentChange so extensions
can simply use it when they want a tag added. And all the actual
tagging is done once at the end.

Change-Id: I8df2fd983c12632337e8d2922fa357808482338c

Do not run exact db match if offset is > 0

When scrolling results on prefix search api the exact Title
match is always at pos 0 even if we want to scroll the results
by setting offset to > 0.

Change-Id: Ib02c9d3e479d739e6fe79014d962db50b6fd9de8

Merge "Remove getSearchEngine() from DatabaseBase"

Merge "Create fields & data for image/file data indexing"

Create fields & data for image/file data indexing

Bug: T145558
Change-Id: I23d4c8235d0e4150eefec31cea4b2cfdd32bf32a

Merge "Clean up array() syntax in docs, part VII"

Merge "Replace DatabaseBase:: with Database:: and update type hints"

Replace DatabaseBase:: with Database:: and update type hints

Change-Id: I3919b04eb2de4fa0bf8a02239fb5bbf17d347511

Merge "EditPage: Allow language of copyright warning to be specified"

EditPage: Allow language of copyright warning to be specified

This avoids global state, and is necessary in places where the
global user language cannot be used, such as load.php.

Bug: T146686
Change-Id: I80a813a76b551643b68430e6a0ab553b2e1dfb11

Clean up array() syntax in docs, part VII

Last part

Change-Id: I38f015e2122ef4fd2d2141718bd889794c29f06c

Merge "Remove a few unused variables"

Merge "Remove/actualize unused imports"

Remove a few unused variables

Change-Id: Ibfc4c6cbbc08b5917f1a84d86d2d4a0855e371a1

Remove/actualize unused imports

Change-Id: I6ef19d5d982aa45dbf5554107ad9ee720442f466

Merge "Fix function name case"

Fix function name case

Change-Id: Ibd4f682d2ed8500a50d85aae38f17281646f7c2d

Merge "AuthManager phpdoc cleanup"

AuthManager phpdoc cleanup

Change-Id: Ib6fa897e76ae479de68acaeb16708c2bdf15ae5f

Localisation updates from https://translatewiki.net.

Change-Id: Id7ff13898d6a89d8ce88794666e940d7a6d94596

Merge "Update wfEscapeShellArg() comments"

Merge "Minor installer/upgrader cleanups"

Merge "Change mid-upload 'cancel' button to 'back'"