Merge "Remove useless $out parameter from SkinTemplate::prepareQuickTemplate()"

Merge "Followup I4049b666: "Removing prefix exception for HTMLCheckMatrix""

Merge "mediawiki.notify: Simplify callback by using .then()"

Merge "Use camel case for variable names in Article.php"

Merge "Don't set the session name in CLI or when sessions are deactivated"

Merge "Allow to send the memory usage with UDP profiler."

Enabled $wgUseLinkNamespaceDBFields

Change-Id: I7aa64c3ef50cd0bc210ea755a0c9d6ee1b7abb7c

mediawiki.notify: Simplify callback by using .then()

Change-Id: Ifff25534d716721ff1d2d4484dbd0f63161ecb43

Merge "Pass $name to default Skin constructor for backwards-compatability"

Merge "Added --lastUpdatedId option to backlink namespace population script"

Merge "Run some parser tests with tidy."

Merge "Revert "Made LCStoreDB try to use a separate DB connection""

Merge "includes/actions/: Use Config instead of globals"

Pass $name to default Skin constructor for backwards-compatability

This feature was necessary when it was only possible to specify the
name of the class a skin used, this was the only way for one class to
dynamically serve multiple skins (such as if the skins were purely
template based with no PHP or if one skin had different themes to it).
It also provided an option for other skins to set $this->skinname dynamically.

See code review on I5c442f3c9e.

Change-Id: I7315fadf2e26d164ccc4f47b6d883945fa2570da

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

Merge "SkinFallbackTemplate: Use SkinFactory instead of $wgValidSkinNames"

includes/actions/: Use Config instead of globals

Changed InfoAction::pageCounts to be non-static, so
it's able to access $this to get the Config object.

Also replaced instances of $wgScript with wfScript().

Change-Id: I4a6a3224e762f13640af04a73e2934b887dffedd

Merge "Fix documentation."

Localisation updates from https://translatewiki.net.

Change-Id: Ie88a32c00c27abb72f24c9585884b7253d938b31

Merge "Update release notes and documentation about skins"

SkinFallbackTemplate: Use SkinFactory instead of $wgValidSkinNames

Change-Id: I463d263fc24a3e0c589ddc7176b77dffde17d105

Fixed spacing

- Added newline at end of files
- Added/Removed spaces around comma, parentheses and negation
- Added space after function word

Change-Id: I2dd338153aeb5f07702ba015945e95c7d0ae673b

Merge "Correct variable names in @param to match method declarations"

Merge "Cleanup - let's make IDEs more useful"

Cleanup - let's make IDEs more useful

http://phpdoc.org/docs/latest/references/phpdoc/types.html

If IDEs have many warnings, we don't look at them.
Let's minimize the number of warnings, and make them useful again.

* Some function docs fixes
* Removed unused $iwprefixes var in ApiQuerySearch.php
* declared private $blockStatusByUid in SpecialActiveusers
* declared private $repo in UploadFromChunks

Change-Id: Ifd20f78b168b9a913fdb8d89dc26a76a173b1c29

Update release notes and documentation about skins

skins/README based on extensions/README. Also updated the latter.

Bug: 68402
Bug: 69372
Change-Id: I6d045af0509aa8fc485255b507554d771d101ee0

Correct variable names in @param to match method declarations

Some @param have a typo in the variable name,
some @param's were in wrong order.

Change-Id: Ie25806831027112b398f6f4a909c59147ac3a5fa

Fix documentation.

It's true that right now the internal skin name doesn't have to be all-lowercase,
however not doing so leads to side effects such as the changing of case of i18n message keys,
if the documentation makes programmers aware that something other than all-lowercase can
be used, they should also be aware of the side effects of doing so.

Change-Id: Ib1ed192b1ba83ae864313c34b450a1151485750b

Use /** for doc comments

/* is only a comment, which is not used when build the documentation

Change-Id: Id5f917928e3a253a90fd17b0d7aad7340f9393f5

Merge "SkinFactory: Improve documentation and comments"

Merge "SkinFactory: register skins in Setup.php"

Fixed some docs

- Added since to @deprecated
- Use lowercase array on @param
- Added missing type on @param
- Swapped type and variable on @param
- Fixed spelling of @return

Change-Id: Idbbc6b4c8aadb6d98c636f53024ac513346c912b

Merge "ApiTestCase: Avoid notices about undefined indices"

ApiTestCase: Avoid notices about undefined indices

We're going to throw an exception right away here, but this is still
problematic because of PHPUnit:

"By default, PHPUnit converts PHP errors, warnings, and notices that
are triggered during the execution of a test to an exception."
http://phpunit.de/manual/4.2/en/writing-tests-for-phpunit.html#writing-tests-for-phpunit.errors

Also change the exception message to something saner.

Change-Id: I766c36076d34428035be78bbb3cf36cc572b565b

Docs: compatability -> compatibility

https://en.wiktionary.org/wiki/compatability

Change-Id: I71d82ae04d4b17609a1d5a7eec4497115e64972e

Merge "EditPage::newSectionSummary should return a value in all code paths"

Set $wgUploadNavigationUrl to false on parser and unit tests

Setting this to default avoids failing parser tests,
when it is set in LocalSettings.php

Bug: 54576
Change-Id: I531d5839e9abe571c6c29f290bb159dabca34798

SkinFactory: Improve documentation and comments

* Document parameters to register()
* Document what the "human-readable name" does
* Document the distinction between autodiscovery and $wgValidSkinNames
  skins

Change-Id: Iee5bd18b3e68f3e48ccd28e386109e60fee31085

SkinFactory: register skins in Setup.php

This un-makes $wgValidSkinNames a legacy thing, and is
more backwards-compatible friendly.

Change-Id: I5c442f3c9e4ee7a4a3980fd02138ee756ef9fa7a

Remove deprecated $wgHTCPMulticastAddress, $wgHTCPMulticastRouting and $wgHTCPPort

Change-Id: Icfbc178d823e59f2e305cbc4bca0b194d2786026

Merge "Remove return from Action::checkCanExecute"

Merge "resourceloader: Wrap only=script responses in "if(window.mw)""

resourceloader: Wrap only=script responses in "if(window.mw)"

We currently have a few legacy requests to the load.php end point
that bypass the ResourceLoader client by coding a request to
load.php via a "<script src>" directly. Most prominently the
request for the 'site' wiki module (aka MediaWiki:Common.js).

Remove the manual wrapping of embedded private modules as this
is now taken are of by ResourceLoader::makeModuleResponse itself.

Misc:
* Mark "jquery" and "mediawiki" as Raw modules. While the startup
  module had this already, these didn't. Without this, they'd
  get the conditional wrap – which would be a problem since mediawiki.js
  can't be conditional on 'window.mw' for that file defines that
  namespace itself.
* Strip the cache-key comment in the unit tests because the hash
  no longer matches and using the generic 'wiki' dbname was breaking
  DB queries.
* Relates to bug 63587.
* See also 05d0f6fefdcc959d which expands the reach of the non-JS
  environment to IE6 and helped expose this bug.

Change-Id: Icf6ede09b51ce212aa70ff6be4b341762ec75b4d

Fix phpcs issues in actions and revisiondelete dirs

Change-Id: I54c1e336b8c44ff8720ac38c8b2d68ede42af305

Merge "actions: Move SubmitAction class out of EditAction.php"

Merge "resourceloader: Fix undefined variable in ResourceLoaderFileModule::getSkipFunction"

actions: Move SubmitAction class out of EditAction.php

Straight move, except for the slight change in the comparison operator
for session_id().

Change-Id: I9404f48ee75ff3d3655f2f5b75a05d66f334b80d

resourceloader: Fix undefined variable in ResourceLoaderFileModule::getSkipFunction

Follows-up 75c08916b02ac87. Looking at the call chain of
validateScriptFile() up to JSTokenzier, the first parameter is the file
name, otherwise "[inline]" is used; so that variable should be $localPath.

Bug: 69214
Change-Id: If7f36449cb352f50ba795a6d306e5d949a3dbd29

Merge "Change "ENGINE=HEAP" to "ENGINE=MEMORY""

Merge "Remove duplicate override in SpecialRecentchangeslinked.php"

Remove duplicate override in SpecialRecentchangeslinked.php

Causes bug in FR

Change-Id: I5efa8953e6e6717ba043fa13a652f1dfcf936bf2

Localisation updates from https://translatewiki.net.

Change-Id: I73acb1878dc7e7d2186219f542bf3e8f4ca3d944

Remove useless $out parameter from SkinTemplate::prepareQuickTemplate()

Part II; follow-up I4fac5b14af (ff4da35).

It makes sufficient time the first part was merged, no need to pass the
parameter anymore.

Change-Id: Icb390995912b4860df4ce2571105d3a9dc97a092

Change "ENGINE=HEAP" to "ENGINE=MEMORY"

"Memory" is the preferred term, see
https://dev.mysql.com/doc/refman/5.0/en/memory-storage-engine.html

Change-Id: I1fd05d027903af6023002dd357998ce5f2606708

Remove return from Action::checkCanExecute

No need to return true on success, because failure gives Exception,
so the return value needs no checking.

Change-Id: Id59bfaebc14bd1c638a721c303f585c1de627508

Make travis notifications go to #mediawiki-feed

Change-Id: If61d0ac868c0932a0bb0aae76db4354190874c35

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

This reverts commit 420f55ad976a4929fdc9b1eff9ed2b6044c633d6.

Change-Id: I4c8aa600c72041daf49ed6158b9375bdcc5c9015

Fix visibility in SkinFallback and StubObject

Change-Id: Ie4c1aabd966fcdf87a9129ceb59c058740292f2e

Add @codingStandardsIgnore tags to parser classes

Change-Id: I16d19de3d2b2461a68030afd3a79aa59c9e948d4

Fix phpcs issues in parser

This fixes all issues except for:
 - class names
 - line length

Change-Id: Ie91b010d5b3eec49d3b80b6e93b125a901ef43c6

Fix phpcs stuff in profileinfo.php

Change-Id: Ia0709ced1ac6365a0c1f7c14c52d2487041a9c3f

Localisation updates from https://translatewiki.net.

Change-Id: I13bfc16a2cba749eb74e0657c337eda0f4a3491d

Merge "Cleanup some docs (tests)"

Merge "Remove some unnecessary IE 6 hacks"

Cleanup some docs (tests)

- Swap "$variable type" to "type $variable"
- Fixed spacing inside docs
- Makes beginning of @param/@var/@throws in capital
- Changed some types to match the more common spelling

Change-Id: Ia041964250d8b7c0349d79dc9b131c5b8696e795

Add release notes for Ibe28a69c9fbab00b81c53b1643df722a3f1fbf19

Change-Id: I0a2b2feafcf2561369145bec80ec14c7929beff9

Merge "Do not allow a user to delete a page they can't edit"

Do not allow a user to delete a page they can't edit

This was probably overlooked in the past because usually the only users
who can delete pages also have permission to edit the relevant
protection levels.

Change-Id: Ibe28a69c9fbab00b81c53b1643df722a3f1fbf19

Localisation updates from https://translatewiki.net.

Change-Id: I5529c6d8115cc05b3de8d3fde9de948f1719bed3

Merge "API: Remove action=paraminfo 'props' and 'errors' result properties"

Fix userrights-interwiki when the 'user' table is shared

At present, this function selects the user table from the same DB as the
user_groups, however, in wikifarms this is not often the case, as the user
table is often shared, but the user_groups table not. Adding a check in
here, and passing the shared table, if needed, ONLY to the user query,
should fix.

Change-Id: Ib85630067c402b8e4f50ff11a33fe7f0eadd4f16

Require a ResourceLoader instance is passed to ResourceLoaderContext

Change-Id: Ie1e5453b47349fc00664325827f88a1e7a6de43f

Merge "Make styleguide mobile first"

Fix documentation for Html::closeElement

Follow-up to d950da075dcb6c6dd3f68a50a0d427569350763a

Change-Id: I916052cf924dd3c6664b6d7d02175fd74a71527b

Merge "Skin: Don't double-check for non-existing skin in newFromKey()"

Skin: Don't double-check for non-existing skin in newFromKey()

normalizeKey() should have already handled this.

Change-Id: Idf2510ec86841ed18d589f392a30db1b41ac0c24

Merge "Never display empty sections on Special:Version"

Never display empty sections on Special:Version

Follow-up to d423112b.

Change-Id: I44c8bfe6f4ca1350a7e35a253785f772602353c1

Move Vector's hooks documentation to mediawiki/skins/Vector

Change-Id: Ide133c1ec0ae701c5864edcbab428be314bd159e

Remove some unnecessary IE 6 hacks

We dropped JS support for IE 6. These workarounds for its bugs are no
longer necessary, because this code is no longer loaded. (The CSS
files modified are parts of modules that require JS support.)

We probably have a lot more, these just came up when grepping for
'IE6' and were obvious.

Change-Id: I7588b03f6ade699ddda4ec28be6c6f512e4b0ea8

Merge "Cleanup some docs (languages)"

Merge "Cleanup some docs (maintenance)"

Merge "Introduce SkinFactory"

Introduce SkinFactory

Modeled similar to ConfigFactory, this lets skins
register themselves via a callback, allowing for
proper dependency injection.

Loading via $wgValidSkinNames is still supported,
but considered "legacy", not deprecated though.

Skin::newFromKey is now deprecated (and had only
one caller in an extension, which I'll update
afterwards).

Change-Id: I1960483f87c2ef55c994545239b728fa376f17f4

Cleanup some docs (languages)

- Makes beginning of @param in capital
- Removed return void

Change-Id: Ie05436c1ef886cb23c62ccde95384f253f83694c

Cleanup some docs (maintenance)

- Fixed spacing inside docs
- Makes beginning of @param/@return in capital

Change-Id: I864b8b3f1f67e34e369aba7aadaf87e7752c894d

Merge "Special:Allpages replace table with unordered list"

Localisation updates from https://translatewiki.net.

Change-Id: I0a3841a41e7f3ff5c19a756f1e7ce36779025204

Followup 248ac9e: set $wgAutoloadClasses to an array after declaring

Change-Id: I07cc526ff3145c38bc08cd9b83fb266dc20a47c1

Merge "Support GENDER in MediaWiki:Blockip and Special:Block"

Merge "mediawiki.content.json.css: Don't use funny fonts"

mediawiki.content.json.css: Don't use funny fonts

* Allow skin-provided fonts for .mw-json.
* Use standard monospace font for .mw-json .value. We need to write
  'monospace, monospace' because of old browsers, see design/monospace.html.

Change-Id: I4ed850408895dd1962a0503227ccb384bf479790

Make mediawiki.content.json.css not trigger JSDuck

Fun fact: JSDuck tests don't run for .css file changes on Jenkins, but
when they do run for .js file changes, they also process .css files
and the documentation comments in them.

Change /** to /*! to prevent that, like we've done in other files.

Follow-up to d2a82fcb.

Change-Id: Ib7f9875ea818bc72fe9aaed167b428e570c6df2e

Use new static instead of new self

See code review on I957d6bda626b8e749a99717b5c89c4db38a36bf8

Change-Id: I79e4a987636509515df044da7a6951d3e4b4f824

Merge "Correct variable types in phpdoc"

Correct variable types in phpdoc

Change-Id: I1a57024868fa47d9537585f1bb738d6f7c725c7a
in: LocalisationCache.php and ImageQueryPage.php

Remove unreachable return in ChangesList.php

Change-Id: Ic00d7b4eaebdaa795b550c4213c518b1c56bd297

Split SkinFallback and SkinfallbackTemplate

Change-Id: I55c866c08d4f442021c43068efe64be871afb96b

Merge "Don't include images/categories when behind a local interwiki prefix"

Merge "Add $query parameter to OutputPage::addBacklinkSubtitle"