Merge "SpecialTrackingCategories: Read from the extension registry"

Merge "registration: Ignore attributes that start with @"

Merge "registration: Sync extension.schema.json with reality"

SpecialTrackingCategories: Read from the extension registry

This demonstrates how we can transition from extensions putting
things into the global scope ($wgTrackingCategories) to instead
storing them in the extension registry. This will increase the
overall performance of the extension registry since it no
longer needs to do an array_merge with $wgTrackingCategories.

For extensions already converted to using the registry
no change is needed as the schema is still the same.

Change-Id: Ie0df4c20b123dac784a1c02eb991edc609a911b6

Merge "convertExtensionToRegistration: Put some keys (name, version, etc.) on top"

Merge "mediawiki.cookie: Remove false documentation for options.expiry as number"

mediawiki.cookie: Remove false documentation for options.expiry as number

Follows-up 7c585b5. There was never an implementation for
supporting an offset from now in seconds. The code and tests
account for no such thing.

When passed a number, it will actually end up being passed to
$.cookie, which interprets it as an offset in days (!).

Change-Id: I5d8ed8cb432ca550c62cf415077ffffd91fe2bbd

Use: addGroup() and removeGroup() should return boolean

Have User::addGroup() and User::removeGroup() methoids return a
boolean when their respective hooks return the respective boolean.

Fix SpecialUserrights to respect this return vale and update the
add/remove arrays accordingly.

This resolves an issue where a hook that prevents a group from
being added or removed still shows that group being changed in
the Userrights log.

Change-Id: I7621cc22b04ff41cf67bd434a1f89d31bdc2cffd

mediawiki.inspect: Use fixed numbers for sizes in bytes

Change-Id: Ieb39322ccfbbc80213ecd1d2a3b4f66bbe346b85

Merge "mediawiki.inspect: Guard against Object.prototype keys as module names"

Merge "Use Context on ProtectionForm for messages"

Provide a backwards compatibility shim for MWLogger

MWLogger was renamed MWLoggerFactory and reduced to a static utility
class in Ie474676. Reintroduce an MWLogger that proxies the public
static methods of MWLoggerFactory to ease the transition of users who
have implemented $wgMWLoggerDefaultSpi configurations that reference
MWLogger.

As noted in the class, this is a temporary bandaid that should be ripped
off before 1.25 becomes an official release.

Change-Id: Iaccb78a510c60aab2ff20a9aa7c0869699657388

Replace MWLogger with MWLoggerFactory

Time wounds all heels. During the code review for the PSR-3 logging
introduction, several people asked me why we needed a wrapper
for Psr\Log\LoggerInterface if the point was to use the standard. At the
time I was convinced that it would be better to introduce the dependency
via a wrapper class so that we could use the wrapper to patch over any
deficiencies that we might find in the PSR-3 API. After going on to work
on a project to disentangle other MediaWiki components from internal
project dependencies I have suddenly and clearly seen the error of my
ways.

We still need a logger factory as PSR-3 does not specify
a standard mechanism for creating Psr\Log\LoggerInterface instances. My
solution is to convert MWLogger into MWLoggerFactory to retain a static
factory interface for creating PSR-3 loggers but remove the MWLogger
wrapper class itself in favor of direct exposure of
Psr\Log\LoggerInterface to the MediaWiki consumer classes.

Change-Id: Ie47467657dcf341991ada00827dca5e8eff95438

Localisation updates from https://translatewiki.net.

Change-Id: I4929a44b21dadb98e226ffccba61059a0b39c389

Merge "Always set group_concat_max_len to avoid awful truncation gotcha"

Merge "Changed FileBackend exceptions to subclass Exception"

Use DatabaseBase::selectRowCount() in two more places

Change-Id: Ic6e03a96c03499f7d62539f12e677b4407def673

Localisation updates from https://translatewiki.net.

Change-Id: I87f18cc6e49a9e0ba2910ea99cd3806f1a108046

Merge "mediawiki.ui: checkbox: Correct alignment of long labels"

Merge "Adding PHP mustache parser (lightncandy)"

Merge "Add option to output referrer policy meta tag"

Add option to output referrer policy meta tag

Bug: T87276
Change-Id: I9ea4797c4d292cd4565f47b88e605b4e7afdb1d1

Document alternative to deprecated function

Change-Id: I89ae5f5062f8e3eb8dac38d7efcca7723d6b62b1

Revert "resourceloader: Omit empty parameters from mw.loader.implement calls"

Works as intended, but didn't account for the first implement() parameter
being omitted client-side. Revert until that is accounted for, then re-try after
that fix is rolled out for > 1 week.

This reverts commit 4ce0c0da42acfbcc5c68527834f85436efd0ebc1.

Change-Id: I36c1619991663c0303636d1d3f037b0021ac79bf

resourceloader: Omit empty parameters from mw.loader.implement calls

Follows-up ebeb297236.

Also:
* Add tests for ResourceLoader::makeLoaderImplementScript().
* Apply ResourceLoader::trimArray to makeLoaderImplementScript (new in c0c221bf).

As always, the client (updated in Ie32e7d6a3c) is backward-compatible with old
(cached) load.php module responses. However, the old client is not compatible
new load.php responses after this commit.

That's generally not an issue, as we don't cache the client very long (~ 5 min).
However people with their browser open and mw.loader clients initialised can
still make new module requests (e.g. modules loaded on-demand, such as when
previewing edits, clicking buttons etc.). That can easily be several hours after
initial page load. As such, client/server bound changes should always be
back-compat and deployed a reasonable time apart to reduce chances of active
sessions making subsequent requests. Ideally we'd find some solution to this in
the long-term, but handling this at all is better than what we usually do...

For deployment: Ensure this is deployed several days after Ie32e7d6a3c09f.

Change-Id: Ic8d7efe49b5d45e3f95a2f04e3a26a014b10af16

UserLogin: Don't rely on jQuery prop('tabindex') to map to tabIndex

Follows-up I9b03d519af43d.

There is a hardcoded set of exceptions in jQuery core right now
that map these common mistakes for back-compat. But this is
undocumented/unsupported legacy behaviour for a very small subset
of properties.

Change-Id: I9cfef0bc6a088e4c0a632b3673df460d26477c37

API: Fix setting of wpIgnoreBlankSummary in action=edit

EditPage checks this using getBool(), not getCheck(), so empty string is
interpreted as false.

Also clean up a long-obsolete comment.

Bug: T87188
Change-Id: I18f376cefad44295874d7b1cd760ba6fc966be29

Localisation updates from https://translatewiki.net.

Change-Id: I845680e24225a0e9491e61efd1455675eb397b20

Templates:Captcha position modified,extend functionality added to QuickTemplate

The ConfirmEdit extension presently inserts captcha in the header of Usercreate
and Userlogin templates. The js,if enabled places the captcha before submit
button.  A separate 'extrafields' placeholder for captcha is added just before
submit button in Usercreate.php so that captcha position remains consistent,
even if js is disabled.  An extend method added to QuickTemplate to append any
data to existing field, rather than setting it with fresh data. Particular use
for 'extrafields' in Usercreate and Userlogin.

Bug: T85192
Change-Id: I82c68814e79cbc5aa250a308862c59fcbb6fd527

Add newline at end of languages/messages/MessagesBgn.php

Change-Id: Ibd95ebb1eaf37e9fafc572818e86be05b839dbb2

Add language fallback for Western Balochi (bgn)

* As on translatewiki.net.
* Added to Names.php with 919dbbfcdd7900c7d213ed1a60e60bc920fdee09

Change-Id: I9648a1e326ea742ad74f2b1265e59599c4201b5c

mediawiki.ui: checkbox: Correct alignment of long labels

Use display: table-cell; to display the checkbox and the label
side-by-side. I am honestly surprised that this works here. Not sure
if it's a good idea to use… although we successfully did it for
OOjs UI in 2498265b, but we have more control over everything there.

Bug: T85241
Change-Id: I381d05360ad3e7ab153f56772fffb650dab5cda9

Add new Polish translation for DEFAULTSORT magic word

This has been done on translatewiki ages ago [1], I was under the
impression that this is updated with the regular localisation updates, but
apparently not so. The Polish Wikipedia community would like to have
started using the new translation a week ago [2], so let's expedite this
please.

[1] https://translatewiki.net/w/i.php?title=MediaWiki%3ASp-translate-data-MagicWords%2Fpl&diff=5944044&oldid=5711187
[2] https://pl.wikipedia.org/wiki/Wikipedia:Kawiarenka/Kwestie_techniczne#DEFAULTSORT_-_mo.C5.BCe_przet.C5.82umaczy.C4.87.3F

Change-Id: I52f9f5962d3b52d7a5e9b16915f7dedabeecd142

LSG: Break module list into text file for readability

Change-Id: I3d66bcacf99da7eecc91a421c521dc650ed9cf82

Merge "Stop using mw-ui-big on login form buttons"

Localisation updates from https://translatewiki.net.

Change-Id: I16b3a2ee08dd95acd0a8ba9edfb42d214abf1aee

Merge "Revision: mark getRaw*() methods as deprecated"

Localisation updates from https://translatewiki.net.

Change-Id: I8fb2ad3395c3dab471bd1b3dfce103f3d98623a2

Use Context on ProtectionForm for messages

Also use Language::userTimeAndDate instead of Language::timeAndDate,
avois using $wgUser for the timezone.

Change-Id: I3547bdc2ee2b787bcb301c4023512e31bc18b67f

Revision: mark getRaw*() methods as deprecated

   Revision->getRawUser()
=> Revision->getUser( Revision::RAW )

   Revision->getRawUserText()
=> Revision->getUserText( Revision::RAW )

   Revision->getRawComment()
=> Revision->getComment( Revision::RAW )

The body of Revision->getRawUserText() has been moved
into Revision->getUserText().
Every usage has been replaced.

Change-Id: Ic6fbfbc0507dcf88072fcb2a2e2364ae1436dce7

Added some job queue comments

Change-Id: If42b982ac339cc6aaf56ef2ed6c78867b83d2e56

Changed FileBackend exceptions to subclass Exception

Change-Id: Ic7d4d6cf0dde3e93ef78758b1a6b03f78c9bcdba

Localisation updates from https://translatewiki.net.

Change-Id: Icdecf03a42a08ce43210b632fd15f7d09795332e

Add bgn to Names.php

It recently passed the threshold and translations were committed:
https://translatewiki.net/wiki/Translating:MediaWiki/New_languages

Change-Id: I2a217b70baf6961c3c7bd7b3a90fbb104238a74f

Made JobRunner bail if wfReadOnly() is true

Change-Id: I97ef66718bf4033768cd820b42521af31539b3f6

Stop using mw-ui-big on login form buttons

This is inconsistent with mobile and is one of the blockers from us
using this form rather than our own on a mobile device.

Also mw-ui-big is a terrible class name and we should be deprecating its
usage.

Change-Id: I56e93429431690cf1b9ef043620b36148e3b4a7a

JsonContent: Support non-object values as root structure

* Remove is_object restriction from isValid.
* Implement rootValueTable to support rendering of other
  root structures.

Also:
* Minor method documentation tweaks.
* Document why quotes are not escaped for strings in value cells.

Bug: T86270
Change-Id: Ic1d10393912fcefa22d675fd4aa2baf437d2a05a

Localisation updates from https://translatewiki.net.

Change-Id: Iac5da9911687d649dec3c748d2b2b5b04469f245

Avoid fatal error with profiler in Maintenance.php

Error Message: "Fatal error: Class name must be a valid object or a string"

Change-Id: Ia21f4d2c45aaaa12a270dd0fda7274008736a899

Add another Br[yi][oa]n to CREDITS

Change-Id: I2f3a5f813bb195f10930af6a7373dcaad96c18a0

Fix HTTPS protocol detection

According to PHP documentation:
http://www.php.net/manual/en/reserved.variables.server.php
The $_SERVER['HTTPS'] is set to a non-empty value if the script was queried
through the HTTPS protocol. There is also note that for ISAPI with IIS, the
value is set to 'off' if the request was not made through the HTTPS protocol.

To follow the PHP documentation the $_SERVER['HTTPS'] == 'on' doesn't seem
to be the correct way how to detect the HTTPS protocol (there maybe e.g. '1'
instead of 'on').

Bug: 46511
Change-Id: I5675fed9b7d54711b96b25702181112ef3692f3c

jquery.tablesorter: Fix "capabilities" typo in comment

Change-Id: Iea1a2956c8927bc29c904569ae594f6978aa8868

Update OOjs UI to v0.6.3

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

Change-Id: If0fc682b3a7f186d53ca70fdb66dded66a7f4814

Adding PHP mustache parser (lightncandy)

Per front-end standards group and architecture cabal

See corresponding change for vendor repo: change Id86b118c

Change-Id: I281acc49c1959bc01d0c1580089fe6896992ba19

Localisation updates from https://translatewiki.net.

Change-Id: Ib5e195c508863f6b9d75e0984a3c608b65deb609

Typofix in wrapWikiMsg() comment, smoother sentence

Change-Id: Ic0ebc94cec68ee1515d4c98d49b26a6f9148e95e

Merge "Userlogin.php: Tweaks and corrections"

Merge "Article: Show error message when Revision::getContent() fails"

Userlogin.php: Tweaks and corrections

* Correct nesting of the block generating "Forgot your password?" link
  – it would be mistakenly only generated if the user could create an
  account.
* Use ->escaped() rather than ->parse() for 'userlogin-resetpassword-link',
  as it is used inside a link and allowing wikitext could lead to
  undesirable side-effects.
* Remove unused $extraAttrs variable.
* Remove pointless 'size' => '20' for some fields (they are displayed
  full-width using CSS).
* Remove pointless .mw-ui-block class, as its styles are always used
  for buttons in a VForm.
* Prettify code for hidden fields generation.

Change-Id: If749620bf2eaf58f815cae559383e1a1c842b870

Merge "Add tests for Sanitizer::escapeHtmlAllowEntities"

file name -> filename

Change-Id: I2b39501b95745b04f3c8f2c5df893c066e5e7867

ParserOutputTest: Fix @covers tags

Change-Id: I4460f0474174a95558301a0e0a261a962dc66bee

Article: Show error message when Revision::getContent() fails

It already checks for other errors though not this specific one
(e.g. missing text row or external storage row).

Change-Id: Id9eed4bd3d8241d445b15948f1544040394012c3

Merge "HTMLFormField: Don't abuse mClass for VForms"

Add tests for Sanitizer::escapeHtmlAllowEntities

Change-Id: I84a1328559d57a424aa77a6d89f837615feffc13

HTMLFormField: Don't abuse mClass for VForms

Various fields use this differently than I thought, breaking things.
Let's add a separate variable.

Change-Id: Ia974a42c2a905cbc9e8405d9f03c10c8ad692864

Fix array index for oldid parameter

This caused an incorrect oldid parameter to be passed to parsoid on html2wt
conversion, resulting in dirty diffs.

Change-Id: If140bbc94f280f9ec5a1c9f6d9c3b4e7a46e1917

Merge "Set stat cache when getting local copy files in Swift"

Localisation updates from https://translatewiki.net.

Change-Id: I7e0c73a076713d6e2f4a317456e2bffb9a1a77f4

Merge "Disable styling for checkboxes and radios on non-js browsers"

Merge "mediawiki.action.view.redirect: Don't unset wgInternalRedirectTargetUrl"

Merge "Use Config in SpecialUpload::getInitialPageText"

mediawiki.action.view.redirect: Don't unset wgInternalRedirectTargetUrl

Follows-up c49bd9cb14a9af678.

No good reason to do it. Dunno why I did it this way.

Change-Id: I9d80492b7ef6c1790c8dcb4a6861f263c450bd6a

Use Config in SpecialUpload::getInitialPageText

Change-Id: I7edfe23278acff8d3089d9ad23b588f937d9e337

Release notes for f338a1cf

Change-Id: Iecf07aea6384b41d4ca4f115ea80644e7ca0bcac

Merge "Release notes for 1dd14dce and b3830611"

Release notes for 1dd14dce and b3830611

Change-Id: I0b933560d7df17246ea91b80ff1cd57abb61ad1b

HTMLForm: Separate VForm code to a subclass

…and in general, work with the existing HTMLForm design for defining
display formats, rather than against it.

Breaking changes:
* HTMLForm::isVForm() is now deprecated.
* You can no longer do this:
    $form = new HTMLForm( … );
    $form->setDisplayFormat( 'vform' ); // throws exception
  Instead, do this:
    $form = HTMLForm::factory( 'vform', … );
  When using FormSpecialPage, override the new getDisplayFormat() method
  instead of calling $form->setDisplayFormat() in the alterForm() method.
  (Other display formats are not affected, only 'vform'.)

Bug fixes:
* Correctly suppress empty labels for VForm fields
* Correctly disable <fieldset/> wrappers for VForms

Other benefits:
* Reduce code duplication related to $getFieldHtmlMethod
* Introduce HTMLForm::factory() method for constructing HTMLForms

Related cleanup:
* Correctly style 'reset' buttons in MediaWiki UI mode
* Label $wgHTMLFormAllowTableFormat as a mobile-specific hack
* Display checkboxes normally in MediaWiki UI mode (removed weird
  override that only broke things). Also, always render checkboxes
  in VForms as .mw-ui-checkbox.
* self:: → static::

Bug: T85285
Change-Id: I086a84f1c8cc6a16710709b7806c7f5f96462b32

Merge "Add plural support for message config-no-db in the installer"

Merge "Add Timing-Allow-Origin header for cross-domain API responses"

Add plural support for message config-no-db in the installer

The message config-no-db contains a comma separated list of items.
Adding the count of this list allows use of plural for better i18n.

Change-Id: I07ccbd3f664709d1a6234cb1e268ff8d7ddfabd9

Made SqlDataUpdate use lazy DB connections

Change-Id: Ic041f620f27e1e8012586542ac6dc60657461d04

Always set group_concat_max_len to avoid awful truncation gotcha

Change-Id: I1000412a0c58a9d76ea25f6ecc37a3a464aa430a

tests: Move npm-test to repo root

This is way overdue as it isn't experimental anymore.
Keeping it in a non-standard subdirectory makes it harder to
discover for people. And makes it harder to run from Jenkins
since this is already being ran on every commit.

Related to 70f3919.

Change-Id: Iaf5cf65616f82640145fbb6395c36129428602ca

Merge "Allow mobile for jquery.throttle-debounce"

Merge "installer: Don't generate $wgDefaultSkin='' when no skins are present during installation"

Add Timing-Allow-Origin header for cross-domain API responses

This makes it possible to get detailed network timing information
via ResourceTiming.

Change-Id: Ie88d4354285420014c0f1612446ba94fc2a8c68f

Allow mobile for jquery.throttle-debounce

This will allow Flow to use core's module:
Ia59e14dc8d7838cfe237b99ee59053529a7235fa

Change-Id: Ia6502da0e10dfa91732b7ac7ac9921dc5cfd218c

Merge "mediawiki.js: Clean up and improve documentation"

Merge "mediawiki.test.js: Document wgSiteName can't be mocked"

Merge "mw.loader: Remove support for IE6"

mediawiki.js: Clean up and improve documentation

* Consistently phrase one-line captions in the imperative mood.

* Consistently start parameter descriptions with a capital letter.

* Document a few missing parameters and callback parameters.

* Rewrite mw.Map example.

* Merge duplicate @return tags in mw.Map#set (only the first one
  is parsed by JSDuck).

* Fix mw.loader#registry code example. In order for the indented
  text to be parsed as a code block, an empty is required.

* Improve IE-support comments with more details.

* Remove redundant String() cast in newStyleTag(). Per its signature,
  it only supports strings. Other values don't make sense.

Change-Id: Iaaeaea4869f7869f5f4d29b27d11c0b05aad2ef5

installer: Don't generate $wgDefaultSkin='' when no skins are present during installation

The bug only affects people installing from Git, not tarball releases.

* Make sure getDefaultSkin() never returns null and just uses 'vector'
  when no skins are available (so the user just has to install it).
* Produce a hidden input field to pass the generated value to the next
  installer page.

Change-Id: I025f1aeb182a818de59a42df01591e01fc9e6236

mediawiki.test.js: Document wgSiteName can't be mocked

Can't make it use mw.config value via setup/teardown because
the src module caches it at load, which means the site's live
config value is used...

Change-Id: Ie941fa138273ec01ab3dc93ea7d75c1da75c5ee9

mw.loader: Remove support for IE6

The client loader doesn't run in IE6.

Since MediaWiki 1.24, the startup module provides IE6 a
javascriptless environment.

Change-Id: I16628e21d6a89a31dc65baf4a22c6fb31dde2535

Localisation updates from https://translatewiki.net.

Change-Id: Id6b7c7c05c7a9d42f2cac2f7fd32cfbe415a9b0e

registration: Ignore attributes that start with @

Allow these to be used for fake comments or other information
that should not be loaded.

Change-Id: Id79cd8b18988b94db565b2ddbc31ee6f17a89fca

Merge "Run the ImagePageShowTOC hook before adding the 'metadata' link"

registration: Sync extension.schema.json with reality

* info-files was never implemented
* path is no longer needed since we already know it

Change-Id: I0212620cb3a32e506717bd39a2fff4c9589f11cd

convertExtensionToRegistration: Put some keys (name, version, etc.) on top

Bug: T86608
Change-Id: I4b8e48e8d2d23c3b7e3f0b49bfa9c8941b5a3434