Merge "Check whether TYPE_COMBINED modules are allowed to be included in the page"

Merge "Added Html::radio, Html::check, and Html::label."

Merge "Just log exceptions instead of spamming them in JobRunner"

Merge "Apply mw-ui-input to captchas in registration form"

Merge "Revert "test: coverage for wfWikiID()""

Revert "test: coverage for wfWikiID()"

Cause unit tests to fail for some reason:

 1) WfWikiId::testReturnsProperDbName
 Failed asserting that two strings are equal.
 --- Expected
 +++ Actual
 @@ @@
 -'known_db_name'
 +'known_db_name-unittest_'

 tests/phpunit/includes/GlobalFunctions/wfWikiIDTest.php:10
 tests/phpunit/MediaWikiTestCase.php:133
 tests/phpunit/MediaWikiPHPUnitCommand.php:42

Thus making the regression job to complain whenever a patch is merged:
https://integration.wikimedia.org/ci/job/mediawiki-core-regression-master/

This reverts commit 4f55b804f20960e039671a3dbf3f3eb4a5a4b81e.

Change-Id: Ib08e40742b74be88b72af79ec4ab6d427e3b6d14

Remove return from MaintenanceFixup::outputChanneled

Parent Maintenance::outputChanneled also has no return value

Change-Id: Idae9d3e1ca70d9f0ed81b345eb4fef21294cea39

Apply mw-ui-input to captchas in registration form

Change-Id: Ie8160985799eefc2b24315008b23c447d4497dd5

Check whether TYPE_COMBINED modules are allowed to be included in the page

With the introduction of the OutputPageScriptsForBottomQueue hook,
modules loaded through it are TYPE_COMBINED, which
OutputPage::makeResourceLoaderLink was not checking on whether
or not it was safe to include them.

Change-Id: I33f39a5643b3d05db5a89e62fa6c86d437dea143

Merge "checkTemporaryPassword should call loadPasswords() first"

Merge "Revert "Add OutputPageScriptsForBottomQueue hook""

Merge "Define fallback skin using $wgFallbackSkin instead of hardcoding Vector"

Revert "Add OutputPageScriptsForBottomQueue hook"

After some discussion with Roan, it turns out this
hook is unnecessary. The proper solution is to use
addModuleScripts/addModuleStyles instead of a
plain addModules.

This reverts commit e7361de181dcadbcc7c32de92b3498c9fb9d0193.

Bug: 68712
Change-Id: Iadbff5f7fbbc5a08d6336674b12315967f45591d

Merge "Remove utilities from styleguide"

Just log exceptions instead of spamming them in JobRunner

* Even the CLI script already shows the error=X snippet.
  Logging the error should be enough, and avoids showing
  output when all we want is the JSON.

Change-Id: Iade412ea61cf427865d841ecab5498e4fcdb7e13

Remove utilities from styleguide

Document them but don't include them. Styleguide is currently written
for elements so it is rather strange.

Change-Id: I4c8bd18430d737445c14ac828d50a0844c456ca4

Revert "Update jquery.ui Vector theme to 1.9.2"

This change messed up jquery.ui.button styling on Commons, so we're reverting.

This reverts commit 7bb0c9a01eb0266696a9c3989572ae3a5b67bfdd.

Change-Id: Ibef35ce9f5a642f89bf70ec891ba1425f678ec1e

Merge "Revert "test: coverage for wfWikiID()""

Revert "test: coverage for wfWikiID()"

Introduces broken test that doesn't pass regression or HHVM.

This reverts commit 4f55b804f20960e039671a3dbf3f3eb4a5a4b81e.

Change-Id: Ib82cc9b89e7392b752e9ef98134e90186e371247

Merge "Clean up search results if scrolled of the end"

checkTemporaryPassword should call loadPasswords() first

Otherwise mNewpassword could be null.

Mlpearc reports on irc getting fatals about mNewpassword not
being an object. I couldn't reproduce, but loadPasswords() should
definitely be called here.

Change-Id: I6d1b74b79c87d1a1870971aafd1349255170123d

Merge "Added explicit type-cast to hash_pbkdf2"

Merge "Disallow top level domains in Cookie::validateCookieDomain()."

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

Merge "Unsetting the email address for a user when the email address is invalidated."

Localisation updates from https://translatewiki.net.

Change-Id: Id59d00bd9402da7fd03a34bd6d0ceb9bd2d00024

Added Html::radio, Html::check, and Html::label.

Migrated some convenience functions from the Xml
class to Html so they can be used. Only added
functions that acted as more than just a wrapper
for another function.

Change-Id: I4cc5876d4be6e04ec554444242cc049a3cff0f58

Merge "Make upload_source_file and upload_source_url more specific"

Merge "Special:Search: Insert a difference between exist and non exist page"

Merge "Restore the number of rows shown on Special:Watchlist"

Special:Search: Insert a difference between exist and non exist page

The css class of "page exist" or "create this page" should
be different when the page exists (instead of "createlink").

Change-Id: I6d4d9d88a561749ae158ca9d27e563c2c5f9b943

Restore the number of rows shown on Special:Watchlist

This reverts the result of Iada3a93762dd25fdaf35fb707304abc40bd4a5b1.
It used a param which was added to function doHeader in
I495d19258205d8713a19aa934a7657aecbe240d4.
It reused the old message to reduce translation interaction.

Bug: 62017
Change-Id: Ie722c509a6ad1346b1624abee778f4cbdc79d457

Clean up search results if scrolled of the end

If the user ends up off the end of the search results then show them a
slightly more sane page:  no more showing results 50 - 20 of 20.

Bug: 68783
Change-Id: I29354e15873054278be18d74b2355b83b86f35f3

Merge "Toolbar: Only show on WikiText pages"

Merge "Have SvgHandler create a directory for its RSVG input files"

Merge "Set "has value" flag correctly for result parameter"

Merge "Clarify comment in UploadBase"

Added explicit type-cast to hash_pbkdf2

It seems HHVM does not like the use of
dynamic typing  in hash_pbkdf2() (also it's
kind of not the best idea in general), so
added explicit (int) casting to parameters.

Follows-up: I0a9c972931a0eff0cfb2619cef3ddffd03710285
Change-Id: I8adae3cc2cdf8029be59e2d62a3ab0fbfb73b7aa

Merge "Documentation fixes for Password API-related changes"

Merge "test: coverage for wfWikiID()"

Merge "Documentation: Update $wgPasswordDefault and BcryptPassword"

Merge "Move PasswordError under includes/password/ file"

test: coverage for wfWikiID()

Basic coverage test, note that $wgDBprefix is used as a suffix!

Bug: 68231
Change-Id: Ic454ce38672b1f18db2a7f2f46c50bcfc5ee83d0

Merge "Add at least one RTL language to the TestSites mock collection"

Move PasswordError under includes/password/ file

Change-Id: I7256325492b2f6451223714efb8b3e07fca9ee3e

Documentation: Update $wgPasswordDefault and BcryptPassword

Slight syntax code change for $wgPasswordDefault in DefaultSettings.php
and fixed reference to global in BcryptPassword.php.

Change-Id: I8d1d12c09ecd2f422f21a586e948f314e29fa605

Merge "Add mw-ui-input to mediawiki ui"

Documentation fixes for Password API-related changes

A few @since 1.23 --> 1.24

Change-Id: I55d00ab9c3f9dae96ed804a22f19d713879663fa
Follows-Up: I0a9c972931a0eff0cfb2619cef3ddffd03710285

Clarify comment in UploadBase

Change-Id: Ied29298d312ca19ecfdd2196fafa89130321987c

Set "has value" flag correctly for result parameter

Change-Id: I97eeef620ddfdda6f09abbfdf0388146d4d02e36

Merge "Resources.php: Remove/deprecate 'skinStyles' references to 'minerva' skin"

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

Resources.php: Remove/deprecate 'skinStyles' references to 'minerva' skin

It doesn't even exist in core: it is part of the MobileFrontend
extension. The skin style overrides should be reapplied there using
$wgResourceModuleSkinStyles: done in I30c8ce13.

OOjs UI has some weird build process that currently requires this, though.
According to James, that will be fixed "in a couple of months".

Change-Id: If82556a41c0510a70796a09b3555d2c91a7abb52

Vector: Make better use of $wgResourceModuleSkinStyles

This gets rid of quite a lot of Vector from Resources.php and resources/.

* New $wgResourceModuleSkinStyles entries:

  * jquery.tipsy: Split off skins/Vector/skinStyles/jquery.tipsy.less
    from skins/Vector/components/common.less. It's just one rule, but
    let's do it for consistency anyway.

  * jquery.ui.*: Converted from Resources.php.
    Files renamed: resources/src/jquery.ui-themes/vector/**
    → skins/Vector/skinStyles/jquery.ui/**

  * mediawiki.notification: Still has the specificity hack, just renamed
    skins/Vector/components/notifications.less
    → skins/Vector/skinStyles/mediawiki.notification.less

* Updated $wgResourceModuleSkinStyles entries:

  * mediawiki.special: Renamed file for consistency,
    skins/Vector/special.less
    → skins/Vector/skinStyles/mediawiki.special.less

  * mediawiki.special.preferences: Renamed file for consistency,
    skins/Vector/special.preferences.less
    → skins/Vector/skinStyles/mediawiki.special.preferences.less

Change-Id: If248781e44300039eb52262d03b2f6722838dd12

resourceloader: Allow skins to provide additional styles for any module

The newly introduced $wgResourceModuleSkinStyles global enables skins to
provide additional stylesheets to existing ResourceLoader module.

This both makes it easier (or at all possible) to override default
styles and lowers the style footprint by making it possible not to
load styles unused on most pages.

----

Example:

Use the file 'foo-styles.css' for the 'mediawiki.foo' module when using
the MySkin skin:

  $wgResourceModuleSkinStyles['myskin'] = array(
    'mediawiki.foo' => 'foo-styles.css',
    'remoteSkinPath' => 'MySkin',
    'localBasePath' => __DIR__,
  );

For detailed documentation, see the doc comment in DefaultSettings.php.
For a practical usage example, see Vector.php.

----

Implementation notes:

* The values defined in $wgResourceModuleSkinStyles are embedded into
  the modules as late as possible (in ResourceLoader::register()).
* Only plain file modules are supported, setting module skin styles
  for other module types has no effect.
* ResourceLoader and ResourceLoaderFileModule now support loading
  files from arbitrary paths to make this possible, defined using
  ResourceLoaderFilePath objects.
  * This required some adjustments in seemingly unrelated places for
    code which didn't handle the paths fully correctly before.
* ResourceLoader and ResourceLoaderFileModule are now a bit more
  tightly coupled than before :(
* Included a tiny example change for the Vector skin, a lot more of
  similar cleanup is possible and planned for the future.
* Many of the non-essential mediawiki.* modules defined in
  Resources.php should be using `'skinStyles' => array( 'default' => … )`
  instead of `'styles' => …` to allow more customizations, this is
  also planned for the future after auditing which ones would actually
  benefit from this.

Change-Id: Ica4ff9696b490e35f60288d7ce1295766c427e87

Make upload_source_file and upload_source_url more specific

See https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Upload_source_file/en

Change-Id: I46e3cc58203667f9c32487aada1e5a5177dd7b6f

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

New changes:
1e7fcaa build: Remove over-ride on jscs's disallowQuotedKeysInObjects and make pass
170f607 Use native style property names instead of jQuery's css map
eb9b38f Localisation updates from https://translatewiki.net.
b812471 Unified demo page
6b06deb SVG fixes
970e35b Localisation updates from https://translatewiki.net.
bbfdfa8 Localisation updates from https://translatewiki.net.
9e8f965 Fix copypasta in WindowManager documentation
7fe05d5 demos: Rename demos.html to index.html for convenience
55beb80 demos: Add example of menu item with a long label
694e81c Update oojs to v1.0.11

Change-Id: I908933a4fa025d48e2d868a83039f7636356911c

Merge "jquery.textSelection: re-add iframe check due to regression"

Added myself to CREDITS

Added myself to CREDITS, as suggested by Vogone.

Change-Id: Ia1759a753b06bbe672a4e9ecd7d9cb5606c37866

Merge "Fixed hook documentation for removed hooks"

Merge "SpecialEditWatchlist: Avoid empty namespace sections"

Fixed hook documentation for removed hooks

Removed hook definitions for UserComparePasswords
and UserCryptPassword, rather than keep them in
the file.

Bug: 28419
Change-Id: I7bfda155986d1407caef4e3890024c5b8a66fdcc
Follows-up: I0a9c972931a0eff0cfb2619cef3ddffd03710285

Merge "Fixed calls to bogus methods in triggerJobs()"

Merge "Edit warning should get value from editors"

Remove File::getPropsFromPath() (deprecated since 1.19)

Change-Id: I11b6c582b9ce60cba03e55d55afb7cd02d218049

Fixed calls to bogus methods in triggerJobs()

* Updates some callers that were missed in 094d901b8

Change-Id: Ie425dfdc7da37331e51e3cd8c8f8d890bb5dd6de

Merge "Added password hashing API"

Merge "Remove File::sha1Base36() (deprecated since 1.19)"

Edit warning should get value from editors

Editwarning needs to make sure that any editor that uses something
other than wpTextbox1 to keep the value of the form, is able to write
this into the form before we are allowed to leave the page.

This is already supported by the textSelection api, so use that to
retrieve the value instead of plain .val()

Bug: 61117
Change-Id: I24d7096929c4eed36376ef9089d7f1b72a0b6682

jquery.textSelection: re-add iframe check due to regression

This is to fix a regression due to change Ibe011b2b of bug 29328.
The replacement jquery.textSelection of CodeEditor was no longer able
to be disable don the fly. This is because the code of textSelection
and WikiEditor is still too much intertwined.

I'm making a new patchset to strip them further apart.

Bug: 68556
Change-Id: I7c4c43ba2cde6f7c869e41ed4c5a51783cd32f82

Add mw-ui-input to mediawiki ui

Upstream component from Flow
Adds mw-ui-input component
Update existing (and undocumented) usage in core of mw-ui-input to
use Flow focus style

Updated usage of mw-ui-input in Special:Search and Special:Userlogin
Packaged in mediawiki.ui.inputs to allow developers to use inputs
separately to other mediawiki ui components.

Change-Id: Ida765b97e6574bdc8fbba88a08ec98ec12f3dc95

Merge "Update jQuery Cookie to v1.3.1"

Update jQuery Cookie to v1.3.1

Source

* https://github.com/carhartl/jquery-cookie/blob/v1.3.1/jquery.cookie.js

Change log

* https://github.com/carhartl/jquery-cookie/blob/v1.3.1/CHANGELOG.md

Notable changes:

* Removed the "raw" option from $.cookie.
  This can now only be set globally via $.cookie.raw

* Bugfix: Properly handle RFC 2068 quoted cookie values.

* New: $.cookie() returns all available cookie.

We should eventually upgrade to v1.4.1, but that has a breaking change.
Namely, it drops support for $.cookie('key', null) which needs to be
replaced by $.removeCookie which didn't exist until v1.2.0.

Change-Id: Ic664dbe7b1be2c9dd67d0f50599c36fa2dab8449

Localisation updates from https://translatewiki.net.

Change-Id: I262a6adfd236603246fa68c162a18f3fcabb72c3

Merge "Truncate block reason for whole multibyte characters"

Merge "Specifiy mixed doc param in FSFile"

Truncate block reason for whole multibyte characters

This applies to Special:BlockList, or to the block message shown to the
user, or API results. Log is not effected, because it already truncates
on its own.

Change-Id: I0a8bae5f43fd1604034d65e96c9461a9f83c0b0a

Merge "Added support for result=json parameter to runJobs.php"

Added support for result=json parameter to runJobs.php

* This can let the caller get info about what was run, just like the web version

Change-Id: I49d7eccee52eb1dc161249fdc0546259ccf1f959

Removed use of cache in JobQueueFederated pop() method

* This does not really buy anything for a number of reason and
  can cause bugs. For example, if the aggregator is notified due
  to undelayed jobs but this cache is not updated, then the runners
  will not see the jobs and will de-list aggregator entry.

bug: 68506
Change-Id: I1eacca108df59fff9c0fcc846aafaf1616cf5895

Specifiy mixed doc param in FSFile

Change-Id: I406e46da6ece2e2f9436783a0280afa7cfa9807c

Collapse nested if statements

Change-Id: I1cecfe5884edb98c8ad0a441f4d82288d597f631

Remove File::sha1Base36() (deprecated since 1.19)

Change-Id: I551a3c7451307e1a4fba3ea9b22320cc4ca72215

Merge "ResourceLoader: Test that all module dependencies are satisfiable"

Define fallback skin using $wgFallbackSkin instead of hardcoding Vector

Bug: 68332
Change-Id: Ibd7485dfa3123fb9ac3e9dd97cbe7129a0969c65

Localisation updates from https://translatewiki.net.

Change-Id: Ifc90dcf97a19e99a99e42ccd5ea10404ce75d083

Added password hashing API

Deprecated the old User::crypt, et. al password hashing
system and implemented an extensible password hashing
API.

The new Password class allows registering of child classes
and provides factory functions for creating new Password
objects. The built-in hash types are the old MediaWiki MD5
types, which are for backwards-compatibility only, and bcrypt.

Also included is support for wrapping existing hashes as well
as encrypting passwords with a configured encryption key.

Bug: 54948
Bug: 28419
Change-Id: I0a9c972931a0eff0cfb2619cef3ddffd03710285

Add comment about how to deploy new xml schema XSD files

Change-Id: I5a20594befaa5afcab44960fd892c6f798c3d729

Merge "Allow message object on HTMLForm::setWrapperLegendMsg/setSubmitTextMsg"

Remove return value from QueryPage::execute

The parent SpecialPage::execute also has no return and not all return
statements in QueryPage::execute actual return a value

Change-Id: If7a38ca5ed1107b6a5a740acae54295534950696

Fix param doc of OutputPage::setHTMLTitle

Method also accept Message object

Change-Id: I3a5254a3776f0215667e64ff9d01d60b80375aca

Allow message object on HTMLForm::setWrapperLegendMsg/setSubmitTextMsg

When a message needs plural support, it should be possible to parse a
message object with the params set.

Change-Id: Ifb67952b589a1821cde452b2be3f327f24e6b534

Merge "Revert "Make a hidden form field to keep the language code""

Remove some duplicate newlines in call to wfDebug()

Spotted on beta:

--------
commonswiki-375c0c3e:

0.0752   1.9M  mime: <image/png> extension: <png>

--------

With lot of requests, that causes them to be interlayed and ungreppable.

Change-Id: Ib99bc9547bdecb9efa59b7d292ddd7d10a552652

Revert "Make a hidden form field to keep the language code"

This reverts commit f3d9e7137ed19dbe1944567909e1f117eba0fade.

Breaks the core functionality of the special page.

Change-Id: I4a4b01e16763a7b418ff137112d2db2d42293ed1

Fix getAutosummary for create blank page

Fixes for change to enable the creation of blank pages. Added test
for this case.

Follow up: I3457c36a909d1dbfaeed04a1f0568c69e0ef3386

Bug: 57238
Change-Id: Id52be6cbe22c6098d1c0824d7a7848579aa05a84

Merge "Remove text output while running tests"

Remove text output while running tests

Change-Id: I65d9ee70adfe53267b7c0acd59fce43d4683777b

Merge "OutputPageTest: Don't assume Vector is the default skin"

Merge "Remove unused XMLReader2 class"