Really allow Extensions to make a new gallery format

Adjust GalleryGetModes call to fit Hooks::run() signature.

Follow up: I33462a8b52502ed76aeb163b66e3704c8618ba23

Change-Id: I916dade31d6d67de4bb82582cc71585e83fec1c9

Moved "nowait:" key code to PoolCounter in core

Change-Id: I5286e6c6052289e1107314a04d72703b44a8fbc6

Fix NS invert on SpecialNewPages

Follow up: I5c03dc543b910aab27a26a5a223341be50893cf3

Bug: T85145
Change-Id: Ia5cec6db48c3e900ebb07169015e55a2fb72b792

Don't allow embedded newlines in magic links, but do allow  

This continues the work started in T67278 to make magic link parsing
more consistent with wiki text parsing in general, and closes two
long-standing bugs.

Bug: T30950
Bug: T31025
Change-Id: I71f8b337543163569c64bbfdec154eb9b69d7264

Localisation updates from https://translatewiki.net.

Change-Id: I27b0e2b659c880905f9bc6988d0c41f84616b480

Fix AutoloadGenerator to work on MediaWiki-Vagrant (again)

The patch set from I5d502b5 re-introduced the use of realpath() within
AutoloadGenerator::readFile() as part of a set of changes to ensure that
path separators are normalized across Unix and Windows systems. As noted
previously in I4623b3d, the use of realpath() in this function will
cause fatal exceptions to be thrown when a file such as
LocalSettings.php is a symlink to a file outside to $IP.

This patch separates the path normalization functionality from
realpath() expansion and uses only the path normalization component
within AutoloadGenerator::readFile(). It also introduces a cautionary
comment in AutoloadGenerator::readFile() that will hopefully keep
realpath() from being reintroduced there.

Change-Id: I4923dfa8370a7bd6077f42ff4c437d7293fdad66

Escape message 'word-separator' on Special:FileDuplicateSearch

Change-Id: Ib00cbca264d5723434e5db476293364267a13a85

monolog: MWLoggerMonologSamplingHandler

Introduce the MWLoggerMonologSamplingHandler which can
be used to probabilistically sample the log event stream.

Bug: T85067
Change-Id: Icd14fc8c44ca9eef0f3f5cc4f1d1d8b68d517f07

Merge "Fix redirect page for RTL"

Escape retrievedfrom message in the skin

Change-Id: Ifd696ecd93c76e56cb21b3f2645367beeeeb5ba0

Localisation updates from https://translatewiki.net.

Change-Id: Ic37ef1c92e555d471595a05c2d4b0d7b520d3e0c

Fix redirect page for RTL

Change-Id: Ic101356245247c745daa2f58428de22b08be5289

Merge "Remove unneeded fields from query in Special:ListUsers"

Merge "Ensure integer compare in Special:WantedCategories"

Remove unneeded fields from query in Special:ListUsers

Since r72864 the fields numgroup and singlegroups are unused.

Change-Id: Ifc2f6c77ec54a9c9e72c4a7a4b8a9c8f9f90013a

Merge "Add batch lookup for user groups on Special:ListUsers"

Add batch lookup for user groups on Special:ListUsers

At the moment for each user on the list a new user object is created and
than the getGroups method is called, which fully init the user object,
which is done in 3 queries (user table, user_properties and
user_groups).
Removed the user object with one query for all the userids and cache the
result.

Also added the group page to the LinkBatch, because now the necessary
pages can be determined easily.

Change-Id: I4a945f83ad28edf5cc040139943cf743cb3d133c

Merge "jquery.tipsy: Unbreak the 'live' option for now"

jquery.tipsy: Unbreak the 'live' option for now

It used jQuery.fn.live, which stopped existing when we removed jQuery
Migrate (c393f874). This patch basically reimplements it.

Both .context and .selector properties used here are deprecated and
will be removed in the future, so we should come up with a better
solution at some point (or intentionally disallow using this option,
rather than breaking it by accident). For now, re-added the tracking
and logging, like what jQuery Migrate provided.

https://api.jquery.com/context/
https://api.jquery.com/selector/

Bug: T69989
Change-Id: Ib1a908dc60ca23c354fb05484d9b00cf54a05ebe

Merge "Hide file links in action=info's 'Number of redirects to this page'"

Hide file links in action=info's 'Number of redirects to this page'

Special:WhatLinksHere gets now passed hideimages=1
if the concerned page is in the File namespace.

A bit of history:
the 'hideimages' parameter has been added to Special:WhatLinksHere
with commit 06ad0d25 (r34267), then removed with commit cdc7e22d
(r34277), and finally added back with 725cb284 (r34320).

Change-Id: I0f358e52c38bb525cee8085ca206118ee276889d

Merge "Update jQuery from v1.11.1 to v1.11.2"

Merge "Fix error in docs of VirtualRESTServiceClient and MultiHttpClient"

Merge "EditPage: Optimise loading of mediawiki.toolbar module"

EditPage: Optimise loading of mediawiki.toolbar module

Follows-up b3830611c4.

Unlike getEditToolbar(), which only runs if the user preference
is enabled, the loading of mediawiki.action.edit is unconditional.

As mediawiki.toolbar has already been separated from mediawiki.action.edit,
it's easy to load it conditionally instead of via a dependency
(mediawiki.action.edit doesn't depend on it for anything else).

Also:
* Remove odd 'false' values passed to User::getOption(). These
  options are part of MediaWiki core and always exist. The default
  value 'false' was ignored.
* Remove redundant closure. The domready callback already provides
  a closure and 'mw' is not used here (similar to jquery.mw-jump).

Change-Id: Ib2f4633b328cf8090df43b8d286cfcd77f95c5ea

Reject out-of-range output when converting to TS_MW

TS_MW is a 14-character string "YYYYMMDDHHIISS", and thus cannot
represent timestamps earlier than 00000101000000 or later than
99991231235959.

MWTimestamp should throw an exception if asked to represent out-of-range
times in this format, rather than returning invalid values that are
likely to be truncated by the database.

Bug: T51580
Change-Id: I744e446356f3ed9193dfaaaec5dc81c611dab4a3

Fix error in docs of VirtualRESTServiceClient and MultiHttpClient

err -> error

Change-Id: I758c026599367b282150a513b09d6f6c837a3a80

Update jQuery from v1.11.1 to v1.11.2

Source

* https://code.jquery.com/jquery-1.11.2.js

Release notes

* http://blog.jquery.com/2014/12/18/jquery-1-11-2-and-2-1-3-released-safari-fail-safe-edition/

Change-Id: Ic972e272be9deea6b44b2375fa31ac128e452dac

Localisation updates from https://translatewiki.net.

Change-Id: Id19350a7d65938614fba54f5eefb6a0051027626

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 2711026e89948a99f85182228563fdab8e2dfad3

Change-Id: I7e1e562273d2cdd5a4199971a975c2e674a519e9

Fix VirtualRESTServiceClient::run

runMulti returns an array of response arrays, not request arrays.

Change-Id: I1d57ff24c43c033d533630c23fe3e1e85b448013

mediawiki.notification: Use link syntax for URL in documentation

The url was rendered as plain text instead of a clickable link.
https://doc.wikimedia.org/mediawiki-core/master/js/#!/api/mw.Notification_

Change-Id: Ic7bd0e3765a53f5e4072124333d02417b7e47940

Merge "maintenance: Don't copy the Apex styles when importing OOjs UI"

Merge "Don't create Language objects during ResourceLoader tests"

maintenance: Don't copy the Apex styles when importing OOjs UI

Also remove the existing ones.

Change-Id: I60450f5ce3d8e5b1136f9ffa3557aaf3687ca814

SpecialImport: Don't access $this->getConfig() in the constructor

The context is only set later on, so call it in the main
execute() function.

Bug: T73376
Change-Id: I34229877df6a8960756565f7df1d680aa8951cba

Merge "Use more pretty output in ResourceLoader debug mode for arrays"

Set mw.config wgFileExtensions only on Upload instead of site-wide

It is just used by mediawiki.special.upload

Change-Id: I433e29866fe184ba80c5dda35722e228e79f9307

Merge "Make HTMLForm::formatErrors non-static to can parse message in context"

Use more pretty output in ResourceLoader debug mode for arrays

Effected:
- mw.language.data
- mw.language.names
- mw.config.set
- mw.user.options.set for defaults
- mw.toolbar

Change-Id: I8a9e718ab15f0b3f80e12b817295c6843a570d46

Make the autoload generator use forward slashes on all OSs

It was previously using the platform-specific directory separator, meaning
that we got backslashes on Windows and forward slashes on other OSs.

Bug: T77004
Change-Id: I5d502b54fddd55272e63d4a2a14b6d5de541263a

Change case of class names to match declarations

Found by running tests under a version of PHP patched to report
case mismatches as E_STRICT errors.

User classes:
* MIMEsearchPage
* MostlinkedTemplatesPage
* SpecialBookSources
* UnwatchedpagesPage

Internal classes:
* DOMXPath
* stdClass
* XMLReader

Did not change:
* testautoLoadedcamlCLASS
* testautoloadedserializedclass

Change-Id: Idc8caa82cd6adb7bab44b142af2b02e15f0a89ee

Make HTMLForm::formatErrors non-static to can parse message in context

One call in core already called it non-static

Avoid:
[GlobalTitleFail] MessageCache::parse called by
Message::toString/Message::parseText/MessageCache::parse with no title
set

Change-Id: Ic91e715177c0a4578825640a31ec68ecba3176e0

tables.sql: Improve description of old_flags

* Changed the name of the 'utf8' flag to 'utf-8', as that is what
  Revision stores. (This was already corrected in the mediawiki.org
  manual page by RichF.) Noted that 'utf8' was, however, mistakenly
  used in an old version of recompressTracked.php.
* Added 'external' to the list of flags. This was already added
  to the manual page (by GreenReaper) yet not here. Copied the
  description from there and added a couple of clarifications.

Change-Id: If15b49a28d7d4b0397481d21e30d877298fc7955

Add documentation for SiteListFileCache

Change-Id: I6c5b7fdfbbd6a4e6d67bd0f4aff539ce4d97cfda

autoloader: Remove incorrect documentation comment

Follows-up I8b1bdb84a969. This is not generated by that script,
the autoload.php file is, but that one has a similar comment
in it already.

Change-Id: Icb44f3dab356820f1e3aa5ba56949219af1b5719

Re-emit unknown tags from #tag

When #tag is given a tag that it doesn't recognize, re-emit it as a
regular tag instead of giving an error. This allows for it to be used with
transparent tags and HTML tags.

Change-Id: I0ceee8a4fdaf2d3142054a108f445ff06597c31a

Fixed comment typo and other docs tweaks

Change-Id: I0368561611071f0e4722661e7f7c146618f3364a

Don't create Language objects during ResourceLoader tests

Mock calls to ResourceLoaderContext::getDirection(), which creates
Language objects to get the directionality of a language.

Change-Id: Ibe6da3013e658aa7cf596c1da2f8ca1314b7cdd3

Merge "Don't break autolinks by stripping the final semicolon from an entity."

API: Avoid MySQL filesort with list=allpages&apfilterlanglinks=withlanglinks

I'm not sure whether r44584 didn't go far enough or if MySQL's behavior
has changed since 2008, but MySQL is now filesorting when a
constant-in-WHERE field is included in GROUP BY.

If all our supported databases used the 1999 SQL standard rules for
GROUP BY[1] this would be an easy fix. But PostgreSQL before 9.1 uses
the older 1992 rules.[2] And then there's Oracle and MSSQL, which aren't
listed as supported[1] but are still in the code. Simplest thing to do
is probably to check if we're on MySQL, Sqlite, or Postgres >= 9.1 and
use the 1999 rules, and otherwise use the older rules.

 [1]: Basically "any non-aggregate field in the SELECT must be
      functionally dependent on the grouped-by fields", meaning if you
      include the primary key you're good.
 [2]: Basically "any non-aggregate field in the SELECT must be in the
      GROUP BY".
 [3]: https://www.mediawiki.org/wiki/Manual:Installation_requirements#Database_server

Bug: T78276
Change-Id: I80b515bb06d194b146897155b318a3d1c908e8b6

Merge "Use generalizeSQL for DBPerformance log entries"

Use generalizeSQL for DBPerformance log entries

Change-Id: I6612e01ba6382bcdc1642848009f6b8e13bd79f2

Revert "Simplify MWTidy"

This is broken, for reasons indicated in
<https://gerrit.wikimedia.org/r/#/c/180384/>. It was broken before, but I made
it more broken. So revert for now, and I'll give this another stab.

Change-Id: I7e67a61f7d6370f90487be6470bebe1449432a4c

Set $wgInternalTidy to false in unit tests when running under HHVM

The Tidy extension for HHVM is nominally usable, but does not provide error
text returns. Tests should use TidySupport to determine whether they should use
the Tidy extension or not.

Change-Id: I0f842ddf8484206fc2a8b01289bc0a6750f34fed

Don't break autolinks by stripping the final semicolon from an entity.

Autolinking free external links is clever about making sure that trailing
punctuation isn't included in the link.  But if an HTML entity happens to
terminate the URL, the semicolon from the entity is stripped from the url,
breaking it.

Fix this corner case.  This also unifies autolink parsing with Parsoid.

See: I5ae8435322c78dd1df170d7a3543fff3642759b1
Change-Id: I5482782c25e12283030b0fd2150ac55092f7979b

Localisation updates from https://translatewiki.net.

Change-Id: I23b96b0ea12c96c9b6fa9f2bfc6a24856986e4bd

Merge "Remove double escaping in Special:Block"

Remove double escaping in Special:Block

This function is used in two extensions, which also pass it to
HtmlForm, so no underescaping there either.

Change-Id: I93325051b3a6ef8fc242437a736b5c84bda56b7d

Escape unescaped messages shown on a diff page

Change-Id: I05c07625a2dbb3c5d3ab46d1cfafeaed6a248bba

Merge "Make PHPCS pass on JsonContentTest"

Merge "Use Context in Article::delete for messages"

Merge "Change bugzilla -> phabricator"

Change bugzilla -> phabricator

Bug: T84855
Change-Id: Ic71206f5c0c8b7c6c56a350a136eb3f0ae7bc229

Merge "Skip ApiFormatWddxTest under HHVM"

Merge "Avoid GlobalTitleFail in HTMLFormField::__construct"

Skip ApiFormatWddxTest under HHVM

wddx_serialize_value() fails to escape the ampersand under HHVM. It has
been fixed upstream https://github.com/facebook/hhvm/issues/4283 but
has not been released yet.

When running under HHVM and detecting the ampersand is not escaped, skip
the test..

Bug: T75531
Change-Id: Ia58ec20b4daf78cd90da1bdf8af6cac86015c5d7

xhprof: Guard against division by 0

Warning: Division by zero in
/srv/mediawiki/php-1.25wmf12/includes/profiler/ProfilerXhprof.php on
line 143

Change-Id: Ibb3d0ce836d30663c511809b6e1dece4baa4da92

Make PHPCS pass on JsonContentTest

Change-Id: I1054d7d0e903e3be7cb815035dddb0f34bcd6334

Merge "Run structure tests on extensions"

xhprof: discard section profiler running totals

The '-total' record from the section profiler used for measuring
sub-function timing is not useful in the cumulative xhprof report data.

Change-Id: Ieb79b0e82e0bed54653fab016c133cc74ec4f637

hygiene: small special:search refactor

* removed some unneeded members
** Grepped for usage through repo and extensions
* added isPowerSearch method
* pull search options out of search profile tabs
  and into its own method
* consistent creation of did you mean html

Change-Id: I0da81609e3bc685c971b85a08e228cdda4455c18

xhprof: Fix magnitude of %real measurements from scoped profiler

Change-Id: I06e5b3d82cfddd1a407c56819c9bdd91f160e928

Merge "ImagePage: Use $this->getContext()->msg() instead of wfMessage"

content: Refactor and fix various bugs in JsonContent

Follows-up d2a82fcb60. These issues weren't previously exposed
as nothing uses JsonContent by default in core, and the extensions
using it (e.g. EventLogging) lock it down very early. As are
most of these methods were never really put to use (they were
called after the extension does its superset of checking, or
too early and WikiPage ignores it).

Bug fixes

* Empty JSON object was converted to an array by PST conversion.
  The beautifyJSON method is intended for prettify purposes but
  actually modified the content stored in the database and made
  it no longer roundtrip ({} != []).

  We can't change getJsonData to return an object since it's
  a public method and people use it as an array. So we can't cast
  it to a PHP object as that would break back-compat.

  Turns out the class doesn't even support non-objects anyway (a
  primitive in JSON can trivially cause a fatal as it wasn't
  consistently considered invalid, though it didn't actually fatal
  due to some lucky spaghetti code in WikiPage).

* Fix beautifyJSON by checking for empty objects to prevent
  implicit {} to [] conversion.

* Add isValid() check to fillParserOutput() as it's called early
  on. Otherwise it throws a warning that 'foreach' (in objectTable)
  iterates over null. In practice it doesn't matter since the
  entire parser output is rejected when WikiPage eventually
  checks isValid (through Content::prepareSave).

* Consider all non- (PHP) array values invalid instead of just
  non-null values.

Enhancements

* Display message "Empty object" instead of a completely blank page
  for an empty object.

* Display message "Empty object" or "Empty array" instead of an
  empty table cell.

* Render arrays as a list of values (without indices).

* Remove italics from table cells for values. The monospace font
  should be enough. It also offsets it from the "Empty"
  placeholders (which are italicised).

Refactoring and clean up

* Use FormatJson::parse so that we can use Status to distinguish
  between null parse result and thus reliably cache it.

  Ideally we wouldn't need to cache it, but right now this code
  is pulled apart and called in so many strange ways that we end
  up calling this several times.

* Improve fairly meaningless test (testBeautifyJson) that was
  calling FormatJson in its data provider, exactly what the method
  being tested did. It also provided the test with data that could
  never end up in normal usage (a PHP-style associated array with
  implied numerical indices).

* Document that this class rejects non-array values.

* Document the problem with WikiPage assumming PST can run on any
  content. WikiPage fundamentally still assumes wikitext, in that
  there's no concept of invalid content.

* Fix incorrect documentation for getJsonData's return value
  (It may return null.)

* Fix incorrect documentation for beautifyJSON's return value.
  (It never returned boolean.)

Bug: T76553
Change-Id: Ifed379ba4674a8289b554a95953951886bf2cbfd

ImagePage: Use $this->getContext()->msg() instead of wfMessage

For the GlobalTitleFail entries coming from ImagePage.

Change-Id: I54409b53a4b19f08848e8558877d069f69947393

Merge "Fix some stuttering in comments and documentation"

Avoid GlobalTitleFail in HTMLFormField::__construct

Pass the HTMLForm parent instance in the constructor so context
is available when parsing a message.

Change-Id: I532c0d95698cbcc57294b9bd2725f33838f393a9

Made a new SectionProfileCallback class that extends ScopedCallback

* This is now used by SectionProfiler and avoids the high overhead of call_user_func_array().

Change-Id: I7ff2c9a35c7cd8ee462f2368b655e766ad33dd63

Localisation updates from https://translatewiki.net.

Change-Id: Icf1de242546dc5a459bc4de4ad1dd185a329f1d1

Fix some stuttering in comments and documentation

Change-Id: I9c0088b9aab37335203cad45a1d6fa8ac3f43321

SECURITY: Fix CORS origin matching in the API

Bug: T77028
Change-Id: I68c1ee5b5a048af2aba4e710bc301b09748389bf

thumb.php: Set proper output formats for messages going into HTML

* Use ->parse() instead of ->text() for wikitext messages that were
  being treated as HTML
* Explicitly specify ->parse() if no output format was set
* Document that wfThumbError() takes HTML

Bug: T76686
Change-Id: Id6e7548b2e081cfda7803772ed0395a15feb1f84

Merge "Logging related comment updates"

Run structure tests on extensions

The tests in the structure PHPUnit testsuite are used to verify the
Autoloader, ResourceLoader definitions and some basic other tests.

They are not run by Jenkins for MediaWiki extensions since we invoke:

 phpunit.php --testsuite extensions

Add the structure directory to the 'extensions' testsuite.  That will
start enforcing them on all extensions, so need to be merged with care.

We will want to backport this patch on all release branches we support
and make sure they pass for all extensions/branches :-/

Bug: T78798
Change-Id: Icb88fb3f10d203cfcb61c36c9c92864b92d6739c

Merge "Remove $wgJavaScriptTestConfig"

Use getHtmlCode() instead of getCode() to set the lang attribute

The lang attribute of HTML elements should conform to BCP 47.

Change-Id: Ia8a088add29daea39e90bb277279a6485990d18a

Simplify JsonContent::beautifyJSON()

Change-Id: I406d5c2967615f818c9ac42abc19ab8a49e1da8d

Remove $wgJavaScriptTestConfig

The config variable itself and the documentation property were added
in MediaWiki 1.19 (r107919 / c447423593).

The testswarm-injectjs propert was added in MediaWiki 1.20 (5e590be3d6).

We never actually ended up using TestSwarm, and this variable
is not used anywhere I can see.

Having a configuration variable for a documentation page seems
odd. I can't find another instance of this. As it's tied to development
(not for users of the wiki), link to mediawiki.org direcly.

Change-Id: Ib16607683a293b6d6661ed0411dad9a3ff551a08

Logging related comment updates

* Pretty up some logging related comments.
* Send wfDebugLog messages to logger at info level

Change-Id: I97b8629095e8d6168dbeb791dfb5e7753d281cbe

Update OOjs UI to v0.6.0 in composer.json too

Change-Id: I0cbeaa6421a940b12ccc3a8df42b743537258e84

Merge "Add findVariantLink to StubUserLang"

Update OOjs UI to v0.6.0

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.6.0/History.md

Change-Id: I0386c3f622355de2d5771427fbcaeedd6d3bff04

Merge "Move Test files under same folder structure where class is (/includes/)"

Merge "Destroy session after running api tests"

Merge "Normalize "\r" newlines in preSaveTransform"

Move Test files under same folder structure where class is (/includes/)

Change-Id: I95f1aa6f0ed2cc3306aa6e588a11f359854315c1

Destroy session after running api tests

ApiLogin and ApiCreateAccount calling wfSetupSession, which leaks a
session over the test.
The test RequestContextText needs a clear session to work, so the api
tests should avoid leaking the session.
Doing this in the ApiTestCase because some tests calling ApiLogin over
FauxRequest and that also starts a session.

Change-Id: Icf5cb4d4a2c24c96698cac5bf32147c0c9149ef3

Merge "Localisation updates from https://translatewiki.net."

Merge "Move advanced search inputs back into the form"