Merge "Various fixes for RecentChange"

Merge "remove specific FORCE INDEX clauses causing trouble"

remove specific FORCE INDEX clauses causing trouble

bug: 45619
Change-Id: I6b8f35bd8d709ccc8e4811836e7731bcedb5520c

Remove no longer needed code for PHP 5.2 compat

Change-Id: I9c1f5f12f51274c443084da55447d12b828a9fb0

Various fixes for RecentChange

* Do not return the value of a function that returns void
* Update documentation

Change-Id: I2b25bf34f578be516a19c84d45860ab6c7f93e0e

Merge "Adding hlist module to mediawiki"

Merge "Remove unused classes Title[List]Dependency"

Merge "Update formatting"

Merge "Address CodeSniffer errors and warnings"

Update formatting

Change-Id: I7586c4d50f7332c515b8377749c4250cc9e04d83

Merge "Add visibility to Status functions and variables"

Add visibility to Status functions and variables

Change-Id: I829a7503693318586e33605d2ac8e2ca63f3c886

Merge "Move RecentChangeTest to subdirectory"

Address CodeSniffer errors and warnings

Change-Id: Ie119db7d6d9e9eb3c1cd9c07b8f84fd155847c62

Merge "Fix indentation of comment in tables.sql"

Merge "Put status handling in EditPage into private function"

Merge "mediawiki.ui: Add right padding to .agora-field-styling()"

Put status handling in EditPage into private function

This bit of code is isolated enough to be separated from
attemptSave and be put into a private function.

Change-Id: I84c532358ee32fec959039c62798e4de0702f14a

Adding hlist module to mediawiki

Bug: 40062
Change-Id: Ie4ac182ce53317654b4cc17b25cfa34596054df7

Merge "Remove unused global in ForeignAPIRepo"

Remove unused global in ForeignAPIRepo

Follow-Up: I6849107a3af773687a31155b23a2d7b4d7a60645
Change-Id: I45d405a4dca9b7928f740fcd231e95c599310689

Merge "Localisation updates from translatewiki.net."

Remove unused classes Title[List]Dependency

Classes TitleListDependency and TitleDependency that are currently not
used. Searched core and extensions in Gerrit.

Change-Id: I51404ce36c4ef55393588817d1673fd17c81b4a2

Localisation updates from translatewiki.net.

Change-Id: I3615ad83da760c512799749618a35cb2e366ba41

Merge "Format email addresses more consistently"

Format email addresses more consistently

Change-Id: I1dbced804ff1e0bf8e162570c17cc537d318872f

Update LinkCache::addGoodLinkObj param defaults to int

Update the defaults for $revision and $model from bool to int, making it
consistent with the documented input. Also update docs and line length.
Change "inval( $x )" to preferred "(int) $x" while changing the method.

Change-Id: Ic19a408aa7c50fb03e2c3aca8df3fa7cedc2420b

Merge "Add WikiFilePage::getForeignCategories() method"

Merge "Tweaked WikiPage::clear() comment a bit"

Add WikiFilePage::getForeignCategories() method

Unlike getCategories(), this will return the categories
on the wiki where the file is hosted (so it should work with Commons).

Bug: 56598
Change-Id: Ice972f023c676b5857707ccf58ee108585a7f021

Merge "Enable module storage for 0.05% of visitors w/storage-capable browsers"

Tweaked WikiPage::clear() comment a bit

Change-Id: I9ec27b575de5547d1ab0569c4c2cdd1307f8307a

Merge "Declare visibility on class properties of LCStore* classes"

Merge "Declare visibility on class properties of LocalisationCache"

Merge "Remove underscore from classes LCStore_*"

Merge "Set visibility on class properties for *Dependency classes"

Merge "Declare visibility for class properties of LocalisationCacheBulkLoad"

Merge "Remove underscore from class LocalisationCache_BulkLoad"

Merge "Break long lines in includes/cache/"

Merge "Update formatting"

Merge "Remove unused local variables"

Fix indentation of comment in tables.sql

Change-Id: Id879b4438f8f004549c090567ff76b64c5072824

Merge "Set visibility for class properties of DependencyWrapper"

Merge "Address errors and warnings in CodeSniffer in api/"

Merge "Update indentation"

Merge "Address CodeSniffer comments in ApiBase.php"

Merge "Allow null to be passed to User::setNewPassword()"

Added fail-over support to JobQueueAggregatorRedis using "redisServers" option

* Also cleaned up various bits of documentation errors and cruft

Change-Id: Icccae5329941e4311fef02b6137a6b90f39f9046

Move RecentChangeTest to subdirectory

matches location of class within includes

Change-Id: I78d3212032ded3deaeef93aaca4d575a35559990

Allow null to be passed to User::setNewPassword()

This allows the temporary password to be cleared. This methods now
behaves the same as User::setInternalPassword() with that value.

Bug: 57075
Change-Id: I2c446864c7eb591dcb64e5971bc9989950715d15

Merge "Add show=unpatrolled to the recentchanges API"

Merge "Relicense jquery.placeholder.js to MIT"

Localisation updates from translatewiki.net.

Change-Id: I5a74c20de10a3203ff81d453de6c739a8a476081

Add show=unpatrolled to the recentchanges API

Adding it only in the API because SpecialRecentChanges only shows
"Hide patrolled edits" option if rcpatrol is enabled.

show=!patrolled returns results that include changes that can't
be patrolled which technically correct (they aren't correct)
but probably not what you're looking for when looking for
unpatrolled changes.

Change-Id: I356a8625c7126b90aa7e7a23efe3bef7d448b502

Set visibility on class properties for *Dependency classes

Set visibility to private for properties that are not referenced outside
of the class itself, and when the class is not currently being subclassed.

Change-Id: I427c6de5a0a29b43cff755db0eb8a750db620173

Set visibility for class properties of DependencyWrapper

Set to private. No subclassing and no uses outside of the class in core or
extensions.

Change-Id: If195d02b8ce853242312de7fb4bc7dc1cbe94365

Declare visibility on class properties of LCStore* classes

No direct references found in core or extensions outside of the classes
themselves. None of these are subclassed, so set all of them to private.

Change-Id: Id599c0830c1e6d3679c7ae55147f4444f8252987

Add/update/remove incomplete/incorrect documentation

Change-Id: I3c2c7f61d9a579ad794ce7b564ccc8a7cfb9b6fc

Declare visibility on class properties of LocalisationCache

Mark all private, except the one that's being used in subclass. Checked
core and extension for direct uses. None found.

Change-Id: If73d65ca094ff56deb19d3b6f3ef99892654726d

Update docs to reduce inconsistent type compatibility in api/

Change-Id: Iaf97aeb514ddb2176491c3820fe891be8e989939

Merge "Add ar_id and el_id sequences for PostgreSQL"

Remove underscore from classes LCStore_*

iPart of program to remove underscores from class names. Checked core and
600+ extensions for occurrences. All uses are in core, and are updated in
this change.

Change-Id: I432dc249d22053728013ae7d0d56c3c398021c5e

Declare visibility for class properties of LocalisationCacheBulkLoad

Mark all as private. Not referenced elsewhere in core or extensions.

Change-Id: Ibe0e854f39261053e5147138309c3a4b1944e5fe

Remove underscore from class LocalisationCache_BulkLoad

Part of program to remove underscores from class names. Checked core and
600+ extensions for occurrences. All uses are in core, and are updated in
this change.

Change-Id: Ib157c3cf209677718ff184552cf872b1d3f4e969

Break long lines in includes/cache/

Change-Id: I23a6f46f89b3cdcaf2a3b982dc63e779eed5840d

Update formatting

Change-Id: I9aa56cbe0989d6f302ee0f47c4a6154101f73e90

Update some documentation in CacheTime and ParserCache

Change-Id: I31be0d4639f2a7189e15b622b34595564fad2a3d

Update indentation in ApiUserrights::execute()

The previous formatting was inconsistent.

Change-Id: I22d7cba00cd39df7069b47f78ca130e3e031ceb4

Simplify ApiUserRights::getUrUser()

Remove unneeded else and local variable assignment.

Change-Id: I166d811b220e925be11da5c49b6fe55f0d8e71c8

Update documentation for ApiDelete::deleteFile()

WikiPage was no longer valid given the current type hint. Added
types and documentation where missing.

Change-Id: Ie0a3a8fff3ad496b723958133867df48a695d3b7

Merge "Add function to clear mPreparedEdit, to use in cases of mutable content"

Add function to clear mPreparedEdit, to use in cases of mutable content

In Wikibase, EntityContent is mutable and needs to clear
prepared edit before performing a save.

Follow-up to I2c34baa

Change-Id: I770bf2dc960c0a67a9b8315b7cdf24f3b89b6480

Merge "Removed excess profile calls"

Fix inconsistent line break

Change-Id: I42098f3bdbf470bd9de1d29ad64005dfd6f7e134

Remove unused local variables

Change-Id: I4162d45e68d65347108f94b3d80bc146f3393c1b

Address errors and warnings in CodeSniffer in api/

Change-Id: I06de371393d50eada33154626874b106d814642b

Update indentation

Change-Id: Id0ff1d21362aa8984529ecde39304c7aedf589c1

Address CodeSniffer comments in ApiBase.php

Change-Id: Ifd8567dcccf51a16f0dc7459a3bbe9e21e928d91

Removed excess profile calls

Change-Id: Idd57e8d7fff56c62a66c7fcaf4627b9096f0c363

Changed use of tag_summary to use change_tag with GROUP_CONCAT()

* Added buildGroupConcatField() method to the DB classes

bug: 53577
Change-Id: I976f297653880e66af429ba9b622a954fefcc512

Merge "Fix incorrect parameter order for dieUsage()"

Merge "Add documentation for DerivativeRequest::__construct"

Merge "Remove useless method overriding"

Set title to Special:Revisiondelete on action=revisiondelete

Bug: 48616
Change-Id: Idb85c797dbe349e956d21b5cde6a59d89dab4fd2

Merge "Call setRelevantTitle during revision delete"

Merge "Show correct link color on cached result of Special:DeadendPages"

Merge "ContentHandler: TextContent::diff should compare to given object, not itself"

Call setRelevantTitle during revision delete

This will make the header tabs be for the page being deleted,
like it is for Special:MovePage and friends.

Change-Id: I52d848ef0763f6691cec9192794e8e7e9d933ed2

ContentHandler: TextContent::diff should compare to given object, not itself

Not sure this is actually used anywhere which might be why this has gone unnoticed.

Also fix the method's docs.

Change-Id: Ifa7a684fe98774cae546b385b09a950cfa7eff32

Show correct link color on cached result of Special:DeadendPages

Reaching this by doing a LinkBatch and than using
Linker::link instead of Linker::linkKnown.
This adds also the stub color to the link.

Bug: 56912
Change-Id: I516f95302546ccdf8d0fdc7101c2ff72daa5f2fb

Fix incorrect parameter order for dieUsage()

Change-Id: Ib163c40d728bacbd16e7c4beaae0c80e79855686

Add documentation for DerivativeRequest::__construct

Change-Id: I20e62d07b6a069b3c2b5b0cb67736ded294bcfb2

Merge "Logic optimization for wfExpandUrl()"

Merge "Remove unused exception in SpecialPage::getTitleFor and add tests"

Logic optimization for wfExpandUrl()

removed redundant checks of the same variable, making the code
slightly more efficient

Change-Id: Ice4d3c45e80ca1214e2c36444baf0ce87b15a59b

Remove unused exception in SpecialPage::getTitleFor and add tests

The way the existing code was, it seems the exception condition
was never hit, as one could enter a bogus special page title in
SpecialPage::getTitleFor and it returns a Title object anyway.

Some tests are also added for SpecialPage::getTitleFor(),
although more refactoring my be required to truly unit test this,
vs. indirectly testing other stuff via static methods.

Change-Id: I6e427d047a2f8e58677eb65b50866f767078c255

Remove useless method overriding

Parent is:
public function __construct( $mainModule, $moduleName, $modulePrefix = '' )

Spotted by CodeSniffer

Change-Id: I71fdcc59584562e5287c43be337934a0ad4975b4

LinkSearch: use default protocol if unsupported

Default $protocol to http:// if part of link up to
colon or double-slash isn't in $wgUrlProtocols.

Also simplify elseif condition above - $protocol is
always blank there since neither if block above it
is executed.

Bug: 33029
Change-Id: I66d2c592998c750fe3da937b228fe1387fcd344c

Merge "Add continue editing link also to js and css pages"

Merge "Avoid empty batch in ListredirectsPage::preprocessResults"