Merge "Update OOjs UI to v0.1.0-pre (46ccd5b3a7)"

Merge "Add prebodyhtml template variable"

Add prebodyhtml template variable

Allows the VectorBeta BetaFeature to add HTML underneath the heading

Change-Id: I6d9326d9e84a92b01dee2a29028208ef3f875767

Merge "Changing URLs of mediawiki.org in scripts to the SSL-based website"

Changing URLs of mediawiki.org in scripts to the SSL-based website

http://www.mediawiki.org --> https://www.mediawiki.org

Part 3

Change-Id: Ica633881b1744fa2854f4b012b79dbf5a7e5e7e2

Merge "Adjust default interwiki result text"

Merge "Allow more-descriptive section names for parserTests"

Merge "Vector: Make animation LESS component"

Merge "Adjust magic bolding in api help to only match lines inside *"

Adjust default interwiki result text

'Results from $1' sounds nicer than '$1 results' for sites that
haven't set the custom text.

Change-Id: I810c8e0503dde238ea8813b7cecb1a1dec927d3e

Merge "Title and redirect snippets don't have a parameter anymore"

Adjust magic bolding in api help to only match lines inside *

list=logevents&leaction has param values with *, which gets bold, when
there was two of them in the same line.

Bug: 61834
Change-Id: Idace9afd4f3a2dce9be539b209a02fa318df8f45

Title and redirect snippets don't have a parameter anymore

Change-Id: I5aae8caf14a5dd569042069fe64bc30736734cd8

Localisation updates from https://translatewiki.net.

Change-Id: I603eeeed9980a1190075cb54092edefdaf3c2ed5

Remove request_with_session/request_without_session from showCacheStats.php

Fix for I0ed1e87574 (42fcd43).

Since request_with_session/request_without_session were removed in the
mentionned commit, the maintenance/showCacheStats.php refuses to show
the other statistics.

Change-Id: I955cc537f8bc9116f759af5a5c3005aa7d60c83b

Remove $wgTitle from all maintenance scripts

Shouldn't be needed and aren't for any core operations. If any
extension relies on these $wgTitles being set in maintenance
environments those extensions are broken and should be fixed.

Change-Id: Ie02a5042ab96e155d783d56d5340dd0da8e3d55c

Update OOjs UI to v0.1.0-pre (46ccd5b3a7)

New changes:
9247e3a Localisation updates from https://translatewiki.net.
0a078dc Localisation updates from https://translatewiki.net.
3027b13 readme: Fix broken Markdown list syntax
d7ab327 Update oojs to v1.0.8
4dd7320 Drop file "OO.ui" file prefix
7c607c2 Localisation updates from https://translatewiki.net.
46ccd5b Make dialogs a little taller

Change-Id: I51caa22711fd10ff464b895e90a69292cbf003f8

Changing URLs of mediawiki.org in scripts to the SSL-based website

http://www.mediawiki.org --> https://www.mediawiki.org

Part 2

Change-Id: I3be61fe3dfb502cc20180486eb1a8016eac151df

Merge "Add --regenerate-all option"

Merge "Set a title for the context during import on the cli"

Merge "HTMLForm: Support 'url' type"

Merge "API: Handle exceptions from ApiBeforeMain hook in a user-friendly manner"

Merge "HTMLForm: Regularize punctuation in messages"

API: Handle exceptions from ApiBeforeMain hook in a user-friendly manner

The immediate impetus behind this change is this series of events:

1. CirrusSearch hooks ApiBeforeMain to handle some setup that requires
   the User object.
2. So User is loaded from the session.
3. OAuth checks the headers as part of loading User.
4. OAuth sees that the headers are invalid, and since it was called from
   the API it throws a UsageException, expecting the API to catch it and
   return an appropriate response to the client.
5. But nothing does so, leading to an unhelpful "Internal Error" page
   being returned to the client.

We can do better than that.

Bug: 62312
Change-Id: Ib5735661eec6ebe57eaa69c67b399e703cc90fc4

HTMLForm: Support 'url' type

SecurePoll will have a field that's a url, so we may as well use the
correct HTML5 input type for it.

Change-Id: Id3ddd8f2efdff08c8a188089d321d143ce5ef9c9

HTMLForm: Regularize punctuation in messages

To make Siebrand happy.

Change-Id: I0f622e4cdd2e6d8b33eb1ff38f0d78f8eec5bce3

Merge "Fix spacing in SpecialRevisiondelete.php"

Fix spacing in SpecialRevisiondelete.php

Change-Id: I5ae28e7e7b937ad4b60b919c87858e373ce85757

Merge "Fix CodeSniffer and warnings in includes/db"

Fix CodeSniffer and warnings in includes/db

Follow-up with a slightly wider scope to e611604d10897f.

Change-Id: I9bb5ef0ceef0cd7a9c044e675b953db6d3b35407

Merge "Fix CodeSniffer errors and warnings in changes tests"

Merge "Fix IGNORE option for sqlite update"

Fix CodeSniffer errors and warnings in changes tests

Follow-up to 1c6b7c74859e7.

Change-Id: Ib42621a03a43487d94ff1cb755544823a8adda9c

Merge "Add BaseTemplateAfterPortlet hook to add html after portlets"

Merge "Add RCCacheEntryBuilder, split from EnhancedChangesList and cleaned up"

Fix IGNORE option for sqlite update

this fixes the issue of the normalized sqlite
IGNORE option being discarded and ignored in
DatabaseBase::makeUpdateOptions.

Change-Id: I01579dee0f939a56c086d13683a60f4400014f62

Allow more-descriptive section names for parserTests

This patch allows `!!wikitext` (for `!!input`) and `!!html` (for
`!!result`). This is more in line with what those sections actually
contain and closer to the semantics of how they are used.

The old names are accepted as aliases to accomodate parser tests to
provide a migration path for extensions and other users of the
parser tests framework.

In addition to `!!html`, this patch also accepts `!!html/*` and the
more-specific `!!html/php`.  This allows tests to include a number
of different "outputs" for a given wikitext input, for example
`!!html/parsoid` and `!!html/php`.

Co-authored-by: C. Scott Ananian <cscott@cscott.net>
Co-authored-by: Arlo Breault <abreault@wikimedia.org>
Change-Id: Ie4e68960ca7c352af495ebb59ba83488935a44c4

Add BaseTemplateAfterPortlet hook to add html after portlets

Allows extensions to move away from using (or abusing) the
SkinTemplateOutputPageBeforeExec hook to add things in places
such as after the list of language links.

Also explicitly check that $template->data['language_urls'] is not false.
Extensions can set it to an empty array to have the language links
header be displayed.

Bug: 57094
Change-Id: I099fb2ff48e02a2e3564a484aea906263e40bb53

Merge "mediawiki.api.watch: Don't use deprecated 'title' parameter"

mediawiki.api.watch: Don't use deprecated 'title' parameter

* Instead handle it client side, and not as a deprecated feature.
* Rename 'page' parameter to 'pages'.
* Add unit tests.

Change-Id: I81ec4282aedd91e3f0bb2ed0d6dd34e437b54f5d

Merge "Fix CodeSniffer errors and warnings"

Add $wgLocalInterwikis to handle multiple local prefixes

This is akin to $wgSkipSkin/$wgSkipSkins. It is quite plausible for a wiki
to have more than one self prefix (e.g. enwiki has w: en: wikipedia: and
maybe others).

Some recent changes code seems to use $wgLocalInterwiki for quite unclear
purposes:
- I removed the line using $wgLocalInterwiki from the RecentChange
  class, as the 'lang' field of $mExtra is not used anywhere in core code.
  Extensions may use it, but it would seem more appropriate for them to
  use something like $wgDBname (or indeed to consult $wgLocalInterwikis
  directly) if they need to identify a particular wiki.
- In the IRC formatter, the first prefix in the array is used (if set).
  Appropriate documentation is added to DefaultSettings.php.

Related to bug 954 comment 3.

Bug: 954
Bug: 955
Change-Id: I9dbb566385b464402c5e78510b95dd2ffb4d9489

Merge "Fix some omitted colons in Spanish magic word l10n"

Fix CodeSniffer errors and warnings

For the files touched in I06ee0f01d8 as logged at
https://integration.wikimedia.org/ci/job/mediawiki-core-phpcs-strict-HEAD/7798/console

Change-Id: I811884f7a5ada4d0551552d0a2c2775d6b17b59f

Add --regenerate-all option

Bug: 62565
Change-Id: I98e052120c4f64d6a11f8363fa17de9c69656183

ChangesListSpecialPage: Separate all functionality for generating feeds

I should have done it at the beginning instead of trying to
extract it from recent changes.

Same for SpecialRecentChanges and SpecialRecentChangesLinked
(subclasses).

Created a new API module for it: ApiFeedRecentChanges.
It's somewhat un-API-like and hackish, but all feed modules are.

Old URLs redirect to new ones, so this should be fully
backwards-compatible assuming sane feed reader clients.

Change-Id: I06ee0f01d896bc66545a1800b24693ce7524e433

Merge "Partly revert 28b4dd5f for zh and zh-cn/hk/mo/sg/tw messages"

Add RCCacheEntryBuilder, split from EnhancedChangesList and cleaned up

This makes it more feasible for Wikibase, Flow, etc. to support
enhanced changes format, and allow better support for the rc_source
column in the future.

Change-Id: I873f6b86007000a94337f0c963df4bf8fec5b715

Merge "tests: Add unit tests for ResourceLoaderStartupModule"

Merge "qunit: Disable sinonjs-ie as it breaks the clock sandbox"

Merge "Check validity and availability of usernames during signup via AJAX"

Merge "Include all of /resources/mediawiki/* in jsduck index"

Merge "resourceloader: Pass jQuery twice to 'implements' module wrapper"

resourceloader: Pass jQuery twice to 'implements' module wrapper

This must be done in two parts. A follow-up change will update the wrapper
signature in ResourceLoader::makeLoaderImplementScript to bind the
arguments to '$' and 'jQuery'.

Because almost all event-handling and DOM interaction is mediated by
jQuery, ResourceLoader could provide a per-module breakdown of the work
performed on each page view by passing each module a copy of the jQuery
object that wraps certain functions (like jQuery.fn.on) in module-aware
profiling code. I expect that profiling would only be enabled on a small
fraction of requests. The effect on ordinary page-views would be a
possible small speed-up as a result of jQuery look-ups resolving in
module (rather than global) scope, but I expect the difference to be
neglible and it is not a part of my motivation.

Because ResourceLoader must bind both '$' and 'jQuery' to module scope,
we pass jQuery twice. A separate commit will update the wrapper function's
signature in ResourceLoader::makeLoaderImplementScript. This has to happen
separately to ensure that no module binds jQuery to undefined due to
being loaded by an out-of-sync version of mediawiki.js.

Change-Id: I0c9edac359ceb16fc273de1e51ea70322f18353e

qunit: Disable sinonjs-ie as it breaks the clock sandbox

The ie fixes file assumes running in the global scope. Since it
runs locally in our environment, it ends up destroying its
references to e.g. the Date object.

Then when calling sandbox.useFakeTimers, the Date constructor
will yield an object with no methods and tests will fail at a
random point when e.g. trying to access date.getTime().

* Remove 'clock' from the sinon properties we inject into the
  test context. We use the sandbox instead (just like for server).

  This also fixes a shadow clash with the tests that stored the
  clock instance in `this.clock`.

Change-Id: I6085bccf5038e4751da48ee765fe81560f58c6e8

Merge "When checking whitelist of extensions, only count last extension."

Localisation updates from https://translatewiki.net.

Change-Id: I87ed70aa87a61057fa81f0eae63363af85fd71a0

Update oojs to v1.0.8

Code:
 http://oojs.github.io/releases/oojs-1.0.8.js

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

Change-Id: I3ae46a0050d74c5c4fa9004455328e0661cab2dd

Partly revert 28b4dd5f for zh and zh-cn/hk/mo/sg/tw messages

For unknown reason (maybe some bug), L10n-bot exported messages in
these languages which had been disabled for translation. Let's remove
them for now and observe whether the bot exports them again.

See I0007fd00579971d4d5bce5e4545492f87ea52867.

Change-Id: I70d384c3a01aff5bbade01917c92e7be9fb936f4

Add release note for removal of preference "noconvertlink"

Follow up 333bf3ae5b412fae1e4f57a62a220c941ef50536.

Change-Id: I7a3e391f56d2e2839b3210d79a9f5b630f0f6fed

Merge "Remove user preference "noconvertlink""

Merge "Add missing DROP SEQUENCE to postgres' tables.sql"

Merge "Cleanup MediawikiTestCase"

Merge "Throw exception when trying to stash unset globals"

Merge "Introduce stashMwGlobals method to MediaWikiTestCase"

Merge "Don't load the apex theme for oojs-ui if we're using Minerva skin"

Don't load the apex theme for oojs-ui if we're using Minerva skin

Change-Id: I12c9f91276d34e56d0e9f5bb2e990a72756dd6b9
Dependency: Ib9d52327425f8a12e12d82f1421f5cab0774c510

Localisation updates from https://translatewiki.net.

Change-Id: Ia03f6dcf21ee8e589758e039566adb661893804f

Merge "Further optimize User::saveOptions() by doing deletes by PRIMARY KEY"

Further optimize User::saveOptions() by doing deletes by PRIMARY KEY

* We do not care about gap locks here, just as in the case
  with new users. Races should be rare and are simply FWW.

Change-Id: Ie483a2d5df501619acbd868bf894dfc6c911a753

CSSMin: Don't do file_exists on random data in getLocalFileReferences()

If it's a URL, it obviously can't be a local file; check that first to
avoid PHP warnings about malformed paths.

Bug: 60960
Change-Id: Id784c089c3de8af79af7524ef5ab5cc2f7b8af9e

Merge "Add a link to Special:WhatLinksHere in deleting-backlinks-warning"

mediawiki.api.watch: Fix promise return value format

Follow-up to Ic77b546c.

Bug: 62422
Change-Id: I2ac9e0da0f1c825e1cefd98d21406fcb089c0827

Check validity and availability of usernames during signup via AJAX

This is done in addition to the server-side checks. Users who have
disabled JavaScript will not notice any difference.

The way this is done (via action=query&list=users API) means that we
can't check for all possible error conditions, both for the username
(e.g. these enforced by extensions like AntiSpoof) and other parts of
the form (e-mail, password…) – these are still handled server-side
only. Thus we intentionally never say that whatever the user typed in
is valid – we only warn when we know it's not.

Doing this "properly" would require some reworking of the internals of
the signup process and this way is already a huge improvement.

(Split off from the reverted a0c72523.)

Bug: 34447
Change-Id: I42c00b54651fd4e7217a862b3116c6113935f5ae

Merge "Let wildcard actions work in list=logevents&leaction="

Merge "Add final period to API module descriptions"

Add final period to API module descriptions

Change-Id: Icae68c1ab1fd0006e00a3a9a56ae8f831d3d0d45

Set a title for the context during import on the cli

Bug: 62467
Change-Id: I9e0b6a219ea4176f9a3d14b8dbbea08b36c76c59

Localisation updates from https://translatewiki.net.

Change-Id: I1d75011f4c5f78746fb1b05ef05858ac6a35eec4

Merge "Remove final period from api module descriptions"

Shorter lines in LanguageUz.php comments

Follows up gerrit change 116914

Change-Id: I75215416277eb8f5d9e16a1bf91637c673be0a48

Wrap long text

Change-Id: I37fe6fbdc8779ba86318e587c3d9e217bede400a

Remove API developer email addresses

They just end up being used for direct support requests that should
really be done on public mailing lists etc

Change-Id: If704ec8dff5fc4669165b580290cc29cf953a25b

Cleanup MediawikiTestCase

Cleans docs
Adds since tags
Fixes typos
Removes totally unused stuff
Adds scopes

Change-Id: I80d542196a0f2265aacdd8ae89f919773832c14c

Throw exception when trying to stash unset globals

If we are trying to stash an unset global then
throw an exception!

Also adds a test

Change-Id: I25f493a0a535201c08ca9624c2c650f61d9e256d

Introduce stashMwGlobals method to MediaWikiTestCase

This method is factored out from the existing
seMwGlobals method.

The Doc from the initial method is also split and
improved and since tags has been added

Also adds tests

Change-Id: I0637194d637abf485a245b00587743f0f6dd495a

Merge "Remove 2 false values returned in execute()"

When checking whitelist of extensions, only count last extension.

When we are doing blacklisted extensions, we count all extensions
as some programs (like apache sometimes) consider extensions that
aren't the final extension. However when doing whitelists we need
to only count the last extension, otherwise people can name files
foo.goodExt.BadExt. For example [[commons:File:Deamado ko.png.bmp]]

I do not believe this represents a security risk as bad files are
still filtered out. However it does allow unwanted files to be
uploaded.

Bug: 62451
Change-Id: Ie27c15f749812710571f432bc5915e498f8017e3

Deprecated useless BagOStuff::replace method

* Almost nothing actually uses this (one extension)
  as it doesn't really solve race conditions like merge().
* Removed subclassed versions too.

Change-Id: I4d4123afcc0576a90a3cbf1e4e6daff7ebce39c3

Merge "Cleanup SampleTest.php"

Include all of /resources/mediawiki/* in jsduck index

Not all classes/methods in these files have documention yet (the
only effective change is adding of mw.Debug), but at least the
other files are now in conforming syntax.

mw.Debug:
* Made method descriptions more consistent (a few of the "build"
  method didn't start the description with "Build").

mw.Feedback:
* There was no @class anywhere. Worked because @class was implied
  for the method name starting with a capital. It was still
  ignoring the @param tags signature for the constructor though.
  Fixed now.

Co-Authored-By: Mark Holmquist <mtraceur@member.fsf.org>
Co-Authored-By: Timo Tijhof <krinklemail@gmail.com>
Change-Id: I74c8dcfeac9754e38f9dec23873081e8fb6d51d4

Split long line

Change-Id: I719e63a9b1d6e1f1fd2031e443e2f40adfbcb4ca

Merge "Use setMwGlobals in ApiEditPageTest"

Merge "Update lessphp to b7cd5c79e8"

Merge "Fix phpdoc of class vars in ResourceLoader"

Merge "Update OOjs UI to v0.1.0-pre (efc7297353)"

Update OOjs UI to v0.1.0-pre (efc7297353)

New changes:
3ab4d6d Balance margins around icon on iconed ButtonWidget
8d69f53 Add missing 75% transparency to add-item
2c0ae23 Add indicators to demo page
1700e98 Localisation updates from https://translatewiki.net.
262cb8d Localisation updates from https://translatewiki.net.

Change-Id: If265e4760b69957a1cc0138f9834ea984763f1f8

Update OOjs to v1.0.7-pre (22e610a5e9)

New changes:
544b0a8 doc: Improve overall documentation and fix minor issues
293f89f inheritClass: Use Class.super instead of this.constructor.super
59c78c7 Registry: Remove redundant type validation logic in #register
22e610a Factory: Use Class.super instead of hard coding parent class

Change-Id: Idae726f126b15e3c884e5bbda24eb2b7c11d801f

Merge "Clean up ExportDemoTest"

Merge "CSSMin: Clean up the logic in getLocalFileReferences()"