Merge "Allow titles with falsy title text in suggestions"

Merge "Use WIS:addWatchBatchForUser in SpecialEditWatchlist"

Merge "Switch Signature of WatchedItemStore::addWatchBatch"

Merge "Avoid master queries on deletion form view"

media: Misc code clean up in DjVu.php

* Use $cache->makeKey() instead of global.
* Access constants via $cache.
* Remove obsolete 'that = this' PHP 5.3 workaround.

Change-Id: I75505d3e3017fde185e43815d4e7acaba3fee69c

Allow titles with falsy title text in suggestions

A valid title, such as 0, would end up being ignored in this clause and
not providing a title. Make the empty string check stricter to allow
these titles through.

Change-Id: I938a7dfe294e979c19f720eadb6f96282b1c4588

Merge "Set doesWrites() for Special:Undelete"

Set doesWrites() for Special:Undelete

Change-Id: Ie9044c9a57cb354a412ab515208b71ce44e52185

Localisation updates from https://translatewiki.net.

Change-Id: I31cbae4f585f73a2a0ab97909206185a0d1c0aef

Avoid master queries on deletion form view

Bug: T92357
Change-Id: I9c785e8e541f0e4fb7fa0f11cab2d33c26f7fb3a

Merge "Define 'zh-hans' and 'zh-hant' localisations for Moment.js"

Merge "Set $wgIncludeLegacyJavaScript false by default"

Merge "Throw an exception when the $key argument to hash_hmac is not a string"

Merge "parserTest: Remove mention of non-existent 'noxml' option"

Define 'zh-hans' and 'zh-hant' localisations for Moment.js

Bug: T123080
Change-Id: I38ac5f1669364f7512be8c63ee51ce1449f6db26

Throw an exception when the $key argument to hash_hmac is not a string

HHVM throws a fatal error when $key is not a string (unlike $data which
is typecast), so we might as well as throw an exception so that
at least we have a stack trace.

Bug: T126316
Change-Id: Iad9a499b51647c7dbcd58e9ab7ac8e8cb6359bba

mw.widgets.TitleWidget.js: Share interwikiPrefixes between multiple instances

The interwikiPrefixes are static and do not need to load separately for every instance.

Bug: T124899
Change-Id: I316bdcc67ba176d22a70cd30fc3eb72bfe0be509

Set $wgIncludeLegacyJavaScript false by default

This setting was deprecated in 0ac4f998 but was left as true by default, which
means sysadmins weren't prompted to find and fix faulty code. Setting this to
false by default gives them a release in which to fix the code (setting this
back to true until such fixes are made) before we remove it entirely in the
next MediaWiki release. Extensions, skins, gadgets and scripts that need the
mediawiki.legacy.wikibits module should express a dependency on it in their
ResourceLoader manifest.

Bug: T35836
Change-Id: If787d0975b6019827e81f4ea0d90898e361d9acd

Use WIS:addWatchBatchForUser in SpecialEditWatchlist

Bug: T129481
Change-Id: I550e295786afaf9970fe96788f31d514827cd0bc

Switch Signature of WatchedItemStore::addWatchBatch

Adding batches of watched items per users
makes much more sense.
Only the deprecated static WatchedItem method
needed the old silly way of passing in objects.

Change-Id: I90f9583b66bd3b5afcf07faefedb38a8a0149f6e

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 5538d868777d2d5abb125dac310550fde3873689

Change-Id: I6750107c92c6dd96fe28ca96e66db43e0235c470

Merge "Dont count RC_CATEGORIZE in Special:ActiveUsers"

Merge "Add redirect=no only to redirects on redirect pages"

Merge "Use WIS::getWatchedItemsForUser in SpecialEditWatchlist"

Merge "Introduce basic SpecialEditWatchlistTest"

Merge "Introduce basic SpecialWatchlistTest"

Merge "Cast row namespace to int in SpecialWatchlist"

Introduce basic SpecialEditWatchlistTest

Change-Id: I9f765c4183314850169f685abd242e979a79bd68

Introduce basic SpecialWatchlistTest

Change-Id: I98a2ad112189a09581a0564da3c3357939a5d7de

Use WIS::getWatchedItemsForUser in SpecialEditWatchlist

This also adds a order option to:
WacthedItemStore::getWatchedItemsForUser

Tests are also updated

Change-Id: Ia683b92846ad79bde3f37068a8e168c5a8bdc201

Dont count RC_CATEGORIZE in Special:ActiveUsers

Bug: T130550
Change-Id: Ia2296586d35bb21085fcf386f26ad6f4e765e534

Cast row namespace to int in SpecialWatchlist

Something similar to this recently got through
and caused exceptions on mw.org

This looks like the same thing, a ns from a db row
being handed straight to the TitleValue constructor
without being cast.

Change-Id: Ic126cd713c5210c9ae3d8e805081849ccf84a7f6

Merge "ZipDirectoryReader: Don't try to parse zero-length files"

Merge "MimeMagic: Prevent PHP warning when trying to read zero bytes"

ZipDirectoryReader: Don't try to parse zero-length files

With the current code, 'zip-bad' would be returned (eventually). I think
'zip-wrong-format' is a more correct response. This prevents weird errors
when trying to upload such a file (and results in saner errors instead).

Change-Id: Ic2c010f318d98df4783da1cf5126e8dd88aa9014

MimeMagic: Prevent PHP warning when trying to read zero bytes

Honestly, there's no reason why fread() should warn when asked to read
zero bytes. It should just return zero bytes of content. Eh, PHP.

Bug: T130813
Change-Id: I56c4826759390bdbf6af04da28e2d9ed07781bca

Always use 'bool' instead of 'boolean' in Doxygen tags

Just like commit f86a5590aae7fbe6d9b8a3d129c7a04a11a27579

Change-Id: Ic9d08bca6524d6bb4baf5170c081ad0f3d738e28

Merge "Improve sign-up portion of the login form"

Merge "Title: Fix inaccurate documentation of getUserPermissionsErrorsInternal"

Title: Fix inaccurate documentation of getUserPermissionsErrorsInternal

Blocks are checked (checkUserBlock).

Change-Id: I3a164d1003f2bea3e3e07081c9bebfabd884ade9

Merge "OutputPage: Minor clean up of <head> and HTML"

OutputPage: Minor clean up of <head> and HTML

* Remove trailing space in self-closing tag.
  Brings parserTest output of Parser and Parsoid closer together.

* Remove various line breaks at begin and end of script contents.

* Remove FILTER_NOMIN from makeConfigSetScript() output.
  This isn't part of any user- or page-dependent module and not minified.
  And Xml::encodeJsCall already ensures compact output for prod mode.

Bug: T127328
Change-Id: I85a5a59fd0955c1a112e8b24b933f0d9e983a156

Merge "jquery.suggestions: Fix typo "singleton" in documentation"

jquery.suggestions: Fix typo "singleton" in documentation

Change-Id: I81af9a1974ea9c9dcd26f201bf055554087ccd99

Merge "Request-local caching of DjVu dimensions"

parserTest: Remove mention of non-existent 'noxml' option

The 'noxml' documentation in parserTests.txt was added in 2006 (7eea2398; r12504),
however it wasn't actually implemented.

It wasn't until 2009 (7aa4a8f9; r54767) that $wgWellFormedXml was created,
which defaults to true and has no relation to this option.

Remove the 'noxml' options from existing tests.

Change-Id: Ie3ae9f491b5747716080607b81b9763bf2bcc889

Merge "Request-local caching of revision text"

Request-local caching of DjVu dimensions

Make DjVuHandler::getDimensionInfo() cache dimension info in memory so that
operations which get the dimensions of each page don't result in multiple
round-trips to memcached.

Change-Id: Idbd11637a8f7d87e4774f76a5b90d7469f3b0857

mediawiki.requestIdleCallback: Implement timeRemaining()

This matches the native API. This allows callers to better batch and spread out
expensive operations based on actual execution speed.

Right now CentralNotice is manually creating arbitrarily sized batches in
kvStoreMaintenance. Instead this can use a while loop with timeRemaining()
to run as quickly as possible whilst still being able to stop and yield when
it runs for too long. This way will naturally take more iterations on slow
devices and less iterations on faster ones - to be least disruptive.

While timeRemaining() is already available in the native interface,
it was previously unsafe to call because the fallback didn't implement it.

* Remove redundant QUnit.test() expect numbers.

* Add a test for the native one if available. This will catch silly mistakes
  like assigning the native one to mw.requestIdleCallback directly that result
  in 'Uncaught TypeError: Illegal invocation' due to missing call context.

Change-Id: I9721fab9e89c561e31101b5556a3748431353548

Merge "Cast namespaceId to int in SpecialEditWatchlist::cleanupWatchlist"

Cast namespaceId to int in SpecialEditWatchlist::cleanupWatchlist

Bug: T130778
Change-Id: I4b62e5b251f8c3d31aa2f76a9988b5cb7c8cf3b7

Merge "SpecialUploadStash: Add links to view a thumb of each uploaded file"

Localisation updates from https://translatewiki.net.

Change-Id: Idf395e9379605a64103df4ee0ce3847b600f5ede

Cleanups to WikiRevision

* Check for blank $source variable more properly.
* Avoid use of ugly File::DELETE_SOURCE flag.
* Remove now-unused $flags parameter to uploadOld().

Change-Id: I6bc610cfead0036700b4f9a82e13f2bd9ab10f32

Merge "Add WatchedItemStore::getWatchedItemsForUser"

Merge "Ensure that User::getId() returns int"

Add WatchedItemStore::getWatchedItemsForUser

This method is also no used in SpecialEditWatchlist

Unit and Integration tests have also been added.

Bug: T129481
Change-Id: Ica136b6fee63be8de4c2fe502288dd2318d99114

Merge "SessionManager: Use existing backend for the ID if one is loaded"

Request-local caching of revision text

It's not uncommon for the revision text of a Scribunto module to be retrieved
half a dozen times or more in the course of a single request. Caching them in
APC is a non-starter, because there is no practical way to determine which keys
are hot, and storing everything in APC is not viable. Request-local caching
gives us much of the benefit with no negative consequences apart from a nominal
increase to memory usage.

Use a MapCacheLRU to hold cache items so we can limit the size of the cache to
10 items, to prevent uncontrolled memory growth for long-running scripts.

Change-Id: I77575d6d0ea2d06fc6c93f664c5407641aab88d9

Merge "Add new index to make updateCollation.php painless"

Merge "Fix typos in docs/memcached.txt"

Fix typos in docs/memcached.txt

expriy, expriry => expiry

Change-Id: I96304392e79499d78551e9d66fb0161d759bf1e0

Update OOjs UI to v0.16.4

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

Change-Id: Ic0f8cf4fc956feff29189c4d3e1d83e071daa8f7

Add new index to make updateCollation.php painless

We want to update categories in order, to minimize disruption
to users. Previous indexes required a filesort to do this, which
exploded things on large wikis. See bug for details

Bug: T58041
Change-Id: Iee6cd997ff87a313a46fda19d8ab063d0fed8ce8

SessionManager: Use existing backend for the ID if one is loaded

This fixes a bug where SessionBackend::resetId() of the PHP session will
fail to properly load $_SESSION because the new session ID hasn't been
saved to the store yet. It's also a reasonable performance improvement,
no need to call loadSessionInfoFromStore() when we already have the
session loaded.

Change-Id: I30f159ef1267442a6325aabbbdfaf69defc10ed6

Localisation updates from https://translatewiki.net.

Change-Id: Iba1610b61c1658b22216e57ed5d60d131073feab

Merge "Revert "Add link to anon's user page; remove "Not logged in"""

Merge "Message: Use RequestContext instead of $wgLang"

Revert "Add link to anon's user page; remove "Not logged in""

This reverts change I049d0671a7050.

This change was reverted in the wmf/1.27.0-wmf.17. Since there is no
clear consensus, revert in master before branching wmf/1.27.0-wmf.18.

Bug: T121793
Change-Id: I2dc0f2562c908d4e419d34e80a64065843778f3d

Names.php: Use ʻ instead of ` as ʻokina in Hawaiian

https://en.wikipedia.org/wiki/%CA%BBOkina:
In the Unicode standard, the ʻokina is encoded as U+02BB ʻ MODIFIER LETTER TURNED COMMA

Change-Id: I030f3b46382d5c5ed5ba81471f7c4156c660b244

Localisation updates from https://translatewiki.net.

Change-Id: I9e4277291fc529f86496f46c8eab1c2cd4590903

Merge "mw.widgets.CategoryCapsuleItemWidget: Pass the right 'this'"

mw.widgets.CategoryCapsuleItemWidget: Pass the right 'this'

Follow-up to 7b57752e65c3cac62d973f43d1060b6bd0ab2a2f.

Bug: T130544
Change-Id: I89c18be911bcc492f3a70e1c1da3d2fd2bf65a05

Merge "Use WatchedItemStore in ApiQueryInfo::getWatchedInfo"

Use WatchedItemStore in ApiQueryInfo::getWatchedInfo

Adds a method for getting watchlist's notification timestamps
for a batch of LinkTargets.

Bug: T129482
Change-Id: I1f84212e7879a84b34bb3b53859069fcea282bba

Localisation updates from https://translatewiki.net.

Change-Id: I3d9b14ae3a5d77fea9694ef113b0180e5677c39e

Merge "Update phpunit to 4.8.24"

Merge "Call Skin::setRelevantUser on Special:EmailUser"

Update phpunit to 4.8.24

See changelog at
https://github.com/sebastianbergmann/phpunit/blob/4.8/ChangeLog-4.8.md

Change-Id: I604f10917e015788acf49cdcd6389957da89fbdb

UploadBase: Set mFileSize, if given, even if mTempPath is unknown

When uploading a file from stash using the action=upload API, with
async=1, UploadFromStash is initialized without initializing a temp
file. dcb5ec5cbf92b9a07f0776b9c194183a13400193 accidentally made it so
that the given file size is ignored if there's no path. This caused
validity checks to fail (because mFileSize is null) and action=upload
to also fail with cryptic 'emptyfile' warning.

This made it impossible to upload files bigger than 10 MB using
UploadWizard, as it uses the async mode for them.

Bug: T130238
Change-Id: Ie35a66a565a370fe9adc66d5fee0866c4d51470e

Localisation updates from https://translatewiki.net.

Change-Id: I67c4a49a16f5cee599db767b1414cac75bc19fd5

build: Bump npm development dependencies to latest

 grunt-contrib-copy     0.8.2  →    1.0.0
 grunt-contrib-watch    0.6.1  →    1.0.0
 karma                0.13.19  →  0.13.22

Change-Id: Idc5c1ac97580e91d9c958c55cdfdda681330c254

Merge "Handle HTTPS when running jobs asynchronously"

ListFiles: Call Skin::setRelevantUser() when applicable

Also validate user name before calling `User::newFromName()`

Bug: T129825
Change-Id: I7481ab0fc1720e5e840f0d552934324f676c0241

Call Skin::setRelevantUser on Special:EmailUser

Adding the relevant user to the skin will add user releated links to the
navigation bar. User related links are: contributions, logs and blocks

Change-Id: I37126a228717addc9879fd69fcbe89057ca94cb7

Merge "Many more function case mismatches"

Many more function case mismatches

Change-Id: I5d3a5eb8adea1ecbf136415bb9fd7a162633ccca

Merge "MediaWikiTestCase::assertException() was removed"

MediaWikiTestCase::assertException() was removed

Deprecated since 1.22. No uses in core or Gerrit extensions remaining.

Change-Id: Icae4077078c73d766d3c1c9e9e569922151296d9

Update grunt-karma to 0.12.2

Reason because this update will include a fix for incompatibility with
grunt 1.0.0 when it is released.

Change-Id: I39f01e3a68b764a4638c3465a1d116aefd517ef5

Message: Use RequestContext instead of $wgLang

This cuts the $wgLang and StubUserLang which is a bit cleaner.
It also makes it more reliable when load.php sets its
RequestContext user interface language.

Bug: T127920
Change-Id: I05302feb9b3ce9e4c29541e07a4260effc4b55b2

Fix numerous class/function casing

Change-Id: I23982bfa0548c9ea3bdb432be7982f1563930715

Merge "API: Allow fetching login token from action=query&meta=tokens on private wikis"

Merge "resetUserEmail: Allow resetting email without scrambling password"

Merge "Actually make $wgNoReplyAddress default to $wgPasswordSender"

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

Merge "Try to fix some other broken-looking legacy maintenance script options"

Merge "Register importImages options without arguments as actual maintenance script options so they are usable"

API: Allow fetching login token from action=query&meta=tokens on private wikis

The problem is that ApiQuery requires the 'read' right even though
ApiQueryTokens doesn't.

So, we introduce an exception: if action=query gets only meta=tokens
(and optionally rawcontinue and indexpageids, since they don't affect
anything), no other modules and nothing in the ApiPageSet,
ApiQuery::isReadMode() will return false.

Bug: T130112
Change-Id: I83dafb0305ff0cb1fc3bac668b88b5d2022e5880

Localisation updates from https://translatewiki.net.

Change-Id: I935a1d0bbc9fa5028f2fa1c9b05272ec2e416a7a