Merge "Added clarifying comments to IContextSource"

Rename JSDuck config to standard 'jsduck'

* Move configuration to /jsduck.json per standard. This way
  it can be run as plain '$ jsduck' without needing a maintenance
  script or custom Jenkins job. Similar to JSHint, JSCS, Grunt,
  and Gem etc.

* Move --processes=0 from maintenace script into config file.
  This should've been in the config file all along and serves as
  workaround for https://github.com/senchalabs/jsduck/issues/525.

* Use grunt-contrib-copy instead of a symlink for resources.
  For local development a symlink works fine, but for publishing
  from Jenkins to doc.wikimedia.org the /docs/js/ directory needs
  to be standalone. This was previously done with a manual post-build
  step that added an additional rsync, but this logic should be
  in the repository so that the doc entry point can be simplified
  and standardised to 'npm run-script doc' for all projects.

Change-Id: Iaaaac50ee78dd9ff8f24f1ef3a3685ad51cf33b2

Make PHP unit parser tests pass again after translation update

- sr-ec.json updated with 17607cf8965bf332f8c80d7a8e25a8c4e1107696

Change-Id: I7dfc9bcea2be85dd42d344858ce02246432f7fe1

Localisation updates from https://translatewiki.net.

Change-Id: Ifc570f5a883ef4269e9d85b9e634f09e95716581

Added clarifying comments to IContextSource

* Also removed some duplicated comments

Change-Id: I2c967d0f3219f3533378ecd5f9fccc40c8f9b219

Don't require JavaScript for addHelpLink styles

And avoid wfMessage.
Followup to 376c0a13df05f30367b2a89c2cac7bdab22600ed

Bug: T45591
Change-Id: I2f02c2a4db609356ff8be30f260490d19f3226d1

Merge "mediawiki.special.upload: One event handling function is enough"

Push enqueue-on-view jobs to the faster "local" queue

* This only matters for multi-DC setups, otherwise everything is local anyway

Bug: T91819
Change-Id: I0756b59ae1ba27a5d24cf7baef6d068b748e6e30

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

Localisation updates from https://translatewiki.net.

Change-Id: I5e91846766f3558874fd179fd7a1a3702901bda4

Merge "Remove isVolatile/markVolatile logic from LocalFile"

Remove isVolatile/markVolatile logic from LocalFile

* This is obsoleted by the "latest" flag
* Follows-up 5085a4b

Bug: T88506
Change-Id: I808c3a6f0edb3466cb600c06c5e742ab030baf14

Merge "Added EnqueueJob class that handles routing jobs to queues"

Merge "Made wfFindFile/wfLocalFile callers use explicit "latest" flags"

Merge "inputs.less: Change focus state"

Set up RuboCop configuration according to coding conventions

Updated RuboCop to the latest version.

Renenerated .rubocop_todo.yml file with “bundle exec rubocop
--auto-gen-config”.

See also:

https://www.mediawiki.org/wiki/Manual:Coding_conventions/Ruby#Base_confi
guration

Bug: T91485
Change-Id: I926d6ee114ccc01076e8df6ec3f95eb2af78c212

QA: Refactor for mediawiki_selenium 1.0

Refactored by following steps outlined in UPGRADE.md under the
mediawiki_selenium project.

Change-Id: I52717292df0541eb489b34ec083ec4a957f7546d

Made wfFindFile/wfLocalFile callers use explicit "latest" flags

* Callers that should not use caches won't
* Aliased the old "bypassCache" param to "latest"

bug: T89184
Change-Id: I9f79e5942ced4ae13ba4de0b4c62908cc746e777

Merge "resourceloader: Call setName() in test suite before calling getStyles()"

Merge "maintenance: Defer initilization of 'main' Config until after Setup.php runs"

resourceloader: Call setName() in test suite before calling getStyles()

This caused a database error due to NULL being inserted as name, which is illegal.

> Function: DatabaseSqlite::replace/single-row
> NOT NULL constraint failed: unittest_module_deps.md_module
> Stack trace:
> #3 includes/resourceloader/ResourceLoaderFileModule.php(420): DatabaseSqlite->replace()
> #4 tests/phpunit/includes/resourceloader/ResourceLoaderTest.php(88): ResourceLoaderFileModule->getStyles()
> #5 (): ResourceLoaderTest->testLessFileCompilation()

This test shouldn't be trigggering database updates, but that's for
a later change to stub out or refactor.

Bug: T91567
Change-Id: Ic451bd41e2ffc188d2efd6b7ce61b03b9de61296

Merge "mw.Feedback: Terms of use always appended and allows for link"

mw.Feedback: Terms of use always appended and allows for link

Bring the terms of use link outside of the general checkbox message
and have it appear regardless of whether the checkbox is shown.
Also make sure it is parsed correctly if/when it has a link.

Change-Id: Ic1072690e33927e472f8ba529b16fa32280a6abc

mediawiki.test: Prevent "Error: expected" from going to the real console

Keeps the build logs cleaner and less confusing.

Change-Id: Ibbbf5dd367cab798885df34b9dd73f84229d3c5e

Remove rel="archives" from HTML output

Follows-up fab055d6a1.
Introduced in 4299d27956 (r45988)

That link type was removed from HTML5 in 2011. It is not recognized
by the W3C Markup Validator.

https://html5.org/r/5925
https://www.w3.org/Bugs/Public/show_bug.cgi?id=11486
https://html.spec.whatwg.org/multipage/semantics.html#linkTypes

Change-Id: I61ddd7ee3df41bfb62423d0498217d31e7952902

Unbreak parser tests

Caused by f988ef004cd4e6c21649fb30a7d421aadae8f495, which would
have failed tests if l10n-bot hadn't self-merged.

Change-Id: I4674178f51c9b67bc057a84b06b3e4153857860b

Localisation updates from https://translatewiki.net.

Change-Id: I8fd55473983eff489284bbc3ff46c1bcffdb1ce1

Merge "Fixed detection of unsigned mysql column in updater"

Fixed detection of unsigned mysql column in updater

MediaWiki running with an old mysql or php without mysqli cannot
use the newly added flags property of Database::fieldInfo.
For backward compatibility add the missing properties using the flags
property to the result of Database::fieldInfo.

Added missing unsigned, numeric, blob and zerofill properties to the
MysqlField using flags in mysqli.
The missing fields in mysqli are taken from
http://php.net/manual/en/function.mysql-fetch-field.php

Follow-Up: I641dc1fad0a5bb14815f55e0357243a562fd672b
Change-Id: I1da904df72f8af7725fc7ff94121ef85b18a36f8

Remove duplicate param escaping code

wfEscapeShellArg() can handle multiple params, escaping each.
This patch changes wfShellExec() to call wfEscapeShellArg() directly
instead of doing the gluing itself.

This patch also extends wfEscapeShellArg() to accept an array parameter
optionally instead of as separate args, which is often useful.

Added also unit test cases for single, multiple args, and single array args.

Change-Id: I7a0761cc2ba98c210a9eacadd12da407d933e42a

Merge "Browser should clear cache for API responses"

ParsoidVirtualRESTService: Don't use "else if"

https://www.mediawiki.org/wiki/Manual:Coding_conventions/PHP#C_borrowings

Change-Id: Ica51fd60e3da75d3b882cf3172ef7500afeab0db

Merge "Made the "bypassCache" flag for file locator methods use the master DB"

Browser should clear cache for API responses

By default we send "private, must-revalidate, max-age=0" for regular
logged-in wiki viewing.  This changes API responses to match.

Later, someone should update the Cache-Control header generation so
that it works the same for API responses as it does for OutputPage.
This is becoming more important since we're using the API instead of
OutputPage for editing with VisualEditor.

Bug: T74480
Change-Id: Ib309df8568de2c7137b6d13b9ca4004150a772dd

Made the "bypassCache" flag for file locator methods use the master DB

* Added a File::load() stub method
* Cleaned up the File loading flags bitfield a bit

bug: T89184
Change-Id: I1aa4b096c0cad5f5ca34321cc897019005c53a76

Fix three more invalid double underscore special page aliases

Cf. f8207aeb17f955a79a726df775d3643c550605fa

Change-Id: I6e72273524c54016296c1a77d978fd6a872a542e

Update OOjs UI to v0.9.0

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

Change-Id: I73d20f6556558bd5675c11addfcd27d417173693

Merge "Profile wfShellExec() calls by caller"

Profile wfShellExec() calls by caller

Change-Id: Ide437ec586b9e123ca73b5abd17278a72375dfb0

Added EnqueueJob class that handles routing jobs to queues

Bug: T89308
Change-Id: Iadb34f24d8bbe94c0f9f119e530c0bbe1060df0a

Add "ValidSkinNames" to extension.schema.json and ExtensionProcessor

skin.json goes through the same exact code as extension.json does, and needs
to be able to add skins.

Change-Id: Idd0acaf6735b4c308d3ab584c49da4e97448347f

Localisation updates from https://translatewiki.net.

Change-Id: I4d53140c8b780ef5d6118ac535ab2a70c7e0082b

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

Merge "API: Properly handle limit and continuation for generator=prefixsearch"

Merge "Add top help link to MediaWiki.org in several pages via indicator"

mediawiki.special.upload: One event handling function is enough

- Common event handling function for all radios
- Only remove `mw-upload-source-error` once per radio change
  (there is no way to disable a radio button without enabling another one)

Change-Id: I1e5c7c403efb26c74df39669491bc49602a9df4f

API: Properly handle limit and continuation for generator=prefixsearch

The search engine is queried for limit+1 titles, so we know whether to
output a continuation value. Actually do this for generator mode,
instead of just returning the limit+1 titles.

Bug: T91503
Change-Id: Ib80c6d3fa9e8b52ae1f5c58fbe74c8e909329e59

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

Add top help link to MediaWiki.org in several pages via indicator

All the chosen targets are translatable public domain help pages
on MediaWiki.org. Mostly special pages and actions for privileged
users for now.

Adapted from the Translate extension, credit to Niklas Laxström
(TranslateUtils::addSpecialHelpLink).

Depends on 6f5b29ff4e6fdf21b5a8cacaf10d6aceaee26a7d, whose commit
message has a typo addIndicator() instead of setIndicator().

Bug: T45591
Change-Id: I2934b1708a0d207dcf3d940264f140613646f203

maintenance: Defer initilization of 'main' Config until after Setup.php runs

Bug: T90680
Change-Id: I72d6306404bc1f6c7a605853130a026e9858b493

Shorten long lines in PrefixSearch.php to make phpcs pass

Change-Id: Ibf18835ff614e53887912b8e997a3230ed29dd68

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