Look up the list of events which should be included, instead of
hardcoding.
Patch by fu.
diffs (14 lines):
diff -r 08ca14543825 -r 7a87e8cad75d cgi-bin/LJ/NotificationInbox.pm
--- a/cgi-bin/LJ/NotificationInbox.pm Fri Apr 01 15:13:37 2011 +0800
+++ b/cgi-bin/LJ/NotificationInbox.pm Fri Apr 01 15:30:58 2011 +0800
@@ -178,8 +178,9 @@ sub subset_items {
sub singleentry_items {
my ( $self, $itemid ) = @_;
+ my %related_events = map { $_ => 1 }
LJ::Event::JournalNewComment->related_events;
return grep {
- ( $_->event->class eq "LJ::Event::JournalNewComment" ||
$_->event->class eq "LJ::Event::JournalNewComment::TopLevel" )
+ $related_events{$_->event->etypeid}
&& $_->event->comment
&& $_->event->comment->entry # may have been deleted, which breaks
all filter to entry comments
&& $_->event->comment->entry->ditemid == $itemid
_______________________________________________
dw-changelog mailing list
[email protected]
http://lists.dwscoalition.org/mailman/listinfo/dw-changelog
