Merge "Add anchor "mw-oldid" for beginning of page content in diff view"

Add anchor "mw-oldid" for beginning of page content in diff view

* #top, #mw-content-text and others exist, but all point to the
  beginning of the diff, not of the actual page content.
* Pick mw- prefix and technical but familiar related name, in
  the hope it's not confusing nor too hard to remember.
* Lowercase letters, similar to #top, #footer, #file etc. were also
  considered and discard, inspired to usual names:
  https://www.wikidata.org/wiki/Q10966628

Bug: T4313
Change-Id: Ia90f2541327480535e9943ac30bfa19791a19620

Merge "Remove unused alias/layer of test abstraction wfShellMaintenanceCmd()"

Merge "Moved some JobQueueAggregator logic out of JobQueueGroup"

Remove unused alias/layer of test abstraction wfShellMaintenanceCmd()

Change-Id: I28d303f9b89ffad052d06eb2dbb9e6f8cfaeea24

Merge "Removed obsolete "containsOldMagic" code"

Moved some JobQueueAggregator logic out of JobQueueGroup

Change-Id: I28ba1a25db225d4cf5f503a6c0f4405f13118151

Merge "Delay language conversion cache construction until needed"

Merge "Removed overzealous caching from JobQueueFederated"

Merge "Profile all external HTTP requests from MW"

Merge "Made master connection expectations actually work"

Delay language conversion cache construction until needed

Instead of instantiating this on every single request. Removes
wfGetLangConverterCacheStorage() and $wgLangConvMemc which were
otherwise unused.

Change-Id: Ic500944a92c2a94bc649e1b492c33714d81dca00

Merge "Allow profiling of Http requests"

Profile all external HTTP requests from MW

Change-Id: Ie980b080da2ef21ec7d9fc32f1accc55710de140

Allow profiling of Http requests

Http::get/set/request() now all take a new $caller parameter
which is like $fname in databases. Pass it __METHOD__ so all
of your external requests can be grouped together in profiling.

Change-Id: Ibad219452903a9678378044595cff1231bf605d8

Shorten long lines in UserRightsProxy.php to make phpcs pass

Change-Id: Ib9c2b2ed4bdfe895764a3af6444e48522f4808d7

Shorten long lines in User.php to make phpcs pass

Change-Id: Id0938687439d046b98d7b3a4b7666ff7ae41760a

Allow dumping raw xhprof data for consumption by xhprof GUI

Change-Id: Iab90cef1c61b92ffc6d46a6bc93a03cf7bc2adb9

Merge "resourceloader: Implement '$pages' parameter to ResourceLoaderWikiModule constructor"

Merge "mediawiki.user: Check crypto.getRandomValues before use"

Merge "Shorten long lines in MovePage.php to make phpcs pass"

mediawiki.user: Check crypto.getRandomValues before use

Firefox in version 16 implements crypto, but not function
getRandomValues, which causes a JavaScript error.

Bug: T91378
Change-Id: If4320e83af74bb1e90cc16d739d143a4bac7471b

Merge "Shorten long lines in Linker.php to make phpcs pass"

Merge "Shorten long lines in Html.php to make phpcs pass"

Merge "Shorten long lines and use consistent tabs in DefaultSettings.php"

Shorten long lines in MovePage.php to make phpcs pass

Change-Id: I4b6b2975f8e2b9508f0a78fb176093974c935cf1

Shorten long lines in Linker.php to make phpcs pass

Change-Id: Ia3376560dfd321252babc2ac3edd88764a16bce1

API: Add caution to docs for ApiBase::getCustomPrinter

Change-Id: I37737cfd65fffa4f19ad35d3443e881acc4918db

Merge "Removed old HTMLCacheUpdateJob b/c code"

Shorten long lines in Html.php to make phpcs pass

Change-Id: I5060b510105aa08f1cb4804e2c33d8e1b00ef390

Merge "Fix and re-enable Dumps' checkpoint tests"

Merge "Allow to set stub read buffer size for TextPassDumper"

Shorten long lines and use consistent tabs in DefaultSettings.php

To make phpcs pass.

Change-Id: Iff20470d1db409ed91577058204d0254c461ba76

Made master connection expectations actually work

Bug: \96T86862
Change-Id: If918904c6c02ea83473dfaf34dfc2787ab610c6f

Merge "Removed pointless memcached JobQueueAggregator class"

Removed pointless memcached JobQueueAggregator class

* On a basic/default install there is only a single wiki
  and nothing uses this. Larger wikis would want to use redis.

Change-Id: Ie5bf1a644ae60b2c6ca72b165fa5510113717611

Merge "Made JobRunner bail sooner for bogus job --type parameters"

Removed old HTMLCacheUpdateJob b/c code

* Also supports simple per-title jobs now

Change-Id: I9c976cdf93b8d1e171988722adc3695935ac43e0

Made JobRunner bail sooner for bogus job --type parameters

Change-Id: I1259682b8a6543e76f1c9a4d99324b457115a277

mediawiki.feedback: Set the message input to be autosized

It's tiny!

Change-Id: I82f689d2cc51500cb3c19300e3b14e14eb71ecb1

Merge "Refactoring mw.Feedback to work with OOUI elements"

Resources.php: Simplify dependencies to single modules

Change-Id: Icaa81662243259f166a28e3b6bb5b8ac6fea4bb3

Shorten long lines in Language.php to make phpcs pass

Change-Id: Ied04ebc64cd5a6e54bd0c3de5845342ec89d4d1c

resourceloader: Implement '$pages' parameter to ResourceLoaderWikiModule constructor

This makes it easier for subclasses to use ResourceLoaderWikiModule. Currently
many subclasses of this simply need to override the getPages() method.

UserModule and SiteModule keep their getPages override due to the set of pages
being dependent on context.

Change-Id: I388531398671afacfec36c6c5746d72267b5bdac

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:"