Merge "Revert "memcached: better error messaging""

Revert "memcached: better error messaging"

This always issues PHP notices when the code is hit.

This reverts commit fe6da52a11375c83d86f73bf4725d14acfbf4cbf

Merge "Add correct rtl value to some languages"

Add correct rtl value to some languages

The language ku-arab and kk-cn weren't defined as RTL in their
message files. This commit fixes that.

Change-Id: Id55c17e6cdb2238993dc52b17b1297a352ba14d3

Do checks in the correct order in Linker::getLinkColour().

There's no point doing potentially expensive checks (exists() and
getLength()) if the page is simply not in a content namespace.

Change-Id: I86a2b3a36071cf508b8e99ef20bdf3e0e20f3c82

Merge "Removed now obsolete compat-code"

Merge "Add @since annotation to Language::formatExpiry()"

Merge "Fix regression in API log events (bug 35635)"

(bug 27610) Add archivename to list=filearchive

Change-Id: I485ff58ad57030cbfb3a0ee7803bf0e25c363d95

Merge "[FileBackend] Bumped Swift listing page size."

Add @since annotation to Language::formatExpiry()

Change-Id: I022762a3a5e6fa6e45824f963c9301a83930b850

Merge "show old protection in prop=info, if no new protection exists"

Merge "make list of 'Languages that support variant conversion' dynamic"

Merge "(bug 35893) Special:Block should load mediawiki.special.block.js"

(bug 35893) Special:Block should load mediawiki.special.block.js

Special:Block is showing checkboxes that apply only to IP blocks when
blocking usernames, and vice versa. The code to hide these checkboxes
already exists in resources/mediawiki.special/mediawiki.special.block.js,
but it's not being loaded.

Change-Id: I875125b56e9f234dbff87bff3b045877fb1614be

Merge "mw.toolbar: Clean up the API of the classic toolbar."

Merge "Remove weird, confusing, unreachable code"

Remove weird, confusing, unreachable code

The removed code was doing first-order dependency resolution if a single
module was passed in as a string.

This is crazy because:
* passing in 'foo' triggered it but passing in ['foo'] didn't
* dependency resolution is resolve()'s responsibility
* all callers were passing in arrays produced by resolve(), so this code
  was never reached

Change-Id: I081e2bf90cead63437e495006c78254cf65e3847

Whitespace fix

Change-Id: I439b39ffad2989afe1c01a1d430082730ae38151

don't use message when the 'nodb' flag is set

Fix for If5f1c26f (69591db):
This breaks maintenance/language/transstat.php when there's no db connection.

Change-Id: I3f4416234ecce34c93ba14a66771379cb427e83b

Pass the local User object to Revision::userCan()

Change-Id: Iffd857a577593642464371a988b7b7829d246748

Reorder methods in the MediaWiki class for better consistency.

Change-Id: I651606b3d675212891c5075b3114c451810a3ec5

Improve documentation of maintenance scripts.

Change-Id: If0af9922ede902c2c6f18b627d5810f3e888c944

Eliminated deprecated functions in TextPassDumper

Change-Id: I9f3148fe6044cc718e57b4aaaaf507f3342dc4d3

Removed unused $wgAntiLockFlags flags.

Change-Id: I56a24db564906256f78b5d1903de4ddd70a13bc4

Merge "(bug 27567) Add file repo support to prop=duplicatefiles"

Merge "(bug 38333) Check global blocks on account create"

(bug 38333) Check global blocks on account create

Before displaying an account creation form, checks against
GlobalBlocking are run, but not when the actual account is
created.

At this time, User::isBlockedFromCreateAccount() does not
run the hooks for which GlobalBlocking is registered.

Change-Id: Ieb3696bc49d6f4c4e3e95d8c55121b7ffa07b506

Remove <i> tags from MessagesKsh.php, they aren't parsed

Change-Id: I24b5ceb61170c0150e805b243642b9335d770aab

Merge "More accurate function description for isAllowed()"

More accurate function description for isAllowed()

Change-Id: Idf95efc20714bfb2595259bcce6ba9eaf9a63fae

(bug 27567) Add file repo support to prop=duplicatefiles

This adds a new method findBySha1s to the FileRepo classes to support
the multiple hash search against all repos (with only one query for each
db repo).

Change-Id: I745cae7a1db3a32c20aa0067b744402fcf1a3122

[FileBackend] Bumped Swift listing page size.

Change-Id: I44cab543246c0ff65b2bd64020d642a18c48cb67

Localisation updates from translatewiki.net.

Change-Id: I7db66b78968b5627b3936631a4d18b12ab77a65a

Add testRomanNumerals

Up Language::romanNumeral() to work upto 10,000

Does anyone know how do do letters with an underscore ontop of them? ;)

Change-Id: Ib1b1415126af855ce5fb55f81b71534c26d84cc9

Merge "Fixed DATA_FOR_UPDATE loading in WikiPage."

Merge "Reverted "FOR UPDATE" in 61f246aa786d824d0653522ed679c16be719da80."

Merge "Fix missing occurence change in I6b69919a (747e89e)."

Fix missing occurence change in I6b69919a (747e89e).

The line above changed in that revision, but not this one,
so I highly suspect that this occurence of deleterevision
should have been changed to deletelogentry.

Change-Id: Ief4ca9acf08bccc8ff064cea5795a2dbeae44d19

Removed now obsolete compat-code

Change-Id: I64dc4beb4f8a084bfc5b7c60cbdaa6b65838b79b

Merge "Use User::getDefaultOption() instead of $wgDefaultUserOptions"

mw.toolbar: Clean up the API of the classic toolbar.

* Several methods were added here recently during 1.20 development
  that should not have been public methods.

* Also in the creation of this new module (replacing the old
  mwCustomEditButtons) a design flaw was made. Instead of using
  a key-value pair object, the signature was changed to a tentacle
  function with 7 (for callers, unnamed) arguments.

* Changed it back with the compatibility fix the other way around.
  So everything is backwards compatible.

* Moved to local scope:
 - buttons queue
 - $toolbar
 - insertButton
 These were recently introduced during 1.20 development but not
 meant to be public. When used too early or too late from outside
 the module it will break or be ignored. For example $toolbar is
 false before dom ready, buttons queue is ignored after domready,
 insertButton will break if called before dom ready because the
 target element doesn't exist yet. These are not bugs, but result
 of calling internal methods before they are initialized.
 The public API takes care of these state differences by using
 the queue and the dom ready handler.

 Scripts should (and do) only use the addButton API.

* Kept:
 - addButton
 - insertTags
 - init (empty b/c function, was already there)

* Improved:
 - addButton: Now takes an object as well, just like
   mwCustomEditButtons used to do.
 - Cache Array.prototype.slice instead of re-grabbing from
   a new dummy array.
 - Store buttons[i] in a local variable in both cases, not just
   for legacy. Saves 2 property lookups. Minor gain, but
   in this case it was already going to be stored in a local
   variable, so might as well do it in the other case.

* Fixes:
 - Clear queue array after it has been used. Though in practice
   it should never happen that it is iterated over twice, just in
   case.
 - Added comment to init() function explaining where it is used.
 - Updated closure arguments per code conventions.
 - Made it a position-top module so that it actually can be used
   before the document is ready.

* Example usages tested:
<code>
// Legacy way from wikibits.js:
// Has to be done before document ready
window.mwCustomEditButtons[window.mwCustomEditButtons.length] = {
  imageFile: 'http://placehold.it/23x22',
  speedTip: 'tool tip',
  tagOpen: 'x-',
  tagClose: '-y'
};

// mw.toolbar: List of arguments
mw.toolbar.addButton( 'http://placehold.it/23x22', 'tooltip', 'x-', '-y' );

// mw.toolbar: Object
mw.toolbar.addButton({
  imageFile: 'http://placehold.it/23x22',
  speedTip: 'tool tip',
  tagOpen: 'x-',
  tagClose: '-y'
});
</code>

Change-Id: Id19819707c937c2c3144ad8177b75baa46f5073c

Merge "(bug 37926) Split 'deleterevision' right for log entries"

(bug 37926) Split 'deleterevision' right for log entries

This change adds a new permission ('deletelogentry') which is required to be able
to delete log entries.

It does not affect who can see deleted content.

Change-Id: I6b69919a1bdc502becc1ae4ac9169b8b0e85bfdc

Merge "[LockManager] Made LSLockManager session 32 chars (128 bits)."

Reverted "FOR UPDATE" in 61f246aa786d824d0653522ed679c16be719da80.

* Previously, this code would select FOR UPDATE unless ALF_NO_LINK_LOCK
was set in $wgAntiLockFlags (which in fact was set for wmf sites for ages
to reduce contention). In this case, it was just loading from the master,
without locking, and relying on the page_latest check. Now, it just never
selects FOR UPDATE, regardless of the lock flags, relying on page_latest.

Change-Id: Ie6301de4415f56e6be3899cf4ab37924963ce8c4

Fixed DATA_FOR_UPDATE loading in WikiPage.

Change-Id: Ifc4259942eb8bf5e91e80ec0009c2a2ab191ae11

Use User::getDefaultOption() instead of $wgDefaultUserOptions

Using User::getDefaultOption() in User::setOption() sets preferences like
'language' to the right default value, whereas using $wgDefaultOptions
sets the language to null because language is not part of $wgDefaultUserOptions,
but it is part of return value of User::getDefaultOption().

This only makes problems when reusing the same user object, because when
a new user object is created, the value of 'language' is correct.

Change-Id: I43a32c66fc3997a4f842c63af374e84d234602b2

Merge "Updated result properties in paraminfo API"

Merge "Revert "Do not show empty categories with list=allcategories""

Execute jobs on all index.php requests.

The problem is that currently deferred updates are not executed
after action=ajax requests or file cache hits. This means that,
e.g. ViewCountUpdates are not executed after file cache hits.
The code to execute jobs (both deferred updates and job queue)
is now in MediaWiki::restInPeace() so that it's also executed
after what is mentioned above, and the remaining code of
MediaWiki::finalCleanup() is moved directly MediaWiki::main()
since that function is only used in one place and it make no
sense to keep a method for such little code.

Change-Id: I6f511a74c59a2e3653c10bfcff9b0d15118e699f

(bug 38190) Add "required" flag to some token params

action=block/action=unblock can not have the token required because when
using the gettoken param, the token param can not set.

Change-Id: I15317f16c06f150d29c1b17de76f41a6cfa84820

Merge "(bug 260) Handle <pre> overflow automatically with a scroll bar"

Use WebRequest instead of $_SERVER in EditPage.

Change-Id: I30cdcffe8c520f9a4031c31e3e08db9b699da597

Merge "Undo feature: Add a wpUndidRevision field that can be used to distinguish (undo) edits from other edits"

[FileBackend] Cleanups for doxygen.

Change-Id: I82cd7e4ef01c6fa10fbdb6ccce5cc8e42e8aac67

cleanup temp files left around by UploadStash

stashFile renames files to add an extension, this way it is no longer
cleaned up by php. So we have to unlink it after storing it in the FileRepo.

Change-Id: Iab697fc4535546dd25231dddf220e18f3b0b73be

Fix bug in moveTo() with $auth=false and $createRedirect=false

Even when $auth=false, moveToInternal() would unconditionally check
$wgUser's 'suppressredirect' permissions and override $createRedirect.
This means it was impossible to suppress redirect creation when moving
pages in a maintenance script, even when telling moveTo() to disable
permissions checks.

Fixed by moving the check from moveToInternal() up into moveTo() and
respecting $auth there

Change-Id: I9b52dc67c7ae2dbda3ca62f78d4d7df118771c0f

(bug 35060) more allowed params to Special:MyPage, Special:MyTalk

I investigated the available parameters and considered which could be
plausibly useful for Special:MyPage and Special:MyTalk, and there are
quite a few. See inline documentation for a list of them. More could
potentially be added in the future if a use case is discovered.

This patch also make it possible for extensions to add their own
parameters to this list, through the new hook:
RedirectSpecialArticleRedirectParams

It has at least one application - with FlaggedRevs, it's desirable to
pass the "stable" flag through Special:MyPage. It also makes it simple
for individual servers that have some special reason to pass on a
particular parameter to do so in LocalSettings.php, simplifying
upgrades.

Change-Id: I3101fbe2eba8712b87a53706583f4e8f3e907478

make list of 'Languages that support variant conversion' dynamic

see bug 33223. Placing this list in LanguageConverter,
so people see the list, when adding new Converters

Change-Id: I088bc1f0ea01d7d6e02c736261311919a0956a57

(bug 38333) test blocked user cant createaccount

This patch adds BlockTest::testBlockedUserCanNotCreateAccount() which
test that a new user can createaccount and that blocking him will
prevents to the createaccount right.

Does not solve bug 38333 though!

Change-Id: Ia10e26825d50d47b4770900ee91dea666337f969

(bug 30836) API siteinfo specialpagealiases should not return nonexistent special pages

Some special pages are conditionally registered (e.g.
Special:Popularpages only exists when $wgDisableCounters is false), or
sometimes an alias exists for a special page that does not. The API
should probably not return entries for these from
meta=siteinfo&siprop=specialpagealiases.

This also fixes bug 38464 correctly, rather than blindly reverting.

Change-Id: Ic021a47754ea7a1574e818dad47151ab6698e06f

[LockManager] Made LSLockManager session 32 chars (128 bits).

Change-Id: I1eba3eb44ebef02c771b518c2bcbdc0d41eb8086

[FileBackend] Make sure multiwrite sub-backends use the proxy backend wiki ID.

Change-Id: I886b0aeb891a108e3a61e9d98e70b5a0bdc2bdc5

Undo feature: Add a wpUndidRevision field that can be used to distinguish (undo) edits from other edits

Change-Id: I85d485d89ab87c9478552183ffdb10b0bfaad648

Merge "escape HTML elements in docblock with double quotes"

Merge "Various doc cleanups for doxygen."

Merge "Free up some memory by unsetting large arrays in backend copy script."

Various doc cleanups for doxygen.

Change-Id: I079f66997111d27d89910505dd5978aac613d383

Merge "Minor documentation tweaks"

Merge "Api method documentation tweaks"

Revert "(bug 30836) API siteinfo specialpagealiases should not return nonexistent special pages"

Caused Bug 38464 - API showing internal error with ?action=query&meta=siteinfo&siprop=specialpagealiases&format=xmlfm

This reverts commit 48523016ef0d4a3d15e7417dfb3048d34cac69f7

Change-Id: I470e87d21d98d4436bd5e27d0a3796559f9c41bb

MediaWikiLangTestCase now call parent setUp/tearDown

Fix the temporary file garbage collection which was not occuring on lang
tests. Fix at least:

(bug 37252) filebackend-unittest temp dirs not deleted

Change-Id: I446d5ad21c8a3b12020e2424d09ba71dde410221

Increase the readability of provideFormattableTimes by adding newlines

Change-Id: I7f96f44c2e18153ecfd7ea6d593229808a835a63

Revert "Do not show empty categories with list=allcategories"

This reverts commit 4b9bd6a4dba57026b2f18d4024b021407e52ff4f.

This change made allcategories less useful,
because it means it's impossible to list empty categories
(previously, acmax=0 did that).

To exclude empty categories, acmin=1 can be used.

Change-Id: I7943507c4cb7d752f82caa69ce3edb0b31ca9463

Use Linker::userToolLinksRedContribs() in LogFormatter.

It is much more self-describing to use that method instead of
Linker::userToolLinks() with true as third parameter.

Change-Id: I30fbef2e06c7a835a66130cc54e10e114e286914

Free up some memory by unsetting large arrays in backend copy script.

Change-Id: I55b6a6f8228370d948821dcd4de22fda51539fd0

Merge "[FileBackend] Removed @since formatting for options to fix doxygen."

[FileBackend] Removed @since formatting for options to fix doxygen.

Change-Id: Ic1cb55dc9317de34fc50b7c590a2cd6e80c800ad

Merge "updateCollation.php size histogram feature"

[FileBackend] Added "bypassReadOnly" flag.

* Added flag to bypass read-only checks to the 6 write functions.
* Made backend copy and sync scripts pass in this new flag.
* Also made sync script use doQuickOperations() since its faster.

Change-Id: Iee47f79ed7ab002cfc2d0adb5321c3a8520f971d

updateCollation.php size histogram feature

Added a feature allowing updateCollation.php to show a histogram of
sort key sizes, to assess the effect of index size truncation. Added
--dry-run and --target-collation options to allow the index truncation
to be assessed without actually changing the collation.

Change-Id: I497b5d0740384f5d6fdebc6d5ccfea5d853fbd37

[FileBackend] MultiWrite code improvements and sanity checks.

* Automatically override a few sub-backend settings in FileBackendMultiWrite.
  We should ignore locking, journaling, or read-only settings of sub-backends.
  Journaling is now done just for master sub-backend paths (under the proxy backend name).
  We don't need to log for each sub-backend, which is a waste of disk space.
* Changed FileBackendMultiWrite::doOperationsInternal() to be similiar to the
  other write functions. It now uses doOperations() of the sub-backends rather
  than manual code. This makes settings like 'concurrency' easier to manage;
  we might want to have some sub-backends with varying setings for that.
* Made FileBackendMultiWrite::doQuickOperationsInternal() compliant with docs.
* Added CHECK_SHA1 option for consistency checking.
* Fixed function visibility in two places.
* Improved multiwrite backend tests by calling consistencyCheck().

Change-Id: Iac7bfe10c77ecd069fb9ef0ec26a01512f5f4eea

Merge "(bug 37249) validate export-demo.xml against current export.xsd"

Merge "Allow extensions to abort (and replace) email notifications"

Merge "API: Cleanup around comment/reason params"

Merge "prop=imageinfo&iiprop=url|thumbmime needs iiurlwidth="

Merge "merge msg script now detects extensions main files"

Merge "Avoid mangling fields in API continuation parameters"

Merge "Notice: Undefined index: page_is_redirect in \includes\api\ApiPageSet.php on line 111"

Avoid mangling fields in API continuation parameters

API continuation parameters encode sufficient state for a subsequent
query to continue where the previous one left off; this may sometimes
include page titles, with or without the namespace. Given that these
page titles are already in the exact format required for the next
request's SQL query, it is not necessary to "normalize" them in any way.
And if normalization does more than just change spaces to underscores or
vice versa (e.g. it canonicalizes namespace aliases or capitalizes the
first letter of the title), it can be actively harmful: see bug 36987
and bug 29290.

Note this patch involves a breaking API change: the values for the
"continue" parameter of various modules have been changed, and some
modules will now return "continue" as the continuation parameter instead
of reusing "from".

Note this patch also corrects a minor logic bug in ApiQueryAllLinks,
changing ">" to ">=". The line is being changed anyway, so I didn't
bother doing a separate changeset.

Change-Id: I459232e919d20f89f6de9d20640fd48c8fd5781c

Add wfProfileIn/wfProfileOut calls to InfoAction::pageCountInfo()

Change-Id: Ice4eff38253b189a60885be36dadd2a3cf5d6297

Only query page_counter if $wgDisableCounters is false

Change-Id: Ie134090cdd6573d11a7f5510e1d90dab9b06f117

Allow extensions to abort (and replace) email notifications

Change-Id: I769ad24fcd6f3e3c03fb67122eab25a8472047b6

Merge "add optional limits argument to wgShellExec"

Merge "[LockManager] Memc lockmanager improvements."

[LockManager] Memc lockmanager improvements.

* Throttled key lock polling and changed the key locking to happen in sorted key order.
  This should reduce deadlocks and wasted RTTs for frequently used items.
* Also tweaked the wfRandomString() argument to be 32 (128 bit).

Change-Id: I12bbc988251ad3c895aa40a08b0215b2bcdb5e4f

Merge "[Database] mark some methods as explicitly public"

Remove some optional messages from ba and shi

Removed pagetitle-view-mainpage and pagetitle from languages ba and shi.
The current pagetitle caused strange display of the title in
Bashkir projects.

Change-Id: Icba106e30916a2c15213fdc771c871a2f5df632e

[Database] mark some methods as explicitly public

Change-Id: I95fca6d346ae812c22b88b577477be02b5a0676b