selenium: Re-add all tests and wdio-mocha-framework, now v0.6.4

This reverts commit c3878c08b5275123972354ba5479d3315c4fefb4.

Now that we're running on node10, this will fail until we fix
things.

Bug: T213268
Bug: T222406
Change-Id: I5935fc5d5bc23978e50275d3c99ac870b3b82f49

Merge "HTMLForm: Remove deprecated unused getErrors() method"

Merge "make sure revision uids are 0 in the xml if missing/0 in the db"

Merge "selenium: Temporarily drop all tests and wdio-mocha-framework"

Merge "Add missing newline between <?php and namespace/use section"

make sure revision uids are 0 in the xml if missing/0 in the db

Bug: T224221
Change-Id: Id9861866fd9e4d2fe8d151c9631403aa24b9a779

Merge "Use ExtensionRegistry::setAttributeForTest() instead of Reflection"

selenium: Temporarily drop all tests and wdio-mocha-framework

This is so that we can migrate to node 10 in CI. It should only land
for an hour or so, and is to be reverted as soon as we get CI up to
node 10 and working, lest our suites of selenium tests become
subject to bitrot.

Bug: T222406
Change-Id: I9473e0213c070a0e963913fd81062c6bbf9ec2f2

Merge "Remove usage of deprecated SkinFactory::getDefaultInstance()"

HTMLForm: Remove deprecated unused getErrors() method

Was deprecated in 1.28 and no longer used, see usage below;

Usage
=====

https://codesearch.wmflabs.org/search/?q=%5CbgetErrors%5C(%20(.*%3F)%20%5C)&i=nope&files=&repos=

Bug: T220656
Change-Id: I542df4f1f7f4180d42282eaf2e069ab0bc9e8614

Merge "Add a "help" button for action=info"

Add missing newline between <?php and namespace/use section

The rest of the codebase is using this code formatting standard.

Change-Id: I4d2ba61757a7e28d40096d9dc5915005c340d4f2

Merge "API: Migrate Title::userCan() calls to PermissionManager"

Merge "always order by page_id for dumps of current revisions"

Merge "Wording"

Add a "help" button for action=info

Bug: T224848
Change-Id: Id72fb9134ce889b9610ca088e53799804580903e

Use ExtensionRegistry::setAttributeForTest() instead of Reflection

Change-Id: If29e8825f1734f8c0efa21aa77f6354fa3468248

Remove usage of deprecated SkinFactory::getDefaultInstance()

Change-Id: I3bd50f8ca5baabd34dbc0e3bbc2f97e94650a17a

Localisation updates from https://translatewiki.net.

Change-Id: Ia730eebb70ecece0daca1d97c3d541a767777ecc

Wording

In GitInfo, the debug sentence "Computed cacheFile" could imply that some
file is written, but in reality "computed" refers to the name. Change to
"Candidate cacheFile".

Bug: T215722
Change-Id: I19448e5790a0f2fcb5202170ca03ab0f5e16d24e

Add a "help" link for Special:PermanentLink

Bug: T224793
Change-Id: I4099b1a794df8477842d5f018d63304cef99d372

Merge "Improve changePassword.php error handling"

Merge "Allow $conds of IDatabase::selectRow to be a string"

Merge "Fix typos in DefaultSettings"

Merge "Inline trivial …_internal() methods in Title/UserArray classes"

Allow $conds of IDatabase::selectRow to be a string

When not having a condition you can pass a empty string or '*'

Change-Id: Id1bb6cf6e6bd89ad2df492e7441253d06fcf2a2d

Merge "rdbms: cleanup code for read-only propagation for master connections"

Merge "Add CirrusSearch dev dependency to require-dev"

Fix typo in docs

Change-Id: I69085afd4f3475ae6449a0e3683a28ccddf47445

Merge "Make sure that each DataUpdate still has outer transaction scope"

Merge "rdbms: make resolveDomainID() use more consistent in LoadBalancer"

rdbms: cleanup code for read-only propagation for master connections

Make "readOnlyReason" immediately propagate from LoadBalancer to the
DB handle even when getConnection() with DB_REPLICA yields a master
DB handle. This can happen when the master has a non-zero "load".
Previously, it would not be set until something later used DB_MASTER
with getConnection(). This didn't really effect anything but seemed
confusing.

Short-circuit getLaggedReplicaMode() when the reader index uses the
master DB since there is no chance of lag. Also simplify it to just
calling getReaderIndex(), which since nothing else is needed to set
the "laggedReplicaMode" field.

Also rename "readerIndex" field to "genericReadIndex" and "loads"
field to "genericLoads" for clarity.

Change-Id: I6dc28933d2f94f0de1e8f9c5b2b8d2cf8a1a9d08

Merge "rdbms: clean up and refactor ResultWrapper classes"

Merge "Consistently use spaces and stars in comments"

Make sure that each DataUpdate still has outer transaction scope

Bug: T221577
Change-Id: I620e461d791416ca37fa9ca4fca501e28d778cf5

Localisation updates from https://translatewiki.net.

Change-Id: Ib076d5736f1bea05df4c649cddd1b12ea6cc3af4

Add CirrusSearch dev dependency to require-dev

Adds symfony/yaml for parsing event schemas used by EventBus
as part of the CirrusSearch test suite. The dependency needs to be
added to core to support CI testing extensions that depend on
CirrusSearch.

Bug: T220723
Change-Id: Iddbfb81fb667980470d6872c34a1e63897f1e4af

API: Migrate Title::userCan() calls to PermissionManager

T208768 introduced the PermissionManager service that can now be used
for page specific permission checks. This change replaces calls to
Title::userCan() with the new service in API classes.

Bug: T220191
Change-Id: I768d07a520ca6473a4eefb88c9f587657bc74357

Merge "Cleanup LinkerTest leakage between provider entries"

Merge "Fix empty auto-summaries triggering a fatal error."

Consistently use spaces and stars in comments

Change-Id: Ia8e005ec65b5a98b12fc1c079604db175b731a88

Fix empty auto-summaries triggering a fatal error.

Aka:  Streamline Linker::formatAutocomments() and add tests

This uses the "streamlining" for the code proposed by Thiemo
in I38edc1ad7720. I have squashed the two commits, so it now
has his code in Linker, but still has my tests as well as his.

Thiemo wrote on his patch:
This also changes the output in case there is no fragment to link to.
Before an empty `/* */` in a summary this would have created a link to
the page. I would like to argue this is not what a user expects.

Bug: T222628
Change-Id: I05408ede0e20dfd976f4057fc5baab461d2ef769

Merge "Revert "Factors out permissions check from User into PermissionManager service""

Revert "Factors out permissions check from User into PermissionManager service"

This reverts commit 7faa7a7420866ec685863c1d6c530bd4999da643.

Reason for revert: T224607

Change-Id: I549810a4cd2e424cc4a438887d2f24614a24cc00

rdbms: make resolveDomainID() use more consistent in LoadBalancer

Use the method in getConnection()/openConnection() and make it
handle the local domain alias as well.

Also:
* Move some code to a new getConnectionIndex() method.
* Make the text of various index/domain exceptions more consistent.
* Improve some auto-commit flag code comments.

Change-Id: I7e0d4f2134ee91ad60b0d34bf01e05115193b04a

Cleanup LinkerTest leakage between provider entries

The problem was that new revisions were being added to the same page
since it is not deleted between each run.

Change-Id: I52e8db8e5ef3cde309c26f928fd44de4e6ec18e4

Merge "jquery.suggestions: Do not show suggestions on prefilled values"

jquery.suggestions: Do not show suggestions on prefilled values

Show the suggestions on load only if the current value differs to the
defaultValue from the HTML. This prevents suggestions if there is
already a prefilled value in the input field.

Also direct call $.suggestions.update() instead of trigger a keypress.

This change is a follow-up to 72f61f7a5930cf03d4e8ddde62c2ef627b05dd69.

Bug: T224524
Change-Id: I501596996a20c62d1497bf66e23c7858b58bf4ea

Merge "Fix typo in comment about class name"

Merge "Print chained exceptions when maintenance script fails."

Merge "Add checkDependencies.php"

Inline trivial …_internal() methods in Title/UserArray classes

This code is quite old. I can't tell why it was written this way.
But I can see these methods are never used anywhere else. We can as
well inline them.

Change-Id: I252f4d2b2b2da822c8d43ffc1f088f5848034c5e

Fix typo in comment about class name

Change-Id: I52e2b9f6553f4d3977c609bc1e2bc86c24e0c9c7

Merge "parser: list the vary-* flags in the NewPP report HTML comment"

parser: list the vary-* flags in the NewPP report HTML comment

Change-Id: I5a4afba2bfdb5b5b56ba0a01ed8ff444a67fbb1a

Merge "WatchedItemStore: Remove deprecated Title param to getNextRevision()"

Factors out permissions check from User into PermissionManager service

The following methods should are factored out of the User class into PermissionManager, leaving only deprecated stubs:

- User::isAllowed -> PermissionManager::userHasRight
- User::getRights -> PermissionManager::getUserPermissions
- User::groupHasPermission -> PermissionManager::groupHasPermission
- User::getGroupPermissions -> PermissionManager::getGroupPermissions
 -User::getGroupsWithPermission -> PermissionManager::getGroupsWithPermission
- User::groupHasPermission -> PermissionManager::groupHasPermission
- User::isEveryoneAllowed -> PermissionManager::isEveryoneAllowed
- User::getAllRights -> PermissionManager::getAllPermissions

Depends-On: I258f02e286b6ba0387e1bff540a744fafb03dc55
Depends-On: Ie4cedf457eaaa93ec3055c37539322855e02ce26
Depends-On: Id274f240d687efa61cb9f7a15033ae2a7a532083

Bug: T218558
Bug: T223294
Change-Id: Ia0d840b772ea5f20c9594ce151cc57adc270e48b

rdbms: clean up and refactor ResultWrapper classes

Change-Id: Ib1d71dbbd31e134cf89dca58f7aa77bdddc828c0

Update OOUI to v0.32.0

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

Bug: T141320
Bug: T163142
Bug: T169484
Bug: T178950
Bug: T180730
Bug: T203859
Bug: T212186
Bug: T213884
Bug: T215644
Bug: T219479
Bug: T220239
Depends-On: Ia65a8053082ac95bf89cb5484aa64668fb780735
Change-Id: I0a48885642b86950180d1fbe4c7a609d323fb87f

Merge "Rename Block to MediaWiki\Block\DatabaseBlock"

Fix typos in DefaultSettings

Change-Id: I692286d68bdf5585694c8a65d9d9d512f813fb9e

Merge "Fix typos in MessageCache"

Merge "media: Fix $ext property name in BmpHandler::getThumbType()"

Merge "EditPage: Remove unused deprecated method ::submit()"

Fix typos in MessageCache

Change-Id: I5ede5ad5687144535545248940ca6f676f514900

Merge "Hard-deprecate Title::moveTo and friends"

Merge "Finish sentence in MultiUsernameFilter documentation"

EditPage: Remove unused deprecated method ::submit()

Was deprecated in 1.29 and EditPage::edit() was made available for
use and replacement of this ::submit().

Usage
=====

https://codesearch.wmflabs.org/search/?q=%5C-%5C%3Esubmit%5C(%5C)&i=nope&files=&repos=

The usage of ->submit() in the search link above are not for EditPage
but for other classes.

Change-Id: Ibd65965c785f84e9bda65cfaa2c491310cda8166

mediawiki.action.view.dblClickEdit: Move preventDefault()

Move the preventDefault() into the if-condition to trigger the
preventDefault() only together with the click() trigger.

Change-Id: Icde2187248b8b18cd4fd5085a92d840e1f77b00d

Finish sentence in MultiUsernameFilter documentation

Change-Id: I933f54f505e0680346f73293c98624e0f48e9292

Merge "Fix doc for $item of ApiQueryBase::addPageSubItem"

Hard-deprecate Title::moveTo and friends

moveSubpages wasn't officially deprecated at all before this release,
but there were no callers in codesearch outside core, so it should be
safe to hard-deprecate.

Bug: T208775
Change-Id: Icea9f1dc45d8bd8c96cc159d631f18c394aab4b7

Remove deprecated mw.language.specialCharacters

mw.language.specialCharacters is deprecated since MediaWiki 1.33.

Change-Id: Idb949a37883e96689cabb734402f3d628851c119

media: Fix $ext property name in BmpHandler::getThumbType()

This is not "text", it's a file extension.

Change-Id: I92e3b70805df974e74e8aaa9e780dcb55ff584b9

Rename Block to MediaWiki\Block\DatabaseBlock

Keep Block as a deprecated class alias for DatabaseBlock.
Update calls to the Block constructor and Block static
methods from external classes.

Also update documentation in several places that refer to
blocks as Blocks.

Bug: T222737
Change-Id: I6d96b63ca0a84bee19486471e0a16a53a79d768a

Localisation updates from https://translatewiki.net.

Change-Id: I09b242d9bd17495e8b300432290a0af37ddc5fe1

Merge "Make improvements to BlockManagerTest"

Merge "Make config-outdated-sqlite parameter numbers consistent with config-*-old"

Partial revert I0d0ef26f59721039b2c24ab322fa3bf6ab16fea1

The parameter swap in meetsMinimumRequirement wasn't supposed to happen

Change-Id: Icf5b237f336084e7a8d11698cd5ea665305e7f3c

Make improvements to BlockManagerTest

Remove reliance on real domain and add more test cases.

Change-Id: Icd67fe8c1c9223a92a3775d34d0453328442d89e

Merge "Remove deprecated 'jquery.byteLength' module"

Localisation updates from https://translatewiki.net.

Change-Id: I0c369e318495e5b68b64315ac311bde601cc08c2

always order by page_id for dumps of current revisions

Also drop ordering of revs within pages, since there is only one
revision being dumped

Bug: T207628
Change-Id: I5e4f0bea7b54506ca389818407c43152a290da6e

Remove deprecated 'jquery.byteLength' module

Deprecated since MW 1.31 (f19c9021a466f414). No longer used
anywhere in Wikimedia Gerrit, and no usage found through mwgrep
either.

Bug: T219342
Change-Id: Ia2fecd68ae01c82f323f6c266230ff3fb2bbe287

Merge "Revert "change phabricator URL""

Revert "change phabricator URL"

This reverts commit 8b61598b39b3ffe709764bd394d2f113d50aea81.

Reason for revert: phabricator.mediawiki.org doesn't work.

Change-Id: I13909393763f4f154b9119e3916fea52fff371f1

Merge "Fix typo of Minimum in variable name"

Merge "Update wikimedia/remex-html from 2.0.1 to 2.0.3"

Localisation updates from https://translatewiki.net.

Change-Id: Icf713ed037b9895977bb5c908759ec379fd20fc1

Make config-outdated-sqlite parameter numbers consistent with config-*-old

Bug: T224374
Change-Id: Iebfb8299234cc9c66db0ecc4abd0c0a32af63602

Fix typo of Minimum in variable name

Change-Id: I0d0ef26f59721039b2c24ab322fa3bf6ab16fea1

Update wikimedia/remex-html from 2.0.1 to 2.0.3

https://github.com/wikimedia/remex-html/releases/tag/2.0.3
https://github.com/wikimedia/remex-html/compare/2.0.1...2.0.3

Change-Id: Iaa7212a362b120ca4d9c91f229ad4cabc39cc578
Depends-On: I6b6a45c2cf6dcc1fa03f59bf54d4c759111b3385

Merge "Switch empty username logging from warning to debug."

Job: Fix typo in exception message for invalid specification

The Job::factory method throws an InvalidArgumentException if a given
job specification is invalid. The error message for this exception was
worded in correctly. This patch improves the message to avoid confusion.

Change-Id: I6cab739263c1d3530c0650823db022dd5a4b60bc

Fix doc for $item of ApiQueryBase::addPageSubItem

ApiResult::addValue defines mixed for $value

Change-Id: I31881426ce7cb81f6430cccd4252881a154cd161

Merge "Rename $search to $engine to match hook docs for SpecialSearchSetupEngine"

Merge "Introduce TransactionRoundAwareUpdate interface"

Merge "rdbms: fix ChronologyProtector client IDs to not be empty"

Rename $search to $engine to match hook docs for SpecialSearchSetupEngine

Bug: T224353
Change-Id: I9629840531183ffd53f2a1c568a8a72c91a30171

rdbms: fix ChronologyProtector client IDs to not be empty

Follow up to 8107fddd8fe3d5

Also improve debug logging and add some unit tests

Bug: T223310
Bug: T223978
Change-Id: I35484385e4da2912bc10f5e8d2fb07cb1097347e