Fix rebuildall.php fatal error with PostgreSQL

The fix for 47055 introduced a fatal error when running rebuildall.php. This
is a workaround suggested by gebhkla on Bugzilla (idle for 10+ days). It just
checks to make sure $options is actually an array before calling array_search
on it.

Bug: 60094
Change-Id: Ib54420b5a749c60b82a4757a9f5fa511f48fdb72

Merge "SpecialWantedcategories: Actually strike the category if it was emptied"

Merge "Add Special:Diff as an internally-linkable redirect to diff pages"

Merge "+Test for Status->CleanParams with a callback"

Merge "Add test that covers Status::fatal"

Merge "Add test for Status->merge with overrideValue true"

Merge "Cleanup RefreshLinksPartitionTest"

Merge "Add release notes for removed RdfMetaData class"

Add Special:Diff as an internally-linkable redirect to diff pages

This is similar to Special:PermanentLink added in r79036 and has been
asked for several times in different places, including:
- on the English Wikipedia (oldid=539308532)
- on mediawiki.org (lqt_oldid=31691)
- on the French Wikipedia (oldid=93029892)
- on the English Wikipedia again (oldid=588408888)

A notable use-case is linking to diffs in the edit summaries, where
external links are not yet allowed (bug 14892).

All of the following are valid usages:
- [[Special:Diff/12345]] (diff of a revision with the previous one)
- [[Special:Diff/12345/prev]] (diff of a revision with the previous one as well)
- [[Special:Diff/12345/next]] (diff of a revision with the next one)
- [[Special:Diff/12345/cur]] (diff of a revision with the latest one of that page)
- [[Special:Diff/12345/98765]] (diff between arbitrary two revisions)

Co-authored-by: Jérémie Roquet <jroquet@arkanosis.net>
Co-authored-by: Bartosz Dziewoński <matma.rex@gmail.com>
Change-Id: I77fdaf8e04375caa1d67ca4a3ec3bd93920c3309

Add release notes for removed RdfMetaData class

Follow-Up: I3454d605a741bc97a51a1ad471a17eb3053d662f
Change-Id: Id1a04ea330da90ed84dae692bde7c64a5ac06cd5

Split Timestamp.php into class and exception files

Change-Id: I438adfe5479a1017baee8f2b663f3fb2e49c685a

+Test for Status->CleanParams with a callback

Change-Id: I4e7570ea041d28303388c630b3ca07225faf8551

Add test that covers Status::fatal

Change-Id: I7cb1e19569f77df5230808d7b35200b0ec9a6698

Add test for Status->merge with overrideValue true

Change-Id: I049c67e80277f8dbbbadae6e7a8f30d66066e7bf

Cleanup RefreshLinksPartitionTest

Change-Id: I9c8e66b7370f8ea8d9b4f8843718b29e992b1180

Merge "Improve organization of TestsAutoLoader"

Update jStorage

Contains bugfixes, new license and smaller file size: mw.loader.inspect
went from 16.6 to 11.9. This is still bigger than what they claim:
"jStorage is really small, just about 7kB when minified (4kB when gzipped)!"

Change-Id: I0b6f083a72a27af1e3fd367f61e8f98f74fea5d8

Add .gitignore to the /skins directory

To make it easier to work on 3rd party skins.

Bug: 55486
Change-Id: I986b6fe71bba429d1274bd86822448838601af64

Added some constants to speed up Setup.php

Change-Id: I528b362d6ef3d07c6c301e7a1767a1be0fc93725

Add support for "tabindex" in HTMLFormField subclasses

It is already set for some fields in Special:Block, but are
discarded by HTMLForm and its fields.

Some notes:
- fields with multiple inputs (radio, select and other, select
  or other) will have the same tabindex set on all elements
- Some items such as multi-select and check matrix are not yet
  implemented

Change-Id: I3e1ba7f16f3a3183f231afcf60dd392ce6d6eb6b

Add direct links to translation interface on Special:AllMessages

The local Special:AllMessages page is the first and most friendly place
where average users will look for the message and try to translate it:
when they find it, add a link to translatewiki.net so that they can start
translating without searching for it from scratch.

Special:SearchTranslations is the only target usable for this: the solr
search is smart enough to present the most relevant results first and the
translation interface is directly available in place.
On the bright side, similar messages popping up from other projects or areas
of the code will provide suggestions and help find other occurrences of
translations which can similarly be improved.

More specifically, we are using a combined language, key and text search:
* the English text is the only thing guaranteed to exist, if the message is
  available, though quoting it can give some false negatives and not quoting
  it would give it too much weight if it's long;
* the key can be used thanks to the feature revealed by Niklas Laxström in
  comments to this patch: it is simpler to handle but allows only exact matches
  and can be wrong if TWN adds a prefix for the extension, so it can't be used
  alone, but in the best case it's what we're looking for;
* the language makes sense in combination with the key match.
We boost key and language to get the current translation of the message where
possible; if it's not, the first result will usually be English text.
https://wiki.apache.org/solr/SolrQuerySyntax
https://lucene.apache.org/core/2_9_4/queryparsersyntax.html

From the initial search, the user can further refine its search and navigate
the interface to find the message(s) and editor where to add or edit the
translation.

Change-Id: I8de3faba829c50cdc0f2d09726f738c585792e00

i18n: Native digits on #wpLocalTime

Change-Id: Ief3c4700247809e59dda30b9b224e3f9dfaa5e62

Merge "Little change of readability in Title::secureAndSplit"

Merge "StatusTest is language dependant"

Merge "More use of Title::hasFragment"

Merge "Add hints to two function comments in Title.php"

Merge "Replace <i> and <b> by <em> and <strong> in i18n"

Add hints to two function comments in Title.php

Change-Id: I47627892c6450396cfb658ddef7617046fdf4209

Little change of readability in Title::secureAndSplit

Move init of mArticleId to bottom, where other fields are filled and
change ternary operator to a simple if for better readability.
Break a double assignment in two statements.

Change-Id: Ib61435e62e883142dd0c5dcb40e0e50e7ca18439

More use of Title::hasFragment

Missing two places for the new method.

Also change a remaining var access of Title::mFragment to accessor.

Follow-Up: I27d9c3e40e6de6800f4488de167cf06e83c88ce6
Change-Id: I5d26b9a7662673b902e3f7147cbc95ca0626d721

Merge "Ignore testRunLegacyHooks in findHooks.php"

Merge "Remove preference "Show table of contents (for pages with more ...)""

Remove preference "Show table of contents (for pages with more ...)"

The table of contents box is auto-inserted, can trivially be hidden or
exposed on a per-page basis with __MAGICWORDS__, includes a sticky
[show|hide] link, and can be easily hidden with site-wide CSS as
necessary. It needlessly adds complexity and user interface clutter.

Bug: 52813
Change-Id: If2139317dae4aa980b373c73d7b81dac627b5af8

Replace <i> and <b> by <em> and <strong> in i18n

Change-Id: If84e78e413fd042ef9451c68df2321fd52a35c79

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

Localisation updates from https://translatewiki.net.

Change-Id: Idf89d2bff84d9c4e74be06a2532222891b0fb30e

Merge "Change behavior of Special:Watchlist when user's watchlist is empty"

Merge "Fix comment on JobQueue::push"

Fix comment on JobQueue::push

Change-Id: I2a7e570ee29ca9dcab2e5c92616ecb69310ae7d5

Merge "User::saveOptions() optimization"

User::saveOptions() optimization

Since we only want to save non default user options, we have to strip
out any user option that match the default ones. We did that by calling
User::getDefaultOption( 'some option name' ); on each of the option.

Since the User mOptions property is a merge of the default option, we
end up doing a lot of unneeded processing. The loop roughly looks like:

 User::getDefaultOption()
   User::getDefaultOptions()
     Language->getCode()
     SearchEngine::searchableNamespaces()
       language->getNamespaces()
       wfRunHooks('SearcheableNamespaces')
     wfRunHooks('UserGetDefaultOptions')

For EACH of the mOptions.

Instead this patch does an array_diff to strip out from mObjects any
default option.  We still skip options whose value is false or null.

Test provided to make sure we only save what we want.

Change-Id: Ie98d3a17edab74401ed32f759ba11f723b56e376

Removed "Enable section editing via [edit] links" from preferences

Unwanted user preference option in MW adding to the clutter.
Users interested in hiding section-edit links can use per-user
(or site-wide) CSS.
Removed 'editsection' from Defaultsettings.php and
ResourceLoaderUserCSSPrefsModule.php
Updated Release Notes

Bug: 52811
Change-Id: I5fc49106621943ca7180ddb37590b624edac67d5

Add Title::hasFragment and use it

Makes checks against the fragment easier to read and all the same.
At the moment some using strval, some use type safe comparsion.

Change-Id: I27d9c3e40e6de6800f4488de167cf06e83c88ce6

Merge "API: Remove obsolete wgRightsCode from siteinfo"

API: Remove obsolete wgRightsCode from siteinfo

This was apparently a companion to $wgRightsPage, $wgRightsUrl,
$wgRightsText, and $wgRightsIcon, added in r3148.

But even then it was commented out as "not yet used", and appears to
have eventually disappeared without ever having been used.

Change-Id: I63f4393c22912f04e4ea690ebb1cd02bae65faab

Fix testValidyntax to testValidSyntax

Change-Id: I6157fec56a5455888200752d757602ddb640fd1e

Merge "AutoLoaderTest: Add support for class aliases"

Merge "Do not load special pages in WatchedItem::load"

Merge "Remove TODO for use of TRUNCATE and replace with comment"

Merge "ChangesListSpecialPage and subclasses: Reorder functions"

Merge "Changes list legend modules cleanup"

Merge "Create ChangesListSpecialPage as a base class for Watchlist and RC"

Merge "Don't pass $this by reference to SkinVectorStyleModules hook."

Merge "Replace a bunch of extension_loaded() checks with checkPHPExtension()"

Merge "Begin abstracting more of the database-specific search backend out"

Don't pass $this by reference to SkinVectorStyleModules hook.

Fix for Ica0b69ad95 (8b6ed4d0).

$this cannot be changed, so there is not any reason to pass
it by reference.

Change-Id: I916ccabbaff9f60edb344550111f62c833bb137b

Replace a bunch of extension_loaded() checks with checkPHPExtension()

Change-Id: I7bd9bc3a753975f45e63532d4f07aeff853b9970

Add $context to the ArticleFromTitle hook

Most of the time people wouldn't need this - this hook is mostly
used to inject a custom Article subclass for a specific
namespace to do fancy things, but lets give users the most
flexibility.

Asked for by leaves on irc.

Change-Id: I3386ff4abe0e5b3eb783604ae222b7703428db44

Fix git repository viewer URLs so they support more variants of the Wikimedia URLs

I've tested this against the URLs that are exposed within gitblit and gerrit, specifically:

* https://gerrit.wikimedia.org/r/p/mediawiki/extensions/AbuseFilter.git
* ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/AbuseFilter.git
* https://gerrit.wikimedia.org/r/mediawiki/extensions/AbuseFilter
* ssh://stwalkerster@gerrit.wikimedia.org:29418/mediawiki/extensions/AbuseFilter
* https://stwalkerster@gerrit.wikimedia.org/r/mediawiki/extensions/AbuseFilter

Change-Id: I36008eced60e32d554a5801bae155611f259f2d1

Create and move some functions for class ArrayUtils

Change-Id: Id9ca20925f49e314918810fb54b3819ba9cf9c39

Follow-up I49b7d8a - Add api module for common metadata

Change-Id: I0d957891e0778ba0454f6fecb06524211506f6b9

Create ApiBase::requireAtLeastOneParameter helper

Added new ApiBase::requireAtLeastOneParameter() helper method for modules
that need one or more of a combination of parameters. The associated help
message function has also been added.

Change-Id: Iae6649ed503fdbf14b313d0be1a82f3dca8d7162

AutoLoaderTest: Add support for class aliases

* Changed the regex to match class aliases, and added a check to
  also ensure that all aliases for a class are in the same file as
  the class (see cb77e59f7c99 for explanation).
* Removed support for the seemingly unmaintained Parsekit extension
  (no release since 2009, and known not to compile against PHP 5.4).
  The test would currently fail if Parsekit were used anyway because
  the class MyLocalSettingsGenerator is commented out.

Change-Id: I7be2352f4659abc7b4952a9f74a14d0cef911614

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

Merge "Add Status outparam for AbortNewAccount hook to fix API error handling"

Localisation updates from https://translatewiki.net.

Change-Id: I6034169b0cb9f50d581683b2b66854a733c552c7

Merge "Split documentation for $wgRCLinkLimits/$wgRCLinkDays"

Merge "Remove `login-throttled` system message reuse"

Merge "Remove the fade in image in favour of background gradient"

Remove the fade in image in favour of background gradient

* Introduces vertical-gradient mixin (tried and tested
in MobileFrontend)
* Add additional LESS variables
* Just dont give versions with no vertical gradient support a fade
effect - it's not a big deal
* Save a single asset download

Change-Id: Ie36708f32edf0d83a0be843c115ad82d8014e4e1

Add Status outparam for AbortNewAccount hook to fix API error handling

With this fix and relevant fix for ConfirmEdit in, an API account creation
attempt that fails the captcha check will return a much cleaner error such as:

{
  'error': {
    'code': 'captcha-createaccount-fail',
    'info': 'Incorrect or missing CAPTCHA.'
  }
}

Abort hooks that use the old interface and send a text message will now
be reported with the generic 'createaccount-hook-abort' message code, with the
string passed back intact.

Previously, the returned result would list the contents of the message
_as_ the error code, making it hard for a client to determine the error.

'AbortNewAccount' hook clients can add a '&$status=null' function paramater
on their signature, and along with the back-compat message parameter
you can set something like:

  $msg = wfMessage( 'captcha-createaccount-fail' )->text(); // back-compat
  $status = Status::newFatal( 'captcha-createaccount-fail' ); // new

This is done for ConfirmEdit in If9cc08e

Bug: 60008
Change-Id: I6ae34c00d1051d34363b6d654424be17dcb1ea30

Merge "Installer: Remove two environmental checks"

Merge "(Bug 60030) Use $content of the hook for toc hiding"

Split documentation for $wgRCLinkLimits/$wgRCLinkDays

Each var should have it own doc block

Change-Id: Iec2b37687c009be9412103081ff5fc020c6dad03

Correct Title::getInterwiki method name case on call

Change-Id: Iad9652b3d43ec1d80a8152c41f114acaa7ec3997

Merge "Generate CSS for KSS through ResourceLoader"

Installer: Remove two environmental checks

* envCheckPHP531: Message is outdated ("downgrade to 5.3.0"; the
  current minimum PHP version is 5.3.2), and PHP Bug #50394 has
  not reappeared in any later version.
* envCheckZE1: zend.ze1_compatibility_mode was removed in 5.3.0.

Also updated the error message for envCheckBrokenXML to remove
the "Upgrade to PHP 5.2.9 or later" wording.

Change-Id: Ifc8754151dd23a0314c6739cb78c7ac61a306bc4

Remove some deprecated global functions

Some uses remained in Gerrit extensions, and patches have been submitted
to replace the usages. See below for details.

* in_string (deprecated in 1.21)
* wfGenerateToken (deprecated in 1.20) (had 2 uses)
* wfStreamFile (deprecated in 1.19) (had 1 use)
* wfUILang (deprecated in 1.18)

Change-Id: Ic67a968da8f37daf44503b98d00710eb99180805

Generate CSS for KSS through ResourceLoader

This ensures the LESS is processed the same, so the behavior and output
exactly match.

Change-Id: I9bba8f2c328c9f9b9f7763cd335e280e973d03d6

Merge "Re-remove experiment code for moduleStorage"

Re-remove experiment code for moduleStorage

The second iteration of the experiment is done, so this can once again be
removed. This patch reverts commit Ifa97d36d3.

Change-Id: If85b392d3e29de57316d32ba81c9b172bef758dd

Remove deprecated ParserOptions::getSkin()

Has been deprecated since 1.18. One remaining use in a Gerrit extension
replaced.

Change-Id: I7eba24593368047ca48a5e1cb60378319c1e51b2

Merge "Remove deprecated MediaWiki::articleFromTitle() and WikiPage::createUpdates()"

Merge "Provide correct type as input for LoadMonitor::postConnectionBackoff()"

Merge "Update/add documentation in the LoadBalancer class"

Merge "Expand documentation around $wgAPI*Modules"

Remove TODO for use of TRUNCATE and replace with comment

The new comment explain why truncate is not possible

Change-Id: I8dacff0784a07a0e20a4da0cc71043379ded6eb0

Remove deprecated MediaWiki::articleFromTitle() and WikiPage::createUpdates()

No uses found in core or Gerrit extensions.

* MediaWiki::articleFromTitle() (deprecated in 1.18)
* WikiPage::createUpdates() (deprecated in 1.18)

Change-Id: I689249b579ee89f883b97fdb4ac79fb3502c45dc

Remove FakeMemCachedClient

Was deprecated in 1.18. Not used in core or Gerrit extensions.

Change-Id: I06bc9c6f01f21386c772fa7fad57b55a7c7b2fe4

Short circuit in notifyRCFeeds

Save instantiating classes unnecesserily

Add getEngine return type hint. Also make public

Change-Id: Ie2610516f99fe7b01742e7e95e13c4b1b90c77d7

Begin abstracting more of the database-specific search backend out

There's a lot in the base search implementations that is specific
to the database backed search. This starts moving some of that out
into a shared base class for those.

For starters, let's not grab a connection to the slave DB for
every single search for backends unless they need it.

Change-Id: Ib66696841eea901e04b21dd309784af889a45ab1

Merge "Clean up whitespace in various javascript files"

Clean up whitespace in various javascript files

Change-Id: I62fe370493b269618a0277ea06ed6ad22b97616a

Merge "Move initial setting of $extNode out of the conditional"

Move initial setting of $extNode out of the conditional

Change-Id: I3d13bfde15085d8c17ef41c6729bec2e9f8f84a2

Check for very old PCRE versions in installer and updater

RE_IPV6_ADD uses (?(-n)) ("relative reference condition"), and CSSMin uses
\K ("reset start of match"), which only work in PCRE 7.2 and later -- newer
versions than the PCRE 6.6 included with Red Hat Enterprise Linux 5 and its
derivatives (e.g. CentOS 5).

Because the WMF developers, in general, do not seem to support maintaining
compatibility with such old software versions, I have opted to add a check
to the MediaWiki installer for these versions of PCRE. Affected users are
directed to a MediaWiki.org page advising the use of a different PHP package
that uses the bundled PCRE version instead of the older system version.

For now, the minimum PCRE version is set to 7.2, the oldest version not
known to break MediaWiki core. Once PHP 5.3 support is dropped, we may be
able to require PCRE 8.12 (bundled with PHP 5.4.0) or later.

The existing check for mere existence of the PCRE functions is removed;
since PHP 5.3, it is impossible to compile PHP without PCRE support.

Bug: 58213
Change-Id: Icf3732b6f84eeb25990178ae8fe3bd0fe4cc833f

Localisation updates from https://translatewiki.net.

Change-Id: I2bf221ca2d246108335c61947a464d23961a6551

Merge "Removed RdfMetaData"

Localisation updates from https://translatewiki.net.

Change-Id: I36b88d4b387ed2313b2160a6ba9fc6be056ded96

Remove `login-throttled` system message reuse

Replaced calls to the `login-throttled` system message to new, separate system
messages for [[Special:ChangeEmail]] and [[Special:ChangePassword]] in order to
minimise message reuse.

* [[Special:ChangeEmail]] now uses `changeemail-throttled` instead of
  `login-throttled`.
* [[Special:ChangePassword]] now uses `changepassword-throttled` instead of
  `login-throttled`.

Follow-up to I565971a90fc383327a51d717a80013cb7a3e9c9b.

Bug: 53655
Change-Id: Icc7ad829b381514627dca16cde62ccf0831967b6