Refactor database-related code in ApiQueryWatchlistRaw

Database queries used to get user's watchlist items in API are quite
complex due to number of options oferred by the API.
Generating the query is moved to WatchedItemQueryService.
ApiQueryWatchlistRaw no longer contains database-related code.

Simple user watchlist item lookups should use WatchedItemStore.

ApiQueryWatchlistRaw tests have been introduced in
I9c07aa237607143985f0efe20ed0065d2bde27e4

Bug: T132566
Change-Id: I875a92074b52c00ac11db1fa05615abbf5262ab1

Merge "registration: Remove broken ResourceLoaderLESSImportPaths support"

Show parser output for diffs unless extension aborts

It was backwards.

Bug: T139433
Change-Id: Ie370da9d77e1d7608e2d5492f9d34094af233d0b

Merge "Revert "Convert Special:MergeHistory to use OOUI.""

resourceloader: Replace SHA1 with 32-bit FNV-1 as hash function

SHA-1 is not secure enough to be used as a cryptographic hash function, and its
implementation in JavaScript is too long and too slow for it to be a good
general-purpose hash function. And we currently throw away most of the work:
SHA-1 produces 160-bit hash values, of which we keep 48.

Although the JavaScript implementation is not exported, SHA-1 is a well-known
hash function, and I'm willing to bet that sooner or later someone will move to
make it accessible to other modules, at which point usage will start to spread.

For ResourceLoader, the qualities we're looking for in a hash function are:

* Already implemented in PHP
* Easy to implement in JavaScript
* Fast
* Collision-resistant

The requirement that hashes be cheap to compute in JavaScript narrows the field
to 32-bit hash functions, because in JavaScript bitwise operators treat their
operands as 32 bits, and arithmetic uses double-precision floats, which have a
total precision of 53 bits. It's possible to work around these limitations, but
it's a lot of extra work.

The best match I found is the 32-bit variant of FNV-1, which is available in
PHP as of version 5.4 (as 'fnv1a32'). The fnv132 JavaScript function is
around ten times faster and eight times shorter than sha1.

Change-Id: I1e4fb08d17948538d96f241b2464d594fdc14578

Merge "Fixes to LocalFile::lock()"

Localisation updates from https://translatewiki.net.

Change-Id: Ia097accd2d28baec1f7932cdb30299e0dc1a7781

TitleOptionWidget: don't subvert OptionWidget's tabindex

OptionWidget takes steps to not interfere in the tab order. TitleOptionWidget
uses links as its labels, which has a side-effect of reintroducing a tab-
focusable element into the OptionWidget. Explicitly set tabindex="-1" on these
links, to maintain the OptionWidget behavior.

Bug: T129792
Change-Id: Ia6b1361e7183e253b7abc58037f007936dfa60ff

Remove unused imports

Change-Id: I3b2ede8a11863c0fb775d851abb251e44ad616db

Revert "Convert Special:MergeHistory to use OOUI."

The new form doesn't check the CSRF token. We should use either just
HTMLForm (which checks the token automatically) or just FormOptions
(and check it ourselves), not a mix of the two.

This reverts commit 598068334e72be83088b9acdf674a79293f040ba.
This reverts commit 475e78c377f81a8837462852cdaab05e01cd686c.

Bug: T138346
Change-Id: Icc100552f3fba2e5e17ae6a2f57c2bfed32fbe83

Merge "Update the way captions show up in packed-overlay and packed-hover"

Fixes to LocalFile::lock()

* Added onTransactionResolution() DB method.
* Use this method so that file unlocks fire on unlockAndRollback()
  as well as on DB errors (via MWExceptionHandler::handleException).
  This prevents locks from getting stuck for minutes when deadlocks
  happen, since the LockManager::destruct() method is not reliable.
* Fix broken reference counting which always released locks on the
  first unlock() call, even if there were 2+ lock() calls.
* Added some type hints to IDatabase methods.
* Fixed DatabaseBase::__destruct() logging to include all callbacks.

Bug: T132921
Change-Id: I684706957f4d794cb6fe61505b0d26b7893de706

Localisation updates from https://translatewiki.net.

Change-Id: I46a80dc4344a68d51e8e20e330d95231ce004beb

Merge "Follow-up 6af8dcf: Remove duplicate styles in shared.css"

Fix comment typos in mediawiki.js

Change-Id: I40f90913f62f611f1bd940130751355f34dac5bf

Merge "registration: Add schema validation ResourceLoaderWikiModule"

Follow-up 6af8dcf: Remove duplicate styles in shared.css

Bug: T138721
Change-Id: I1733fcf1b3155b43dee8b79036f91b8d6cc25d05

Localisation updates from https://translatewiki.net.

Change-Id: Ic18771748bb4ac83b3d7810a1cb123345da8c452

Merge "API: Log non-whitelisted CORS requests with session cookies"

registration: Remove broken ResourceLoaderLESSImportPaths support

It never worked to begin with since it did not prefix paths, and the
documentation for it says it should not be used by extensions, so
let's stop pretending it works.

Bug: T108271
Change-Id: I665d4439d14406038f2ac767ee5ec6eb5b67fca6

registration: Add schema validation ResourceLoaderWikiModule

And improve the "groups" description for ResourceLoaderFileModule.

Bug: T134213
Change-Id: I03ecc56d1af565ace556ba579a9a9c495334da9c

Localisation updates from https://translatewiki.net.

Change-Id: I1b12a0e7251d41e3eeb7f000dc2b766b9ac0c637

mediawiki.action.edit.stash: Document ignored key codes

Follows-up eca800c7f02.

Change-Id: Ic0d9620eb3a5e371cc9825248ebb40abb0dea965

Localisation updates from https://translatewiki.net.

Change-Id: If018806a6e265fdbea44aa5812b17ceb40e7dbc9

Merge "Sync up with Parsoid parserTests."

SearchInputWidget: Use isset on dataLocation in case it isn't

Change-Id: I88f4d1624ee0e1f64d8f591dd8936e875211c53d

Remove Profiler specific code from fileOpPerfTest

This code was referencing classes that no longer exist, so it was
removed. Profiling can still be accomplished the same way normal maint
scripts are profiled, by populating StartProfiler.php and passing
'--profiler ProfilerOutputText', or something similar, on the maint
script command line.

Change-Id: Ie037f6b224ec361c86b86369e2a4fff641908053

Fix a few incorrect annotations

Found by applying etsy/phan to the mediawiki codebase, in preparation
for getting it running more regularly via CI. Has no effect on how the
code runs, simply does a better job of documenting the code.

Change-Id: I4c844a51df3ffe4fbb96cac25f3eff2f7a9faca2

Repair interwiki search via api

This was throwing exceptions, probably for many months now, when
requesting and finding interwiki search results. Probably this went
unnoticed due to interwiki search only being enabled on itwiki, and
the requirement to pass a special parameter to turn it on.

Change-Id: I68ec35bb072b5ad62dd66789936bc2ae47d5256f

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit dd8e644d320aec076f76da4e2bd70a8527e0dfd8

Change-Id: Id41989ae0973804c51ed10353d0ac81cef5a6211

Don't reuse variable names

This code was rather hard to follow due to reusing the same variable
multiple times. Lets give them unique names.

Change-Id: I8f134b6314d593a3e40b19765c7b71496b8e524d

Fix undefined classes

Applying static analysis to mediawiki core found a short list of classes
that were undefined. Fix those up.

Change-Id: Ib7f9dbd847ada287b35afb799782fc04a3b39ce4

Fix reference to undefined class

This error class was moved into wikimedia/cldr-plural-rule-parser
project, but in the conversion process this exception was not
converted over.

Change-Id: I36a2466a89e2ed6f0b98b45fcb945b6fdf56fbcf

Merge "Correct two mistaken htmlform annotations"

Correct two mistaken htmlform annotations

Change-Id: Ica807b656af0eb5a82ec0b1ec2d7894fd79a6639

Merge "Factor out preview parse"

Merge "Use a log group for LocalFile lock errors"

Drop check for mbstring in copyFileBackend maintenance script

Already checked in PHPVersionCheck.php as of 41d1fa1c3a.

Change-Id: I35e55b63378f1768076be0c45392fdc5d48168bf

Merge "Changed array instances from "array()" to "[..]""

Merge "Try to predict the rev_id when preparing edits"

Merge "jquery.makeCollapsible: Special case of content inside <li>"

jquery.makeCollapsible: Special case of content inside <li>

Bug: T137676
Change-Id: Ie4228e63a63157e3909e938a4086f46dcbbf6767

Merge "mw.action.view.filepage: Remove higher than necessary specific selectors"

Merge "Move EXIF metadata CSS from mediawiki.legacy.shared"

Merge "Prepare to split create/modify button label"

Merge "Make non-existent messages be html safe regardless of output format"

Prepare to split create/modify button label

Doing the i18n change in advance to allow translators lots of time to
create the translations before the following commit uses them in
practice.

Change-Id: If86485354da4e48a02aaabd0b1642ccea6a0b462

mw.action.view.filepage: Remove higher than necessary specific selectors

Change-Id: I0479fc3e75842f468f25448912acafa8021f8582

Move EXIF metadata CSS from mediawiki.legacy.shared

* ...to mediawiki.action.view.filepage.
* Also move the mw-metadata-show-hide-extended class
  from mediawiki.action.view.metadata to
  mediawiki.action.view.filepage.

Bug: T89981
Change-Id: I62cdcb625d0a5c280b8ac1ac989c7b8fa6cce47e

Merge "Special:MergeHistory: Merge all revisions if no particular box/button is ticked"

Use a log group for LocalFile lock errors

Bug: T132921
Change-Id: I55cafc9e2fff302e55d40a678bf7e25b4bf24026

Merge "Remove incomplete and dated 'wgUseETag' experiment"

Merge "Move HistoryAction CSS from mediawiki.legacy.shared"

Make non-existent messages be html safe regardless of output format

If you have a non-existent message in the output, chances are its
user-controlled. If the message has the ->plain() or ->text()
format, the output used to be not safe for html. Obviously people
should not be using those format types where html is being outputted,
but sometimes that happens. I think we should prioritize always being
safe over the fallback content not potentially being double escaped.

Additionally switch the enclosing brackets to be fancy unicode
characters, to sidestep the escaping issue on the enclosing brackets.

So previously, wfMessage( 'script>alert(1)</script' )->text() would
have outputted <script>alert(1)</script>. Now it outputs
⧼script&gt;alert(1)&lt;/script⧽. No sane message key will include
< or >, so this would really only come up if the user can control
the message key name.

This goes somewhat against T68199.

Change-Id: Ic8a60892b8e847e6021494c10968814aac391731

Merge "Improve HTMLCacheUpdate job CDN purge de-duplication"

Merge "Adding a bunch of hooks from wikiHow into DifferenceEngine"

Localisation updates from https://translatewiki.net.

Change-Id: If18a985dcb92fee1328dd4f2c863cb10116c922c

Merge "add option to XML dump stubs of page ranges with explicit rev_id ordering"

Merge "mediawiki.ui: Improve focus states of primary buttons"

Merge "Allow specifying OutputPage object in Skin::subPageSubtitle parameters"

Allow specifying OutputPage object in Skin::subPageSubtitle parameters

To avoid ending up with API callers like this:
https://gerrit.wikimedia.org/r/#/c/281463/1/ApiVisualEditorEdit.php

Bug: T97700
Change-Id: Id27380de9577ad54cb2580b76a4f236e8dd491dd

Merge "Resources: Remove very old comment about OOjs UI incompatibility"

Merge "Add API module to receive CSP reports."

Update OOjs UI to v0.17.5

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

Change-Id: I17caee666b1d1cc2da670e073ef57122a66ecfc6

Merge "Deprecate the 'UploadVerification' hook"

mediawiki.ui: Improve focus states of primary buttons

Follow-up to T137658 in OOjs UI. Improving `:focus` states of primary
buttons to be more harmonious with normal buttons. Also normalizing
`padding` in Firefox, removing obsolete comment and minor clean-up.

Change-Id: I584644c47be73a9770ac487b2b738d0f8bdf69d1

Drop RELEASE-NOTES-1.27, merge into HISTORY

Now that it's a release.

Change-Id: Ib8374f7916a74bfc564c347d2ea312a1e66a1dad

Try to predict the rev_id when preparing edits

During both the edit stash and first parse in on page save,
guess what the rev_id will be and use that instead of null.
Only reparse if it turns out to be wrong. This avoids extra
parsing on wikis that have low-medium traffic, and does not
cost much. The parsing that can be avoided is:
a) in doEditContent() by using the stash
b) in doEditUpdates() by using the doEditContent() result,
   whether that was able to use the stash or not itself

Also improved the parse operation logging in save paths.

Bug: T137900
Change-Id: Ic6faae70a78b4e223e4d3585cefd482c0fa00677

Merge "kafka: Implement ack handling"

Run maintenance/generateLocalAutoload.php

Change-Id: I0cf208b927d28a09a8603604fdfe0a6c1e9beb6b

Merge "Refactor database-related code in ApiQueryWatchlist"

Merge "autocomplete: fix duplicate/missing tracking events"

Merge "js suggest: better keyup/keydown handling"

Refactor database-related code in ApiQueryWatchlist

This moves generating of a complex Watchlist and RecentChanges
related query to a WatchedItemQueryService class.
ApiQueryWatchlist class no longer contains any database-related
code.

Bug: T132565
Change-Id: I5a5cda13f8091baa430ac1a8e2176e0efd1ae192

jquery.accessKeyLabel: Avoid needless mw.Message#parser in updateTooltip()

When calling mw.util.addPortletLink(), Chrome on a MacBook Pro spends 3ms
in mw.Message#parser for each link that doesn't even have a title.

> addPortletLink: updateTooltipAccessKeys: updateTooltip: msg: mw.Message.parser

Change-Id: Id6c26e31eb2c1431fe9725e608813bdf40069207

Localisation updates from https://translatewiki.net.

Change-Id: I6a1e0bab0d201e631b0c6434fd6839245f6dc0c1

Merge "skins: Minor code clean up"

Deprecate the 'UploadVerification' hook

It has been replaced by 'UploadVerifyFile' a long time ago,
but never officially deprecated.

Change-Id: I345dca48c28ee5e1e2ad35bb6f42bbc03a1f4dd1

Add API module to receive CSP reports.

There are two expected usecases for this:
* The proposed builtin CSP support at I80f6f4
* Setting CSP headers on media served from upload.wikimedia.org

This was split from I80f6f46

For details on CSP, see http://www.w3.org/TR/CSP2/
See also https://www.mediawiki.org/wiki/Requests_for_comment/Content-Security-Policy

Related to (but not directly a fix for) T117618

Bug: T135963
Change-Id: Id92126ca7707186757e77fe50cd336ff1acb8b3f

Merge "mediawiki.ui: Align `.mw-ui-input:focus` to design spec"

Merge "Allow Message::newFromSpecifier to handle ApiMessages"

Allow Message::newFromSpecifier to handle ApiMessages

Instead of constructing a new Message from the Message as
a MessageSpecifier, just clone the existing Message which will preserve
subclass data.

Also, make use of this to simplify the logic in ApiBase::parseMsg().

Change-Id: I9545acb8da752c0c21e16d8b1d37d8802fcb329d

Merge "ApiQuerySiteInfo: Expose wgFixArabicUnicode and wgFixMalayalamUnicode"

Merge "mediawiki.js: Correct a typo in documentation"

UsageException: Validate that $codestr is a non-empty string

Current theory on T138585 is that something is passing in false, so
let's throw an InvalidArgumentException that should log a backtrace for
us.

Bug: T138585
Change-Id: Iba0dad424c56340b8bc73322bb1a7ae6d4de314a

ApiQuerySiteInfo: Expose wgFixArabicUnicode and wgFixMalayalamUnicode

These variables control how MediaWiki normalizes page titles.
Clients need to know these settings to not be surprised by such normalization.
See T94826.

Change-Id: Id77f584b8c3a3d9c065993bd74525f166112b20c

mediawiki.js: Correct a typo in documentation

Change-Id: I03808e49d67c5d412929472d3e0cde2390e1a8e1

Merge "Fix misleading comment about svg filtering."

Merge "mediawiki.api.edit: Add edit() and create() methods"

Localisation updates from https://translatewiki.net.

Change-Id: I41d9dca5b60db89e21609a1eff34494298daf6f3

mediawiki.api.edit: Add edit() and create() methods

Doing edits "The Right Way" is non-trivial due there being mulitple strict options that
need to be known and enabled. By default, the API encourages bad behaviour:

* Edit is unexpectedly saved as anon after session becomes invalid.
* Other edits are silently overwritten.
* Accidentally re-creates a deleted page.
* Accidentally creates a new page when an edit was intended (eg. if title was wrong).

Implement abstraction methods for edit and create that handle all this.
Thus guarding JS edits with the same protections as EditPage.

Change-Id: Ic6a35902cbae262971c704b9b8127e54733dac79

Merge "Expose $wgMaxArticleSize in siteinfo query api"

Expose $wgMaxArticleSize in siteinfo query api

Currently, an user of the api has to try to save a page to see,
if it's too big, or not. A much saner way would be, that the api user
reads the siteinfo of the target wiki to know the maximum size of a
page and trim the edit to not exceed this limit.

This commit exposes the value of the configuration variable, which
handles the maximum article size.

Bug: T138669
Change-Id: I25a4c1b9013f98e5b0a263a637c8b006dccfd5ad

Remove incomplete and dated 'wgUseETag' experiment

Follows-up:
* 8ff36ebd (r9793) Set ETag header on article views.
* 8167b89e (r16205) Add $wgUseETag, false by default (broke Squid; T9098).

ETag is a useful alternative for If-Modified-Since (see T33639, T49529 about
use cases and bugs it may fix). However the current incomplete implementation
isn't useful.

It's a feature flag that has been disabled since 2006. Enabling it stores
ParserCache::getETag in OutputPage and sends it out as a response header.
It's ignored on incoming requests. We don't check If-None-Match for
'304 Not Modified' anywhere.

ResourceLoader has a working implementation that could be leveraged.

To be re-evaluated in the future. ParserCache::getETag() could use
improvement as well.

Change-Id: I37fd9d0ce1ce9bffb79243c6ae340403fdfd769f

skins: Minor code clean up

* Make Skin::initPage visibility explicit.
* Make Skin::preloadExistence protected (not used outside this class).
* Remove use of empty().
* Avoid empty if-block.

Change-Id: I85bac34ded3414af5b998af614d554402ce92f21

Fix misleading comment about svg filtering.

Follow-up 551d79a3e6754. See also Bug T122653.

Change-Id: I0662dc6618596bb3a3bad345de45b054b6f7f968

Factor out preview parse

Mildly refactor preview parsing, to avoid code duplication in the parser
migration tool that I am developing, which subclasses EditPage.

Change-Id: I3eee8a1ef3007353ba8b129d968d9f4e4d40ef5d

Move HistoryAction CSS from mediawiki.legacy.shared

...to mediawiki.action.history.styles

Bug: T89981
Change-Id: I0b5c513ebd73aeed287f8fe681e231ffc9b6ab08

mediawiki.ui: Align `.mw-ui-input:focus` to design spec

Aligning `.mw-ui-input:focus` to design specification
https://phabricator.wikimedia.org/M101

Change-Id: I4e4d20d38047e9b2e069a84a073fc57480303452

Follow-up 9e45850: Temporarily duplicate styles back to shared.css

Per Bartosz and Timo's comments on
Ifd90292fc54875403d871429ce6e5020036d4797

Change-Id: I90940d3f67780af6f1efb8507a0bc038e0496220