Merge "mw.loader: Avoid use of deprecated QUnit.asyncTest/QUnit.start"

Merge "Remove temporary hole in abstractions"

Remove temporary hole in abstractions

Was introduced temprarily to facilitate event logging, now the only caller is gone.

Bug: T166247
Change-Id: I0f9cdccc6b3a566d9f7ebf02515f724348205071

RCFilters: Allow experimental live update feature to be enabled with query string parameter

For testing. Adding ?liveupdate=1 to the URL will unhide the live update button.

Bug: T167743
Change-Id: I2696da959d34adf952f7f2fa6477ab756ef259bd

Localisation updates from https://translatewiki.net.

Change-Id: I272241868eec5790ebc164f74b3d8f6982153776

Merge "Add urlencode for URL encoding of lang values"

Merge "ChangesListSpecialPage: Don't short-circuit the ChangesList, instead format zero rows"

RCFilters: Scroll to top on view switch

Bug: T170651
Change-Id: I07376569397651ebee0815c1e1d6e945ef1704dc

Localisation updates from https://translatewiki.net.

Change-Id: I3d1065d5b04461626dac32532b635373b28ade4f

Merge "Special:ListUsers: Use HTMLForm and OOUI"

mw.loader: Avoid use of deprecated QUnit.asyncTest/QUnit.start

Deprecated since QUnit 1.16, removed in QUnit 2.0. (We're on 1.23 currently.)

Migrate to assert.async().

This is a fairly atypical use of QUnit.start(), because it functions here
as a cross-script callback, where lexical scope cannot be used to share
the async() callback directly.

Other mw.loader tests already solved this by using a static callback instead
which inherits the lexical scope from the test to call done(). The old
'qunitOkCall' script is no longer used after this and thus removed.

Change-Id: I430df14b35a69c71df8685494d1379e22af0d6df

Merge "Maintenance script to purge specific page"

Merge "HTMLUsersMultiselectField: Handle empty value properly"

Merge "RCFilters: Don't call ChangeTags::tagUsageStatistics() for now"

Merge "SpecialUndelete: Follow-up to 6cf13604aec40e2210c16ccdbeba850de9428d19"

SpecialUndelete: Follow-up to 6cf13604aec40e2210c16ccdbeba850de9428d19

Per my (late) code review comments on that commit.

Change-Id: Ia5bb6410b430992b66f34274e86480e6c9566525

Localisation updates from https://translatewiki.net.

Change-Id: Ice8d28e7cb190e7c0b2ec4d95a3e3e2e835b17bb

Merge "DeleteAction: Remove now-unnecessary RL modules"

Merge "Fix cleanupTitles.php"

Merge "Convert rest of Special:Undelete to OOUI"

Merge "Give some idea of time remaining"

Merge "Advise extensions not to modify $wgWhitelistRead"

DeleteAction: Remove now-unnecessary RL modules

After 97a8fb78faa4ca2d9604140e5547f5deae5a2524 these are never used.

Change-Id: Ibb4ee45ad991651afab0f0363056d3066e740b24

Merge "FilterTagMultiselectWidget: Use frameless buttons and fix height issues"

Convert rest of Special:Undelete to OOUI

Also fix the 'invert selection' button:
* Don't use type=submit so it doesn't try to post
  if JS is disabled
* Don't invert the wpUnsuppress checkbox, just
  checkboxes in revision lists.

Bug: T113758
Change-Id: I84785a5ddcc3343d74b5b7addb4aa4817a469f33

Merge "Convert article delete to use OOUI"

Merge "Convert Special:Undelete search form to use OOUI"

Merge "Use OO.ui.MultilineTextInputWidget"

Convert Special:Undelete search form to use OOUI

Bug: T113758
Change-Id: I955a07a5903c37d5a054c7b2427c4ec0615a3c36

Merge "Sanitizer: Allow attribute names to use any Unicode "Letter" or "Number""

Use OO.ui.MultilineTextInputWidget

Replacing deprecated OO.ui.TextInputWidget({multiline: true}) as of
OOjs UI v0.22.2 with OO.ui.MultilineTextInputWidget.
Also removing `multiline: false` since it is the default value.

Bug: T170275
Change-Id: Iad7c5bb32d7fb9c38b1781cce00eec8f1e99f6f2

Merge "Add (hacky) check for webm audio files"

Merge "Add hook for cleaning up data that depends on purged recentchanges rows"

Advise extensions not to modify $wgWhitelistRead

Instead they should use the TitleReadWhitelist hook. I've seen this
pattern in multiple extensions now, each causing different problems, so
it should be explicitly documented.

Change-Id: I670b0acde660bdc27a6c7e91c4b1b3baeea8b2e5

RCFilters: Don't call ChangeTags::tagUsageStatistics() for now

We need to fix its performance first, it currently takes >30s on wikidatawiki.
Fake all hit counts to be zero. Instead of sorting by hit count,
sort by display name.

Bug: T169997
Change-Id: I4075ea4d43a8f75e21a87a892211699ba3bc7058

Update OOjs UI to v0.22.3

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

Depends-On: If676bfab1d2040a9d360c741a5ab4e1c81b36b87
Change-Id: I684822b564663a94e03fc8b559617974502c8e52

Localisation updates from https://translatewiki.net.

Change-Id: I7a1a699abbe0d0a35d753e9d8e29a292e4722b5f

Add (hacky) check for webm audio files

If there is no video track in a webm file, it's supposed to be audio/webm,
but since we assume every webm file is video/webm, that's never worked.
This should slightly improve searching for audio files.

Bug: T156135
Change-Id: Iac749233e87b3622ff416ad503aedef3df0d0f1d

HTMLUsersMultiselectField: Handle empty value properly

If the string value is '' (empty string), the array
value should be [] (empty array), not [''] (array of one element
which is an empty string).

Bug: T169384
Change-Id: I558f3890af05efb6eaa18403467c0a05f44af12a

Merge "Make label on Special:Undelete search reflect whether search is full-text or prefix"

Merge "Use OO.ui.MultilineTextInputWidget"

Use OO.ui.MultilineTextInputWidget

Replace deprecated OO.ui.TextInputWidget({multiline: true}) as of
OOjs UI v0.22.2 with OO.ui.MultilineTextInputWidget

Bug: T170275
Change-Id: I8f310510a318fb943969fde2a366c08d88bda0e7

mw.loader: Log unknown modules in load() to console

Follows-up If8ff31b530dfbd882. Now that we have resolveStubbornly,
we don't need to filter out unknown modules ahead of time.

Aside from removing the needless filter condition, it also has
the benefit of causing a message to be logged to the console,
which can help discover problems. Previously these were silently
ignored.

Change-Id: I700db4931dfd0a412a8eca66c4a74b8831ab0086

Merge "Make multiple colons escaping interlanguage links invalid, consistently"

Merge "Resolve successful 1-chunk uploads"

Resolve successful 1-chunk uploads

Existing code would assume there's always more than one chunk,
or it would otherwise not resolve.

Upload dialog started doing chunked uploads recently:
Ib9cc2c77a27658accf728fb37f8cebc879a467d2

Change-Id: I29a85c5ee6fbf6cea7d07e3281279b4ea3d87188

Merge "Refactor UploadBase::checkWarnings into smaller methods"

Merge "RCFilters: Align TagItemWidget highlight in Safari"

Merge "Use shell_exec instead of backtick"

Merge "RCFilters: Change legend heading for the beta feature"

RCFilters: Change legend heading for the beta feature

Bug: T168043
Change-Id: Ide56598b160c93e1dfa8019c0e6e99fb377ea311

Merge "mw.loader: Skip modules in load() with unknown dependencies"

Merge "RCFilters: Add white background to legend"

Fix cleanupTitles.php

It was failing in production with duplicate key error in
moveInconsistentPage(). Live-patching it to check for existence
for updates resolved the issues. Committing the fix now.

Change-Id: I1c0bf632b5040b44f5a49c8fb95779e7ed19dea9

RCFilters: Add white background to legend

Bug: T170190
Change-Id: Ib0fc1a492d8e6d431e73bc749178f19f25863965

RCFilters: Respect ?enhanced=0 url regardless of preference

Change-Id: Iffb8cc17b86c5358eea5ae904f55c88965ba4ee0

Localisation updates from https://translatewiki.net.

Change-Id: I2d28957e14a7a90a5cba88bfe1a60060387d7cdc

Merge "RCFilters: Only show 'no filters' when actually none were found"

resourceloader: Add unit tests for ResourceLoaderSkinModule

Change-Id: I299eff8f5172e047c0d54d9c824b17529dd6d190

ResourceLoaderSkinModule: Fix SkinStyles extending of known media queries

If any of the styles given in its module definition (in the
'styles' or 'skinStyles' properties) used the same media queries
as the module's own CSS (e.g. 'all'), the module would fail with
"PHP Fatal error: [] operator not supported for strings" because
FileModule defaults to merging all the stylesheets into a single
string.

Fix this by ensuring they are arrays before trying to extend them.

This previously made it impossible to use $wgResourceModuleSkinStyles
for modules that use SkinModule (instead of plain FileModule), such as
the 'mediawiki.skinning.interface' module.

Bug: T168088
Change-Id: I3effcaa4982728e707fbf9efeec4e5e78fc8aab6

RCFilters: Only show 'no filters' when actually none were found

Bug: T170161
Change-Id: I50da0d21e84d7e0582437dd43760c93ecb557a56

Add hook for cleaning up data that depends on purged recentchanges rows

Adds RecentChangesPurgeRows hook which gets called in the same
transaction where old recentchanges rows are purged, and gets the
list of deleted rows. Extensions which store data for joining to
recentchanges can use this to purge their own data.

Bug: T159753
Change-Id: I03f1d485a1a3004412e0859d9d878b7895c95b40

Remove auto-generated "Constructor" documentation on constructors

Having such comments is worse than not having them. They add zero
information. But you must read the text to understand there is
nothing you don't already know from the class and the method name.

Change-Id: I994d11e05f202b880390723e148d79c72cca29f0

Special:ListUsers: Use HTMLForm and OOUI

Also update the hooks documentation. Now that it is using HTMLForm the
<fieldset> is closed before the submit button is added. The old code
was closing the <fieldset> after adding the submit button so the
documentatio made sense.

Bug: T111999
Change-Id: I109065100e40fef0c56a010c444de04a40950479

Use <link rel="license"> instead of <link rel="copyright">

"copyright" is not in the list of the allowed values for the attribut rel:
https://w3c.github.io/html/single-page.html#allowed-keywords-and-their-meanings

Bug: T167284
Change-Id: I09205bb146001d1d1d90e6a0f25d21851f3f697c

Rename ckb to Kurdish

Bug: T116761
Change-Id: If5abc7c859c3fbf13e3ebf1be98693ee6fc04d9a

mw.loader: Skip modules in load() with unknown dependencies

We already skip unknown modules at the top-level, but dependencies
still cause a run-time exception from sortDependencies, resulting
in the entire queue not being loaded.

Bug: T36853
Change-Id: If8ff31b530dfbd8823c47ffd827fcdba807c05b3

Localisation updates from https://translatewiki.net.

Change-Id: I7e6128d08d6a47a747d1d7732ae7df2beea3acce

Merge "statsd: Rename MediawikiStatsdDataFactory to IBufferingStatsdDataFactory"

RCFilters: Only build tag list when experimental views are enabled

Follows-up 7725c755867ac1.

It's a waste otherwise, and buildChangeTagList() has perf issues.

Change-Id: Ifd7f81792fd7b5ee6e41ada756abbf0248f3eb36

Merge "Autodiscover parser tests for extensions, deprecate $wgParserTestFiles"

Merge "Use Sanitizer::stripAllTags( $x ) instead of html_entity_decode( strip_tags( $x ) )"

Autodiscover parser tests for extensions, deprecate $wgParserTestFiles

This implements autodiscovery of extension parser tests that are located
in the tests/parser/ directory. Any *.txt file in that directory tree
will be treated as a parser test.

Core parser tests are now defined in ParserTestRunner::$coreTestFiles,
and $wgParserTestFiles is marked as deprecated.

Bug: T143976
Change-Id: Ia24fd8ef52e6732c698153b17bb679a5f511a2a7

Merge "SpecialRecentchanges: Globals for config are bad, m'kay"

Merge "RCFilters: Basic implementation of live updates"

statsd: Rename MediawikiStatsdDataFactory to IBufferingStatsdDataFactory

Follows-up I2874175647e (7fdc3d09a3b).

* Avoid "MediaWiki" in class names. If unavoidable, always camelcase.
* Use a more descriptive name, matching the implementing the class ("Buffering").
* Use "I" prefix for consistency with other MediaWiki-provided interfaces.
  (Avoid "Interface" suffix, which is foreign to MediaWiki and is only used
  in the StatsdClient namespace, which this interface is not in).

Bug: T166354
Change-Id: I06de59122625f9c23e7c1a1bfa69a7ddabbf379e

SpecialRecentchanges: Globals for config are bad, m'kay

Change-Id: I9ca7aead42ffee1e891525fc80f3438157f4fb2f

RCFilters: Basic implementation of live updates

Adds a live updates button that refreshes the changes list every 3 seconds.
For now this is pretty dumb in that it re-requests the entire list every time;
the next step would be to make it only load new changes using the &from=
query parameter.

Bug: T167743
Change-Id: Ic2ddea840e5c46f42b32ae4fff91138cacc28ec0

Use Sanitizer::stripAllTags( $x ) instead of html_entity_decode( strip_tags( $x ) )

We have a utility function for this, so let's use it.

What I don't understand though is why Sanitizer uses custom PHP implementations
for both tag stripping and entity decoding, instead of the built-in functions.
If there's a security reason for this or the built-ins are inadequate, that's
fine, but then that should be documented (and we should possibly ban usage
of the built-ins).

Change-Id: I2ba2ecd388cb3d9cd2360ecaa236f3d444f0eabf

Localisation updates from https://translatewiki.net.

Change-Id: I43c7687a6be0a4b411287ccd2c3541c9edab5c30

Merge "Fix font size / alignment of timestamps in enhanced recent changes"

Fix font size / alignment of timestamps in enhanced recent changes

font-family: monospace; causes the timestamps to have a smaller font size,
but font-family: monospace, monospace; doesn't.

Bug: T170010
Change-Id: I20a4ab12803d43e26ced429fe3b68939a1f2c13e

Merge "Adjust Shortpages query with multiple content namespaces"

Merge "Add Database::unionConditionPermutations()"

Merge "mediawiki.UI: Remove deprecated `.mw-ui-constructive` class"

Merge "SpecialNewFiles: Swap from/to date serverside"

SpecialNewFiles: Swap from/to date serverside

Bug: T168695
Change-Id: I760096138da15a6f3b84a6db68ab5590f5a3e59b

Documentation: Fix return value of IDatabase::insertSelect()

Implementation at Database returns bool, not IResultWrapper

Change-Id: Ic560bfcb45f0e9cd1f4e8334e3b5dfc4e682452a

Disable VisualEditor welcome dialog when running Selenium tests

Bug: T164721
Change-Id: I7b5f73c6fbaaa221ac1b689344f9015fb903220f

Merge "Create users and pages for Selenium tests using action API"

Create users and pages for Selenium tests using action API

This will make tests slightly more robust.

Bug: T164721
Bug: T167502
Change-Id: I9b2fea77b28af4f7f521490a0105e7d04730bc87

Merge "parser: Add unit tests for parser output flags ('showflags' option)"

Merge "parser: Add parser tests for 4 and 5 tildes in PST"

Merge "Push all DeferredUpdates to POSTSEND queue when running that queue"

Merge "Avoid high edit stash TTLs when a user signature was used"

parser: Add unit tests for parser output flags ('showflags' option)

Add logic to ParserTestRunner to allow testing of ParserOutput flags,
and use it in the existing {{REVISIONID}} test, and for the added
flag for user-signature from I77de05849c (T84843).

Change-Id: I96e3d050e17a2d7e3d0478c702ecd53310259f56

parser: Add parser tests for 4 and 5 tildes in PST

Add logic in ParserTestRunner that sets the same fake timestamp
used by the Parser magic word expansion, also in ParserOptions,
which is used by preSaveTransform().

Change-Id: I5adacffccb1212651c3031ca2fc4c20f717ff24a

Merge "Clarify what $params is for ApiBase methods to get Title/WikiPage"

Merge "objectcache: Use a separate postgres connection in SqlBagOStuff"

Clarify what $params is for ApiBase methods to get Title/WikiPage

Change-Id: I1bfae270072ba08db967a02a8e30047bc607e3a2