Merge "Fixes to Special:BookSources form"

Merge "Make concurrent runs of FileBackendTest independent"

Merge "Add gender to 'passwordreset-emailerror-capture' message"

Add gender to 'passwordreset-emailerror-capture' message

Bug: 46646
Change-Id: Ib775597d7d5b5afd84717fc7064a73721bc1a01c

Make concurrent runs of FileBackendTest independent

Make each run of the FileBackendTest be for a different "wiki"
so you could have multiple runs at the same time without them
interfering with each other.

There's been intermittent failures on jenkins for this test. I'm
not sure if its some race condition (which wouldn't really make
sense given each run should be independent I thought), but this
can't hurt.

Change-Id: I3b639f1c783efc140028c3acdce35ef3f58bf66f

Localisation updates from translatewiki.net.

Change-Id: I29386c98e8d9308a6cbebe8b654834996ac36b0c

Merge "Set up the profiler in CLI mode."

Merge "[FileBackend] Simplified the shard list iterator to use AppendIterator and FilterIterator."

Merge "Reduced DB contention in User::saveOptions()."

Merge "Small doc fix to JobQueueRedis."

Merge "Set 'skin' template parameter rather than override parent method"

[FileBackend] Simplified the shard list iterator to use AppendIterator and FilterIterator.

Change-Id: I22f79447b1edec4fa6d7fc06d67f3f831a484f16

Reduced DB contention in User::saveOptions().

Change-Id: Ic91501cd6476dae54b54b85f2f06c25bd2577c9b

Small doc fix to JobQueueRedis.

Change-Id: I217b3642c67338f7d45995d81e5e18887242479a

Set up the profiler in CLI mode.

Change-Id: I7f36786573870b66b4f1e93c2fc2e444f1ae1252

Move release notes from 1.21 to 1.22

Several change sets that added entries to RELEASE-NOTES-1.21 were
actually submitted after the REL1_21 branch point. Those change
sets that are unlikely to be backported to 1.21 (i.e. not yet
backported) are listed here.

* I0f2d81dd (bug 23393) (bug 45803)
* I226349aa
* I3faa9c3e (bug 23343)
* I6e7f4cf6
* I719d84c5
* I97b04673
* Ia98f54f4
* Idab489fc (bug 25325)
* Ifaf40c66 (bug 46680)

This patch moves the release notes to the RELEASE-NOTES-1.22 file.

Change-Id: I6a51e3ee07fe7622b9c708c78563795d7a1118fc

Rebuild MessagesEn.php

php maintenance\language\rebuildLanguage.php --lang=en

Change-Id: I6a51f75b393e7c85b93a79bc1f09c57ad7e79f93

Clean up old way to disable variants (using a special name 'disable')

$wgDisabledVariants is the preferred setting now since 1.16. For some
reason, the 'disable' magic name hasn't been cleaned up completely and
only affects skinning.

Someone may want to run rebuildLanguage.php later because messages.inc
is modified here. I didn't run it because it introduces too many
unrelated edits messing up this patch.

Change-Id: Ifb005e7cc9efd0b4a4f87fd491a71c8d5e1a5748

Set 'skin' template parameter rather than override parent method

BaseTemplate's getSkin() method is perfectly adequate if you simply set a
skin value where it expects it. Doing so obviates the need to reimplement
getSkin() in UserloginTemplateVForm and UsercreateTemplateVForm.

Change-Id: Icd22de2218fc96cb67c11d0c1763b607743fe3b6

Support HTTP timeout detection in curl backend

We already do this for the pure-PHP backend, and it's easy to
catch this single error and pass a more specific message

Change-Id: I8a81dd90b691c35c51a19f8790f06578dd067613

Localisation updates from translatewiki.net.

Change-Id: Ibba6af6cc653791657989c97d74ed2a6a4c25014

Use onTransactionIdle() for page invalidations when not using the job queue.

* This will do the page_touched UPDATE in autocommit mode to avoid
  locking a bunch of titles for some unknown amount of time that
  depends on how big the transaction is.

Change-Id: I6ec16a73c11fbf19c24d2b7fe7d01f91b9dcf33a

Some tweaks to the database commit callback docs.

Change-Id: Ifc1687f5393046d000a49aa7850c6454d03154b8

Account for $wgSquidMaxage in OutputPage::checkLastModified().

* This deals with the fact that seldom edited pages can end up cached
  with very stale resource (JS/CSS) references since the response to
  IMS GET requests will be 304 Not Modified if page_touched is ancient.
  When squid revalidates its stale cache it will keep getting 304s and
  renewing the TTL on the stale cache.

Bug: 44570
Change-Id: I3889f300012aeabd37e228653279ad19b296e4ae

Merge "Cleanup and performance tweaks for BacklinkCache."

Pushed updateCategoryCounts() to transaction end to reduce contention.

* This can really be a problem if locks are held during file related operations.

Change-Id: I00107ffc9951915d3af6002b258b3dae2de5fa92

[Database] Added onTransactionPreCommitOrIdle() function.

* Also fixed case where onTransactionIdle() could start uncommitted
  transactions if there was no transanction and DBO_TRX was set.

Change-Id: I0bf6171fac692cf3d6e04011321bed075f58724b

Merge "Set Agora font stack to mw-ui-container in Vector"

Merge "Localisation updates from translatewiki.net."

Merge "Disable external entities in Import"

Merge "Add git HEAD date to Special:Version for core and extensions"

Merge "Disable external entities in XMLReader"

Merge "Sanitize $limitReport before outputting"

Disable external entities in Import

Temporarily disable loading entities in XMLReader when calling read()
during import.

bug: 47251

Change-Id: I0b39386e6cf4ec0244aab8ebc4095922511e2964

Disable external entities in XMLReader

Temporarily disable loading entities in XMLReader when calling read()
with libxml_disable_entity_loader(true).

bug: 46859

Change-Id: I0b2ef270f15c7b4da17edee680bf7e2410919915

Sanitize $limitReport before outputting

Prevents possible injection of "-->" and other HTML by extensions using
the ParserLimitReport hook.

bug: 46084
Change-Id: Id97b6668da6df3e5e4c0acefffa00c82cac3c44a

Add git HEAD date to Special:Version for core and extensions

The patch adds the localised commit date of i) core and ii) extensions
in the Special:Version page tables. It requires the Git version control
system being installed, which is checked during the installation.

Introduces a new parameter for the git binary in DefaultSettings.php:
$wgGitBin = '/usr/bin/git';

Patch authored by DaSch <dasch@daschmedia.de> and updated and fixed
by Wikinaut<mail@tgries.de>.

Bug: 38783
Change-Id: I0931400ecacf91ed2ab4fc7aa46dceac17661768

Localisation updates from translatewiki.net.

Change-Id: Ie18fd3023c90008b80d8bf3eb888dce4d872fd90

Merge "Localisation updates from translatewiki.net."

Merge "Moved JobQueueDB::recycleAndDeleteStaleJobs() function below overriden ones."

Merge "(bug 42607) $.tablesorter: require separators when detecting dates"

Merge "5 new tests (3 Parsoid serializer, 2 parser) & fixed 4 tests."

Get rid of wfScript in MediaWikiSite::normalizePageName

wfScript should not be used to construct URLs for calls to other wikis

Bug: 47242
Change-Id: Id5aaa58f10d513e082b217aeb8d88b8228341a48

Removed old cache update b/c code from 2007.

Change-Id: Ia19062cb48f7748501c061544b2949bf3beaba4d

Merge "Fixed incorrect parameter passed to LoginForm::userBlockedMessage()"

Merge "Added spaces before and removed spaces after 'array'"

Fixed incorrect parameter passed to LoginForm::userBlockedMessage()

The error is Catchable fatal error: Argument 1 passed to
LoginForm::userBlockedMessage() must be an instance of Block, null given.

The problem is that User::$mBlock is not always the correct block, when
checking that a blocked logged-in user can create an account.

bug: 47138
Change-Id: I43f35cfa342737f05a08a693a86949c14348321f

Added spaces before and removed spaces after 'array'

Fix some other spacing while at it

Change-Id: I13f81533eaf40e06c13cf377c0f08ef5cef01d00

Merge "Use a provider for Xff blocking tests"

Update/add documentation

Change-Id: I897d5833cdaf7dc2d3bca5bb5551958611f22bd9

Use a provider for Xff blocking tests

Move the block insertion to addDBData(), and use a provider to
itterate through the test strings.

This seems like the more correct way to run these tests, although it
still seems hacky, since the db isn't cleaned up after each test.

Change-Id: Ib22ce69797ec03a61b0cc330e8572865290ebd31

Merge "Added/Removed spaces around string concatenation"

Localisation updates from translatewiki.net.

Change-Id: I019d72ecef04cd1851c951d76818afb81ae3f763

Merge "JSHint: Re-order jshintrc options to split common from project settings"

Added/Removed spaces around string concatenation

And added/removed spaces around some other tokens,
like +, -, *, /, <, >, =, !

Fixed windows newline style

Change-Id: I0b9c8c408f3f6bfc0d685a074d7ec468fb848fc8

JSHint: Re-order jshintrc options to split common from project settings

There are certain options required by our coding conventions.
The others are specifically overriding it for core. Either
because we have certain code that needs laxing or because we
can be stricter.

Comments can be used in jshintrc since a while now. Though
they are invalid JSON, many parsers (including JSHint) allow
them for documentation purposes.

Change-Id: I43ace21208121e866900b2cbe1efbcebadd2946a

Merge "Various code and doc cleanups to ChronologyProtector."

Set Agora font stack to mw-ui-container in Vector

We moved the language links outside the new compact vertical form
layout, hence also need to apply the font stack to mw-ui-container when
using Vector skin.

Also
* a comment-only CSS change
* set relative_assets = true in Compass sourcefiles configuration

Bug: 47187
Change-Id: Id064dc8ada5c43c33e17d1e66d327ba05e918657

Merge "Fix sequence issue with /signup URL form"

Some comments for Preferences.php

Change-Id: I65d448acf1a53c95efe94587b695718f3231f538

Merge "Update/fix docs"

Update/fix docs

Change-Id: Ifa3cfc6fe38ea45e70b50683c36e7cbf3bb4ef82

Fix sequence issue with /signup URL form

/signup subpage affects form display and title (just like ?type=signup),
so check for it earlier in execute().
Also rename execute()'s $par parameter to $subPage to match SubPage.php.

Change-Id: Ida615a1039bfa1d8328f5fda24c4e3da66b9d162

Merge "Update limit/urlwidth param doc of prop=imageinfo"

Merge "Localisation updates from translatewiki.net."

Made $wgMaxBacklinksInvalidate actually work.

* The check never worked before due to the 200 limit passed to getNumLinks().

Change-Id: I009c757c6437f22544a8d7d09d3534950c32c887

Localisation updates from translatewiki.net.

Change-Id: Ie707fb7d84d7ceb47089a618707a05c45c29e7f1

Merge "Support for linking directly to sections in ApiFeedWatchlist.  It works only if a section name is present in item's comment."

Use wfAppendQuery to append a query

Also pass a array to wfAppendQuery, which than does the urlencoding.

Prefer a wfScript() over the global or string

Change-Id: Icada534cb4c99c9441938a2e8dcbc11a142360c6

Merge "Add devunt to CREDITS as patch contributor"

Add devunt to CREDITS as patch contributor

Contributions to core are in Gerrit changes I316685262, I42be68c6a, and I45d04691a.

Change-Id: Ic917af8e0c7c9f49528a7309e22c0a60505ba5e8

5 new tests (3 Parsoid serializer, 2 parser) & fixed 4 tests.

* Updated html output of two parsoid-centric tests to reflect
  changes in parsoid output.
* Updated 'disabled parsoid' options on two tests to 'parsoid'.
* Added three more tests for the Parsoid serializer.
* Added two tests documenting differences in PHP and Parsoid
  output for overlapping HTML tags (sometimes seen on wikipedia
  pages).

Change-Id: Id72fc5de5a09490028b64a522db3461970008960

Merge "Revised design of Special:Userlogin"

Various code and doc cleanups to ChronologyProtector.

Change-Id: I4d76e4c627e399f733d078c5cc97020f363094c4

Merge "Add ChronologyProtector to ExternalLBs"

Revised design of Special:Userlogin

If a global variable is set or if you add ?useNew=1 to the query string,
Special:Userlogin loads a different login template (UserloginVForm.php)
with the new Vertical Form appearance and different messaging.
Otherwise the current unchanged template renders so that wikis can cut
over to the new look when ready (with messages and links). Once they do
so, the variable and flag will be retired.

The new template applies mw-ui-vform and mw-ui-button styles defined in
a new 'mediawiki.ui' CSS module in core to create a compact vertical
form.  The mw-ui styles specify a Helvetica font stack (that we tested
in the Account creation experiment) in the form area, but NOT if the
user is using some other skin than Vector.

The CSS code is developed using Sass. The patch includes the
Sass scss files, along with a Makefile that uses their Compass build
configuration (config.rb).

The redesigned Special:Userlogin also:

* Displays a "secure login" link if HTTPS is available.
* Loads additional CSS for its form features (e.g. more attractive
  errorbox, "Join wiki" messaging).
* Defines new "userlogin-xx" messages; many are the same as
  existing messages but without ':' on the end.
* Uses a distinct title for Login instead of generic "Log in /
  Create account".
* Removes the [mailmypassword] code branch from its login template as it
  is never executed.

Bug: 44628
Change-Id: I489042c50aa060c90ca18b05097dbe25c4ae6395

Merge "Moved ChronologyProtector to its own file."

Merge "Fixed use of __METHOD__ in a closure."

Merge "phpcs: Fix Squiz.WhiteSpace.ScopeClosingBrace.ContentBefore violations"

Moved ChronologyProtector to its own file.

Change-Id: I01bf0206322454e36633921887d10356a48ab5cb

Fixed use of __METHOD__ in a closure.

Change-Id: I41dc0ba4aff39d5a65cd1ccb3fba6c102974e6c2

Merge "Localisation updates from translatewiki.net."

Merge "Disable Special:PasswordReset when $wgEnableEmail is false"

Localisation updates from translatewiki.net.

Change-Id: I6c19864b175b2cbb84273326a981467af2db8610

Disable Special:PasswordReset when $wgEnableEmail is false

Previously one could still navigate to the page by entering its URL directly,
and from there be able to request a password reset e-mail for any user who has
an e-mail address configured. This patch adds a check for $wgEnableEmail in the
SpecialPage code, replacing the interface with an error message when e-mail is
not enabled.

Change-Id: I42be68c6a068aafdf45623eb35bb21aa684f54f6

Merge "Minor fix to redis queue docs."

Merge "Consistency tweak: Wrap variable names into <code></code>"

Minor fix to redis queue docs.

Change-Id: I27394ddc8187ab3e9c275e2d72911318cc496604

Merge "Added a quick test for getAllQueuedJobs()."

Update limit/urlwidth param doc of prop=imageinfo

Follow up Id0ec6a and to clarify things like bug 46782

Change-Id: I2215d022a47cd8ceeadd2f5d38064495f3197772

Merge "Remove externaledit/externaldiff preference"

Update documentation and styling

Remove some trailing whitespace, break long lines, make PHPDoc consistent
and update it in places.

Change-Id: I340b43a47e121a44ebd4aaa2a45dff12e945e645

Add ChronologyProtector to ExternalLBs

Change-Id: Ic6c4e86b520f7899172e3ab43e81c4d1d9637d82

Remove externaledit/externaldiff preference

Change-Id: Ic3791d188688f556753ab5556a4d9733bbf2e85e

given name correction in CREDITS

Change-Id: I1afdad0129980a988e5626351cff94c38d01688f

phpcs: Fix Squiz.WhiteSpace.ScopeClosingBrace.ContentBefore violations

ERROR: Closing brace must be on a line by itself
Squiz.WhiteSpace.ScopeClosingBrace.ContentBefore

- For non-empty scopes it means the closing brace must be on a separate
  line. This is already the case in most classes in some cases the "lazy
  closing" is still used.
  array(
   'x' ); // Moved } to next line
  function () { return 'x'; } // Moved } to next line
  case 1:
    stuff; break; // Moved break to next line
- For empty function it serves as a visual distinction between there not
  being a scope block / function body and there being an empty function
  body.
  function foo(); // No body
  function foo() {} // Empty body - violation
  function bar() { // Empty body corrected
  }

Change-Id: I0310ec379c6d41cc7d483671994d027a49f32164

Add @since line for User::groupHasPermission()

Change-Id: Ibcd20a3921058a010334e0e0983e2d9541aea734
Follows-up: I41edb091fa35c8c68b6f95cc5fd208ea99418cdb

Merge "Minor coding style whitespace cleanup"

Minor coding style whitespace cleanup

Change-Id: Ic066eda3a1a2db51b2a953a81829ce7f92568125

Localisation updates from translatewiki.net.

Change-Id: Id27f47305ed695264d422df9fa51d6492b838b88