Merge "build: Prepare for mediawiki/mediawiki-codesniffer to 0.9.0"

Merge "Remove "@author Bryan Davis" and "Brad Jorsch" annotations"

resourceloader: Add basic tests for getScript() and buildContent()

Bug: T162719
Change-Id: I37d64da77682adfef61e78033d639b623d7c9c2b

resourceloader: Add @covers for stripBom and makeModuleResponse tests

Change-Id: Ie32178cdd03a79a0ab871122acf081c65e5a9f4d

resourceloader: Add @covers for stripBom() test

Change-Id: I3403b6617121f5965ef7aa125eb2528db7c89e95

resourceloader: Complete test coverage for FileModule::getTemplates()

* Add coverage for the 'else' branch for file not existing and
  an exception being thrown.

Change-Id: Ia7d608f1ed72f07f46b969046d9c9910fbfd738d

User: Avoid deprecated Linker::link()

Change-Id: Id50615c1cd4c49ea7b04747f15e353448e971b66

DidYouMeanWidget: Don't use deprecated Linker::linkKnown()

Change-Id: Ie5464c9d8e3207e7d307bbd31f3c3026ac18fcf2

RCFilters: Remove <hr> if extension table is empty

Bug: T168325
Change-Id: I5f537722aeef8d52cd5b0555519be97a7675491f

Merge "RCFilters: Add 'single_option' group type"

RCFilters: Correct default view title

The key was changed from "label" to "title" but that was not changed
in the default view. Change the definition key and adjust the name
of the method so it isn't confusing.

Bug: T168855
Change-Id: Ib94f57d3bcce94a02fac25d66b893a68aa648e84

RCFilters: Add 'single_option' group type

Group type that only allows a single option to be selected
from its range of items.

Bonus: Add the ability to have a view that is hidden from
the menus

Bug: T162784
Bug: T162786
Change-Id: Ide93491a49c1405926ac171c7924a469e94c0e0a

RCFilters: Simplify "Saved filter settings" to "Saved filters"

Bug: T167876
Change-Id: I5beb2433a9fb0ba8493ec571c1c4d37cd4ee26ba

RCFilters: Always use the filled bookmark icon (unClip)

Bug: T168828
Change-Id: I436f00bc87db143116125a185abd8000132df195

Merge "Replace some usages of Linker::link with LinkRenderer"

Merge "RCFilters: Invert the muted state of inverted namespace options"

RCFilters: Invert the muted state of inverted namespace options

Since the inversion happened, the background of selected items was
overridden by "muted" background, so we had to change a few more
details to get things to work properly:
- Override selected color even if the item is muted (previously,
  'selected' cannot be muted by definition, but this patch changes
  that and hence needs to make sure selected color overrides the
  muted color state)
- Set correct muted state on the namespace items to be reversed.
- The highlight/select/mute background issue exposed a secondary
  issue when switching views by clicking the 'tag' item, whereby
  the change of view itself causes the input to empty, and triggers
  the updateItemVisibility in MenuSelectWidget, which then selects
  the first item. This shouldn't happen when the user is actively
  selecting an item from the tags, so a method of recognizing that
  the user is actively selecting an item was added.

Change-Id: Idf5c2232216ad6a38b29bbc103bbca66f56533b7

Remove "@author Bryan Davis" and "Brad Jorsch" annotations

Undo traces of a practice we carried over from past projects and
existing examples that is neither universal nor actively encouraged in
the MediaWiki codebase.

Bug: T139301
Change-Id: I5c9c89b72a45a44aa4264a5e57b003c1a86cdf6e
Co-Authored-By: Brad Jorsch <bjorsch@wikimedia.org>

Merge "mw.filewarning: Increase the area of the warning button"

Merge "mediawiki.Title: Remove needless $.type() and minor clean up"

Localisation updates from https://translatewiki.net.

Change-Id: I457987aceec9e936608988e63a88686d1e98675d

tests: Reset LanguageConverter conversion tables between test cases

Conversion rules defined in a previous test case were leaking into
subsequent test cases.  Existing tests had worked around this by defining
non-overlapping rules, but it's better to just fix the problem at the
source.

Change-Id: I65736520cd04bfe8949b29ade07338a6e1b88a4d

build: Prepare for mediawiki/mediawiki-codesniffer to 0.9.0

The used phpcs has a bug, so the version 0.9.0 could not be enforced at the moment.
Will be fixed in next version, see T167168

Changed:
- Remove duplicate newline at end of file
- Add space between function and ( for closures
- and -> &&, or -> ||

Change-Id: I4172fb08861729bccd55aecbd07e029e2638d311

Downgrade 'found writes/callbacks pending.' from error to info

This generates useless logspam and should be logged with a lower
log level.

Bug: T149353
Change-Id: If5e84b89eca098ac7f8c788ca3b0d268adf799fb

mw.filewarning: Increase the area of the warning button

So that the user can move the mouse cursor to the popup
without the popup disappearing.

Bug: T157544
Change-Id: Ide9db6f6fd092caeea1bacfc97ec79f9acc3793e

Merge "Parser: Emit deprecation warnings for ParserLimitReport hook"

Fix highlighting for phrase queries

I think the bug was introduced during a cleanup in Iaabc10c.
I don't think that " should be part of the legalSearchChars at query
time, it seems to break the regex.
The strategy here is to distinguish legalSearchChars used query time vs
the ones used at index time by introducing:
SearchEngine::legalSearchCharsForUpdate()

Bug: T167798
Change-Id: I61dc53665e26d3c6c48caed78dd3bbde9a33def7

Merge "Perform input validation for file $page at some strategic locations"

docs: Use oojs-ui-wikimediaui, instead of -mediawiki

Change-Id: Iac6a526c928fe384f5f7df5af6b0156146bf2f33

Localisation updates from https://translatewiki.net.

Change-Id: I3311079076af3f9bad7b06c4ac0a532a2ecd75ba

Merge "Update weblinks in comments to msdn.microsoft.com"

Merge "Rename canTalk methods"

Merge "Parser: Ensure function and tag hook callbacks are callable upon registration"

Merge "FileDeleteForm: Create log entry even if the page didn't exist"

Rename canTalk methods

This renames Title::canTalk to Title::canHaveTalkPage
and MWNamespace::canTalk to MWNamespace::hasTalkNamespace.

Bug: T165149
Change-Id: I342a273a497b31282388b13bf76dadfb1122dcbb

Parser: Ensure function and tag hook callbacks are callable upon registration

This is to safeguard against issues like T167238, where the callback was invalid,
but was only noticed when the magic word was used in an article.

In general, type checks should be applied when things get
registered for later use. Fail early, stay sane.

Change-Id: Ifb7005ee214829c98cec534261c0db7d13f50f35

Merge "API: Allow deleting files without corresponding pages"

Remove newline at end from english messages

Newline get lost on translation, but it seems there are not needed.

Change-Id: I51cfc598052d4d9b457c2d7a4f25178d580d8ee3

Localisation updates from https://translatewiki.net.

Change-Id: I18ef4dcda34a029f16c6a85c3474eeef6a71502b

Parser: Emit deprecation warnings for ParserLimitReport hook

Change-Id: I351f0867132acf8d5cc9ee51b575744445b28b77

mediawiki.Title: Remove needless $.type() and minor clean up

* Use typeof instead of $.type() for primitives.
* Remove needless type check before instanceof.
* Use early-return pattern.

Change-Id: Ibfdbc3912961fd017cefb7115951f07b92f6d435

Merge "API: Fix capitalization of SpecialRecentChanges* classes"

Localisation updates from https://translatewiki.net.

Change-Id: Id56fbcea3887cfa2a0ba2a3bdc83a180c20bdc92

API: Fix capitalization of SpecialRecentChanges* classes

And use the magic ::class instead of plain strings.

Noticed by logging added for T166759.

Change-Id: I7694c272c255b5b1f7455e55e70ecc240a4ba61c

SpecialContributions: Fix capitalization of DateInputWidget class

The namespace was improperly capitalized, move it to a "use" statement
while we're at it so it's only included once.

Noticed by the logging added for T166759.

Change-Id: I3fe6533a0f06dcbc37017361679dad9e77808830

Merge "RCFilters: Unify reading filters by views and adjust unit tests"

Merge "RCFilters: Change the input icon on interaction"

RCFilters: Unify reading filters by views and adjust unit tests

- Have the initialization method read filters by views and treat
  all filters, regardless of their views, the exact same, including
  evaluating their subsets and conflicts.
- Clean up unit test file and make pass with the new initialization.

Change-Id: I200720ffde8ad8ff7a63d0cd82615d358e74949a

Factor out byte limit label logic from edit page to utility function

Other input widgets have byte limits and may want to display the counter.

Change-Id: I61b72ab75007d80b9e0bba0f391c88cbac2bce3d

Merge "Set $wgGalleryOptions in Setup.php"

Merge "Use $.noop instead of function () {}"

Set $wgGalleryOptions in Setup.php

Defaults will be added to the config array at a later point,
without overriding any that have already been set.
This'll make it less prone to errors in case $wgGalleryOptions
is incomplete.

Bug: T168479
Change-Id: I5f127e5ac8c8a1583301dd7aed2ae8e434987950

Merge "resourceloader: Backport jquery-migrate.js patch for exceptionHook"

Use $.noop instead of function () {}

$.noop (https://api.jquery.com/jQuery.noop/) already provides an empty
function.

Change-Id: I7f1ca20a894d0469cb7bcb72c81722070e040956

Fix border-radii on FilterTagMultiselectWidget

Change-Id: I14211db541f02a48e7aa0a95f967d7882c59280e

Merge "RCFilters: Reposition TagItemWidget to be center on the widget"

Merge "RCFilters: Have the model accept multiple views"

Merge "RCFilters: Correct unbalanced margins in active filters display"

Merge "RCFilters: Remove view triggers before checking emptiness of string"

Merge "RCFilters: Change to the new views redesign"

RCFilters: Change to the new views redesign

- Allow for multiple footers for the menu, each configured
  for its own view (or multiple views)
- Move the switch view buttons to the right of the input
- Add a 'back' button on the menu header when we're in
  advanced filters view, moving back to default view

Bug: T167384
Change-Id: I9fd0243f88f92ddacb4c912ff974f7d325f32f5d

resourceloader: Backport jquery-migrate.js patch for exceptionHook

jQuery Migrate has to monkey-patch jQuery.Deferred, but in doing so
it also effectively disables exceptionHook, which means exceptions
are no longer logged to the console.

<https://github.com/jquery/jquery-migrate/pull/262>

Bug: T168086
Change-Id: Id46abcc7fb5e65954532b114636bf0f1f000a551

Merge "RCFilters: Actually recognize a saved query default and delete when removed"

Merge "Fix the tableExists method of MysqlBase"

Fix the tableExists method of MysqlBase

The table name generated by Database::tableName() does not work in
MysqlBase's tableExist method, as the syntax of SHOW TABLES does not
function like other foreign db queries. It instead should use FROM
$database rather than the database name in front of the table name.

Bug: T168207
Change-Id: I7806090eaa647959fd34de8bc606eeb952161529

Localisation updates from https://translatewiki.net.

Change-Id: Ie2c6873b323f3668546ee1467865fdb342900a77

RCFilters: Have the model accept multiple views

Organize the way we create and process multiple views, to stop
hard-coding namespaces and tags.

Change-Id: Ib28c166052781b23a3b35c1b7b0af7d26f102e3f

RCFilters: Actually recognize a saved query default and delete when removed

Compare ID to saved default even if those were saved in different
formats (number vs string) when setting default on initialization
and when deleting the default when deleting the item.

Change-Id: Ia211a7166b3ff514c0e2a34bb370cf2d50057745

Merge "Linker: Remove whiny FIXME comments"

Linker: Remove whiny FIXME comments

These are not actionable. Such is life, deal with it.

Change-Id: I81df045f54052b98cf798d3d42a3d730407d68ae

Merge "API: Display message-per-value style help for submodule parameters"

Merge "Add 3D filetype for STL files"

Merge "Remove execute bits from non-executable files"

Remove execute bits from non-executable files

Change-Id: I4d6a202c63bef3e52b67b55fb6daef04dc2fdd62

CollationFa: Avoid PHP 7 Unicode escape syntax

We still support PHP 5.5.

Change-Id: I587cb794cded95afe7ad493614a6090a108efe6c

Merge "Ability to create tests with nested modules"

Add SearchIndexField::getEngineHints()

Allows search engine clients that implement custom definitions
to pass engine hints used at index time.
Hints are a way to fine tune the behavior of the search engine
when handling a particular field.
As of now this is introduced for CirrusSearch to let SearchIndexField
implementations to control how the noop script is configured.
The noop hint with CirrusSearch allows to (for example):
- ignore an update if a numeric value does not change for more than X%
- control the merge strategy of complex fields

Bug: T166589
Change-Id: Ia560e41d33013c30ac47e5a60543f8cb133e61fb

Merge "RC Filters: Fix tags for integrated filters"

RCFilters: Remove view triggers before checking emptiness of string

Bug: T168220
Change-Id: I3ae6fba358456745cb570840211319f7eac754ce

Localisation updates from https://translatewiki.net.

Change-Id: If8ff147d26d1ccf1d4c540ee7bc25cabead1da5a

Merge "resourceloader: Add unit tests for ResourceLoader::isFileModule"

Merge "Allow $wgResourceModuleSkinStyles for any ResourceLoaderFileModule subclass"

resourceloader: Add unit tests for ResourceLoader::isFileModule

Change-Id: Ic21ae9e9cc418868ad7b93fe65bd09495f38d1b2

Merge "Make MalformedTitleException errorMessage non-null"

Merge "Fix enhanced RC data attribute sanitizing"

Allow $wgResourceModuleSkinStyles for any ResourceLoaderFileModule subclass

All ResourceLoaderFileModule subclasses are now required to support
the 'skinStyles' property in their module definition, and ideally
make use of it (but at least they must not fail violently).

Follow-up to 51eede02837a3a9d378cb35fa7c27a9f4e8e5f4e.

Bug: T167478
Bug: T168088
Change-Id: I35a12a451bf2695818702df1bbd1708173a3f9ce

API: Don't handle non-preflight OPTIONS as CORS

If it's not a preflight, it's either a simple request or an actual
request after a preflight. But simple requests can only be GET, HEAD, or
POST, and we only return a successful response for a preflight if the
actual request is going to be GET or POST. So either way it shouldn't be
handled as CORS.

This also adds a response header to indicate to the end user why
something they probably intended as a CORS request wasn't handled as
one, to help with debugging.

And it renames a local variable that confused me when looking back at
this code.

Bug: T168558
Change-Id: Ia66c74e1cc8536c6c478237540abb4d7a98b9803

Merge "Factor out new qualifiedTableComponents() Database method"

Merge "Load mediawiki.action.edit.styles on mobile"

Make MalformedTitleException errorMessage non-null

A null $errorMessage constructor argument has not been supported since
I43d988602b, since wfMessage (via the Message constructor) throws an
exception if the message key is null. It follows that getErrorMessage()
can never return null.

Bug: T99818
Change-Id: I2cbf4909e7237b0a91d100cc5478fb4cfee1d748

RC Filters: Fix tags for integrated filters

Change-Id: Ibf06ffe9338b12951caa7259304594d4fcc928d5

Factor out new qualifiedTableComponents() Database method

Change-Id: Ib453a5a8e0cf8bb1b77e65eb6f9569819d4eb5b2

Load mediawiki.action.edit.styles on mobile

Hopefully uncontroversial.
Mobile non-js editing is currently feature flagged
by $wgMFAllowNonJavaScriptEditing so this shouldn't have
any impact on production but will ensure the editor gets
styled consistently with future changes here.

Change-Id: If8994370e9ec7fc424ce7cb22df922d029cfc035

Merge "Replace Linker::link() with LinkRenderer in special pages"

Merge "Replace a Linker::link() call in InfoAction.php"

Merge "ApiSandbox: Indicate when login is suppressed"

Merge "API: Add the ability to flag parameter values as deprecated"

Replace Linker::link() with LinkRenderer in special pages

Bug: T149346
Change-Id: I2735750803dc1b2e36be3989caaabbf3232b2923

Localisation updates from https://translatewiki.net.

Change-Id: I1d2050c1a5d2cadf8f8e152037f7b80530612dcd

API: Display message-per-value style help for submodule parameters

The module's summary will be used as the description of the value.

Since these are often very long lists, this also changes ApiSandbox to
collapse <dl> lists in parameter descriptions by default.

Bug: T123930
Change-Id: I205b68a52a94cae4c1cdf7ec9fd3e8a04d565919