resourceloader: Allow modules to be registered via a factory callback

This should work the same way as registering API modules via a factory callback.
Point in case: Ifb8611473a971 could avoid global state using this mechanism.

Change-Id: Ifbf29006141ce2a2dff42efa352f406502a06bc6

Merge "Print: prefer JS print over printable version."

Print: prefer JS print over printable version.

Bug: T24256
Change-Id: Ic3dc15e578ce4a94959285c9ae579e9088423544

Merge "Fix EnhancedChangesList::getDiffHistLinks null exception"

Merge "Match Parsoid's attribute sanitization for video elements"

Merge "HTMLUsersMultiselectField: Make auto-infusable when used inside a legacy HTMLForm"

Fix EnhancedChangesList::getDiffHistLinks null exception

If a page is deleted, rc_cur_id is missing, but it takes
a while before job queue deletes RC entries. If we encounter
something like that, just skip it since its bound for
deletion anyways.

Bug: T164059
Change-Id: I286109a9707e54939c0da31656ef54fd29acf481

Merge "Allow namespaces defined in extension.json to be overwritten locally."

Localisation updates from https://translatewiki.net.

Change-Id: Id9a991314902c759c4fac6202896c582e346737c

Merge "ApiSandbox: Fix HTTP error handling"

Merge "Various test fixes for postgres"

Merge "Added increase editcount upon page move"

Merge "Improve grammar of message on Special:GoToInterwiki"

Merge "Remove untranslated "editinguser" message from tyv (tuvinian)"

Merge "Typofixes in two apihelp messages for search and templates"

Improve grammar of message on Special:GoToInterwiki

Change-Id: Idd19a8455b82c86cf5a891efb17199fc9f625286

Added increase editcount upon page move

Bug: T163966
Change-Id: Ie055f676534e98963c68f8b8185cca65fa8e76b1

Typofixes in two apihelp messages for search and templates

Spotted by Dgstranz:
* https://translatewiki.net/wiki/Thread:Support/MediaWiki:Apihelp-query%2Bsearch-param-enablerewrites/en
* https://translatewiki.net/wiki/Thread:Support/MediaWiki:Apihelp-query%2Btemplates-param-namespace/en

Change-Id: I5655d95c6cdd74433d358f15873d4a3a30738f50

HTMLUsersMultiselectField: Make auto-infusable when used inside a legacy HTMLForm

Change-Id: I23b3a86f06a10b066e6671e398da37c62e3e61b1

Remove untranslated "editinguser" message from tyv (tuvinian)

https://translatewiki.net/wiki/Thread:Support/MediaWiki:Editinguser/tyv

Change-Id: I7ac5da1d7bfeca09e88a8f7dc13073eed0195e69

Allow more customization on the default toggle system

The naming convention remains unchainched. Default toggles are added if no mw-collapsible-toggle child element is present. Premade toggles have already one ore more mw-collapsible-toggle elements defined.

Default and premade toggles (mw-collapsible-toggle) still cannot be mixed/combined with remote toggles (mw-customtoggle).

* The default toggle got less rigid by removing dependency on <a> elements.
* Support for multiple premade toggles was introduced.
* The expand/collapse messages can be used by premade toggles via mw-collapsible-text.
* Removed the linksPassthru option. This step allowed merging premadeToggleHandler into actionHandler.
* The pass through functionality is now applied to all <a> elements since the default toggle does no longer depends on those.
* Removed mw-collapsible-bracket because it was not used and is deprecated by mw-collapsible-text.
* The test suite was adapted to reflect the latest changes.

Change-Id: Ic8627c4c185e8e4175e6fef1c8e1c2190e54edaa

Merge "Disable broken TemplateCategoriesTest unit test"

Merge "Add HHVM version check"

Disable broken TemplateCategoriesTest unit test

Bug: T165099
Change-Id: I370e48965fdfc948aa02533ebfeb0a540fb53c12

ApiSandbox: Fix HTTP error handling

Since bf69459, ApiSandbox seems to stall out when an API request
results in an HTTP error (e.g. due to a PHP fatal error). Before that
revision, it displayed the 'apisandbox-results-error' message in this
situation.

Apparently the jQuery 3 changes to Deferred behavior caused it to be
impossible to have a then() filter return `this` (or anything else) in
order to avoid replacing the existing promise that's being resolved or
rejected.

Bug: T165857
Change-Id: I3f646cdfe7fe8987437980790788821f51e728d1

Merge "Make sure all functions in Database.php are documented"

Merge "Fix Closure detection in MediaWikiTestCase"

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 Closure detection in MediaWikiTestCase

Sometimes the closure are hidden in arrays, catch this.

The $maxDepth check is just for sanity, I don't think it's
actually needed.

Follows-Up: c2c7452577e

Bug: T111641
Change-Id: Id5e036ce4949b8106873fd938f54c2774d3d6a4a

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"

Make sure all functions in Database.php are documented

Will add @inheritdoc in a follow-up commit for functions
documented in the parent classes.

Part of 2017 MediaWiki Documentation Day

Change-Id: I002a1f6451940ecbcacea7b3ca2fc6ad0f4eba47

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"

Add HHVM version check

HHVM brings it's own version number, so relying on the PHP version
string in the PHP_VERSION const on an hhvm server maybe does not
increase in different versions of hhvm. Therefore, this change
implements a hhvm server version check, if MediaWiki runs on an
hhvm server, instead of checking the php version.

Bug: T123436
Change-Id: I91b0a4e1eded61d3ba63ecbd14cdc85c57295892

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

Various test fixes for postgres

* Add another TS_MW timestamp coercion to WatchedItemStore.
* Do not expect timestamp conversion for pager mOffset.
  They have been used in Pager URLs for ages.
* Do not assume the user_id for UTSysop is 1; use the row value.

Bug: T75174
Change-Id: I590a4b7363ccff26d9ac9f51b95fc1aa072503b2

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