Use Html::rawElement() instead of hardcoded string in Special:Contributions

Since there is generally no classes in the list items, this avoids having a
lot of useless class="" in the output.

Change-Id: I804af2269def1f06d023145c9ced6f1afa0d0c71

(bug 39062) Show revs without rev_parent_id in Special:Contributions.

This is a follow-up to Ifd63cdf1 (c8e0ec3f)

Currently if rev_parent_id is not populated on a row we just
output an empty bullet - <li class=""></li>. (Which can
happen if an update is performed properly. There are (were?) some
examples of this on en wikipedia. I was just talking to a user
who had some of his revisions not be populated for that field,
so it can happen.) Heck there's even code that checks for this
case inside the if statement (before this commit) that stops
non rev_parent_id rows from being processed.

This seems very very wrong to me. Change the check for if a revision
is valid to look for rev_id (which is the primary key, so
going to be there, and always has been there).

Also in the
case we don't have a valid revision and can't format it
(For example, an ext made a hook that added random rows
to the Special:Contribs query, but forgot to format some),
instead of outputting an empty bullet, output just an
html comment, (and a debug log entry).

Thank you to Isarra for reporting this issue.

Change-Id: I7ef562d6e829877dd8ea4ffb1e7e04b46c943ec1

Merge "API list=alltransclusions, rm unique+gen & continue restrict"

Localisation updates from translatewiki.net.

Change-Id: Icf200b83397e2d0c7d01269a063d23759c397b95

API list=alltransclusions, rm unique+gen & continue restrict

* list=alltransclusions added to enumerate every instance of page embedding
* list=alllinks & alltransclusions now allow both 'from' and 'continue' in
  the same query. When both are present, 'from' is simply ignored.
* list=alllinks & alltransclusions now allow 'unique' in generators, to yield
  a list of all link/template target pages instead of source pages.

Change-Id: I4137e2d790f988e0cef638703d9d6961a47fe662

Merge "ApiOptionsTest: Group medium (takes more than 2 seconds)."

ApiOptionsTest: Group medium (takes more than 2 seconds).

Various jobs are failing in Jenkins due to this test exceeding
the maximum execution time of 2 seconds.

https://integration.mediawiki.org/ci/job/mediawiki-core-phpunit-api/684/console

Follows-up 13f0f58, fb7c95f.

Change-Id: I41a9a571a50e538de514586618d58438c5dd6148

(bug 41733) Hide usermessage element from printable view.

Change-Id: Iefa8158d28c8a78f21ad8155eba11d4870573b59

Add plural support for protocols in 'linksearch-text'

Change-Id: I6b043b4b3f7bba585abe6bd44d29a300a35d697c

(bug 42949) API shouldn't assume all exceptions are MWException

Since I20525253, the exception-handling code in ApiMain has
inadvertently assumed that all thrown exceptions will be subclasses of
MWException. This change restores the check for MWException while
preserving the new check for not UsageException introduced in that
commit.

Change-Id: I5016143a07c871bae910e93b22fefe61a959780c

Localisation updates from translatewiki.net.

Change-Id: I734196d547973e07c8b129523ce693c318c4ee1a

Fix typo in ResourceLoaderFileModule.

Change-Id: Ia4460fb8bba7e08f235016fa8b59adf97e48a4d3

Fix some validation errors on Special:Import

Fixed errors are:
* Duplicate ID mw-import-table
* Duplicate ID mw-interwiki-rootpage
* The for attribute of the label element must refer to a form control
(2x)

Added also a id for another input element for easy user scripting

Change-Id: I9c7aa3b2af2d49cf3aed42a057ab13aa1275ee90

Use NS_* constant for default namespace in api

Change-Id: Ic46e7fe1877c4067b1b6bbea71216c0cdf594cb2

Merge "allow OldChangesListRecentChangesLine hook to return false and omit line from RC"

Remove unused globals

Change-Id: Ie1920c5b6f0a38c7e697a493aef88e54873c07ab

allow OldChangesListRecentChangesLine hook to return false and omit line from RC

- this enables an extension to do some processing on the RC line,
and if processing fails or for other reason, the extension has the
option to not output the line at all in Special:RecentChanges and
Special:Watchlist.

Change-Id: Ibf3a217afba5b4ad7992919399710fe3da301025

Localisation updates from translatewiki.net.

Change-Id: I2abbe72f8a76d574a436a4b29755e3f3b889d94e

Merge "Add mobile target to modules needed for mw.msg()"

Merge "(bug 24692) Make the padding support RTL languages"

Merge "Creating new GetDoubleUnderscoreIDs hook"

Creating new GetDoubleUnderscoreIDs hook

Creating a new hook so that doubleunderscore (behaviour switch) magic
words can be defined from Extensions (so that I can solve bug 6754
from an extension rather than from core, as requested).

Change-Id: I734a4a1639be61052924838b19ed9db93a28e50b

Merge "(bug 25398) Remove upgrade1_5.php"

Localisation updates from translatewiki.net.

Change-Id: Ie29068d717b09723f85975886e4478b145862d63

(bug 24692) Make the padding support RTL languages

Change-Id: I5be38f001dcc13a57e213fdbb505f05623070af3

(bug 25398) Remove upgrade1_5.php

The script has stopped working for quiet sometime now. If one still has
a 1.4 (or previous) wiki, it should upgrade to 1.5 first.  I do not
think it is worth anytime to try to port the updater nor to mention a
release note entry.

Change-Id: Ic0e366e2c7086e371d486b05ca222f2853f64ccb

Made wfBaseconvert actually work with $lowercase=false.

* This used to return false for any non-digit input characters.

Change-Id: Ie7fca2669724dea29f4733bbc77a559d4f48a5b1

Allow setting a form message prefix in FormSpecialPage

Change-Id: Iff14a626853c98190a1aba8263b8a7ec79e9b9b5

RELEASE-NOTES entries for I7761a635 (af72dd6) and I2b2ca3da (24584ca)

Change-Id: Ib4bcf9811a5727d8d2835aed78d58eca1d77cb7e

Merge "(bug 43382) Alt+shortcut key does not work in Chrome/Linux"

(bug 43382) Alt+shortcut key does not work in Chrome/Linux

For chrome in linux, the shortcut keys given are ALT+[KEY].
This works, but fails for keys like E, F, D etc.
ALT+E or ALT+F opens the settings menu, ALT+D highlights URL.
To make it work always with all keys , it should be presented as
ALT+SHIF+[KEY] as done for Windows.

Change-Id: I87a9d498ac99d381c31c94b03d6cc72fa265e839

(bug 43379) History page fail gracefully if rev_len unavailable

If the length of a revision is unavailable, do not show
the page size (or char difference) instead of pretending
it is 0 and showing wrong info.

Change-Id: I2b2ca3da480e9222911ff6a91f3ab1ca4a8b9ba1

Merge "(bug 34581) RemoveUnusedAccounts: Ignore newuser log."

Fixing some variable names and comment formatting - no functional changes

Changing $_ to $number; changing $numberpart to $integerpart

Also adding a unit test for the commafy function

Change-Id: Iaf6dd027bd70722d316d1a9c10c9913fff8300ce

Merge "(bug 42269) namespaceDupes.php has to use the interwiki cache"

(bug 34581) RemoveUnusedAccounts: Ignore newuser log.

Patch-Source: https://bugzilla.wikimedia.org/attachment.cgi?id=11554
Change-Id: I7761a635261881f5a7ff7aaa8f8513d3d3a2149b

Merge "Fix database access in ApiSetNotificationTimestamp"

Localisation updates from translatewiki.net.

Change-Id: Ie228538204b005bd62307a130258c23ec2e74606

Merge "Hook to allow manipulation of image data"

Hook to allow manipulation of image data

Adds a new hook, ThumbnailBeforeProduceHTML, to ThumbnailImage::toHtml
method, in order to permit the user to manipulate the image and link
attributes before the HTML is rendered. For details see bug 41835

Note: Patch given by Victor <vdanilchenko@vistaprint.net>

Change-Id: Ib94967997e1303522d0081c7387e29af9c0ef0f0

Merge "Improve usage of which and that"

Merge "Only variables can be passed by reference..."

Merge "Suppress warnings around copy call"

Update CREDITS

* Adding Ryan Kaldari, Ori Livneh and Patrick Reilly as developers
* Adding every contributor with 5+ changes merged in core as patch contributors

[ Notes ]

(1) This operation has been facilitated by a script to compare CREDIT
    entries and the Gerrit contributors base.

    Query used: status:merged project:mediawiki/core NOT owner:L10n-bot

(2) Contributors with at least one patch merged will be added later.
    If you're concerned, to speed the process, see the next note.

(3) Names sources used:
- WMF staff & contractors list
- Name used in Git commits
- Name on meta. or mediawiki. user page

Change-Id: I94131a691ef9e0b5f999aba63169157d9e5b433b

Localisation updates from translatewiki.net.

Change-Id: I518f7ce098fc59f3f34d2898138421895024270c

Fix database access in ApiSetNotificationTimestamp

ApiBase::getDB() doesn't take a parameter indicating whether DB_SLAVE or
DB_MASTER is wanted. Use wfGetDB() instead.

Change-Id: Ic2057a1561ad31a0638eea6bae887982a4fe8d80

Only variables can be passed by reference...

Change-Id: Ifc0f9a25148e8728b3b57885ca241853f0c2a894

Suppress warnings around copy call

Attempt to shut some noise in production logs

Change-Id: I4bac4027d82e4194b6ec46658d52f49038db573c
Warning:  copy(/mnt/thumbs2/wikipedia/commons/thumb/8/8c/Map_of_USA_AK_full.png/120px-Map_of_USA_AK_full.png) [<a href='function.copy'>function.copy</a>]: failed to open stream: No such file or directory in /usr/local/apache/common-local/php-1.21wmf6/includes/filebackend/FSFileBackend.php on line 254

Documentation improvements in ChangesList.php

Change-Id: I7e2a35d2f15ed72338779b39e7dd64f4ee5a3989

Merge "(bug 36751) InstantCommons doesn't follow redirects"

(bug 36751) InstantCommons doesn't follow redirects

This fixes r56837: now redirects are only skipped if the redirects query
parameter is not set. This was presumably the intent of that commit, see
https://www.mediawiki.org/wiki/Special:Code/MediaWiki/56837#code-comments

Change-Id: I27485bba9fe196211baeeb84238e4a397c620a1d

Simplify Xml::encodeJsCall

Change-Id: Idf2838a1b32170ca6336eb4a67f16cd3335218f9

Additional list tests for Parsoid.

Change-Id: I166850e241af6c2e678d92aacb5ac767b4353397

Merge "Use wfWikiID() instead of $wgDBname in wfIncrStats()"

Localisation updates from translatewiki.net.

Change-Id: I1a79e5cdde73006f67e0b43d760cf14ba76256c2

Use wfWikiID() instead of $wgDBname in wfIncrStats()

This allows to have distinct stat groups for people using database table prefix and
it makes wfIncrStats() consistent with ProfilerSimpleUDP that already uses wfWikiID().

Change-Id: Iecef9f490df8c5685a4887cc81dcb4a7e4d3172c

(bug 43316) Don't fetch revision ID when a page doesn't exist.

Fixes I90446b7b.

Change-Id: I429653942bc388b6658b956d36d16398bbcf04e5

Merge "Improve documentation of content handler stuff"

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

Improve documentation of content handler stuff

- Split file and class documentation
- Add some missing file/class descriptions
- Add missing @ingroup Content

Change-Id: I7f7e3056570ca13a92f36a408c9b961c938f09a1

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 42269) namespaceDupes.php has to use the interwiki cache

Change-Id: I8fdf3a44bef8044d064db9d4489425e20e04d004

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

Improve usage of which and that

I tried to improve the usage of which and that in messages
and comments here, and also made a few other grammatical changes.
I think that it's an improvement, but I am not a native English
speaker, so it's possible that I made mistakes. Please review
it carefully. It will be great to invite your grammar-loving
friends to check it, too.

Change-Id: Id7d8d97950a32c126378fbb15da5ff616ebf599b

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."