rdbms: minor improvements to LoadBalancer logging

Change-Id: I03f54a65b1932b99ac4e129000547a6c2640c489

tests: Remove IE9 mobile from Grade A

Change-Id: I0e1da9d155c778edaa85d19a3e556fe14f83fdd2

Fix infinite loop in JpegMetadataExtractor

One of the skip-over loops was missing an feof() check and could
cause infinite loops.

Includes test file created by truncating a tiny tiny .jpeg at
the right place...

With the fix, it doesn't loop but dies on an exception, which
is good!

Bug: T184048
Change-Id: Ica13d6b68c3c12f7ce414edd081bf0886714e465

Add TimedMediaHandler's images/transcoded to .gitignore

Avoids git looking in your transcoded audio/video files
for changes...

Change-Id: I5610354a1575bb97f4523d58d3e9713512416305

Merge "sql.php: Provide --json output mode"

Merge "Drop Grade A support for IE10"

Drop Grade A support for IE10

Bug: T187869
Change-Id: I68c882e64fba95d5e1bf41f64658fcef7524dcf5

Merge "Contributions: Only add form JS when the form is added"

Merge "Update README file for Selenium tests"

Merge "Truncate tag filter descriptions"

Merge "Fix English name for lb"

Update README file for Selenium tests

Bug: T187862
Change-Id: Iadd41307a48a1b50f64a495e7efcee3b88d0ff68

Fix English name for lb

This version has been in MediaWiki longer than my email history,
since 2005 at least. This spelling is not present in
https://en.wikipedia.org/wiki/Luxembourgish

Change-Id: Ibda7e6428a2c79b9f7d88892ef1c16e9921ae934

Merge "Factor OldRevisionImporter & ImportableOldRevision out of WikiRevision"

Merge "Factor UploadRevisionImporter & ImportableUploadRevision out of WikiRevision"

Contributions: Only add form JS when the form is added

Special:Contributions can be transcluded ('including') on to another
page. In this mode, the form is not present, but the OOUI JS infusing,
payload and styling, was added regardless. This caused errors on pages
transcluding such lists, as the OOUI HTML was missing, causing the JS to
throw errors.

Bug: T187710
Change-Id: I7d45dd3ddd3ba75d44f7d24e03cef5e95301f4ed

rdbms: make sure non-native replace() uses one transaction

This is similar to what upsert() already does

Change-Id: Ide83eefe0d937fb2cdc20aa3c7dc9654c4d34beb

Merge "mw.Title: Use $.trimByteLength from jquery.byteLimit module"

Merge "tests: Remove dead code in jquery.byteLimit tests"

Merge "Switch the sidebar cache to using checkKeys"

mw.Title: Use $.trimByteLength from jquery.byteLimit module

Change-Id: I39ebd559a72ce72743cbe3bb86abe679d108d94a

tests: Remove dead code in jquery.byteLimit tests

These options do nothing.

Change-Id: I15cc91b9208e8cfd3f3f42bb0a9c91514d6532e3

Merge "parser: Deprecate string type for $lang arg in DateFormatter::getInstance"

Merge "jquery.byteLimit: Handle characters outside BMP (surrogate pairs) when trimming"

parser: Deprecate string type for $lang arg in DateFormatter::getInstance

The parameter $lang is not used with type string in the Git repository:
https://codesearch.wmflabs.org/search/?q=DateFormatter%3A%3AgetInstance
This function is only used with type Language and without parameter.

The support for the language as string will be removed with Change-ID
I677ab561d67b63f0c86f65fadf9319e41444a22a.

The removal of the support for the type string simplifies the code.

Change-Id: I0c0ab46ffc0fab6b68672940e979ad7e5754a7d1

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

Localisation updates from https://translatewiki.net.

Change-Id: Iea6405478e926c0bce200213a211bb66da7efbf3

Merge "mediawiki.cldr: Use require() for pluralruleparser"

Fix for Ia97b750b

Change-Id: I2eb5ba754334864f65dede92b138f9a81a210e70

Merge "Add support for 'es-formal'"

Merge "mw.loader: Document mw.loader.require() as private"

Factor OldRevisionImporter & ImportableOldRevision out of WikiRevision

This is to be used within the FileImporter extension to allow
adding custom loggers to this import process.

Change-Id: Ib094d4829764ccc8e5bd2619fb827d701ae06d43

Factor UploadRevisionImporter & ImportableUploadRevision out of WikiRevision

This is to be used within the FileImporter extension to allow
adding custom loggers to this import process.

Change-Id: I4a6c573fc0a69b06d696cd2afca9226fb492a9bc

mediawiki.cldr: Use require() for pluralruleparser

Use require( 'mediawiki.libs.pluralruleparser' ) instead of
mw.libs.pluralRuleParser. (Added in 1.27, with bc4e07b6f63b0)

Change-Id: I7388cf631068458596250b8da82edec3f7cbd98b

mw.loader: Document mw.loader.require() as private

Only public for debugging purposes.

Change-Id: Icc44e11ec6874d7f74ed527e87a80dd06b4edb3a

Merge "Cleanup IDEA warnings in MessageCache"

GD zero filesize error

Checking if the filesize is zero before passing to GD. This can result
in a fatal error.

Bug: T182387
Change-Id: I26a24286f3035f233de682caed21a8e8c2d5aaad

Merge "Pass change tags to NewRevisionFromEditComplete hook"

Merge "rdbms: tweak var names in LoadMonitor::scaleLoads()/getLagTimes()"

Merge "tests: Remove verbose logging settings from DevelopmentSettings.php"

tests: Remove verbose logging settings from DevelopmentSettings.php

Follows-up fdb7d36903. It seems these make unit tests on Travis CI
fail, per T187581. And when we briefly enabled DevelopmentSettings.php
in our Jenkins setup, the PHPUnit tests also failed.

Remove these from DevelopmentSettings.php to confirm that the
tests pass that way. Then, once our own Jenkins setup has switched
back to using DevelopmentSettings.php, we can re-consider adding
these in a way that is verified by Gerrit/Jenkins before merging.

Bug: T187581
Change-Id: Ib81c5909849598bdad02955f6414d68c038742c5

jquery.byteLimit: Handle characters outside BMP (surrogate pairs) when trimming

Bug: T186364
Change-Id: I6282d97bcd637ae8e86d70996adb468582c8f02f

Add support for 'es-formal'

Spanish formal address. First exported on Id132ebc6.

Bug: T187750
Change-Id: Ia97b750bde01ceee7db0ea826e4f87bb9a2f0a93

Localisation updates from https://translatewiki.net.

Change-Id: I11bbae1702de3352fd8b59efa3abe3ad47b49acc

Merge "Update links in comments to developer.mozilla.org"

Merge "build: Updating mediawiki/mediawiki-codesniffer to 16.0.0"

Switch the sidebar cache to using checkKeys

The avoids the long delete() loop in MessageCache::replace()
and has better separation of concern.

Change-Id: I0acb0119058fa92fcafb52a5850f5dad4aaa94d2

Cleanup IDEA warnings in MessageCache

Change-Id: I0ced7c7289918f78bb6f3682ae974fe0c04a1e04

Update links in comments to developer.mozilla.org

Change-Id: I2310cf73ab494375655067833cea11b18483a396

Localisation updates from https://translatewiki.net.

Change-Id: I613493a91cd01032529bd161947b9231819b99cb

rdbms: tweak var names in LoadMonitor::scaleLoads()/getLagTimes()

Also mark the methods as "final" due to their delegation

Change-Id: Ie32f53445f749d9a00e77fce43d75830e260c039

Localisation updates from https://translatewiki.net.

Change-Id: Idc8c6ef5a94c61cac176895e089f9624428dc2af

build: Updating mediawiki/mediawiki-codesniffer to 16.0.0

Change-Id: I59b59f79bbf3ce4feff3b3a20c1c31bc16370531

Merge "rdbms: make DatabaseMysql::masterPosWait() handle inactive GTIDs"

rdbms: remove "m" member prefix from various classes

Change-Id: Iade8e8f70bb1307b96683d979d7e3650f4107515

Merge "rdbms: remove "m" member prefix from DatabaseMssql"

Merge "objectcache: add IExpiringStore::TTL_SECOND constant"

Merge "Soft deprecate DeferredStringifier class"

Soft deprecate DeferredStringifier class

As of 3041b5c, the DeferredStringifier class is no longer used in MediaWiki core or any Gerrit-hosted extensions.

Bug: T187290
Change-Id: Ic363daf0847c0371a6b4c4041f9ea394377185f3

Merge "RCLFilters: make sticky filters default and current value equal"

objectcache: add IExpiringStore::TTL_SECOND constant

Change-Id: Iec59ce72f3dbb0de68c8f6704d86f1522f86827e

Merge "site_stats: Unbreak counting newly created pages"

sql.php: Provide --json output mode

stdObject is a pretty useless format unless you're working with PHP

Change-Id: I7549347a630768223fba5b282a930361dfe6f2d3

site_stats: Unbreak counting newly created pages

Fixes a bug introduced on 331c9f9e2e that stopped newly created pages
from incrementing the counter of ss_total_pages in site_stats. The array
accepted by SiteStatsUpdate::factory is not meant to take a value with
the key 'total'. What's meant here is 'pages'. Let's change the variable
name as well to avoid confusion in the future.

Change-Id: I9c82d926206d9b46f333c6f6cb9d3c7d41db3a53

Localisation updates from https://translatewiki.net.

Change-Id: Ib373192166d7f9d43bbd15df341f8ad770353685

rdbms: make DatabaseMysql::masterPosWait() handle inactive GTIDs

Change-Id: I543deef24f6cbf99094a4f3bee7cabe768fa221a

rdbms: remove "m" member prefix from DatabaseMssql

Change-Id: I270cd4f4048e0ae751ea1c078d31d45c9db59690

Merge "Follow-up I0bb4ed7f7: Use correct 'this'"

Follow-up I0bb4ed7f7: Use correct 'this'

Bug: T187523
Change-Id: I7ced05b239d3ce0da1bcd6acdb3a30f65872645c

Don't write exceptions to STDERR from BadTitleErrorTest or ThrottledErrorTest

It's annoying and pointless. Instead, have MWException write them to
standard output where we can catch them with ob_start().

Bug: T170028
Bug: T170029
Change-Id: Icd99c1c39d4a30d78c511d33948ef639e1b92455

Merge "rdbms: do not bother making DBO_TRX transactions in IDatabase::lock()"

Update wikimedia/php-session-serializer from 1.0.4 to 1.0.5

https://github.com/wikimedia/php-session-serializer/releases/tag/v1.0.5
https://github.com/wikimedia/php-session-serializer/compare/v1.0.4...v1.0.5

Change-Id: Ib9f301f8f86031faed698beca65a56407859a1aa
Depends-On: I0f85f206fdd5566641784037132d2a6046de2c2d

rdbms: do not bother making DBO_TRX transactions in IDatabase::lock()

Named locks are session-level constructs and this transaction agnostic.
Also make lockIsFree() a bit more consistent when the thread has the
lock itself.

Change-Id: Ief51196161bbc50c798740f3c738fd0e39880508

Merge "rdbms: remove "m" prefix from Database fields"

Merge "rdbms: make MySQLMasterPos handle inactive GTIDs"

Merge "tests: Commit initial version of DevelopmentSettings.php"

rdbms: remove "m" prefix from Database fields

Done using the PhpStorm refactor->rename tool.

Also move "defaultBigSelects" declaration to DatabaseMysqlBase
as no other classes uses that.

Change-Id: I424a2d9815de3a5d4cca2522f3db23a5efe6b592

Merge "rdbms: improve ILoadBalancer comments about reuseConnection()"

Merge "rdbms: remove leading table name space hack in Database"

Merge "Salvage site_stats row with negative values in miser mode"

Merge "rdbms: remove "m" prefix from LoadBalancer fields"

rdbms: make MySQLMasterPos handle inactive GTIDs

Change-Id: I328142a9fe10bccb2717a0a392e0c50d0c070390

rdbms: remove leading table name space hack in Database

This was added in r19879 (3de6fd8c8) with no provided use case.
There also appears to be nothing exploiting this.

Change-Id: I1641d4ed16d612ab6f08ee1eeca1580af73888bf

rdbms: remove "m" prefix from LoadBalancer fields

Change-Id: I68fc74859bd58ef0c7591beef1c21411198d45e9

Use `lock(,,0)` instead of `lockIsFree && lock(,,1)` for non-blocking locks

This pattern is already used elsewhere and seems like a more efficient
way to acquire locks in a non-blocking way.

Change-Id: Idb369e7cb03b793d5f8295e956fecd8d1f849e17

rdbms: improve ILoadBalancer comments about reuseConnection()

Change-Id: I94f8e95f5ed9c3357d118e9bc66c55b389061e7f

Merge "Drop the 'CategorizedRecentChanges' feature"

Localisation updates from https://translatewiki.net.

Change-Id: I820327da7a4aa07879b4fa23f8187783bf35befa

Merge "objectcache: improve WANObjectCache reap method comments"

objectcache: improve WANObjectCache reap method comments

Change-Id: Icec0846b0e66229af39bb819d75144734fd111bf

Merge "Be more db-friendly when purging expired userrights"

Merge "build: Add grunt target for 'karma:firefox'"

Drop the 'CategorizedRecentChanges' feature

It is off by default, it's very rarely used (GitHub finds four config
users setting it true), it has over time had fatals going unnoticed or
fixed for years. Let's just kill it.

Change-Id: Idac2341ed7ecf053ee84069df657e424e50b896c

RevisionStore logging use wfBacktrace instead of wfDebugBacktrace

wfDebugBacktrace doesn't work very will with logstash.
The whole message ends up being too long and the log ends
up in the 'jsonTruncated' channel.
Instead use wfBacktrace which will be smaller as it just gives
us a string.

Change-Id: I5309ac2ad4592fb6dbb1474526d382893fb7e6c1

Merge "EditPage::getBaseRevision can return null. (fix phpdoc)"

EditPage::getBaseRevision can return null. (fix phpdoc)

If !$this->mBaseRevision then the code to populte $this->mBaseRevision
is run.
This code either calls Revision::newFromId or Revision::loadFromTimestamp
both of which are documented as being able to return null.
As a result EditPage::getBaseRevision can alos return null.

Bug: T187378
Change-Id: I60ad9ddcfbe6e1060cab1ad6aa2194c1a3406cbf

Merge "Hard-deprecate ParserOutput stateful transform methods"

Merge "resourceloader: Fix comment on Mobile Safari requirement"

resourceloader: Fix comment on Mobile Safari requirement

Mobile Safari 5.1 doesn't support Function#bind.

Change-Id: Ifdbd593352ab6d682a37010b17a13370f3c96176

resourceloader: Remove ResourceLoaderModule::getPosition

@deprecated since 1.29

Bug: T184257
Change-Id: I1a59a1f0a5fbedfea9f0b28bee8c298710557a8d