Merge "Follow-up I47412b67: pass an array of query parameters to Linker::link()"

Merge "(bug 39099) Changed error-levels in Uri.php"

Follow-up I47412b67: pass an array of query parameters to Linker::link()

Spotted by Daniel Werner in that change.

Change-Id: I8ded52b8285b2af0f69cc586abe532ea56cc18bb

Merge "Pass the 'returntoquery' parameter on cookie check."

Merge "Remove unused variables/function values returned"

Merge ".gitignore: Add .DS_Store for Mac OS X; Organize and a-Z sort."

mediawiki.user: Reduce repetition of code

Created a local utility function getUserInfo(), which is called by
both mw.user.getGroups and mw.user.getRights.

This function uses the same API request to get both pieces of
information.

Also reduces the number of jQuery.Callbacks instances by
using it directly instead of jQuery.Deffered with '.always()'.

Change-Id: I590175dba1c9761da19a84efe69a5305220d8eb7

Merge "Allow session storage to be configured independently"

Allow session storage to be configured independently

* Introduce $wgSessionCacheType to allow the session storage to be
  configured independently of $wgMemc.
* Renamed $wgSessionsInMemcached to $wgSessionsInObjectCache to reflect
  the new versatility.
* Modernized the relevant code, removing the explicit require_once(),
  used the autoloader instead. Moved it to ObjectCacheSessionHandler.
* Tested with memcached, external sharded MySQL and APC, it all seems to
  work just fine.

Change-Id: I473334bb56cafb4e21ac1c1304d69095676fc0c4

.gitignore: Add .DS_Store for Mac OS X; Organize and a-Z sort.

Change-Id: Idddfac5af757aa2869a2db25f10ca7ef6f4820a9

Remove unused variables/function values returned

Fix trailing whitespace

Change-Id: I53abf75e142f0166032b98e4adb3dabe06643017

(bug 39099) Changed error-levels in Uri.php

Switched component aliases and invalid URLs to wfDebug()
instead of wfWarn() so that it fails gracefully. Also
changed Uri::setComponents() to throw an exception for
invalid components instead of warning in order to be
the same as the other methods.

Change-Id: I2f2b552f8b13b7063c6b85b6a02d05eaba470b90
Signed-off-by: Tyler Anthony Romeo <tylerromeo@gmail.com>

Merge "Follow-up SVN r98012: pass Title objects to LogEventsList::showLogExtract()."

Merge "Set form name for HTMLForms in FormAction subclasses."

Merge "Localisation updates from translatewiki.net."

Merge "Followup c17651/bug 39032"

Localisation updates from translatewiki.net.

Change-Id: I7f355864088419d9fa96788a2b329c858457f183

Merge "Added some docs, fixed some docs and made implicit checks explicit"

Merge "Linker::link() prints deprecated warning if $query is a string"

Followup c17651/bug 39032

makeHelpMsgHelper made a list of "allowed generators" at the same
time as generating stuff that was used to build the api help docs.

Change-Id: Idcc0b19ff62ab8da95c09c75a5222d3116ed0dc5

improve wfStripIllegalFilenameChars()'s readability

make the preg_replace regex more readable,
and expand documentation a little.

Change-Id: Id26d9d42be7d3f2780e82da27dd2477d62d8e106

(bug 33037) Special:newfiles treat its subpage parameter as a limit

This restores the older behaviour of Special:newfiles. This allows
people to easily include variable sized galleries via
{{Special:newfiles/12}} syntax (people could of course previously do
{{special:newfiles|limit=12}} but less people know that.

Also makes IndexPager::setLimit validate the limit in the same way
it validates the limit coming from a url, and makes it so that
calling setLimit takes precedence over its built in determine the limit
code.

p.s. This is my first commit in our great glourious git future, so
let me know if i did anything wrong

Change-Id: I7fc148e56e43b7c453a1a9559ef74b8b3119c6fe

(bug 11142) Improve file ext error reporting in API upload

The API action=upload returns a generic error message when one or more
of the file extensions are blacklisted. It should return a more
informative message, and also return the list of blacklisted extensions
in a machine-readable format.

This changeset introduces a new message,
"api-error-filetype-banned-type", which is to
"api-error-filetype-banned" what "filetype-banned-type" is to
"filetype-banned". As a starting point, I've copied (with minor changes)
the existing filetype-banned-type message to
api-error-filetype-banned-type for all languages for which both
api-error-filetype-banned and filetype-banned-type are currently given.

This changeset also incidentally fixes an internal server error with
format=xml when the filetype-banned error is returned by the API.

Change-Id: I6b0c58fbc5b19aa55286c56fa7da2195ea683ae0

Pass the 'returntoquery' parameter on cookie check.

Change-Id: I6d80ac4ec6a129f7670b046ec5fe4ab35fc4e352

Added WebRequest::getMethod() to get the HTTP method of the request.

This is to replace the usage of $_SERVER['REQUEST_METHOD'].

Change-Id: I45084254c5452b00b0665df78628cfd214e39cab

Merge "Allowing Message objects to be passed as error page title."

Allowing Message objects to be passed as error page title.

Changed OutputPage::showErrorPage() so that the title parameter
can be either a string or a Message object, just like the
message parameter. This allows titles that take parameters.

Change-Id: Icb7f7b0db8599984774f438333b60d66956b6075
Signed-off-by: Tyler Anthony Romeo <tylerromeo@gmail.com>

Linker::link() prints deprecated warning if $query is a string

Change-Id: I47412b67b964ff0d868f181903628de3fd41365a

Merge "Add a Uri class."

Add a Uri class.

Add a Uri class matching our mw.Uri JS class for handling uris.
This class should be helpful in a bunch of places where we end up doing manual
concatenation of things like the path + '?' + query of a url parsed with wfParseUrl.

[tylerromeo@gmail.com: Removed cat() function, fixed wfWarn() usage for aliases
 and added visibility to all functions. Also added test for aliases.]
Signed-off-by: Tyler Romeo <tylerromeo@gmail.com>
Change-Id: Iefdedb7c80cf1d4aab58050edab3ab44ba868a58

Merge "Fix wrong return value in Preprocessor::getChildrenOfType"

RefreshLinksJob performance tweaks.

* Made refreshLinksJob2 always spawn smaller jobs. This can reduce
  the problem of all runners doing the same refresh jobs by increasing the
  granularity of the work to single pages parses per job.
* Avoid master queries when fetching the latest revision for refresh links jobs.
  Also avoid the master for template fetching on parse. A LoadBalancer waitFor()
  call is used instead. The main reason for hitting the master to fetch templates
  was this job itself.
* Fixed bug in refreshLinksJob2 where one missing page would cause all the
  remaining updates for pages to be aborted.
* Factored out some code duplication between the two refresh links job classes.

Change-Id: Ieca51567a888f50a6f15b6c2606323da80d6584b

[FileBackend] Fixed curl call broken by 10ec7a1.

Change-Id: I661d2f586d3e3177c2f233153787597095872535

Merge "Always save the cache if stuff was added to it - dont require deriving class to call save"

Merge "Memcached PHP client improvements"

Set form name for HTMLForms in FormAction subclasses.

Used FormAction::getName() to set the form name for
FormAction subclasses. Without a form name, HTMLForm
creates weird messages names for certain fields (e.g.,
in a form with sections, HTMLForm will use the message
"$prefix-$key" for the legend, but if no prefix exists,
it just becomes "-$key").

Change-Id: I1c19775e44163f4335e30f1d1f73a6e002cb57ec
Signed-off-by: Tyler Anthony Romeo <tylerromeo@gmail.com>

Merge "Fixes to Special:Mostlinkedcategories."

Merge "Use a better message when a revision is not found."

Localisation updates from translatewiki.net.

Change-Id: If5dfc80fe392a76f111dcfe23cdd2432022bc0ee

Merge "[FileBackend] Fixed bogus sync-check status errors."

[FileBackend] Fixed bogus sync-check status errors.

Change-Id: Iefb3a9575e241cfdc3220301a89667d6ad96a307

Add @deperecation stuff to message functions

Change-Id: I69168ed9d850baf02b3dd7c8bd0e820f3f00f43f

Merge "Move message importtext into the fieldset"

Merge "Followup ac8b7d54f81f7b3879a7e21c96b0d182a00618d2 fix broken php detection"

Move message importtext into the fieldset

Than it is the same place as message import-interwiki-text

Change-Id: I0eb800b05848cbea123f895180da02cc1b9af913

Split CacheTime into its own file

Change-Id: Iaa546fc5a30dad4bc0a0dbd0dc18dd2e05bf9d7f

Added some docs, fixed some docs and made implicit checks explicit

Change-Id: Ib8ed516bc1114c58bec6e6213b52658fb34f5c98

In WebRequest, don't use isset() to check that a variable is null

Change-Id: If4577b911e774e1e0c6a1d90c585bbb027d6b7cd

Use a better message when a revision is not found.

This comes from
https://translatewiki.net/wiki/Thread:Support/MediaWiki:Missing-article
that asks for a link to the deletion log in the
'missing-article' message.

Instead of simply changing that message I replaced
it with two news messages 'missing-revision' when trying
to view an old revision and 'difference-missing-revision'
when showing a difference. This removes the "lego"
construction of 'missing-article' with 'missingarticle-rev'
and 'missingarticle-diff' submessages.

I had to keep current messages since they are still
used in extensions.

Change-Id: Ibe60c84cfea922a78b3ad22cc48185a109cd617d

Followup ac8b7d54f81f7b3879a7e21c96b0d182a00618d2 fix broken php detection

Change-Id: Id78f14c220bbaa21b78c0c669a324f6169b3109d

Merge "change some rc_new to rc_type == RC_NEW"

Tweak 'disambiguations-text'.

Change-Id: I51fc0b48f5af8f9d50f7c968d5ecdcf94406a623

Should be used from fallback.

Change-Id: I91ce057883d902d9dc55a781e200136971c94f02

change some rc_new to rc_type == RC_NEW

The field rc_new is obsolete and should be removed, see bug 34320

Change-Id: I411ec2cb0fe3eb078094eded210d4aefee5f8e74

Fix wrong return value in Preprocessor::getChildrenOfType

I am not sure, but this looks wrong, because it adds the type to the
array and not the child.
This method is unused in core and wmf extensions, maybe
removing/deprecating is a better idea, but I am not sure, if that is
possible.
I have only see this possible error, while looking through the
preprocessor.

Change-Id: I5b7492d455989a8a3e71b5db6d31091b986c502a

Merge "Localisation updates from translatewiki.net."

Merge "Rebuild MessagesEn.php"

Localisation updates from translatewiki.net.

Change-Id: I9939a879f65f468d7d3392b5f41c7db2eeffcc0f

Fix grammar issue and add quotes around parameters.

Change-Id: Idd4ff6ac8bc9176a69ad46769e73dc070cd5f3ad

Rebuild MessagesEn.php

Change-Id: Ibaa94493225ee9f20674c3c30df18f3da8afe28a

Support GENDER in 'passwordreset-emailelement'.

Change-Id: I1b14dbb8d1fb06c688064af4a6a02e03293dc5f0

Improve documentation of maintenance scripts.

Change-Id: I557f85e8526a3e4b48107fbf299ff39f6af1ac12

Follow-up SVN r98012: pass Title objects to LogEventsList::showLogExtract().

It seems that I forgot to update Article.php in that revision.

Change-Id: Ia29cc62dc88abb7f58d5f8cec68f67b126763f6f

Fixes to Special:Mostlinkedcategories.

- Don't execute the LinkBatch if there are no rows.
- Checks for invalid titles and output a message
  if this is the case.

Change-Id: I934c151204ee26f2bbfe0dbc8068b96ae7465625

Merge "Fix possible error texts in action=options"

Merge "remove 'cantpurge' from possible errors in action=purge"

Merge "Rename 'file-session-key' in api message map"

Merge "unknown error 'filerevert-badversion'"

fix list of possible errors for list=blocks

Change-Id: I4a091eeb788f465fe6cdbb7fe6d296606185ba3c

remove 'cantpurge' from possible errors in action=purge

Was removed in r74944

Change-Id: Ib13be4a3a9956b5c48f2d435092affe518fd7cd3

Fix possible error texts in action=options

Change-Id: I465b9cf109bd4f22542747ca72876caa3725a353

Rename 'file-session-key' in api message map

Follow up r92009

Change-Id: I4752d47e54f4b13101400bed097eccc525e8506e

Merge "* Bug 39032 - ApiQuery generates help in constructor."

unknown error 'filerevert-badversion'

Adding 'filerevert-badversion' to message map
use some local vars in ApiFileRevert

Change-Id: I2341f3f212af484d446a7a84c1b5c7984597468d

* Bug 39032 - ApiQuery generates help in constructor.

Functions called unnecesserily, as their return values aren't used
or even cached.

Removing them to reduce the pointless overhead of calling them

Change-Id: I0d8ddab9492ea79d5de4118944e77ac774275506

Merge "Improve detection of php binary."

Improve detection of php binary.

- Add php binary and a program which might
be set as $PHP.
- Check for the correct php version.

The break shell builtin is not too commonly used,
but should beperfectly safe to use, since it's specified by POSIX
http://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#break

Change-Id: Iefbe991426f0e9a7178363a30b613f57110e1953

Add a test for paragraphs with extra newline spacing

This test simply documents the current PHP parser behavior, so that it remains
compatible while we are tweaking Parsoid.

Change-Id: I807efb93da6c69da124bcab5c7fbd531b4deaa2b

Merge "Added line to rel notes about CacheHelper class"

Merge "Bug 38676 - password-change-forbidden isn't properly localized"

Merge "use FileRepo api to delete and check for files"

Merge "Make rebuildLocalisationCache.php also obey --memory-limit if set."

Merge "Fixes to Special:Mostlinkedtemplates."

Merge "Use WebRequest instead of $_SERVER in ApiMain."

Merge "Change from wgMiserMode to isCached on Special:WithoutInterwikis"

Merge "Introduce default expiry time for special:block (bug 32178)"

Merge "(bug 38351) Image alignment should follow the page content language"

Localisation updates from translatewiki.net.

Change-Id: Ieac1b68dd05ab55e1129fb94f32354fc86143aa2

Memcached PHP client improvements

* When there is a read error, close the socket and mark the server dead,
  instead of continuing to send requests through it, causing a protocol
  violation.
* Also check for write errors.
* Increase the default stream and connect timeouts from 100ms to 500ms.
* Remove the commented-out _safe_fwrite() code that I wrote in 2005,
  that PHP bug is fixed now, so writes can be handled the same way as
  reads.
* Fix E_STRICT errors in get_multi() due to a resource being used as an
  array index.
* For performance, flush the read buffer less often (only on persistent
  connect).
* Move the fread() loop from _load_items() into its own new function.
* Be stricter about stripping expected newlines, don't just run trim() on
  everything.
* Make MemCachedClientforWiki into a simple alias since the distinction
  with MWMemcached was lost many years ago. We even renamed the class
  "MWMemcached".
* Remove vim modeline made incorrect by 19d6293 (Nov 2009).

Change-Id: I81de31049b87038999e37054d8d174a782c78a68

Change from wgMiserMode to isCached on Special:WithoutInterwikis

This makes it clearer why the input from is not needed.
It does not change anything, because isCached checks isExpensive and
wgMiserMode and isExpensive is true here.

Change-Id: Ib5f775d5ff20b53b91732ca544ec273543afe7d1

link to HTMLForm doc on the wiki

URL provided by Clément Dietschy.

Change-Id: Ibea0266618c09b2dfbac5c356beba93dd8bebba2

Fixes to Special:Mostlinkedtemplates.

- Don't execute the LinkBatch if there are no rows.
- Checks for invalid titles and output a message
  if this is the case.

Change-Id: Ibd3794fbe165b13f3b386a9dabe4b6fabb7ba1d8

Merge "Fix bug causing API to list anons as autoconfirmed in certain cases"

Merge "Fix jquery.localize tests, change number of expected assertions"

(bug 38351) Image alignment should follow the page content language

The alignment of image thumbs should follow the page content language instead of the wiki content language.
For this it needs the parser context, and because it makes sense to have it as first parameter, I renamed makeImageLink2() to makeImageLink(), the 2 seemed to be redundant anyway.
The old function name keeps the old behaviour, but can be removed quite soon since almost no extension is using it.

Change-Id: I0c35b06a85528dcc43fdd0578dc9b327c495cf4a

Make rebuildLocalisationCache.php also obey --memory-limit if set.

Same as https://gerrit.wikimedia.org/r/17473 for runJobs.php

Change-Id: I352df6c2f46606fe575ef5f52a39175b4aad9f75

* (bug 38953) --memory-limit switch not working for runJobs.php.

In runJobs.php the memory limit was hardcoded to 150M, so passing
--memory-limit would have no difference. Changed to using what was
passed as --memory-limit if something was, else just use the 150M
as usual

Change-Id: I3e3bafc6a0fe4e88792a65f01b432c8f4f6677cf

(bug 38904) prop=revisions&rvstart=... should not blow up when continuing

A query using prop=revisions in enum mode will return a query-continue
using rvstartid. If the original query included rvstart, the client will
therefore send both the rvstart from the original query and the
rvstartid from the query-continue. And the API will then return an error
that rvstart and rvstartid may not both be used in the same query.

Since there is no way for the API to instruct the client to ''not'' send
the rvstart when continuing, we'll just use rvcontinue instead. Then the
module can ignore both rvstart and rvstartid to get the intended
continuation.

Change-Id: I145215996b8a7818196cdfb583a1fdacae973fee

Fix bug causing API to list anons as autoconfirmed in certain cases

On a stock install, the autoconfirmed requirements default to zero, so
anons qualify for autoconfirmed. They don't actually get it because
User::getEffectiveGroups() only checks for autopromote groups for
logged-in users. However, ApiQueryUsers::getAutoGroups() (which
duplicates this logic for some reason) didn't use the same rule and
applied autopromote groups to anons too, which caused discrepancies
betwen the API output and wgUserGroups.

Krinkle noticed this because a QUnit test for mw.user.getGroups() was
failing while logged out: the API response included autoconfirmed but
wgUserGroups didn't.

Change-Id: I0b781c11e06d3cc7176b2fb3ba06979d3637f970

Fix jquery.localize tests, change number of expected assertions

jquery.localize: Handle basic replacements (1, 4, 5)

Expected 3 assertions, but 4 were run

Change-Id: I95d89d8ff37befd54735edd565db8d0184a8e39a