Add debug call to check that config passing to the image scalers works

Change-Id: I2d1f7de7985e230752cdc8a9c4accae027e5592f
Mingle: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/600

Merge "Removed incomplete/unused DatabaseType interface"

Merge "hhvm: Detect json-c parser"

Merge "Deprecate MWFunction::newObj() in favor of ObjectFactory"

Removed incomplete/unused DatabaseType interface

Change-Id: I95b64f0a74f0203092464c4900fef2ac97e3ba35

Merge "Make allowing site-wide styles on restricted special pages a config option"

Localisation updates from https://translatewiki.net.

Change-Id: I5ac5f394a10a1e5c560435522128209591864471

hhvm: Detect json-c parser

Add detection for a json-c powered json_decode() implementation. Default
loose parsing behavior was previously attributed to HHVM, but additional
investigation has found that this is actually a "feature" of json-c.
HHVM optionally links with json-c for distributions that wish to avoid
being encumbered by the "shall be used for Good, not Evil" clause of the
JSON.org parser's license. The test case implementation stays the same,
but a feature flag style check is added to determine if the parser being
used is lenient or not.

Bug: 72778
Change-Id: Ieb8dddd2e324c90f45f2fc27ac13ab5fff733302

Merge "Minor code comment tweaks for spelling and consistency"

Localisation updates from https://translatewiki.net.

Change-Id: I6a2b7b0f75a1f4800566f43db77de6a0bb55e9ea

Merge "mediawiki.ui: checkbox: Fix gap between bevel and border"

Minor code comment tweaks for spelling and consistency

Change-Id: I51391f45d0f81e4245ccc0e435a71ccd5b0e3ca3

Make allowing site-wide styles on restricted special pages a config option

This mostly reverts commit 614d7e5c274d927f99bfc52ac3a1e6c7e5902408.

Many wikis use MediaWiki:Common.css and associated pages to create a
custom "theme" for their wiki, which would no longer load on login
or preference pages, creating an inconsistent UI.

This re-adds the difference in module origin for different types
(styles, scripts, etc.), and now OutputPage::disallowUserJs()
checks the value of the "AllowSiteCSSOnRestrictedPages" config setting
to determine whether to allow site-wide CSS styles or not.

By default this feature is disabled to be secure by default.

Bug: 71621
Change-Id: I1bf4dd1845b6952c3985e179fbea48181ffb8907

Localisation updates from https://translatewiki.net.

Change-Id: I76ed020202fc84c9e4a4a64bbff1cd7497eeefe4

Fix phpunit test tags

The "@group Database" is only effective when it is in the
comment immediately before the class definition.

Merge the two comments into one.

Bug: 69612
Change-Id: Icfc7c9587534115154d90e20d510c854359fa8f0

Merge "Add WebResponse::getHeader()"

Add WebResponse::getHeader()

Equivalent to FauxResponse::getHeader()
Also change case of FauxResponse::getHeader.

Change-Id: I569b2ebbcd166f5d0a5a5f2dfa913a6aa49e13f4

Enable Travis CI for PostgreSQL

Setting up Jenkins tests for PostgreSQL is not a priority for WMF, so
this change enables Travis CI tests run against a PostgreSQL database.

At the moment, those tests will fail because various fixes for
PostgreSQL issues are still pending review.  However carefully testing
individual commits (cf. for example
https://travis-ci.org/scfc/mediawiki-core/builds/40319020) shows that
the test setup itself is working as expected.

Travis CI's hhvm does not support PostgreSQL, so for now this change
only tests PostgreSQL with PHP 5.3.

Also the range of branches that tests are run against is widened to
include travis-ci/*.  This allows developers to enable Travis CI in
their GitHub fork of wikimedia/mediawiki and then push to such a
branch without having to mess up the master or wmf/* branches with
their defined meaning.

Bug: 37602
Change-Id: I9c2efa3fbc589d2d4198120ca80501124b1b16b9

Merge "Schema is not PostgreSQL connection parameter"

Remove unused global in wfLogProfilingData

Change-Id: I873efb8967cd17c89de5b32fcde6c738eefa03e6

Remove unused globals in the web api

Change-Id: I9ae428ed9f14f95c649953f62d729a9a9be7869b

Let users type their username for 1 s during registration before barking

250 ms are not enough to finish typing your username and the warnings on
the username not being available are fully relevant only when you're done
typing, while being actively harmful if you're still typing an actually
valid name.

We should wait as much as possible, as long as most users are given the
warning before submitting the form.

Bug: 71744
Change-Id: I646d76d2b29b8049c4464f156320d32ea4bd207a

Allow the empty string as the wiki DB

* This is useful when cluster= is set but the DB is not there yet

Change-Id: I1766c564448502c7359c7da964f0becd2e3222c6

Merge "Update OOjs UI to v0.1.0-pre (12b66051ff)"

Merge "Add two hooks to allow for extensions to expose log_search values in the UI"

Remove Brad from Patch Contributors as already in Developer list

Change-Id: I572336a72ff846e2412dd85a95f0e1a290257d22

Merge "Add Brad Jorsch to core authors list"

Merge "1.24 -> 1.25"

Deprecate MWFunction::newObj() in favor of ObjectFactory

Change-Id: Iaa803311409cf7b649f64f69bafe2935a418d31c

Localisation updates from https://translatewiki.net.

Change-Id: I251434c76c10f0ee0ac4012e9c7c41a140d44c3b

1.24 -> 1.25

Change-Id: I7c970e44ad8086dbc2362f55548b16cfb32d3c2f

Fix typo in CREDITS

Change-Id: I32979bb02217c06bca6713ecae8495a77eccd677

Add Brad Jorsch to core authors list

Change-Id: Iba0dcdc5ba23cb0d8665db440889d77b4cfde56a

Merge "Profiler: Move debug(Group)() to ProfilerStandard, not used elsewhere"

Profiler: Move debug(Group)() to ProfilerStandard, not used elsewhere

Change-Id: I6066d0e2a80d04808f772396f578521857d6c550

Profiling: remove "m" prefixes from variables since they're pointless

Change-Id: I88cdbac935d6d908fa3745634922a0bb70c0dbc8

Update OOjs UI to v0.1.0-pre (12b66051ff)

New changes:
0bcb0a1 Style mediawiki theme checkbox
227aefa paralell -> parallel
51ee44c Remove nonfunctional $icon/$indicator code from OptionWidget
735ba0e Use Object.prototype.hasOwnProperty where appropriate
324b8cb Localisation updates from https://translatewiki.net.

Change-Id: I2919d80b9ea5231f45b1e14ae1393a0f7665ec0c

Merge "Make sure that the temp zone has an .htaccess file"

Merge "Update OOjs to v1.1.2"

Make sure that the temp zone has an .htaccess file

Change-Id: I357c6f6a60b4eb531636132180a717f4186b1797

Merge "Don't try to verify XML well-formedness for partial SVG uploads"

Update OOjs to v1.1.2

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fcore.git/v1.1.2/History.md

Change-Id: If0634147a2a01e10c66acc3c8f335d3d15cf7180

Merge "Changed TransactionProfiler to only work via the DB classes"

Changed TransactionProfiler to only work via the DB classes

* This makes it work with xhprof/xdebug, though it will only examine DB queries.

Change-Id: I999b66e6c3af88dda408b2252c248e085af9eea4

Move Cookie(Jar) to libs/

This doesn't rely on any other parts of MediaWiki since we can
replace MWException with InvalidArgumentException here.

Change-Id: Id4512b184d24efc6674d486a5d452aa44dc871c6

Merge "Allow mediawiki.cookie module to be used on mobile"

Move ArrayUtils into libs/ as there is nothing tying it to MediaWiki

Change-Id: I6910c85279858895f5462324134bade580a0cae7

Allow mediawiki.cookie module to be used on mobile

Change-Id: If9cce75738e11a552eee620bdab8b2e2c5938392

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

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

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

Bug: 72939
Change-Id: I9b4a734753af2c9dd7256eec60781553bc76edbf

Fix wrong closing tag in Special:Version

Bug: 73081
Change-Id: I808dce238f6eab4c4202f366b0b8d3752087002c

Remove deprecated functions from MWFunction

* MWFunction::call()
* MWFunction::callArray()

Change-Id: I9b3acd59bc85d54298711e596397b61e4deeff75

Use "callable" instead of "array" in doc block for ArrayUtils::findLowerBound()

Change-Id: I1207963c53941a8f94350aaced414c8e601fb4e4

Merge "Use CSSJanus via composer"

Resources.php: Add <ooui-dialog-process-continue> message, sort list

Change-Id: I6ebc20dd4eab8ba66728e4d5ae37cdf7fee7772f

Use CSSJanus via composer

Requires I7de49f443 in mediawiki/vendor

Change-Id: I94837938f0e0f3daa4d1f53c390457f843000605

Localisation updates from https://translatewiki.net.

Change-Id: Ibddfa10940494144d82487caa2b4fb61815f5e18

mediawiki.ui: checkbox: Fix gap between bevel and border

Use an inset box-shadow instead of adding another pseudo
element. This removes the gap and also keeps the look
consistent on different zoom levels.

Change-Id: Ibf75a09ef9955a5d1cb9d80bff2cce001b60d7c6

Make rubocop ignore remote repositories

The introduction of rubocop (Commit 70f3919) missed blacklisting files
which are not part of mediawiki/core.  Developers running rubocop
against their working copy would end up with errors reported against
code which is out of the mediawiki/core repository.

Instruct rubocop to ignore some well known paths:

Place holders for additional repositories:
- extensions
- skins
- vendor

And tests/frontend/node_modules which is populated via npm install.

Change-Id: I1abdaa3ef110d981e3c017667f87e698016d6805

Merge "Add an "ArticleDeleteAfterSuccess" hook"

Merge "Cleanup mw ui checkbox margins"

API: Make uselang=user the default

After consultation, it is thought that the reduced cacheability won't be
an issue.

This also adds "uselang=content" as an option since it's no longer the
default.

Change-Id: Idc24bfc8bff7392c2047a6b856760ad26a4e9800

Cleanup mw ui checkbox margins

The margin right should apply to the checkbox itself to create a
gap between the pseudo element checkbox and the label.

The existing setup causes rendering issues on io8 and is unnecessary
here. The purpose is to create a space for the checkbox. margin-right
doesn't make any difference to an absolutely positioned element and applying
margins to the label interferes with the pseudo elements themselves.

Bug: 72727
Change-Id: I72af4a53e22b1adb3a70af85f5453a1bc9830eb9

Update OOjs UI to v0.1.0-pre (b38d485723)

New changes:
5a88296 Further mute keyboard shortcuts when disabled
0cdba98 Allow errors to be warnings, and give continue rather than retry button
31a1cbb Normalize configuration initialization
42ce966 Populate LookupInputWidget on mousedown too
3ed0d9c Removing text-shadow from readonly/disabled text in TextInputWidget
68eb012 Add missing message docs for ooui-dialog-process-continue

Change-Id: Ie55491851f7509295fdddbc5229bab2a32e83e8e

Merge "Remove Profiler::setInstance()"

Remove Profiler::setInstance()

This is a weird construction that only existed for dumpHTML.
DumpHTML will no longer require this as of Ia38aa904.

It won't work with future plans to separate profiling & profiling
output

Change-Id: I4d7806596fa233a2b2d1d691c8adfa68724f20f5

Localisation updates from https://translatewiki.net.

Change-Id: Id2c318131a2873993f774034d176b25c7934e366

Merge "Support empty DB names in LoadBalancer more fully"

Merge "Profiler code cleanup"

Merge "wfDebugLog(): remove unused global"

Merge "Use MWLogger logging for wfLogProfilingData"

Profiler code cleanup

- Put Profiler, ProfileSection and TransactionProfiler in their own
  files and rely on Autoloader to use them (maintenance has been
  using the autoloader here for some time--we don't profile the
  autoloader manually)
- This reduces overhead in WebStart/doMaintenance by only loading
  three functions at profiler initialization and defers until the
  first profiling call happens
- Inline callback functions in ProfilerSimpleText rather than having
  public static functions.
- Small comment and code formatting changes in various touched files.

Change-Id: Idf27677c068c50b847152c523a33e7f0c33fdeeb

wfDebugLog(): remove unused global

Change-Id: I336b022c7c87550729ca4e97b0960ba6adec7bb5

Merge "Select column page_lang in prop=info"

Merge "API: Split list=deletedrevs into prop=deletedrevisions and list=alldeletedrevisions"

API: Split list=deletedrevs into prop=deletedrevisions and list=alldeletedrevisions

list=deletedrevs has always been an odd one: it pretends to be a prop
module sometimes in taking titles from ApiPageSet, but when the pageset
supplies no titles it acts like a list module. This causes problems
such as bug 71389, and makes the whole thing unnecessarily confusing.

The solution is to split the "prop" and "list" aspects into separate
modules: prop=deletedrevisions when input should come from ApiPageSet
and list=alldeletedrevisions when not.

At the same time, let's take advantage of the situation to clear up some
other bugs. And let's share the revision-formatting code with
ApiQueryRevisions instead of partially reimplementing it.

Bug: 23489
Bug: 27193
Bug: 44190
Bug: 71396
Bug: 71389
Change-Id: I3e960d5c655bc57885d6d4ee227e67104808add7

WikiPage::selectFields(): Do not include page_counter

Follows-up 90d90dad6e4d. Also removed a mention of "counter" in
a doc comment.

Change-Id: Id6b28e699de010d08e1f30fb5dd379ddc14d1489

Merge "Correctly parse <indicator/> contents, Parser rejiggering"

Merge "mw.Title: Add new static methods `newFromFileName`, `newFromUserInput`"

Select column page_lang in prop=info

The new field needs to be part of the select to init the title object
correct.
Only needed when $wgPageLanguageUseDB = true

Change-Id: I30e8bb1dd493148cba9fec214718f80d04b7638b

Profiler: Call getContentType() only once in logData()

Each call potentially does a regular expression check per output
header. No point in multiplying that by 3.

Change-Id: I30e9a6b5df1a773e9448ec9b77d0e79339f19738

Make load.php parse in PHP 4 again

Not sure what for, but if it "needs" to parse, then let's make it parse…

Change-Id: I28ec3b3a86f6eb90e3341f967649fb4c910966d7

UserGetLanguageObject hook should accept any IContextSource

Bug: 72951
Change-Id: I7c385a2cac0696493dd1c97268e5c4d03829234a

Merge "Prepare repository for running RuboCop after every push to Gerrit"

Merge "mediawiki.ui: Remove dead code from checkbox"

Correctly parse <indicator/> contents, Parser rejiggering

includes/parser/Parser.php
  * Pull out a chunk of code we need to reuse from parse() to
    internalParseHalfParsed(). This is a fully backwards-compatible
    change.

    Code changes:
    * Add a guard for running ParserBeforeTidy and ParserAfterTidy
      hooks, as extensions might not expect them to be called for
      snippets, only full page content.
    * Change $options to $this->mOptions.

    The bulk of parsing work is now done in internalParse() and
    internalParseHalfParsed(), parse() only handles four things:
    * Resetting parser state when a parse starts/finishes
    * Page title language conversion
    * Outputting limit report and limitation warnings
    * Running ParserAfterParse hook (dunno why, but it's documented)

  * Expand documentation for recursiveTagParse(), with some uppercase
    warnings so that no one does the stupid thing I did ever again.

  * Add new public method recursiveTagParseFully(), which is a
    recursive parser entry point that produces fully parsed HTML ready
    for inclusion in HTML output. Compared to Parser::parse(), it
    doesn't produce limit reports and doesn't run the ParserAfterParse
    hook.

includes/parser/CoreTagHooks.php
  * Use the new recursiveTagParseFully() method.
  * Use Parser::stripOuterParagraph() to remove silly tags.

Bug: 72887
Change-Id: I89ae9a50b82245f9a9e4a903563aeb1c51b6103e

mediawiki.ui: Remove dead code from checkbox

This element has 'position: absolute;', which means that 'display' is
explicitly ignored and 'margin-right' doesn't do anything.

Change-Id: Id71193a522f023f30120da6742c2d05397e76fdf

Merge "jquery.getAttrs: Replace deprecated nodeName/nodeValue with Attr.name/value"

Merge "jquery.getAttrs: Remove compat code for IE6/IE7"

Merge "mw.ui:checkbox: Fix indentation and add zero before decimal"

Merge "Clear DeferredUpdates state via setUp()"

mw.ui:checkbox: Fix indentation and add zero before decimal

'margin: 0 .4em;' can easily be misread as '0.4em'

Change-Id: I8cc8ad76ed9da9907809ac6c39b614ce5b0451b7

Merge "Fix fatal error in I0349cfe0e322"

Fix fatal error in I0349cfe0e322

Change-Id: I4ac41d56c2e0f98cd8f026c7deac1e8ab69e8d54

Merge "Update article count when pages are moved"

Update article count when pages are moved

If a page is no longer countable (or becomes newly countable) after a move,
we need to update the site statistics (count of content pages).

Bug: 64333
Change-Id: I0349cfe0e32229706e01d654aacf8ccbe5ebabf3

Merge "Split doc for $wgHKDFSecret/$wgHKDFAlgorithm in DefaultSettings.php"

Merge "die() with explanation when \Psr\Log\LoggerInterface is missing"

Support empty DB names in LoadBalancer more fully

* This convention to avoid selecting a DB is already supported for
  new connections but not when an existing foreign wiki connection
  is reused. This avoids giving PHP notices.

Change-Id: Ic6e9ed031bc8ea6d6f519b6998f0414364c087d0

Merge "Always return string on HTMLTagFilter::getInputHTML"