Merge "Changing default copyright notice per WMF Legal's suggestion."

Merge "Replace single to double-quotation marks"

Localisation updates from translatewiki.net.

Change-Id: If490bc68434b06359037fac93ec613d0e7265f44

Replace single to double-quotation marks

Give grep a chance to find the usages

Change-Id: I95431f8e5cc684c322fdd7cda041388120b7c49b

Localisation updates from translatewiki.net.

Change-Id: I82effb99c8a9b2f7bedc874008835e29f1d9ae4d

Merge "Deprecate assertException in MediaWikiTestCase as it adds no value"

Merge "SearchEngine::userHighlightPrefs needs no user"

Merge "Simplified some code and broke lines in thumb.php"

update.php clears caches twice

There was a bad rebase on I96b4cfd4c02e9cbf46cc6a0499b87fb3b89020a0,
because it partically reverts Ice42a31dee1e6b41da4aa0a47e8786579382aff1

Bug: 44690
Change-Id: Iff2c8d3cb1a10da7cd0ab18ee189454761d7db6f

Added {{REVISIONSIZE}} variable to get the current size of a revision.

This avoids giving the own page name to {{PAGESIZE:}} and allows
PAGESIZE on subst: to give the size before save.

Viewing a oldid page will give the corresponding size to that revision.
(Like REVISIONUSER or REVISIONTIMESTAMP)

This partially reverts Idfac13de37d05317f65e4131534543e66cf74873
because there is no need to let PAGESIZE handle extra, when the given
title is the own page.

Change-Id: If9e608d54904c21ac6b095e91ff6e0a15da0fb4c

drop unused fields rc_moved_to_ns/rc_moved_to_title

Follow-up I68e8c0bb23f185c0f996a8905f6d437db3080aa1

Deleting those fields also from tables.sql definition
for mssql, oracle and postgres

Bug: 34960
Change-Id: I1867536c9161abed0b23428d3c840d16a05cba56

Tidy up log deletion rows which didn't get subpages

Bug: 37714

Change-Id: I904988c864d777d039b2aae956aef7b19092cf83

Cleanup Preprocessor_DOM::preprocessToObj wfProfileOut()s

Simplify the multiple if levels used for profiling out.

Change-Id: Id0530207f99daca49a6a76ce256476b677a4108f

Add a way to redact certain function parameters from exception stack traces

Bug: 30714
Change-Id: I0a9e92448f8d9009dd594cb4d7f5dc6fdd4bcc86

Remove reduntant regex from calls to StringUtils::isUtf8()

I've cautiously moved the regex out of the most used code path.
There is no string that will match that regex check that will not also be
passed by mb_check_encoding.  I think the regex was intended as a shortcut
evaluation, but it is no faster than mb_check_encoding which will often
need to be run anyway.

I think it could just be deleted, but I have limited motivation to
risk introducing a bug to improve performance on old PHP vesions and
unusual configurations, so I've moved it to the fallback code path.

Change-Id: Ie9425cc23ba032e5aff42beeb44cbb1146050452

optipng -o7 "$png" && advpng -z -4 "$png" && advdef -z -4 "$png"

Change-Id: I2b267d23da582e4daf95c6898e3c687ad63d6e6b

Changing default copyright notice per WMF Legal's suggestion.

Changing from "Content is available under $1." to...
"Content is available under $1 unless otherwise noted."

This is especially relavent to wikis that use InstantCommons.

Change-Id: I878372de6c42bcb7660b111911d0e12bb351d2c5

Note about removing portal-url, currentevents-url and helppage from default Sidebar

A note about the removal of portal-url, currentevents-url and helppage from
the default Sidebar as requested at the related bug report.
Follows up: Icb0e1b91e40bd682629f8a6f7291dbf8588b586

Bug: 53862
Change-Id: If7a624cd3a62c31af84a711f79ea396b53868fd2

Deprecate assertException in MediaWikiTestCase as it adds no value

setExpectedException, provided by PHPUnit itself, can and should be used instead

Change-Id: I06833eeeb492d2cc4e66b14aaebd5b2061a29e3a

Localisation updates from translatewiki.net.

Change-Id: I4d3fce0836a6e2bd2c7c79159fb74411a02b40b6

SearchEngine::userHighlightPrefs needs no user

Was removed with r87215 from the function,
but not from the function call

Change-Id: Iacb247d5962fe77db5a29084ece941261bf306ec

Simplified some code and broke lines in thumb.php

Change-Id: I99fc63ab473a3f2fe33e0227e5e283e0c195f01b

Pass user to Title::userCan in ApiFeedContributions

Change-Id: I15b95f4e4bf4a07b55b98baa360078411c0f42cc

Merge "Collect stats about RL cache hit rate"

Collect stats about RL cache hit rate

Change-Id: I7cf2f2248f60b0e686e90faf9cb9989b723c88d1

Pass context to Action::factory

Fixes a PHP Notice that was introduced on every page (follows-up 01d81d88)

Bug: 54344
Change-Id: I24f01c7244089e73b0cf2e0af37c365f9dd051b2

Reduce use of FORCE INDEX in LogPager

* This currently hurts some change tag queries by orders of mangitude.
  MySQL/Maria query planning is also a fair bit smarter than it used to
  be. Lastly, WMF table statistics are better maintained now.

Change-Id: I4d0ac7a35c9cec6b47771aae718b44bfc03bdfbe

Localisation updates from translatewiki.net.

Change-Id: Ifc090ee9782be50af70146738908549ca31b6f96

Merge "Reduce complexity of SiteSQLStore saveSites method"

Allow RedisConnectionPool to be used with HHVM's Redis library

As of commit 2811e9d, HHVM includes a pure-PHP Redis client library with
method-for-method compatibility with phpredis: <http://git.io/ylDAyQ>.
RedisConnectionPool appears to work well with it save for the
extension_loaded() check. Practically a freebie!

Bug: 54158
Change-Id: I3ac12293a51a45e8f6a61a35bff7a3c063e16964

doc: Fix grammar in GitInfo documentation comment

Change-Id: I67da49398f1281b83700884475d364f6b30c05c8

Merge "Adapt StringUtils::isUtf8 to the top of Unicode at U+10FFFF"

Fix initialization of Context in Action class.

When Action::getActionName calls Action::factory, $context
must be passed explicitly, since WikiPage does not have a getContext()
method (only Article does).

As a failsafe in Article::getContext(), use $this->page->getContext()
only if $this->page is an Article. If $this->context is null and $this->page
is not an article, fall back to RequestContext::getMain()

Change-Id: I6d8db29dfd1aaa68f3de4f8f418841205c2379af

jquery.suggestions: Fix up trailing comments from 2f5125e8

Change-Id: I8f2f8a9f1e0da3e460928003b56a2c99668473da

Factor out ChangeTags::tagUsageStatistics() from SpecialTags

It's prettier if we keep all explicit DB queries to those tables
in one place.

Change-Id: I85b6854e26ffc656b990079d463a085ba39556df

CSSMin: factor out image encoding into encodeImageAsDataURI method

The code for encoding suitable images as base64 data URIs was previously part
of the remap method of CSSMin, which greps CSS files for /* @embed */
annotations. This patch moves it into its own method, encodeImageAsDataURI.

Change-Id: I6f57116c6a6c18ff9d4e28760dd484ea2c8fc493

Adapt StringUtils::isUtf8 to the top of Unicode at U+10FFFF

RFC 3629 defines the legal range of characters as U+0000..U+10FFFF
and forbids overlong forms (encodings of a character that use more
bytes than necessary). Let's make StringUtils::isUtf8() match the
specification.

* Changed the maximum value in the pure PHP code path and added a
  check for overlong forms.
* Added another check, specific to PHP 5.3's mbstring extension,
  for values above U+10FFFF.
* Fixed the mbstring test errors in PHP 5.4 using changes to
  StringUtilsTest by Platonides <platonides@gmail.com>.
* Uncommented some other tests that could fail because of the
  missing check for overlong forms.
* Added additional tests for extra continuation bytes, overlong
  sequences/forms, and values in the UTF-16 surrogate range.

The changes to the function were so extensive that I might as
well say I rewrote it.

Bug: 43679
Change-Id: I56ae496d17ffc3747550e06a72dacab3ac55da61

Merge "Separate RevDel logic from UI"

Reduce complexity of SiteSQLStore saveSites method

split code that generates an ORMRow from a Site object

Change-Id: Ie1f7ed834a33befb43536355ad2b9b98a13d4c79

Merge "Add gzdecode fallback to GlobalFunctions"

Merge "Localisation updates from translatewiki.net."

Merge "Changed tableName so it returns uppercased table names (+prefix) Changed tableExists so it returns boolean (and closes resource when it's done in case it fails)"

Merge "Changed quoting function for oracleDB."

Localisation updates from translatewiki.net.

Change-Id: Ia363d8fc374b04d948cb0b6599269f9c0e1c4117

Fixed check of BagOStuff set() return value in mctest

Change-Id: I6e69cdc24b9b40132641205cb40f64bb5db45f70

Separate RevDel logic from UI

Move functions implementing the actual revision deletion out of
SpecialRevisiondelete, so they can be used from elsewhere (e.g.
ApiRevisiondelete).

Change-Id: Id0f6b1c8dab474bdfd802c07b211885d398c94e5

Merge "Use wfResetOutputBuffers in ResourceLoader"

Merge "Fully log exceptions within ResourceLoader (including traces)"

Merge "RefreshLinksJob: Replace use of empty() with explicit check"

Merge "mediawiki.api: Simplify done/fail calls by chaining"

mediawiki.api: Simplify done/fail calls by chaining

Change-Id: Ic0c1928708612d31abcd6c5c7184f68b2e7c1a76

Fully log exceptions within ResourceLoader (including traces)

Change-Id: Icb479c76cd855244429fe65b29667783dcca8f53

legacy upload.js: Clean up and remove from jshintignore

* Coding style:
  - Braces
  - Single quotes
  - Dot instead of bracket access for properties
* Fixed various implied global/static variables, most significantly
  the for-loop variable 'cached' etc.
* Changed == '' to ! since === '' will require it to be a string
  and == '' returns true for almost anything that is falsy, no
  good way to choose a strict alternative.
* Changed setAttribute('disabled', ..) to using .property instead.
* Using setTimeout as a global instead of as a window property.
* Using other globals from wikibits.js through window property,
  as they are defined.
* Use a closure instead of string eval() in setTimeout().
* Made local equivalents for some global variables for access
  inside the closure.
* Don't create functions in a loop. Moved the onchange function
  outside the loop and re-used it for each one (safe in this case,
  and more efficient).
* Call setTimeout/clearTimeout as global, no need for the window
  object as host.

Change-Id: I9df912ee48d30c189394bf07f5e49014220f36a3

Use wfResetOutputBuffers in ResourceLoader

No need to replicate its logic there.

Bug: 46836
Change-Id: I0c6ad90c327e39b9bc35fc3a9ecab39487a1d8b5

Merge "jquery.makeCollapsible: Use promise().done instead of when().then"

Merge "doc: Clean up documentation for StringUtils classes"

doc: Clean up documentation for StringUtils classes

Change-Id: Ie016c1d1686c9dce7944864e77e6c3bdf001d8c3

jquery.makeCollapsible: Use promise().done instead of when().then

Follows up b340bd4f6b.

They both work, but in this case there is no need to use $.when
since there is only 1 promise. And there is no need for the
filtering proxy of Promise#then either, a straight done is enough.

 $.when( $foo ).then( callback )
 $.when( $foo ).done( callback )
 $foo.promise().done( callback )

Change-Id: I4df755a32c124481ca690078191a735d2e2b511a

Sanitizer: Remove redundant @private tags

Follows-up 64dcaf4e8a.

Change-Id: Ib6f406e4881e7c7ba39cdb132ffb95f4c1a19209

RefreshLinksJob: Replace use of empty() with explicit check

Using empty() is a maintenance nightmare as it completely removes
any meaning of what the author intends to check for (whether the
variable exists at all, it being set to an empty array, 0, '0' etc.)

Follows-up I445e10da122e5f24e589 which changed it from isset()
to !empty() with commit message of checking for false.

Change-Id: I561f03b0526685f51138d723f9eb7196725e0b93

Merge "Make CSSMin::getMimeType public rather than protected"

Make CSSMin::getMimeType public rather than protected

There's no reason why it should be protected. People sometimes use
'protected' to mean 'ancillary to the main purpose of the class', but
'protected' comes with the added cost of making the class API less
flexible. I intend to call this method in a future revision of Id052a04dd2,
but I think it should be public regardless.

Change-Id: I56365df3152df1180ce73de98202e3a91def1485

Merge "Store boolean values as integers with SQLite"

Store boolean values as integers with SQLite

Since SQLite doesn't have a boolean type, and doesn't enforce types, make
sure booleans get stored as integers, to prevent undesirable behavior,
such as false values being stored as empty strings.

Change-Id: I2a96065826412fa25f98ec298d806e73ebe155ba

Add gzdecode fallback to GlobalFunctions

* Follows-up I6383e933b.
* Replaced existing usage of `gzinflate( substr( $data, 10, -8 ) );`
  with `gzdecode( $data )`.

Change-Id: Ie2916584f92ff72c7d7ddca73acf1c94dda38cca

Merge "Clean up legacy protect.js & remove from .jshintignore"

Clean up legacy protect.js & remove from .jshintignore

- Declare all variables w/single var statement.
- Un-quote property names.
- Braces / whitespace / quoting / strict equality checks.

Change-Id: Iaabd3bbaeb10a661adf53dc9ad39813b14661f77

Localisation updates from translatewiki.net.

Change-Id: I87722e0db8d910030b237290b78e7f8e4e7c59dc

Merge "Protect against non-arrays when fetching headers"

Protect against non-arrays when fetching headers

PHP documentation says this can theoretically return false on failure.

HHVM actually returns null when running from the command line, so
this does protect against that.

Change-Id: I0d75b8ed209128a9667ce1e7189597ae9ebc8af6

Merge "Guard against non-object returns from File::getHander()"

Merge "Show the revision ID on error message when content is missing on difference page"

Merge "Removed unneeded else branches in DifferenceEngine"

Merge "mediawiki.user: Clean up documentation and minor optimisation"

mediawiki.user: Clean up documentation and minor optimisation

* Optimise mw.user#id implementation.
* Make documentation more consistent (describe the intent instead
  of the exhibited behaviour, e.g. "Get foo" instead of "Gets foo",
  like most other JS documentation does already).

Change-Id: If9e5dcb7cfed27785c4a15bc3492868f083517f3

Merge "Add TitleMove hook"

Add TitleMove hook

some things need access to the old file before its moved,
adding hook to allow extensions to do something before
file is moved.

Change-Id: Ic5e659abc79c41e3331d42074e7f21eec9b9ba7c

Show the revision ID on error message when content is missing on difference page

Currently if the revision exists but its content is not found, it will make
loadText() return false; which in turn will make a call to showMissingRevision()
from showDiff(). However since the revision exists; it will not match the
condition in showMissingRevision() and thus an error message will appear without
any revision ID.

Change-Id: Idd8a5f20a3c082a3b02bd77557e0f4dc4c66e876

Removed unneeded else branches in DifferenceEngine

The "if" always exit, so there is no need to use else branches;
also added blank lines for better readability.

Change-Id: I7d8321652a90fbba99e53fa0c1fe018492883b8a

Guard against non-object returns from File::getHander()

MediaHandler::getHandler() can return false when a handler cannot be
determined for the given file's derived mime type. This change adds
guards to invocations that I could find that did not properly check for
this potential return result.

Bug: 53820
Change-Id: I8c0165311cc75f9920ac30ce2b38ccd207439198

Rebuild MessagesEn.php

Change-Id: Ic91207589544ec2a7c0ca844ac51fac11a0e542f

Merge "Sync up with Parsoid parserTests."

Merge "Minor bugfix to IP::prettifyIP."

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 8da66db733e02a5a886a62075ab9d9e9c6db6e3a

Change-Id: I0b429b4affbacb4ed97718bce8ba9c886d268216

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I01c4d11140387c16202ffa9c15cc358cf035a455

Merge "Don't ask users to mark own page creations as patrolled"

Remove 'mediawiki.legacy.wikiprintable' module

It's a relic of simpler times, no longer used by any core skin. The
'mediawiki.legacy.commonPrint' module can be used instead.
(SkinTemplate-based skins do it automatically.)

* The 'mediawiki.legacy.wikiprintable' module has been removed.
* The skins/common/wikiprintable.css file has been deleted.
* Skin#commonPrintStylesheet has been deprecated; its return value is
  ignored.

Dependency: I96e66ff8905416bea906d40cdd72ba646399191b
Change-Id: Icbcebc8f539f7786d037b717d262684e9931aca6

Localisation updates from translatewiki.net.

Change-Id: I1f5fe0b2664bc67e8f3fb2e56558834f2f1a427c

Merge "Drop unnecessary CSS selectors for a.external[href^="gopher://"]"

Merge "mediawiki.util: Detect Iceweasel for accesskeys"

Merge "Add optional message to define default description for Upload"

Remove remaining calls to MWInit methods

... as well as the require_once statements in includes/WebStart.php
and maintenance/doMaintenance.php, now that the autoloader lists
MWInit (since r85807 / c68957c5e3ada65806294bc9bc5938aa18c6e50c).

Also removed code paths in maintenance/userDupes.inc that seem to be
dead (class_exists( 'Revision' ) should always be true) and useless
global/require_once statements in languages/Language.php.

Follows-up Ic3e769f1fbad4f7ad26dd819406796fee48c6b45.

Change-Id: I48fd6810fdb923b3065ae98024912eb18d394415

Merge "Web installer: correctly show note for I4ecd0659"

Web installer: correctly show note for I4ecd0659

InstallDocFormatter didn't escape the double underscores in
__TOC__. I also noticed that wfEscapeWikiText had a similar
bug, which I reported as bug 53658 (fixed separately).

Change-Id: Ib5346b9c234f1c5ef361cc4357352b09def5dc25

Drop unnecessary CSS selectors for a.external[href^="gopher://"]

* They don't do anything anyway - gopher:// links are not styled by
  anything else, so they'd just get the same styles as regular
  external links.
* They cause problems in browsers not supporting the attribute
  selector (like IE6), as the browser drops the entire rule, as
  required by the CSS specification.

Also removed some code in monobook/IE60Fixes.css which seems to be
simply a workaround for the issue described above.

Bug: 31495
Change-Id: Ib33d47d5372dd73097bdcef8d00ebd073f6bbd68

mediawiki.util: Detect Iceweasel for accesskeys

Bug: 54015
Change-Id: I105f22756c5df32aef2e4581f5f0d8e3db059dfc

Merge "Always pass user to LogPage::addEntry"

Localisation updates from translatewiki.net.

Change-Id: Id3cc828ebb6b0c06ea052977fcabeec78cacd606

Always pass user to LogPage::addEntry

This avoids getting $wgUser in LogPage::addEntry

Change-Id: Ie105cef4ebc3d69aeaef5ee40ce6ee0087a74c69