InsaneJournal Announcements

Comment rot solved

InsaneJournal Announcements

Comment rot solved

Previous Entry Add to Memories Tell a Friend Next Entry
I found the cause of comment rot. There was an old legacy setting that I didn't know what it did. It was among the last of the things to trace out in the code. I traced it out and found the offending code that was deleting comments and entry properties (status, userpic, etc) and removed it. Please let me know in THIS post if any NEW comment rot occurs after this post.

This has been a very very long and trying road and I want to thank all of you who have stuck it out here in spite of this major bug.


sub new_entry_cleanup_hack {
my ($u, $jitemid) = @_;

# sanitize input
$jitemid += 0;
return unless $jitemid;
my $ownerid = LJ::want_userid($u);
return unless $ownerid;

# delete logprops
$u->do("DELETE FROM logprop2 WHERE journalid=$ownerid AND jitemid=$jitemid");

# delete comments
my $ids = LJ::Talk::get_talk_data($u, 'L', $jitemid);
return unless ref $ids eq 'HASH' && %$ids;
my $list = join ',', map { $_+0 } keys %$ids;
$u->do("DELETE FROM talk2 WHERE journalid=$ownerid AND jtalkid IN ($list)");
$u->do("DELETE FROM talktext2 WHERE journalid=$ownerid AND jtalkid IN ($list)");
$u->do("DELETE FROM talkprop2 WHERE journalid=$ownerid AND jtalkid IN ($list)");
}
</pre>



Edit: After looking at the code even more closely I am sure that this was the cause, I think we can call comment rot a closed issue. Now to getting to writing a mass icon uploader.
  • i have another account that's got over 1500 notifications on it and the notification page will never load when i try to go and delete some. it just takes me to a white screen. i'd like to be able to use the notification feature on that account but i think it's just got too many comments. if you've got a solution for that it would be wonderful. i'm sorry to comment off topic to this post.
    • I have this same problem. On one of my accounts I somehow have 5000 notifications (.... I like commenting, k. don't judge me!) It was up to 4900+ the moment the link first appeared for me. I haven't been able to load it at all, while it loads fine on this account (which has no current notifications).

      I'd like to at least clean out the notifications on the account with so many of them. So I second the question for a possible solution.

      And while I have never had a comment rot problem on any account of mine (knock on wood), I do still want to say thank you, for the work put in to solve it.
Powered by InsaneJournal