Simplify HTMLTitleTextField::validate

Title::makeName already provide a way to build a title string with
namespace text

Change-Id: I21518ca9d7c6101fc866f8d667a88cc7bdf5ae7c

Merge "Inject SpecialPageFactory into Parser"

Merge "Pass through 'helpInline' to OOUI FieldLayout and make true by default"

Merge "mediawiki.util: Only call $.fn.updateTooltipAccessKeys if accesskey is set"

Localisation updates from https://translatewiki.net.

Change-Id: Ie7ab020ed7722038335b1a27c6faaaa7e07f8f3f

Merge "Mass conversion to SpecialPageFactory service"

Merge "Add index on rc_this_oldid"

Merge "Improve profileinfo.php documentation"

Pass through 'helpInline' to OOUI FieldLayout and make true by default

This change is meant to improve the layout of Special:Preferences in
particular, but it will affect any OOUI form using help messages. It
should be a harmless or beneficial change for most of them.

Previous behavior can be restored by passing `'help-inline' => false`
to the HTMLForm factory after 'help-message'/'help-messages'/'help'.

For example:

* Special:Preferences?ooui=1#mw-prefsection-watchlist
  * Before: https://phabricator.wikimedia.org/F25025213
  * After:  https://phabricator.wikimedia.org/F25025181

* Special:ChangeEmail
  * Before: https://phabricator.wikimedia.org/F25073327
  * After:  https://phabricator.wikimedia.org/F25073328

* Special:BotPasswords/foo
  * Before: https://phabricator.wikimedia.org/F25073324
  * After:  https://phabricator.wikimedia.org/F25073326

Bug: T181854
Change-Id: Ica67fe4081dfaa8eb9e8f56fdb93530750e47012

Merge "jquery.expandableField: Remove module unused since 1.22"

Merge "resourceloader: Increase minification cache version"

Merge "JavaScriptMinifier: Fix bad state after ternary in object literal"

Merge "jquery.suggestions: Remove dead code for returning options"

jquery.expandableField: Remove module unused since 1.22

Was originally created for Vector, which was originally part
of MediaWiki core. It hasn't been used for a while, and Vector
is in its own repository now.

Found 0 uses of this anywhere in Wikimedia Git, nor elsewhere
indexed by MediaWiki Codesearch.

Bug: T202154
Change-Id: I72ab5ad7f25be87b2f6d2f5f41a672b6a48b8005

Inject SpecialPageFactory into Parser

Change-Id: I6a6a94cbdafdc724ce02408cd9e744e7b3eda92b

Mass conversion to SpecialPageFactory service

Change-Id: Ia6e1e819ec6cbe8bf75b820109f51d47863e31fc

Merge "Make SpecialPageFactory a service"

Merge "Deprecate wfGetCache() and wfGetMainCache()"

Merge "Deprecate $wgParser"

Merge "Add release notes for $wgContLang deprecation"

Make SpecialPageFactory a service

Calling SpecialPageFactory methods statically is now soft-deprecated.

SpecialPageFactory::resetList() is a no-op, and I changed tests
in core to use overrideMwServices() instead.

Methods that fell back to $wgUser now require a User object being passed.

Depends-On: Ie1f80315871085b9fd4763a265b588849d94414d
Change-Id: Id8a92d57743f790b7d8c377c033cef38d1bb24de

Deprecate wfGetCache() and wfGetMainCache()

The former is already a wrapper around ObjectCache::getInstance().

The latter was identical to ObjectCache::getLocalClusterInstance().

Bug: T115890
Change-Id: Ib4e43bc8d3f4ac9f7a453e36dcce9b3d962666ba

Deprecate $wgParser

Bug: T160811
Change-Id: I7bcf4bbd14268ec069d97848c016e0098eaf0be4

jquery.suggestions: Remove dead code for returning options

    returnValue = (key in obj) ? undefined : obj[key];

By standard, this cannot return anything other undefined.
This code has been broken since the introduction of the module
in 2012 (32377424b, r72349).

The functionality also didn't have test (naturally), and isn't
used anywhere in Wikimedia Git. Simply remove it.

Also remove the confusing $() wrapping of the non-getter return
value that provides chainability.

The 'this' inside a jQuery method is already an instance of jQuery,
pulling it through $() again achieves very little.

Change-Id: Id13861d42b65a759d74670d8c5850aecee4daf7d

Add release notes for $wgContLang deprecation

Bug: T160815
Change-Id: I4dfdd8516b243f3eeccb69fe518a932178534aa2

mediawiki.util: Only call $.fn.updateTooltipAccessKeys if accesskey is set

Avoids overhead of mw.Message#parse, jqueryMsg.parser, and various
other DOM operations.

Bug: T202151
Change-Id: Id6304c9223806af6d469d9b4c67e60263f4b17ac

Message: Fix error on getTitle() without language

Steps to reproduce:

Old:
> $msg = new Message( 'brackets' )
> print $msg->getTitle()
Error from line 478 of /.../includes/Message.php: Call to a member function equals() on boolean

New:
> $msg = new Message( 'brackets' )
> print $msg->getTitle()
MediaWiki:Brackets

This got broken in 796d62d0.

Change-Id: I52cde36787e348378c4df18d90aa577a5600e06b

Improve profileinfo.php documentation

* Move $wgEnableProfileInfo to DefaultSettings.php

* The configuration variable to enable the entry point was added
  together with the entry point itself in 9af3c09e5c (r9846).

* Change references to StartProfiler.php to refer to LocalSettings.php,
  given the former is deprecated since 1.31 (I4e8dd9558132).

Change-Id: I7ca5f2deace8645f06bebd915630c1de05c84bc5

Merge "Html: Add test coverage for inlineScript()"

Merge "languages: Add coverage for 'ar' and 'ml' normalize()"

Merge "mediawiki.jqueryMsg: Combine the two generated data scripts"

Merge "mw.ForeignStructuredUpload: Use 'helpInline' instead of custom styles"

Merge "Fix eslint warnings and switch to error code"

Merge "Warn against certain jQuery methods"

Merge "benchmarks: Create $normBenchs before the run instead of during"

mw.ForeignStructuredUpload: Use 'helpInline' instead of custom styles

Before: https://phabricator.wikimedia.org/F25025765
After: https://phabricator.wikimedia.org/F25025764

Change-Id: I423c786e30570b8febc888b61450e8fce9185ba6

Merge "Remove incorrect timezone conversion from date parameters"

mediawiki.jqueryMsg: Combine the two generated data scripts

No need to output two mw.jqueryMsg.setParserDefaults() calls.
Follows-up 7e65b6b3a766c1.

Also:
* Fix an inline comment that referred to a non-existent method.
* Document why we pass deep=true.

Change-Id: I7414d4188a4d933e2132f80135cbda87241580c0

Localisation updates from https://translatewiki.net.

Change-Id: Icc72d5230b3c3c09c5095b52cc3d2191751a109f

Merge "rdbms: remove some dead code in LoadBalancer::setDomainPrefix"

benchmarks: Create $normBenchs before the run instead of during

Separates the concerns a bit better, and also makes the code easier
to debug with less distracting steps during the running of the
benchmark.

Change-Id: Ia5a18216cb77d39fd60cd76b23ebceee7324a250

resourceloader: Increase minification cache version

Follows-up 1f5f6fc204.

Bug: T201606
Change-Id: I0b5af067d1d44880c8122343f8e4a1b47e998619

JavaScriptMinifier: Fix bad state after ternary in object literal

The following pattern of input (found in jquery.js) triggered
this bug:

    call( {
        key: 1 ? 0 : function () {
            return this;
        }
    } );

The open brace changes state to PROPERTY_ASSIGNMENT (for object literals).
The colon after 'key' sets state to PROPERTY_EXPRESSION.

Each individual parts of an expression (identifiers and literal values)
is recognised with state *_EXPRESSION_OP, such as PROPERTY_EXPRESSION_OP.

The '1' after 'key:' correctly sets the state to PROPERTY_EXPRESSION_OP.
Upto there it goes well, but after that it goes wrong.

The question mark (TYPE_HOOK) in this context was wrongly switching
back to PROPERTY_EXPRESSION. That is a problem because that does not
handle TYPE_COLON, which meant '0: function' was seen together as a
sequence of continuous PROPERTY_EXPRESSION_OP where TYPE_FUNC may
not be handled.

Fixed by changing handling of TYPE_HOOK in PROPERTY_EXPRESSION to
switch states to EXPRESSION_TERNARY, and also performing a push
so that ternary handling can pop back to the property expression.

This mirrors the handling that already exists for ternaries in
the regular handling of EXPRESSION/EXPRESSION_OP (as opposed to
the variant for object literal properties).

Bug: T201606
Change-Id: I6104c839cfc3416257543b54a91b74cb4aa4193b

rdbms: remove some dead code in LoadBalancer::setDomainPrefix

Change-Id: I594a390b446c00eb604b0be63f67b2c8cbd45758

title: Disable the failing tests from TitlePermissionTest

Bug: T201776
Change-Id: I088bd797225e0c60c66de4d4d1aa12d0b5bf67d8

Merge "Set getEditConflictMainTextBox parameter type to array"

Merge "selenium: undo page"

Set getEditConflictMainTextBox parameter type to array

This is only called at one place in core and used in the TwoColConf
extention. The passed argument is always an array.

Change-Id: I4ca94442b79dc8871ed72943be733d100e093e44

Unbreak Special:JavaScriptTest

Follow-up to dec8009

Change-Id: I812253cc1a504a8bb876700026dbfd35a727458b

Merge "rdbms: use Database::getDomainId for transaction logging calls"

Merge "Fix spelling of word "necessary""

Merge "rdbms: Disable DBConnRef::selectDB() for sanity"

Fix spelling of word "necessary"

Bug: T201491
Change-Id: Ic983889a5702b8b8552c88ebc4d676f35e9fe61e

Fix some of the common typos

* supress -> suppress (Except in backup_LogTest.php)
* recomend -> recommend
* becuase -> because
* accross -> across

Bug: T201491
Change-Id: I8faa4e6cc688b3ee204b3f79ab55eb7b65cc1fdd

Merge "Have WANCacheReapUpdate::getEventAffectedKeys() accept LinkTarget"

Merge "Actually inject InterwikiLookup into MediaWikiTitleCodec"

Merge "Have LinkCache::getMutableCacheKeys() accept LinkTarget"

Merge "Title: Access properties directly instead of wrapper functions"

Localisation updates from https://translatewiki.net.

Change-Id: I4b07075f2c8b2965f542f08cfbb62d71b23d9099

Merge "Make LinkBatch::setCaller chainable"

Merge "Update OOUI to v0.28.0"

Update OOUI to v0.28.0

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

Bug: T109529
Bug: T183601
Bug: T183645
Bug: T185751
Bug: T189113
Bug: T189953
Bug: T190441
Bug: T190581
Bug: T194104
Bug: T197004
Bug: T198845
Bug: T201329
Bug: T201871
Depends-on: I3c000504661e190ac8752f69419bed28785601b5
Change-Id: I0c101a942011f6095678ffbd90f9bd536132091c

Merge "Add option to populateChangeTagDef not to update the count"

Merge "Remove dead code from Title::secureAndSplit()"

Merge "TitleValue: Don't use Assert for basic type checks"

Merge "Have TitleParser::parseTitle() default to NS_MAIN to match Title"

Add option to populateChangeTagDef not to update the count

I need this when I'm picking this up to continue populating

Change-Id: I52c74d3c1c59ee966027d5d9cb1a2bd3cf6dfc7a

Have WANCacheReapUpdate::getEventAffectedKeys() accept LinkTarget

Nothing specifically requires TitleValue here.

Change-Id: I62ee1995c49071db2871149b941277b29836a94a

Make LinkBatch::setCaller chainable

To allow for one-liners such as $lb->setCaller( __METHOD__ )->execute();

Change-Id: I45cd81d9e583f09aacb578ffb87d7c11b0e13d57

rdbms: use Database::getDomainId for transaction logging calls

Change-Id: I360ca633b145dc8e510bb00b542ac44933283943

Actually inject InterwikiLookup into MediaWikiTitleCodec

Change-Id: Ic469f0fb8068bfb06b741d957b21e66dfa839cd6

Merge "rdbms: split out private LoadBalancer::openLocalConnection method"

Have LinkCache::getMutableCacheKeys() accept LinkTarget

Change-Id: Id84590b3aa285e31a358877167fe8ff502077f3c

Title: Access properties directly instead of wrapper functions

Since this is pretty hot code, avoid the performance hit of a function
call when we can directly access the property.

Change-Id: I287de5eb003582b5704e9d93e2fc99d16c0ffb2a

Merge "Make ExternalStoreDB "wiki" context override the server "dbname" field"

Merge "JavaScriptMinifier: Merge $push and $pop into $model"

Remove dead code from Title::secureAndSplit()

Change-Id: I08810f0ab20fc1608f2d3b2248dcd5106044080b

rdbms: split out private LoadBalancer::openLocalConnection method

Change-Id: I2138b3d28f8f1a8ea706165fbe3a7e3f0d858447

TitleValue: Don't use Assert for basic type checks

Previously, `new TitleValue()` was roughly twice as slow as
`Title::makeTitle()`. Switching to basic is_int/is_string checks
makes TitleValue roughly twice as fast as Title!

Tested with benchmarkTitleValue.php.

Bug: T201801
Change-Id: Idccf159983145c1c11af9182292e1d5f704ba077

Have TitleParser::parseTitle() default to NS_MAIN to match Title

Matches the default parameters of Title::newFromText().

Change-Id: Id08312d424be9ae7e64688bb21687204ae77c560

Merge "MergeHistory: Fix flaky test due to relative timestamp"

rdbms: Disable DBConnRef::selectDB() for sanity

Bug: T193565
Change-Id: I4276d1a7d77a019e0e60dab4b9ec36c93e418037

Make ExternalStoreDB "wiki" context override the server "dbname" field

This is all that is needed for b/c and going beyond that can break
foreign wiki (domain) external store access if matching per-wiki DB
names *are* used for external store.

Follow up to 92e4ace7eae61.

Bug: T200471
Change-Id: I877139ff659f542da04d4d8f5ef3297dbfcfd734

Merge "DateTimeInputWidget: Rename LESS var to align with WikimediaUI Base"

Merge "resourceloader: Remove $wgResourceLoaderLESSVars support"

MergeHistory: Fix flaky test due to relative timestamp

In PHP, `strtotime('tomorrow')` evaluates to the first midnight
after the current time.

Which means on Aug 14 at 23:59:59, strtotime('tomorrow') refers
to a timestamp merely 1 second in the future (Aug 15 00:00:00).

This causes the test to fail because this timestamp is computed
in a data provider, before the test itself starts. In order to
pass, it needs to be far enough in the future that it will also
be after the point in time where the test creates the page and
revisions in question.

Fix this by:
* ... moving it into the test itself, where even plain time()
      should probably suffice, but that's for another time.

* ... using +24*3600 instead of strtotime().

Bug: T201976
Change-Id: Ice5d54af4fb7cffa8db9b657796ba6ebecdaffa0

DateTimeInputWidget: Rename LESS var to align with WikimediaUI Base

Change-Id: I1d3a33cb9911250793dc07758d384d7a625de1c5

Merge "Parser: Call firstCallInit() in getTags/getFunctionHooks"

Merge "Fast manual restart for populateContentTables.php"

Merge "RemexDriver: improved tracing"

Merge "password: Move commonpasswords.cdb to includes/password/"

Merge "collation: Move first-letters-root to includes/collation/data"

Merge "language: Add generate-normalize-data maintenance script"

languages: Add coverage for 'ar' and 'ml' normalize()

* Exclude the data files from PHPUnit coverage.
* Add tests covering the normalize() implementations.
* Fix a small todo about using data providers.
* Set explicit visibility.

Change-Id: Ib104cc3215a36901cff853ad5969d92a6e0cf6a0

Merge "Fix reset interwiki table between tests"

Fix eslint warnings and switch to error code

Follow up to I4552191f639b1022cb085ef5eb4abda835c2d91c

* Add eslint-ignores to various files
* Correct various uses of isFunction, trim, map, grep
where easy fix is available
* Switch from eslint warning to error

Bug: T200877
Change-Id: I8420db21e548772332e51769aa8ec92d5958715f

Merge "Follow-up fc88eea: Add new oojs-ui.styles.icons-editing-citation pack"

Merge "Upgrade wikimedia/remex-html to 2.0.0"

Merge "Fix common typo "MediWiki" in MW core"

Parser: Call firstCallInit() in getTags/getFunctionHooks

So callers don't need to do this manually. Pointed out by Tim in T201799.

Depends-On: Ia6c36d5a650095e35093bf47e275e081e89b3daf
Change-Id: Ida62767f3ca53f99609cae01d3a20051bb92ccab