Shorten long lines in Html.php to make phpcs pass

Change-Id: I5060b510105aa08f1cb4804e2c33d8e1b00ef390

Shorten long lines and use consistent tabs in DefaultSettings.php

To make phpcs pass.

Change-Id: Iff20470d1db409ed91577058204d0254c461ba76

Resources.php: Simplify dependencies to single modules

Change-Id: Icaa81662243259f166a28e3b6bb5b8ac6fea4bb3

Shorten long lines in Language.php to make phpcs pass

Change-Id: Ied04ebc64cd5a6e54bd0c3de5845342ec89d4d1c

Merge "resourceloader: Clean up UserModule to be more like UserGroupsModule"

Merge "Text color for quiet buttons should not be lightened"

resourceloader: Clean up UserModule to be more like UserGroupsModule

Change-Id: Ic988c36c034b2862e6404bb6b6b7fe24d171fef1

Merge "API: Handle invalid titles in action=feedwatchlist"

Revert "Made insertMessageBlob use (start|end)Atomic in case DBO_TRX is off"

Per T91399.

This reverts commit 80a3a30d6a90b21f68cca4dfe5b3623cb9e5e7fc.

Change-Id: I2aacfbac2ab25a4662d6bee544e3f953770dc925

API: Handle invalid titles in action=feedwatchlist

When new interwiki prefixes or the like get added, it can cause existing
watchlist entries to become invalid. Let's handle this more gracefully than
"Fatal error: Call to a member function getFullURL() on a non-object".

Bug: T44274
Change-Id: I9476fa7a86aaae810b8c0c2da08a9317451a0bdf

Merge "mediawiki.user: Simplify extension of mw.user"

Add explicit array cast in Language::getGenderNsText

Similiar to r102414

Bug: T43194
Change-Id: I868b31b3862539d4782912d5b6e5d3ce5d9b4f60

Merge "Add hook EnhancedChangesList::getLogText"

Made insertMessageBlob use (start|end)Atomic in case DBO_TRX is off

Change-Id: I6b730e8b4a6cb2d4a82a0cf471c7625ccfd08a0e

Merge "Populate User::$idCacheByName in User::loadFromRow() if possible"

Merge "Make sure hash_hmac() always works even if no secret key"

Add hook EnhancedChangesList::getLogText

Enhanced RC generates these "(3 changes | history)" links for
every block of grouped recentchanges. That changes-link links
to a diff page.

For Flow, that is all wrong: we have different ids (not integers),
on a different page (&curid=&oldid=&diff= means nothing). Even
the concept of a "diff" page seems wrong here for us - a new post
is not part of some document that can be diffed.
In short: we'll want to generate a different link, and we'll need
a hook to let us change them.

Meanwhile also split the code that generates those links into a
separate method.

Bug: T72513
Change-Id: Ib32fb9552b80f9581d89b3b47da6e5d32e3d84a3

Make "name" a required value of extension.json in the schema

The code considers it to be required, so indicate that in the schema as well.

Change-Id: I9c1a279d5675eb10210a3dba5a15fe169619d719

Merge "Sync up with Parsoid parserTests."

Text color for quiet buttons should not be lightened

This is part of the effort to synchronize button styles between
MediaWiki UI and OOjs UI. According to the designers, the text
colors for quiet buttons should be the raw non-quiet equivalent
button background colors (not lightened).

Bug: T88449
Change-Id: Ib6ff7167a62c03d0b0427a7161d396e11e173046

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 7608aeabdc1e47ce3dbfffd3ded85eecad647260

Change-Id: I4cb8f04b9d7988e5138c8f040fadeb0b7a9348f9

Populate User::$idCacheByName in User::loadFromRow() if possible

Change-Id: I4c7dab517ca6b7adcdec0c945e21d6be0b7b7d66

Clarified RequestContext doc comment

Change-Id: Ibb2a30634b192f4bc77e0b6b1afebc65b2ed3ec9

Merge "Explicitly mark HTMLCacheUpdateJob jobs that are recursive for clarity"

Localisation updates from https://translatewiki.net.

Change-Id: Ib5e92f58763d20ad7f75b3631ebd1db2b1a79ffe

Merge "Hide section anchor links from screen readers using aria-hidden"

Merge "Made SqlBagOStuff avoid tripping TransactionProfiler"

Merge "$timeout was optional in Http::get()"

Merge "Moved JobQueueFederated partition exceptions to their own log"

$timeout was optional in Http::get()

Change-Id: I6cd2918412395d1d5f811f0d47bb080f22b598ec

Moved JobQueueFederated partition exceptions to their own log

Change-Id: If495e9b1a631b587cb3b8d83974837eef1d2d4c6

Merge "Remove $timeout parameter from Http::get()"

Merge "installer: Remove unused references to wgResourceLoaderMaxQueryLength"

Remove $timeout parameter from Http::get()

It is inconsistent with request() and post() and there's a couple
of mistaken usages in core and extensions that think they have the
same signature.

Change-Id: I834278639a3648edec7bcb57db7bb61f456d2a92

API: paraminfo html strings shouldn't include unnecessary <p>s

If the output is a single paragraph, there's not really a need for
explicit <p> tags around it.

Bug: T84995
Change-Id: I4406121fe0ec099c4a4ade849ef495edad90c2ad

Hide section anchor links from screen readers using aria-hidden

Hide from screen readers, as it is inside a heading (the programs use
headings to navigate, so the symbol is being read during navigation).
Pending T13555 being fixed.

See <https://phabricator.wikimedia.org/T18691#1076371> and
<https://phabricator.wikimedia.org/T18691#1076857> for reasoning.

Bug: T18691
Change-Id: I736cb7f52d5ce37eb7e8d3a3648ffd0bd40f2c83

Allow a lonely "-" as positional argument

Change-Id: I658988c8f955103d6bc4bd1047819a37de0d505d

Merge "resourceloader: Let $wgUseSiteCss protect MediaWiki:Print.css as well"

Localisation updates from https://translatewiki.net.

Change-Id: Iee49f1dd94d5cc94647b7b47277aee2625db56e1

resourceloader: Let $wgUseSiteCss protect MediaWiki:Print.css as well

Follows-up 12a9f0564d1af13.

Change-Id: Id67f98e9d624191d69def262763dbba33050606f

Add composer lint, phpcs, and test commands

lint uses jakub-onderka/php-parallel-lint to check the syntax of the provided
PHP files or directories.

phpcs uses the MediaWiki codesniffer standard to check the provided files or
directories.

test runs both lint and phpcs together. phpunit is not included at this time
because our phpunit tests require a database to be set up and are very very
slow.

Example usage:
  composer lint .  # Lint all files
  composer phpcs includes/FooBar.php  # Check code style of one file
  composer test extensions/FooBar  # Lint and check an extension

Change-Id: I3c7ac7a02668776de6400ac268f37e7e6b35a3c6

Localisation updates from https://translatewiki.net.

Change-Id: Id79fa4748516563b5c8429e08e7edce047b97963

Merge "refreshLinks.php: Tweak exit condition in deleteLinksFromNonexistent()"

installer: Remove unused references to wgResourceLoaderMaxQueryLength

* Added in 40e18e45343.
* Removed in 648bed9f839 (r87494).

The latter left the Installer::values array still populating this
variable eventhough it was no longer used in the generators output.

The example in overrides.php is also outdated as modifying this
variable there would have no effect.

Change-Id: I0c86e94a004c034702c5fcd83257c0f4c3d15a57

Merge "LivePreview: Wrap content preview with mw-content-*"

Avoid access to array key that does not exist

Accessing an array element that is not set
causes a PHP notice. This change first, checks if the
array key is present.

Bug: T91127
Change-Id: I468a95851e6acdb8186a06b0a2ac73499cc4611f

Merge "Escape messages 'word-separator' and 'parentheses' in InfoAction"

refreshLinks.php: Tweak exit condition in deleteLinksFromNonexistent()

Instead of exiting the do...while loop only once a query returns zero
rows, exit whenever fewer rows than the batch size are returned. This
could save quite a bit of time when the highest nonexistent page_id
found is a relatively low one.

Follows-up 40e300b8273d.

Bug: T44180
Change-Id: I14d2d48c2405fcc0bd05a3181ba6293caef5298c

Merge "Gracefully handle redirects in SpecialMyLanguage"

Make sure hash_hmac() always works even if no secret key

Otherwise a fatal error is possible.

Change-Id: Icda96bac3e75f424be068cdad30ad618b503a8e1

Localisation updates from https://translatewiki.net.

Change-Id: I853bf0e029de0348e1ecdcd5cfa624cb12a68947

Merge "Made ConfigException extend Exception"

Made ConfigException extend Exception

Change-Id: Ic63c9cbfcb653a2fb39445a3b6ec4b88d8084d7a

build: Work around jshint failure for global JSON object

jshint is now failing with errors about how it doesn't
recognize the global 'JSON' object, even though we have
browser:true set. This worked in 2.6.0 but broke in 2.6.1.

Other people have reported similar but different problems with
jshint: https://github.com/jshint/jshint/issues/2213 .
It looks like 2.6.1 is a seriously broken release, but we
can't work around it in package.json because grunt-contrib-jshint
depends on 2.6.*, assuming that point releases are backwards
compatible. Which they should be.

Change-Id: I85bcf1c97f2f8c82e139ebf2f1b6c4248b72f45e

Update OOjs UI to v0.8.2

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

Change-Id: Ie6722b1aae1970d628a82c491c8d525768d01925

Made SqlBagOStuff avoid tripping TransactionProfiler

* Set a custom profiler with no expectations to avoid tripping
  the "0 write" expection. This avoids useless log entries.

Change-Id: Iac849a729eb36b1a8affb0dbc8b8c195fab4b03a

Merge "installer: Use HTTPS links for licenses"

Merge "refreshLinks.php: Get IDs in batches in deleteLinksFromNonexistent()"

Merge "Fix fatal "Call to setExpectation() on non-object" in MediaWiki::main()"

Merge "Fix testGetCanonicalName PHPUnit test:"

refreshLinks.php: Get IDs in batches in deleteLinksFromNonexistent()

... instead of making an unbuffered query, which is discouraged in the
doc comment for DatabaseBase::bufferResults().

Also used NOT IN for the antijoin instead of LEFT JOIN...IS NULL; when
combined with DISTINCT, the latter causes MySQL to use a temporary
table rather than an appropriate index, according to EXPLAIN. (Using
GROUP BY instead of DISTINCT also avoids this problem. I don't know why.)

Bug: T44180
Change-Id: Idca85fac7dd7879f9fbef2712b6aa83343099e02

Merge "New convenience method ApiBase::lacksSameOriginSecurity()"

New convenience method ApiBase::lacksSameOriginSecurity()

For consistent handling of API requests with the 'callback' argument.

Change-Id: Ic6e3483f5e8819498c693650a11728efc1bafcc0

Fix fatal "Call to setExpectation() on non-object" in MediaWiki::main()

Follows-up b403a757726e, 2762888e0d0ee50d.

Caught by the qunit tests via Special:RunJobs. The test is still
work-in-progress (T50002) but it's already paying off :)

Change-Id: Idb37ecb97ff77950a981bfe10a46f27e532e1abb

Merge "Introduce HashSiteStore, mainly for use in tests"

Introduce HashSiteStore, mainly for use in tests

HashSiteStore comes from Wikibase (see I783bd95), where it was
called MockSiteStore.

This enables some phpunit tests, related to Site objects, to no longer
depend on a database, memcached or other external storage. This makes
tests faster and more simple.

Bug: T90874
Change-Id: I048d37bd2aaa5f17c9fe16b2855df8bf9fe7bc8c

Fix testGetCanonicalName PHPUnit test:

* assertEquals takes expected, prior to actual
  https://phpunit.de/manual/3.7/en/writing-tests-for-phpunit.html#writing-tests-for-phpunit.assertions.assertEquals
* ' trailing space ' test was failing with wgCapitalLinks false.  Since the test is not about that,
  I changed it to ' Trailing space '.  The alternative would be to force wgCapitalLinks to true
  with setMwGlobals or skip the test if it's true.

Change-Id: I677273045bdabb6248ffe2c2f003a3673b3fa51c

Merge "Sync up with Parsoid parserTests."

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 47ddba73a5f6078ac265f091d9c9e841c0fdead6

Change-Id: I07bd6edd2d62291f75bea79ab7825e390ccb314e

Gracefully handle redirects in SpecialMyLanguage

Currently if you have a link that goes through SpecialMyLanguage
and that page is redirected you will no longer get it in your
language. This adjusts to follow redirects before resolving language.

Bug: T58036
Change-Id: I88fcfeb86d81a33a8eccc6119693c2ecb857afb2

Made "maxAffected" query expectation actually apply

Change-Id: Ib7796fe0975b50b96c5e1d53384079735eea881d

Merge "Made insertMessageBlob only catch DBError for sanity"

Made insertMessageBlob only catch DBError for sanity

Change-Id: I1614e7fde052e7b6f9d86d86af6eed1ba52dc430

Localisation updates from https://translatewiki.net.

Change-Id: I49e15ff89e58153a9ce674de36127c9ed52e1ad9

Escape messages 'word-separator' and 'parentheses' in InfoAction

This change will not break the usage of the messages, because there are
usually used escaped in mediawiki/core.

Change-Id: I049134e2fbfadab04ac228090d17fd18c5baca3d

Merge "Escape extra input messages on Special:UserLogin/signup"

PostgreSQL: Fix timestamp in Special:ListFiles

The database field img_timestamp needs to converted
from database timestamp format to the wf internal format.

Bug: T72931
Change-Id: Ifbac6805a43a809962372016668c3e41fcaeb7c0

Add UserArray::newFromNames()

Change-Id: I91bb8349dd846e982be9294c7f4f3858995ad2eb

Merge "Add srcset for images in <gallery>"

Add a hook run after EditPage::attemptSave for WikiEditor

So that we can determine whether a save attempt succeeded or failed,
to log saveSuccess and saveFailure events from the server to Schema:Edit
on meta.

Bug: T88027
Change-Id: Ib861262603872e67600d1aab9bde3b58a8dd1738

Merge "Exclude null rows from Special:MediaStatistics"

Escape extra input messages on Special:UserLogin/signup

TitleBlacklist and AntiSpoof can register extra input fields with a own
message, this message would now be escaped to avoid use of raw html.
Messages: 'titleblacklist-override' and 'antispoof-ignore'

The messages are now handled similar to checkbox 'createaccountmail'

Bug: T85864
Change-Id: I14326c3844904560b66eb8e8eb7d36706caa47da

Update OOjs UI to v0.8.1

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

Change-Id: I5ad8d6aac0fb4ef146ef4f36459e4b0e398a66e8

Update OOjs to v1.1.5

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fcore.git/v1.1.5/History.md

Change-Id: If43d0b4704f95abe4a0438a0d5b0756eb77cc95f

Merge "Allow configuring the transaction locking mode for SQLite"

Allow configuring the transaction locking mode for SQLite

Bug: T89180
Change-Id: Ib18347299ea78f9b31e56313b2acf749f693cddb

Add srcset for images in <gallery>

Bug: T64709
Change-Id: I476beb6cf072c5a38570143d1f56baa3f49d23d4

Made the SQLite installer make a separate objectcache DB

* This avoids severe lock contention if CACHE_DB is used and also makes
  cache sharing between wikis simple (keys are always prefixed as needed).

Bug: T89180
Change-Id: I864272af099a9ebe2b3648e2714d2ccdee20305d

Exclude null rows from Special:MediaStatistics

The commons db has a random null row in the image table, adding
a non-sensical row to this report.

Bug: T74535
Change-Id: Ie290e588af56aee7d5303954202f7923b64d2272

Localisation updates from https://translatewiki.net.

Change-Id: I13f209cdcc50ab68829450d27e64a0f6c6d0f002

Merge "Always decode Blob objects from Database::addQuotes"

Merge "PostreSQL: DatabaseBase constructor cleanups"

PostreSQL: DatabaseBase constructor cleanups

Change I29b79a3c1bb2b5e51c1c8f5e04cd08c71e0662a3 rendered
it uninstallable on PostgreSQL.

Make the corresponding API changes to PostgreSQL installer.

Bug: T90288
Change-Id: I2019893ad705d99f3dbbce13690cd808cfb71d4a

Run maintenance/generateLocalAutoload.php

I thought a unit test was supposed to fail if this hadn't been run, but
I3936417b and I40fd6aaa recently got merged despite it.

Change-Id: Idb9b4b77a4a1c82ed74ec829fad8c12a9c39af09

Don't alias tt special pages to titles with double underscore

Such titles are invalid, as far as I can see. I get "no such special
page" at https://tt.wikipedia.org/wiki/Special:Unusedimages etc.

Change-Id: Id9869a9af5c67c18af78a925c8c4fdbaef062d32

Merge "Start moving objectcache into libs/"

Merge "Avoid calling rollback() in handleWriteError() if there is no trx"

Merge "Fix parameter order for block logs"

Start moving objectcache into libs/

The base BagOStuff class and some of the implementations are
not dependent on MediaWiki and can be in the libs folder.

Change-Id: I299f9abb778abc65a70461a2aff3015c02a1f04d

Allow subscribing to module loading exceptions

Provide access via mw.track to module loading errors which are
otherwise swallowed by mw.loader.

Bug: T513
Bug: T85263
Change-Id: Ifc13c207af36dc56be931a3b140063354513a4ec