Forum OpenACS Development: FYI: Fix applied to openacs.org forums

1: FYI: Fix applied to openacs.org forums

Posted by Lars Pind on 12/20/02 02:26 PM

This is an FYI for the openacs.org team.

I deleted a couple messages from a spammer, and that caused the forum-view page to break, because the message_id's were cached, so the cached result referred to a message that didn't exist any more.

I fixed it by flushing the paginator cache in www/moderate/message-delete. While I was at it, I also added a context bar to that page.

And finally, I also added a cache flush to the message-post page, since people have been complaining that their newly posted messages don't show up right away. They do now.

Hope my fix is okay with everybody.

/Lars

2: Re: FYI: Fix applied to openacs.org forums (response to 1)

Posted by Jun Yamog on 12/20/02 02:49 PM

Thanks Lars... bug fix + added feature!

3: Re: FYI: Fix applied to openacs.org forums (response to 1)

Posted by Don Baccus on 12/20/02 07:09 PM

Did you put it in the openacs.org CVS tree or just patch it directly? (there's been some inconsistency in this area but things should be changed in the CVS tree for our website).

4: Re: FYI: Fix applied to openacs.org forums (response to 1)

Posted by Lars Pind on 12/21/02 03:42 PM

I fixed it in-place, and then did a cvs commit of the modified files.

5: Re: FYI: Fix applied to openacs.org forums (response to 1)

Posted by Jonathan Ellis on 12/21/02 03:56 PM

three cheers for lars :)