Parser::replaceInternalLinks: fix batching

The parser unnecessarily made individual checks for existence of
pages that were neither in LinkCache nor linked only with a fragment.
A Title::isKnown() call in Parser::replaceInternalLinks2() (added in
bca8b8ad7d2f) caused this.

Title::isKnown() was used to avoid treating a link to a distinct page
as a self-link even when the title happened to match one of the variants
returned by Language::autoConvertToAllVariants(). This change fixes
the bug by moving the problematic portion of the self-link check into
LinkHolderArray::doVariants().

Change-Id: I586e11e8b47308980ea04087ebc4246c397a8f53

Use restriction-level-* messages in Linker::formatTemplates

The current implementation of Linker::formatTemplates only checks for
"sysop" and "autoconfirmed", and completely ignores any other protection
levels. That needs fixing.

Rather than introducing additional messages "template-protected-*" to
match template-protected and template-semiprotected, let's just use the
existing restriction-level-foo messages and wrap them in parentheses.

The old messages are kept for backwards compatability, but could
probably be considered deprecated.

Bug: 43462
Change-Id: I0c507f4ecc1921f599acbda834fa55e96388fa7b

Avoid master queries in Revision::newFromTitle(..., Revision::READ_NORMAL)

The fallback to master makes sense in Revision::newFromId(), since a
revision ID known to exist by some other means might be present in the
master but not in the slave. But it doesn't make sense for
Revision::newFromTitle(), where by far the most common cause of this
case being hit is the page being nonexistent.

Change-Id: I41189fcde1b8065ac134982241376d63b78732a1

Merge "Fix Redis increment behavior by using BagOStuff->incr instead"

Merge "Test BagOStuff->incr method"

Fix Redis increment behavior by using BagOStuff->incr instead

Due to the PHP serialization, the Redis native INCR and INCRBY do not
work.  This instead falls back on the parent class's incr, which uses
lock and unlock, and incrementing on the PHP side.

Bug: 55986
Change-Id: I500199aeb935963ca118fde163bd7ba0cba8f79f

Test BagOStuff->incr method

Change-Id: I3b72f19df82ee302dee47dcf22b69ed9bb6ff8e0

lockmanager: QuorumLockManager subclasses can get EX/SH locks at once

* Also reduced rount trips in doUnlockingRequestBucket().
* Also removed some redundant doc comments.

Change-Id: I81878e92332509bd7fda9ddeef950b774f5b015d

Cleanup /languages/* tests

This change:
 - Adds method scope
 - adds @covers tags
 - adds various @todos
 - fixes some comments

Before the changes tests ran with:
   1383 tests, 1412 assertions 10 skips
After changes the results remain the same

Change-Id: Iee57447bdb47026952ef5dcce6fed5dad0f80e52

Add more @covers tags and test cleanup

Other cleanup includes
 - Adding method scopes
 - Fixing php comments
 - Adding todos

Change-Id: I0a231008e6a59110ffcab6af1bd8c4d3ee13f21d

Merge "Mention the new HTMLForm 'vform' display format."

Merge "Fixes for duplicateStderr (I833aeb3a)"

Mention the new HTMLForm 'vform' display format.

Change Id03d185bbee990595bfc469a61163cc598fc3441 added the feature.

Change-Id: I17ccb127e47047dc32f930ce1775496ea22410d2

Localisation updates from translatewiki.net.

Change-Id: I0539a27c72b9a270db4e3b29a4c357e40d806353

Merge "mediawiki.inspect: add CSS report"

Merge "Added a few more trx sanity checks to DatabaseBase"

Merge "Have list items occupy their own line"

Localisation updates from translatewiki.net.

Change-Id: I922e660fafc7a952129beebeaed001712f5c2bc1

Merge "Clean up old title on move before reset article id"

Merge "jquery.localize: Fix incorrect use of void tag for <html:msg>"

Merge "Fix typo erronously -> erroneously"

Merge "Add 'RedisPubSubFeedEngine' feed engine"

Fix typo erronously -> erroneously

Follows-up 1aed5382d2.

Change-Id: I1e0cbc3dfa529816446c3aec6372eb2b821ee540

jquery.localize: Fix incorrect use of void tag for <html:msg>

Follows-up 55dac1f which errounously changed a few <html:msg />
to <html:msg> while trying to fix up <span/></span>.

Bug: 40598
Change-Id: I987fa288cc1b97a8a5fb367aae2626d7d0b48238

Add 'RedisPubSubFeedEngine' feed engine

This patch adds a class which implements the RCFeedEngine interface by
publishing recent change notifications to Redis. The class handles the
'redis://' URI scheme. Recent changes are PUBLISHed to the channel 'rc'; a
different channel name may be specified as a path component.

Change-Id: I846036c091c45059a8947245a1efe92c9800dcf4

Added ceph comment

Change-Id: Ice9efd7ada10e1896afd0185a6305f6852e9605d

Added a few more trx sanity checks to DatabaseBase

* Also track the caller that added the callbacks

Change-Id: Idc4d9e4a89fcee28bd4b18ad6d8c6b09b9fe7e75

Declare visibility on class properties

As pointed out by phpcs:

 The var keyword must not be used to declare a property
 (PSR2.Classes.PropertyDeclaration.VarUsed)

Set to public, as this is also done for class properties in classes these
ones extend.

Also break some long lines to silence CodeSniffer.

Change-Id: I77dd45eb2bf1d405d429603b97d5cbfb7ea04ad0

Add @covers tags for more tests

Change-Id: Iff3af78e9b41c445b7f066b6c0d0f4a87d2d6c4e

Fixes for duplicateStderr (I833aeb3a)

* Separate code from data. $2 in limit.sh is an option list.
* Don't duplicate stderr on linux when limit.sh is not used, unless
  duplicateStderr is specified

Change-Id: I8e6e74810864830c12af627bfbe75e74ba34e3e3

Merge "Simplify boolean attribute handling for Html::input in templates"

Merge "Make it possible for EditPage::getCopyrightWarning to return HTML"

Merge "Remove proxy check in EditPage.php"

Remove proxy check in EditPage.php

$wgBlockOpenProxies, $wgProxyPorts, $wgProxyScriptPath, and
$wgProxyMemcExpiry have been removed, along with the open proxy
scanner script they were added for.

Bug: 54597
Change-Id: Id4c7521443f04995df3d8762d23629c18ada76f8

Merge "Remove 'Debug' hook from wfDebug and wfDebugLog"

mediawiki.inspect: add CSS report

This patch extends mediawiki#inspect to add a notion of 'reports'. Instead of
there being one single view for all ResourceLoader module debug data, you can
now request that a specific report be run. Each report highlights a set of
metrics that are unified by some common theme.

The previous, unqualified report is now branded the 'size' report. In addition
to it, this patch also adds a CSS report, which prints the count of selectors
and the count of selectors that match against the current time

Change-Id: I0e7a47b5a93a66ecb103fe30a3de8a2ca8a4eb84

BaseTemplate: Exclude two more attributes in makeLink()

'context' and 'primary' are used internally by some SkinTemplate
functions and Vector. They should not be passed on to HTML.

Bug: 55659
Bug: 55660
Change-Id: I1f7a656aacc93f9bf289c481f258824ae28ccf4e

Merge "Localisation updates from translatewiki.net."

Merge "Don't match HTML entities in language conversion syntax"

Simplify boolean attribute handling for Html::input in templates

The Html class supports this handy shorthand.

Change-Id: Ie34790fa650c7ecad02c2d91130bf0db9172ea3f

Localisation updates from translatewiki.net.

Change-Id: Ifc418e11643e44130c7d5d563aaae870222ba536

Clean up old title on move before reset article id

Calling WikiPage::onArticleDelete without a page_id will reload the
page id, maybe from a (lagged) slave, than the title has the page id
set, and a call to Title::exists return true, which shows the link in
blue and the wrong message.

Bug: 45348
Change-Id: If415dc1729b0bb1cfe7fc75f9e29638a6797d478

Make it possible for EditPage::getCopyrightWarning to return HTML

Adds a $format option to let it return various output formats
so code that needs HTML doesn't have to do hacky stuff.

Change-Id: Ie4ff9738773399f9f00994c6cc587703ce99a681

Have list items occupy their own line

Ensures that HTML list items are on their own line, which fixes a
CSS issue with .hlist; it depends on a breakable character between
list items, and a newline does provide one. Should also unburden
HTML Tidy, as output now matched that of Tidy.

Bug: 39617
Change-Id: I82fb4d749a200cfc049b30d2ee6e96a5ff7574e2

Merge "FormOptions: Document getValueReal(), consistency fixes"

Merge "Split changes list classes into separate files"

Split changes list classes into separate files

Having them separate improves readability of the code
and makes it easier to work with this code.

Change-Id: Ic6aaf1ace3640f66ad41c2d85b4f7d740b259e0c

Merge "Tweak bidi on Special:Contributions/newbies"

mediawiki.notification: Expose close method to hide a Notification

Bug: 51338
Change-Id: Icd7fd1ac328094ddc2560d4804dd52fc85879867

Merge "Added more Message parameter functions"

Added more Message parameter functions

Similar to numParams(), added functions for other parameters
that can be formatted by the Language class. Adds functions for
expiry, size, timePeriod, duration, and bitrate parameters so
that the formatting doesn't have to be done at the caller.

Change-Id: I7b435fcc11824ead55e4c0f5512418187eae9a6f

Merge "Rename Toolbox to Tools"

Merge "Correct release notes for I9dd9f0fd499c37f2"

Correct release notes for I9dd9f0fd499c37f2

Change-Id: I8f6e84b958d1bdadb578de655c4306f6b9d1c873

Merge "deleteEqualMessages: Exclude messages that are empty by default"

filebackend: try to combine SH and EX lock acquisition

* getPathsToLockForOpsInternal() now returns an array in the format LockManager can use
* Also fixed getScopedLocksForOps() for multiwrite backend (it should accept proxy backend paths)
* Updated getScopedFileLocks() docs

Change-Id: Id0dc50c159c5fbc8fca10a9e46c5be23afcb3e9a

Small JobQueueFederated performance tweaks

* Made "find non-empty queues" and isEmpty() checks start with the most
  weighted partitions first
* Cleaned up a few exception catches to use JobQueueError
* Added short-circuit for doGetSiblingQueuesWithJobs()

Change-Id: I4d96a8c61c6b1c41dbb44000ccda0fa05a418a09

Fixed a bunch of annoying deprecation notices

Change-Id: Ic2eb0ea94d8e27574f7dc76f1b448ccd6c01bb98

Rename Toolbox to Tools

The name "Toolbox" is strangely anachronistic. Other functions in the
sidebar such as languages and navigation use very simple, functional
names.

Most translations already use a single word with meaning equivalent to
simply "Tools"; several use literal translations of "Box of tools",
which tends to look rather awkward.

Bug: 54910
Change-Id: I9ba287ec6426632847fe7d3a0d71948838496317

Reduce excess lock contention and chance for failure on rename

* DB locks and transactions were getting held up by sometimes slow purges (10+ seconds)

Change-Id: I0eead60e8faf5aee6b2ddf59cfdd928f9c8083ff

Localisation updates from translatewiki.net.

Change-Id: I239f39d2a75aabffa1586776bcb8fb3c447c5df4

Merge "Split change_tag sql patch"

Merge "Add @covers tags for /content files"

Merge "Add @covers for GenderCache + MessageCache tests"

Merge "Add @covers for main Database test types"

Merge "importImages.php: Correctly generate automatic summary"

Merge "Removed unused const QoS_Atomic constant (odd casing)"

importImages.php: Correctly generate automatic summary

If the summary parameter is not set, then use the page text of the
given page, NOT that of the first image uploaded.

Bug: 55885
Change-Id: I00ced9d2ce62fc1a00f26ff513866359cf96d535

Don't blindly overwrite $options in importImages.php

Bug: 55883
Change-Id: I695e080000f6abec3ded966f12af3e25c585c3ba

Add @covers for GenderCache + MessageCache tests

Change-Id: I379f2788c1b308354a3dbb73c889aa4a90b2cd20

Add @covers tags for /content files

Change-Id: Ied40e5afb1027f7d8ff46ffbf722ff6558eed880

Add @covers for main Database test types

Change-Id: I010ff5ce7afd85c7f9d528a98f2448547874b8b3

Tweak bidi on Special:Contributions/newbies

Special:Contributions/newbies when page title is not RTL will mixed
username and page title.

Change-Id: Icccf953cd2b6784f83fdd6a5775b6af8b1216ed5

Removed unused const QoS_Atomic constant (odd casing)

Change-Id: I2b03ad1700e3a1184f7209394096cde61d23c6a5

jobqueue: various cleanups to JobQueueFederated

* The relevant partition information is now stored in
  the fields instead of the general information for all wikis.
* Also improved hash ring caching and made getCoalesceLocationInternal()
  more robust.
* Improved availability of sibling queue methods.

Change-Id: I7ac268bb1db4615e0f735576c506593d5688e17e

Try to make sure jobs will not get pushed twice on failures

* Some queues like the DB one, might split up batches into chunks
  which we do not want when used by federated queues (it already
  manages chunking anyway). If only X of Y got inserted, then they
  would all get inserted into another partition, causing duplicates

Change-Id: Id73b34f3500f8aeaf456e6bcac862a1e4832c870

Merge "Fix SQLite exception when $wgDBuser is set"

Remove 'Debug' hook from wfDebug and wfDebugLog

Per the bug report, the 'Debug' hook was triggering an infinite loop when
wgDebugFunctionEntry is enabled.  The Debug hook is used if an extension wants
to stop a debug message from being sent out.  Ideally the wfDebug and related
functions should be as low-level and avoid calling other code as much as
possible to avoid situations like this.

Bug: 55818
Change-Id: I679782489b683503fc624cfea3c7ad72a989b005

Fix SQLite exception when $wgDBuser is set

Fixed from several different directions:
* The error reporting was screwed up, so I wrapped the PDOException
  in a DBUnexpectedError.
* Make $db->open() trigger a close/open sequence if called a second
  time, like MySQL, instead of breaking mTrxLevel.
* Don't call $this->open() twice from the constructor, even if $user is
  set.

The bug dates to r84485. The nasty hack allowing you to construct a
do-nothing database object with "new Database()" dates back to 2004 and
has probably outlived its usefulness -- noted this. It was formerly used
by LoadBalancer::reportConnectionError().

Bug: 49254
Change-Id: I571f9209bec8e8ed5058b6327e1738eb4315d5a0

Add $wgEnableParserLimitReporting to control whether the NewPP limit
report is included as a HTML comment.

Bug: 26792
Change-Id: I2b2b3c9d76487168243b733c1154c4c26c7de3d6

Merge "mediawiki.api: add postWithToken and getToken"

mediawiki.api: add postWithToken and getToken

Similar to postWithEditToken in mw.api.edit, but more generic.

Change-Id: Ic2af66b91a996bc9b46dc3e999b4099541573b08

Merge "If $wgShowHostnames is true, add parsed by to NewPP limit report"

Localisation updates from translatewiki.net.

Change-Id: I3560103a9de1e613c5878422172e070878ec7d43

Add archive, externallinks PK

* New fields: ar_id, el_id. el_id is presently not used for
  anything, but will help with online schema changes.

Bug: 15441
Bug: 39675
Change-Id: Ib6b0fc3736d173fa4ba7b786ecfc710b2f4711bb

Fix mysqli support

Exception was thrown if $dbType=mysql, regardless of whether a driver
was available. Also, the installer support was broken if the mysql
driver was missing.

Change-Id: I29df3fc9b3bf5743c1365f757e0cf899d5b96ae9

Resubmit "Add support for mysqli extension"

This reverts commit 1d7e12179df0db332f9ac139e8b7d831e04a6a10.

Change-Id: Id1a3fc34e94006b880f58de83e5e7c70b21d26ed

Merge "Revert "Add new recentchanges field rc_source to replace rc_type""

Revert "Add new recentchanges field rc_source to replace rc_type"

No feature flag, and also not deployed on WMF wikis. Asking for schema changes by the time we branch tomorrow is just unfair

This reverts commit e478c230f6d3f1b7638d4443682eff0df7650def.

Change-Id: I125c30e76fa04679cc04ae7b8728ae3a61f35194

Vector: Fix font size regression

As Krinkle noted the font size of the body content ballooned with
Ica0b69ad95051e15cd8ed9888927e034c13fbf92 - this rectifies it.

Change-Id: I3a5a159cbb04a1a24277596ca304805a84d3b77f

Merge "Revert "Add support for mysqli extension""

Revert "Add support for mysqli extension"

Throws exception unconditionally, installer support is totally broken.

This reverts commit b2d64432ee02661ea29cb727686e10e7a4980dfb.

Change-Id: I2edb78440806754c42d48c5da2ff74b4cb40fa2d

Merge "Added a "syncviadelete" param to copyFileBackend script"

Merge "Add new recentchanges field rc_source to replace rc_type"

Added a "syncviadelete" param to copyFileBackend script

* Added a bit of tabbing to clarify the output

Change-Id: I4b0095595d742452b151e6ed6765a9dc805a55ae

Add support for mysqli extension

DatabaseBase::factory()'s parameter array now takes an optional
'driver' variable to allow specifying a specific driver for the
database class if we support more than one.

This is so we can gracefully support mysql/mysqli as the former is
deprecated, but the concept could be extended to other databases
as well.

Bug: 45288
Co-Author: Chad Horohoe <chadh@wikimedia.org>
Change-Id: I6733fe21c4aa7443e409c5dfa7c789552b2b62b7

Localisation updates from translatewiki.net.

Change-Id: Id8e5920d1a5f58fc904a812369df35fe2f8f448e

Merge "Keep returnto, returntoquery on Special:CreateAccount redirect"

Add new recentchanges field rc_source to replace rc_type

The existing field to differentiate between kinds of recentchanges rows is
the rc_type field. We want to allow extensions to insert their own custom
data into recentchanges, but we have learned via the NS_* series of constants
that requiring extensions to "register" a specific number is very error prone.

The solution, which this commit implements the first phase of, is to utilize
a new 16 byte string field rc_source.  Within that field change types will be
prefixed strings such as 'mw.edit' and 'mw.new'.

This commit adds the new field and begins populating it with data.  At some
point in the future the rc_type field will be dropped.  While WMF wiki's will
simply wait out the 30 day recentchanges history, other wiki's have the option
of letting update.php populate rc_source, or manually applying the db change and
utilizing the PopulateRecentChangeSource maintenance script.

Change-Id: Iaddd6c446373a68d31586ed54346db7d04e13b2c

Merge "Use max() for job count for sanity"