Suppress "Headers already sent" in PHP 7.2 too

The "h" is now capitalized, so we need to update the regex.

Change-Id: I1111e1228868ec66d930c7a3b0d7972e5c6356b9

Localisation updates from https://translatewiki.net.

Change-Id: I37d7b1dfa4bbe393b32d62f8f8ac564ebb24d149

Localisation updates from https://translatewiki.net.

Change-Id: Ib87a638392b5f4d20ce69a0af2895692f11197d4

Fix numerals for Saraiki

Change numerals from Persian numerals to
Eastern Arabic numerals.

Bug: T202943
Change-Id: I9ef7c18d40b9aa3f1c3b3db7407d3f61feb7fd1d

Merge "messagecache: use MergeableUpdate for the deferred replace() update"

Merge "Add link to protect log to action=info"

Merge "Remove trailing spaces from IP addr in Special:DeletedContributions"

Replace Media namespace redirects with File namespace

If a user creates a redirect that goes to a [[Media:example.jpg]]
page, then an exception is thrown because NS_MEDIA is a virtual
namespace. This change catches this case and changes the namespace
to an NS_FILE namespace and the redirect works correctly. This
change only happens when we are dealing with a redirect so other
uses of the NS_MEDIA namespace shouldn't be affected.

Bug: T203942
Change-Id: Ia744059650e16510732a65d51b138b11cbd43eb4

messagecache: use MergeableUpdate for the deferred replace() update

This combines the load loop for multiple messages for a language code.

Bug: T203925
Bug: T193271
Change-Id: Ie5e1e83d6740344b7ca641c99fb3bd4ad5718492

Localisation updates from https://translatewiki.net.

Change-Id: I16181fad87c2d0afe3c43188c9f7807b6a953698

Merge "UIDGenerator: Remove the clock skew problem"

Merge "Write Latin and other scripts with captial letter"

Merge "search: Fix DYM typos in widget"

Write Latin and other scripts with captial letter

Change-Id: I16c660e54191b63cd6eb3407cb00504665930c4e

Remove trailing spaces from IP addr in Special:DeletedContributions

* Trim "target" to remove trailing spaces from IP address in
  Special:DeletedContributions that triggers MW internal error.

Bug: T187619
Change-Id: Ic6b0d8020553ecce4dcf97f9c78487d3174444d8

Merge "wdio-mediawiki: Add 'fragment' parameter to Page"

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

Merge "Test ApiUnblock"

Merge "Fix List* in MessagesKo.php"

Test ApiUnblock

Full coverage (except for the examples/help URL).

Change-Id: I4c618506b5446680f06fa9b605d3babe7194de94

Merge "filebackend: Add normalization for stat errors"

filebackend: Add normalization for stat errors

Bug: T205567
Change-Id: I75f1eb6dc2cbff0ea0dc0706cca0ad79c54fc612

Merge "Enforce no-session constraint in opensearch_desc.php and profileinfo.php"

Localisation updates from https://translatewiki.net.

Change-Id: Idf339b32a190b7a770c0465e4545948861ab979d

Merge "Clean up UIDGenerator field comments"

UIDGenerator: Remove the clock skew problem

In a nutshell:

This commmit removes the use of drifting microtime()
in favour of non-drifting time(). Then, we increase the size of
the counter by x1000, and consider every 1000 increments as 1ms
for the purposes of UUID.

Why:

This means we eliminate the whole code that can produce a fatal
exception when the clock drifts by more than we can wait for,
which puts us in a logical conundrum we can't get out of, hence
it previously fatalled.

Not aborting random end-user requests and jobs is good.

This also means we avoid the vast majority of cases where
MediaWiki would busy-loop for upto 10ms in a likely-to-fail
attempt to sync the clock. This means the method runs faster,
which is a nice win, albeit not the main objective.

Bug: T94522
Change-Id: Ia8a847078ec76d633854db6823a20f0961c80f80

Add link to protect log to action=info

Bug: T52190
Change-Id: Id983d4585a253ebe34e07f07472548f6d1d20bce

Merge "Update OOUI to v0.29.1"

Merge "Avoid fatal when finding no base revision for a null revision."

Avoid fatal when finding no base revision for a null revision.

Bug: T205675
Change-Id: Iae67649a1be9597086033ad34d9d00556ba35730

Update OOUI to v0.29.1

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.29.1

Local fix to rcfilters for renamed parent method.

Bug: T130434
Bug: T165650
Bug: T183299
Bug: T196756
Bug: T205472
Bug: T205692
Bug: T205979
Bug: T206153
Bug: T206155
Depends-On: Ie847465af6762fd0322a5be4adf601cabda2d458
Change-Id: I704e3197e86d484be3ab91b9282d6ac1b7db2251

Merge "ParserOutput::getCacheTime should stay the same after the first call."

Merge "Ensure OutputPageTest works when Translate extension is loaded"

Merge "Use Remex for TextContentTest subclasses"

Merge "Disallow overriding services that were set"

Merge "tests: Allow string to be passed to getTestUser() etc."

Merge "Minor cleanup in ApiBlockTest"

Merge "Only use "*Test.php" for actual PHPUnit tests"

Merge "Re-enable tests from TitlePermissionTest"

Merge "registration: Let extensions add PHP extension requirements"

search: Fix DYM typos in widget

* DYM link was triggering the 'go' feature due to fulltext misspelled
* all ids are of the form mw-search-*, except one. Make it the same
  (WikimediaEvents code was expecting everything to be mw-search-*)

Bug: T206232
Change-Id: Ib91ccf28879b8af2a8c0f2a40b9fe3ec4259a730

tests: Allow string to be passed to getTestUser() etc.

If there's only one group, let's not require the brackets.

Change-Id: Ib46ef1facbbbf2ba01c13b8290ebbe3a8e08cb4f

Minor cleanup in ApiBlockTest

Use tablesUsed instead of cleaning up ourselves.

Change-Id: Ibe470a8f292d65ceeaa098d5c033b8585501a9f3

Localisation updates from https://translatewiki.net.

Change-Id: I3a709a2ec2f0e13ca2b8015aa9de36e9e7d4e063

ParserOutput::getCacheTime should stay the same after the first call.

Previously, getCacheTime would default to the current time, potentially
causing the return value to change over subsequent calls. With this change,
the value is determined on the first call, and then remembered for subsequent
calls.

Bug: T205464
Change-Id: If240161c71d523ad5b0d33b9378950e0bebceb6e

UIDGenerator: Misc clean up

Various miscellaneous clean ups with no change to any of the
logical code. This to make the next commit have a cleaner diff
for easier review.

* Make internal millitime() non-static.

* Improve documentation and add missing @covers annotations.

* Correct getTimestampedID88() documentation to state that
  only two values need to be passed, not three.
  This is significant because the Flow extension is actually
  using this method in a subclass and passes only two values.

Bug: T94522
Change-Id: I2a0c51bea58df4cc0c253c1c10de3ac383f04c8e

wdio-mediawiki: Add 'fragment' parameter to Page

In some cases it's useful adding a fragment to the url, e.g. when
accessing the beta features preference page. This patch adds support
for that on the Page object.

Change-Id: I4b6aa5e7e89e60f05e217fc557bf6786441855e6

Clean up UIDGenerator field comments

Change-Id: I0c6d125ec03a56fe0b3456cff7d498d231315e0d

Phabricator: Use Tddddd instead of Bug ddddd in comments

Change-Id: Ic9fe03cab270bd6be738af346164ad5d31a0d780

Merge "rdbms: clarfiy some comments about commitMasterChanges() methods"

Merge "rdbms: clarify IDatabase::setTransactionListener() comment"

Merge "MessageCache: do not store the EXCESSIVE array as it is only needed for HASH"

rdbms: clarfiy some comments about commitMasterChanges() methods

Change-Id: I2c9f3cb9b4dcd16f561e55d119849b526e9d402b

MessageCache: do not store the EXCESSIVE array as it is only needed for HASH

This saves space as the number of entries becomes larger.

Bug: T193271
Change-Id: I32d7512d7f05ae7c9a69b88cd19df64d7a21b4a1

Merge "Fix option name in maintenance/importDump.php"

Special:Preferences: Drop isOouiEnabled() before it gets released

Depends-On: Ic1df4e965df61c2d164b2ffd2fa07ed68b028ddf
Depends-On: If0e83751089d010d026a736040d77b9a0851ef42
Depends-On: Id9c840e979b723806883bb3e63d7f2f691fea629
Depends-On: I9d5dbf449c624233cef2ccf0e3e4200d9eb2b7c3
Change-Id: If2a21a6a40802ab187b30de78fb521f016dd10cd

Merge "Special:Preferences: Drop non-OOUI legacy form version"

Merge "Drop 'SpecialRecentChangesQuery' & 'SpecialWatchlistQuery' hooks, deprecated in 1.23"

Merge "Database: close() should not commit transactions"

Make UID clock drift error have more details.

Change-Id: Ic682e48b06161c2a286f5da9016f4831ee72a31e

Move test assertion to mirror parameter order

Follow up from I2df0551c5837adc578b27082ab6ba2ac95d937f8

Bug: T206130
Change-Id: Ib669c77fdb709846d0182cb28796cf53914114c4

Merge "SECURITY: Fix permissions check for patrol action"

Enforce no-session constraint in opensearch_desc.php and profileinfo.php

Attempts to use session data via these endpoints will now cause
exceptions to be thrown.

Bug: T127233
Change-Id: I28b080a84e7b928e4336a5a1ec770bb57b05d009

SECURITY: Fix permissions check for patrol action

Return existing errors instead of empty array in checkUserConfigPermissions().
Returning an empty array wiped out previously-found errors.

Also add test coverage for patrol action.

Bug: T206130
Change-Id: I2df0551c5837adc578b27082ab6ba2ac95d937f8

Database: close() should not commit transactions

Transactional databases normally roll back when a connection is closed
with an open transaction rather than committing them, so MediaWiki
committing them is unexpected.

There are two cases being changed here: automatic transactions without
writes and manual transactions. For the former it shouldn't make a
difference if we commit or roll back since no writes were done anyway.
The latter has logged a message since MW 1.31 (I0992f9a8), and that
warning has not been logged in Wikimedia production in the past 60 days
so we should be ok there too.

Bug: T206147
Change-Id: Ieceef4deb49044db8f0622d38ee76c9d9f39704c

Fix option name in maintenance/importDump.php

Bug: T200138
Change-Id: Icc5d9878bc3ebdf03efec9161b220ad0bf55dc9a

Merge "Show copyright based on $output->hasCopyright()"

Merge "Remove unused function: isStructuredFilterUiEnabledByDefault"

Only use "*Test.php" for actual PHPUnit tests

Otherwise I get errors every time I try to run PHPUnit on includes/ or
includes/parser, because it tries to run ParserIntegrationTest.php and
fails.  Apparently the <exclude> in suite.xml doesn't work if PHPUnit is
invoked on a directory.

Bug: T201278
Change-Id: I7d09576bee2705d8516152e8fa671da8dac40233

Show copyright based on $output->hasCopyright()

Introducing setCopyright/hasCopyright in OutputPage to
make the showing of the copyright message controllable
by pages other than articles.

hasCopyright defaults to isArticle() to preserve the rule
that all article should show the copyright.

This is used by Flow to show the copyright
on various pages that contain user-generated
content.

Bug: T184960
Change-Id: I3a50dbcedc6b119b9262c50cb3a84b0dd230fb3d

Localisation updates from https://translatewiki.net.

Change-Id: Ic0af665f26aa1af0bd706ee26bb811d0a482ec2b

Re-enable tests from TitlePermissionTest

Let's see if they work now that services are reset between tests.

This reverts commit 7f843b0c0472b9da909d7abf7c2829eec17a5b3e.

Bug: T201776
Change-Id: Iea7c74f8c77a97d83385b4399e500cf8129a1158

Fix List* in MessagesKo.php

The primary one does not state it is "list". Fixes that where that
is a primarily a list.

Change-Id: I56374841b972be71d25ffa10b5dc33cc3ea3b140
Signed-off-by: Yongmin Hong <revi@pobox.com>

Deprecate Language::setCode as public method

setCode changes the language code for the Language object but it also
replaces the whole language codes for all Language objects.

> $lang = Language::factory( 'fr' )

> $lang2 = Language::factory( 'fr' )

> $lang->setCode( 'it' )

> print $lang2->getCode()
it
> $lang3 = Language::factory( 'fr' )

> print $lang3->getCode()
it

Better assign a new Language object.

Also add more tests for Language::equals.

Depends-On: I61439bac82021344c3f9a6056cccd937b3450af2
Depends-On: I2d9e551d6eb33f28f42aeaf48160eba21b83881f
Change-Id: I201b479f58e63c9c40fb8a3ec9575a551fb35235

Merge "API: Handle empty xxnamespace parameter in ApiQueryBacklinksprop"

Merge "AutoloadGenerator: Filter PSR4-compliant classes instead of ignoring directories"

Merge "Migrate image descriptions from image_comment_temp"

Merge "RCFilters: better vertical alignment of checkbox and text in menus"

RCFilters: better vertical alignment of checkbox and text in menus

- Remove min-height so menu items are a little smaller and
  their content is vertically-aligned.

- Force `vertical-align: middle;` for checkboxes by overriding
  OOUI with extreme specificity.

Bug: T200364
Change-Id: If0b2b5c1b749f913badbd0c02a85852ec0007020

Merge "MessageCache: remove confusing and unused $isFullKey parameter from get()"

MessageCache: remove confusing and unused $isFullKey parameter from get()

Follows-up cba0fb1c1576324e87b27, which removed the last caller.

Change-Id: I00c17fedff39b1b35519cff2a0f8eac3e4d6f2ab

Language: Don't return aliases to namespaces that don't exist

On a multi-wiki server using a l10n cache, the cache will probably
include aliases from all extensions enabled for any wiki while the
namespaces themselves will only be defined on the wikis where the
extension is loaded.

Bug: T206030
Change-Id: I985dfa3eb17944d5fc0c5be0979be21504fb1341

Merge "rdbms: make * consistently act like  in select/insertSelect methods"

Merge "Add audience parameter to PoolWorkArticleView"

Disallow overriding services that were set

Otherwise setService() calls (including indirect, like via
setContentLang() ) will be silently overridden.  This bit me several
times already while working on tests.  If someone actually wants to do
this, they should probably split their test in two, because it evidently
has two separate phases that they're testing (with/without services
set).

Depends-On: I9acb81c0de95eb5a6bed543d757ae62523ea6041
Change-Id: I8c60e37c179320e61684cbc11281c509e525e8fb

Migrate image descriptions from image_comment_temp

image_comment_temp was always intended to be temporary, until an
expensive schema change on Wikimedia Commons's image table could be
done. Now that that has been done, stop writing image_comment_temp and
add a migration script to copy existing data into img_description_id.

Ic8efeddc will remove the reads from image_comment_temp and drop the
image_comment_temp table.

Bug: T188132
Change-Id: Iab5f521577a415b2dc213b517ee8a0dca4fdd0aa

Merge "Use pseudo elements to style changelist-separators on Contributions"

Add audience parameter to PoolWorkArticleView

The old behavior was that the audience was RAW if the revision object
parameter got passed in, otherwise PUBLIC. This was undocumented and
not used outside core; this patch gets rid of it in favor of an
explicit argument.

Bug: T205578
Change-Id: Ic7cdb38f658f6d85c48ff13c7f84c64a45c9b1ee

Merge "Add type hint to invalidateImageDescriptions()"

Add type hint to invalidateImageDescriptions()

Change-Id: Ie6322f25c959f5de4bbccd8f47d40f112817d00b

Merge "Replace WikiExporter streaming (unbuffered) mode with batched queries"

Drop 'SpecialRecentChangesQuery' & 'SpecialWatchlistQuery' hooks, deprecated in 1.23

Depends-On: I9822bb2bc1bcc13eec7924208bf3819d3512ad26
Change-Id: I67c06bb99d9c9582a638f061f523661c821bca8d

Merge "ApiOptions: fix resetting some preferences to default"

Merge "Use varargs in global functions"

Special:Preferences: Drop non-OOUI legacy form version

Change-Id: I65b89385c3ec28ef01b86dd933dae3801e503631

Remove unused function: isStructuredFilterUiEnabledByDefault

Follow up Id3799fefd21cd9bea0e089a5e12576ee9ea1085e

Bug: T196033
Change-Id: Ib2ad41be8648f7ff8b4a218fd66c328dee3924d9

CategoryTagItemWidget: Drop backwards-compatibility "Capsule" name

Not used for months.

Change-Id: Ie85d2d0ef6ebc3e1e2502519a02b8fadd8d53b7a

Use pseudo elements to style changelist-separators on Contributions

Currently, these separators are unnecessarily hardcoded in HTML.
This is problematic, as it limits the extent to which skins such
as Minerva and Timeless can skin these pages.

If the pseudo selector is not available (e.g. legacy browsers)
this degrades nicely to whitespace and the page is still useable.

Right now, these changes are scoped to pages uses the ContribsPager
(used by Special:Contributions)
but we can adopt them in other pages once we have this approach set in
stone (CSS)

Bug: T205581
Change-Id: Ia2c485e6058ebf4282ed2f7eb7b046b2deb03f57

Hard-deprecate password handling in User

Add wfDeprecated for User::checkPassword(), User::setPassword(),
User::setInternalPassword() and User::checkTemporaryPassword().
With AuthManager mediating between the caller and a set of
authentication providers in a potentially multi-step process,
a password check or change now cannot always be expressed
with a single-step succed-or-fail process. Callers should use
AuthManager::beginAuthentication() with a PasswordAuthenticationRequest
for password checks, and AuthManager::changeAuthenticationData()
for a password change.

Bug: T91699
Change-Id: Ib0ae8f1ff10ae6c2655d529db8b3a32e0cb489b0

Merge "Remove temporary rcfilters variables"