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

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

Localisation updates from https://translatewiki.net.

Change-Id: Ib334b266f7a061203444ca34e6b3fa5add2d9d22

Merge "Follow-up ebe7205c: Edit stash: Unbreak jQuery selectors"

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

Bug: T134812
Change-Id: I3184b86fb751724a624f568b6f45599f9340fad4

Follow-up ebe7205c: Edit stash: Unbreak jQuery selectors

Change-Id: I4d72948ba62cb5c1cc29277f30363c29e42fa25b

Localisation updates from https://translatewiki.net.

Change-Id: I8526fd44dca56b8cbd902e909936137a1b108c1f

Merge "Mock Http in MediaWikiPageNameNormalizerTest and add more test cases"

Restore load position override for 'site.styles' module

Follows-up 93ed259 which changed the module to be 'site.styles', but
the override was still looking for a module called 'site' (which isn't
listed in that queue anymore).

Bug: T138586
Change-Id: I3052a2e378f804d0d4d117f8ed446fe9b4b59f99

Mark the APIEditBeforeSave hook as deprecated

Bug: T137832
Depends-On: I9d0b7e7713c805ebc7bf59f55456e69c6491e265 (AbuseFilter)
Depends-On: I27f244631e9dcd160bffff70349e5034f2a537ea (ConfirmEdit)
Depends-On: I7b0e1d125b196379dd5bf26658ff642d32ba832a (ProofreadPage)
Depends-On: I97f6a3c20476f1a42e3fadc701df5870a30c790c (SpamBlacklist)
Change-Id: I6947d5373aa33224141f8d21cf3a1ee1f641d952

Mock Http in MediaWikiPageNameNormalizerTest and add more test cases

Bug: T138570
Change-Id: Ia156ba61afb90c9a3b1a2e1e0bf967056bd87fc8

Merge "Chinese Conversion Table Update 2016-4"

Merge "Extended login: Don't use a $wg config variable, add UserName"

Merge "build: Bump grunt-stylelint to v0.4.0 and make pass"

Merge "registration: Support Change/RemoveCredentialsBlacklist in extension.json"

Merge "Make deprecated TestUser properties private"

Merge "Edit stash: Support CodeEditor"

build: Bump grunt-stylelint to v0.4.0 and make pass

Change-Id: I00e469cf9b484c9d4d4ad145480fa6e66dd7e8cc

registration: Support Change/RemoveCredentialsBlacklist in extension.json

Change-Id: I4d796a46d99daa8841fcdcbe76dddf9db46af9ed

Edit stash: Support CodeEditor

Use .textSelection( 'getContents' ) instead of .val() for '#wpTextbox1' and '#wpSummary'.

Then the normalization of line endings and dependency on 'jquery.getAttrs' is not necessary.

Bug: T137861
Change-Id: I26f7021a2a02cf719c9e5f44787b95cfc3239959

Localisation updates from https://translatewiki.net.

Change-Id: I5105d0a0b0e35fdafbe9604122a49ae692cabe5a

Fix unit tests running on non-DB environment

Bug: T138551
Change-Id: Ie1d3c8b24e5271d4e12f4190400531cbe606bfd4

Merge "mw.jqueryMsg: Add support for {{PAGENAME}} and {{PAGENAMEE}}"

Extended login: Don't use a $wg config variable, add UserName

CentralAuth needs 'User' as well for this to work.  However, this
shows the exact cookie names are an implementation detail that should
not be exposed as a 'wg'.

Instead, use a function in the CookieSessionProvider.  That way,
CentralAuth can override it properly without requiring users to change
$wg's.

I also added UserName. provideSessionInfo will fail to return
session info if UserID and UserName are both set and don't match.

Also, the UserID<->UserName mapping is public, so there is no
additional privacy issue.  Thus, it seems we should expire them
the same time.

Bug: T68699
Change-Id: Ia3259846433980408f79d44f665e17e15670e8ee

Merge "Count subject and talk page conflict separately"

Merge "search: did you mean should always go to SERP"

kafka: Implement ack handling

By default the kafka implementation we use doesn't require any kind of
acknowledgment, it just throws messages into the wind and lets them sit
where they may. Implement an option for KafkaHandler to specify the
number of acks (number of replicas that must record the message) and
some error handling to throw exceptions as necessary when there is a
problem.

Bug: T135159
Change-Id: I859dc791072db407f908b2f36be0d6704f1a6256

Merge "Remove unused MediaWikiPageLinkRenderer class and interface"