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

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"

Support empty explicit plural forms in JavaScript

PHP supports them as well.

Change-Id: I7f7ed0a5b3e6ccbb904cde08d8818c2a665ac52e

Merge "registration: Load extra autoload files before executing callbacks"

Explicitly require all the required PHP extensions

Change-Id: I2bca4ece0c49c3f4f802d3f8f543882ebd14d445

registration: Load extra autoload files before executing callbacks

Callbacks should be able to safely assume that anything (or at least mostly)
defined in extension registration is already loaded and processed when
the callback itself is executed. To make sure, that this applies, callbacks
should be executed after all extra autoload paths are loaded.

Bug: T131978
Change-Id: I2c6624423957a8a00523b126fa7209d9c283aa9e

Kill mbstring fallbacks

In the age when we require PHP 5.5, pretending that mbstring emulation
is not slow and silly is silly.

Bug: T129435
Change-Id: Ic8235c9da9a926df63ec7388900c44eab454eebe

Unbreak tests

Bug: T131981
Change-Id: I1078b2a9f792f95fd451da4342b6441d6fd4d892

Localisation updates from https://translatewiki.net.

Change-Id: I1d8329f703a364fbe36a51994cdc0467cdd40387

Fix phpdoc in StatsdAwareInterface

Change-Id: Ibb3cac402f2cbe723b79785f4d3bbcb9c0b10ebe

Merge "Add a hook for collapsible content"

User.php: Update 'setEmailWithConfirmation' for notification email

In order to minimise the chances of successful account cracking,
a notification email should be sent to the user on their previous
registered email address, informing them of the change in the
registered email address of their account.

Bug: T31856
Change-Id: Ib1ef76d7eb45ed0fa7ee0c2e7e39af21289367d2

Merge "mediawiki.Title: Reuse local variable fragment"

mediawiki.Title: Reuse local variable fragment

Change-Id: Ifc1153ef3246a387d2343e68e5b5f3837dd809cd

Don't replace !! in elements

 * 55313f4e almost got it right, but missed the str_replacing table
   headings.

 * Thankfully, this was doubly broken before that patch since the
   StringUtils::explodeMarkup would have skipped the || which would
   go on to be explode by table cell attribute parsing. The test case
   provided would look like,

   <table>
   <tr>
   <th class="">|">ha</div> ho
   </th></tr></table>

   Suffice it to say, noone is using this in production.

 * Note that we can't just entity encode the ! since that would break
   style attributes with !important.

 * Also note, Parsoid already gets this right.

 * Adds a StringUtils::replaceMarkup

Change-Id: Iab3ae4518fcb307b795d57eece420ba48af0a3bf

Merge "mw.Upload.BookletLayout: Handle additional warnings/errors"

Bump grunt to 1.0.1, remove grunt-cli

Change-Id: Ic04c3a0b0237c8785f54b83f1bcfb64a3978d768

Swap exif-pixelydimension and exif-pixelxdimension messages

Somehow we had width and height confused.

Bug: T107717
Change-Id: I06acb28d665302e77e6b2584910b4df001156e91

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit d206ffe4026dcbce9ef27f1546d023a7027ea7e9

Change-Id: I9f7f0ddf87bf3ee14a1929599e72c5d837e15fbf

mw.Upload.BookletLayout: Handle additional warnings/errors

Warnings:
* 'exists-normalized': Same handling as 'exists', the only difference
  is exact match vs only matching after title normalization.
* 'was-deleted': Added new l10n message 'api-error-was-deleted',
  based on 'filewasdeleted' but shorter/simpler.

Errors:
* 'protectedpage': Display 'protectedpagetext' message.

Bug: T131709
Change-Id: Ia9f65fc39d59018dc62b758f9a2ae98de7d187fe

Merge "Update monolog to 1.18.2"

Localisation updates from https://translatewiki.net.

Change-Id: I17458dc07e3b940238dc2395e2fcc08833e303ab

SpecialLog: Don't put assignment within another expression

Change-Id: I0a0cd9ee0647e8778d2e835cc02c571a83b3a8c2

Update monolog to 1.18.2

https://github.com/Seldaek/monolog/compare/1.17.2...1.18.2

Update KafkaHandlerTest.php

Depends-On: I32e9ebe0a10d97a1ccb0f4d605bd0ee9ba4e97f6
Change-Id: I35eedb6d95b68ebf90c00bb54469c9ff5731cdab

ImageListPager: Fix improper use of Title::newFromText().

The $defaultNamespace parameter to Title::newFromText() only applies if
a namespace prefix isn't included in the text. So if a username was
"Help:FooBar", the NS_USER prefix would not be applied.

Instead use Title::makeTitleSafe( NS_USER, ... ); which was designed for
this specific purpose.

Change-Id: Iafce73d352887947e61ae7c906ddf965e04fbf6f

Preprocessor: Don't allow unclosed extension tags (matching until end of input)

(Previously done in f51d0d9a819f8f1c181350ced2f015ce97985fcc and
reverted in 543f46e9c08e0ff8c5e8b4e917fcc045730ef1bc.)

I think it's saner to treat this as invalid syntax, and output the
mismatched tag code verbatim. The current behavior is particularly
annoying for <ref> tags, which often swallow everything afterwards.

This does not affect HTML tags, though. Assuming Tidy is enabled, they
are still auto-closed at the end of the page content. (For tags that
"shadow" a HTML tag name, this results in the tag being treated as a
HTML tag. This currently only affects <pre> tags: if unclosed, they
are still displayed as preformatted text, but without suppressing
wikitext formatting.)

It also does not affect <includeonly>, <noinclude> and <onlyinclude>
tags. Changing this behavior now would be too disruptive to existing
content, and is the reason why previous attempt was reverted. (They
are already special-cased enough that this isn't too weird, for example
mismatched closing tags are hidden.)

Related to T17712 and T58306. I think this brings the PHP parser closer
to Parsoid's interpretation.

It reduces performance somewhat in the worst case, though. Testing with
https://phabricator.wikimedia.org/F3245989 (a 1 MB page starting with
3000 opening tags of 15 different types), parsing time rises from
~0.2 seconds to ~1.1 seconds on my setup. We go from O(N) to O(kN),
where N is bytes of input and k is the number of types of tags present
on the page. Maximum k shouldn't exceed 30 or so in reasonable setups
(depends on installed extensions, it's 20 on English Wikipedia).

Change-Id: Ide8b034e464eefb1b7c9e2a48ed06e21a7f8d434

Improve comment to localizers in MessagesEn.php

Follow up to Ia670512e0cb375335873e7f9a08b638bbe039e45.

Bug: T116020
Change-Id: I8634585e52ce5339dbfcd5b19c5d1b6c9ef5687e