Cleanup to ImagePage::openShowImage()

- Removed useless $showLink variable and related check since the variable is always defined to true
- Moved $linktext up so that there's no need to use a temporary variable to modify it
- Moved $longDesc down where it's used
- Change "else { if ( ... ) { } }" to elseif for better clarity

Change-Id: I65dcbb76c7f0a0441a6420e76affe797a3dc9c45

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: If69487fb62fda379ce9fc2c38ca4dc6fe0c07292

(bug 43307) L10n recache needed in LogFormatterTest::tearDown

LogFormatterTest replaces all extension localization files in
$wgExtensionMessagesFiles with its own, and correctly restores this
variable. But it neglects to refresh the localization cache after
restoring the variable, so subsequent extension tests are missing their
localization.

Change-Id: I74809af47830a5e09c36f8fec20c9c65b1e3aa17

Set correct context for ImageHistoryPseudoPager

ContextSource::getContext (ImageHistoryPseudoPager): called and $context
is null. Using RequestContext::getMain() for sanity

Change-Id: I0430791a705f42ca3e5d02692070f0fb0dadc897

Merge "(bug 41470) Use numParams in Language::formatDuration"

(bug 41470) Use numParams in Language::formatDuration

Change-Id: I3f9642d445ef6f3b3c77a824e385ce259210e46e

(bug 32434) API allows reblocking the user without reblock parameter.

This change adds special case in processing of the block form.
An unused parameter 'AlreadyBlocked' was replaced by 'Reblock'.
Parameter 'PreviousTarget' was added so API user can block him/herself.

Change-Id: I29275eb4b5f425b7382ba8b7dec050209878f81b

add function in ChangesList to getTimestamp without inserting it

Change-Id: Iae28a6a36f3ab1de3c74cd9eba88793c85faa322

Merge "misc style fix"

misc style fix

* makes booleans lower case
* add spaces before open braces

Change-Id: Id88884e08bc23d7730361ee91646f54f5e16920b

Merge "(bug 42168) Nicely handle missing revisions in ApiQueryRevisions."

(bug 42168) Nicely handle missing revisions in ApiQueryRevisions.

ApiQueryRevisions had fatal errors when the Content object of a revision was null.
Should be fixed now.

Change-Id: Ic41d03fc55413ba7ba1c96b4f06d71d79fb7c4eb

Merge "[JobQueue] Cleaned up DuplicateJob factory function."

Merge "Add missing message in messages.inc and rebuild MessagesEn.php"

(bug 42519) Message "enotif body intro deleted" mentions revision

"See current revision" was just a way to call the article URL.
I'm leaving the URL and removing the incorrect definition.
URL is still useful to have something to click and also
to see the talk page, the excerpts from logs, etc.

Also replaced "changes" with the more generic "activity" in enotif_body.

Change-Id: If77117e208b350dc318471dc84cef339d653f10d

Fix ApiQueryLogEvents::addLogParams for unknown types using the new format

Fixed ApiQueryLogEvents::addLogParams for unknown (probably extension)
log types which use the new log format (with keys like 4:foo:paramname).
Until now such keys we're directly put into the XML output resulting
in invalid XML (see bug 43221, which will be fixed by this). To prevent
this we just remove everything except the plain parameter name and use that
as key for the output.

Change-Id: I1a3c7ac624eb575b879d068d47d3a13c9972b1a1

Merge "[MessageCache] Cleaned message cache load() error handling."

cleanup WatchedItem class

* removed $id, $ns, $ti which were set on construction and replace them
  with meaningful accessor getUserId(), getTitleNs and getTitleDBkey()
* removed a commented out hack from September 2004

Change-Id: Iaa9e851516e0522492b0452430c7583fe3526ffc

Add an option to importImages to search subdirectories recursively

Change-Id: I73a0d42884c1006492cd3de912eb1dc6dab08c5e

(bug 39957) Document $wgUnwatchedPageThreshold var

Added in I1cb6ee22d50d871a8c7083c5b7a091d31cf640f5.

Change-Id: I35df67c66855c9aa1b8ee74ee2c57ff109823634

fix and provide more documentation in EditPage for mergeContentInto functions

Change-Id: I2a5aed29a6e8ff05ddea5d46f854740b9f3b03b1

Add missing message in messages.inc and rebuild MessagesEn.php

Change-Id: Ie40b26ba357db133c4aa5f0f42eaa0d56ad49bd5

Merge "(bug 39957) Added threshold for showing number of page watchers."

Improving function descriptions in Linker

Change-Id: Id3aff70f8b5813f80e5b44bf14cb8198b2e16d4b

Merge "(bug 37195) Doc fix. $from and $until are arrays since 1.17."

(bug 39957) Added threshold for showing number of page watchers.

Added configuation variable that allows bypassing the unwatchedpages
permission when the number of people watching a page is greater
than a certain threshold. The default value is false, meaning the
unwatchedpages permission is always required.

Change-Id: I1cb6ee22d50d871a8c7083c5b7a091d31cf640f5

Fixed comment typo.

Change-Id: I5f68cff001a6059806d1dd5e99db59754290ce71

(bug 37195) Doc fix. $from and $until are arrays since 1.17.

This makes $from and $until of the constructor be documented as
arrays instead of strings. It also changes the default to be
an empty array instead of the empty string. In theory this
could use some back-compat code, but the change was made
way back in 1.17 so I don't think there's a point.

Change-Id: I6f995ea925960b90bc4a458db5af4f7aa768c1ae

[MessageCache] Cleaned message cache load() error handling.

Change-Id: Ic8d62fa4162b54c776d934bd7106978fb36a6d32

Localisation updates from translatewiki.net.

Change-Id: I4c875b76b6cdb7d88d5cb6744a4d344d9f9a713b

Merge "make rebuildtextindex script aware of content models."

Merge "Pass user to LogEventsList::getExcludeClause"

Merge "(bug 43184) bogus script path in Special:Version"

Pass user to LogEventsList::getExcludeClause

This avoids $wgUser in that method

Also removed the double strencode for logtypes,
that is already done by Database::addQuotes/Database::makeList

Change-Id: I5f7f6da06594d92375f74ca48c04d180505642a9

Merge "Fix PEAR part of install-phpunit.sh."

(bug 43184) bogus script path in Special:Version

Corrected display of script path change so that when the script path is
empty, we default to showing / as the path.

Change-Id: I8bb6ff820fa3f8d7c3cdd608f531c0f3c36389ca

Fixed description for async parameter in upload module

The description was already present,
but it used incorrect syntax, so it didn't show.

Change-Id: I7522265362f019c73523125df182366dfd274be2

Merge "Localisation updates from translatewiki.net."

Merge "(bug 39490) Add caching to InfoAction."

(bug 39490) Add caching to InfoAction.

Added caching to InfoAction. The cache key uses the ID of the
latest revision so it is auto-invalidated when the page is
changed. Also, the cache key is deleted whenever the page is
purged.

Change-Id: I90446b7bcb4517959605aa38eacfada2b785060b

Localisation updates from translatewiki.net.

Change-Id: I8881e68353fccbbd7c4b8da737045747dc5c1c05

Fix PEAR part of install-phpunit.sh.

* Discover additional channel, update, and use alldeps.
* Remove obsolete pear.symfony-project.com
* Per https://www.mediawiki.org/wiki/Manual:PHP_unit_testing/Installing_PHPUnit#Using_PEAR

Change-Id: I7f348c967c1526736ec7f34d610d315f8aabd504

Merge "(bug 43218) Fixing PHP notice: undefined SERVER_PROTOCOL key"

(bug 43218) Fixing PHP notice: undefined SERVER_PROTOCOL key

Change-Id: I1ce7605a7c8503aed37ed9b3e7c3f3c13083db6f

Merge "Update docs"

Document ResourceLoaderWikiModule::getPages.

Change-Id: Id3b3df20efcec90dfcc7cd496aecbe74e5cd9915

[JobQueue] Cleaned up DuplicateJob factory function.

Change-Id: If2303776bdbc9b78a07907e7b290c2da5af7e493

Localisation updates from translatewiki.net.

Change-Id: I0ad670d4b2dc9410e0b1021c29d79cd209209b78

Suppress warnings around unlink call

Attempt to shut a lot of noise in production logs

e.g.
1 Warning:  unlink(/mnt/thumbs2/wikipedia/commons/thumb/f/f2/Members_only_jacket.jpg/800px-Members_only_jacket.jpg) [<a href='function.unlink'>function.unlink</a>]: No such file or directory in /usr/lo
cal/apache/common-local/php-1.21wmf6/includes/filebackend/FSFileBackend.php on line 422

Change-Id: I9e1407c4b11301f468f8e291105955ade9a731d0

Merge "(bug 43096) Fix querying suppression logs"

Merge "(bug 43207) Fix regression in API generator=duplicatefiles"

(bug 43096) Fix querying suppression logs

Change-Id: Ia9709ed5f694e07b7544887b49c99db5ed7063df

Merge "use null rather then false and doc fix"

Merge "(bug 43177) API: Fix regression in case handling for sha1 params"

Merge "(bug 43137) Don't return the sha1 of revisions through the API if the content is revdel'd"

(bug 43207) Fix regression in API generator=duplicatefiles

Prior to change I745cae7a, generator=duplicatefiles would return all the
files that were duplicates of the passed files. Since then, it will
return only one of the input files if any duplicates are found, possibly
one that doesn't even have duplicates.

The problem is the use of a wrong variable.

Change-Id: I8e8493718ab1f7f5cb6ae718790738a7cf3c72c7

(bug 43137) Don't return the sha1 of revisions through the API if the content is revdel'd

Change-Id: Ic2cad08cdae428d04dbdc31665c531d8a4759cbb

(bug 43177) API: Fix regression in case handling for sha1 params

While the change in Ic980fd71 makes sense since MediaWiki stores the
hashes with lowercase letters, historically aisha1 and fasha1 accepted
both upper and lowercase because wfBaseConvert accepts both.

To avoid a backwards compatibility break, let's adjust the code to
explicitly lowercase the input for those parameters. And for good
measure, let's also accept both cases for the corresponding sha1base36
parameters.

Change-Id: I704935193398c722c22a302dc9d23f2c0e2e3e39

Update docs

Change-Id: I96c04c4ce9384816e120baef3572dfc4fd216b9a

Fix the little documentation that FormOptions::consumeValue() has

Change-Id: I9a4bfaace4f19ba5e3e8a405118fe1bae3a1696d

(bug 42184) Send second variable for large-variable message

Change-Id: Iad4bb0d0ab25248ec6296af4f39401eeb59c306f

Merge "ResourceLoader: Move safeFilemtime() to ResourceLoaderModule."

ResourceLoader: Move safeFilemtime() to ResourceLoaderModule.

This is a useful method not just for inside and sub classes of
ResourceLoaderFileModule (i.e. it could've been useful in
VisualEditor's ResourceLoaderModule class as well)

Also moved up getTargets() to be in the right section (looking at
the file as a whole).

Change-Id: If696ffbdc5aa7f0a51603bcf9d52adab38b9c686

Merge "applying code conventions and cleanup in mediawiki.special.*"

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: Iecaef86923bb32e102dd69ec12651824565bc3ed

(bug 41342) jquery.suggestions: Call cancel before fetching.

This caused race conditions in the calling code, if it used AJAX
and they completed in different order than they were started.

For example in mediawiki.searchSuggest, when one slowly types
"wikipedia", 9 requests get sent (suggestions for "w", "wi",
"wik", etc.), but they might not necessarily arrive in the same
order, especially on slow (e.g. mobile) connections.

Then you get the race condition effect - suggestions for
"wikipedia" flash briefly and then are replaced by suggestions for
"wik", even though the bolding is correct.

This commit fixes the issue by using a function that was already
present, that calls the callback as well as clearing the internal
timeout, which for some reason wasn't used in this code path.

Change-Id: I880a61e5e45a64bea6d679ed160c4be3da085dfa

Merge "Remove nocreatetitle"

Remove nocreatetitle

This message was removed from OutputPage in commit 3801a3cd
and it doesn't appear to be used elsewhere.

Change-Id: Iba5de5ba6f6e699bac5b1e21efd8869006398f1b

applying code conventions and cleanup in mediawiki.special.*

Using standard closures per [[CC/JS]], typos in comments, indentation
inconsistencies, some other minor changes.

Change-Id: I53eef75006d87c8b55d6780d36165317066d1368

(bug 28401) Make RC en RCL honour options for rss/atom

RecentChanges and RecentChangesLinked should link to atom/rss feeds using the same options as specified for the pages themselves. E.g. if somebody is looking at Special:RecentChanges&hideliu=1&namespace=1 the feed(s) should link to Special:RecentChanges&feed=atom&hideliu=1&namespace=1. All options available on RC and RCL are already supported by their feeds, except the maximum number of changes in feeds is bound by $wgFeedLimit.

Change-Id: I8fb00ccd7efdd962a8a42f0bce3eac5177e2cfbd

Make wfSuppressWarnings kill E_STRICT in PHP 5.4+

Since PHP 5.4+ E_STRICT is now included in E_ALL. Hence
wfSuppressWarnings was not filtering E_STRICT.

There was some complaints on project:support_desk about
timezone warnings popping up. See also bug 43092

Change-Id: Ie1ace158dac1733e6b2b2c1d533004d9bcab8c80

Trimming 'addedwatchtext'. The previous text was too long and overly detailed.

Change-Id: If7c5ba8e2a7dd7e6e628118a72f3c4527528e4c4

Moved LocalisationCache under /cache.

Change-Id: I54b230376ed682d6a6b7d6fa4fd5ab133c8c8b1b

(Bug 35987) Clarify change password on Special:ChangeEmail

Continuing Ravichandran's work, I fixed issues raised by IAlex.

Change-Id: Ib8bec70bf0aa451d26971569214b4b13c2abddff

Merge "drop "@fixed" annotations in CologneBlue skin"

xLocalisation updates from translatewiki.net.

Change-Id: I1167ecea2f384d226f18dcba864e6ea69c40381d

drop "@fixed" annotations in CologneBlue skin

These were only useful for me when I was refactoring the code,
and are not needed anymore.

Change-Id: I30aca1619817102a412f9d3cf83da8cf52fa62af

Add new message key from  I28ef70f7 to maintenane file

Change-Id: I56c3d200ff22b12c395ccfefba9c163f250f6348

use null rather then false and doc fix

Change-Id: Ifb8798a66d32da36fc8134e2039ec880c0f1dc1a

Removed commas before 'because'

If I understand correctly, a comma is not needed before because
in these messages. It's possible that I'm wrong, however, so
if you know English punctuation well, please review this.
You can also ask your grammar-sensitive friends.

Change-Id: Ic9c1e42be28ce6d23bdbd15714f1c512a888ed75

Merge "Add some entries removed in I41f1995d back."

Merge "(bug 42787) Make import robust against corrupt content."

Use $IP instead of relative paths.

Change-Id: Ic49ed466d3974c1640378831f0db61fda89f4e9e

Use $IP instead of hard-coded relative path.

Change-Id: I6e631116ca73896527eee7a607204071c1467414

Merge "jquery.badge: Add ability to display the number zero."

jquery.badge: Add ability to display the number zero.

This is needed by Echo extension.

Follows-up Id5e7cbb1.

Change-Id: I88c5f819c42d9fe1468be6b2cf74413d7d6d6907

[LockManager] Moved ScopedLock to its own file.

Change-Id: I7430604d3a23a25e8b82036b022336a73cbbc996

Merge "Revert "Adding ability of jQuery badge to display the number zero if requested.""

Revert "Adding ability of jQuery badge to display the number zero if requested."

This reverts commit b170b56f09d1ec5ab5e34e997447546239e008fd

Merge "(bug 37714) Use log type in target object when deleting logs of the same type"

Merge "Test a newline-only template parameter"

Merge "Localisation updates from translatewiki.net."

Test a newline-only template parameter

Change-Id: I99f17767e0aed915e84962a72403ebfb3b1307be

Localisation updates from translatewiki.net.

Change-Id: I55a9766a29e8bb5513bcb472cad30c4ed979283c

Merge "Adding ability of jQuery badge to display the number zero if requested."

Merge "Changed LoginForm::addNewaccountInternal() to return Status."

Add link to localisation instructions in the installer

The installer didn't give any clue on how to localise MediaWiki,
while localisation is the main "customisation" need for a wiki.
"Localisation" now does have some human-redable sections with pointers.

Change-Id: Ia08eaababcaaf281cae6d8b509cead3a66a62111

Merge "(minor) Always use getSites() to get sites."