Merge "Get timestamp from WikiPage, instead of Article"

Merge "Fix a PHP 5.5 TODO"

Merge "Fixup contenttype stuff in UserMailer"

Merge "Updating docs for UserMailer::sendWithPear() which calls $mailer->send()"

Merge "Remove no-op parameter from UIDGeneratorTest"

Merge "Initialise array with contents in one go"

Merge "Fix typo and improve code comments in LinkTarget"

Updating docs for UserMailer::sendWithPear() which calls $mailer->send()

$mailer->send() call in UserMailer::sendWithPear() which is correct. $mailer is an
instance of Mail_smtp not UserMailer so the arguments passed to the send() method are
correct. The docs have been updated appropriately.

Bug: T68673
Change-Id: Ia85f957c60f4dafd3f450cda469b70bdcc166a8f

Merge "Move WatchedItem::duplicateEntries to WatchedItemStore"

Merge "OOjs UI: Fix #gatherPreInfuseState called incorrectly, causing TypeErrors"

OOjs UI: Fix #gatherPreInfuseState called incorrectly, causing TypeErrors

This bug is currently live. Try opening
https://www.wikidata.org/wiki/Special:NewItem
and have a look at the JavaScript console.

Introduced in If07d40a.

This patch is a manual cherry-pick from I748a4bc, which is the same
fix in the original repository. I feel we should backport this to
production.

Bug: T106313
Bug: T128075
Change-Id: I25a5deab2c287e97086726d0acbb3afeee842e0b

Merge "Add missing null type hints to Content::isCountable and implementations"

Merge "ApiSandbox: Convert all pluses in URL to spaces, not just the first one"

ApiSandbox: Convert all pluses in URL to spaces, not just the first one

This would prevent linking to any query that contains spaces.

Change-Id: I45607c1eb465d4d9bf7bbb1de7098bb1450b5043
Bug: T128053

Merge "Message.php: Update comment to current implementation"

Add missing null type hints to Content::isCountable and implementations

Change-Id: I6e3128dddd296f6d33da23332b8ff8393837b61b

Merge "Fix autofocus on Special:Activeusers"

Merge "ApiSandbox: Visual separation of fields"

Merge "SessionBackend: skip isUserSessionPrevented check for anons"

SessionBackend: skip isUserSessionPrevented check for anons

For anon requests, the call to SessionManager::isUserSessionPrevented(
$this->user->getName() ) is both expensive (because of the need to
sanitize the IP) and pointless, because the session-prevention feature
is intended for named accounts. So short-circuit the check if the user is not
logged in.

Change-Id: I17386b97e229b492723b46db1e1ae16fd4b0fc5a

IDatabase::commit: Docs re 'flush' exception on explicit transaction

Change-Id: I44087e92d2186f4e931b3f0766caba1f90f4ebc4

Merge "Fix validation error on Special:Emailuser"

Localisation updates from https://translatewiki.net.

Change-Id: Ia35e1b9cc7ac053de010cde3accc1f988a7e965a

Fix validation error on Special:Emailuser

Xml::inputLabel does not know about boolean parameters like autofocus,
so add it with an empty value

Fix validation error:
Bad value 1 for attribute autofocus on element input.

Change-Id: Ide77585fcf24f8ecd26dddac3610a134ba78dce9

Message.php: Update comment to current implementation

Change-Id: I04f6b42380e6b8eeb5bf0d679e8fb65cb4696d1a

Fix autofocus on Special:Activeusers

Xml::inputLabel does not know about boolean parameters like autofocus,
so add it with an empty value or omitted it.

Fix validation error:
Bad value 1 for attribute autofocus on element input.

Change-Id: Idb7a7b58079d6c44b5a7f1fa226218bb685c8241

Add $lang parameter to SpecialVersion::getVersion

Usage in Scribunto with I712e048367d9d.

Bug: T127233
Change-Id: Id14733aaef3e52a2e315bffe74baeb926d46e238

Change "edit" to "create" in the message noarticletext

Change-Id: I0437c30969afbe65723bb0c8130b3636ea059246

Merge "Add used messages to LogFormatter::getMessageKey overrides"

build: Tweak jscs jsDoc rule 'checkAnnotations' and make pass

* Don't override "private", the default is to require empty
  values for the tag

* Move some @private tags after descriptions so the latter
  are not mistaken for tag values

* Require some value for @context, @source and @see tags

Change-Id: I583bb2ff3ec6537e1419732aaf1644ea8feb2255

Merge "Add missing 'public' keywords to some MediaHandler methods"

Add missing 'public' keywords to some MediaHandler methods

* getParamMap
* validateParam
* makeParamString
* parseParamString
* canRender
* mustRender
* isMultiPage

Change-Id: I0c00a61abf9be400dca5407370ca2c232d57ae17

Merge "mediawiki.ui: Buttons should receive `max-width`"

Initialise array with contents in one go

Change-Id: Id9ea7a8f17905a0e1e3c0ff12a9ba6c54de54dbe

mediawiki.ui: Buttons should receive `max-width`

Applying `max-width` to mw-ui-buttons and center-aligning subgroup
when class `mw-ui-block` is available.

Bug: T95367
Change-Id: I4aaf24042f451c3b5489f49098f4b9e7eeef8098

Fix parserTests: Set scriptPath in setupGlobals() as well

Follows-up to 2eeda9bc7, which only set it in setUp().

Change-Id: Ifafaeaab679f51d452419bea223a955afcc63b51

Localisation updates from https://translatewiki.net.

Change-Id: I38e60872965bf5f1f0074c41a9b3773487331fa8

Merge "Sync up with Parsoid parserTests."

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 239d2e60d82c736905a935525db748ebf6b45e23

Change-Id: Iec62b6824ab7d4c5d660999af130c777f8621571

Revert "Log multiple IPs using the same session or the same user account"

This reverts commit f22549a60539c9aa5c5390c8417c984ba8eef5b2.
Per T125455#2054194.

Bug: T125455
Change-Id: Ic2049381e98586e91974fc5b47d9e857a73414a4

Merge "resourceloader: Remove $wgUser optimization that uses session "

resourceloader: Remove $wgUser optimization that uses session

Remove optimization that avoids unstubbing $wgUser because $wgUser
is not a plain object for the current user name.

It's a stub with mFrom='session' until methods are called that need
the info, such as getName(), which then lazy loads the object based on
the session information.

We want to make load.php session-less.

Bug: T127233
Change-Id: Ica482e5d1892cb29456e6f2a91cd70017cf414c5

Merge "JsonContent: Call member functions with '$this->', not 'self::'"

Merge "Avoid pointless doBatchLookups() query for user contributions"

Merge "CachedBagOStuff: cache backend misses"

Merge "Fix class name of BcryptPasswordTest"

Merge "Add toggle checkboxes to action=history"

Merge "Remove wfCheckEntropy() as unused and deprecated in 1.27"

Add missing namespace to @covers comments

PHP_CodeCoverage_Exception:
> Trying to @cover not existing method "SwiftFileBackend::sanitzeHdrs".
> Trying to @cover not existing method "LineFormatter::normalizeException".
> Trying to @cover not existing method "MonologSpi::mergeConfig".
> Trying to @cover not existing method "ProcessCacheLRU::het".
> Trying to @cover not existing method "BitmapHandler::swapICCProfile".
> Trying to @cover not existing class or interface "checkParseSafety".
> Trying to @cover not existing method "Article::__call". (was removed).
> Trying to @cover not existing method "ExtensionProcessor::extracttExtensionMessagesFiles".
> Trying to @cover not existing method "FileContentsHasher::getFileContentHash".

Makes code coverage run fail at the moment. These used to be warnings
in PHPUnit 3.x, but are now hard exceptions in PHPUnit 4.x when requesting
a coverage report.

Change-Id: If7f45ca57fd7d480d35b1414a889398837c0c472

Add missing namespace to @covers comment in LegacyLoggerTest

> Trying to @cover or @use not existing method "LegacyLogger::interpolate"

Makes code coverage run fail at the moment.

Change-Id: I8417b5c2f1fc116583758c7507770c796127bb67

parserTest: Make $wgResourceBasePath match $wgScriptPath

Right now it forgets to reset $wgResourceBasePath, which means it
is inherited from the wikis's (or Jenkins') default settings which
is typically '/w'. That caused parser tests to behave as if pointers
to /extensions were outside /w.

Also update wgScriptPath to be '' instead of '/'. Otherwise this
can cause double-slash prefixed urls.

Change-Id: Ic455d62fca8fcac2c4ecc055cc0d7e311b70a94a

Merge "Upgrade from PHPunit 3.7.x to PHPUnit 4.8.x"

Merge "Fix parameter parsing in parserTests.php"

Merge "Fix resource path check when ResourceBasePath is an empty string"

Fix parameter parsing in parserTests.php

Change-Id: Iafb8a60f040cd94eb1b8c30ea5ace9cff4272c0e

CachedBagOStuff: cache backend misses

Cache misses from the backend cache the same as hits.

Bug: T127772
Change-Id: If2fe1920411b24862acea888c627db13717da8bd

Fix a PHP 5.5 TODO

Change-Id: Ieb3bc27e4f33e170b2217ab8eab58f5309083e80

Upgrade from PHPunit 3.7.x to PHPUnit 4.8.x

Major changes:

* (4.0.0) Limited support for stubbing and mocking static methods was removed.
          (introduced in PHPUnit 3.5)
* (4.0.0) The addRiskyTest() was added to the PHPUnit_Framework_TestListener interface.
          Classes that implement this interface MUST implement this method.
* (4.5.0) The "--strict" catch-all option was deprecated. Use specific options instead.

Notable changes:

* (4.6.0) Tests not annotated with @small, @medium, or @large are no longer default
          to matching in group @small.
* (4.6.0) Result printer now checks hasColorSupport().
* (4.8.0) The name of the default group was changed from "__nogroup__" to "default".
* (4.8.0) This is the last release to support PHP 5.3, PHP 5.4, and PHP 5.5.
          PHPUnit 5 will require PHP 5.6 or higher.

PHPUnit 3.7.17 was released March 2013.
PHPUnit 4.8.0 was released August 2015.
PHPUnit 4.8.23 was released January 2016.

Details:

* https://github.com/sebastianbergmann/phpunit/wiki/ChangeLog-for-PHPUnit-3.7
* https://github.com/sebastianbergmann/phpunit/wiki/Release-Announcement-for-PHPUnit-4.0.0
* https://github.com/sebastianbergmann/phpunit/wiki/Release-Announcement-for-PHPUnit-4.1.0
* https://github.com/sebastianbergmann/phpunit/wiki/Release-Announcement-for-PHPUnit-4.2.0
* https://github.com/sebastianbergmann/phpunit/wiki/Release-Announcement-for-PHPUnit-4.3.0
* https://github.com/sebastianbergmann/phpunit/wiki/Release-Announcement-for-PHPUnit-4.4.0
* https://github.com/sebastianbergmann/phpunit/wiki/Release-Announcement-for-PHPUnit-4.5.0
* https://github.com/sebastianbergmann/phpunit/wiki/Release-Announcement-for-PHPUnit-4.6.0
* https://github.com/sebastianbergmann/phpunit/wiki/Release-Announcement-for-PHPUnit-4.7.0
* https://github.com/sebastianbergmann/phpunit/wiki/Release-Announcement-for-PHPUnit-4.8.0

Bug: T99982
Change-Id: I70ad81513812aa76a84fa55be14135aac94d55e8

Avoid pointless doBatchLookups() query for user contributions

This shows up in slow query reports a lot.

Change-Id: Ic1cf45fae87655e0b9d09fdc96aa2b2c22de2c11

Merge "Update OOjs UI to v0.16.0"

Merge "Enforce MW_NO_SESSION, add MW_NO_SESSION_HANDLER"

Update OOjs UI to v0.16.0

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

Change-Id: I9c170fce1c65ae65b0b33342572fe4d3bb613774

Fix resource path check when ResourceBasePath is an empty string

If you have MediaWiki installed in the root of the domain, then
$wgScriptPath and $wgResourceBasePath is an empty string. In HHVM and
PHP, passing an empty string as the second parameter of strpos() causes
a warning and returns false, which will cause the condition to fail, as
if the path were not within the base path.

So, normalize such paths. Using substr() instead of strpos() for a
"starts with" check would have worked except that RelPath also fails
when given an empty string.

Bug: T127652
Change-Id: If7e94ae638d6834f7cc0f31f67a5fe6a2f74771c

build: Update grunt-contrib-jshint from 0.12.0 to 1.0.0

Changelog at
https://github.com/gruntjs/grunt-contrib-jshint/blob/v1.0.0/CHANGELOG

This release includes a fix to the checker because sometimes it would
not show where the error was coming from.

Change-Id: Ia2e7d6520b760bdc80f7e24051ec578954aa0706

Localisation updates from https://translatewiki.net.

Change-Id: I2d18d11d274f565e92623563b5f0550a5ed527c1

JsonContent: Call member functions with '$this->', not 'self::'

Change-Id: I7dab69ff77c98439f90e7d79f31d0e9128728683

Fix function call casing

Change-Id: I20600c865b9f239ac281a1f8c5776ad4dbec4359

Add used messages to LogFormatter::getMessageKey overrides

Change-Id: I55e943ab2788e7fee022d40cc3d74ed479788f3b

Fix class name of BcryptPasswordTest

Tests are named Test, not TestCase

Change-Id: I3725f8bebde5896fde24e36bd6b6e25dd8492a91

Merge "Fix session store logging"

Merge "Use hex2bin() instead of pack()"

Enforce MW_NO_SESSION, add MW_NO_SESSION_HANDLER

When an entry point specifies MW_NO_SESSION, actually enforce that by
having both SessionManager and PHP's session handling (session_start()
and friends) throw exceptions.

If an entry point needs the old behavior of using PHP's default session
handling (as defined in php.ini), it should define
MW_NO_SESSION_HANDLER instead of or in addition to MW_NO_SESSION.

This also makes PHPSessionHandler be installed in CLI mode, where it
wasn't installed before.

Bug: T127233
Change-Id: I2a3db06ee8e44a044096c57a819b5fd5e51c5c5c

Fix session store logging

Two things were wrong here:
1. SessionManager shouldn't override the configured logger of the
   underlying store.
2. CachedBagOStuff shouldn't override the configured logger of the
   backend store on construction.

Change-Id: I24ed5a81d5b238a42934a7890dd8d0a9f9bb3b6f

Merge "Remove some unused variables"

Merge "mw.Title/NamespaceInputWidget: Use wgNamespaceIds for namespace constants"

Merge "Tidy: <source> and <track> are empty elements"

Fixed Selenium tests by removing skins not installed by default

Looks like the tests were developed targeting
en.wikipedia.beta.wmflabs.org that has 4 skins by default. The commit
removes checks for all skins but Vector, which is installed by default. (At least in
my MediaWiki-Vagrant installation.)

Bug: T94150
Change-Id: I68f864bd194dbb251acab3d94699f1e701300ab2

mw.Title/NamespaceInputWidget: Use wgNamespaceIds for namespace constants

Avoids duplication of numerical constants.

Fix instance of variable changing meaning (ns = ns.toLowerString()).

Change-Id: I0e21c51d0e1bdf8340d5d7c927cee7619171e14b

Merge "Deprecate PrefixSearchBackend"

Merge "Don't URL encode titles when revdeling from Special:Undelete"

Match html5 unquoted attribute parsing

 * Brings us closer to the html5 attribute parsing algorithm described
   in http://www.w3.org/TR/html5/syntax.html#before-attribute-value-state

 * There's a similar patch for the Parsoid in,
   I2160a23b2a3c914eb369347bbf5d58328440041d

 * The spec says <div class=  style="123">hi</div> should parse as
   <div class="style=\"123\"">hi</div>, which it now does, whereas it
   used to yield <div class="" style="123">hi</div>.

 * Merge with caution. This is going to break pages like,
   frwikisource/La_Mirlitantouille_(Lenotre)?oldid=4669681

Bug: T108134
Change-Id: Ic2fc1b573a55a847e6c05707678b58c1189ecc52

Merge "Compress PNGs with zopflipng"

Localisation updates from https://translatewiki.net.

Change-Id: I9d1b78ddb3fa02956e4b01c137691cf6f290b958

Don't URL encode titles when revdeling from Special:Undelete

URL encoding is here handled by getFullURL so calling wfUrlencode resulted in
double-encoding.

Bug: T127091
Change-Id: Ibc32afb80bd220b87d2c4a816398cc10345e7ac4

Add toggle checkboxes to action=history

This breaks out the toggle checkbox code into a separate class in
includes/ListToggle.php

Bug: T92230
Change-Id: I8d1aefb83008053e63d59abf8b8915b93e15fcc2

Compress PNGs with zopflipng

Zopfli is the most efficient DEFLATE compression algorithm, trading run-time
performance for file sizes that are typically 3-8% smaller than those produced
by zlib with the maximum compression setting. Its output is Deflate-compatible,
so no specialized decoder is needed.

This change was created by running zopflipng against all the PNG files in this
repository. The exact invocation was:

  git ls-files --exclude-per-directory=.gitignore -- '*.png' \|
    parallel zopflipng -m -y {} {} \;

Files which zopflipng was not able to compress more efficiently were left unmodified.

Bug: T127608
Change-Id: Ia9c70cf35da489985af39df1f8b58469eb4813fd

Tidy: <source> and <track> are empty elements

Seems these got accidently added as inline items, even though they
should be and are output as empty elements. This should correct that.

Bug: T122787
Change-Id: I6e75529c9d349050479c1b7ad758320d1e948e78

Merge "Add --wait option for runJobs.php"

Merge "Document memory-limit as a possible exit reason"

Add --wait option for runJobs.php

It is now very easy to implement a simple low delay job runner:

while true; do php runJobs.php --wait; sleep 1; done;

Change-Id: I50cf00aed4b15d90384a2cc12bddd64d96d5a1ad

Merge "CatMembChange use timestamp from revision where possible"

Merge "resourceloader: Add content hash to static debug urls"

resourceloader: Add content hash to static debug urls

This way, they automatically bypass any stale client or server cache.

Remove an old wfExpandUrl() call that would interfere with
transformResourcePath()'s ability to recognise the path.

Expanding the url there wasn't needed anyway. Whether it's on a separate host
(eg. bits.wikimedia.org), or path-absolute, either way works as-is.

Bug: T90983
Change-Id: I64eb3291adcfc3733ef96690399c09c06e19b3aa

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

mediawiki.api.options: Use formatversion=2 for API requests

The format of the JSON response does not change.
But the request is different so the test also have to be updated.

Change-Id: I391a221f81fd92731c331363a8202c21d6b933b5

Localisation updates from https://translatewiki.net.

Change-Id: I80bb2ffabf103ac061320924ff1bee7f6c13769c

Document memory-limit as a possible exit reason

Change-Id: I96748cd8397969447bd2315f0e6b6e7989ddfc4f

Remove some unused variables

Change-Id: Ia3a290555f06c564c063591710380367a04cb1cc

Remove wfCheckEntropy() as unused and deprecated in 1.27

Change-Id: Ib2eda8c5ab7b124a283b9e0ec934c034d3da864c

Use hex2bin() instead of pack()

This function was added in PHP 5.4.0 and can be used now that MediaWiki
only works with PHP 5.5.9 or higher.

Also fixed a bug in ApiQueryCategoryMembers::validateHexSortkey() that
allowed a single line feed at the end of the string to pass.

Change-Id: I5b577e7dcc5fb6a06ab550429aae657dbcc79083