Committer: vad
LJSUP-7966: Brenden's Notes show through on Featured Post widgetU trunk/cgi-bin/LJ/TopEntries.pm
Modified: trunk/cgi-bin/LJ/TopEntries.pm =================================================================== --- trunk/cgi-bin/LJ/TopEntries.pm 2011-02-16 03:52:52 UTC (rev 18312) +++ trunk/cgi-bin/LJ/TopEntries.pm 2011-02-16 05:07:20 UTC (rev 18313) @@ -149,7 +149,12 @@ url => $entry->url(), time => $entry->logtime_unix(), userpic => $userpic->url(), - poster => $poster->ljuser_display(), + + ## Do not store results of ljuser_display() anywhere + ## becouse they depend on $remote user. + ## this field should be generated on fly. + # poster => $poster->ljuser_display(), + timestamp => $timestamp, comments => $entry->reply_count, @@ -263,12 +268,22 @@ $self->{featured_posts} = $struct->{spots} || []; $self->{timelimit} = $struct->{timelimit}; + my $users = {}; + $users = LJ::load_userids( map {$_->{posterid} } @{ $self->{featured_posts} } ) + if ref $self->{featured_posts} eq 'ARRAY'; + ## Update comments couter foreach my $spot (@{ $self->{featured_posts} }){ my $entry = LJ::Entry->new($spot->{journalid}, jitemid => $spot->{jitemid}); next unless $entry; + + my $u = $users->{$spot->{posterid}}; + next unless $u; + $spot->{comments} = $entry->reply_count(); $spot->{logtime} = $spot->{time} = $entry->logtime_unix(); + + $spot->{poster} = $u->ljuser_display(); } } $self->{min_entries} ||= 3;