(bug 39062) Show revs without rev_parent_id in Special:Contributions.
authorBrian Wolff <bawolff+wn@gmail.com>
Thu, 27 Dec 2012 07:28:17 +0000 (03:28 -0400)
committerBrian Wolff <bawolff+wn@gmail.com>
Sat, 29 Dec 2012 05:07:58 +0000 (01:07 -0400)
commitede90ac05abc60bc87d59b27711cbc63f8cf899f
tree5fe1e885663280c3cc7f7e05696de0888d8aeb64
parent8543b735ef5afaabaf021deced3030ac7dad5f14
(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
RELEASE-NOTES-1.21
includes/specials/SpecialContributions.php