resourceloader: Improve ResourceLoaderClientHtmlTest tests

The example for a mixed user module is outdated. Current user modules
all separate styles and scripts.

To make the effective change by the next commit easier to review,
update these tests first seperately.

Change-Id: I76a8a96eed8a8c39863c535362b1d0144ef2f866

Merge "Remove space after cast"

Merge "clone keyword must not be used as a function"

Merge "RCFilters: Fix typo in removing a saved query"

Merge "RCFilters: Correct display of save filter popup"

RCFilters: Fix typo in removing a saved query

Bug: T169107
Change-Id: I7c1535f6ae75159a069ebde7db781f6a875fcb3c

RCFilters: Followup I200720ffde8ad: Hide extended footer if gater is disabled

Bug: T169163
Change-Id: I62f63cbf06a7e42864faca1a196903eb34ea077b

Merge "resourceloader: Use "\n" instead of ";" as separator for scripts"

Merge "Add SearchIndexField::getEngineHints()"

Localisation updates from https://translatewiki.net.

Change-Id: I889cf1f5e93554486b452922edd474ef865bec7f

Paramaterize TransactionProfiler log entries

Change-Id: I4e468d818af48dba21cd2c567f4e6a58beb0924b

Record PostSend master queries via TransactionProfiler

Bug: T166199
Change-Id: Ib2f858f09aae65af6aa8324edb72d8a16094f2b3

RCFilters: Correct display of save filter popup

- Correct language in the 'apply' button
- Add a placeholder to the input
- Make the 'apply' button disabled if the input is empty
- Remove the use of the OOUI-built-in validation, since all
  we do is "validate" that the input isn't empty, and there's
  no need to show error mode (red border) for that, especially
  since the 'apply' button is disabled in that case.

Bug: T169042
Change-Id: I5e3600b1ac8e63d8a25c0540468fe42febfc3a70

clone keyword must not be used as a function

Also changed a require_once

Change-Id: Ib8030ea5320a62ee4f114d8d506382c5665e3e02

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

Merge "User: Avoid deprecated Linker::link()"

Use visibleByteLimit on Special:MovePage

Change-Id: Ie9e2bef8397851ca5899bfb7a3849853d07590ff

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

Use Devanagari digits for list items on mai.wikipedia

Bug: T168781
Change-Id: I0f715daaacbe2cee236d28866b23893c342922a2

Merge "Remove exception and drop cloned tables in PostgresSQL too"

RCFilters: Followup  I436f00bc87db: Remove onPopupToggle listener

Bug: T168828
Change-Id: Ie6d73f38335ff1c5bc88602119ddf3e2138eba60

resourceloader: Use "\n" instead of ";" as separator for scripts

This fixes two bugs:

* 1) When two modules are requested, and the first one ends with ";"
     inside a comment, the second module might become part of
     that comment.
* 2) A request with script=only where the requested module content
     ends in a statement without ";", but has a comment after it
     that does ends with a semicolon, then in debug=false, mw.loader.state()
     would be appended directly after the semicolon-less statement because
     the check is performed before minification.
     Previously:
     script> foo()
     script> // bar();
     states> mw.loader.state( {} );
     Became (minified separately):
     script> foo()
     states> mw.loader.state({});
     Became (concatenated)
     > foo()mw.loader.state();
     Which is invalid code.

Both of these are now fixed.

Bug: T162719
Change-Id: Ic8114c46ce232f5869400eaa40d3027003550533

resourceloader: Add more concat test cases for makeModuleResponse

Bug: T162719
Change-Id: Ic8d4a022dbe8c7998924fb20d727936e68cb11d2

Merge "Record TransactionProfiler violations beyond the first-by-type"

Merge "Allow SVGs using an older proposed recommendation DTD"

Remove ParserOptions::optionsHashPre30()

The pre-1.30 version of ParserOptions::optionsHash() was kept
temporarily as ParserOptions::optionsHashPre30() to prevent a cache
stampede on WMF sites when the hash format was changed in I7fb9ffca9.

Now that the cache has been rebuilt, it's no longer needed and we should
clean it up instead of leaving it forever to bitrot.

Change-Id: I037d8dfdefe72a295547bd331bc1454e69cb418d

Merge "Add constants for schema migration feature flags"

Merge "benchmarks: Add benchmark for JSMinPlus"

Record TransactionProfiler violations beyond the first-by-type

Beyond giving a more complete picture, this can be used to adjust
the PostSend criteria to log master queries under the restInPeace()
entry point.

Bug: T166199
Change-Id: If5cf704ba10291d0eba63e66b388666fde3ca649

Merge "Fix SearchEngineTest test descriptions"

Merge "Fix highlighting for phrase queries"

Merge "Remove "@author Aaron Schulz" annotations"

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

Remove "@author Aaron Schulz" annotations

Bug: T139301
Change-Id: Ib5248e8e27d60611c7373bce4b29dd5e85aa3489

Merge "benchmarks: Add reporting of resource usage"

benchmarks: Add benchmark for JSMinPlus

Example usage

> mwscript maintenance/benchmarks/benchmarkJSMinPlus.php --file resources/lib/jquery/jquery.js -v

Bug: T31784
Change-Id: I3c11574adb864a4bfbfceba137e285fdb8dc66d2

Merge "Fix typo in comment."

Merge "tests: Use TestingAccessWrapper to reload LanguageConverter tables"

Merge "tests: Reset LanguageConverter conversion tables between test cases"

Merge "RCFilters: Move legend into results (both js and nojs output)"

Fix typo in comment.

Change-Id: Ia656c55ca22fa30b5a911b8cb3a6218c916d3316

tests: Use TestingAccessWrapper to reload LanguageConverter tables

Make the LanguageConverter::reloadTables method actually private,
and use the TestingAccessWrapper to call it when running parser tests.

Follow-up to I65736520cd04bfe8949b29ade07338a6e1b88a4d.

Change-Id: I43b81b8fef6441ad50b858ff7757732ecb5eef91

Merge "Fix border-radii on FilterTagMultiselectWidget"

Merge "RangeChronologicalPager: Don't clear $offset when $this->rangeConds is set"

Localisation updates from https://translatewiki.net.

Change-Id: I27707dbaae9edf04ef400300801c339a22df3eb3

Merge "RCFilters: Always use the filled bookmark icon (unClip)"

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

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

RCFilters: Move legend into results (both js and nojs output)

Move the legend from the fieldset area to the results area.

Bug: T168043
Change-Id: I40454b194fe3975d8d4ac13d96774bfa5b5d5a46

Merge "RCFilters: Make extra filters footer non-sticky"

Merge "RCFilters: Correct default view title"

Merge "RCFilters: Remove <hr> if extension table is empty"

RCFilters: Make extra filters footer non-sticky

Bug: T168851
Change-Id: I504847e4b07d473786a4026b053a55ae460897e8

Avoid checking double-prefixed table names in mysql tableExists()

Change-Id: I9d91a5305227171d0776ffa443d4e538fbe1b15f

Remove space after cast

One remaining file was fixed

Change-Id: I95f77028219046e14c887cd2e0d5f6ffe98b9cd8

Fix SearchEngineTest test descriptions

These should describe the test not the failure.

Change-Id: I82601795d925d3dd7548909f18c325eeaebd83ff

Allow SVGs using an older proposed recommendation DTD

Dia software seems to use this DTD (at least in some versions)

Bug: T168856
Change-Id: I51ad7ff4a935d4edb78e091142be9c58017dd3af

Merge "resourceloader: Add @covers for stripBom and makeModuleResponse tests"

Merge "resourceloader: Add @covers for stripBom() test"

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

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

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