Merge "resourceloader: Sanitize lang code before creating Language object"

Merge "Minor fixes for existing jsduck documentation in resources/mediawiki/*"

Minor fixes for existing jsduck documentation in resources/mediawiki/*

mw.loader.store:
* Turn url into a link.

mediawiki.language:
* Turn url into a link.
* Fix whitespace.

mediawiki.Title:
* Fix broken <pre> rendering.

mediawiki.Uri:
* Add protocol to url, otherwise <> won't link it.
* Remove bogus dependency listing.
* Turn url into a link.

Change-Id: Ie8e2d1869843bcf594763250c7a9bf94bf874298

Merge "mwjsduck-gen: Fix duplicate resources link"

Merge "Add jquery.arrowSteps to jsduck"

mwjsduck-gen: Fix duplicate resources link

If jsduck failed, it would make a second symlink to ../../resources/
in resources/ which made no sense.

Change-Id: Iedf10466acdbd8e70a751ae898dc453a9be62275

Merge "mediawiki.html: Fix jsduck syntax for list, backtick html, link url"

Add jquery.arrowSteps to jsduck

It's probably a doomed module, but we may as well document it.

Change-Id: I5c28d1abc5364de839806ed21c2edc0d1d655f86

mediawiki.html: Fix jsduck syntax for list, backtick html, link url

* For the list to render as a list (instead of a paragraph
  with hyphens in it and collapsed linebreak whitespace), it has
  to be in block context. Meaning, a line break first.

* Backtick the <br> tag, otherwise it becomes an actual line
  break as Markdown supports <br>.

* Turn the url into an actual link (unlike wikitext, Markdown
  doesn't do magic linking of urls).

Change-Id: I0e3913cd5f97bf37381a99cbdc6af651a8872c70

Localisation updates from https://translatewiki.net.

Change-Id: I31ee0d1685cdf6b315cb5336b30e548c76d67a95

Merge "Remove button styles from mediawiki.ui"

Merge "Suppress section edit links with action=render"

Merge "Stop make kss from opening browser"

Merge "Use Agora oojs-ui theme on mobile"

Merge "Update OOjs UI to v0.1.0-pre (1c7875205a)"

Stop make kss from opening browser

Instead add make kssopen that opens the browser with the styleguide.
Most people will probably keep the style guide open in the browser and
refresh it themselves if they need to.

Change-Id: I5fd82d01da1c7f4bbf48ad921c2e7cb0407974b3

Use Agora oojs-ui theme on mobile

Change-Id: I53e15fca1facfc8b55eace07de42fced4d07efcc

Update OOjs UI to v0.1.0-pre (1c7875205a)

New changes:
a14026f Move some styles to the Apex theme
9df7af9 Add basic styling for Agora theme
deefcf3 Localisation updates from https://translatewiki.net.
1c78752 Add initially enabled ToggleButtonWidget to demo

Local changes:
* Change local update script so that it also copies oojs-ui-agora.css.

Change-Id: I77e6876899809db3f6e246579f82c1bf8bc51a45

Merge "Have ?download parameter trigger Content-Disposition: attachment"

Merge "Comment generated code to explain the i18n JSON migration."

Comment generated code to explain the i18n JSON migration.

Change-Id: I4d41be5a70eab5ceeec3be0bd544e4a87504572a

Merge "Declare visibility on class properties of SpecialRevisionDelete"

Merge "Declare visibility on class properties of SpecialUndelete"

Merge "Make CodeSniffer checks pass on includes/specialpages/"

Merge "Declare visibility on class properties of RedirectSpecialPage"

Merge "Add a 'revdelete-selected-file' message on Special:RevisionDelete"

Declare visibility on class properties of SpecialRevisionDelete

All class properties are used solely in the class itself.
SpecialRevisionDelete is not being subclassed, so made all properties
private. Also:
* Added missing class properties.
* Renamed $list to $revDelList to be more specific.
* Updated some documentation.

Change-Id: I9f989a020d46173cb313d4d3dddc33f6b92b1942

Declare visibility on class properties of SpecialUndelete

All class properties are used solely in the class itself. SpecialUndelete
is not being subclassed, so made all properties private.

Change-Id: I32792fb5d21178d850764c0efe3024d957fca2a8

Make CodeSniffer checks pass on includes/specialpages/

Fixes the one remaining warning.

Change-Id: Id54a1eb148aaf6953db61e8900a9253591f05208

Declare visibility on class properties of RedirectSpecialPage

Change-Id: I54c0300af1ff62be7cc5a9519e97b7f825ddf09c

Merge "Remove "include redirects" option from search"

Merge "Update formatting for includes/specialpage"

Remove "include redirects" option from search

Always include redirects. Search results are worse without them
and MWSearch has never respected this option anyway.

Change-Id: I0ebe321a3b14e7960aa98e3225dc5a19ba916e7d

Update formatting for includes/specialpage

Change-Id: I6f5ebe5408464cde78fae3584e19392b26d2b2a1

Merge "Fix CodeSniffer errors and warnings in includes/specials (2)"

Merge "Try to make a dent in CodeSniffer reports on includes/ (1)"

Fix CodeSniffer errors and warnings in includes/specials (2)

Change-Id: I9787d2a1515b2a8d9310ea636650e20d041acc3d

Merge "Fix CodeSniffer errors and warnings in includes/specials (1)"

Merge "Declare visibility on class properties of LoginForm"

Merge "Set visibility for some methods and remove unused ones"

Fix/update inline documentations in GlobalFunctions

I mainly fixed PHPDoc comments that are highlighted by PHPStorm
as being wrong or incomplete.

* Fixed some wrong inline documentations, e.g. wrong @return types.
* Used the style suggested by the Doxygen and phpDocumentor
  documentations: @param type [$optional,...]
* Doxygen and phpDocumentor suggest the order: @param type $var
* Avoid unclear "varargs" and "mixed" if possible.
* Lowercased "array" and such, they are no classes.

Change-Id: Iff16e05fed6652cefec29e3d254807f52d84c9ab

Merge "Revert "New Hook rc/watchlist hook ChangesListBegin""

Revert "New Hook rc/watchlist hook ChangesListBegin"

This reverts commit 01798c3813b602bd3e8f9a666249e6da58526109.

The patch set changed a RecentChanges interface used in at least the
extension CleanChanges so that it was generating a lot of warnings.
Additional parameters to existing methods that are used elsewhere
should have defaults to not break backward compatibility.

Change-Id: I1851e23e186ba7aaeb001ba212e56888657a3ae0

Merge "Use /usr/bin/ as default folder for DjVu tools in unit tests"

Merge "Fix/update inline documentation in WikiPage class"

Merge "Fix custom local MediaWiki:Helppage values"

Merge "New Hook rc/watchlist hook ChangesListBegin"

Set visibility for some methods and remove unused ones

* private: checkUnicodeCompliantBrowser(), makesafe(), unmakesafe().
* protected: safeUnicodeInput(), safeUnicodeOutput().
* unused and deleted: safeUnicodeText(), sectionAnchor().

Also updated documentation for touched methods.

Change-Id: I1a46113a6380505aeea71150b6d106ec68423002

Fix custom local MediaWiki:Helppage values

Followup to 3a6ea89d4aa4a0fbc36a209fa13b1c6134110ca9

Change-Id: I7effec352d263f536d1d1763e9cd56d2f5154f13

Merge "Reduce connect timeout in MediaWiki::triggerJobs"

Reduce connect timeout in MediaWiki::triggerJobs

The fifth parameter to fsockopen is the number of seconds
to wait before declaring the connection a failure. When not
provided it defaults to the php ini value of 'default_socket_timeout'
which is 60s.  Under no circumstances should we wait 60s to
connect to ourselves,  100ms seems like a reasonable timeout
since its explicitly connecting to itself, but the exact timeout
could be slightly longer.

Change-Id: I18d328274ddf1e0848fce40ebf39e0466b5c4d5d

Merge "Remove wfDebugLog() call from wfSetupSession()"

Localisation updates from https://translatewiki.net.

Change-Id: I083069e8633deaa49055a255e05692dcb1072193

Merge "Sync up with Parsoid parserTests."

Merge "Handle exceptions when testing row validity"

Remove wfDebugLog() call from wfSetupSession()

Since Iffba121a99 (00b7f76) with the removal of wfHttpOnlySafe(),
session cookie's parameters are based only on configuration
settings, so there is no point to spam the "cookie" log group
with predicitible values.

Change-Id: I8b1cdea929cefc32dd8b01c2ecbf2d76bb64189f

Fix/update inline documentation in WikiPage class

* Add missing "new" to methods that actually support it.
* Use more precise type hints in PHPDoc comments.
* Lowercase "string" and "array", they aren't objects.
* Drop a useless comment that only repeats the method name.
* Add some line breaks.

Change-Id: I2f50179eedffcb1809c46fa7ee1c062144a94ffe

Lowered high lock timeout in mergeViaLock() and lock() in BagOStuff

Change-Id: Ib5cd0a98ce6ec47edb24f8cc9f4df2c941ce1c38

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit b2e209c4d370a4a83f33bb1a9f6f7794e7fda3ca

Change-Id: I27e0373436cbfd16b1558ca2b2e998144b808f4b

Fix CodeSniffer errors and warnings in includes/specials (1)

Change-Id: Iafbd82756b00b1ddd5e38a6603037845238c08fd

Merge "Remove $wgHttpOnlyBlacklist"

Merge "Remove unused style for #loginend and #signupend"

Merge "Adding function documentation for Skin::footerLink()"

Have ?download parameter trigger Content-Disposition: attachment

This parameter triggers Content-Disposition: Attachment which
makes the browser download the image instead of displaying it.

This is needed by Media Viewer to allow users to click a button in
order to download an image at a given resolution or the original.

Change-Id: I470a24a09139ac65588312104995e34d97a89b0f

Remove unused style for #loginend and #signupend

loginend in skins/common/commonContent.css was added in
https://gerrit.wikimedia.org/r/53614
and is now unused.

clear:both for #loginend and #signupend in skins/monobook/main.css
is not necessary because there are no floating elements above.

Change-Id: Ic472f43f1d1a7ab4b7abb9f19067574f12ce6a6b

Use /usr/bin/ as default folder for DjVu tools in unit tests

It's the path in most GNU/Linux distributions

Allows also tests to run in Travis

Change-Id: I2fead84184dca94e1407104287f42ebf6112d712

Merge "Fix documentation of wfDebugLog()"

Merge "Set up node-jscs via Grunt (and pass it)"

Merge "Update formatting for includes/specials/"

Merge "Autoloader incorrect case warnings fixed"

New Hook rc/watchlist hook ChangesListBegin

An extension hooking into recent changes may need to load additional
data that did not fit into the recentchanges table.  This hook gives
revisions an opportunity to see the full result prior to rendering
and batch load where approprite rather than loading piecemeal during
the render process.

Change-Id: I28d4e41437e485e518f2a23b6da00cdc430a8c23

Update OOjs UI to v0.1.0

New changes:
4f4fc2c Localisation updates from https://translatewiki.net.
ea88cce Localisation updates from https://translatewiki.net.
37af34f Add the ability to clear the SearchWidget
1cd4b32 Make PopupWidget clippable.
81f2edc Localisation updates from https://translatewiki.net.
fa6e8e5 Make popup toolgroup labels a little closer to their icons

Change-Id: I63fb1f9edb24a86cc882b204075540427ac4d11b

Set up node-jscs via Grunt (and pass it)

* Set up Grunt via package.json (run `npm install` in mediawiki-core)
* Add grunt task for node-jscs (NEW)
  This is a style checker (as opposed to jshint, which is for
  code quality). There are a few small style-related things that
  JSHint can check (camelcase, onevar etc.) but those are being
  deprecated in JSHint v3, people should use more sophisticated
  tools like node-jscs for this instead. As such this commit
  removes moves of those options from our jshint configuration.
  See: http://jshint.com/blog/jshint-3-plans/
* Add grunt task for jshint
  This will use the same jshint configuration as we use on
  Jenkins but makes it easier to run locally from the command
  line by being part of the same `$ grunt test` task list.

Also:
* Changed jshintignore to use "dir/**"" instead of "/dir" or "dir"
  because the latter is not compatible with Grunt for some reason.
  See also https://github.com/gruntjs/grunt-contrib-jshint/issues/126.

Examples of coding style rules that were being violated that we
can now catch in node-jscs:
* Operator "," should stick to preceding expression
* Missing space after "if" keyword
* Multiple line break
* Empty block (in jquery.textSelection and mediawiki.language)

Bug: 54218
Change-Id: Ib9d7eab9f0d5cea5fb33f0b9f82e5554897fdfe0

Localisation updates from https://translatewiki.net.

Change-Id: Icad3f57cf182107ce7e813297f00473e75e4ad6f

Adding function documentation for Skin::footerLink()

Change-Id: Ib38173ed7bdc434274ff17dfd8c85e5ec95bfb29

Followup af4085a: reset $prev interwiki to null in between result sets

Change-Id: Ie2ba7b061e532e0ab749b0519a7e72bc018a1b5a

Merge "Automatically add a new line at the end of wfLogDBError()"

Allow interwiki searches to return arrays of interwiki results

This allows them to group by interwiki in more sane ways.
Also remove no longer used $terms paramter

Change-Id: I5d420595edfd35a1ce85662d2722ebbe2d357ce3

Improve docs for $wgExtensionCredits (and add 'license-name')

Follows-up 357eb3d488b9d8c which implemented 'license-name'.

Change-Id: I2b2c17965230558fd1650e3db92abafb082737d1

Autoloader incorrect case warnings fixed

Change-Id: I610de60b49840a26ddb7ceeb14264aec77dc8b96

Add getRobotPolicy()

Bug: 57764
Change-Id: I5d53b9b36b8ec5263bf7ebb13c7ef8c102c1ce68

Test against php 5.6 on travis

Change-Id: I7401f1d059437eadd3712c012a93a6a5b6ac4403

Merge "Use correct Creative Commons license abbreviation"

Merge "Removing Wikitravel from the default interwiki list"

Use correct Creative Commons license abbreviation

Change-Id: I7df989de9dca2ee056030ff6f5830d5f0124185e

Tweaked "latest" handling of filebackend stat entries

* Let "latest" stat entries override non "latest" so
  that future getFileStat() calls with the "latest" flag
  can actually have a cache hit.

Change-Id: I1e9391039537d608b89773b4d51575e3b364a751

Merge "[Regression] Delete autoblocks when removing autoblock flag"

Merge "DatabaseMysqlBase: Remove broken check for Percona Server"

Merge "Remove double wfDebug() call in Database.php"

Merge "Add support for specifying headers in FileRepo::quickImport"

Merge "Add Special:ListDuplicatedFiles expensive query special page."

Merge "Add default implementation of getLength to base MediaHandler class"

Add Special:ListDuplicatedFiles expensive query special page.

I saw some comments recently on commons suggesting that
better ways are needed to manage duplicate files (There are tools
for if a specific file is a duplicate, but no backlog of
outstanding duplicate files).

This seems like a fairly easy first step in that direction.

Wasn't sure if this should be an image gallery type
query page, or just a list. I think in this case a plain
list is more useful.

Change-Id: Ibe4b9da71ca6451ec4e6b0050feaf3ca70e1b888

Add support for specifying headers in FileRepo::quickImport

Previously only content-disposition headers were allowed.

This is needed so that TimedMediaHandler can put
X-Content-Duration headers on ogg transcodes.

Bug: 62928
Change-Id: Ic053be63b3edf348da95a88fc494fcf334430265

Add default implementation of getLength to base MediaHandler class

It was being called in File::getLength, but was only implemented
in a subclass, that's not even in core. Any method of a handler
class called from a method in File class should work on any
subclass of MediaHandler.

Change-Id: I94eda90ec3dd64b6ff2e3a5732aa539366cec521

Localisation updates from https://translatewiki.net.

Change-Id: I30f333c40b241dfeefea11e617f3f80c70a84bb1

Merge "Button group adjustments"

[Regression] Delete autoblocks when removing autoblock flag

Since I5403f6e1d7f8f07273cd5da8068b928fdddcdfc9 on reblock the block
will be updated and not deleted/inserted and that update did not trigger
a delete of existing autoblocks.

Bug: 62748
Change-Id: I50384a41d819689944907e653e2495391253831b

Merge "Misc fixes to UploadSourceField"

Update formatting for includes/specials/

Change-Id: I478dedd0b692eae002a0ca9b0ec15cb2c357411f