Introduce ApiFeedContributions::feedItem hook

ContribsPager, which is used by ApiFeedContributions, can return more
than just revision rows.  This is handled in the html side within the
ContributionsLineEnding hook.  ApiFeedContributions had no special
handling so here I have added a simple hook the provides the data
from ContribsPager and allows subscribers to provide the appropriate
FeedItem instance.

Bug: T85229
Change-Id: I27c77cc682ba801c40361c76b67398108ca1a592

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

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

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

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

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

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

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

Disable styling for checkboxes and radios on non-js browsers

Change-Id: I0b9691daa61fddb484b6adff759f4413d201ae03

Merge "mw.Map: Check presence of value argument in set()"

Merge "mediawiki.js: Move mw.format definition to where it's exposed"

Merge "jsduck: Update references to renamed resources/ directories"

Merge "Move DeferredStringifier into libs, add tests"

Move DeferredStringifier into libs, add tests

Change-Id: I384d1a3854e957315584d30ec58c48c02fee6a2c

Add missing @covers tag

Change-Id: I491929e3d77688bf07640db4218ef99d6b888a82

Merge "Add User::equals"

Merge "Add plural support for message upload-permitted on Special:Upload"

Add plural support for message upload-permitted on Special:Upload

The messages upload-permitted, upload-preferred and upload-prohibited
containing a comma separated list of items. Adding the count of this
list allows use of plural for better i18n.

Change-Id: I3fb09db4788bff6ea89731f77848d78fb33dd493

Add User::equals

Seems stupid omission. Title has one. Why do I need to think how
to determine how to users objects point to the same user. Allows
more expressive code.

Also fixes a bug in multiple places where users "0" and "00" were
considered equal.

Change-Id: I682392e564b332b77ab489f2ad394fa2d28098a5

mw.Map: Check presence of value argument in set()

Add back the `arguments.length > 1` check (accidentally removed
in 24f84b0). Otherwise it inadvertently uses the `value`
parameter, causing it to set undefined as the value.

There was already a test ensuring undefined can be set as value,
but a test to ensure it doesn't default to undefined was missing.

Change-Id: I4c69f0c11f165640a9b387a72c77c48eb6aa9e72

mediawiki.js: Move mw.format definition to where it's exposed

No need for it to be a local and public method at the same time.
The Message#parser can just use the public one.

Change-Id: I57e040cda6da57cda447ea83f5b9b6b28b538158

jsduck: Update references to renamed resources/ directories

The inline examples in the generated documentation due to a
404 Not Found for the javascript files.

1. Run $ ./maintenance/mwjsduck-gen
2. View http://localhost/w/docs/js/#!/api/mw.Map
3. Try "Live Preview"

Change-Id: Ic44f029c789042d82ec14e35d385f9ccc59959aa

Don't auto-select first alphabetical item in import subproject list

Currently when you load Special:Import, the first subproject in
alphabetical order gets selected. Instead, the first subproject in the
list should be selected by default (i.e. no <option> element should be
given the "selected" attribute).

Change-Id: I442a9afadc75f42f63d8c9c32ee2920374ceb26d

Handle more cases in convertExtensionToRegistration.php

* Handle constructs like $wgResourceModules += array(...)
* Support $wgMessagesDirs['FooBar'] = array(...)
* Handle stripping paths where it is the current directory
* Use FormatJson::ALL_OK when writing the file

Bug: T86311
Change-Id: I9434f6634fd6e460a5c2f6dac95c9065da35a51a

registration: Add ConfigRegistry to the extension.json schema

Change-Id: I6918215e3b5b2dd4c993d8a8767dc3ce998bd5ed

exception: Hide suppressed errors in 'error' log, flag in 'error-json'

Follows-up 399ba2f.

Bug: T75619
Bug: T45086
Change-Id: I1f312660c058a3940bf1e9425f86cfd531121ba3

Merge "Tweaked the message for non-MW exceptions"

Merge "Skip cache hint about 0 seconds on Special:ActiveUsers"

Don't call $wgOut->disable() from wfHttpError if $wgOut isn't set up yet

For example, if the database cannot be connected to, the language is
something other than 'en', and the l10n cache is LCStoreDB, then the
call to Language::factory() in Setup.php line 604 will do it when it's
trying to get the fallback list.

Bug: T86398
Change-Id: Ia0e9ea440dcc4c8ab7bab1f33802aeff487eb3f0

Localisation updates from https://translatewiki.net.

Change-Id: I8622b53bb08154439b751c93d0b218f12f892c24

Merge "Check $wgCheckFileExtensions on client side on Special:Upload"

Check $wgCheckFileExtensions on client side on Special:Upload

Setting $wgCheckFileExtensions to false but having
$wgStrictFileExtensions still on true will reject unknown file
extensions on Special:Upload with javascript.

Bug: T25583
Change-Id: I260107bb5c3206044f766d308572ad27378715bb

Merge "mediawiki.action.edit.stash: set timeout to 4 seconds"

Localisation updates from https://translatewiki.net.

Change-Id: I557d9716255fa29881eef5909cfd9bf5c02e352c

Add *.orig to .gitignore

People and tools sometimes accidentally commit these, let's make it
more difficult to do.

Follow-up to abb4271a. NO ONE SAW THIS EMBARASSING MISTAKE.

Change-Id: If5f427586814c304b30e5783a59f33814bfeaa54

Add .*.orig to .gitignore

People and tools sometimes accidentally commit these, let's make it
more difficult to do.

Change-Id: I05574d46186642940054aaecec055723c7ade8ac

Escape unsafe messages in CategoryViewer.php

Bug: T85864
GCI: http://www.google-melange.com/gci/task/view/google/gci2014/5811264311787520
Change-Id: Ie59fb389b804f031b761f93f2c91a548742326ab

Merge "Surround edit notices with appropriate classes"