Merge "Rewrite collapsibleNav as LESS file"

Merge "Strip excess newlines from formatting test"

Rewrite collapsibleNav as LESS file

* Rewrote css file as a LESS file for better maintainability
* Make use of core embed mixin
** Add .background-image-svg mixin for rendering svgs as background images

Change-Id: I72a66553d6ecee5018d4efaabfd0fa0df6c4135f

Merge "LESS embeddable(): Use lessc::toBool"

Merge "Mouseover explanations for interlanguage links in native language"

Strip excess newlines from formatting test

The result's basically the same either way, only difference is
things like

 "emptyObject": {

 },

Are now

 "emptyObject": {
 },

Results are still pretty, now passes in both Zend and HHVM

Change-Id: I6afa0fd49a7aef3f6b7ca1b4574f3de555521348

mw.Title: Add missing dependency on jquery.byteLength

Follows-up 4894793ab.

Bug: 54977
Change-Id: If884324674198e26636785c0e95e5d61fd5c543e

Merge "Fixed silly unlock bug in LocalFile"

Fixed silly unlock bug in LocalFile

* Removed the lock instead of just purging some cache data

bug: 54978
Change-Id: Ida007c28e07b0fe1803ab8ed027bc2ec4f55af19

Merge "Allow appending text to redirects from moved pages"

Mouseover explanations for interlanguage links in native language

Adds a language name in User's language from CLDR
to an interwiki 'title' attribute. Believed to be
RTL-compilant.

New message added: tooltip-iwiki, added to $preloadedMessages

Bug: 5231
Change-Id: I3dcb00cdc28a10711b6703fa509b8b8d06e325a2

Allow appending text to redirects from moved pages

Add message move-redirect-text, which is appended to redirects created by
page moves (empty by default). This could be used on enwiki to automatically
add {{R from move}} while still allowing non-admins to revert the move, for
example.

Change-Id: Ic2158897ce46177b695a7fb90c8ed3349df18bde

Restrict <abbr> formatting to where a title attribute is present

Reasoning:

* It does not make sense to draw attention to an <abbr> element that
  doesn't have a title attribute because there is nothing special to
  see there.
* It matches Gecko's default stylesheet [1] and the HTML5 style [2].
* It does not negatively impact compatibility, as far as I know
  (IE 6 does not support the [title] selector, but does not support
  the <abbr> element either).
* <acronym> elements are currently not allowed and won't be because
  they are non-conforming in HTML5 (cf. bug 671, comment 37), so this
  part of the rule can be dropped.

[1] http://mxr.mozilla.org/mozilla-central/source/layout/style/html.css
[2] http://www.whatwg.org/specs/web-apps/current-work/multipage/rendering.html#phrasing-content-1

Bug: 54729
Change-Id: I1b00ddb7996911e9b477365e7a84458cefd5da90

Support GENDER for "contribsub2"

Also remove BC code that was added in MediaWiki 1.9. Spotted by Protnet at
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Blocklogentry/el/reply

Change-Id: I37eefd1de95d6ef3429403cd351db315bc3ab2bf

Merge "mw.Title: Rewrite from scratch (porting logic from Title.php)"

LESS embeddable(): Use lessc::toBool

With Ib6bc76736 in place, we have access to lessphp's custom function helpers,
which provide some nice encapsulation for interpreter internals like boolean
nodes. This patch makes embeddable() call $less->toBool() rather than construct
its own AST nodes.

Change-Id: I0a015564dff19f5ced764af182558eb351e3d816

Merge "Update lessphp to 261f1bd28"

Update lessphp to 261f1bd28

From https://github.com/leafo/lessphp/blob/261f1bd28f/lessc.inc.php

Changes, 6e8e724fc7..261f1bd28f:
* 261f1bd Merge pull request #480 from atdt/master
* 0f85200 Miscellaneous small touch-ups
* 5504066 Improve handling of /* comments */ when 'preserveComments' is set
* 667702c Make function helpers public

Bug: 54673
Change-Id: Ib6bc76736d460081dc8f48aee09ca32066dae296

Merge "Added wiki ID to getCoalesceLocationInternal()"

Added wiki ID to getCoalesceLocationInternal()

Change-Id: Ia1ab1883808b669e3c3b442d860c1c562e984564

Merge "Avoid extra HEADs for stash clearing"

Merge "Maintenance: detect missing parent::__construct()"

Merge "Add missing default true value for 'vector-simplesearch' preference"

Localisation updates from translatewiki.net.

Change-Id: I7f8298c727621e51c73f689ea5287da44dea3af4

Avoid extra HEADs for stash clearing

Change-Id: I96e78b219fe12b43daf4a313eb297b98577ef955

Merge "Use HPHP_VERSION constant to detect HipHop"

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 3d6d0723f93e2a838df717b5036e5cd6b0345eca

Change-Id: Iadc98cc4cc33c7725e5fbf4a16df7e1a898aee80

Maintenance: detect missing parent::__construct()

The first time I forgot to include that line in a maintenance script,
I spent a few minutes debugging "Notice: Undefined variable: IP".
Hopefully, this change will save someone else a few minutes of his
or her own time.

Change-Id: Icd26a5dc59e25d2bcc6da9bf0a2f6e95d494e5a6

Merge "Removed more FORCE INDEX clauses for logging"

Use HPHP_VERSION constant to detect HipHop

Follows-up Ic3e769f1fbad4f7ad26dd819406796fee48c6b45.

Change-Id: I0210002c05efd79242b00592639b7be50a79020b

Merge "Allow AbortLogin hook to override error messages"

Remove wfSpecialList()

Has been deprecated for several releases, nothing in Git or SVN
uses it.

Change-Id: I899a0a76a5cb23686caaf6fd244f926a6938fe77

Merge "Show whether tags are active on Special:Tags"

Merge "Correct attribution for Wikimedia project visitors and restore grammar"

ParserTests: allow test of precedence between interwiki links and namespaces

Bug: 51680
Change-Id: I8ad82c596139824fb97d513a62513edf2941b958

Allow AbortLogin hook to override error messages

The AbortLogin hook has a $msg argument that allows for specifying a
MediaWiki message to display to the user. But this message is only used
when the hook is aborted with LoginForm::ABORTED. There's no reason not
to use this same parameter for the other possible failure constants.

Note this is a slight change in behavior for the hook, as previously the
$msg parameter would be coming in as "login-abort-generic" unless an
earlier hook altered it, while now it will be null.

Change-Id: Ib03a1aed9dfb566a027c31c015aa30619174a6c2

Removed more FORCE INDEX clauses for logging

* Same reasons as 7003a7c5e825f391c1a83f8e46cdf332a228d874

bug: 54876
Change-Id: I743bd21e81395c2866b4241730caa8c5990cc73f

Merge "Installer: remove dependency on JSON extension"

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

Add missing default true value for 'vector-simplesearch' preference

It was off and only enabled (unconditionally) by the Vector extension
which is being killed off per bug 45051. The preference was added in
r68177, moved to core in r68224.

Change-Id: Id5e96f5ed9dd532b33bc011104388a39f57277dd

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

mw.Title: Rewrite from scratch (porting logic from Title.php)

Changes:

* Add support for fragments.

* Use wgLegalTitleChars instead of the old clean() method
  that stripped out characters instead of throwing an exception.

* Implemented various other parts of Title.php to make it more
  strict like Title.php. It is still slightly looser, but it
  now takes care of the following that Title.php did already:
  - Directory patterns ("../" etc.)
  - Extra initial colons
  - Titles in NS_TALK that don't round-trip to NS_MAIN
  - 3 or more consecutive tildes
  - Limited title size (255 bytes)

* Extracted parsing logic into a private static #parse method
  and introduced mw.Title.newFromText (a constructor that returns
  null|Title instead of throwing an exception).

* Extended test suite to cover the added features and fixed bugs.

* Since the PHP test suite was lacking these, added them there
  as well.

Bug fixes:

* Fragments are now excluded from the title instead of causing
  the input to be invalid or malformed (e.g. "Foo#bar" was being
  normalised to "Foo_bar").

* ".com" now parses and round-trips properly. The extension and
  rest of title are still separated, but only at the very end
  after all other processing, so though title cannot be empty,
  since we only do a lazy split afterwards, it will split into
  title="", ext="com" internally and join back together when
  needed (bug 38081).

* "Example.js " (trailing space after extension) was previously
  incorrectly parsed as title=Example.js,ext=null.

* "Foo    bar" (multiple consecutive spaces) was transformed
  into 1 space correctly, but "Foo___bar" was not. This has been
  fixed to match the PHP implementation (it merges underscores
  and whitespace of any kind).

Clean up:

* Removed various redundant private helper methods.

* Removed fixNsId as getNsIdByName uses wgNamespaceIds which
  always yields a valid value. The fixNsId was verifying
  something that was already valid.

* Yoda conditional in Title.php, got rid of.

* Use newFromText in jquery.byteLimit.test. It was previously
  using a very basic invalid test (=== '') and no try-catch.
  Since we're getting more strict, typing 'User:' results in
  an invalid title, which should result in the same behaviour
  as it previously did for the lazy === '' check.

Bug: 38081
Change-Id: Ief1c11dabadc2f822065c40be91e04d655933e4f

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

Installer: remove dependency on JSON extension

* Remove one of the two uses of Xml::encodeJsVar(): to encode an
  unused JS variable dbTypes. The remaining use is only after
  installation is complete.
* Add an environmental check to ensure the JSON functions are
  available before attempting installation.

Bug: 54774
Change-Id: Ifb8859026b853ef3f328b796945a973c4add7eeb

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

Correct attribution for Wikimedia project visitors and restore grammar

This change largely reverts commit 5ab2d62a7321c973ddecb6cb2b3ee2aa05c48ccd.
The exception being the change from "regulars" to "community".

Change-Id: I28081364fae5de6ea5bcd07dc135251610e6b7d9

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