Get rid of wfScript in MediaWikiSite::normalizePageName

wfScript should not be used to construct URLs for calls to other wikis

Bug: 47242
Change-Id: Id5aaa58f10d513e082b217aeb8d88b8228341a48

Removed old cache update b/c code from 2007.

Change-Id: Ia19062cb48f7748501c061544b2949bf3beaba4d

Merge "Fixed incorrect parameter passed to LoginForm::userBlockedMessage()"

Merge "Added spaces before and removed spaces after 'array'"

Fixed incorrect parameter passed to LoginForm::userBlockedMessage()

The error is Catchable fatal error: Argument 1 passed to
LoginForm::userBlockedMessage() must be an instance of Block, null given.

The problem is that User::$mBlock is not always the correct block, when
checking that a blocked logged-in user can create an account.

bug: 47138
Change-Id: I43f35cfa342737f05a08a693a86949c14348321f

Added spaces before and removed spaces after 'array'

Fix some other spacing while at it

Change-Id: I13f81533eaf40e06c13cf377c0f08ef5cef01d00

Merge "Use a provider for Xff blocking tests"

Update/add documentation

Change-Id: I897d5833cdaf7dc2d3bca5bb5551958611f22bd9

Use a provider for Xff blocking tests

Move the block insertion to addDBData(), and use a provider to
itterate through the test strings.

This seems like the more correct way to run these tests, although it
still seems hacky, since the db isn't cleaned up after each test.

Change-Id: Ib22ce69797ec03a61b0cc330e8572865290ebd31

Merge "Added/Removed spaces around string concatenation"

Merge "JSHint: Re-order jshintrc options to split common from project settings"

Added/Removed spaces around string concatenation

And added/removed spaces around some other tokens,
like +, -, *, /, <, >, =, !

Fixed windows newline style

Change-Id: I0b9c8c408f3f6bfc0d685a074d7ec468fb848fc8

JSHint: Re-order jshintrc options to split common from project settings

There are certain options required by our coding conventions.
The others are specifically overriding it for core. Either
because we have certain code that needs laxing or because we
can be stricter.

Comments can be used in jshintrc since a while now. Though
they are invalid JSON, many parsers (including JSHint) allow
them for documentation purposes.

Change-Id: I43ace21208121e866900b2cbe1efbcebadd2946a

Merge "Various code and doc cleanups to ChronologyProtector."

Merge "Fix sequence issue with /signup URL form"

Some comments for Preferences.php

Change-Id: I65d448acf1a53c95efe94587b695718f3231f538

Merge "Update/fix docs"

Update/fix docs

Change-Id: Ifa3cfc6fe38ea45e70b50683c36e7cbf3bb4ef82

Fix sequence issue with /signup URL form

/signup subpage affects form display and title (just like ?type=signup),
so check for it earlier in execute().
Also rename execute()'s $par parameter to $subPage to match SubPage.php.

Change-Id: Ida615a1039bfa1d8328f5fda24c4e3da66b9d162

Merge "Update limit/urlwidth param doc of prop=imageinfo"

Merge "Localisation updates from translatewiki.net."

Made $wgMaxBacklinksInvalidate actually work.

* The check never worked before due to the 200 limit passed to getNumLinks().

Change-Id: I009c757c6437f22544a8d7d09d3534950c32c887

Localisation updates from translatewiki.net.

Change-Id: Ie707fb7d84d7ceb47089a618707a05c45c29e7f1

Merge "Support for linking directly to sections in ApiFeedWatchlist.  It works only if a section name is present in item's comment."

Use wfAppendQuery to append a query

Also pass a array to wfAppendQuery, which than does the urlencoding.

Prefer a wfScript() over the global or string

Change-Id: Icada534cb4c99c9441938a2e8dcbc11a142360c6

Merge "Add devunt to CREDITS as patch contributor"

Add devunt to CREDITS as patch contributor

Contributions to core are in Gerrit changes I316685262, I42be68c6a, and I45d04691a.

Change-Id: Ic917af8e0c7c9f49528a7309e22c0a60505ba5e8

Merge "Revised design of Special:Userlogin"

Various code and doc cleanups to ChronologyProtector.

Change-Id: I4d76e4c627e399f733d078c5cc97020f363094c4

Merge "Add ChronologyProtector to ExternalLBs"

Revised design of Special:Userlogin

If a global variable is set or if you add ?useNew=1 to the query string,
Special:Userlogin loads a different login template (UserloginVForm.php)
with the new Vertical Form appearance and different messaging.
Otherwise the current unchanged template renders so that wikis can cut
over to the new look when ready (with messages and links). Once they do
so, the variable and flag will be retired.

The new template applies mw-ui-vform and mw-ui-button styles defined in
a new 'mediawiki.ui' CSS module in core to create a compact vertical
form.  The mw-ui styles specify a Helvetica font stack (that we tested
in the Account creation experiment) in the form area, but NOT if the
user is using some other skin than Vector.

The CSS code is developed using Sass. The patch includes the
Sass scss files, along with a Makefile that uses their Compass build
configuration (config.rb).

The redesigned Special:Userlogin also:

* Displays a "secure login" link if HTTPS is available.
* Loads additional CSS for its form features (e.g. more attractive
  errorbox, "Join wiki" messaging).
* Defines new "userlogin-xx" messages; many are the same as
  existing messages but without ':' on the end.
* Uses a distinct title for Login instead of generic "Log in /
  Create account".
* Removes the [mailmypassword] code branch from its login template as it
  is never executed.

Bug: 44628
Change-Id: I489042c50aa060c90ca18b05097dbe25c4ae6395

Merge "Moved ChronologyProtector to its own file."

Merge "Fixed use of __METHOD__ in a closure."

Merge "phpcs: Fix Squiz.WhiteSpace.ScopeClosingBrace.ContentBefore violations"

Moved ChronologyProtector to its own file.

Change-Id: I01bf0206322454e36633921887d10356a48ab5cb

Fixed use of __METHOD__ in a closure.

Change-Id: I41dc0ba4aff39d5a65cd1ccb3fba6c102974e6c2

Merge "Localisation updates from translatewiki.net."

Merge "Disable Special:PasswordReset when $wgEnableEmail is false"

Localisation updates from translatewiki.net.

Change-Id: I6c19864b175b2cbb84273326a981467af2db8610

Disable Special:PasswordReset when $wgEnableEmail is false

Previously one could still navigate to the page by entering its URL directly,
and from there be able to request a password reset e-mail for any user who has
an e-mail address configured. This patch adds a check for $wgEnableEmail in the
SpecialPage code, replacing the interface with an error message when e-mail is
not enabled.

Change-Id: I42be68c6a068aafdf45623eb35bb21aa684f54f6

Merge "Minor fix to redis queue docs."

Merge "Consistency tweak: Wrap variable names into <code></code>"

Minor fix to redis queue docs.

Change-Id: I27394ddc8187ab3e9c275e2d72911318cc496604

Merge "Added a quick test for getAllQueuedJobs()."

Update limit/urlwidth param doc of prop=imageinfo

Follow up Id0ec6a and to clarify things like bug 46782

Change-Id: I2215d022a47cd8ceeadd2f5d38064495f3197772

Merge "Remove externaledit/externaldiff preference"

Update documentation and styling

Remove some trailing whitespace, break long lines, make PHPDoc consistent
and update it in places.

Change-Id: I340b43a47e121a44ebd4aaa2a45dff12e945e645

Add ChronologyProtector to ExternalLBs

Change-Id: Ic6c4e86b520f7899172e3ab43e81c4d1d9637d82

Remove externaledit/externaldiff preference

Change-Id: Ic3791d188688f556753ab5556a4d9733bbf2e85e

given name correction in CREDITS

Change-Id: I1afdad0129980a988e5626351cff94c38d01688f

phpcs: Fix Squiz.WhiteSpace.ScopeClosingBrace.ContentBefore violations

ERROR: Closing brace must be on a line by itself
Squiz.WhiteSpace.ScopeClosingBrace.ContentBefore

- For non-empty scopes it means the closing brace must be on a separate
  line. This is already the case in most classes in some cases the "lazy
  closing" is still used.
  array(
   'x' ); // Moved } to next line
  function () { return 'x'; } // Moved } to next line
  case 1:
    stuff; break; // Moved break to next line
- For empty function it serves as a visual distinction between there not
  being a scope block / function body and there being an empty function
  body.
  function foo(); // No body
  function foo() {} // Empty body - violation
  function bar() { // Empty body corrected
  }

Change-Id: I0310ec379c6d41cc7d483671994d027a49f32164

Add @since line for User::groupHasPermission()

Change-Id: Ibcd20a3921058a010334e0e0983e2d9541aea734
Follows-up: I41edb091fa35c8c68b6f95cc5fd208ea99418cdb

Merge "Minor coding style whitespace cleanup"

Minor coding style whitespace cleanup

Change-Id: Ic066eda3a1a2db51b2a953a81829ce7f92568125

Localisation updates from translatewiki.net.

Change-Id: Id27f47305ed695264d422df9fa51d6492b838b88

Vector: Move 'editwarning' feature from Vector extension

* Created a new Resource Loader module
* Created an entry for the RL module in the Resources.php
* Added messages to languages/MessagesEn.php and languages/MessagesQqq.php
* Called the new RL module from the EditPage.php
* Added required entries in the messages.inc, DefaultSettings.php

Corresponding change in the Vector extension: I164f17255bc927

Bug: 46514
Change-Id: I7bdbd09f4083ccb316156307400ccfacdfec79e1

Merge "Use HTMLForm::setWrapperLegendMsg() in FormSpecialPage::getForm()"

Add release notes for Iea1f7842

Change Iea1f7842 (Add timezone support to Language::sprintfDate) did not
include an update for RELEASE-NOTES-1.22.

Change-Id: Icad1228b90b1e30130cc820faaba7c0c8f2254a2

Use HTMLForm::setWrapperLegendMsg() in FormSpecialPage::getForm()

There is no need to call $this->msg() directly for the fieldset legend
as there is already a method in HTMLForm to do it.

Change-Id: I771520ea230301b51cece685f47e6a606ad860be

Merge "Remove dependency on ORMTable from ORMRow"

make error message pretty

Change-Id: I45d04691aca8a1e2c7bb94537da8e04d2bbf0481

Add nosummary, minor and watchthis to Special:MyPage and friends

These are options usable with action=edit and allows better options
for the inital edit view from urls or the InputBox extension.

Change-Id: I696c47ddf9f29764bc8cc6136c618c56dbc2d726

Remove dependency on ORMTable from ORMRow

IORMRow implementing objects take a IORMTable object in their constructor.
The later is a hard to construct service object while the former ideally
should just be a simple wrapper around a database row. This means IORMRow
objects are tightly coupled with IORMTable objects, which makes them
inflexible and makes various things such as testing logic contained in
them needlessly difficult.

This commit gets rid of this nonsense by allowing for construction of
ORMRow objects without providing an ORMTable. All methods dependent on
the table field have been deprecated. Most of these methods have a new
alternative in ORMTable. For instance, saving an ORMRow can now be done
by passing an instance of ORMRow to the updateRow method of an ORMTable
instance, rather then calling save on the ORMRow instance.

Backwards compatibility has been retained except for the fields passed
in the constructor no longer undergoing magical unserialization if it
looks like this is needed. I do not expect this will affect any existing
code though.

Change-Id: I86368821fc2cd0729df5342b8572eb470c0f77a0

startup: Lower blacklist for Firefox from 4 to 3.

Though Firefox is 3.6 is both unsupported by Mozilla, by jQuery
and by MediaWiki JS, it is not known to have fatal errors in
javascript run time execution so moving from Grade B (blacklist)
to Grade X (unsupported but given Grade A source code at own risk).

This is done following a thread in English Wikipedia's Village
pump requesting to get the Grade A resources again.

Follows-up 02fe025e6364ebf6f.

Change-Id: Ia54dd738b3ce0995fab6aae7a2729cc7b1a0dcb1

Fix message cache expiry semantics

* Use the stale message cache while the new one is being generated
* Revert I811755d4 (make message cache load failure fatal). This
  escalated several very plausible temporary site issues from barely
  noticeable to complete downtime -- for example, memcached being down
  on a site with only one memcached server.
* Remove $wgLocalMessageCacheSerialized, it's always been pointless
* Clarify a couple of comments.
* Increased lock wait timeout to 30s
* Make lock() fail immediately on memcached connection refused

Tests done:
* With local cache enabled: normal cold refill; refill local from
  global cache; use stale local cache during remote refill; use stale
  global cache during remote refill; cold cache wait for remote refill;
  saveToCaches() failure; memcached connection refused.
* With local cache disabled: saveToCaches() failure; cache disabled due
  to "error" status key; memcached connection refused.

Setting a 1-day expiry in memcached, with a ~10s CPU cost to replace, is
not the best idea since it inevitably leads to a cache stampede. Dealing
with the stampede by waiting for a lock is not ideal, even if it were
implemented properly, since it's not necessary to deliver perfectly
fresh message cache data to all clients.

This is especially obvious when you note that barring bugs, expiry and
regeneration always gives you back the exact same data, because we have
incremental updates (MessageCache::replace()). Keeping all clients
waiting for 10s just to give them the data they have already is pretty
pointless.

So, continue to serve the site from the stale message cache while the
new one is being generated.

One caveat: if local caching enabled, when the message cache becomes
stale, a sudden spike in network bandwidth may result due to the full
array (also typically stale) being fetched from the shared cache.

Bug: 43516
Change-Id: Ia145fd90da33956d8aac127634606aaecfaa176b

ApiFormatXml: remove broken xmldoublequote param

r55641 was a refactoring of ApiFormatXml::recXmlPrint() that added
a $doublequote parameter, yet the parameter was not actually passed
along in recursive calls, rendering the formatter's xmldoublequote
parameter useless.

XPath support was the sole reason this parameter was added. It was
argued that because in the XPath 1.0 grammar, a Literal is:

    '"' [^"]* '"' | "'" [^']* "'"

it is not possible to match attribute values that contain both an
apostrophe and a double quote.

However, conforming XPath implementations include a concat() function
that can be used to work around this limitation. Furthermore, XPath 2.0
allows escaping a double-quoted " as "" (and a single-quoted ' as '').

Because there was no Bugzilla report (other than mine) filed in the
last 3-1/2 years, and working around the XPath flaw is possible, I
think it's safe to remove the parameter. The worst that could happen
is that a not-very-robust API client chokes on the "Unrecognized
parameter" warning.

Bug: 46626
Change-Id: I9a4d8c13b78ffd2634d03c8c8b4bbeff69f4bc08

Merge "add htmlform-no and htmlform-yes messages for generic yes-no questions"

Merge "Added a script to copy jobs from one queue to another."

Merge "Honor transform limit for images with many old revisions"

Merge "(bug 33454) Add timezone support to Language::sprintfDate"

Added a script to copy jobs from one queue to another.

Change-Id: I9173f21563089e094d67deaf203797d90aa84990

Merge "ResourceLoader: Don't cache ob_get_level"

Honor transform limit for images with many old revisions

When preparing Id0ec6a0a, for some reason I thought the thumbnailing
only applied to the current revision. Fix that oversight.

Change-Id: I2e8aecc76a1190bac353a2b9855bc6a19ec06dec

(bug 33454) Add timezone support to Language::sprintfDate

Add an optional timezone parameter to Language::sprintfDate, add format
characters eIOPTZ, and correct crU.

While we're at it, remove backwards-compatability code for 'N' and then
merge the existing switch cases for cr and wNzWtLoU that are basically
identical, since all those cases need to be changed anyway.

Bug: 33454
Change-Id: Iea1f78428bc0d32d6395818311dbe4b94d776c42

Merge "Fixes to Special:PagesWithProp"

Merge "Add SVG version of user icon in Vector personal portlet"

ResourceLoader: Don't cache ob_get_level

Follows-up I0eba7fcc9ce. This was br

Bug: 46836
Change-Id: Ida0e48931781be77327073dc4ed3966949f4b082

Merge "mediawiki.jqueryMsg: Implement HTML support and improve HTML escaping."

Add SVG version of user icon in Vector personal portlet

An SVG version of the user icon is provided only for browsers supporting
SVG. To ensure browser compatibility, the SVG version is provided in a
two-layer background where the first layer is a transparent gradient.

The fact that browsers supporting CSS gradients are a strict subset of
those supporting SVG guarantees an appropriate fallback.
Embedding is used to avoid extra http requests.

Bug: 35341
Change-Id: I914da0649459744ccca9e1a78e9f48fe66e1a77f

Added a quick test for getAllQueuedJobs().

Change-Id: Ia39461bc1e2356da1a694bf1898b8ca8e4200ff8

Merge "Localisation updates from translatewiki.net."

Merge "Cleared ResourceLoader blob store after update.php finishes."

Localisation updates from translatewiki.net.

Change-Id: I83961608dc6bc3c0cea008592300972c027b5de1

Batch all StatCounter stats into one datagram in flush().

Change-Id: I50a2cd24764e0a284c4a1138f8424ebf3990792e

Merge "Make sure a job is logged as OK if the status is NULL for sanity."

Support for linking directly to sections in ApiFeedWatchlist.
 It works only if a section name is present in item's comment.

Change-Id: I2547c10b01fab2f835fce856f6213386f309f0af

Cleared ResourceLoader blob store after update.php finishes.

Update.php is supposed to clear all localization caches. This
makes sure to clear the ResourceLoader's own message cache
as well.

Bug: 44524
Change-Id: I15ceb60f5e2e4d3ed9ba818fe6cd619f9adce5b2

Merge "jsduck-gen: Add --version parameter"

Merge "startup: Blacklist incompatible mobile browsers"

jsduck-gen: Add --version parameter

Follows-up I1d5b626.

Change-Id: I7a48e549e6acd2bdb7be9f6f2c34ff0554bb791c

Merge "Revert "Remove is_numeric check from Title::checkUserBlock""

Revert "Remove is_numeric check from Title::checkUserBlock"

Breaks unit testing (see bug 47031). The commit managed to land in
the branch because of a bug in Jenkins (bug 46723).

This reverts commit 8cc0b601aa2db6db09ac0e4d70847293d75875aa

Change-Id: I4b3fadccaae9c35964a0c47d63b22c4f35148a24

Merge "Remove is_numeric check from Title::checkUserBlock"

Remove is_numeric check from Title::checkUserBlock

This should allow the usernames of administrators such as "7"
to show correctly on permissions error pages.

I extracted the working code from UserBlockedError::__construct
into a separate method Block::getPermissionsError, called from
both places with context provided as an argument.

Bug: 46768
Change-Id: Ic3fa926a5a4c109faff35fffbccb60fb06ea4a18

Get rid of preg_replace( '/.../e', ... )

This is deprecated as of PHP 5.5, and the remaining uses are quite
silly. Tim said I should remove his easter egg from Special:Version,
as it already was broken, and a new one can be added in a separate
commit.

Change-Id: I0f09f4efc7afe5933c8317462026a475530a5324

Allow search backends to return statuses instead of result sets

This will allow a backend to propogate more useful errors up
the stack.

Bug: 43544
Change-Id: I54ab520762381e8f46d5ca17c0bad8820eacb964

startup: Blacklist incompatible mobile browsers

These user agents have been blacklisted on the mobile site for some time.

They have been re-reviewed alongside the A grade phones supported by
jQuery mobile [1] and jQuery [2]

[1] http://jquerymobile.com/gbs/
[2] http://jquery.com/browser-support/

Change-Id: If7743c476e95002174763514428c17dfcbe7da4c

Merge "bug 42649 - PHP Fatal error: Call to a member function isLocal() on a non-object in Title.php"

Remove first letters that have an overlapping prefix.

First letters are supposed to be primary collation elements.
However, we do not want expansions to be considered
as firstletters (aka thorn "þ" -> "th" which isn't
the same as any other first letter (since "t" !== "th" )
however if þ was a first letter, the word "the" and
even worse the word "too" would be sorted under it, which
is wrong.

Looking for feedback if this all sounds sane. I have tested
it, it got rid of the contractions while at the same time
not removing any letter it wasn't supposed to.

Once this is merged, we could get rid of all the
-<langcode> entries. The other firstLetter array
entries for tailorings could be merged into
generateCollationData.php too, since incorrect
things would get pruned automatically, which
would probably make the logic in Collation.php
simpler.

Bug: 43740
Change-Id: I4bd3d39ec2938a53e2c6728adc48ee6cf9778d74

bug 42649 - PHP Fatal error: Call to a member function isLocal() on a non-object in Title.php

Change-Id: Ie6027c818785b12f6be9c47caab77ffa4a587a79