Move some more classes to comply with class per file

Change-Id: Ibecea848fcca24709847931f5e02a7c981c4d277

Merge "Add \b to regexes in BlockLevelPass to avoid confusing tr & track"

Merge "More magic word translations for Catalan (ca)"

Add release notes for T151633

This is a follow-up for f87b3b68860e.

Thanks to @Sfic to have submitted this solution.

Bug: T151633
Change-Id: Ia87d925f81be68a69b9b5f9893b7fc71afe950db

Merge "Use Devanagari digits for list items on ne.wikipedia"

Merge "Fix the web updater"

Add \b to regexes in BlockLevelPass to avoid confusing tr & track

With TimedMediaHandler in video.js mode, videos can be inline,
without a wrapper div.

Previously, in this mode two paragraphs where one contained a
video would end up merged into one paragraph, due to BlockLevelPass
matching "<track .../>" against "<tr" in its regexes.

Added \b to a couple of the regexes to protect against such errors,
and corrected a parser test case that had bad output listed, where
"<link .../>" matched against "<li".

Bug: T165817
Change-Id: I06e82b881f5ebddae5e7df7fb940adfa54f6b659

Hack around icu breakage for fa sorting

Bug: T139110
Change-Id: I35bcdaf309f595258289f01bbe5713ce6d1ffad1

Fix the web updater

It kept failing with Error: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'ESCAPE '`'' at line 1 (localhost)

Bug: T165805
Change-Id: I2671415fc34b1e6226104c88ca21fafc860bb7fb

Localisation updates from https://translatewiki.net.

Change-Id: Icd2487d8cf826e9736277723e84e2520106b2a5c

Merge "Add type hint to ThumbnailRenderJob::hitThumbUrl"

Merge "Add workaround for HHVM issue 6206"

Merge "Add missing doUpdates() call to refreshLinks.php"

Merge "Avoid postgres PrefixSearchTest failures due to collation differences"

Add type hint to ThumbnailRenderJob::hitThumbUrl

Change-Id: Ifab9d74ae9147349b0a3417f35088b96ee47b6fa

Add missing doUpdates() call to refreshLinks.php

Follows-up b8b01aa589ce which accidentally removed this call.

Bug: T157545
Change-Id: Id4efdccb60e7829269df601b83f64288f6190db4

Add workaround for HHVM issue 6206

Just use the actual global on HHVM, like we do on
Zend, but avoid hitting the warning.

This is a workaround for https://github.com/facebook/hhvm/issues/6206

Bug: T111641
Change-Id: I15ef0e90827c94a8b2609484b0dddcb78f04284c

Use Devanagari digits for list items on ne.wikipedia

Nepalese uses Devanagari, so we wish to use this alphabet for
list items' digits.

Bug: T151633
Change-Id: Id9cb42932fd8b8aa2cfb5418c4a3cd16ec37a60e

Merge "MediaSearchWidget: Only resize new results"

Merge "Improve HTTP logging"

MediaSearchWidget: Only resize new results

When repositioning images based on the "change" event,
ignore images that have already been assigned to a row.

Bug: T163727
Change-Id: Ida7556d8848f98d97d82aef9caf5c1d43bb47d7b

Merge "Tests: correct order of assertEquals arguments"

Merge "MimeAnalyzer: Detect magic bytes for mp3"

Tests: correct order of assertEquals arguments

Change-Id: Ib8d1e8aa92172a9abc0495dc0c18ea5df850dd5e

MimeAnalyzer: Detect magic bytes for mp3

Unit tests in follow up patch

Bug: T115170
Change-Id: Ie1a63aa319cebbb2efb3cdc478416f5947078cbe

Merge "ContribsPager: Use one TemplateParser instance"

Merge "EnhancedChangesList: Use one TemplateParser instance"

Merge "TemplateParser: Improve cache integrity check failure scenario"

Improve HTTP logging

* Log HTTP debug lines to the "http" channel instead of wfDebug()
* Add the ability to do debug logging to MultiHttpClient
* Add a static method Http::createMultiClient() which provides a
  MultiHttpClient configured similarly to the way individual requests
  are configured, respecting the wiki's $wgHTTPTimeout and writing debug
  logs.
* In EtcdConfig, pass the logger instance through to the MultiHttpClient
  backend.

Change-Id: Ic5bdcb0cae95d7b3715ab5261758be082751c3ff

Avoid postgres PrefixSearchTest failures due to collation differences

Bug: T75174
Change-Id: I6b1df27ca6ae5c2da46134be3dbd7aed404d9601

Localisation updates from https://translatewiki.net.

Change-Id: I9a4201e523e0e37756d4b70c7b85fc670a6fa1d7

Merge "Make sure unique keys carry over to postgres table duplicates"

Merge "Introduce wgRelevantPageIsProbablyEditable"

Introduce wgRelevantPageIsProbablyEditable

This is equivalent to the existing variable wgIsProbablyEditable and in
keeping with wgRelevantPageName/wgRelevantArticleId; this is done as an
alternative to making a breaking change to the semantics of the former,
as that might cause issues for users who expect wgIsProbablyEditable to
return true on [[Foo]], but false on [[Special:Move/Foo]] and similar.

Change-Id: I6c6ca1cfd93e7be917952980f1e1d57aec3a1292

Remove Selenium tests written in Ruby

New Selenium tests in Node.js are in tests/selenium.

Bug: T164004
Change-Id: I3ddeecb4f012ba4a614b3ada81c5926da28556f4
Depends-On: I062d68458d8f472d636f1ae15f4fc37110460854

Localisation updates from https://translatewiki.net.

Change-Id: Ie2f793b769f66378edf72e616dfdd2a344e95987

Merge "Apply EXIF rotation to X-Content-Dimensions"

Merge "ApiLogin: i18n for 'Aborted' reasons"

Resources.php: Add missing dependencies to module 'mediawiki.user'

Change-Id: Ie68418a47d4b4b98fac2b275def392294bd01909

Merge "RCFilters UI: Actually backwards-compat instead of overriding false"

Make sure unique keys carry over to postgres table duplicates

This fixes BotPasswordTest failures.

Change-Id: I81003187af0c57020f9fc4cbcd2606e71907a903

RCFilters UI: Actually backwards-compat instead of overriding false

Another typo in the code made highlight feature always load as false
when you loaded a saved filter.

Bug: T165437
Change-Id: I0e0efa3576659e4145b20013e4af7a06e4e7c0ff

ContribsPager: Use one TemplateParser instance

TemplateParser has an instance cache to avoid reading from APC
repeatedly for the same template, but that only works if the code uses
the same TemplateParser object.

Change-Id: I59e37306c5a3216cf8ed3bfef9c80d50934a9146

EnhancedChangesList: Use one TemplateParser instance

TemplateParser has an instance cache to avoid reading from APC
repeatedly for the same template, but that only works if the code uses
the same TemplateParser object.

Noticed while investigating T163154.

Change-Id: I645895a0965f7150e9a5aebc5a7788f27aa5a26d

TemplateParser: Improve cache integrity check failure scenario

Previously, if the cache integrity check failed then it would emit a
warning but then continue to use the code. The integrity check could
genuinely fail if the secret key was changed, if the cache was
truncated, or other edge case scenarios.

Now TemplateParser will recompile if the cache fails the integrity
check, and then update the cache with the newly compiled version.

Bug: T163154
Change-Id: I9a6c8d528f84cfbabf402cfaf6468c162fab1f15

Update OOjs UI to v0.21.4

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.21.4

Depends-On: I676c9aadbf4739437b94ec729cc4fe47ed19c886
Change-Id: Ie9f8e42c46948688495cbd80e778b484c7b03982

Merge "Sync up with Parsoid parserTests.txt"

Sync up with Parsoid parserTests.txt

This now aligns with Parsoid commit d163deefa3aaeb0926af09a91075d6a611b36363

Change-Id: I9e543f0803247ffc264e634bf66b6bd0e143f187

Merge "Improve output of refreshImageMetadata and refreshFileHeaders"

ApiLogin: i18n for 'Aborted' reasons

Ia6c402a4 did it for 'Failed', but overlooked 'Aborted'.

Change-Id: I3f7379af3609c30f10ea8c648f2ec840310e8d97

Apply EXIF rotation to X-Content-Dimensions

Also adds integration tests for a few formats.

Bug: T150741
Change-Id: I686f7ef42457faf5bc688e60e6ce09a8550ca5aa

Merge "Resources.php: Add missing dependencies to module 'mediawiki.util'"

Improve output of refreshImageMetadata and refreshFileHeaders

Bug: T150741
Change-Id: Ie5f787fd77ecd31b8852d0f66de912baced4ca46

Resources.php: Add missing dependencies to module 'mediawiki.util'

Change-Id: Iac3070adc36533ae046f57ecc7c772247b42420d

Merge "registration: Move attributes out of the top level"

registration: Move attributes out of the top level

This moves attributes out of the top level, and namespaces them under
each extension. If the extension that it belongs to is not installed,
the attribute is not exported and dropped.

The full name of the attribute is the name of the extension plus the
name of the attribute key. This enforces the recommendation that the
attribute name start with the extension's name.

Add test coverage for attributes under manifest_version 1 and 2.

Bug: T133627
Depends-On: I5a148763f68989c8da313a4fb1d0213658ee4495
Depends-On: I5a148763f68989c8da313a4fb1d0213658ee4459
Change-Id: I8613a027c56e2c9d2c6a83ca14749eb1c8fc23be

Merge "registration: Deprecate not having manifest_version set"

registration: Deprecate not having manifest_version set

Emit a wfDeprecated() warning for each extension that does not have
manifest_version set.

Because we don't have any mechanism to cache warnings, just disable the
cache if any deprecation warnings are emitted. This ensures that the
warnings are shown on every page load instead of probably once a day.

Bug: T155610
Change-Id: I163a84ae1d381ca5bd67b4b317c04f9f51c066ea

Merge "OutputPage: Remove deprecated addParserOutputNoText, getHeadLinks methods"

Merge "Removed deprecated ExtractThumbParameters hook"

Merge "EditPage: Preserve &ooui=1 / &ooui=0 from URL after preview"

Merge "ParserOptions: Include wrapping class in options hash"

RCFilters: Actually be consistent when storing 'highlight' state

Saved queries are storing *filter* definitions, not parameters, so
highlight state (dictating whether highlights are enabled) are supposed
to be stored as boolean.

A previous fix that was meant to make sure we store and handle all
*parameters* as strings '0' and '1' was a bit overzealous, and stored
highlight state in saved queries as '1' and '0' too. The current state
then compared that to the filter-state, which produces false/true.

This resulting in saved queries only being matched to the current
state as never true, because the comparison goes by actual obejct
key-values, not symbolic values.

Bug: T165381
Change-Id: Ifdec3a8c058a0e004304e48a857c58639928b36f

EditPage: Preserve &ooui=1 / &ooui=0 from URL after preview

Bug: T165237
Change-Id: Ia546159b45fabb1256f63f888412703ca3eba2e9

Merge "RCFilters: Rephrase 'unset' to 'remove' in default option menu"

Merge "RCFilters: Always put highlight values in the URL"

Merge "RCFilters: Fix getFilterRepresentation to consider '0' as false"

OutputPage: Remove deprecated addParserOutputNoText, getHeadLinks methods

Change-Id: Idc55e7bb0f7bde8466192a81886f205de60c092c

Removed deprecated ExtractThumbParameters hook

Depends-On: I827c61a5a340d4f107478174263e4a5d385398f4
Change-Id: I97e820d78d8041c40fb6e553c4081c676602462c

ParserOptions: Include wrapping class in options hash

Avoids polluting the cache when things take advantage of the option.

Bug: T165115
Bug: T165161
Change-Id: I5be25c6de68012df58b6a0cbf92e2f972be2b68a

Localisation updates from https://translatewiki.net.

Change-Id: I4b723b8eac7d7df85781495c396db74ea0021dbc

RCFilters: Always put highlight values in the URL

Up until now, we only populated highlights in the URL if the item had
a highlight, otherwise it was not in the url at all. However, now that
the system can load defaults from saved queries that can have highlights
themselves, then every time we reload the page (and the system checks
to get defaults merged with the URL query) nothing actively overrides
the default highlight value if it exists.

This meant that if you have a saved query default with any highlights
in it, every time you load the page from the URL it will **also add**
the default highlights that you have saved.

To prevent this, the URL now needs to always populate items with
highlight value, even if that value is null. When we literally ask
for defaults or when we actively load a saved query, that value will
be overridden, but if we have a URL with highlights enabled at all,
the defaults will not override and add a redundany unneeded highlight
just because it existed in your saved query.

Bug: T165231
Change-Id: Ia43b5c777c0b4e238e99818696a3a60dda0daca9

RCFilters: Fix getFilterRepresentation to consider '0' as false

Because '0' is a string, it's true, but for our purposes, it's
supposed to be false. Thanks JavaScript.

This bug was actually pretty horrific, it meant that when you refresh
the representation is all wrong (all items in the group were considered
true if the group was 'send_unselected_if_any' which meant that most
of those (that are full coverage) 'corrected themselves' to be all-false
which meant you lost filters when refreshing, even though the parameters
appeared in the URL (the url helpfully corrects itself based on the model
but the model was wrong.)

How did this pass unit tests, one might ask. Well, the unit tests were
treating parameter values as numbers, rather than strings, a fact that
is promptly fixed in this commit.

Also, for consistency and proper data validation, all parameters are
now always stored and handled as strings, in the model.

Bug: T165230
Change-Id: I16d8d95be067b3e48e557ef25f8eb6a49736aa4e

Localisation updates from https://translatewiki.net.

Change-Id: Ia4e0aad6e4f8ee1f1cba85cf72c77d83bffae7f3

Merge "resourceloader: Add filename to validateScriptFile cache key"

Merge "resourceloader: Simplify validateScriptFile() with getWithSetCallback"

resourceloader: Add filename to validateScriptFile cache key

* Add fileName to cache key to fix T52919. The cached parsed error
  message contains the filename, this should be part of the cache
  key as otherwise two identical user scripts may report the same
  error message, including " on line X of page Y" where Y is whichever
  of the two pages first created the cache entry.

* Make the cache key global instead of per-wiki. There is no need
  for this to be per-wiki.

Bug: T52919
Change-Id: I6c2718c53be7f6384a6486a4a8718ae7f423d216

Merge "ApiQueryRevisions: Don't be so tricky handling rvstartid/rvendid"

RCFilters: Rephrase 'unset' to 'remove' in default option menu

Bug: T164861
Change-Id: I13174ac6812532fe0537cbba0f649cf330bf0c57

Localisation updates from https://translatewiki.net.

Change-Id: I411ee26a2f6dd7199c42bc3a8e70fde852255ca7

resourceloader: Simplify validateScriptFile() with getWithSetCallback

* Simplify by using early return and getWithSetCallback.
* Add TTL (previously indefinite, now 1 week).

Bug: T52919
Change-Id: Ic95ba392cdb3bcc8081c77d2c2a3240548bed366

ApiQueryRevisions: Don't be so tricky handling rvstartid/rvendid

If someone strangely supplies 0 as an id, the tricky version confuses
that with null and winds up making the DB layer throw an exception. Just
do it more straightforwardly.

Bug: T165100
Change-Id: Id8376d419e7758c0bf92ad50dd7b7ac283a73101

Refactor filter-specific logic into group models

Make the view model more generic and ready to accept other types
of filtering, like namespaces and tags, by handing off the responsibility
and logic of the parameterizing and filter relationships to the groups.

Change-Id: I9c333719e89088d96d044d72ddb9c39feeeb68ca

RCFilters UI: Add a placeholder to SavedLinksListWidget when it's empty

No longer hide the button when the menu is empty, but instead
show the placeholder item when the *model* is empty.

Bug: T164861
Change-Id: I96e5e375de5f35946663042f6731d7b69e53308b

RCFilters UI: Add tooltip to saved links items

Bug: T164861
Change-Id: Ic7c38c59c53d4d83b959eff36d4b1d198bb654ea

Merge "Pager class for filtering by date range"

Merge "RCFilters: Treat saved query with multiple spaces as empty"

Merge "RCFilters: Actually read/write highlight parameter"

Pager class for filtering by date range

New abstract class RangeChronologicalPager to provide shared date range
filtering capability (with unit tests)

I18n msgs to use as common labels for date range inputs

Expose some ReverseChronologicalParser::getDateCond logic so we can
convert year/month date filters to datestamp date filters

Bug: T120733
Change-Id: I65fdc00368f406f5fa2492600e95e07ce442c165

More magic word translations for Catalan (ca)

Newly translated magic words and new translations for
some already-translated magic words, sometimes moving
existing translations to a lower priority.

Change-Id: I18fd3481f880cf84cdb9b9b486e7062c9f6fbdcc

Merge "Add X-Content-Dimensions support to DjVu"

Merge "Factor out changetag name validation check"

Merge "RCFilters UI: Add down indicator to SavedLinksListWidget button"

Merge "RCFilters UI: Don't remove removed saved links twice"

RCFilters: Treat saved query with multiple spaces as empty

Because it is empty, even if there's multiple spaces. Also, if the
input is invalid and we didn't save the query, don't close the
popup.

Bug: T165113
Change-Id: I30e68b0b90e2a4077fb4068c1462d40b03cbfe8c

RCFilters: Actually read/write highlight parameter

The highlight parameter enables/disables the highlight view. It was
not properly updated to the URL, and a separate typo made it not update
the model correctly.

Bug: T165107
Change-Id: Ie711d4d08fb3eeffca50c2953016e02e2620f898

Merge "Gate option to save RC filters to default false"

Merge "mw.Upload.Dialog: Define .static.name"

Merge "Check for null before calling a textResults method to (fixes T165091)"

Merge "Fix comment typo in MimeMagic.php"