No such method Exception::getText()

It looks like a recent change converted this try/catch
statement from MWException to Exception, unfortunatly it
is still calling a method that only exists on MWException.

We arn't doing anything with the exception but exiting.  Lets
just allow the exception to be uncaught and fail naturally. The
uncaught exception will still output an error and return
a non-0 exit code.

Change-Id: I3e0e9d283c255d2aba8139d675943e93d7cb021a

API: Don't dump Title object into API result

The intention here is to return the title as a text string, not to dump
all the internal fields of Title.

Bug: T88397
Change-Id: I43ad3c98bc1fc056a13f57e2e95a122108049334

Merge "registration: Fix handling of MessagesDirs array and add tests"

Merge "Update OOjs UI to v0.6.5"

Localisation updates from https://translatewiki.net.

Change-Id: I03277966c3b2a86316dbdae595370b9540f982bb

Merge "Gruntfile: Clean up Karma config"

Merge "Output wikiid/api in meta=siteinfo only when non-empty"

Update OOjs UI to v0.6.5

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.6.5/History.md

Change-Id: I1278a9d0f3b63977f293a79c73283593c65bf910

Merge "Remove dir=auto from firstHeading"

composer.json: Correct license identifier

According to <https://spdx.org/licenses/>, "GPL-2.0" means "GNU General
Public License v2.0 only", not "GNU General Public License v2.0 or later",
which has the identifier "GPL-2.0+".

Also made the same change in the documentation for $wgExtensionCredits.

Change-Id: If25c15d03a71d7c50c9c85a04c2a589a3d978ad0

Merge "codepointToUtf8(): Throw exception rather than calling die()"

Merge "Pass user to FileRepo::getUploadStash"

Pass user to FileRepo::getUploadStash

This avoids use of $wgUser in UploadStash

Change-Id: I82ca69818317508109b4d5f4823a20de47f29b01

Merge "Revert "CategoryView modified to use css columns""

Revert "CategoryView modified to use css columns"

The new feature does not work very well, and the commit
breaks several unrelated things. Sorry, let's try again.

This reverts commit cb5a0f20a436b90a5cfabfba84554e712b32794a.

Bug: T85496
Change-Id: I018050ec0bf773e6e22e99819fe3cad91171bbd2

codepointToUtf8(): Throw exception rather than calling die()

Change-Id: I0f2c05be95d34e069a3e9965f581d2adfc0a1b9d

Gruntfile: Clean up Karma config

Change-Id: I8197c2b0169983c32a44397faec95ad7eb47ab86

Update OOjs UI to v0.6.4

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.6.4/History.md

Change-Id: I75746e6f702e2108e5282f9a37827a1a533ab57a

Merge "StringUtils: throw InvalidArgumentException and move into libs/"

Merge "Kill hitcounter remains"

objectcache: Use Psr\Log\LoggerInterface instead of wfDebug

ObjectCache::newFromParams() will handle a 'loggroup' parameter
specially, getting an instance of \Psr\Log\LoggerInterface to
pass to the BagsOStuff.

BagOStuff implementations can use $this->logger which will be
an implementation of \Psr\Log\LoggerInterface. As this is set in
BagOStuff::__construct(), all subclasses must now call the
parent constructor.

The goal of this is to make the logging in BagOStuff non-MediaWiki
specific, in the hopes of separating it out into a separate library
in the future.

Change-Id: I8a8e278e6f028814499d8457d6d5341d03eabc7a

Update karma to latest version

Per Timo, hopefully this should fix our issues with Chromium timing out
after 10 seconds?

Related patch: https://github.com/karma-runner/karma/commit/840ee5f7

Change-Id: Idbac350e2c612290f860ae740ef6ba242c9e0896

Merge "PostgreSQL: Drop unneeded foreign key constraint"

PostgreSQL: Drop unneeded foreign key constraint

Change I1c7f3a84f10df05d6b37dccbad4c8232edf51580 causes
an existing foreign key assumption (under PostgreSQL) to be
violated upon deleting a page.  This foreign key assumption does not
explicitly exist in MySQL, and is not implied via documentation.  So
it was probably never needed in the first place.

Don't create the foreign key constraint in PostgreSQL, and drop it
if it already exists when running update.php.

The constraint was previously created with an implicit name, so
drop the constraint involving the specified column name (rc_cur_id),
rather than hard-coding the name of the constraint itself.

This bug probably exists under Oracle and MSSQL as well, but no attempt
was made to address it there.

Bug: T76254
Change-Id: I2abd650c8ce83c5b725aec0545fff14a927a305a

Merge "Tweaked BagOStuff::lock() retry times slightly to be faster"

registration: Fix handling of MessagesDirs array and add tests

Previously the code was designed to handle:
  "MessagesDirs": {
    "FooBar": "i18n"
  }

However, it can also be an array, and some extensions (VisualEditor)
use it like:

  "MessagesDirs": {
    "FooBar": [
      "i18n",
      "also-i18n"
    ]
  }

This properly handles both strings and arrays and adds tests to verify
the behavior.

Change-Id: Iff1523b86f754cac1f5b8d822d4324c5fbfc1a50

Localisation updates from https://translatewiki.net.

Change-Id: I88b24823903debcef363e6c0f79bbf86e0b7a978

Merge "break up long line that phpcs whinges about in DefaultSettings"

Merge "Fix typo in Release note"

break up long line that phpcs whinges about in DefaultSettings

Change-Id: Ic2cc583c112e8235725dffa78e98f18ba6998b72

Output wikiid/api in meta=siteinfo only when non-empty

When using interwiki data from the database the value for iw_wikiid and
iw_api maybe empty strings, because the database does not allow NULL
here.
Suppress the output of the attribute in that case, because there are not
useful.
This should not break clients, because using .cdb (like WMF wikis)
already missing these attributes in the result.

Change-Id: I60b97b4b58fdeaf699951544d418e5dc06e3830a

Fill the iw_api column of the interwiki table when installing

Was added in r69542 for interwiki-transclusion, but not populated.

Also make link for 'mw' consistent with 'mediawikiwiki' by using HTTPS.

Bug: 46992
Change-Id: Ieb6c4121ae4cd4e1aefbc372bbe82bdb0ce53b5d

Merge "CategoryView modified to use css columns"

Escape unescaped messages shown in action=info

Change-Id: Id16d8c8dff73fdacad6c9a4ff7f2919945b7e893

Add missing </ul> in Article::getRedirectHeaderHtml

Change-Id: I673213fc424c859ce81b1cf1a85e6ed9e508d0b6

Fix typo in Release note

Follow-Up: I433e29866fe184ba80c5dda35722e228e79f9307
Change-Id: Ic35ee4a3acefd23ee006a50210a4cb2eedbdacba

CategoryView modified to use css columns

The categories on Category page were listed using table layout. This layout has
been changed to the responsive css column structure, which is 3 column wide
beyond 768px otherwise shrinks to a single column.

Bug: T85496
Change-Id: I2d3583efd470500c374965a433538845fcfade26

Merge "Escape message 'version-poweredby-others' on Special:Version"

Merge "convertExtensionToRegistration: Add --skin option"

Localisation updates from https://translatewiki.net.

Change-Id: I4af632da08565b289d5fca22efd6bb6df0d115b8

Merge "normal: Add @covers tag to UtfNormal test"

Merge "DeferredStringifier: Clean up comments and indentation in tests"

Merge "ProcessCacheLRU: Clean up doc comments and add @covers tags to tests"

Merge "Sync up with Parsoid parserTests."

Escape message 'version-poweredby-others' on Special:Version

Bug: T85864
Change-Id: I1e90962c3c94c8b72aadfc4c0d041d52e2cde3f1

normal: Add @covers tag to UtfNormal test

Change-Id: Ib0d64bda44cbb1f72ea69025785442967bb26d74

convertExtensionToRegistration: Add --skin option

To write to skin.json instead of extension.json

Change-Id: I0e7dfbd871b7f80958fe7033d4595a205bb34293

Merge "Avoid double escaped expiry time on action=protect"

DeferredStringifier: Clean up comments and indentation in tests

Change-Id: Ifdf2fe711a0be495465255c6fabb55fe34b79b27

ProcessCacheLRU: Clean up doc comments and add @covers tags to tests

Change-Id: Ie7e8f4b31d89fc0bee748c03d94cb54b23ab4f83

Merge "Escape all return values from ProtectedPagesPager::formatValue"

CSSMin: Optimise isLocalUrl by moving fast comparison forward

Change-Id: I668e248bd51fb75dfeeef3b74973f517a188c7ba

CSSMin: Add tests for isRemoteUrl and isLocalUrl

Change-Id: I5a84eb62eaac96d0dd7f7e27bf76f64e7d7657cf

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

Localisation updates from https://translatewiki.net.

Updates for magic words.

Change-Id: Ic564d36de395c9a30be5f1390591da723f47edce

Merge "Fully escape return value from FileDuplicateSearchPage::formatResult"

Localisation updates from https://translatewiki.net.

Updates for special page aliases.

Change-Id: Ic2b34a948d527cee6b76abd7e24a6d8a33b65bd2

StringUtils: throw InvalidArgumentException and move into libs/

Bug: T87863
Change-Id: Iac5bd958c27cad834e35930d0b99adb75c60411b

Merge "Use context when parsing messages in Article::setOldSubtitle"

Merge "Set context when parsing message 'redirectpagesub' in Article"

Merge "Set context when parsing message in SpecialLog::addHeader"

Merge "Pass user to Title::quickUserCan in DifferenceEngine"

Set context when parsing message 'redirectpagesub' in Article

Change-Id: I53fc0855fc8dd57cdfcae68751c9800969208310

Use context when parsing messages in Article::setOldSubtitle

Change-Id: Iec4985021b862d6e99f5531545b7db287325263b

Set context when parsing message in SpecialLog::addHeader

Also direct passing a message object to OutputPage::setPageTitle

Change-Id: If88ca673bffa033f9cd9cc7a680b73aa701578f3

Pass user to Title::quickUserCan in DifferenceEngine

Change-Id: I2e984a51b7ad357a08ed1827b76f7f9803785cfc

ReplacementArray: Set explicit visiblity on functions

Also add a type-hint

Change-Id: I78a322dfa11a71e22a3d0b7064b808aec6bfbf83

Move ReplacementArray into includes/libs/

Change-Id: I95b748c77522027753fb5bd0237f25e254938c16

ExplodeIterator: Set explicit visiblity on functions

Change-Id: I407280a432098d13ad75ff2d3468aa6a7d653da7

Move ExplodeIterator into includes/libs/

Change-Id: Ibb3b36dbce223454b9cd485743d7e7845f729bfb

replacers: Make Replacer abstract, and add abstract Replacer::replace()

Change-Id: Ib00dc8585e8ba599491e51e0b99a8667c3b4cd63

replacers: Set explicit visiblity on functions

Change-Id: I4f35ea9b4bd2503bc612dc25dc8d34fc5ca040a0

Move "Replacers" into includes/libs/replacers/

Split into separate files while we're at it

Change-Id: I0bba4dcea686de088bd96964833fe6fb649a41e9

Turn Utf8Test into a PHPUnit test case

Change-Id: I7e709f475a37a592fb3052f367894ebce32310f3

Move includes/normal to includes/libs/normal

Change-Id: Ib530ad9dbe1d3a33dc53ef8b9620f61d4e1a2d62

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 88605a4a7a37a61da76238db6d3fff756e8514f1

Change-Id: Ia6d103bec95019eaece64166172a166f8eabdd09

Merge "Output COPYING files as plain text in Special:Version/License/*"

Merge "Add OutputPage::enableOOUI() to set up OOUI stuff when needed"

Merge "Update and enrich API documentation"

Escape all return values from ProtectedPagesPager::formatValue

Bug: T85864
Change-Id: Ifb58bd3823aa633135a701ae3e2ae9c955b422c0

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

Localisation updates from https://translatewiki.net.

Change-Id: I421056addc820a66a5912b9d4c23255302768cba

Output COPYING files as plain text in Special:Version/License/*

The vast majority of them consist of the GNU GPL, which is in
plain text format and not supposed to be converted into wikitext.

Change-Id: Iaa30b8994b8e4034ace61df8202a8b5002093ff2

Escape return value from Skin::getUndeleteLink

Bug: T85864
Change-Id: I7a5dfc8ecd2f84da0dce3e1357af462a3c0da23f

Merge "Special:AllMessages: Ignore case of first letter when sorting"

Merge "Only try to load UtfNormalDefines.php once"

Merge "Fix doc for ManualLogEntry::setParameters()"

Only try to load UtfNormalDefines.php once

It's already loaded in Defines.php, so no need to do it again

Change-Id: Ie6f3bf3fbbcbaadd85cfc8e54527b93e98f7b898

Update and enrich API documentation

As discussed previously with Brad. I've done the first 10% or so of the
documentation. Once this has been merged, I'll continue with the rest.

Change-Id: Iaef77a14b2bc184ca28bac02087e67744d4573d4

Fix doc for ManualLogEntry::setParameters()

Numbered-and-named parameters are "number:type:name" (where "type" is
often empty), not "number:name".

Change-Id: I417a3d6caf929a94ffed5e315692950295b3be0d

Kill hitcounter remains

Followup to Ieeb558f9523c11965cbc1941cad4f316c00c85c5
No need to create the table and drop it then

Change-Id: I11409c6e5a76d65365c5652039c584e249dbf556

Pass user to Title::userCan in InfoAction

Change-Id: Id4dac8ca3b36839abdb184d0ad538b2400aa80ff

Don't split regex string unnecessarily

Change-Id: Id5912e64916ce5c7be2991478c32531596917540

Add plainlinks class to <h2> in Special:Version

Make "Parser extension tags" and "Parser function hooks" <h2> external
links to plainlink by adding plainlinks class to <h2>.

Bug: T70426
Change-Id: Ibbef62ff5bb0457eb25b9964c190c3e8283b2610

Fully escape return value from FileDuplicateSearchPage::formatResult

Also avoids unneeded recreation of a title object

Bug: T85864
Change-Id: I0298887e2ee5da9c1694393fb06cfa5eed0e46d3

Make Sanitizer::removeHTMLtags() and ::removeHTMLComments public

The functions are used in at least 5 extensions in Gerrit, so not private
at all.

Change-Id: Id6d7b82d7333184ff2b1dcd592a80af30a50a889

Merge "SpecialMovepage: Correctly apply target namespace to subpages"

Merge "Import: Fix error reporting"

SpecialMovepage: Correctly apply target namespace to subpages

Previously, moving a page from a subject namespace to a talk namespace
(or vice-versa), resulted in the subpages being moved to the new page
name, but still in the old namespace.

Bug: T44024
Change-Id: Iefc5b495e777840dc6a2fe1226b8869d807f2d94

Merge "Run clearSearchIndex when mysql in rebuildTextIndex"