Merge "Remove convertPlural methods already served by CLDR plural system"

Localisation updates from translatewiki.net.

Change-Id: Ia5773eca5bdbe62ab8d4123bb417c2f71a6aa9f5

Merge "Standardise wording of verbs relating to revision deletion"

Make sure LocalFile::lock() works for brand new uploads

bug: 54736
Change-Id: I10f950939d34ff5917d168ef1d905c86e2cfc075

Vector: Add navigation collapsing feature

* Moved from the Vector extension (removed there in I47950b5375cae38e2).
* Removed bucket testing
* Removed "new" version of collapsible nav
  (the language portal splitting feature) as it didn't seem to be active
  on any wikis. Can be revived later if considered useful still.
* Killed the vector-collapsiblenav preference
** Usage on Wikimedia wikis:
  - 0.008% on en.wikipedia
  - 0.02% on de.wikipedia

Bug: 46512
Change-Id: I87149d7e15931f02ab700164e9e1d3d707b5e6a5

Merge "Added comment about not using ignoreErrors()"

Added comment about not using ignoreErrors()

* Using this makes it easy to forget to restore the old value, which then
  complicates anything that inserts to the DB.
* Also removed mention of the flag from some places.

Change-Id: I838183222518e0d008dee47ec8df6b3f3dc578e9

Merge "Fix comment typos from 5c51cb96eaea76d22b9c48928044a942c4be2feb"

Fix comment typos from 5c51cb96eaea76d22b9c48928044a942c4be2feb

Change-Id: I56f37010bce8a325af65eab62e7add232df108f4

Localisation updates from translatewiki.net.

Change-Id: I901fdf6051c34c1b35cc024b6f8b36dd92f76aaf

Merge "SearchUpdate should read from master."

Merge "Add Vary: X-Forwarded-Proto for https redirects"

jobqueue: improved performance of JobQueueGroup::getQueuesWithJobs()

* Also added a JobQueueGroup::getQueueSizes() function.
  This function is now used by the SiteStats to make is
  meaningful for when queues are not in the DB. This number
  is already exposed via the API.

bug: 45072
bug: 50635
bug: 9518
Change-Id: I75c16ffa14c963e7f8fb7cb390e6cc4cde0a5804

SearchUpdate should read from master.

This should prevent slave lag from being interpreted as a page delete
from the perspective of search.

Bug: 54652

Change-Id: I2b6673a9d2950cb9e88e9191e8d359aa5dd36d06

Move 'main page as default title' further down to unbreak diff urls

The feature that oldid and diff parameters override the title was
not working if wiki's main page is a special page, because that
behaviour was excluded for special pages. Hence moved the main
page as default title code further down, so that we will hit the
no title provided condition for the oldid and diff parameter only
links.

Bug: 54452
Change-Id: I62cc6e2568a4db988fcf113955ba02710e42a1d2

Merge "Localisation updates from translatewiki.net."

Merge "Detect invalid IPs in XFFs and give a more useful error message"

Localisation updates from translatewiki.net.

Change-Id: Ia1719853a82193be525c15112b9fab8ca5404175

Merge "Make thumb_handler.php use MediaHandler::parseParamString"

Add Vary: X-Forwarded-Proto for https redirects

From mark's comments on https://gerrit.wikimedia.org/r/#/c/86268/, we
should be setting Vary: X-Forwarded-Proto when the response varies on
the protocol, which is the case for redirects from http->https.

Change-Id: Ie6dc9da4e804199b99e3a7abbea7775a181801a0

Merge "Maintenance script to check LESS files for validity"

Maintenance script to check LESS files for validity

A reworked version of script from I068686854ad79e2f63a08d81b1af02f373110613

Move leccs instantiation code to ResourceLoader because it doesn't depend on any
particular module's state.

Change-Id: I733b53171dca77f50a30e5bd0bd5f1b456e4c85d

Merge "Redirect to https for signup"

Redirect to https for signup

Redirect to https for most user, since they will be sending their
password with their signup request.

GeoIP prevention of the redirect is still respected.

Bug: 53651

Change-Id: I12646fcd268c78fd446c6dbf2ba201eee3846444

Merge "Move getRotation and getImageArea to the base class"

Merge "Reduce chance for parser cache race conditions"

Merge "Factored out a small filebackend getContentType() function"

Merge "Add way of including all stderr output when executing command"

Merge "Fix comma errors in various js files (new jshint warning)"

Merge "Add a header on thumb.php saying which thumb server on error."

Fix comma errors in various js files (new jshint warning)

We're about to upgrade to JSHint 2.x. Among various improvements
it now properly detects incorrect use of the comma operator.

This helped uncover two mistakes that could solve a bug, though
in this case the difference was purely in the semantics. The
endresult is unchanged.

Incorrect use of the comma operator can lead to assigning the
wrong value to a variable or causing an implied global variable.

> $ jshint --version
> jshint v2.1.11
> $ jshint .
> resources/jquery/jquery.textSelection.js: line 298, col 71, Expected an assignment or function call and instead saw an expression.
>
> tests/qunit/suites/resources/mediawiki/mediawiki.util.test.js: line 99, col 44, Expected an assignment or function call and instead saw an expression.
>
> 2 errors

Change-Id: I6fdae5f66ef69b6526121cd7b1b2fbb0d9df7a62

Merge "Add meta=filerepoinfo for getting file repo info"

Merge "Add explicit GENDER support for 'revertpage-nouser'"

Localisation updates from translatewiki.net.

Change-Id: I3be3f9075e1d3deabe21db9fffd22698f4f250ff

Merge "Update README"

Standardise wording of verbs relating to revision deletion

Per discussion at the bug.

Bug: 19592
Change-Id: I4b92b68476c87d70ecad5a0e5235dc95fa0f5cfd

less: Use new addParsedFile method, replacing embeddedFiles hack

Follows-up b67b9e1, lessphp now has a public method to add to the
list of files for compilation cache.

Change-Id: I62a6c7cdeb34ea742fa7547e3ca10e24ee484b97

Move getRotation and getImageArea to the base class

This way we know we can always safely call getRotation on any image.

In particular, PagedTiffHandler doesn't extend BitmapHandler, and I
want to make VipsScaler work with PagedTiffHandler, and it calls
getRotation.

Change-Id: I63f2be2a6f31398918b8562e15343f8a839d91c1

Add a header on thumb.php saying which thumb server on error.

Only added if $wgShowHostnames is on. Adding the host name as a
header makes it much easier when debugging with command line tools,
since one often throws away the response body. This still keeps
the host name in an html comment in the response body as well,
like it was previously.

Change-Id: I9d15bd5e2abb4e7a8d3fe2a17fc546a1c5f3b70f

Make thumb_handler.php use MediaHandler::parseParamString

Use MediaHandler::parseParamString to parse the parameter string
of a thumbnail. This seems to make much more sense given the
design of the Handler class. Also mark ExtractThumbParameters hook
as deprecated.

This should be reviewed by someone familar with the 404 thumbnailing
infrastructure. I'm unclear if there was a reason for why it wasn't
using MediaHandler::parseParamString in the first place.

Bug: 49629
Change-Id: I43a0e7d6f0e45536e4e24b72140157791f5f027f

Add way of including all stderr output when executing command

This adds an option to wfShellExec (and convenience function
wfShellExecWithStderr), to make sure all stderr is duplicated
to stdout. The previous method of doing this was to include
2>&1 on the command line. However this did not redirect errors
from limit.sh (For example cgroups not set up, or if a command
reached the file size limit set by ulimit).

Not sure if this is the best approach, but it seems to work well,
and compared to most other approaches I considered, actually gets
the ulimit errors redirected too.

Currently some files fail to render with no error whatsoever,
hopefully this patch will make what went wrong more obvious.

Also fix a comment in wfShellExec that was incorrect (trailing \n),
and make the initial value of the return value variable be 200, so
if there's ever a bug in php where its not being set properly, it
would be immediately obvious what is happening.

Bug: 53824
Change-Id: I833aeb3ab9da726ecb97331369ea187daad7e795

Allow descending-order sort of Special:ListUsers

Add an option to Special:ListUsers allowing it to be sorted in descending
order.

Change-Id: Icfff90a18a62f8f0b70e42313fc3285cf7e44c82

Fix grammar of action-patrol message

Now I look at it, this one is just plain ungrammatical.
"Others' edit" is nonsense in English. It's either "another's edit"
(i.e. "another user's edit") or "others' edits".

Change-Id: I659c04592fbc199bbf440ec959d742824535ec49

Merge "Fix wording of action-import/action-importupload messages"

Fix wording of action-import/action-importupload messages

Bug: 54420
Change-Id: Ib85b8ca05bb6721ba2742859567c246750838244

less: Add '@since 1.22' comments

Change-Id: I673705423cf7618e207c0ee47b01761d6c89ce5f

lessphp: Update to upstream 6e8e724fc7

Summary of changes:

- lessphp throws errors when referencing undeclared variables and mixins.
- lessc::addParsedFile is now public.

See <https://github.com/leafo/lessphp/compare/785ad53840...6e8e724fc7>.

Change-Id: Idffe6946f4029659e677f9dfb62566096cc8d5b5

Merge "Capitalized bullet points and updated last sentence to say MediaWiki instead of WikiMedia"

Merge "Deprecate wfDebugDieBacktrace"

Update README

slight revision of introduction

GitHub: https://github.com/wikimedia/mediawiki-core/pull/9
Change-Id: Ibeff01313618346414f3c03a00f77a25fbe04e6d

Capitalized bullet points and updated last sentence to say MediaWiki instead of WikiMedia

Change-Id: I2961f9b07e71f9dcf3e05bce11371562e74c602f

Merge "In wfArrayDiff2_cmp only compare $a and $b as string if they both are strings"

Deprecate wfDebugDieBacktrace

Change-Id: I12336983e29524a450fc61ed9df4d840066be9b6

Merge "Hide filebackend listing exceptions for thumbnail purges"

Merge "Clean up language test cases objection construction"

Add meta=filerepoinfo for getting file repo info

We currently only return the image repository name for imageinfo, which
is not particularly useful, especially since image repositories aren't
accessible via the API. This patch adds meta=filerepoinfo, which will
return a list of repositories and relevant information (like the API URL
for ForeignAPIRepo ones)

Also returns the local repo now, and defaults to "all" in a more useful
fashion.

Change-Id: I1aaed0895d2a0bc224c82e93975ecf6afd8cb6b8

Merge "CSSJanus: Support text-shadow and box-shadow flipping"

In wfArrayDiff2_cmp only compare $a and $b as string if they both are strings

Noticed during bug 54662

Warning: strcmp() expects parameter 2 to be string, array given in
/usr/local/apache/common-local/php-1.22wmf19/includes/GlobalFunctions.php on
line 127

Bug: 54662
Change-Id: Icdd7fa229e49d7e83aa2a7992895f497806448e1

Hide filebackend listing exceptions for thumbnail purges

bug: 54674
Change-Id: I20db9004fc8b5509f7731b1e678efc6f0aed6cdb

Merge "resourceloader: Never defer execution for top loading modules"

Merge "Title: Add byte class to unicode class conversion for js"

Merge "Remove duplicate file extensions from output messages"

resourceloader: Never defer execution for top loading modules

There was a bug where if a module was in the top queue but had CSS,
its execution would be deferred until its CSS was ready. This would
cause the module's execution to occur after document ready and after
user scripts had run.

This is bad because top-loaded modules are supposed to be guaranteed
to run before document ready.

Fix this by skipping the wait-for-CSS logic when loading synchronously.
Top-loading modules' execution is now guaranteed to block loading the
rest of the page, but their CSS isn't guaranteed to be ready at this
point. Which is fine, because they shouldn't be measuring things in the
DOM before document ready anyway.

Change-Id: I5779cdef336be184741ad97ea84113eb7790dd39

Remove duplicate file extensions from output messages

If a file type was added to $wgFileExtensions by both local configuration
and defaults in an extension (eg TimedMediaHandler and LocalSettings.php
both adding 'ogg' and 'ogv') it was being listed twice in the UI messages
listing acceptable types.

Runs array_unique() over the array on various outputs.

Bug: 54378
Change-Id: I14cd098d8b27099f8f803630535f33549740295c

Merge "Added new MWTimestamp::getRelativeTimestamp for pure relative."

Title: Add byte class to unicode class conversion for js

The upcoming rewrite of mw.Title needs to use wgLegalTitleChars,
but for that to work, it needs to be converted into something
that can work in javascript.

Signed-off-by: Timo Tijhof <krinklemail@gmail.com>
Signed-off-by: David Chan <david@sheetmusic.org.uk>
Change-Id: I163f3d7e3a680d52640a93f4bd195d8209669918

index.php: Remove copyright line and use HTTPS url for mediawiki.org

None of the other entry points (or any file for that matter)
lists the main MediaWiki copyright holders. This is already in
README and SpecialVersion. Afaik there is no special value for
index.php to have this as well (plus, it has become outdated
and incomplete).

Change-Id: Ic3e1ab5b452002160563750fa1537dfbdd66e312

Merge "OutputPage->getJSVars: Maintain symmetry between JS & PHP vars"

Merge "Remove legacy scripts from .jshintignore"

OutputPage->getJSVars: Maintain symmetry between JS & PHP vars

The function is more difficult to read than it need be by dint of some things
two names. I can't be the only one who finds 'wgCurRevisionId' => $latestRevID
ugly, right? Anyhow, lots of JS code depends on the JS variable names, whereas
the PHP variables exist only in the scope of that method, so it's clear that
it's the PHP names that should be brought in line.

Change-Id: Ibddd5d2fc8d75e0ade18ff3433714d52605f2911

Merge "Clean-up of skins/common/IEFixes.js"

Remove legacy scripts from .jshintignore

- skins/common/config.js only needed a singlequote fix to pass JSHint.
- skins/common/IEFixes.js passes thanks to I1ee94b184.

Change-Id: If23ff6e13ea1e0060e7366346196ee61c4cbb841

Clean-up of skins/common/IEFixes.js

- Wrap script code in an IIFE.
- Create scope-local bindings for global vars.
- Use strict comparisons.
- Use single var statement to declare variables.

Change-Id: I1ee94b184f5bb2355d01fae554c50ef314147298

Support rate limiting backlink purges, including null edits

Change-Id: I86e45b7d0eba7d87f65150b34e190e04d8342b3e

Clean up language test cases objection construction

I have no clue how this ever actually worked under Zend, but it
definitely didn't with HHVM. Now it works on both.

Change-Id: I521dfb74f30306736adda5662598fd036ad9849b

Merge "Move LESS function definitions from DefaultSettings to ResourceLoaderLESSFunctions.php"

Move LESS function definitions from DefaultSettings to ResourceLoaderLESSFunctions.php

Intermittent error in production from top-level lambda functions:
https://bugs.php.net/bug.php?id=52144

Change-Id: I9785a601fd5d57c9d8d84b90ee29eef61a6f2e77

Merge "Use utf8 charset for searchindex MySQL table"

Add explicit GENDER support for 'revertpage-nouser'

Requested in https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Revertpage-nouser/en

Change-Id: I56d6d6186c1b2f36eff8c43574f2382a927d805f

Add explicit plural support for limit report messages

Requested by Purodha Blissenbach in
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Limitreport-postexpandincludesize-value/ksh
and
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Limitreport-templateargumentsize-value/en

Change-Id: I39bea8a45bdd941ebe417b823936aa599185ae03

Merge "Clear legacy restriction field on page restriction update"

Localisation updates from translatewiki.net.

Change-Id: I643a45c8f96b3c530a488c4a5ed2a0bbf6d7aa24

Fix typo in comment block in mediawiki.js

asynchrounous -> asynchronous.

Change-Id: I65cf79c14fdc8359875e7dda3821678e99712cb1

Use utf8 charset for searchindex MySQL table

This prevents the DB error "1283 Column 'si_title' cannot be part of
FULLTEXT index", which happens when the "binary" charset is used.

Note that 2ab7c9ec3b fixed the problem for creation of new DBs, though
not for reuse of existing DBs. It is still necessary to specify the
character set for the table itself.

Bug: 47191
Change-Id: Ib4f30fd2730839af1323295898e6dce47b27068d

Merge "Add wgRevisionId variable to ResourceLoader"

Merge "Rename lessc->embeddedImages to lessc->embeddedFiles"

Merge "Vary on forceHTTPS cookie"

Vary on forceHTTPS cookie

Varnish seems to be returning the cached version of pages for users
after they have logged in over https, but access an http page. This
seems to occure because only the forceHTTPS cookie is sent on the
request, but varnish doesn't vary the cache based on that cookie.

Bug: 54513
Change-Id: Ia97ed80622191669ee5ca37af809d307bbdb61ae

Merge "Replace Pakaran with Punjabi"

Replace Pakaran with Punjabi

These are Punjabi digits. I couldn't find a "Pakaran" language.

This was added in 5d00cad9.

Change-Id: I94649b7e5c3c6c9666accee561b12166552d50cb

Clear legacy restriction field on page restriction update

This causes problems where some pages can't be unprotected
as we still take the legacy field into account. See also
bug 33334.

Change-Id: Ibaec827289b1070ee110635d6c9d7b60e061384b

Merge "ResourceLoader: expose registry as mediaWiki.loader.moduleRegistry"

ResourceLoader: expose registry as mediaWiki.loader.moduleRegistry

This patch exports the ResourceLoader registry object as a public member of
mw.loader. I understand that it is not properly a part of the public API
ResourceLoader exposes to JavaScript code; it is an internal data structure and
ResourceLoader does not expect to have anything modify it except ResourceLoader
itself. At the same time, it provides a trove of extremely valuable data for
debugging and profiling, and it's annoying as hell not to be able to poke and
prod it from the JS console. So my hope is that we do the Python thing of
treating 'private' as a tip and social convention rather than something that is
enforced by the code.

Change-Id: I1c47970aa9fceb0b2ded9008efc926b46fa08f51

Merge "Add Special:CreateAccount to Special:Specialpages"

Factored out a small filebackend getContentType() function

Change-Id: I221483ab4a0ed36ea34c19db1740dfaecf223677

Merge "bug 23839: "Refreshing links table" -> "Refreshing links tables""

bug 23839: "Refreshing links table" -> "Refreshing links tables"

There's not a single links table; there are several tables. :)

Change-Id: Ie2a76da3d047a41d102442be1919796bcd7bf2d2

Merge "Add option to chose what language to fetch file description in."

Added new MWTimestamp::getRelativeTimestamp for pure relative.

Since MWTimestamp::getHumanTimestamp just makes a pretty
timestamp, and not a purely relative one, this commit adds
MWTimestamp::getRelativeTimestamp, which does just that.

Change-Id: I8b3e3fc6eeb4afd58f85fa7d05b5ea1a51b0afb6

Add wgRevisionId variable to ResourceLoader

mw.config provides wgCurRevisionId, which has the latest
revision number. This adds wgRevisionId, which indicates
the revision number currently being viewed. This is useful
when looking at old revisions.

Bug: 51594
Change-Id: Ie0d00f3a9a8af8533ab28204b5bb483a0092c710