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"

Merge "Add better error message for files which exceeds $wgMaxImageArea"

Merge "Explaining why user groups are sometimes returned by getRestrictions"

Add better error message for files which exceeds $wgMaxImageArea

Added a TransformTooBigImageAreaError to allow setting an extra message.
Added also size-*pixel messages to show the value of $wgMaxImageArea
with some formatting.

This error is still throwing for all files, to fix bug T34387 this needs
a follow up with a proper check. I am not sure, if a File::isLocal() is
okay, because files from a DBForeignRepo maybe transformed on the same
server, so the check needs to be done also for this. For APIForeignRepo
the check is done on the foreign server.

Change-Id: Ieba12e424c8bddb1961a30d3f9ea5c8ff241abb5

Merge "SpecialBlockList: Remove unused message 'infiniteblock' in formatValue()"

SpecialBlockList: Remove unused message 'infiniteblock' in formatValue()

Change-Id: I24bf357386366e1bbe3ec1e5463b7b6bba9df0e8

Add findVariantLink to StubUserLang

Add the full signature of Language::findVariantLink to StubUserLang so
that pass-by-reference variables are properly passed to the un-stubbed
Language object.

Bug: T78427
Change-Id: I51f57a17c814c43742f514df0e42a1760c406c48

Localisation updates from https://translatewiki.net.

Change-Id: I6a7a02bb4f5a4f6be8d5af9a8bba62cd0e5d20e4

Escape parentheses message on watchlist pages

Change-Id: Ie9943d7da81d16fcd95558f681af68ef9d392bfe

Explaining why user groups are sometimes returned by getRestrictions

Title::getRestrictions is assumed to return user rights, but in most
real contexts it actually returns user groups. This is super
confusing and has caused serious permission bugs. Adding an
explanation to the function comments.

Change-Id: I58013ba5e553d1feecacf6bfef066e1569fcc236

Correct documentation for $wgResourceModuleSkinStyles

Change-Id: I56195b2193477f8f7ff2b56b241702b0dbd7de0c

Minor spelling comment fix

Change-Id: Ic56f4e73e56e6dca4825c93b0a95f4d9de835fd4

LocalisationCache: Use file_get_contents instead of DOMDocument::load

DOMDocument::load fails to load the plurals data during the import
process.

This is a work-around for https://bugs.php.net/bug.php?id=64938. This
bug only rears its head when using Special:Import, because that is
essentially the only place in MediaWiki where we fiddle with
libxml_disable_entity_loader.

Bug: T58439
Change-Id: Idcb4ab1cef2a7b080543e7cc1cee5464fc476456

Made StashEdit log a bit less verbose

Change-Id: Ib1272862efa5518854533af8bf28df97f37f350e

Localisation updates from https://translatewiki.net.

Change-Id: Ie0f2b699f277133fcb6c6e4005784a7087c65437

Merge "Apply coding conventions for JavaScript"

Merge "Get to the point about howto download Vector :p"

Merge "skins: Update .gitignore"

Localisation updates from https://translatewiki.net.

Change-Id: I5021f4e172b346279df2bed793ab619bf59a519c

Convert more wfRunHooks() --> Hooks::run()

Change-Id: I1b1e3cb6871721fc05dda881a8020c9a5d30a514

Merge "GlobalFunctions: Fix 'Undefined index' notice in wfBacktrace"

Merge "Special pages: Change some wfMessage to $this->msg"

GlobalFunctions: Fix 'Undefined index' notice in wfBacktrace

Notice: Undefined index: line in
/var/www/wiki/includes/GlobalFunctions.php on line 1840

Caused by fc2e6df64ffabe9236b8b5eee43d3d89e235670a, previous versions
used isset here.

Change-Id: Ie9434f09e0905346ace19909fb316d47c75e0440

Special pages: Change some wfMessage to $this->msg

Change-Id: I797a791c0e6738857fbd01bacff088c777e91461

Merge "Removed some unnecessary code in LocalFileDeleteBatch"

Merge "Do not require that a Monolog handler has a formatter"

Merge "resources: Switch OOjs UI from the 'Apex' to the 'MediaWiki' theme"

Merge "Resources: Split oojs-ui module styling into oojs-ui.styles"

Merge "Add dependency upon oojs/ui's PHP port"

Merge "Guard against empty integerPart in Language::commafy"

resources: Switch OOjs UI from the 'Apex' to the 'MediaWiki' theme

Bug: T78054
Change-Id: I712b066a8cd70e1ef81a4e43e24801fe1750b3a4

Resources: Split oojs-ui module styling into oojs-ui.styles

Change-Id: I56e7802c58ccf546c83901382abd8fcbe370fd0a

Add dependency upon oojs/ui's PHP port

Depends upon Ibca93b05 in mediawiki/vendor.

Change-Id: Ibca93b05623377dc645a28ddd337027e53983552

Guard against empty integerPart in Language::commafy

Fixes:
    Notice: Undefined index: 0 in languages/Language.php on line 3348

Change-Id: I1a00f3af95d3b9b0fd3ad3cc690358fe5b90f58f

Merge "Update OOjs UI to v0.5.0"

Merge "Pass config to UsercreateTemplate/UserloginTemplate"

Do not require that a Monolog handler has a formatter

Not all handlers require a formatter.

Change-Id: Ifb31aa278d4e90b7fa3a2b7bf3b20173b8345afd

Update OOjs UI to v0.5.0

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

Change-Id: I8503e846b8d240facb7a579952104285d7f9a7fc

Pass config to UsercreateTemplate/UserloginTemplate

Avoids:
QuickTemplate::__construct was called with no Config instance passed to
it

Change-Id: Ibfa1885dbe5609f1458dc3022dd367eb17b614b1

Merge "Check hasTitle() in RequestContext::setWikiPage()"