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

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

Merge "Consistently follow conventions for documenting parameters"

Consistently follow conventions for documenting parameters

Per MW:CC/PHP, the type goes before the variable name, and built-in types are
not uppercase.

Change-Id: Ibb753acd9529ace3579d57654adc47673fa49719

Merge "wfMkdirParents: recover from mkdir race condition"

wfMkdirParents: recover from mkdir race condition

If mkdir fails, check again to see if dir has been created
since our initial check, and return true if so.

Also, in initial check, only return true if $dir is really
a directory, not a file.

Bug: 49391
Change-Id: I2b331669fae70948ce79ba1477c05968a3095c3d

Merge "Make line breaks in <blockquote> behave like <div> (bug 6200)."

Merge "Localisation updates from translatewiki.net."

Merge "Remove "Misc" tab from Special:Preferences"

Localisation updates from translatewiki.net.

Change-Id: I96c07ba05dd7466f54cd6f3cef315705fc716588

Merge "SpecialPrefixindex: Try not to generate unclickable links"

Merge "jquery.tablesorter: Support genitive month names"

Merge "Adding support for query string params to mw.util.wikiGetlink"

Minor bugfix to IP::prettifyIP.

The offset returned by preg_match with the PREG_OFFSET_CAPTURE is an
absolute position in the string.  You shouldn't *add* it to the
current position in order to advance past the match.

Change-Id: I3631c34e02d9b830bf841bf960855626fbc88eb0

Make line breaks in <blockquote> behave like <div> (bug 6200).

This is an old, old bug: the earliest filed dup is bug 1857, on 2005-04-10.
See bug 51086 for a modern discussion, and bug 52763 for some non-obvious
consequences: indented text inside a blockquote must not trigger
creation of a <pre> block (unlike <div>).

This patch should bring the PHP parser and Parsoid closer together.

This also fixes (or works around) bug 15491, which is really a bug in tidy.
But because <blockquote> content is typically wrapped with <p> tags now,
we don't trigger the tidy bug (see
https://bugzilla.wikimedia.org/show_bug.cgi?id=15491#c7 for details).

Credit to Aryeh Gregor (https://bugzilla.wikimedia.org/show_bug.cgi?id=6200#c8)
and Vitaliy Filippov (https://bugzilla.wikimedia.org/show_bug.cgi?id=6200#c37)
for almost-correct patches for this bug, which saved me a bunch of effort.
Thanks to Subramanya Sastry for pointing out bug 52763 and preventing a
bunch of broken articles on enwiki.

Bug: 6200
Bug: 15491
Bug: 52763
Change-Id: I3696d4ab7b8ad6ebccf8483d6da1722353c1697d

Merge "Clarify "unknown gender" option is a non-answer and resort"

Merge "Fix English gender-unknown message"

Fix English gender-unknown message

Fix for I81f02f03: the verb "detail" means to give details, i.e. to
say more than a single word in response to a question. It does not make
sense in this context.

Bug: 53311
Change-Id: Ifddf5c9a07dc62bc22dd0a3d2986e41a55b8ef33

Optimize Parser::doQuotes().

Performance improvements to doQuotes(), since it is a hot function.

Co-authored-by: Tyler Anthony Romeo <tylerromeo@gmail.com>
Change-Id: If78d4372a2acd78d58b020385da400978716cbf5

Made HTMLForm wrap the submit buttons in a container for styling

Change-Id: I6510493a80c18ba5a466a16e3ca39396f9a60f60

Merge "API: Enforce limit max in ApiQueryBacklinks"

API: Enforce limit max in ApiQueryBacklinks

Change-Id: Id0f0943771e3593c30563469df4820437ded9a99

Adding support for query string params to mw.util.wikiGetlink

This will enable us to easily migrate MobileFrontend from
using M.pageApi.getPageUrl (mobile custom version) to
mw.util.wikiGetlink.

Includes a unit test.

Change-Id: I5224a0910a822f1c3b1b34f505dbcdf879052b39

Add StubObject::unstub, and use it for $wgAuth->getUserInstance()

PHP's __call magic method can't handle functions with reference
parameters correctly,[1] which means that we need to manually unstub
StubObjects before calling such a method. Or you could use call-time
pass by reference with call_user_func_array, which is rather backwards
and verbose.

This change adds a convenience function to do this, and applies it to
the two calls which brought it to my attention.

[1]: https://bugs.php.net/bug.php?id=40694,
https://bugs.php.net/bug.php?id=62455

Change-Id: I74a0c04bf358e164f753b82fbefbd6205d9b2451

Localisation updates from translatewiki.net.

Change-Id: I7f09413ca5fd56cc4a0fc8b806464495577c101e

Merge "Improved search backend fallback in Special:Search"

Improved search backend fallback in Special:Search

Use the srbackend parameter if it's valid.

If it's not, use $wgSearchType if it's not null.

Finally, fall back to the search type of the database.

Bug: 53982
Change-Id: I3605cb4add270543aa20afb037401b2dfbe462f0

Localisation updates from translatewiki.net.

Change-Id: Ib9eb5a43ec80cd1fd531c2aa60770c17a328794b

Merge "Fix inaccurate jQueryMsg comment"

Fix inaccurate jQueryMsg comment

It's done by mediawiki.js, as it says at the bottom of the file.

Change-Id: I2505cde86ff9d9532fce75f3924f88e898a5f5f6

Merge "ToC: Use display:table, so that we can behave like a block element"

ToC: Use display:table, so that we can behave like a block element

The downside to inline-block is that you don't clear other inline
elements. Although inline elements surrounding the ToC are rare, they
are common enough that this introduced some problems on English
Wikipedia.

Instead, let's just display as table. This is a block element and has
all the boxmodel behavior that we want (that of a table, an intrinsic
wide block element). It's not the nicest perhaps, but it works in all
modern browsers without known problems as far as I'm aware.

We still fallback to inline mode on browsers that don't properly
support display:table.

Followup to: I4ecd0659d1f955a9b593d281a3fef0a81c218a52
Bug: 658
Change-Id: Ib8e2801f9a143de370589728aad01926906ffae0

Merge "mediawiki.action.edit.preview: Add .limitreport to live preview"

mediawiki.action.edit.preview: Add .limitreport to live preview

Change-Id: Id65c93322d2d9de2983ba7278f269f9b061e3ade

Merge "Use redirect=no for links to file redirects in "file usages" section"

Localisation updates from translatewiki.net.

Change-Id: I4f52889299ffc8ae8d49634fd8b36d73c66d8b5a

Allow toggling elements with space

This seems more convenient than having the space key
do nothing as it does right now.
Furthermore this is more consistent with the various
other collapsible things we have around than to just
scroll the page on a space keypress.

Bug: 53797
Change-Id: If9292ef6a8fd82b097c597dc166e38c8ca31f4ff

Merge "jquery.byteLength: Improve documentation and tests"

jquery.byteLength: Improve documentation and tests

Change-Id: I6793487b7cd9f58b23554bc29c853bd3f02da49c

Use redirect=no for links to file redirects in "file usages" section

Bug: 50051
Change-Id: Ie77af42b2d9183ba2f56a300a6604c3f71ee89b8

Merge "Removes portal-url, currentevents-url and helppage from default Sidebar"

Removes portal-url, currentevents-url and helppage from default Sidebar

portal-url ("Community portal"), currentevents-url ("Current events")
and helppage ("Help") lead to empty pages in a fresh MediaWiki install,
when most projects won't need these pages ever. Removing the
corresponding lines.

Bug 53862
Change-Id: Icb0e1b91e40bd682629f8a6f7291dbf8588b586b