Merge "Update docs/hooks.txt for ShowSearchHitTitle"

Merge "Fix Selenium tests"

Fix Selenium tests

The tests were broken because the feedback meesage is no longer
displayed as part of the page, but using HTML5 `required` attribute.

Bug: T155640
Change-Id: I9c46db374ec7f50262f84d721a21110aa6e81699

Update docs/hooks.txt for ShowSearchHitTitle

Reasons:
* $text is vague. It's the label of the link. There is a function named
  getTitleSnippet, which is what I'm reusing for the variable name.
* $page is vague. It's not the page of the search result, but the search
  special page.
* $queryString is not a string but an array of strings.

I also updated https://www.mediawiki.org/wiki/Manual:Hooks/ShowSearchHitTitle
accordingly.

Change-Id: Ia18dbf5ccf7f9fcc8890ad01641d0b7ff3e6eee8

Localisation updates from https://translatewiki.net.

Change-Id: Id4e35d1705d767dfd1130533acfb0d721cc5829e

Merge "Make most of DateFormatter private"

Merge "SpecialContributions: Username input is not really required"

Make most of DateFormatter private

As discussed in https://gerrit.wikimedia.org/r/#/c/332702/ , these
methods and fields shouldn't have been marked public in the first place.
No outside users. Also, declare a couple of fields and remove unused ones.

Change-Id: I7775978c87d983784a484ee2ad901d25c42499b3

SpecialContributions: Username input is not really required

Bug: T155780
Change-Id: If5cd6867029c005cf56a1628efc239955cd49931

Merge "Remove unused $resourceLoader"

Remove unused $resourceLoader

Change-Id: I6b5aebe2c6221ee101ec3bc4c84eafd9aeb7b0e2

resources: Update OOjs UI with fixes on top of v0.18.4 (master)

Cherry-picked fixes:

* be46e1773cd0f6b04510c325054d49835714b5f3
* 629389a777b35d131289a12cecf986e52ff65534
* 92dcdcf5351f6d499624d247b2a3ebff66346aca
* f991c382ce3dd80ed3498bb09e81885b94f8795e

Bug: T155711
Bug: T155728
Change-Id: I741db3a17eec812b50b4befcc7eba603943b759c

Merge "Un-blacklist PhanUndeclaredVariable"

Merge "Use HTTPS instead of HTTP for Amazon"

HTMLTextField: Set the 'dir' attribute in OOUI mode

Bug: T153636
Change-Id: I7ddf6a358f8150ef252d8d8e76661fe1eb93a790

Use HTTPS instead of HTTP for Amazon

This change is performed using:

sed -i -e 's/http:\/\/www.amazon/https:\/\/www.amazon/g' languages/messages/Messages*.php

Change-Id: I22c5e7a8d33a80b8cbe814f9735e7c24f1b004f9

Merge "Use parsed HTML error responses instead of api-error-*"

Merge "Add deprecated annotation to Article::doEditContent()"

Merge "Add more $wgPoolCounterConf comments"

Merge "Update tests/phpunit/MediaWikiTestCase.php with support for apcu"

Add deprecated annotation to Article::doEditContent()

Change-Id: I5a15b22c8b12fe77105fbd31427e8a15cde327c6

Update tests/phpunit/MediaWikiTestCase.php with support for apcu

Change-Id: I22d54e96c2c002ea53dabfd2772d2ba4c7eb96b1

Mark Article::doEditContent() as deprecated

Bug: T155696
Change-Id: Ie91b428f3a5992ba85c6ca2b2909953d8fce139b

Remove deprecated Parser::replaceUnusualEscapes()

Deprecated since 1.24, no callers.

Change-Id: Ib780f1a7b77d3ce624112f59c8e57820fecb6bf2

Remove deprecated function Article::doEdit() and WikiPage::doEdit()

Bug: T61113
Bug: T154498
Change-Id: Ie6c1bb4ee5ee45ac2db432877a7038659231c091

Revision::getText() was removed

Bug: T154498
Change-Id: I29dcf768a9c176ca27b94359ac99cdc2b5ea930b

Article::getContent() was removed

Bug: T154498
Change-Id: I27956d494cf19d1204988807130b43f77186db9b

mediawiki.rcfilters: Add the remaining MW core filters

Add JSON definitions and i18n messages.

Bug: T149452
Change-Id: I287e9e5db9ff625bde026056efc81ce4b2401607

Add more $wgPoolCounterConf comments

Change-Id: Ic7b376dcada2b58a2f2ddc13b79e392e9d4fae6f

Merge "Unbreak extensions using SpecialSearchResults hook"

Merge "Unbreak extensions from ShowSearchHitTitle signature change"

Unbreak extensions using SpecialSearchResults hook

Changing hook signatures with no deprecation period is a no-no.

Fixes Ib527fc3a3c39eb2e56985e5d1e4905fc4562353c
Reverts I153a7d590faceb1e2c4fe7d95a5cb931cda0c301

Bug: T155625
Change-Id: I2b408691fc4435ef24fc62360bd7ad6114f385c7

Merge "Avoid fatal error with FlaggedRevs when running rebuildFileCache.php"

Unbreak extensions from ShowSearchHitTitle signature change

Fixes If78cb0c29ae394f16e465c15a8e8246c1b56dcea

Change-Id: Ia56f2eb4a5ed98f727f924666322e1c97b3ee183

Removed deprecated ContentHandler hooks.

Plus removed unused "ContentHandler::runLegacyHooks"

Bug: T154498
Change-Id: Ie398fd4e06d3e286fe8e24112d0c8b4ac7d883dc

Un-blacklist PhanUndeclaredVariable

Undeclared variables are a very common error type that we want to catch
as often as possible. To avoid needing to refactor a variety of global
level code (mostly in old-style maintenance scripts) this ignores
undeclared variables in global scope. This is still a good improvement
over what was happening previously.

Change-Id: I50b41d571724244552074b9408abbdf6160aca59

Localisation updates from https://translatewiki.net.

Change-Id: I2ce862aae175fb1287086106a1fefb3e31de488b

Merge "Remove deprecated mediawiki.UI Less variables"

Merge "Remove deprecated `.mw-ui-constructive` class for text and anchors"

Merge "OutputPage: Ignore protocol-relative urls in transformResourcePath()"

Merge "tests: Add unit tests for OutputPage::transformResourcePath"

OutputPage: Ignore protocol-relative urls in transformResourcePath()

transformResourcePath is only supposed to be called with something
that starts with a '/' (absolute path from document root).

While the primary caller (CSSMin) only calls this method if the path
makes sense as a local file path, ResourceLoaderSkinModule did not,
and wgLogo may be set to a full url that includes a domain.

While chance made it so that protocol-including urls were already
discarded, protocol-relative urls were mistaken for being a path
inside the file system root with a duplicate slash (e.g.
'//tmp/foo.txt', as 'foo.txt' in /fmp, instead of '/foo.txt' at
http://tmp).

This should be fixed upstream in the wikimedia/relpath library,
but workaround it for now since it really shouldn't be called
with urls in the first place.

Bug: T155310
Change-Id: I9b063f1219ddeca5cc2c8a48832cdf8c9eaffe58

Merge "Tweak formatting of 'restrictionsfield-help'"

Merge "Pass the language code to the MessagesPreLoad hook"

Avoid fatal error with FlaggedRevs when running rebuildFileCache.php

Change-Id: I8d335c64cfc74a227528fbd85d333eaaa8157f41

tests: Add unit tests for OutputPage::transformResourcePath

Bug: T155310
Change-Id: Ic5a4af705c3734a7992037e59dc43fafe447ba12

Merge "RCFilters: Clean up focus handling in capsule widget"

Tweak formatting of 'restrictionsfield-help'

`<pre>` looks better for multiline code blocks than `<code>`.

Before: https://phabricator.wikimedia.org/F5302938
After: https://phabricator.wikimedia.org/F5302939

Change-Id: I1a180fee5e635e5f08c6730a9c390d7c4325ff9d

RCFilters: Clean up focus handling in capsule widget

Don't let the parent class's weird implementation run when the
widget is focused through other code paths (e.g. clicking on
the non-input part of it).

We still need to fix the behavior when you click in the
input twice (the first click shows the popup and the second
click hides it), but this is much better already.

Related to I21b8cff4048 in oojs-ui.

Bug: T149452
Change-Id: I08743c64ea32feace757e2a8626df2f5310cfbc9

Merge "Update OOjs UI to v0.18.4"

Update OOjs UI to v0.18.4

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

Change-Id: I09eacfe7597b26e33ef50e606afcee0cefca1d1c

Extract 'did you mean' widget out of SpecialSearch

Bug: T150390
Change-Id: I41d767550ab4112fcd9cc4094b27a14c7d29169b

Extract main search result rendering from SpecialSearch

Has a pre-existing problem related to the link offset of secondary
inline results. Specifically the offset of secondary inline starts
at the provided offset, even though there may be primary results
displayed above it.

Bug: T150390
Change-Id: Id1d6b357f45a2cf615d9412cc95dd597c724e8b6

Merge "SpecialSearchResults hooks does not use by-ref anymore"

Merge "Extract search form from SpecialSearch into widget"

Extract search form from SpecialSearch into widget

Bug: T150390
Change-Id: Ibda84349e1f552641805d6236605c8718540817f

Merge "Cleanup execution flow through SpecialSearch::execute()"

SpecialSearchResults hooks does not use by-ref anymore

In general, there's no reason to use by-ref on objects unless
the hook is supposed to replace them (this one isn't).

Change-Id: I153a7d590faceb1e2c4fe7d95a5cb931cda0c301

Merge "Add test case for Opus file check"

Cleanup execution flow through SpecialSearch::execute()

Not a big change, but makes it so there is only one path into
SpecialSearch::showResults(). Makes things a little easier to follow.
Also moves the code for checking if full text search is disabled into
execute(), makes sense to check before even trying to make a search
engine. Also moves some setup code out of execute and into the setupPage
function

Bug: T150393
Change-Id: Ib527fc3a3c39eb2e56985e5d1e4905fc4562353c

Add test case for Opus file check

Test case for Opus file check so we don't regress!
Confirms the fix for T151352 that Opus files return
type 'audio' and not 'multimedia'.

Bug: T151352
Change-Id: Ib868793d90b60609ccec25726e6fbaa28bffebaf

Merge "fix line length warnings from Jenkins"

Localisation updates from https://translatewiki.net.

Change-Id: If5b73fd92e39b1d19c581109a09b1b18600beb79

Merge "Special case opus mime detction"

Use parsed HTML error responses instead of api-error-*

There is no point in keeping these api-error- messages.
They were originally introduced in UploadWizard, then moved
to core, where nothing else (apart from some in BookletLayout)
uses them. They are mostly duplicates of other messages in
core, and these are very easy to miss when updating a message
in core.

Instead of building the messages on the frontend, we're much
better off requesting a parsed error response from the API
right away. This also lets us get rid of sime hacks (e.g. for
AbuseFilter)

And then a few other api-error-* messages have been removed
in favor of their alternatives (which are also used elsewhere)

And stashfailed is not a warning. Maybe it was at some point,
but stashfailed now dies with an error.

Bug: T47843
Change-Id: I41d88e2aad308a1fecd10d97085345a17d0c3603

Merge "Remove trailing empty lines in PHP"

Typofix: Categorie -> Category

Follows-up f81bb12266

Change-Id: I902522e8a3003cf7fa15c29208fdf7e45e4610f8

Merge "Un-deprecate ArticleAfterFetchContentObject hook"

Remove trailing empty lines in PHP

Performed using
find . -name \*.php -exec sed -i -e :a -e '/./,$!d;/^\n*$/{$d;N;};/\n$/ba' {} \;

Change-Id: I5d0627f94c73690cf3a8a453539c22c760c2aa60

maintenance/importImages: Don't sleep after the last upload

Currently, the importImages maintenance script offers an option
to sleep an arbitrary amount of seconds after each upload.

The current logic implementation had a side effect to also sleep
after the last image upload. This is not a desired behavior, as
the tasks are done and a clean exit with a report are welcome.

Instead, sleep will now occur before the second upload and the
ones to follow.

Change-Id: Ia675f210582c3075e9b59b002a00cb1fc4b44cac

Merge "Update .mailmap for the generation of CREDITS"

fix line length warnings from Jenkins

Example: https://integration.wikimedia.org/ci/job/mediawiki-core-phpcs-HEAD/15477/console

Change-Id: I80f5c2ec4de0c6c06f7084ed4cc8a2ec006f6f70

Merge "mw.htmlform: Fix 'hide-if' disabling OOUI fields"

Fix Hebrew grammar transformation for letter He

It must only be transformed in the beginning of the word.

Change-Id: Iec6e2032775b9efd92cc2d649944f4892f813b24

Redocumented $wgAutopromote

Completely rewritten the documentation of $wgAutopromote to make it
easier to understand.

Change-Id: I0746c09f7a97dd18feeb774643367fa1a0c09fd4

Merge "Fix broken variable inclusions in API unblock parameter documentation"

Fix broken variable inclusions in API unblock parameter documentation

Change-Id: I06bf55b3f7f0e05e059f648d92c0cb2d54255d18

Localisation updates from https://translatewiki.net.

Change-Id: I85b324ab91de1c83d35657233501f7439e8c2842

mw.htmlform: Fix 'hide-if' disabling OOUI fields

Follow-up to 5d5dbc83da4bc339e6c2f7fb3d346ba259192b68.

The logic here is supposed to be the same as in the block above, but
it was inverted. I think this code used to be `.setRequired( false )`
at some point, and I changed the 'setRequired' but forgot the 'false'.

Change-Id: Iabb112d3d0519df079bc2c9afab7d4436fff6562

Un-deprecate ArticleAfterFetchContentObject hook

Accidentally done in I58a0c46b95b8ef02ff55851cfdec7e9d6eeed83c

Bug: T154672
Change-Id: I900232ea01a434b40091fca4627739310d7b3f1d

Merge "Added "autoHideSeconds" setting in mw.notify()"

Special case opus mime detction

Bug: T151352
Change-Id: I44be5d6b407752b39fc8148d31e9f01de28014dc

Add custom InterwikiLookup to MediaWikiTitleCodec

This patch allows for a custom InterwikiLookup service
when constructing MediaWikiTitleCodec instances.
If not specified, it continues to get the service from
MediaWikiServices::getInstance()->getInterwikiLookup().

This patch will allow a fully customized instance
creation, without any global state.

Change-Id: Ica87aff5df4534aae0a32e307b27d88b3df023b4

Added "autoHideSeconds" setting in mw.notify()

"autoHideSeconds" is now option of Notification, instead of global variable.

Bug: T155228
Change-Id: If0d141b67515546f1605f24c2f4a50a05d5f2ba4

Fix namespace handling for uncategorized-categories-exceptionlist

Some categories have a namespace at begin, which is not handled correct
when just using Title::newFromText

Change-Id: I7e12b66028a8e8292473e3a58e2b271c7f3a2274

Merge "Add parameter to API modules to apply change tags to log entries"

What's %MediaWiki? Remove random % from comment

Change-Id: I5e13a7b408885c390bb0b7b3cc1f585d43e327b2

Remove wiki.phtml

Change-Id: Ic2d422e16e3db236cf26731eafcec18f29cbcfb0

Merge "Remove check for MW_NO_SETUP"

Merge "Move libs/time/defines.php inclusion from GlobalFunctions to Defines.php"

Merge "Remove fully initialized debug message, dubiously useful"

Move libs/time/defines.php inclusion from GlobalFunctions to Defines.php

* Verified none of these constants depend on each other.
* Move them to the top of Defines.php for consistency.

Change-Id: I19c77e706d69bff513e4abafd0300207de0dbb1d

Remove check for MW_NO_SETUP

This protection is in place for a single extension, Maintenance.
Said extension is probably broken anyway and has been for quite
some time.

Plus, it doesn't even really work like it would intend, the extension
uses special pages and this is a require_once, so a subsequent
request to WebStart.php (which isn't even called by extensions)
wouldn't re-require it.

tldr: This is pointless

Change-Id: I22e7418d2b46c00d4009c370c24ac4b8bc43190a

Merge "Remove random check (and possible inclusion) of Autoloader from Setup"

Remove fully initialized debug message, dubiously useful

Change-Id: I406e44a3751788c4282f5cb22377d258f06671c3

Remove deprecated mediawiki.UI Less variables

Removing Less variables, which have been deprecated since MW 1.24, and
are not longer in use in any Wikimedia projects as of today.

Bug: T154950
Depends-on: I9bcd90ebbebbaece13e0cf9f36afd5964d49dc3b
Change-Id: Icbf146eb409b85a89c4dc4fb8b46eda4cf2d3193

Remove random check (and possible inclusion) of Autoloader from Setup

This is not even possible. You would've fatal'd 800 ways to Sunday
before you get remotely this far.

Change-Id: Iaf469e56394db93215af1aa54dc96876d5079ea9

Remove deprecated `.mw-ui-constructive` class for text and anchors

Removing deprecated, no longer in use `.mw-ui-constructive` modifier
CSS class for mediawiki.UI text and anchors.

Bug: T155203
Change-Id: I9bcd90ebbebbaece13e0cf9f36afd5964d49dc3b

Add parameter to API modules to apply change tags to log entries

Adds support for tagging log entries for the block, import,
managetags, and move API modules, using a 'tags' parameter.

Bug: T97720
Change-Id: I9d75d2cece317a7704c4bc6d734ad3cafe24544e

Localisation updates from https://translatewiki.net.

Change-Id: I1b63f27592e62509f88b38ed8ab282208cc59f46