Make $wgEmailConfirmToEdit only affect edit actions.

Previously it would affect all actions that use Title::userCan.
This used to be less noticable, but recently was expanded to include
the 'read' action. This only affected the case where both
$wgBlockDisablesLogin and $wgEmailConfirmedToEdit were enabled.

I don't think anyone was relying on the old behaviour as it was
undocumented, and only affected obscure permissions (checked with
Title::userCan and not depending on "edit" rights)

Follow-up b675be2083

Bug: T143790
Change-Id: I4ad93ed78de4f1ed444f73df6dc26d405a67e553
(cherry picked from commit d561f646b9b8424bd79cfc14729a622a143d9f12)

Follow-up Ieaeb3113a: Add RELEASE-NOTES

Change-Id: I79f1d28b54532a7495fb8e205c9b6636016587d7
(cherry picked from commit 0d37539a7c9191f893dec6e77591ae753e6b461a)

Ensure $user is passed by reference in TitleMoveComplete hook on HHVM

Suggested by Anomie in T118683#3688320.

Bug: T118683
Change-Id: I57e05c5ae42c8b85d0a8f9631a459c5686b27893
(cherry picked from commit d5e4bf920c7ff48bf4c907bee1ee47fa96907774)

registration: Allow the mtime of extension.json files to be zero

When creating Docker images of MediaWiki using the Bazel build system, I
noticed that I'm not able to load any extensions. This is due to the
fact that Bazel always generates container layers with mtimes of files
set to 1970-01-01 for determinism/reproducibility.

Relax the check a bit to only fail when the mtime is false, which
happens when filemtime() fails.

Bug: T196672
Change-Id: Ieaeb3113a7d9c44f29cca2d062c5bb11ebeada0d
(cherry picked from commit d7d5d3c82a84338bd2669284427193ea7a80bdbc)

installer: Fix display of UPGRADE by disabling InterwikiLookup

Since 129067c907ea65f62, parsing section titles has looked up interwiki
prefixes with InterwikiLookup. In the web upgrader, this triggers
database access, and since that service is disabled, it throws
exceptions, causing parsing to fail.

Work around that by using a dummy InterwikiLookup service that knows
about no interwiki prefixes. Maybe one could be written to just read
from the stock `interwiki.list`, but that's a project for another time.

Bug: T196607
Change-Id: I13485a9af79297b552a1128240cb8597c2ef83d8
(cherry picked from commit a498abf272c7aff376c5225a6f593349e3cc3eaa)

Fix UploadBase::checkXMLEncodingMissmatch() on PHP 7.1+

file_get_contents() started supporting a negative offset in 7.1+. But
we really just want to start with 0.

Also fix the order of arguments to assertSame() so that the expected
value is first.

Bug: T182366
Change-Id: I84c92652de5b51a43f6e2b58cd235d2889093453

Add vendor submodule

People will no longer need to clone it separately, just run
`git submodule update --init`.

If people want to run composer locally and have already checked out the
submodule, a `git submodule deinit vendor` will disable the submodule.

Change-Id: I1fdaf8dc3c2a8e246bb7d2987a42a41272dffc29

Move RELEASE-NOTES for I4678250331a48db4d50d1fc6c489c991a4dee920

Change-Id: Ic880a54427bf8bd9adf96863da25c485cb3c5662

Merge "objectcache: update MemcachedPeclBagOStuff for pecl memcached 3.0.0" into REL1_31

Populate 1.31 rc.0 -> rc.2 release notes

Change-Id: I15fa23e881c0e177d12a6f6374bc08af3b197435
(cherry picked from commit 6e3bb339713db677df2272d7ff6f634dff1f244d)

objectcache: update MemcachedPeclBagOStuff for pecl memcached 3.0.0

The get() $cas_token parameter was changed into $flags, which can act
as a switch to make the return value an associative array of details.

pecl and PHP-based memcached BagOStuff class both pass all tests now.

Bug: T196125
Change-Id: I4678250331a48db4d50d1fc6c489c991a4dee920

Deprecate $wgDBmysql5

Bug: T196185
Change-Id: Iea74b5dd1fc49c911dcc0070604f05100e56ce0d
(cherry picked from commit 7271d50bab79231efcedeabf5fbe4db0aa13fa4d)

Merge "installer: Don't allow setting $wgDBmysql5" into REL1_31

installer: Don't allow setting $wgDBmysql5

It was possible to pass --dbmysql5 to the command line installer to
enable this experimental option. This removes that ability, and removes
setting of $wgDBmysql5 in the generated LocalSettings.php, so new
installs will fallback to the default of false.

Bug: T196185
Change-Id: Id5ff69f493d14cf7b16157f729fbe4f4b4e2e8c8
(cherry picked from commit c9339ba552cb5f87586e7045b46519bd9db04fe0)

installer: Hide binary/utf-8 charset option from users

This is still used internally for the preUpgrade() checks that see if
the user has an existing table using the utf8 charset, but hide it from
users so they don't accidentally pick utf8.

Bug: T196092
Change-Id: Iaace2e5bf9df7563e1d233496366c95b3f1faf16
(cherry picked from commit d5cde75416893983bfc65d161049798c26e33c54)

registration: Initialize PSR-4 namespaces at same stage as normal autoloader

readFromQueue() injects the content of AutoloadClasses to
$wgAutoloadClasses however it missed doing the same for
AutoloadNamespaces.

When using the installer with an extension having AutoloadNamespaces
set, its classes would not be found.

Make ExtensionRegistry append to AutoLoader::$psr4Namespaces, and add
a test to cover the new behavior.

Bug: T195783
Change-Id: Id61155867a4ca7d9bc4a347f8671da74b0fa490b
(cherry picked from commit 224864ebddab2e448f9ae4f247b85c652d9df42e)

Update git submodules

* Update skins/Timeless from branch 'REL1_31'
  to 58301ba4468f4667c0c180f67bda238f0fedd58e
  - Fix condition for 'emptyPortlet' class

    Follow-up to 6b9616d1ca337308d773441dcfb263d9c68fdab6, which
    accidentally inverted the logic.

    This is a manual cherry-pick of 70b5f24986a5162c06ab343820e54c989753eeb4.

    Bug: T196026
    Change-Id: I98bc7529a1894b09ec3d80aab244327f526c022c

Bump version from rc.0 to rc.2

Should've been bumped awhile ago to rc.1, meaning the rc.1
tag had the wrong version number. Rather than re-issue the
tag, we'll just move forward and not bother with rc.1

Change-Id: Ie89caf3826c35c3a15fe1390c4caec1f8b7d19cc

Update git submodules

* Update skins/Timeless from branch 'REL1_31'
  to 97ba1dd2996811899b6b1a96dce3fa20dd613b21
  - Fix $count to === against array instead of using count that fails

    Bug: T195613
    Change-Id: If2b51af8c5335394b8c0266ac2765e1c7ce49618

Release notes for I61749f1d864cf68afe90cd9e15ba2d7a74252501

Change-Id: Idd0a8f5bbc1da161817276cef44ff61a0e72f8b2

Only load PEAR Smtp mailer from vendor/composer, not from `pear` itself

Change-Id: I91c42a3eafd079039ddbc17a11ad3c7b6579e3c1

mail: Refactor checks looking for PEAR libraries to be clearer

I don't think this makes any difference towards the linked bug, but
it makes it more obvious that MediaWiki doesn't try to use any PEAR
classes before attempting to require them.

Bug: T186456
Change-Id: I342f41903a15f13e72e76464499351412a9afa34
(cherry picked from commit 5d30435e81427fcce3353d900faaf8d9fa9a8709)

build: Updating mediawiki/mediawiki-codesniffer to 18.0.0

Change-Id: Id39e8dcf22262d1b3c57628467163a995fe4053b
(cherry picked from commit 5a024697a272981fc491e7502d6d5fddef4d2c22)

installer: Do not wrongly hide namespace input field

When the radio button is already on "other" then the input field for
the namespace should not get hidden by JavaScript on load.

Also remove the readonly attribute in HTML because the readonly
attribute can not get removed with disabled JavaScript.

Change-Id: I8cfde90d791765234572caf00b731881ac2eda48
(cherry picked from commit 31a472655b246df3c072d36959afcb7d2897889d)

Fix warning "ob_end_flush(): failed to delete and flush buffer" during uploads

Bug: T186565
Change-Id: Ia4b29611ccee4acac11717f5220ff9e0fdbd55a9
(cherry picked from commit 30d72ec3d0914bb905d4114dee9256f02cbdcc7f)

Don't require trailing slash in PSR-4 autoloader directory

This avoids the confusing error “Fatal error: Uncaught
InvalidArgumentException” or similar.

Bug: T195211
Change-Id: Ifda59a26f8bd968a2d0acbdb157d81dc0bf6aab4
(cherry picked from commit 3b85e362a4098832a08529adc89571a86bf6e9a2)

Update git submodules

* Update extensions/ReplaceText from branch 'REL1_31'
  to a027ec972c1d98e92483230e1cbf71cc08a31150
  - Fix path to maintenance directory.

    Change-Id: Ife7a7076ee87938cadb84aa56982cc667bd90e2f
    (cherry picked from commit f7b59db9b3aba2e5523a11d7922f668ef05806ee)

Merge "registration: Improve duplicate config setting exception" into REL1_31

Merge "IcuCollation: Use codepoint as tiebreaker when getting first-letters" into REL1_31

registration: Improve duplicate config setting exception

We don't keep track of what set a specific global, so at least mention
the name of the extension that is setting a duplicate for easier
debugging.

Also, fix the case where if the first extension to be loaded was setting
a core setting, it would not throw an exception since config was being
processed before the rest of extension.json. Now we process config after
all core settings, going only before attributes.

Bug: T194319
Change-Id: I4fd96e7d167cf0652ee3e8e66167c86f2b91b992
(cherry picked from commit 5490b1270a0a7447f28d2407736fc3c1deb41d44)

installer: Don't shell out if it's disabled

Bug: T191947
Change-Id: I16a82d271157cd0024aa14d7eaec80b4870947b5
(cherry picked from commit 2e473413a8fd1e205142b5c5980addb34aaf369f)

Update git submodules

* Update extensions/MultimediaViewer from branch 'REL1_31'
  to 1273d3e0b2189e33fed0567f589a3e7bc4ae6d4e
  - tests: Comment out failing tests

    Bug: T192932
    Change-Id: Iba81676ebb13407679bd99b0e08d29e225a7ebde
    (cherry picked from commit 2078fbfb53acd357c96ea187f31a6ba878afc053)

IcuCollation: Use codepoint as tiebreaker when getting first-letters

This prevents unexpected cuneiform digits from acting as headings for
2 and 3 on category pages.

Bug: T187645
Change-Id: I0424a24769899cb23b28704f97e1002fa44999fd
(cherry picked from commit 390ff7fca179e26ac177810145d27d98fe2fff43)

Add default edit rate limit of 90 edits/minute for all users

Previously there was no rate limit at all if you were autoconfirmed.

This initial setting is set on the high end of things, we may
adjust later. See also T192668, T194204.

Change-Id: I7cb002900bae2da3f4f4758671186659c1720e1d
(cherry picked from commit cefdcefdb8f15ffdec8345b93aff2036db92d1f7)

Add ReplaceText submodule

Bug: T191741
Change-Id: Iea4e0e4e04b2ec1b4fc07b11fb0af1ac3ee6347f

Update git submodules

* Update extensions/TitleBlacklist from branch 'REL1_31'
  to 1143a1f61e049e4f8e4de317e7299091a2ca0707
  - Upgrade phan-taint-check-plugin to 1.2.0

    Change-Id: I9c02ca49f9e35873adc2e2462df1020fd30eb934
    (cherry picked from commit 616a7afb885bd8707c143bb271831fb31fc8d377)

Update git submodules

* Update extensions/Renameuser from branch 'REL1_31'
  to f8e515711db88fdd2aae02b760d2e11df3d70489
  - Upgrade phan-taint-check-plugin to 1.2.0

    Change-Id: I3af50c0a15b7ec0ecea087fefc3d026cced99749
    (cherry picked from commit 52ac2bdc3221a65a15d383f7fafc56f47e31b08b)

Update git submodules

* Update extensions/SyntaxHighlight_GeSHi from branch 'REL1_31'
  to a7d04ae33439d1c1b595d68fbad08c43f930fdcf
  - Upgrade phan-taint-check-plugin to 1.2.0

    Change-Id: I2fa1c622dc495e3f4969d7684934f1839f9caf00
    (cherry picked from commit b185d9892f5c45d02c0bb3c620644c831945d7bb)

Update git submodules

* Update extensions/SpamBlacklist from branch 'REL1_31'
  to f6fa789ad2c07a4fe89e4d1e636aea1126f26afc
  - Upgrade phan-taint-check-plugin to 1.2.0

    Change-Id: Iaefb223d2217762f6fa91a854d0f963270362c07
    (cherry picked from commit 9a87cc3c060eab4d576f3561f0f197f91ac090c2)

Update git submodules

* Update extensions/Poem from branch 'REL1_31'
  to c4098df5ea8132826164f71c0c18b54ae8450edf
  - Upgrade phan-taint-check-plugin to 1.2.0

    Change-Id: I38ce057fc41a0164922f403b77d998df2b4f1099
    (cherry picked from commit 8552303267a4f45a38048f90595385572efe30f0)

Update git submodules

* Update extensions/PdfHandler from branch 'REL1_31'
  to 21f2fe96901077927bde13ec15496c186a208197
  - Configure phan-taint-check-plugin

    Change-Id: I934702057ecee687d9966a6f042ee8aa7c68f2d7
    (cherry picked from commit 5f0b10d2a5e8ea8e5aa1f05ab1c9ddbab07a1584)

Update git submodules

* Update extensions/WikiEditor from branch 'REL1_31'
  to 277159cd3ae6fdafb7f49db3dbfedfa023e81d35
  - Upgrade phan-taint-check-plugin to 1.2.0

    Change-Id: Ib6953c3f1eb5b9c9edc7d58735391b965ac6b27a
    (cherry picked from commit 9ba9e3e4e12a8004fe7198285a8864e752259091)

Adding Timeless skin to 1.31

Change-Id: I57d868eb596f5067690fbfdb9c546dcbdbaf5be2

Update git submodules

* Update extensions/InputBox from branch 'REL1_31'
  to dfec2613ddd6b2bfc6cf291be74429b5bfebea89
  - Bump phan-taint-check 1.1.0->1.2.0

    Bug: T193909
    Change-Id: I1bf4d64e68bff516b5f8b6266d85f670200084df
    (cherry picked from commit 4d4aefd2f24e9e8e996e3fba29df3f5cae60c7b7)

Update git submodules

* Update extensions/InputBox from branch 'REL1_31'
  to 0fdeec73bd212f4409bfbdfcfc1aa977694d3c43
  - Fix a phan-taint-check false positive

    phan-taint-check gets confused when you escape a variable and
    assign the escaped version to the same name as the unescaped
    version.

    Change-Id: I1bf4d64e68bff516b5f8b6266d85f67020008433
    (cherry picked from commit 252fed5d27cc67701c7fbf98370e4af0d50fcb38)

Update git submodules

* Update extensions/CodeEditor from branch 'REL1_31'
  to 886d7971ae1b04edc463bc7e11d7f5d282ae4f10
  - Add phan-taint-check as version 1.2.0

    Bug: T193909
    Change-Id: Ibbfa1554ed1fe4dcaba2629abf608b59a2dd2733
    (cherry picked from commit e9df595a2670e967fd5005b87fa401287cc18a48)

Update git submodules

* Update extensions/CiteThisPage from branch 'REL1_31'
  to f224f44304637e739ac470f7fca25498486cb0fe
  - Bump phan-taint-check 1.1.0->1.2.0

    Bug: T193909
    Change-Id: I1bf4d64e68bff516b5f8b6266d85f670200084df
    (cherry picked from commit 860ece0d63309b077d605526790e198907bbd185)

Update git submodules

* Update extensions/ParserFunctions from branch 'REL1_31'
  to 7ec426b9a1d5fabfa8ba79e08806c5b3f796837a
  - Update phan-taint-check version 1.1.0 -> 1.2.0

    Change-Id: Ib3eec1e70d8c1e4fdcfe811168adff7e32878d3b
    (cherry picked from commit b67b90c6a4918396c221c7154a827495d3d6d942)

Update git submodules

* Update extensions/Interwiki from branch 'REL1_31'
  to b78d2d1d83f403241a5c882c62fcb5019fdcf5a2
  - Bump phan-taint-check 1.1.0->1.2.0

    Bug: T193909
    Change-Id: I1bf4d64e68bff516b5f8b6266d85f670200084df
    (cherry picked from commit 14279ac1774db1d9585cf1b97074de359194556d)

Update git submodules

* Update extensions/Gadgets from branch 'REL1_31'
  to bac29aee151272c14b0675af2f4c273dc3139a1b
  - Bump phan-taint-check 1.1.0->1.2.0

    Bug: T193909
    Change-Id: I1bf4d64e68bff516b5f8b6266d85f670200084df
    (cherry picked from commit ee633a273bda1f16a63897c6adfe4f6baa889347)

Update git submodules

* Update extensions/ImageMap from branch 'REL1_31'
  to 6ca1ad79b0afb0d7b80f4ffccd844cc557e22db9
  - Bump phan-taint-check 1.1.0->1.2.0

    Bug: T193909
    Change-Id: I1bf4d64e68bff516b5f8b6266d85f670200084df
    (cherry picked from commit 08909bdffa0ab08f5fa5bbeec79555ca898dcdab)

Update git submodules

* Update extensions/LocalisationUpdate from branch 'REL1_31'
  to 0249f52dca6b034edc249894efdc849e161eb925
  - Bump phan-taint-check 1.1.0->1.2.0

    Bug: T193909
    Change-Id: I1bf4d64e68bff516b5f8b6266d85f670200084df
    (cherry picked from commit d9ed872a2adec76915b0e86c2fd3d84bc9309e0a)

Update git submodules

* Update extensions/ConfirmEdit from branch 'REL1_31'
  to b52469cf5716165dcf55e0db103d45af045b2acc
  - Bump phan-taint-check 1.1.0->1.2.0

    Bug: T193909
    Change-Id: I1bf4d64e68bff516b5f8b6266d85f670200084df
    (cherry picked from commit 544f0712f04bc790782541fdc63cae34976ede4b)

Update git submodules

* Update extensions/OATHAuth from branch 'REL1_31'
  to cf9f8c126b5213b77fb4dc7b7f49d775c8189119
  - Bump phan-taint-check 1.1.0->1.2.0

    Bug: T193909
    Change-Id: I1bf4d64e68bff516b5f8b6266d85f670200084df
    (cherry picked from commit 19c9b40836b40cb2d82e8be957e0c615eaae20c8)

RELEASE-NOTES-1.31: Synchronise with master

Patches:
* 0e757428d RELEASE-NOTES-1.31: Add notes for Ia76dd6db3
* 0b7d183e2 RELEASE-NOTES-1.31: Add notes for I8228cdd3b
* 360fac542 RELEASE-NOTES-1.31: Add notes for I94377faa5
* b4674e56a RELEASE-NOTES-1.31: More cleanup

Change-Id: I78d01cd1616c4ec9cdc3f7b798d52cfbf7ba46ad

Drop archive.ar_text and ar_flags

This should have been done long ago. Now it is being done.

This also changes ar_text_id to NOT NULL, since it should never be null
anymore, and DEFAULT 0 in preparation for MCR stopping writing it.

Bug: T33223
Change-Id: I18f1c740b7537c7dc3cfeba9b241d0a9f31caa34
(cherry picked from commit 21c6ae1163b07da7ac49938f50613c0e1cf262c3)

RELEASE-NOTES-1.31: Wrap to 80 chars and fix up

Change-Id: I0a090a3ac057021ac2e3544f5e05b2fadd0b9155

Fix MySQLism in populateRevisionLength.php

Use addQuotes() to make a quoted string instead of MySQL's nonstandard
use of double-quotes.

Change-Id: Ie2d017a33b694417a3db140b2798a8d55f911985
(cherry picked from commit 5e6efe18d01f64f2c9a0a8bcb213ffe30999530f)

Make populateRevisionLength fix rows with ar_len = 0.

Due to T192189, some rows in the archive table may have ar_len set to 0
erronously. This can be detected by checking if they have ar_sha1 set to
"phoiac9h4m842xq45sp7s6u21eteeq1", which is the hash of the empty string.

This patch makes populateRevisionLength.php detect and fix such rows.

Bug: T192189
Change-Id: I9b6cd62f4b8c5bf93cd305b56af4352a527e19c9
(cherry picked from commit e420790f5200568422dce9473ef44d4f7cefe675)

Update git submodules

* Update extensions/CodeEditor from branch 'REL1_31'
  to 422d0186bd4f5f5653683b8cb79c0f7cb6fe2a92
  - Mention Ace license in "license-name"

    Change-Id: Ibbfa1554ed1fe4dcaba2629abf608b59a2dd279a
    (cherry picked from commit 99a38b36a316fed4c4fdbc6ecf688d0fd442606f)

Update wikimedia/html-formatter from 1.0.1 to 1.0.2

https://github.com/wikimedia/html-formatter/releases/tag/1.0.2
https://github.com/wikimedia/html-formatter/compare/1.0.1...1.0.2

Bug: T193318
Change-Id: I5f7a3e2300f343f675124df399cfe12909f56e14
Depends-On: Ia69102af64a0102e05b96b10f63ffcf6cbf292fa
(cherry picked from commit b5088a20c96b2a1be15850a72e637f384b231606)

Update git submodules

* Update extensions/PdfHandler from branch 'REL1_31'
  to ab596dbb7dc3f162c2772832456a1ad4e6266608
  - Fix for pdfinfo changes in poppler-utils 0.48

    PDF metadata querying was done with pdfinfo's "-meta" and "-l" options
    at the same time, which was supported in poppler 0.26 but not in
    poppler 0.48.

    Upstream change: https://bugs.freedesktop.org/show_bug.cgi?id=96801

    Local change is to run the two as separate commands, then send the
    output together into the existing processing. Should work with older
    poppler-utils on Jessie as well as current one on Stretch.

    Bug: T117839
    Bug: T193200
    Change-Id: Ib4ee9cf12ac04304c576087727eff5dc521ae751

Merge "import: Don't try duplicate detection when the sha1 is unavailable" into REL1_31

import: Don't try duplicate detection when the sha1 is unavailable

Trying to compare a string field with 0 (cast from false) raises an
error in PostgreSQL, and we're expecting it to not detect duplicates in
that situation anyway.

Change-Id: Ibff6daee6bbec00e37532e03c9ece041ba612de0
(cherry picked from commit f09d66de2ab88e3ed39fc21b3805cea90eee5bfe)

mail: Always quote uncoded names in MailAddress

Otherwise, names that contain whitespace (but no special characters
that need Q-encoding) will be sent as-is in an email header. While
that's valid by RFC 1036, RFC 5322 and others, some mail programs
fail to correctly process spaces. Avoid these problems by simply
quoting the name always - except for cases which are Q-encoded,
which must remain unquoted to be valid (and also cannot contain
spaces, thus unaffected).

Before:
> From: Foo Bar <fb@example.org>

After:
> From "Foo Bar" <fb@example.org>

Update test to work with the new criteria.

Bug: T191931
Change-Id: I4b2a0a80e8b43ecc943939fc5ebaa356a4f840e0
(cherry picked from commit 2b28a56d8ac9a2393924a0b201cc39bd86fc4d15)

PHPVersionCheck: Properly check for `xml` extension in PHP 7.2+

In 7.2, the `utf8_encode` function was moved into core PHP, and is no
longer a part of the `xml` extension. Use a different function to check
for the existence of it.

Change-Id: Ib50c1a59dd9c2a8777b07b2f7a3dd638a55832c6
(cherry picked from commit 9e5785b7204e6a927e7f60ce4cbdeedad8f45818)

Fix handling of ar_length and ar_sha1 in RevisionArchiveRecord.

This makes sure that length and hash are calculated if not known.

This patch also adds missing unit tests for RevisionArchiveRecord,
and consolidates unit tests for the different RevisionRecord
subclasses using a trait.

Bug: T192189
Change-Id: I5e1d17ba96e61e068b6aa5ac9c45ac0f657905a6
(cherry picked from commit 040d5b25525a9a9c8fd27d8dca7fe1b7452cf05b)

resourceloader: Disable ChronologyProtector on load.php (again)

This was already disabled in 2016 with 98c6743413118f by calling
setRequestInfo() to override the default 'ChronologyProtection' value
from Setup.php.

This stopped working since 4ccb228bde9294d96d which made
getDBLoadBalancerFactory (in ServiceWiring) obtains a LoadBalancer
from MWLBFactory::setSchemaAliases, and requestInfo is not
re-evaluated after that.

Bug: T192611
Change-Id: I68748e8f15bcd7bfa8d7921964788e2798682e05
(cherry picked from commit 0612ce28964e35e9ccb988e7372b1f40c00314e8)

RELEASE-NOTES: Fix typo in CSSMin release note

Follows-up c058bc4af3.

Bug: T189375
Change-Id: I21200dba6649d775d519249ae09cc21660dc864b

Merge "Fix and improve PHPDoc type hints in FileBackend and FileRepo" into REL1_31

Merge "resourceloader: Disable a flaky MessageBlobStoreTest case" into REL1_31

Merge "build: Switch QUnit package from deprecated 'qunitjs' to 'qunit'" into REL1_31

Merge "selenium: Abstract test command behind 'npm run'" into REL1_31

Merge "CSSMin: Fix breaking of quoted urls with outer spacing" into REL1_31

resourceloader: Disable a flaky MessageBlobStoreTest case

Sometimes fails on the mediawiki-phpunit-php70-jessie job.

Bug: T176097
Change-Id: Id8c142954d597e1efaa326795f419d48b7af8bdf
(cherry picked from commit 353f09fac98be288d933d73145cffa14773e31d8)

build: Switch QUnit package from deprecated 'qunitjs' to 'qunit'

This should fix the deprecation warning from npm in CI logs.

Change-Id: I5d02dd8d6a86123d7935db51b8d0b5d88218869f

selenium: Abstract test command behind 'npm run'

This makes it possible to use in Quibble. Right now, Quibble
cannot use 'npm run selenium' yet because it sets up its own
chromedriver, which would conflict with selenium.sh.

But, in preparation for Id95f18927b5, we can at least abstract
the test command so that it can easily be changed without having
to update Quibble and without having to worry about it being the
same in all MediaWiki branches.

Bug: T179190
Change-Id: I622ec3bf36746502cae891cc6bec23982b21f876

Merge "Make getScopedLockAndFlush() in Category::refreshCounts non-blocking" into REL1_31

CSSMin: Fix breaking of quoted urls with outer spacing

Bug:  T189375
Change-Id: I9226ec9bd21d7dedc964fbc1dc1f0914abe458d5
(cherry picked from commit 5333d63cbc3d97689c3eb0ab520abc4b264dc0dd)

Fix and improve PHPDoc type hints in FileBackend and FileRepo

Change-Id: I311b4a6777946b451d70c6fdb2133dbbd73f159f
(cherry picked from commit 645d96b0e1560e58285c88abfe8f8e726acb2e64)

Make getScopedLockAndFlush() in Category::refreshCounts non-blocking

Change-Id: I675ec38e064f57a4e960b3934c472833a945b695
(cherry picked from commit 0b5ff6454a2ae32d88bf907dac9ee3937f59e0fd)

rdbms: make sure cpPosIndex cookie is applied to LBFactory in time

Once getMain() was called in setSchemaAliases(), the ChronologyProtector
was initialized and the setRequestInfo() call in Setup.php had no effect.
Only the request values read in LBFactory::__construct() were used, which
reflect $_GET but not cookie values.

Use the $wgDBtype variable to avoid this and add an exception when that
sort of thing happens.

Further defer instantiation of ChronologyProtector so that methods like
ILBFactory::getMainLB() do not trigger construction.

Bug: T192611
Change-Id: I735d3ade5cd12a5d609f4dae19ac88fec4b18b51
(cherry picked from commit 628a3a9b267620914701a2a0a17bad8ab2e56498)

Merge "Adding CategoryTree" into REL1_31

Merge "Test ApiMove" into REL1_31

Adding CategoryTree

Change-Id: I0e6a4c44596f1cd9f063b346f32b37bfd2d920d4

Fix typo: bytePointLimit -> byteLimit

Change-Id: Ifafc4bdd6663e1ccfa6be938e5fc72fd1442a9a5
(cherry picked from commit 8ad7cd92b559632ff8f49fa55e33771670ffcf07)

Hard-deprecate wfUsePHP()

Deprecated in 1.30, no callers anywhere.

Change-Id: I5168234f4e8cfffbccc9dd43026255ee2dcd878d

Test ApiMove

One bug caught: the watch and unwatch params were not being honored.
They have now been removed.

Depends-On: Ia21a974f2b463afc9324182137b95c80db86a6aa
Change-Id: I0e214339c9ae3f0fb5a40c88a84190bc32503151
(cherry picked from commit 80c7158f06b61acf5e8d8f46ee45c4c49829837b)

Merge "rdbms: correct atomic section examples in comments" into REL1_31

rdbms: correct atomic section examples in comments

Follow-up to f0f4f5f5b820

Change-Id: I0402be7942f615f3e6f7ee3edd6f7b356a6bc3ef
(cherry picked from commit d1e70af98d46cb16b8e77c7d11ced16cfdd80d8b)

rdbms: make cancelAtomic() handle callbacks and work with DBO_TRX

Make transaction callbacks aware of cancelled sections. If the
statements of a section are reverted via cancelAtomic(), then the
dependant callbacks are now cancelled as well. Any callbacks for
onTransactionResolution(), which does not depend on COMMIT, will
see the triggering event as a ROLLBACK, since the unit of work it
was part of was rolled back.

Also fix the handling of topmost atomic sections with DBO_TRX.
These still need their own savepoint to make cancelAtomic() work.

Follow-up to 52aeaa7a5.

Change-Id: If4d455c98155283797678cfb9df31d5317dd91a2
(cherry picked from commit 20777d73998fb7024593752b0b3211d9d0e79b79)

Merge "Don't pass USE INDEX to a $dbType parameter" into REL1_31

Merge "API: Fix PHP Warning for "count(): Parameter must be an array"" into REL1_31

Merge "Deprecate $wgUseAjax in 1.31" into REL1_31

jquery.makeCollapsible: Only add '[' ']' to autogenerated toggles

These should not be added if a pre-generated toggle was provided in
page HTML.

Follow-up to 46a7aa7e3e8ea98b69ae4bd0aff51465da57bcd3.

Bug: T192140
Change-Id: I4b7a7091c1db3559a474de94f8286c0df0af8a67
(cherry picked from commit 0b593253ba57b84c7c0a7688aa889d50cbad85eb)

Don't pass USE INDEX to a $dbType parameter

Bug: T192584
Change-Id: Ia2b761bad570532696419b4f23f84bc85ff2c0e8

Deprecate $wgUseAjax in 1.31

This setting was intended to control the legacy AJAX framework, but some
extensions mistakenly used it to control any AJAX functionality at all.

But keeping this around when we're deprecating things like $wgEnableAPI
makes no sense.

[Retro-back-ported after bf61efd87 had landed in 1.32.]

Change-Id: I7b767478f9fb44236e6514b0e83228d9fb8e856f
(cherry picked ish from commit 434f280ba9b27c9f3e3b4dbeb9b260834913649d)

API: Fix PHP Warning for "count(): Parameter must be an array"

Found by the new ApiUserrights test introduced in 5ab1bee6bcf:
> includes/specials/SpecialUserrights.php:462
> includes/specials/SpecialUserrights.php:405
> includes/api/ApiUserrights.php:116

The $tags parameter of the UserrightsPage::doSaveUserGroups()
method must be an array and is not nullable. Fix the caller
to not pass null, but an empty array.

Bug: T182377
Change-Id: Ic9be6a0bddfac023de765c810c07a64d651c33b4

Change php requirement in composer from >= 7.0.0 to >= 5.6.99

Change-Id: I13e75ea955d2d0034cfbaad86ae9c9118fac6e89

Follow-up 2c74b043568e: Restore use of 'clearyourcache' message

Bug: T191800
Change-Id: I632fff6489e929847e7abc1d944277e3b0358314