Merge "Enforce MW_NO_SESSION, add MW_NO_SESSION_HANDLER"

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

Fix function call casing

Change-Id: I20600c865b9f239ac281a1f8c5776ad4dbec4359

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

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

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

Deprecate PrefixSearchBackend

Change-Id: I9a8d3113f0428294db847638365fb0233c93d9ac

Merge "mw.widgets.CalendarWidget: Lazy-initialize calendar buttons too"

Merge "mw.widgets.DateInputWidget, CalendarWidget: Lazy-initialize calendar when first shown"

Merge "mw.widgets.DateInputWidget: Don't construct moment() objects when not needed"

mw.widgets.CalendarWidget: Lazy-initialize calendar buttons too

Bug: T126788
Change-Id: I9b0e67ea5b9854745928e3c63e73d701c6a83daf

mw.widgets.DateInputWidget, CalendarWidget: Lazy-initialize calendar when first shown

Bug: T126788
Change-Id: I469cd7dd77108435459b0f1feba953febd82e91b

mw.widgets.DateInputWidget: Don't construct moment() objects when not needed

Fixed a typo too.

Bug: T126788
Change-Id: I3b2dac752a43945e5525a8ea9349a4b46a7549b2

Localisation updates from https://translatewiki.net.

Change-Id: I72d9974730d1ae1dfbd1b02968a9e1dfab71b525

Follow-up 6281b0a: LookupElement is still not in core

Change-Id: Ie5300ae042d31a64d2b5d40bcf289f4474d3f898

Align mixin whitespace to CSS/Less coding guidelines

Aligning mixin whitespace to CSS/Less coding guidelines. Also fix some minor
inconsistencies and add browser support comments.

Change-Id: I50841e6062e59513def49719dc7a04956002de0e

Merge "Resources: Narrow down dependencies on 'oojs-ui' to the smaller modules"

Resources: Narrow down dependencies on 'oojs-ui' to the smaller modules

I wasn't reading through them very carefully, so it's possible
that some of the 'oojs-ui-widgets' could be narrowed down further
to just 'oojs-ui-core', but this is good enough for me. At least
we're no longer loading toolbars everywhere.

Change-Id: I58799e22f9c0a2f78c1b4a02c4b7af576157883a

Parsertests: Fix the video parsertests to run using phpunit

Bug: T126723
Change-Id: Ib3777413e1f86ddca9e460b41a8e3ef0eb8fb7cc

Update comments about language codes

* RFC 3066 is obsoleted by RFC 4646.
* RFC 4646 is obsoleted by RFC 5646.
* All are also known as BCP 47.
* Language codes must treated as case insensitive.
* There exist conventions for the capitalization of some of the subtags.
* Write a space between BCP and 47 in comments.
* http://www.adobe.com/devnet/xmp/pdfs/XMPSpecificationPart1.pdf is not available.
* https://wwwimages2.adobe.com/content/dam/Adobe/en/devnet/xmp/pdfs/XMP%20SDK%20Release%20cc-2014-12/XMPSpecificationPart1.pdf
  is the new position.

Change-Id: I91e305fd5754cf4dedf8fd7460f7d3375f792c89

EditPage: Style anonymous editor warning as a warningbox for consistency

Most projects override this anyway, but for default installs, let's
be consistent with other warning messages so it's clearer what is
going on.

Bug: T127403
Change-Id: I22c51a46a249247abccda709d74356e8e31a19a8

Merge "rc_old/new_len null for CategoryMembership RC change"

MimeMagic: Recognise .htc as text/x-component

Matches behaviour of nginx:
https://github.com/wikimedia/operations-debs-nginx/blob/be600c8b/debian/conf/mime.types#L15
>  text/x-component                      htc;

Without this, wmfstatic fails to serve "csshover.min.htc".

Bug: T99096
Change-Id: I90c546ff5204ddd9ffa99fdaebee78c7856c32ed

Merge "Timing: Fix flaky TimingTest test"

Timing: Fix flaky TimingTest test

> 1) TimingTest::testMark
> Failed asserting that 1455828402.4503 is greater than 1455828402.4503.

Use usleep(100), similar to the other tests already there.

Change-Id: Ic45c99b7e928df397809d872ab8e41ada6ecf1e7

Merge "Minor code clean up in SpecialBooksources"

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

Merge "Stop doing $that = $this in includes/specials"

Localisation updates from https://translatewiki.net.

Change-Id: I3dece5b748fbe9b4c57cbf823c60f8464ccf8de3

Don't send RC_CATEGORIZE events to the IRC feed

Bug: T127360
Change-Id: I21e4c0edf25fc0b7f16ef3029de879d3b7a2923d

Merge "feedback: Update Phabricator URL broken by upgrade"

Merge "Changed 'expiry' to American English 'expiration' in en.json"

Merge "SessionManager: Autocreate should use READ_LATEST when necessary"

feedback: Update Phabricator URL broken by upgrade

Change-Id: I4e52476b386cc0c196cdb7ca8d1ed98a4d4889da

Changed 'expiry' to American English 'expiration' in en.json

Added British 'expiry' strings to en-gb.json.

https://translatewiki.net/wiki/Thread:Support/British_English_in_Mediawiki_messages

Change-Id: I3913449fc697770122ce01c1733f15ee3a7976de

Merge "registration: Avoid double slashes in localBasePath"

Merge "ApiQueryWatchlist: wlshow=unread should filter revisions, not pages"

registration: Avoid double slashes in localBasePath

Noticed it in APC usage for ResourceLoader that various keys from
FileContentsHasher contain double slashes. All from extensions that
use the `"localBasePath": ""` trick in extension.json (e.g. Citoid).

Change-Id: I5bac1e2e05e063aa7ff251ce7ffaa965a3451db9

Merge "Allow auto suggestion for subpages of Special:BotPasswords"

Title.php: Support DB language in getPageViewLanguage()

If a page language in database is set then use the current preferred variant
of this language.

Bug: T117667
Change-Id: Idfe1165346ae0068a4b73de441dc9dfa107b8e8b

Allow auto suggestion for subpages of Special:BotPasswords

The autocomplete search allows special pages to define the list of
subpages to be excepted. Fill up the function to show auto suggestion
for subpages of Special:BotPasswords

Change-Id: I8e4037f058ac2f65b12c4bcad81b4847c408bcce

Minor code clean up in SpecialBooksources

Change-Id: Ife378fe87c237123c299ded9a79286abeb8fa34b

Merge "Do not unauthenticate if autocreation fails due to a race"

SessionManager: Autocreate should use READ_LATEST when necessary

If we failed to fetch the user ID using the default flags but succeeded
with READ_LATEST, we should also be passing READ_LATEST to
$user->loadFromId().

Change-Id: If8fc332810c2703773a153760ad076c398a84e4e

rc_old/new_len null for CategoryMembership RC change

This will mean that EnhancedChangesList does not
do odd things.
Log changes also have this set to null

This also adds a tiny bit of cleanup for the past which
could be removed in a month or 2.
This will fix all current RC entries as well as new ones.
Once no old RC entires for CategoryMembership changes exist
we can kill the type check.

Bug: T126428
Change-Id: Ib19819373af70e10b0c750ffdb06156764b7fa3d

Merge "database: Add extra sanity check to selectFieldValues()"

database: Add extra sanity check to selectFieldValues()

Change-Id: Ia2788ea84bd099550655930188365984e5274678

Merge "Avoid master queries in Title::getLinksFrom()"

Make destination URL of a page save configurable

With this, if a gadget put a hidden input named "wpExtraQueryRedirect",
can guide MediaWiki to keep that extra query parameter even after the
save of the page.

Bug: T124986
Change-Id: Ie45d6d80e83298e4c349f1e2dedd4eaa66b7697a

CatMembChange use timestamp from revision where possible

This should mean that where possible the timestamp for
the category membership change will be equal to that of
the time of the revision that caused the change.

The IDs still may have space between them.

Bug: T126048
Change-Id: Ia270451ccb02f2fca979ccb1fcefa5bdb4a96722

Merge "CSSMin: Make isRemoteUrl and isLocalUrl really private, now that we can in PHP 5.5"

Do not unauthenticate if autocreation fails due to a race

Bug: T70012
Change-Id: I523ee94744ac943ede78af59ab381b65ae26e672

Avoid parsing HTML when creating <input> nodes

We used to create them like $( '<input type="checkbox" />' ), which
actually does HTML parsing under the hood, because on IE 8 and lower
trying to change the 'type' of an <input> node throws an exception.
But we dropped support for IE 8 recently and no longer need to do that.

Upstream change for OOjs UI is I166f5ab0dce5ab47dc6a1f4e2e5ad012635911ed.

Find:         \$\( '<input type="(\w+)" ?/?>' \)
Replace with: $( '<input>' ).attr( 'type', '\1' )

Change-Id: Ie86f8917e8ce100de22006516daa542ad178aab6

Merge "Changed linking of articles to Linker::link in ChangeList"

Changed linking of articles to Linker::link in ChangeList

When non-existent categories appear in the changelist
they wont get marked reddue to the use of
Linker::linkKnown.

Bug: T126854
Change-Id: I3dc9746c0fe55e81ffd2df1c04ade6950efcc020

Remove superflous `background-size` property & make use of mixing

Removing superflous `background-size` property, which is overwritten
few lines below and make use of correspondent mixin in `:checked`
state.

Change-Id: I4f24c84c2f93bd85f495ff74ae21fbc210c18f57

Stop doing $that = $this in includes/specials

Closures support $this as of PHP 5.4

Change-Id: I452761a6189f50f93158f6de063d31c029bc2ed3

Merge "Fix unmatched @codingStandardsIgnore in parser folder"

Merge "ExportTest: Simplify code that failed on 5.3 because of reserved word 'namespace'"

Merge "Use short array for generator output"

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

Localisation updates from https://translatewiki.net.

Change-Id: Ic6d74b8085cf4b940f82bcf03a49b207e319c965

ExportTest: Simplify code that failed on 5.3 because of reserved word 'namespace'

Reserved words, like 'namespace', are perfectly fine now to use as object
keys, but it seems they used to not be in PHP 5.3, necessitating workarounds.

Change-Id: Iecb79fda19803461dfb3800b8af4d31e971551b8

Merge "Remove comments about isset( $string['foo'] ) being true on PHP 5.3"

Use short array for generator output

The AutoloadGenerator and LocalSettingsGenerator should generate short
array syntax

Change-Id: I5772640556fd6274db0d522341a98c0acfe5794d

Fix unmatched @codingStandardsIgnore in parser folder

Fix outstanding phpcs errors

Change-Id: I7b857be88354f2ffa27d76406253ec9e9710b91d

Remove comments about isset( $string['foo'] ) being true on PHP 5.3

...but don't remove the code. Calling isset( $bar['foo'] ) without checking
that $bar is an array seems not very nice to me.

Change-Id: I822c925b6f36bf34902f8075e54f71fe4f6d2566

LinkFilterTests: Uncomment test cases that should pass on PHP 5.5

Change-Id: If62f964c3a8e0eaaab6e7193d426cc4ddaa47ade

Merge "Don't show patrol links if RCP/NPP are disabled, but file patrol isn't"

Merge "build: Update mediawiki-codesniffer to 0.6.0, add "composer fix""

ApiQueryWatchlist: wlshow=unread should filter revisions, not pages

It's currently showing all relevant revisions from pages with at least
one unread revision, where it would be more sensible for it to show just
the unread revisions. Something similar applies to wlshow=!unread.

Bug: T125729
Change-Id: I552901efb98081af9a1a677f4403a2a1a73bd94f

CSSMin: Make isRemoteUrl and isLocalUrl really private, now that we can in PHP 5.5

Well, protected, because we want to have unit tests for them, apparently.

Change-Id: I734b88599e5860aa59a07a89cc5389eb73b48813

Browser test: fix log in Selenium tests

LoginPage is removed from mediawiki_selenium Ruby gem. This repository
is the only repository that needs it, so the file is copied here.

Also, user used to be logged in via the API, not really testing if log
in via the web site works. It is fixed now.

Bug: T127042
Change-Id: Iff85d66a9a6d3343c212fe648c589539e889a313