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

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()"

Check hasTitle() in RequestContext::setWikiPage()

Calling getTitle() right away can lead to reading from
$wgTitle, so check with hasTitle() first.

This avoids a globaltitlefail in ApiEditPage.

Change-Id: I749cc33dd4f1fd6bc7598be34f37e591d822c3f2

Log to DBPerformance when queries affect many rows

Change-Id: If82b596187b638888760f800bba2fea20fc7ae8a

Merge "Always set a title on images when provided"

Always set a title on images when provided

If an image has a title but isn't linked, set the title on the image
itself rather than not setting it at all.

Bug: T23454
Change-Id: I6302246cf28c84ade805a6a7d4a5e131b6f42e86

Add more context to globaltitlefail logs

Currently stuff like:
 MessageCache::parse called by Message::toString/Message::parseText/MessageCache::parse with no title set.

is showing up in the logs, and is totally useless.

Change-Id: I0ac3ca906c5a67e19974e3434d0e929e6b27fbb2

Merge "Typo fixes and non-code tweaks"

Typo fixes and non-code tweaks

Skipped replacements:
* prefered → preferred
* prolly → probably

Skipped files:
* resources/lib/jquery.ui/jquery.ui.mouse.js
* resources/lib/jquery/jquery.form.js

Change-Id: Ib7923f362ddfca1b892bf5d601785d6b5aa5d44c

Merge "Remove obsolete note from Title->canExist()"

Apply coding conventions for JavaScript

Change-Id: I1ab432ec54e82b05b7c096baa5d4e56dce11aaa9

mw.loader: Guard against Object.prototype keys as module names

This avoids conflicts where we confuse an Object method for a
registered module.

Change-Id: I1b1c2db355f0c698be4a5fe797daa55dedc25258

Merge "Cleaned up LoadBalancer::getConnection"

Fix case insensitive string comparison for http header names

As of RFC 2616 sec 4.2, http header names are case insensitive.

Change-Id: Ib72df2572f5a9c825c5b3ad7735126f3687aad79

Avoid blocking on ChronologyProtecter check in LoadBalancer::getConnection()

Change-Id: Iccf324d87d117972cc6321b2abf1ff101d98fa65

composer.json: Update issues link to point to bugs.mediawiki.org

Change-Id: I67f4403f1797a38bc3f5d7b190d7f84f6fdf457c

Localisation updates from https://translatewiki.net.

Change-Id: I0d198f282e5db225f16d172a25c7375f48b033f4

Merge "Update README from [[mw:README]]"

Fix ProfilerStandard sorting and a profile error

* Sort $this->collated even if collateOnly is set. Also I don't think
  arsort works that way.
* Fix a profiling error in every DB query, which was due to the two
  scoped profiling sections being destroyed in the wrong order.

Change-Id: I6af05f37a5c0391acfa80d54ecbca7a08ad81250

Fix parenthesis placement in multi-line condition

After talking with the folks in #mediawiki-core, I decided that I can
put up with it, under protest, on the basis that it's arguably
consistent with PSR-2.

Change-Id: I5f0c0f8f0172674797970d306efe6439ce1c2b67

Enforce spaces around magic links (RFC, PMID, and ISBN).

Ensure that there is a \b boundary before and after RFC, PMID, and ISBN
links.  (Previously we enforced \b boundaries only before free external
links and after ISBN links.)  Consistency is a good thing!

In addition:
* \b is not a PHP escape sequence, so you don't need to write \\b inside
  a string.
* \b before the numeric part of an ISBN is pointless: by the structure
  of the regexp there will always be a space on the left and a word
  character (a digit) on the right.

Bug: 65278
Change-Id: Ic315b988091a5c7530a8285b9249804db72e55db

Merge "Stash edit when user idles"

Stash edit when user idles

Try to anticipate edit submissions earlier by looking for pauses in keyboard
input, on the assumption that users often take a few moments to look over an
edit before submitting it to the server.

Change-Id: Ib2de3f15b93e5d32c66a03363e98606412d5d53b

Merge "ResourceLoaderImageModule: Allow use on mobile"

ResourceLoaderImageModule: Allow use on mobile

Change-Id: Ic7913aa1a1e1fb8d3742503cbe8a6cb3427fc826

Fix check for function wddx_deserialize in tests for hhvm

The @requires just works for other tests, not for php functions.

This fixed a failure for the hhvm test server:
1) ApiFormatWddxTest::testValidSyntax
Failed asserting that null is of type "array".

Change-Id: Iadd8238ce62d4803cdf56508886ad016f8ac8f1d

Merge "Adding aliases for speacial pages in Hebrew"

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

Localisation updates from https://translatewiki.net.

Change-Id: Id102078400328a8f1032b03e926e56645adf00b2

Fix type hint to match php docs

From http://php.net/manual/en/function.posix-isatty.php

bool posix_isatty ( mixed $fd )

Determines if the file descriptor fd refers to a valid terminal type device.

fd

    The file descriptor, which is expected to be either a file
    resource or an integer. An integer will be assumed to be a file
    descriptor that can be passed directly to the underlying system
    call.

    In almost all cases, you will want to provide a file resource.

Change-Id: Ief19ac1f09e0a955c8f5d5af28b87b8e8d427a2e

Language: wfRunHooks() --> Hooks::run()

Follows up I8ee996f237fd111873a

Change-Id: I0ffc5903c96219676b9bcb46928fd72bb7133927

Replace wfRunHooks calls with direct Hooks::run calls

* This avoids the overhead of an extra function call

Change-Id: I8ee996f237fd111873ab51965bded3d91e61e4dd

Merge "API edit: allow ConfirmEdit to use the merged parse"

Adding aliases for speacial pages in Hebrew

Change-Id: If5e4a535862f40dc7a87527fe525f1fb2f1fe2c1

Merge "Fix case of various newFromID/newFromId functions"

Merge "Xml: Simplify processing of attributes in Xml::label"

Fix case of various newFromID/newFromId functions

User::newFromID -> User::newFromId
Revision::newFromID -> Revision::newFromId
RecentChange::newFromID -> RecentChange::newFromId

WikiPage::newFromId -> WikiPage::newFromID
Title::newFromId -> Title::newFromID

Change-Id: I310f26653613951e9b21ce915a92d205a87937f1

Xml: Simplify processing of attributes in Xml::label

Comment inserted by Antoine Musso with d86a3bc2177f79.

Also fixed wrong indentation.

Change-Id: I054762929413f90090b446c603b60ed9544c06d1

Fixed internalClean class/method existence check for HHVM

* Follows up 4f281083fda91879a77fb87d64d8a9533526bd0c

Change-Id: I5fa406ed1c4f2eefd1c22e9ab90e72655f31d162

hhvm: Check for tidy function instead of class

Bug: T78166
Change-Id: Ie60e23ffbafd698a3458eed1efce92d54c8d0c2a

Avoid preloading unused messages in ContribsPager

* newarticle
* rev-delundel
* rollbacklink

A bit of history:
* messages preloading
** added by Brion Vibber
   commit 935bca8ee764847868ce6c597b43eb74e0908c10

* usage of newarticle
** added by Erik Moeller
   commit d61d3486f0eb7713e58d80f984891cb8882cf7ef
** removed by Alexandre Emsenhuber
   commit 84be101a9098166731fe6a1e30615d2f9b094eeb

* usage of rev-delundel
** added by Aaron Schulz
   commit 0ed239362c54656b95d4d602fcc33d6f5ace8932
** fixed by Raimond Spekking
   commit 349ef821d66a54ee0c6b7ce6024bded399518e95
** removed by Aaron Schulz
   commit 90dcb3c1f2c4b03f2cb9358b40666af1eff88a99

* usage of rollbacklink
** present in Lee Daniel Crocker's 'initial revision'
   commit d82c14fb4fbac288b42ca5918b0a72f33ecb1e69
** removed from old version by Aryeh Gregor
   commit 3a0c45e93cf7cdcfadfd2545a497562968e023ee
** unused but still preloaded in Ilmari Karonen's new version
   commit 2df501fa0d2723bf2a196390c2507a6043cdcdaf

Change-Id: Ib9fce5c1af4f0fd7dd3dcfefa4a6611d61123e02

Fixup 4f0b2f42419c283e8c94a5be6147d54ae7731a6b

It messed up a couple of entries in RELEASE-NOTES-1.24,
now brought into HISTORY.

Change-Id: I84795de70b0cb24af122acd0aa6ebc8367a782d5

Fix documentation of hook 'BeforeParserFetchFileAndTitle'

By placing the notice "If 'broken' is a key in $options
then the file will appear..." added by Aaron Schulz with
commit 9d572d1844380c0db268b456f297bd7675eb609f.
This solves a 'FIXME' added by Siebrand Mazeland with
commit b33c77a52562975aa2503dac66e716e6a5a7ad2e.

Change-Id: I9d100588276faac5d5b2be979d8140389e5ed85a

Merge "API: Allow redirecting a page to itself using action=edit"

Merge "jquery.mwExtension.test: Use better sample data for isDomElement"

Merge "Always use 'bool' instead of 'boolean' after '@param' and '@return'"

Always use 'bool' instead of 'boolean' after '@param' and '@return'

The former is by far the most common.

Skipped:
* resources/lib/jquery.ui/jquery.ui.datepicker.js
* resources/src/mediawiki.special/mediawiki.special.upload.js

Change-Id: I73c93797e745128ba703e4865080c36784caa474

API: Allow redirecting a page to itself using action=edit

Follows-up 02660685708747d5.

Change-Id: I60aa15ac9f33c560c940c55cf98626bb9ff4b822

Cleaned up LoadBalancer::getConnection

* It will also no longer call getReaderIndex( false, ... ) twice
* Removed various related ampersands

Change-Id: Ia79e2007dbf84e7437f9439aa6371333aa3e1b23

Merge "Added some abstract methods to DatabaseBase"

jquery.mwExtension.test: Use better sample data for isDomElement

* getElementById, getElementsByTagName()[0] and jQuery#get(0) all
  return an HTMLElement. Test that only once.
  Avoid using random internal QUnit elements like #qunit-header.
  These don't always exist (e.g. when running tests via Karma).
  If anything, it should use #qunit-fixture. But might as well
  create a simple DIV and test that.

* Use 'null' instead of getElementById('not-found'). We're
  testing isDomElement, not getElementById.

* Add test for TextNode.

* Separate tests for isDomElement and isEmpty.

Change-Id: If9284fbbd066897561ae8e679f5f7ceab9255e18

Simplify MWTidy

* Make the internal MWTidy::*clean() functions always return an array of two
  elements: the output buffer and the error buffer.
* Make MWTidy::externalTidy() always read both stdout and stderr. We can read
  stderr after stdout because tidy.c produces output in the same order.
* Remove the $stderr parameter from the private MWTidy::*clean() methods, since
  error output is always returned.
* Merge MWTidy::phpClean and MWTidy::hhvmClean, since the difference between
  them is now small enough that splitting them up is not warranted.
* On HHVM, MWTidy::internalTidy() always returns an empty string for the error
  buffer.

Change-Id: I178b42d6ebdd1a5b9bd5921eb093a6c5014ffa49

Merge "Add parser tests for tables with pipes and wikilinks"

Added some abstract methods to DatabaseBase

Change-Id: I75781ee55d48d9fda40d600228100708d6524df8

Merge "Allow moment on mobile web"

Merge "Made SectionProfiler cache the ScopedCallback closure to lower overhead"