Scoop || Collaborative Media for the Masses

Front Page · Everything · News · Code · Help! · Wishlist · Project · Scoop Sites · Dev Notes · Latest CVS changes · Development Activities

Is the new comment notifier broken?
By phill , Section Wishlist [] Posted on Tue Nov 05, 2002 at 12:00:00 PM PST

After upgrading to 0.9 and 1.0 when a user comments on an article and returns to either the section listing or the index page, the comment they just made shows as a new, unread comment. Was that routine modified for .9 and 1.0? If so, why?

Since I'm seeing very few questions about something like this, it may be site specific (read operator error).
If you want to check it out, here is a test story and here is the section it's in.

< Shopping cart development - whats the state of play? | MD5 Passwords >

Menu

· create account
· faq
· search
· report bugs
· Scoop Administrators Guide
· Scoop Box Exchange

Login

Make a new account

Username:

Password:

Related Links

· here is a test story
· here is the section
· More on Bugs
· Also by phill

Story Views

18 Scoop users have viewed this story.

Is the new comment notifier broken? | 5 comments (5 topical, 0 hidden)

Yes (none / 0) (#1)
by theantix on Tue Nov 05, 2002 at 08:34:08 AM PST

That bug is indeed real -- in my understanding it's been in scoop for a long time. It's in the list of bugs in the bug tracker, and will eventually be fixed.

i'm sure it's not that big of a deal.. by phill, 11/05/2002 09:42:16 AM PST (none / 0)

found it. (none / 0) (#3)
by phill on Tue Nov 05, 2002 at 11:19:20 AM PST

in Comment.pm, rusty commented out the line

$S->update_seen_if_needed($sid) unless ($S->{UI}->{VARS}->{use_static_pages});

Uncommenting does not fix the issue but changing it to

$S->update_seen_if_needed($sid);

does fix it.

-p

well now... by phill, 11/05/2002 11:23:32 AM PST (none / 0)
status so far.. by phill, 11/05/2002 04:38:11 PM PST (none / 0)

Is the new comment notifier broken? | 5 comments (5 topical, 0 hidden)

create account | faq | search