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.
  • Awesome--thank you so much for all your hard work! The comment rot issue was the only problem I had with IJ, and this makes me much more eager to purchase one of the "Hospital Benefactor" accounts, should you decide to hold the sale.
  • Awesome!

    You're great, Squeaky! I'm sure that everyone here really appreciates it :D All the hardwork and actually putting effort into fixing old problems, unlike GJ and LJ!
  • Woot! You are God! :D

    Thank you so much for working so hard to figure this one out. I know it had to be very upsetting and time-consuming to deal with. :D
  • You are much loved for how you kept pursuing this until you chased it down and didn't just give up!

    YOU ROCK!

    :D
  • Thanks, Squeaky! I hope this means the other outstanding smaller bugs can be addressed.
    • Yes it does. I will start working on some of the other bugs soon.
  • thank you for fixing it! i had it happen twice to the same entry in my last journal ~dannygokey so i am really relieved it is over with!
  • Hey, just letting you know that I just lost 594 comments on a filtered entry in this journal a minute ago.
  • Wow. I've lost so many comments with comment rot on other accounts. I really truly hope from this point out that it won't be a problem anymore. I know there's nothing you can do about the comments we lost, but thank you Squeaky.
  • You're really the greatest. Love that you keep giving us updates about what's going on.

    Thank you so much for fixing the rot and yay for the possibility of a mass icon uploader.
  • Thank you SO much, Squeaky! You ROCK!
  • Mass Icon Uploader?! One more reason I love IJ! That would be made of awesome....let us know when it's up and running!
  • (Anonymous)
    [Error: Can't call method "new_message_count" on an undefined value at '/home/lj/htdocs/allpics.bml' line 9. @ ijwebsever-tugboat2.insanewebhosting.com

    i keep getting that underneath "view" on the left side of my layout whenever i log out of my journal.
    • Let me know if it continues to occur. That shoudn't be happening.
  • Here's hoping! Good job, Squeaky!
  • Now to getting to writing a mass icon uploader.

    Don't forget that the good people at have already written one; you can probably take the code from http://bugs.dwscoalition.org/attachment.cgi?id=213 and apply it with the minimum of changes. The multiple icon uploading doesn't currently work with IE, but that's a known bug. (http://www.dreamwidth.org/)
  • Fantastic! Thank you so much, [info]squeaky! All your efforts are truly appreciated! Hope you have a great weekend :)
  • As always, thank you, Squeaky!

    Are you running for LJ council again this year? The least we can do is vote for you, if you are!
  • This is awesome! W00t! I've never had comments deleted, but I've experienced the reset to default icon if I edited a post (though I must admit while it was annoying, it was livable with). It's to great to here this issue is hopefully solved for good! :D
  • UGH YES THANK YOU! You're phenomenal :]
  • I have an entire thread a friend and I did back on March 25th that has just today had all the comments disappear from it. Is there any chance that I'll be getting those comments back? I've never had any problems with the site before now, and usually it runs really great for me.
  • Thank you for all the hard work you do for us. I am so happy that my RP will find a home her now that the rot is done and over.
  • i just lost over 100 comments last night on this account. after i had edited the post they were gone so is that why it happened..?
Powered by InsaneJournal