Don't allow some E_NOTICE messages to end up in the LocalSettings.php

There is probably a better, more comprehensive way to produce a
LocalSettings.php file so that E_STRICT and E_NOTICE errors don't end
up in the file, or to notify users that there is a problem with the
generated file, at least.   This only attempts to address those issues
reported as a result of c978cee8562f018dab158b41e15266fcf873bf1b.

Bug: 66922
Change-Id: I0fe70446d9fe23aed2a147e20e5ff3851371726d

Fix comment for @param

Change-Id: I3838ba86c3d0d361643cfcef7f6a64950d941427

Merge "Fix capitalization of "MonoBook" in findHooks.php"

Merge "Kill $wgEnableNewpagesUserFilter"

Fix capitalization of "MonoBook" in findHooks.php

Follows-up ec1ccf436751 and 4bbd1c44d90f.

Change-Id: I08e4ff78b2bb2a5582ef612b0248e0c7338436a2

Merge "Separate MonoBook skin from core"

Kill $wgEnableNewpagesUserFilter

A temporary WMF hack from 2008 (r34065). No longer needed.

Bug: 58932
Change-Id: Ia25ddc32a0f08587301461c669499e4b8bdf3adb

Fix some findHooks.php errors

* Correct typo in docs/hooks.txt.
* Add a few directories.

Left unfixed is "Undocumented: Special{$this->getName()}BeforeFormDisplay",
which will be addressed separately.

Change-Id: I4fda8960642c23500bd20e0b89c1d1327456313b

Merge "Removed deprecated WebRequest::escapeAppendQuery()"

Removed deprecated WebRequest::escapeAppendQuery()

This function was deprecated in 1.20 and slated for removal in 1.22;
there were supposed to be warnings given in 1.21 but there does
not seem to be any use of this function otherwise in core.

Change-Id: If70a4e2c1509bf3527d5f8960c14115ce4ab713e

Separate MonoBook skin from core

This makes it behave exactly like a custom skin.

* Renamed directory to reflect skin name.
* Split skin classes to separate PHP files.
* Removed core autoloader entries for skin classes.
* Changed the hack in Setup.php to require_once the skin PHP file, as
  the skin is now registered there.
* Extracted skin-specific localisation messages.
* Extracted skin-specific resources.

Change-Id: Ife9926d12b6baaa84cd2aa9a415f1183415863c8

Separate Vector skin from core

This makes it behave exactly like a custom skin, with the caveat that
it is still hardcoded in several places :(, most notably lots of
skinStyles in Resources.php, the installer and some tests.

* Renamed directory to reflect skin name.
* Split skin classes to separate PHP files.
* Removed core autoloader entries for skin classes.
* Changed the hack in Setup.php to require_once the skin PHP file, as
  the skin is now registered there.
* Extracted skin-specific localisation messages.
* Extracted skin-specific resources. Did not touch skinStyles yet.
* Hacked up the installer not to fall over entirely if Vector is
  missing.
* Adjusted hardcoded paths in some more places...

Change-Id: Idfffc1430790b3a104cc9835a6367137bcbf0e4e

Merge "PrefixSearch (ApiOpenSearch) now supports searching in multiple namespaces"

PrefixSearch (ApiOpenSearch) now supports searching in multiple namespaces

I thought there was just an issue with capitalization, but in fact the code
explicitly only searched one namespace anyway. Fixed that while taking
capitalization differences in namespaces into account.

Bug: 65752
Bug: 30323
Change-Id: I3487bb69eae1867832e0ec9ece497538eca32a2a

Merge "Removed getLang() from IContextSource and subclasses"

Merge "SearchResultSet: remove hasResults(), unused"

Merge "Article: Add RL modules for the ParserOutput when showing CSS/JS page"

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

Article: Add RL modules for the ParserOutput when showing CSS/JS page

It previously added only the page HTML, which meant that ResourceLoader
modules associated with the page and other metadata was lost. This
affected the new way of doing things in SyntaxHighlight (Idf4ad439).

We still do not show categories, language links and other things
on the pages (although internally they behave just like in wikitext)

Also needs a fix in DifferenceEngine because our code sucks so much.

Bug: 64859
Change-Id: I5867985693b27ccc9195fddde8b922e67463e836

Localisation updates from https://translatewiki.net.

Change-Id: Ie8652e0ef6486e4942929361e105f79554ed49a6

OutputPage: Add addParserOutputContent() for more fine-grained control

We previously had addParserOutput(), which added everything and did
some other magic, and addParserOutputNoText() which, as the same says,
added everything but the text.

I renamed addParserOutputNoText() to addParserOutputMetadata() and
created two more functions:

* addParserOutputText(): This is almost identical to adding the raw
  HTML, but calls the OutputPageBeforeHTML hook like other
  addParserOutput*() methods.

* addParserOutputContent(): Like addParserOutputText(), but also adds
  the ResourceLoader modules and variables associated with the parser
  output. This is important especially for some extensions like
  TemplateData or SyntaxHighlight which add styles to the page to
  enhance the display.

Change-Id: Iead541886fd1ccdbdf1cb06af71b34cd04644985

SkinTemplate: Move $stylename to Skin and soft-deprecate

It is *used* by Skin, so let's define and document it there as well.

Also, it was never necessary to set this to anything, but we were
pretending it was. This only served to increase the amount of magic
skin creators have to accept.

Change-Id: I61c6e952ede72bad37db38fefe1299da798914b8

Removed getLang() from IContextSource and subclasses

Change-Id: I94c3df814c06c54c071f371101882dcc71c99417

Merge "Some improvements to Special:MergeHistory"

Remove SearchResultTooMany

Only worked for Postgres, and only worked halfway at that. Result
sets could be false for many reasons. No results, bad query, database
went to Mars. We shouldn't assume they're always "too many results."

Leave it up to the normal query error logging and move on.

Change-Id: Ieddd163e440ae54b152541d727c1afdbc4ab4fbd

Set MYSQLI_OPT_CONNECT_TIMEOUT in mysqli

* There is no php.ini setting for this, and it is unclear how
  high the default is. The retry behavior of MediaWiki also
  makes little sense without knowing this value.
* Also removed the retry loop which will mostly result in
  giving up the position in some FIFO queue only to come
  back at the end of the line.

Change-Id: Id24f6635508ba87288a2e40f71d4e8016a1805b0

Merge "Allow interlanguage link prefixes that are not language codes"

Merge "Update documentation of what a "section" is"

SearchResultSet: remove hasResults(), unused

SolrStore, MWSearch and CirrusSearch all implement this and are
now free to drop their implementations.

Change-Id: Ia04bd3fc8526c35c3c9590c7fe4e2db2bc120283

Merge "Put wfDeprecated() on obsolete use of $wgSpecialPages"

Fix copy-paste error in wfIsConfiguredProxy()

It should call IP::isConfiguredProxy(), not IP::isTrustedProxy().

Follows-up e323bc33934d.

Change-Id: I3caf8261f6058aa1a243c055b746b2fa61b1bf49

Allow interlanguage link prefixes that are not language codes

$wgExtraInterlanguageLinkPrefixes holds a list of interwiki prefixes to be
treated as language codes if $wgInterwikiMagic is true.

To set the display text for the interlanguage links generated by this
code, you need to create MediaWiki:Interlanguage-link-foo, where "foo" is
the interwiki prefix.  To provide a friendly site name for the link title
text, use MediaWiki:Interlanguage-link-sitename-foo.  On the WMF cluster,
these messages could be set using the WikimediaMessages extension.

Information about extra language links (in the site language only) is
provided via the API in meta=siteinfo&prop=interwikimap.

Bug: 32189
Change-Id: I3d04760e2d9fb3320bb71e3d5ad115eed54a899c

Merge "Some doc fixes to MediaHandler. No code changes"

Some doc fixes to MediaHandler. No code changes

Change-Id: I2d79fce7aa55b9a0f2988c09a55594db4e8849f1

Merge "Travis CI: Drop 5.3 as a build target"

Merge "Remove use of strencode() in buildLike()"

Merge "ApiExpandTemplates: Start parse before creating frame"

Travis CI: Drop 5.3 as a build target

On second thought, I'd really much rather only target hhvm-nightly for now. On
the one hand, this is monopolizing / hijacking .travis.yml to a specific
Wikimedia project (HHVM deployment), but on the other hand, it's a big enough
project, so I'd like to ask you (dear reviewer) to consider it, as a short-term
arrangement. There are things like IRC notifications, overall build status,
etc. that are scoped to all configured builds and having 5.3 in there adds
noise. It also sows confusion about where we run our tests. People should
really turn to Jenkins for 5.3 testing. I think we're in the final couple of
months of hardcore sprinting on HHVM. I wouldn't mind opening this up after.

Change-Id: I0131667135ef3ca75d01ac5645dbc89bda306d98

Update Travis CI build configuration

* Target 'hhvm-nightly', not 'hhvm'.
* Don't target Zend PHP versions other than 5.3.
  There is value in targetting different versions of the PHP interpreter, but
  in the short term there is more benefit to be had from targetting just HHVM
  and PHP 5.3 (results are produced more quickly and are easier to interpret).
* Don't update submodules manually; Travis does that by default if there is a
  .gitmodules file in the repository root.
  <http://docs.travis-ci.com/user/build-configuration/#Git-Submodules>
* Update Composer before installing PHPUnit.
* Make the wiki name and dbname agree.
* Allow Composer to install whatever is specified in composer.json, rather than
  hard-coding an install of PHPUnit 3.7 (3.7 is incompatible with HHVM).
* Ping freenode#mediawiki-core with the test results, but only when the build
  status changes. We can turn it off if it gets annoying.
* Don't hack include_path or date.timezone ini settings; those were kludges
  needed to get HHVM to run at all at, but they're no longer needed.
* Invoke hhvm via 'php'; Travis ensures that it points to the target
  interpreter.

Verified on my personal fork:
 <https://travis-ci.org/atdt/mediawiki-core/builds/27983653>

Change-Id: I442b40e0d3e31d3cbd963e1219578842f4564004

Merge "Made pool counter better handled nested calls"

Localisation updates from https://translatewiki.net.

Change-Id: I3ee8d9c83834b69ff919de0a9fcfc7d96375fc77

Make RECOVER_ORIG preserve extension tags

Add PPFrame::NO_TAGS, set by PPFrame::RECOVER_ORIG, to preserve extension
tags rather than expanding them.

Bug: 22683
Change-Id: I427333a20d32eb711a7b5d5ac8b780ef89c752a1

Put wfDeprecated() on obsolete use of $wgSpecialPages

Gotta remove it some day, this is another step in that direction.

Change-Id: Id8a559a671e1b9229b9d2235bf459f31d575c154

Merge "filebackend: Avoid using popen() when "parallelize" is disabled"

Merge "mediawiki.page.image.pagination: Clean up, optimise and fix various bugs"

mediawiki.page.image.pagination: Clean up, optimise and fix various bugs

Follows-up 427b0e1, 5e77f39.

* Remove comment stating the obvious (initialize variable).

* Unset 'xhr' variable in the success handler. This allows it to
  be freed from memory, and prevents us from uselessly trying to
  abort an already finished request.

* Rename 'xhr' to 'jqXhr' as it is not an xhr.

* Use the Promise of jQuery.ajax instead of the long-deprecated
  callback option 'success'.

* Rephrase comment stating that jQuery.load is used (which is
  not true).

* Don't re-query 'table.multipageimage' multiple times. Instead
  query it from the document once and cache it.

* Remove useless 'window.history' check. This is not a new global
  in HTML5. The 'history' global, infamous from history.go() and
  history.back(), has been a part of the web for a very long time.
  We can safely depend on them, just as we depend on 'location',
  'navigator' and 'document'.

* Add an identifier to the history pop-state to prevent an
  exception when dealing with the state object of another
  application. Lots of gadgets use the History API these days
  (as well as extensions like VisualEditor and MultimediaViewer).

* Don't store the location.href inside the pop state data. This
  is not needed because it is already stored by the browser
  natively. A history event contains a url, page title and
  custom data stored as JSON. There is no need to store the url
  again in the custom data.

* Despite the width/height transferring from the old content
  to the spinner container, the scroll position still jumped
  because it wasn't accounting for margin, padding and border.
  Use outerHeight() instead of height().

* Avoid using the DOM to store and retrieve information. Use
  application state instead of a DOM query to know whether we
  have a spinner already. This increases performance but also
  avoids false positives where (unlikely) other similar elements
  and/or spinners from a different script are on the page.

* Properly replace the entire table contents() instead of just
  dropping the <tr> straight into the emptied <table>. This way
  the script makes no assumptions about whether there is a
  caption/thead/tbody or multiple rows etc. Plus it saves another
  recursive DOM query (the one for "tr"). And fixes a potential
  bug if there are nested tables (since it was doing a recursive
  query).

Change-Id: I8b64a0860b73a5dcd8051b5e7a1fcb65107228a6

Merge "Delete sr.json, it blocks messages from sr-ec"

Merge "Clarify fileexists-extension message"

Merge "Add macronless namespace aliases for Old English"

Merge "Actually define jMY date format for Arab"

Allow using wildcards for minor type in Special:MIMESearch

e.g. Allow [[Special:MIMESearch/application/*]].

Change-Id: Ic098b10a64e3346d58c94f2dbfbb552dd809e93d

Make CloneDatabase error out on shared tables.

Due to the way prefixes are handled, this doesn't work properly.

Make it quit early to prevent confusion. I don't think there's
many people who need shared table support, so I don't know if
its worth the effort to actually fix.

Bug: 65654
Change-Id: I6cd701998fc9066d8c7a4fc86d93a4282cc7f9bd

Fixes to Special:TrackingCategories

- Message::isDisabled is using the unparsed message text, so this is
always true and not false when a #switch will return '-'.
Using a compare against '-', the same as in
Parser::addTrackingCategory.
- Title::makeTitleSafe can return null, check that
- Add 'trackingcategories-disabled' when array is empty,
this can happen when a '{{' is given, but no categories was found

Change-Id: I88322512db5c4baedc5c571ce9ccbcd0caebb61d

Make Special:TrackingCategories fully detect namespace switching

Special:TrackingCategories has special logic to detect people
using {{NAMESPACE}} in their tracking category messages, and
handle it. Make it detect any '{{' to deal with localized
variants. This will make it work for
[[commons:MediaWiki:Broken-file-category]]

Change-Id: Ibac0decc5992d30ae536efc91138ad9b42a569e9

qunit.completenessTest: Correct walkTheObject behaviour wrt breadcrumb

The function documentation states that currName is not included in
parentPathArray, although it actually is. This change makes the
implementation match the documentated behavior.

Change-Id: I6e44ad84de2e31320309438b8aa09ec929d15e8e

Update OOjs UI to v0.1.0-pre (7a0e222a75)

New changes:
ef2f66f Element: Add unit tests for focusin/focusout hack
54b09fd Element: Remove focusin/focusout hack in onDOMEvent
91ba7d3 Accessibility: Set role=dialog on dialogs
0447150 Accessibility: Restore element focus when a window closes
715f1ef Localisation updates from https://translatewiki.net.
9f343c0 Localisation updates from https://translatewiki.net.
6be64bc Tidy up SVG sources and fix pixel snapping
7a0e222 Fix scrollIntoView() in the case where the scrollable container is the <body>

Change-Id: I9d86a238546acc867f20aeb45b102c7499108c89

Update Language::userAdjust() $ts parameter hint to be string

Will technically still take an int (it's parsed around anyway), but
should be passed as a string

Mostly for HipHop stuffs

Change-Id: Id940028d581e34ca2e37a0730ca5766a08fc10e2

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

filebackend: Avoid using popen() when "parallelize" is disabled

Bug: 66467
Change-Id: Iaf8eb2ecfad166e26e265a98933399dc4bc255a3

Fix $wgDjvuPostProcessor comment about default being pnmtopng

Change-Id: I643332fcfb2dfae40dc61e71f2e151d6510ebbcd

Localisation updates from https://translatewiki.net.

Change-Id: Ia0ebbe849a5458406d8e7c7bee7d930bdba26cfc

Fix DjVu test (Make sure to check for djvutxt binary)

Bug: 61070
Change-Id: Ib5e7c89a51d0872b32bbcd6b7e3db830c2e5e8a6

Ignore days of random failing tests in wfTimestampTest

This switches 2 of the testcases to use a regex
instead of an exact match

Bug: 64946
Change-Id: I24b64b34787eb1b6bbf6e6b6178a2bbf5eb7c32c

ApiExpandTemplates: Start parse before creating frame

The code added in Id12cbe4a is creating a frame from the parser object
before actually starting a parse, meaning the frame ends up with no
title, which causes problems for other stuff that assumes the frame
*does* have a title.

While we could possibly change from calling ->preprocess() to
->recursivePreprocess() to avoid re-starting the parse, there are some
hooks called from ->preprocess() that it's probably best to continue
calling to not break BC.

Bug: 66798
Change-Id: If2ca8ab94ab81ee10f33ce1f4ee75655b7c1e680

Merge "The siteinfo API uses '' as an attribute value, not `true`."

Merge "Removed useless "while" loop"

Merge "Mime search: Trim leading and trailing whitespace from input"

Add macronless namespace aliases for Old English

Requested on Bugzilla 'for the sake of convenience.'

Bug: 58712
Change-Id: I971717a3ff72f643e1912d7fe78416db7bc8ff44

Clarify fileexists-extension message

The message is a warning, the user can chose to ignore it, and
there are valid cases for ignoring it. However the message read
like a hard error.

Change-Id: I47693d1c08784a44b9a1360a8c2d28433b104437

Make ResourceLoaderFileModule#getAllStyleFiles include all skin styles

* Add the ResourceLoaderFileModule#getAllSkinStyleFiles method,
  which returns all of the skinStyles files for a given module
* The LessFileCompilationTest and checkLess.php script, which use
  the #getAllStyleFile method, now validate all LESS style files

Bug: 63343
Change-Id: Ib2eb5761919c648aea4ae58f8d0531799fe7982b

Merge "ResourceLoaderFileModule: Implement remoteSkinPath option"

Merge "Fixed FSFileOpHandle comment"

Merge "Some phpcs-strict changes on includes/revisiondelete/"

Mime search: Trim leading and trailing whitespace from input

Change-Id: I4f8142224d7585bff65ac3cbab4fce8b302c82bd

Merge "Actually format search error Status objects nicely"

Actually format search error Status objects nicely

htmlspecialchars() on wikitext that's passed to addHtml()?

Change-Id: I6cff09ebc052841b923bbed8b9fa02a208e7d61c

The siteinfo API uses '' as an attribute value, not `true`.

Bug: 66723
Change-Id: Ibef772a72bac6b700ba2a3725832f50b057bf503

Merge "shell script fix using shellcheck lint"

ResourceLoaderFileModule: Implement remoteSkinPath option

remoteSkinPath works the same as remoteExtPath but is relative to skins/ instead of extensions/

This will allow skins to register modules using:
 $wgResourceModules['skin.myskin'] = array(
   // ...
   'localBasePath' => __DIR__,
   'remoteSkinPath' => 'myskin',
 );

Instead of using:
 'remoteBasePath' => $GLOBALS['wgStylePath'] . '/myskin', // or
 'remoteBasePath' => &$GLOBALS['wgStylePath'],

Change-Id: I0e8c4a37a224e9528c9c5aa5417f0f56dbb89b97

Allow use of GENDER in "fileexists" message

Requested by Protnet in
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Fileexists/el

Change-Id: I51fdc106938977a5c9abd78de4fd6f297e2ac880

Merge "Reword typo in file header for EditPage.php"

Merge "Capitalise Message in SpecialAllMessages files"

Merge "Rename SpecialAllpages to SpecialAllPages"

Merge "Revert "mediawiki.mixins: Add border-radius mixin""

Removed useless "while" loop

Change-Id: Idc8ebc3a90f10f2179fcea0872207fe17cc2b265

Localisation updates from https://translatewiki.net.

Change-Id: If9f9760aecc14851052ac63f3cd9da15152f500e

Revert "mediawiki.mixins: Add border-radius mixin"

Prefixes for border-radius haven't been needed in a long time. This
degrades gracefully and isn't worth the overhead.

In addition, only a few weeks ago did we merge commits that removed all
this prefixes (e.g. in Ie5ce3202b4). Adding this mixin would encourage
doing the reverse.

This reverts commit 591ba359f283c5ab4ba1b2a0359a98692a3b204b.

Change-Id: Ic25a001de3fe1516cc4939b3159e362e87cb790b

Followup e48ecbc5: fix fatal error due to undefined variable

Bug: 66669
Change-Id: I195a5ff539f09addb92d2bb07d53bdb3aef0db1f

Minor: more specific parsoid image styling with child selectors

Bug: 51245
Bug: 66616
Change-Id: Iff228e932bbb3d548e9a0430a561020cdccf1b4c

Rename SpecialAllpages to SpecialAllPages

Change-Id: Ic137bc7adbc4a9ed96448ba0fee4807b67b3112d

Capitalise Message in SpecialAllMessages files

No internal changes to Special:Allmessage etc made

Change-Id: I21a164af0b6ec123bf654cd1e4e7085b1192f067

Merge "mediawiki.mixins: Add border-radius mixin"

mediawiki.mixins: Add border-radius mixin

Change-Id: Idb46dee0ac8a29d489ec7fb405f8a3149a729b3b

Split messages for recent change legend and letter tooltips

This makes it possible to use formatting in the legend ones
(e.g. links to help pages) and to use different wording
(the one for tooltips is sometimes too verbose for the legend).

Documented the format of $wgRecentChangesFlags better.

The default text in not modified in this change.

Bug: 58715
Bug: 60235
Change-Id: Ie9872c8fb3e43620877300ee8536bb5ddee5ab46

Reword typo in file header for EditPage.php

Change-Id: Iefad919e09632b11bb06abcab390a756455c2ddd

Merge "Add tests for API's assert={user|bot}"

Revert "Revert "Officially deprecate skin autodiscovery""

The CI issue that made the revert necessary is fixed now.

This reverts commit d672aa3123a612583ef9ec67115992aafce5b2af.

Bug: 66429
Change-Id: Ib2b4a895c5d01e5ccc7e4edfdbfdcaf0a93c267c

Merge "Fix some line mis-wrapping and typos in RELEASE-NOTES-1.24"

Merge "Update jquery.jStorage.js"

Merge "Collapse duplicate $existingExpiry == 'infinity' case into else"