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"

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

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"

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"

Localisation updates from https://translatewiki.net.

Change-Id: Ibc0afaa9fdc8383545aac46f5b886c20a594a697

Move advanced search inputs back into the form

Follow up to Ic6ec29817ab7deadc6bc7125f8b4cd16686baa04

Bug: T78553
Change-Id: I6c8892e7742bac730154a66ab2bbb9c37ef3af9d

Minor clean up of release notes

* Consistently use 'Git' when referring to the system (not 'git').
  https://github.com/git/git/blob/v2.1.3/Documentation/CodingGuidelines#L483-L486

* Remove angle brackets from a few urls. This is pseudo-wikitext.
  And even as plain text, it only seems confusing. Most other
  urls here, and in HISTORY, don't have it either.

* Remove version numbers from the "External libraries" message.
  We generally announce upgrades in the "New features" section.
  Having to keep this section up to date as we upgrade libraries
  seems unnecessary.

Change-Id: I4a8a75193cf6723ae734067f5d91ffa75692b5fd

Merge "Update RELEASE-NOTES for wikimedia/cdb package rename"

Merge "Deprecate wfErrorLog"

Update RELEASE-NOTES for wikimedia/cdb package rename

Change-Id: Ic71087e3b72566771b184bbe31bf766886401ffc

Merge "Create a generic UDP transport class"

Deprecate wfErrorLog

When a logging service other than MWLoggerLegacySpi is used, the
behavior of wfErrorLog is not guaranteed.

Change-Id: I8543bfd556aa752665f7a3daa855d3c2f7fc8956

Merge "Revert "SECURITY: Do not show log action if revdeleted" and fix UI message"

Create a generic UDP transport class

Code moved from MWLoggerLegacyLogger::emit(), which was formerly
in wfErrorLog().

Bug: T74572
Bug: T78599
Change-Id: I9e0e63d41bed6ccb468f3a9f1d52f156acc355a8

Normalize "\r" newlines in preSaveTransform

The behavior of the different preprocessors differs when given \r or
\r\n newlines. We already normalize the latter here, so may as well do
the former here too.

Bug: T78488
Change-Id: Id6390f64a73ea01088729f25d79103388c1fe7e8

Fixing wikitable border formatting in shared.css

Order of parameters was wrong.

Change-Id: I55671add3b35aaa3a78dc7d8c6ec01bae90f0724

Localisation updates from https://translatewiki.net.

Change-Id: I9709428a8db4ec79a2471ff8ec835d186995c82c

Use Context in Article::delete for messages

Also escape a 'word-separator' message while at the line

Change-Id: If6212dfa685378978b61755cc56fe6d88971d625

EditPage: Don't warn if the page already redirects to itself

Follows-up 02660685708747d5.

Also improved the wording of the "selfredirect" message.

Change-Id: I89a4647c03ccc31db4812dedac107869b2721833

Skin: handle invalid titles gracefully

Linker::link throws a notice if given null instead of title. Title
is constructed based on messages, so it can easily be null.

Change-Id: I5451135966cbb3bbd09d2568cec6b936f3b88aa3

MessageCache: Improve GlobalTitleFail log message

Same as 18d952dbcb6c7

Change-Id: Iabb1eb7481026cfdf0610b0125d5b577febcc0f4

API: Don't use an empty LinkBatch in ApiOpenSearch, part 2.

While Ie4609b08 took care of the empty $titles case, we get the same
problem if $titles is non-empty but contains only Special-namespace
titles.

Bug: T78074
Change-Id: Ic6572628e1da9953ef8212e49d4944e94d799f44

Merge "Fix prefix search for special pages"

Fix prefix search for special pages

Prefix search for special pages was returning only the first match.

Change-Id: I5849696de76ca588f7e626d7da319b8bddb3dce9

Followup cf5f641: pass $params by reference again

&$params was changed to $params, but hooks.txt still documents
this parameter as being passed by reference, and VipsScaler expects this.

Giuseppe and I suspect this may be what's causing the HHVM segfaults
we're seeing in beta labs.

Change-Id: Ib018f23bc44c247aefc277a0c75ff8577f309ab4

Merge "Made StashEdit log a bit less verbose"