SECURITY: Add data attribute to patrol links so it can't be spoofed by user

Javascript used to look just for the patrollinks class, which
could be set by the user in order to patrol an arbitrary page.

Bug: T103239
Change-Id: I13fcc3ce479c0a4a90a6217c2e5244f051eaf862

Signed-off-by: Chad Horohoe <chadh@wikimedia.org>

API: Add "standard" header and hook for lacksSameOriginSecurity()

The header is intended for use with XMLHttpRequest when the request
might be part of an XSS. The hook is for extensions that might need to
add additional checks of some sort.

Bug: T98313
Change-Id: I0e5f2d3b29a79a12461dc33c90c812a56810f536

Signed-off-by: Chad Horohoe <chadh@wikimedia.org>

Improve unstyled updatedmarker

This is the "changed since your last visit" marker that you'll
see on ?action=history for a page on your watchlist, marking
edits that were made since you last viewed the page.
updatedmarker is styled by some skins, but not all of them.
Updating CSS selector and involved colors to address some of the concerns
and improving consistency.

Bug: T134515
Change-Id: Ib21ee453f0fa71fc8f516db1fec71096d962d194

Merge "Enable AuthManager by default"

Merge "build: Bump grunt-stylelint to v0.3.0"

Merge "jsduck: Sort list of globals in jsduck.json"

Merge "mw.loader: Optimise hot code paths in addEmbeddedCSS()"

jsduck: Sort list of globals in jsduck.json

Change-Id: Ifb920c055740575edcda0b4f460cc8c5b377ba87

Merge "Limit DELETE in purgeExpiredRestrictions() and use primary key"

Undo translations of message changes from 6ffabb26

I reverted them all on translatewiki.net by hand, then imported here with
a quick script: https://phabricator.wikimedia.org/F4032850.
(Changes were then reviewed and some weird ones undone by hand.)

All the important work was on translatewiki.net, this commit could as well
be done by the localisation bot, but we want it now to deploy it.

Bug: T135773
Change-Id: Iaa6797939fa52619d1bbd8d7e0dad8409687d3cd

Merge "Revert "Convert Special:WhatLinksHere from XML form to OOUI form""

Revert "Convert Special:WhatLinksHere from XML form to OOUI form"

This cannot be the design that was intended here...

This reverts commit 6ffabb260b60497a732b9b46610ec9fd0f9f71f9.

Bug: T135773
Change-Id: Id6ab2ba04c443981cc406c4372c9a809c937c566

Merge "PHPUnit: turn off verbose option"

Limit DELETE in purgeExpiredRestrictions() and use primary key

This should help reduce contention in some cases.
If too many rows are expired, subsequent updates will
clear them out.

Bug: T135470
Change-Id: Iada496d0db9b00e77037320d7c65124a8c8f68c0

Merge "Localisation updates from https://translatewiki.net."

Localisation updates from https://translatewiki.net.

Change-Id: I23caefae3374028db6475f66244c1ec3627f49ec

Use inNamespace in CategoryPage

Change-Id: I55dae5163d62c25536fb040764cab99cd4deb9b7

Merge "exception: Create generic BadRequestError based on ErrorPageError"

exception: Create generic BadRequestError based on ErrorPageError

Ideally this would be an option in ErrorPageError (perhaps even the default),
but its constructor isn't very suitable for that.

After this lands, uses of ErrorPageError should be audited to see if it makes
sense to emit a 400 status code.

Change-Id: I4beb6a4f256446b98b66d5e4bcdbab8f247441a8

API: Handle shorthand "all groups" configuration in ApiQuerySiteInfo

In $wgAddGroups, $wgRemoveGroups, $wgGroupsAddToSelf, and
$wgGroupsRemoveFromSelf, boolean true in place of an array of groups to
allow adding/removing is shorthand for "all groups". Handle this
appropriately in action=query&meta=siteinfo&siprop=usergroups.

Bug: T135467
Change-Id: I6ae61ef14ac7932e3369155c56bad60a9d72060d

Merge "Highlight new requirement"

build: Bump grunt-stylelint to v0.3.0

Change-Id: I012d8b8cb2851270180f1a981c49a8a5043ae218

mw.loader: Optimise hot code paths in addEmbeddedCSS()

addEmbeddedCSS() is a big part of the hot code path that moves a module from
state "loaded" to "ready". Especially on repeat views (where most loads
are cache hits from local storage), this is the main thing that JS spends time
on before running scripts (which must wait for the styles to apply first).

* newStyleTag: Avoid use of jQuery.
  Before
  - jQuery()
    - jQuery#init
  - jQuery#before
    - jQuery#domManip, jQuery#buildFragment, jQuery#inArray
    - Node#insertBefore
  - Node#appendChild
  After
  - Node#insertBefore
  - Node#appendChild

* getMarker: Store raw Node instead of jQuery object. Makes it easy for other
  code to avoid jQuery. And for those that don't, creating a jQuery object is cheap.
  Also use querySelector directly since it's ensured by our feature test.
  The only cases jQuery/Sizzle accounts with querySelector is IE8 (already excluded
  by our feature test), and Opera 12 (in an edge case that doesn't apply to this
  selector).

  Before
  - jQuery
    - jQuery#init
  - jQuery#find
    - Sizzle
    - querySelectorAll
  - jQuery#pushStack
  After
  - querySelector

* addEmbeddedCSS: This was needlessly calling the fairly slow .data() method for
  all style tags in all browsers. It should've been guarded by IE<=9 if-statement.
  The consumer of this data property already had that check. The setter did not.

  Before:
  - getMarker
    - ..
  - newStyleTag
    - ..
  - jQuery#data
    - jQuery#each, jQuery#data, internalData, ..
  - fireCallbacks
    - ..
  After
  - getMarker
  - newStyleTag
  - fireCallbacks
    - ..

Change-Id: Ie5b5195d337b5d88f0c2ca69d15b13a4fb9d87e2

Merge "API: Avoid duplicate IDs in API documentation"

Merge "ResourcesOOUI: Remove deprecated oojs-ui.styles module"

Merge "mediawiki.page.patrol: Use this.href instead of $(this).attr('href')"

Merge "build: Enforce stylelints on fonts"

ResourcesOOUI: Remove deprecated oojs-ui.styles module

Change-Id: I4e6b8897d9a443a11d20fd5d870b6e38e21afd1d

Merge "build: Enforce stylelints on spacing around declarations"

Merge "build: Enforce the rest of the colour-related stylelints"

build: Enforce stylelints on fonts

* Font families should be quoted unless keywords (''Times'' not 'Times')
* Font weights should be named where possible ('bold' not '700')

Change-Id: I20194c2998efb71db4da5ea79234a81dc90b55ea

build: Enforce stylelints on spacing around declarations

* A space before but not after a 'bang' (' !important' not '! important')
* A space after but not before a colon ('display: block' not 'display :block')

Change-Id: Ice36c2034e94e62377bac8a110ff4b16b7187e85

build: Enforce the rest of the colour-related stylelints

* Hex colours must be in short form where possible ('fff' not 'ffffff')
* Hex colours must be used over named colours ('fff' not 'white')
* Hex colours must be valid ('fff' not 'ffq')

Change-Id: I2ba04cc3ad9898c17fee3c65bb3bead834c3a1fd

Merge "Fix alias capitalization warning"

Merge "build: Bump grunt-karma and related tools to 1.0.x"

Merge "mediawiki.special.search.css: Fix chmod"

Merge "build: Introduce stylelint"

Merge "Log logouts"

Log logouts

Not terribly useful but we log all other auth events so why not.

Bug: T125184
Change-Id: I636f0eb5e1e9ffc49023941801c7c286410514d8

build: Bump grunt-karma and related tools to 1.0.x

Change-Id: I3d8ad9fbd3b6950c13bf400790eb97fe4e94f849

mediawiki.special.search.css: Fix chmod

Change-Id: Ib26b4e882ef33fb291df656214ef0fa244603557

build: Introduce stylelint

Initially, with just a single rule to make this a smaller change that we can
incrementally extend up to the full Wikimedia coding conventions preset; namely,
forcing hex references to colours to be in lower-case.

Note the odd src definition to avoid referring to mediawiki.less the directory in
the glob.

Change-Id: I34f0c8e3c8280e3fce0aa22d7537100b850cbb8d

Merge "Allow resources to be salvaged across service resets."

PHPUnit: turn off verbose option

It's very common these days for test runs on ordinary rigs to skip hundreds of
tests, which makes getting the part of the results you care about (the test
failures) an arduous trip up hundreds of lines in your terminal buffer.

Change-Id: Iaf714b86473379e1ad32e4d2d93e83018a757782

Allow resources to be salvaged across service resets.

NOTE: This also changes the semantics of MediaWikiServices::resetGlobalInstance
to only reset services instances, not service wiring. The wiring will be copied
from the old global MediaWikiServices instance to the new one.

Bug: T132707
Change-Id: Ie2ca3ff99aa74fffa9eb6c8faccab857dc0874f7

Merge "Fix resetServiceForTesting() param type in comment"

Merge "Disable Cucumber scenarios that are broken in daily Jenkins jobs"

Merge "tests: Remove duplicate unit tests in HtmlTest.php"

Fix alias capitalization warning

This avoids:

"Found alias defined for Revisiondelete when searching for special page
aliases for RevisionDelete. Case mismatch?"

Change-Id: I35ba1d3d4afbf3e3aca093e17c2de459413d05d3

Merge "Make LinksUpdate only wait on the DB with the link tables"

Make LinksUpdate only wait on the DB with the link tables

Bug: T135690

Change-Id: If79ca385884f422e5ed11e77b29033c8cabf494c

RecompressTracked: Remove useless member variable

It's only checked once and has no external users

Change-Id: I6951b799759f1d8b0063a3059bec5c1c64a35815

Enable AuthManager by default

Bug: T135498
Change-Id: I6695aa3da42fb2b088eaa8d1883ccbb67f2c0c38

Merge "Don't use deprecated User::checkPassword() in TestUser"

Merge "Allow to chaining calls in setSubmitDestructive and setSubmitProgressive."

tests: Remove duplicate unit tests in HtmlTest.php

Follows-up ee4d5c6ee ("Remove support for $wgWellFormedXml = false")

Remove now-redundant duplicate tests that were previously there
to test the same thing again with $wgWellFormedXml set to false.

Change-Id: I91623a8c10e5108ffae13f67328aece2cf33c719

Localisation updates from https://translatewiki.net.

Change-Id: I8f24e94f2d29d5befd905c8e4cb5fa89002ecd38

Fix resetServiceForTesting() param type in comment

Change-Id: I05ba45a812a6d6aa7e9c90ea17b5a24b606e815c

Fix-up for I52c511be04: call parent setup / teardown methods

Change-Id: Ic66fa5ae61f1d1822d7cc20eafa6c01a2bdb6e37

Merge "Revert "Make an empty "?action=" parameter default to "view"""

Merge "In MediaWikiTestCase::stashMwGlobals(), prefer shallow copies"

Revert "Make an empty "?action=" parameter default to "view""

This reverts commit b287ec2cc649a96bb19c893c46f88734c6392204.

Change-Id: I59e996dfe627e8a978ed849a8e82ad4e0b165d7f

Merge "Use WIS::getNotificationTimestampsBatch in ApiSetNotifTimestamp"

Merge "Add ApiSetNotificationTimestampIntegrationTest"

mediawiki.page.patrol: Use this.href instead of $(this).attr('href')

This is the difference between prop('href') and attr('href'). The attribute
returns the (usually relative) url authored in the HTML, whereas the property
returns the effective url, which is what mw.util.getParamValue expects.

Also remove the single-use variable and needless use of jQuery
(property access is more idiomatic through, well, properties).

Change-Id: Id8589dbb24869d5a88351daa65d51ab8602f5c77

Merge "Increase BotPasswordSessionProvider's default priority"

Merge "Fix unstyled updatedmarker"

Merge "Add actual documentation for ContentHandler::getActionOverrides"

In MediaWikiTestCase::stashMwGlobals(), prefer shallow copies

MediaWikiTestCase::stashMwGlobals() performs a deep copy of globals by
serializing and then unserializing them. This is actually unwarranted in the
common case of simple scalar values and flat arrays -- and it's expensive, too.
So before attempting a deep copy, first check if a shallow copy will do.

Change-Id: Iaba1c8e1f6bae9de0a7a1fb411cac94f7e4dfb23

Don't use deprecated User::checkPassword() in TestUser

I423f09f added this call in an attempt to speed things up. However,
User::checkPassword() was deprecated in I2c736ad7, and the call causes
some tests to fail when $wgDisableAuthManager is false.

Since this is trying to determine whether the user_password column in
the user table needs updating for the unit test, let's test that
directly instead of the much-more-complicated User::checkPassword().

Change-Id: I410de706f9074edea3f3988769a3e99231d8dca3

Allow to chaining calls in setSubmitDestructive
and setSubmitProgressive.

Change-Id: I926fa717edf146c64bb7e5287f9dc051e30f6566

Increase BotPasswordSessionProvider's default priority

CentralAuthSessionProvider is using 50, BotPasswordSessionProvider needs
to be higher than that. Since the current 40 is a bit close to
CookieSessionProvider's 30, let's go higer here instead of lowering
CentralAuth.

Change-Id: I1230d23c53dacc8e01db955507aacba0e98e62d3

Fix unstyled updatedmarker

This is the "changed since your last visit" marker that you'll
see on ?action=history for a page on your watchlist, marking
edits that were made since you last viewed the page.

updatedmarker is styled by some skins, but not all of them.
I figured it's probably better to style it from history-specific
CSS in core than to fix Vector to style it.

Bug: T134515
Change-Id: I123f0b19e3869d3885bb38b7a5609941fcee600b

Merge "Title->getContentModel: Get new content model with GAID_FOR_UPDATE"

Add actual documentation for ContentHandler::getActionOverrides

Change-Id: I7fd803204ad137dd3510ea321ff7e6fa11d910c0

Title->getContentModel: Get new content model with GAID_FOR_UPDATE

If a new revision has been inserted (e.g. with updateRevisionOn),
the content model may have changed.  This happens with e.g. undeletion.

Bug: T122262
Change-Id: Ia0ed86a9c24809256215418673e9ee8e263d1349

Merge "MWDebugTest: disable MWDebug on test teardown"

Merge "Speed up password-handling in the unit tests"

Use WIS::getNotificationTimestampsBatch in ApiSetNotifTimestamp

Bug: T134387
Change-Id: I560ae6a29fa27c1e4f1f62aa647e14542b0cc8a9

Add ApiSetNotificationTimestampIntegrationTest

Change-Id: If0bc1f56533102f54c0031eea548c20d8abe1818

Merge "add setNotificationTimestampsForUser to WatchedItemStore"

Merge "Factor InterwikiLookup out of Interwiki class."

Merge "Make an empty "?action=" parameter default to "view""

Factor InterwikiLookup out of Interwiki class.

This keeps the existing app logic for looking up interwiki information
intact in ClassicInterwikiLookup. The idea is to seamlessly switch to a new
implementation when it becomes available, while also allowing us to
switch back in case of problems.

Change-Id: I7d7424345d0ce3ce90ba284006ee9615e3d99baa

MWDebugTest: disable MWDebug on test teardown

MWDebug::init() is currently irreversible -- once MWDebug is enabled, it cannot
be disabled in that execution context. This means that the MWDebug test suite
(which enables MWDebug) has a nasty side-effect -- all the tests that run after
it run with MWDebug enabled. So add an MWDebug::deinit(), and call it on test
teardown.

Ostensibly this is a great use-case for services and dependency injection. The
reason I am not going that route is that it's not entirely clear to me what the
MWDebug class is supposed to represent. If I were going to spend any
substantial amount of time on this, I would be trying to move it out of core
and into an extension, not converting it into a service.

Change-Id: I52c511be049bc276d203d07283e3aa0944f22d34

add setNotificationTimestampsForUser to WatchedItemStore

Bug: T134387
Change-Id: Ia6abe7687b51aabe67e8461375075692db28c9a2

Improve diff docs

Change-Id: I42332051f6f1b94f459b06d6c69625bc2a1fa9cf

Merge Wikidiff3 into DiffEngine

Change-Id: Ib4d083a5200824e4d032de6921c375e455e77fb2

Remove unused stuff from diffs

Change-Id: Ie7d46b9ed290912039c88bbf4548d4c3d97153c9

Refactor diffs

* Merge MappedDiff into WordLevelDiff
* Rename <something insane>WordAccumulator into WordAccumulator and namespace
* Better variable names

Change-Id: I5847f2bb89402d0537b9e99cccd24c547ce4e4e2

LinkerTest: Fix passing test description to $this->assertEquals()

It was being passed to Linker::userLink() by accident.

Change-Id: I13ba5014193e91a467511259a0c7c1659d8bc0e6

Remove "Not logged in" from the personal URLs array

The array of personal URLs should only contain links to the personal tools of the
user. "Not logged in" is a status message, not a personal URL, so it should not
appear in that array. The status should instead be indicated to the user by the
skin itself in whatever way seems best for the skin in question.

The following patches depend on this one to reintroduce the message:
* Vector skin: Ic560d3fd
* Monobook skin: I7001e311

Bug: T121793
Bug: T127758
Change-Id: I5cfa9b4e2f4eb420d5fda6b1c6ce28926cb8e738

Localisation updates from https://translatewiki.net.

Change-Id: I170960df4963c50ee00e0f9d3d876061a7f93378

Improve edit stash hit rate for logged-out users

Check whether they made intervening edits just like logged-in users.

Bug: T134620
Change-Id: Id1e0808caee0e474570c4f2e4b1cf845572e17e0

Make an empty "?action=" parameter default to "view"

When no action is given, e.g. in
https://en.wikipedia.org/wiki/URL
the action defaults to "view". Just like you called
https://en.wikipedia.org/wiki/URL?action=view

But when the action is empty, e.g.
https://en.wikipedia.org/wiki/URL?action=
you get an error message telling you that "the action specified"
can not be "recognized". Wait, I did not "specified" an action.
That's why I left the parameter empty. From the users point of
view I expect the empty string to behave like null/undefined.

This is a resubmit of I331924d without the problematic bits.

Change-Id: I07847600bb24ae078276acf98e6eb244039414d7

Fix installer issues introduces by MediaWikiServices

This fixes three issues with the installer:

1) Make sure LocalizationCache can find the installer's i18n files.
2) Make sure we don't try to use an SqlBagOStuff for caching before we have
   a functioning database.
3) Don't try to output HTML when redirecting (this is unrelated to
   MediaWikiServices, but came up during testing)

Bug: T135169
Change-Id: I7caa932024cd771d6fa226a3ac6001c3148ecc9c

Merge "Let BagOStuff::merge() callbacks override the TTL"

Merge "Increase LocalFile lock() timeout and improve error message"

Merge "mediawiki.util: Use RegExp.test() instead of String.search()"

Don't override all Moment locales to English

Calling moment.locale() not only defines a locale, it also sets the
current one. The call at the top of moment-locale-overrides.js would
always set it to English.

Follow-up to c81ab8ae5f89056d23a4b0d36066120b9701d07e.

Bug: T135462
Change-Id: Ie1b70bc410ff0c436bed5bcbfffb8d4433a1923f