Revert "mediawiki.api: Use action=query&meta=tokens instead of action=tokens"

This reverts commit aacdb664a10d5eca2b55891e5a75ca8c0368a11b,
which unintentionally broke backwards-compatibility, affecting
at least four extensions of which two deployed in Wikimedia projects.

Change-Id: I2e7e761281dae6a22fc520c9153734b55403b120

mediawiki.api: Use action=query&meta=tokens instead of action=tokens

api.php?action=query&meta=tokens has different token types.
A mapping keeps the JavaScript API stable.

Bug: 72094
Change-Id: I429b609b626af35ae0abcdf6f1c13e8afb6503b9

Merge "Properly mark UserGetImplicitGroups hook as deprecated"

Merge "TransactionProfiler now shows the delay periods between queries"

Merge "Special:Contributions: add "mw-contributions-list" class"

TransactionProfiler now shows the delay periods between queries

* Also tweaked the slave query display threshold to show them more

Change-Id: I0d9002b2f12e846a7355992443287d4ec43a7e88

Localisation updates from https://translatewiki.net.

Change-Id: Ia37088f36e8431d223c2eaa1c846ad022dd02c1c

Merge "mediawiki.ui: checkbox: Fix states according to spec"

Merge "mediawiki.ui: checkbox: Remove unnecessary nesting and group variables"

Merge "jquery.tablesorter: Remove unused config variables"

Merge "Allow limiting Monolog output using legacy settings"

Merge "Add Monolog formatter that mimics legacy log output"

jquery.tablesorter: Remove unused config variables

Change-Id: I8640621a2e337f9db363d7730a3d55c668efb7e9

Allow limiting Monolog output using legacy settings

Add $useLegacyFilter option to MWLoggerMonologHandler constructor that
will use MWLoggerLegacyLogger::shouldEmit to decide if a given log
message should be emitted.

Bug: T845
Change-Id: If311308faad35348fdc7e85155a1bc16bbf75c85

MediaWiki UI: Checkbox follow up

Reset the font of the label and checkbox elements as defined
in the user agent stylesheet to avoid overlap

Bug: 72727
Change-Id: I3d02989768bea400e6498fe9e84b1a7d965a2529

Merge "Fix errors in UIDGeneratorTest::testTimestampedUID"

Add Monolog formatter that mimics legacy log output

Having a log formatter for the Monolog stack that mimics the legacy wf*
logging function output will ease the transition for users wishing to
use Monolog who have tooling that expects the legacy log formats.

Bug: T845
Change-Id: I06295ccc4b068c61d7971024213366004b69c03d

Merge "Fix AutoloadGenerator to work on MediaWiki-Vagrant"

Merge "Update OOjs UI to v0.1.0-pre (d4cfcce969)"

Merge "mediawiki.ui: button: Add specific hover and active rule for color"

Update OOjs UI to v0.1.0-pre (d4cfcce969)

New changes:
f6adfbc Give help content box a reasonable line height for legibility
af5d71c Remove CSS hack for positioning popupButtonWidget
c0768f9 PHP: Add missing OOUI\Exception class
76800b4 PHP: Wrap __toString() in try-catch to avoid unhelpful fatals
d399920 Revert "Followup ca6a16f: use parent selector both for hiding and showing windows"
234cd9f Revert "Show the modal overlay as soon as openWindow is called"

Change-Id: I317b4a747389de35411aca4ffdbbc2306b5b74cc

Fix errors in UIDGeneratorTest::testTimestampedUID

Currently, this test passes in Jenkins, but not with Travis
CI's hhvm where apparently code execution is slower so not
all UIDs are created within the same millisecond.  This also
previously masked that the test did not indeed test if the
host bits are identical as instead it tested the MSBs twice.

Bug: 73668
Change-Id: Ic68af599599ace5faa6a687d115d2d3802f6ba1b

Fix AutoloadGenerator to work on MediaWiki-Vagrant

The use of realpath() in AutoloadGenerator::readFile() causes the
LocalSettings.php symlink to be dereferenced. Since the target file
lives outside of $IP, AutoloadGenerator would fail with an exception.

Change-Id: I4623b3da9b984026999189d70349ffb4754812a5

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

Localisation updates from https://translatewiki.net.

Change-Id: I23cc232726bb0d01ff13ed87b5e44494421441ab

Merge "Update OOjs UI to v0.1.0-pre (20c61ec865)"

Merge "Add array type hints to LoadBalancer classes"

Merge "Fix/update inline documentation of LoadBalancer"

Update OOjs UI to v0.1.0-pre (20c61ec865)

New changes:
20c61ec Follow-up Id9597fde: Restore toolbar-bar class

Change-Id: Ia16292b3ee46f5b38ee216927a9796a8fdf9235d

Add @since 1.25 to $wgMangleFlashPolicy documentation

Change-Id: I2b2c3b3f5ebe501bfb5fed39264345077895c26b

Merge "API: Work around wfMangleFlashPolicy()"

Merge "FormatMetadata::fetchExtendedMetadata: Ignore multiple EXIF/XMP values"

Add array type hints to LoadBalancer classes

Change-Id: I3dde038a71da8f775074916d47fa7f34eb37bf3f

Fix/update inline documentation of LoadBalancer

Plus two very minor fixes in the code,
* an unused variable and
* a broken error message.

Change-Id: I40a39c81bd0d6d6054f983c82880f4bf31c21275

API: Work around wfMangleFlashPolicy()

The things wfMangleFlashPolicy() does to the output break things in the
API. For JSON we can work around it, while for PHP we just have to error
out. XML isn't affected because <> are escaped anyway (unless something
somehow uses 'cross-domain-policy' as a tag name), and the rest are
going away soon so they're not worth the trouble.

Bug: 66776
Change-Id: Idc5f37bd778288a9cde572f081dc753d681ec354

Remove deprecated (since 1.21) Sites class

this is an 'alias' for SiteSQLStore and is unused
afaik in Wikibase and elsewhere.

Change-Id: Ibee3e2a89399b97fa4de48e52c3d527714cd0731

Special:Contributions: add "mw-contributions-list" class

To make the contributions' list easier to identify for user
scripts etc.

Bug: 73253
Change-Id: Ief380b9d11df7d91752abea60d7dbed8c4a74fb6

Merge "Made $timeout in BagOStuff::lock() actually work"

Unbreak lock()/unlock() for MemcachedPhpBagOStuff

Change-Id: I49ce96b65fe6a6e2cba86a990962fe4efc43bbb8

Fixed BloomCache handling of network partitions

* As documented, it should return true on error, so that DB is
  checked if the filter is down.

Change-Id: I883fafc9f5f3a84f85207de6e916f1630c78d1a4

Fixed regression that made log queries happen in spite of the bloom filter

Change-Id: I50f280a1db30cb1f9901a55a8a9558ebb2a9dffd

mediawiki.ui: checkbox: Fix states according to spec

 * Color changes
 * Introduces hover state
 * Separates focus and active state
 * Removes need for inset border
 * Adds white icon for disabled & checked state

Design specification on Trello-
https://trello.com/c/JETLmm7F/7-check-boxes

Change-Id: I891f05c8edd010b81bd5f35eeae5d5dd22169933

Made $timeout in BagOStuff::lock() actually work

* Also added a separate $expiry option for things that take a long time

Change-Id: Ie5f81dea031f3c3f3ca8d61ad4cb322a5b876f61

mediawiki.ui: checkbox: Remove unnecessary nesting and group variables

Change-Id: I4caeaf52d5c38391d88f4e6218d9f3ee26346981

mediawiki.ui: Fix label text for inputs in KSS

Change-Id: I9d49bce9b197646409b115632895088909bb9458

Merge "resourceloader: Update cssjanus to v1.1.1"

Update OOjs UI to v0.1.0-pre (1fa4eb7a73)

New changes:
549ff9a [BREAKING CHANGE] Rename *Menu and *MenuItem widgets

Change-Id: Ifded7f9e17923dbb56a722b7424ccd2854194c21

resourceloader: Update cssjanus to v1.1.1

Bug: 61440
Change-Id: Id0221e9f68786758e10b2f222f5e9170898e320d

Update OOjs UI to v0.1.0-pre (7922a50558)

New changes:
491d279 Change document order of tools and actions to fix floating
25840d5 Add lang and dir attributes to the accelerator element
8a87ddf build: Update various devDependencies to latest
b0de093 Fixed RuboCop Style/AndOr offense
577ed71 Fixed RuboCop Style/AsciiComments offense
565251b Fixed RuboCop Style/EmptyLineBetweenDefs offence
328710e Fixed RuboCop Style/EmptyLines offense
f2a1811 Fixed RuboCop Style/NegatedIf offense
51e041a Fixed RuboCop Style/SignalException offense
6b3bc5c Fixed RuboCop Style/SpaceAroundEqualsInParameterDefault offense
9224811 Fixed RuboCop Style/NilComparison offense
6ff2b19 Fixed RuboCop Style/SpaceAfterComma offense
22c86e2 Fixed RuboCop Style/TrailingBlankLines offense
33fc646 Consistently use @return annotation
709546c Localisation updates from https://translatewiki.net.
6e6dfae build: Clean up fileExists config

Change-Id: I528111c92b89e0302d679ed817a98671035cd8aa

Merge "Hygiene: Update mixin file to use single line comments"

Merge "Remove less.js"

Localisation updates from https://translatewiki.net.

Change-Id: Idb44769ae33d7beaa3430c9133c958b0745145df

Merge "Make calling wfMangleFlashPolicy configurable"

Make calling wfMangleFlashPolicy configurable

Add $wgMangleFlashPolicy so sites that define a safe master flash
policy can disable mangling of the <cross-domain-policy> string.

Bug: 66776
Change-Id: Ic0db939aae9d0bb6bdfc3cd9ce282661bad46783

Merge "Avoid sending empty function names to TransactionProfiler"

Avoid sending empty function names to TransactionProfiler

* This could happen if the Profiler was a stub

Change-Id: Id4f635f899ee3c0ca85acf1b1e4cde47dcbbdc75

Handle missing parser cache keys better in pool counter

Change-Id: I493fd1ee5e9ab6c3a49a7f478460cbfe54393ca0

Merge "doc: Clean up Doxyfile for doxygen 1.8.6"

Merge "Remove calls to legacy Profiler::debug()"

Merge "Placed ProfilerStandard logic directly into SectionProfiler"

doc: Clean up Doxyfile for doxygen 1.8.6

Latest stable is v1.8.8, but Ubunu Trusty (which we now use on the
Jenkins slave that generates our documentation) ships doxygen v1.8.6.
Time to migrate away from 1.7.x stuff.

Warnings:

 Tag `SYMBOL_CACHE_SIZE' at line 84 of file maintenance/Doxyfile has become obsolete.
 Tag `SHOW_DIRECTORIES' at line 118 of file maintenance/Doxyfile has become obsolete.
 Tag `HTML_ALIGN_MEMBERS' at line 226 of file maintenance/Doxyfile has become obsolete.
 Tag `USE_INLINE_TREES' at line 253 of file maintenance/Doxyfile has become obsolete.

Regenerated with:

 $ doxygen -u -s ./maintenance/Doxyfile

Which automatically added new configurations settings, removed
obsolete ones, migrated formatting to their new standard.

Change-Id: I372ea06c439762578cb4c68686344eb446925c04

Merge "Make Title::makeTitleSafe() not need user variant"

Remove calls to legacy Profiler::debug()

Profiler::debug() was moved to ProfilerStandard. Since we have
a proposed patch set already that will remove
wfProfileIn()/wfProfileout() calls that wrap whole methods this seem
like an easy problem to solve by removing this logging functionality.

Change-Id: I429ec29fe4393ce886361b896b650fd39480eda5

Remove less.js

Change-Id: I2ec714e79463835016dcfbffcf03f052167a0b61

Use cdb library and provide a back-compat layer

The new cdb library is pulled in via composer. Since the
library uses namespaces, a backwards-compatability layer
is provided for the old class names:
* CdbReader
* CdbWriter
* CdbException

The PHP/DBA-specific classes should never have been used directly.

Depends on I98302bdf1 in mediawiki/vendor

Change-Id: I39549ac8540b262cf91f7d1830d36327afb3033d

Make Title::makeTitleSafe() not need user variant

In trying to avoid hitting RequestContext::getLanguage() from a call to
User::idFromName(), I05aacd30 made it no longer safe for user input.

A closer analysis of the call stack involved reveals that
Title::makeTitleSafe() is constructing a prefixed dbkey using the
localized name for NS_USER, and then Language::getNsIndex() is needing
to get the variant in order to handle that localized name. But if we use
the canonical name for NS_USER, Language::getNsIndex() short-circuits
and skips the problematic code path.

And it turns out that it doesn't actually matter which prefix
Title::makeTitleSafe() uses, since the prefix doesn't make it anywhere
into the resulting Title object. So let's revert I05aacd30 and Ibeef0409
and just do that instead.

Change-Id: Ib902573996c69d1e77527cc7b2faf4e7fa5d3daf

Add logging context to database logs

* Add optional $context parameter to wfDebug, wfDebugLog, wfLogDBError
  and wfErrorLog that will be passed to MWLogger.
* Add support for PSR-3 style log message parameter interpolation in
  MWLoggerLegacyLogger.
* Add context information to wfLogDBError calls made from DatabaseBase,
  DatabaseMysqlBase and LoadBalancer instances.
* Deprecate wfDebugTimer() which now appears to be unused.

Change-Id: Ic90d593d00a2b0b5b80ed205908cbe624042603c

mediawiki.ui: button: Add specific hover and active rule for color

Overrides general a:active and a:hover rules.

Bug: 73587
Change-Id: I36fa7d66d21f0133ea5525962a8b8e8ffab90494

Merge "Bug: Append to wgAutoloadClasses"

Bug: Append to wgAutoloadClasses

When generating $wgAutoloadClasses for an extension the generator
currently outputs an assignment (=), but it is unlikely this is
the desired result. An extension wants to append to the existing
$wgAutoloadClasses. This bug is an unintended consequence of I75403ace
which changed the generator from assigning one key per line to using
an array literal.

This patch changes the output only when generating $wgAutoloadClasses
to the += operator which adds to the array any value that is not
already in the array.

Change-Id: I7d42ee5dc829991c6562878f0c90a06fadb1b6a6

Fix vim folding

The "@{" in these two doc comments was being interpreted as an opening
mark and was causing almost the whole file to be in a single section.

Change-Id: I818ecd2f7621a22b6e8f5a4b93faee6fbf3664dc

Merge "Follow-up to I05aacd30b: normalize case in User::idFromName"

Follow-up to I05aacd30b: normalize case in User::idFromName

Change-Id: Ibeef04093338bfef8e65f40b1f9db588c8d5a79d

Merge "Omit 'window.' when accessing browsing location"

Merge "Expose ID of relevant page in JS variables"

Merge "Don't re-apply EXIF rotation to chained thumbnails"

Omit 'window.' when accessing browsing location

The location object is a global, just like document.

Using it via 'window' needlessly adds complexity and, for example,
makes it harder to catch typos in static analysis.

Also standardise on location.href in place of the many different
variants, like:

 location =
 location.href =
 location.assign() =

And each with 'window', 'document' and without host object.

Change-Id: I77510294d8b5bd4b8a1b08e06817762a7839d43d

Merge "Cleaned up template profile report tabbing"

Cleaned up template profile report tabbing

Change-Id: I46abfc856d718d4db73d0510bde3e2b589341b10

Placed ProfilerStandard logic directly into SectionProfiler

* The former class should eventually be removed

Change-Id: I4e6df3bd0c7047f5f5110e7f115b1cf6fad0316e

MW_NO_OUTPUT_BUFFER is never defined

It may have been defined at one time, but now it's just
function overhead to check on every request.

Change-Id: I05963a0e496e0a960a4feb6877a03e67c5a6fc85

Merge "profiler: Simplify code for class name mapping"

Merge "Fixed some incorrect Profiler comments"

Merge "Use upsert() in ProfilerOutputDb"

profiler: Simplify code for class name mapping

Follows-up 000c6529d. Handle errors first.

Change-Id: I70308783a2e8b140b3c850758cce36544b32eb20

Fixed some incorrect Profiler comments

Change-Id: Ibadc9c7aa0e199e3bd0776a3d563fb6afc1ff7cd

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

Use upsert() in ProfilerOutputDb

Change-Id: I807ee78c602b67c99530ecaab90af74410106fc7

Localisation updates from https://translatewiki.net.

Change-Id: I72e27a8e125fc604c5fe47f73b243b96b9b50cc6

Merge "ProfilerOutput: Remove logStandardData() and make log() abstract"

ProfilerOutput: Remove logStandardData() and make log() abstract

Became unused layer of abtraction as I266ed820 went through iterations

Change-Id: Ic739fb4519bb2e23e679828b191c11c0158ade78

Profiler: Explicitly convert output type to class name

Follows-up b8d93fb4fd06.

Change-Id: I50faa25fdc4fb980e0ff71014b8d1b2a5818af80

Merge "Refactor profiling output from profiling"

mediawiki.action.history.diff: Rework print styles

* Remove extremely old print styles for diffs from commonPrint.css,
  added back in r6970. These were created for the old yellow-green-red
  diff styles and look terrible now.
* Add new, minimal print styles, that replace background colors with
  text decoration (underline for inserted text, line-through for
  deleted). Motivation is the same as in r6970: light colored backgrounds
  work great on displays, but not very well in black-and-white print.
* Fixed diff styles demo (mediawiki.action.history.diff.html), updated
  with print styles and better example.

Bug: 73544
Change-Id: Ibf08fa11f84ac0cf8c7ed4da8af2b8804de6ab6f

Merge "filebackend: Log more details about backend-fail-internal errors"

Refactor profiling output from profiling

* Added a standard getFunctionStats() method for Profilers to return
  per function data as maps. This is not toolbar specific like getRawData().
* Cleaned up the interface of SectionProfiler::getFunctionStats() a bit.
* Removed unused cpu_sq, real_sq fields from profiler UDP output.
* Moved getTime/getInitialTime to ProfilerStandard.

Co-Authored-By: Aaron Schulz <aschulz@wikimedia.org>
Change-Id: I266ed82031a434465f64896eb327f3872fdf1db1

Avoid calling Title::makeTitleSafe in User::idFromName

We don't want to call Title::makeTitleSafe yet, since that call path
ends up needing the user language, which ends up trying to load the
user object, which ends up back in User::idFromName.

Bug: 54193
Change-Id: I05aacd30be66fe505b5c211113ea938fa82e2492

Update OOjs UI to v0.1.0-pre (f1abca8e82)

New changes:
5883e65 Add 'indeterminate' state to progress bar widget
fbe9248 Fix documentation for getRelativeSelectableItem
27f8805 Localisation updates from https://translatewiki.net.
2edfdaa Localisation updates from https://translatewiki.net.
40d08f0 Localisation updates from https://translatewiki.net.
ce8438f Update OOjs to v1.1.3
5721411 Prepare repository for RuboCop
f1abca8 Followup ca6a16f: use parent selector both for hiding and showing windows

Change-Id: I7b49bbc81e9b8ba501f0593daaacf7025f16bf00

filebackend: Log more details about backend-fail-internal errors

Bug: 73094
Change-Id: I9e45284e3cfd55bc34d87009387a2c1766e2db4a
Mingle: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/991

Localisation updates from https://translatewiki.net.

Change-Id: I2e0ff8b47f799752f656915729510dad81c00d4a