mediawiki.special.upload: Don't fail when invalid title is given

If an invalid title was given (for example, anything including '['),
the destination title check would throw an exception and a spinner
would spin forever.

Change-Id: If6f02051383675817a0e2ad406dd71cab33fb5bf

Merge "SpecialRunJobs: delegate error handling to MWExceptionHandler"

Merge "Fix UID test and make debugging easier"

SpecialRunJobs: delegate error handling to MWExceptionHandler

Make the custom error handler that is installed to suppress Cookie
header errors delegate unhandled errors to the default MediaWiki error
handler. This will preserve fatal error logging for errors raised by
jobs.

Bug: T89169
Change-Id: I52ef544fa18351bf310b04b17ab4e0239020fe20

Localisation updates from https://translatewiki.net.

Change-Id: I7264e3c32d5a088999af65c1e597a2ed6a400e27

Merge "Follow-up Idb1202579: Add special-characters-group-greekextended to RL module"

Follow-up Idb1202579: Add special-characters-group-greekextended to RL module

Bug: T130535
Change-Id: I3e26d08a34283eaafe129dbef0944ea10c086162

Remove unused import in WatchedItemStoreUnitTest

Change-Id: I56abf221e6e601ad0bceaae1a15eb8f1097f5baa

Merge "Use WatchedItemStore for SP:RC watchers count"

Merge "CatWatch don't show the number of pages transcluded"

Use WatchedItemStore for SP:RC watchers count

Bug: T132568
Change-Id: Ic6a0ee77cb767c6f8e4a8d359feb0405c9a990d1

CatWatch don't show the number of pages transcluded

Providing the perfectly correct number of pages
that have acctually been added to or removed from a
category is extermly hard.

Rather than providing data that is most likely wrong
and a bit useless ONLY provide a link to Special:WhatLinksHere.

This touches on the following bugs and may mean that
some of them don't need to be thought about any further.

Bug: T126855
Bug: T126407
Bug: T126139
Change-Id: Ida06d822d1955091595c17c9c6c2968a40a93bcd

Allow description text caching for ForeignDBFile

A large amount of real time is spent in fetching Commons
descriptions pages from wikipedias, according to xenon
flame graphs.

Change-Id: I3935bf8bf23da58b734f897d7a1979e9b2f7e5c8

Merge "Add a rewind function to SearchResultSet"

Add a rewind function to SearchResultSet

This is implemented in CirrusSearch so the result set can be looped
through multiple times. In general having a rewind function on any
iterator is probably a good idea, so pushing it into the upstream
interface.

Change-Id: Iaac9c11d8742288610011a2a0f6282944d35d5f3

Localisation updates from https://translatewiki.net.

Change-Id: If98c4a1a477e70076aa3ebf8e12a64b5b111cafc

Merge "mediawiki.api: Log warning when requesting legacy token types"

Revert "Hack IEUrlExtension::haveUndecodedRequestUri() to always return true"

This reverts commit 30c5c55bd28f2fe35867286b73100f42f829c448.

Change-Id: I219ae8db4f730f6ff19e395976ab08d7b9cfc50c

Hack IEUrlExtension::haveUndecodedRequestUri() to always return true

Interacting badly with I1f108eb74. Will revert this shortly.

Change-Id: I58259aa7ccf00a07d19db6e6e23df48f0b78103c

mediawiki.api: Log warning when requesting legacy token types

Change-Id: Ia2c795a95b58f62f4687501a5537e5890d90144e

Merge "Track which web request created a job"

Track which web request created a job

We currently push a request id into structured logging (monolog/
logstash) to allow seeing all logs that were triggered by the same
request. This extends that to pass the id through jobs so jobs triggered
by a web request also share the same id and can be tracked together.
This web request id will follow jobs both directly created by a request,
and jobs created by those jobs.

This should give us some more visibility when debugging into what
started a particular job, and if a large number of jobs blowing up the
job queue are somehow related.

Change-Id: Iedbd031e6e9bb18fd6f7b923c8c305102255ab4b

resourceloader: Implement modern module loading (2/2)

* Send 'module' and 'require' parameters to module closures.
  This depends on Ia925844cc22f143 being deployed one cycle earlier.
* Patch Moment and OOjs to ensure these libraries continue to expose
  their module as globals as well. AMD/UMD-compatible libraries
  only expose a global *OR* an export, not both. We need both
  for back-compat.
* Update pluralRuleParser to make use of module export to allow
  usage via require().

To test, check out the patch and run:
> mw.loader.load('moment');
> mw.loader.require('moment')()
> mw.loader.require('moment')('2011-04-01').fromNow()

Bug: T108655
Change-Id: Idbd054880ee70d659ec760aef8fcb38d0704a394

Merge "Use english messages for background use of Status::getWikiText"

Switch to external HtmlFormatter

wikimedia/html-formatter is already present in mediawiki/vendor
as of 3954ca36ce3cbedc76c1763ad2694320c1327ce6.

Bug: T125001
Change-Id: Ie98096e5e3d325cde583bc66b21b8b41f2bba8b2

Merge "Convert File::getDescriptionText() to getWithSetCallback()"

Convert File::getDescriptionText() to getWithSetCallback()

Change-Id: Ia27d6fc851598162367ad6e749feb6d76f2569bc

Merge "OOjs UI: Apply texfield -> textfield typo fix on the library build"

OOjs UI: Apply texfield -> textfield typo fix on the library build

Broken in I7212e07, this was fixed in I7922121 which will be part of
OOjs UI v0.16.6 in a couple of weeks' time. For now, let's apply it
as a local patch to unbreak the Search form. Also, whoops.

Change-Id: Ie882ae6ee823d6cbcfc899e336c1170ef6f9f092

Merge "Bump grunt to 1.0.1, remove grunt-cli"

Merge "More FSFile docs"

Merge "filebackend: Make resyncFiles() easier to use"

Merge "virtualrest: Small doc fix"

Merge "Revert "Measure commitMasterChanges() run time""

filebackend: Make resyncFiles() easier to use

Pass in the resync mode as a flag instead of using
the object instance.

Change-Id: I2da661a65afd712403db936b6e31fba8e8069096

More FSFile docs

Change-Id: I1ca3e0bc9d5db4ca043f313fb15a780ae102c50a

virtualrest: Small doc fix

Change-Id: Ie162eb88b4afc492b10c75f00b9725767e21ffeb

Localisation updates from https://translatewiki.net.

Change-Id: I95fb9e364f3268eddab39b9abe4c68c62a1ec11a

Merge "Don't report image cache hits / misses"

Revert "Measure commitMasterChanges() run time"

This reverts commit efb441883937. This is the biggest metric
by statsd traffic volume on the Wikimedia cluster. Please re-introduce this
only with sampling.

Bug: T101141
Change-Id: I7bc3b3aaa98dcefe1a80a4f458a83232c29a27ea

Merge "Do not validate relevant user name"

Don't report image cache hits / misses

This single metric is responsible for a substantial proportion of overall
statsd traffic on the Wikimedia cluster, per the task cited below. Nobody is
closely monitoring this statistic. We can reintroduce it with sampling at a
later date if it is deemed to be useful.

Bug: T101141
Change-Id: I6457cb91d527ce42c0918ac4e80613e6b59061ac

Use english messages for background use of Status::getWikiText

Status::getWikiText is used for internal logging, api error messages and
maintenance scripts. All this places are usually in english, so pass an
english language to getWikiText.

Change-Id: I3010fca8eb5740a3a851c55a8b12e171714c78f7

Merge "Add preference for watching uploaded files"

Fix core DB data in unit testing

Iec4ed4c8 made it so addCoreDBData() only gets called once. Which is
nice, except for the fact that it means any test that puts
'user_groups' into $this->tablesUsed is going to cause UTSysop to no
longer be a sysop, so any later test that expects it to be one is going
to mysteriously fail.

The fix is to make resetDB() actually clear out the 'user' table when
requested, but then to re-call addCoreDBData() so UTSysop is still
there.

This might break extension tests if they're relying on the 'user' table
never being cleared even if it's specified in $this->tablesUsed, but
hopefully that's not often the case.

Bug: T132411
Change-Id: If251739fd486544f54a0c07edcc24aeef0998342
Depends-On: I0546a4f18e0751d209b6b7c6cd42973f25828313
Depends-On: I4924ae941b3844b39dd3f44c6986c3bf29b0d62a

Merge "Reduce code duplication in MessageCache"

Reduce code duplication in MessageCache

Change-Id: I3432958c8f81b7a33079b7933e85b87ced7363fa

Flag triggerOpportunisticLinksUpdate() behind $wgMiserMode

This has caused job flood or DB load problems too often.

Bug: T132318
Change-Id: Ieeaf047e748bdfb980d4d5511c22890bd6f28c67

Cache first-letter data in APC, if available

I noticed that `frwiki:first-letters:fr:fr:4.8.1.1` was at the very top of keys
sorted by bandwidth (that is, reqs/sec * size) on one of the memcache servers
on WMF prod.

The data takes ~60 - 80ms to compute, in case of a cache miss. That's not
enough to justify using a tiered cache abstraction here, IMO.

Change-Id: If81ce8f86f2c378565f1f6a0dd2c04dee825c4e9

Merge "Convert Special:Search input to OOUI"

Increase triggerOpportunisticLinksUpdate() backoff TTL

Make use of the expected TTL for the links to become
possible stale, instead of the hard-coded value of
60 seconds. Use a minimum of 3600.

Heavily used Commons templates, for example, lower
the parser cache TTL to 86400. Enqueuing jobs before
that time passes is a waste.

Bug: T132318
Change-Id: I9a3c088db9dd1747b8e99ccf7ec4633e7f01fe7b

Merge "Thumbnails: allow overflow inside thumbnails"

Merge "jquery.makeCollapsible: Don't ignore clicks on fake links without href"

Revert "Allow reset of global services."

Completely breaks login.

This reverts commit 8e7a0a0912bb98a4a12375b354e23f03262bf213.

Change-Id: Ide7ab5632e987e81374c21173df6ab3998649df7

Merge "Revert "Make WatchedItemStore use MediaWikiServices""

Revert "Make WatchedItemStore use MediaWikiServices"

In order to cleanly revert "Allow reset of global services"
which breaks login.

This reverts commit 99e8d45b50993113b26ced7c8bb91e44c7d908b1.

Change-Id: Iae7f7b7b7083345dd50c313ee68e2e814f130f1a

Merge "Fix RebuildLocalisationCache bug from MediaWikiServices"

Localisation updates from https://translatewiki.net.

Change-Id: I065f082606ccb1d6df353f503ee8f393d8d6466b

Fix RebuildLocalisationCache bug from MediaWikiServices

Revert changes to rebuildLocalisationCache.php introduced in Ie98bf5a.
The forked children do not need separate service connections and the
reset process fails due to MW_SERVICE_BOOTSTRAP_COMPLETE being defined.

Bug: T132370
Change-Id: I0d7ddcd66e7c6945fabd91fea29a48805bf14793

Merge "Add templateOverridesBySection to multi LBFactory"

Merge "Normalize `input[type="search"]`"

Merge "RightsLogFormatter: Use DB key to generate foreign user link"

Merge "Finish stash uploads with upload dialog"

Run generateLocalAutoload.php

Ie98bf5af5 was apparently done by hand.

Change-Id: I323391340dbcc2f25bc849bbbb489142e79bb5ed

Make WatchedItemStore use MediaWikiServices

Change-Id: I5c8c1a652a35e055d8d65753a60c0d1684e1c37d

Allow reset of global services.

This change provides a mechanism to reset global service instances
in an orderly manner. There are three use cases for this:

* the installation process
* forking processes
* integration tests (which must of the existing phpunit tests are)

Depends-On: I5d638ad415fc3840186a0beaa09ac02ea688539b
Change-Id: Ie98bf5af59208f186dba59a9e971c72ea0b63e69

Add missing use MWException to SessionManager

Change-Id: I098e719cd888400191438bb0c4deeaaff1aa32dd

Thumbnails: allow overflow inside thumbnails

overflow:hidden on the thumbnail has been in place like this for a
long time. One of the primary reasons was for long sentences in the
thumbcaption. For that, modern browsers now support word-wrap.
I know of no good reasons why overflow would have to apply to the
rest of the frame. Therfor moved to thumbcaption only, and added
word-wrap.

Bug: T132306
Change-Id: I1953683fa2a6d22ed82bac1acd37e5cb12636381

Merge "Swap exif-pixelydimension and exif-pixelxdimension messages"

Localisation updates from https://translatewiki.net.

Change-Id: I01a967bb1fc77fa9a4f8dd9c8bab4e238b682cd5

Merge "User.php: Update 'setEmailWithConfirmation' for notification email"

Normalize `input[type="search"]`

Limiting property `-webkit-appearance` to `input[type="search"]` only,
as it's the element this rule is targeting for instead of unnecessarily
applying it to all `.mw-ui-input` elements. Also adding rule to
remove proprietary IE 10-11 clear button. Bringing it on
par with OOjs UI.

Change-Id: Id375fdb86bf7ef3cfbb2f7e198d59be4947e4edb

Do not validate relevant user name

- This makes Special:Log's behaviour consistent with other special
  pages, e.g. Special:Contributions.

Change-Id: Ic0a9322d920d7514bacac1f29a1da4fe57cae854

Merge "Add note that IP::isInRange() can return unexpected results for invalid args"

Merge "Show absolute block expiry in user timezone on block logs"

Show absolute block expiry in user timezone on block logs

For this add an user parameter to Language::translateBlockExpiry.
This allows the function to display the absolute block expiry in the
user's time zone. Use this when formatting block log entries.

This also avoids the use of $wgUser

Bug: T131241
Change-Id: If0a1d3c88bb4242a016eb9b2df115413de786149

Merge "Switch Special:FileDuplicateSearch to OOUI"

Merge "SpecialLog: Don't put assignment within another expression"

Merge "Add possibility to filter for log_action in all core logs"

Make some TablePager methods actually protected

* getTableClass
* getNavClass
* getSortHeaderClass

Change-Id: I459ca456149635e3300c7ca7cfed4c1edc1c12b1

Merge "Run generateLocalAutoload.php"

Merge "Use pg_result_error, not pg_last_error, as the latter gives false negatives."

Merge "Clean up after "Kill mbstring fallbacks""

Add preference for watching uploaded files

Adds a preference in the Watchlist section for watching uploaded files
for an account. Also works from API-based upload methods, so
UploadWizard and other tools should work fine.

Bug: T33313
Change-Id: If962e667de12b35904b2d1b2d9e99c26b588ec2a

Finish stash uploads with upload dialog

Adds a 'filekey' option to the upload dialog's booklet layout
so it's possible to complete a stashed upload with the dialog,
even if the dialog didn't stash the file.

Proof of concept gadget/userscript for testing purposes:

https://phabricator.wikimedia.org/P2783

Or use this ImageTweaks patch to test:

I8f8421697a6d44ec47b66496ad9ada548c4a7d0b

Change-Id: I2cdea08a29cd481eb7fe5cbd83b9e4b6941a6380

Localisation updates from https://translatewiki.net.

Change-Id: I3bd0664b4ce71182be57e062204c0ae039fd8f84

Merge "Change \\ to \ in phpdoc"

Change \\ to \ in phpdoc

Remove "\\" in namespacing. This is a Doxygen compatibility hack but
does not seem needed anymore, Doxygen reads namespaced class names
correctly, see e.g. https://doc.wikimedia.org/mediawiki-core/master/php/classMediaWiki_1_1Services_1_1ServiceContainer.html

PHP IDEs, on the other hand, were broken by the double backslash.

As an unrelated small doc fix, add parameter docs to PermissionError
constructor (parent has different arguments so the inherited
documentation is wrong).

Change-Id: I6da0f512b8c84f65fd20e90e4617108fe6a8fcd2

Fix mbstring check in update.php

Maintenance::error() will not cause the script to exit unless the
$die argument, when cast as an integer, is greater than 0. Since
the default value is 0, specify a different one, as in the other
calls from within the same function.

Follows-up 943563062f0a6995.

Change-Id: I3af72b1cd87c63d7f57436866633e86c272e904f

Merge "SkinTemplate.php: Add missing @param documentation to wrapHTML"

Localisation updates from https://translatewiki.net.

Change-Id: Ib3d0e6a16e61f0443aaa7e929d8bf66c45f5d4df

SkinTemplate.php: Add missing @param documentation to wrapHTML

Follows-up to: cab0275c7

Change-Id: Ie40229e92c0ce93628811ed7da4224ea7c15e72a

Merge "Ignore DBPerformance warnings for ?action=purge for users with "purge""

Run generateLocalAutoload.php

Change-Id: I618f831f5c33d0163b873dd0494cc37799450001

Update OOjs UI to v0.16.5

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

Change-Id: Ib5b0dc12259ac85ff1589c0e1507238f1089da3f

Merge "Revert "Split ClassCollector class into own file""

Clean up after "Kill mbstring fallbacks"

* Removed fallback code from Language, the associated data file
  (Utf8Case.ser), and the code to generate that data file.
* Removed comment in LanguageFi that "mb_substr has a compatibility
  function in GlobalFunctions.php".
* Removed check for mbstring in bench_utf8_title_check.php.
* In the tests for StringUtils::isUtf8():
  * Removed separate test for the non-mbstring code path.
  * Removed mentions of mbstring from function names and assertion
    messages, since mb_check_encoding() is now always used.
* Also updated the comment in StringUtils::isUtf8() referring to
  PHP 5.3, which is no longer supported in MediaWiki, to indicate
  that the same issue also exists in old versions of HHVM. (If
  we don't have to support 3.4 or older, then the function could
  be deprecated and removed if desired.)

Follows-up 943563062f0a6995.

Change-Id: I55e5cd534b849c6ea06a7fadacbbf34a12d87ebe

Merge "Allow to pass a rev id to a log entry without making it unpatrolled"

Merge "Move action filter logic to LogPager"

Merge "Support empty explicit plural forms in JavaScript"