Merge "Warn against certain jQuery methods"

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

Merge "Remove incorrect timezone conversion from date parameters"

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

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"

Merge "Fix reset interwiki table between tests"

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

Fix common typo "MediWiki" in MW core

Fixed typo "MediWiki" to "MediaWiki" in core.

Bug: T201491
Change-Id: I55cb4af63b72cfa08de079544c43ae9f1366a457

Merge "Feedback form: tweak field spacing"

Merge "Swap SET and WHERE statements in ChangeTags::undefineTag"

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

Change-Id: I8a0426fc288a499e40f02f1da7a9f1b09e6dbd5f

Merge "build: Update eslint-config-wikimedia to 0.7.2"

RemexDriver: improved tracing

Use the new RemexHtml trace features. Add two more tracing modes.

Fix missing member variable declarations and remove unused local
variables.

Change-Id: I512462e1019f9a466684abfa4aab7697b324d5b1

Upgrade wikimedia/remex-html to 2.0.0

Depends-On: Ie13945649314853cbd5707363f3a10da55752743
Change-Id: Ib6c8aaa797c128c273cde8095eb0bb1527fc0e21

Merge "rdbms: Avoid numRows() warnings for mysqli after table creation"

Localisation updates from https://translatewiki.net.

Change-Id: Idca283c3e5caa43b345988307d55b263e6fa9fc0

Swap SET and WHERE statements in ChangeTags::undefineTag

Follows-up 417d8036ae9.

It should work the other way around, I'm so stupid.

Bug: T201934
Change-Id: I69132b2c237d05242ec6ed1a1e3aca7886edf2bc

resourceloader: Remove $wgResourceLoaderLESSVars support

The use of global variables was deprecated in favour of
ResourceLoaderModule::getLessVars() on a per-module basis.

Also moved testLessFileCompilation case to the appropiate file as it
covers ResourceLoaderFileModule.php, not ResourceLoader.php.

Bug: T140804
Depends-On: Ib1b2808df2384473bfac47f53a5d25d7c9bbca2b
Depends-On: I96047f69d01c4736306df2719267e6347daf556f
Change-Id: If708087c85c80355c7e78f1768529b5f2e16ed07

rdbms: Avoid numRows() warnings for mysqli after table creation

Bug: T201900
Change-Id: Ie86a7b8e680d79ad3f9be6ca4ec260b0589e5d0e

Merge "resourceloader: Optimise getCombinedVersion() with string concat"

Merge "resourceloader: Remove support for `state(name, state)` signature"

Merge "Move l10n_cache table to a separate DB for sqlite via the installer"

build: Update eslint-config-wikimedia to 0.7.2

Change-Id: Ibdf9cc013e1265ef461169ec2227652eaf917900

Feedback form: tweak field spacing

Bug: T201871
Change-Id: I80ab0c7a11ef05ed063b8f58e27bd18433ea5807

Move l10n_cache table to a separate DB for sqlite via the installer

This does not set 'db' as the cache type so that admins can still
easily set the cache directory to use the file-based cdb system.
If they do not, then at least the second DB file will be used to
avoid heavy contention.

Bug: T93097
Change-Id: Ib3912f00cf12de99801ebda4f06135b2987ce71a

Merge "Avoid MapCacheLRU error when MessageCache fails to load"

Merge "Improve MapCacheLRU error message"

selenium: undo page

Add basic undo page object and test showing a previous edit can be
undone.

Bug: T201901
Change-Id: Idc1b3889abc6e655032bc1576f1b9d591abd6c8d

Fix typos

Bug: T201491
Change-Id: I25a27d11faabe2f5fa02950c7a4fb58b13fb3662

Avoid MapCacheLRU error when MessageCache fails to load

Bug: T201893
Change-Id: I6093113a3ffa8092dea3351a6ed6c815c7ff7162

Improve MapCacheLRU error message

Bug: T201893
Change-Id: I74ef2cf31d83186f68e676da1b80c4c44ca28d69

Parser: Add accessors needed by CodeMirror

Change-Id: Ia2d98baf6caed2cd779cb00aceba5785cf13d633

Fast manual restart for populateContentTables.php

Currently it takes hours to scan the table if interrupted halfway
through.

Change-Id: Ibcc04b43629039c92abf9ec2cc2a1da07685d033

Fix reset interwiki table between tests

This fixes the following SQL error, when the interwikilinks table was
marked as used:
Fatal error: Uncaught exception 'Wikimedia\Rdbms\DBQueryError' with message 'A database query error has occurred. Did you forget to run your application's database schema updater after upgrading?
Query: INSERT  INTO `unittest_interwiki` (1,2,3) VALUES ('Array','Array','Array')
Function: MediaWikiTestCase::resetDB

iterator_to_array on the result of the select of the interwiki table
returns an array with the first entry having the key 1. For
multi insert to work, the array must start on 0.
Change-Id: I3dc1be9e2f016709f451fe6e8d7cc60862c1ffea

Merge "JavaScriptMinifier: Turn $goto into a generic $model"

Merge "Make MapCacheLRU throw errors for bad $field arguments"

Make MapCacheLRU throw errors for bad $field arguments

Change-Id: Ibde33e0ff671d3428b73c66766478f58763726e1

resourceloader: Optimise getCombinedVersion() with string concat

This is used many times in the hot path. The main motivation for
this is to reduce memory churn in that path.

By building up a string directly instead of first mapping into an
array, and than using Array#join() - we avoid some of that churn.

It terms of speed, avoiding Array#join resulted in the same
or better performance for the browsers I tested.

|                | Chrome 60     | Safari 11.1   | Firefox 61
| map+join       | 241K/s (± 2%) | 189K/s (± 5%) | 161K/s (± 6%)
| forEach+concat | 297K/s (± 1%) | 398K/s (± 2%) | 159K/s (± 13%)
| reduce         | 437K/s (± 2%) | 396K/s (± 1%) | 138K/s (± 12%)

For Chrome and Safari, 'reduce' were marked "fastest".
For Firefox, JSPerf marked all three as fastest due to the
variable ranges overlapping. Re-runs produced similar results,
each time with three cases laid out in a slightly different order
in the 130-165K/s range. The test used a copy of en.wikipedia's
current registry with the module queue of a random article.
https://jsperf.com/array-map-and-join-vs-string-concat/1

Change-Id: Ifbd8f6509ac118657c5ad2833f54fd6e8e63f9ce

Merge "Use separate insert/delete statements to work around Blazegraph bug"

resourceloader: Remove support for `state(name, state)` signature

This has no usage in non-test code anywhere in Wikimedia Git,
and was only used in the test suite for mw.loader in core,
and in the test suite for the NavigationTiming extension.

The core usage is as part of this commit. The usage in NavTiming
is updated by I240ced4e65988f9.

Bug: T127328
Depends-On: I240ced4e65988f96c7ece3772378c2c9a335fb9a
Change-Id: Ic17c797e528feaf07a4777709d705615fea353e5

Merge "rdbms: add "maxReadRows" limit to TransactionProfiler"

Use separate insert/delete statements to work around Blazegraph bug

Bug https://github.com/blazegraph/database/issues/100 happens on DELETE/INSERT
but not on separate delete and insert, try to work around it

Change-Id: Iba6a98dde6f4bea9051cd6dee91fb19db5eca10e
Bug: T201217

Merge "Use LB server configuration to force DB domains in ExternalStorageDB"

Merge "Add @since to LoadBalancer getServerInfo() per a3096440627"

Merge "benchmarks: Add benchmark for JavaScriptMinifier"

Merge "benchmarks: Add a default to the 'file' option of benchmarkTidy.php"