Details
Sub-task
Closed
Minor
Description
1) Go to any Using Moodle forum on moodle.org eg
2) Rate a post
3) Unrate the post by selecting the first item in the menu
4) Repeat 2 and 3, it looks like this screenshot.
Attachments
Activity
The printout of "null" is easy to fix. Only IE does that (instead of printing nothing).
And, curiously, the problem about rating texts being accumulated and accumulated on the right is caused because one "fallback code" I added, that only should be executed if no previous "forumpostratingtext" span is found. And that should happen never, because that span always exist. But IE7 isn't able to find it. 
This is the line not finding the span, that is 100% YUI code, is this:
var ratingsSpan = YAHOO.util.Dom.getFirstChildBy(ratingsDiv, function(el){return el.getAttribute('class') == 'forumpostratingtext';});
I'll investigate a bit more and will try some workaround to make it work under IE7 (grrr). Ciao 
This is the line not finding the span, that is 100% YUI code, is this:
var ratingsSpan = YAHOO.util.Dom.getFirstChildBy(ratingsDiv, function(el){return el.getAttribute('class') == 'forumpostratingtext';});
I'll investigate a bit more and will try some workaround to make it work under IE7 (grrr). Ciao So, fixed. Not sure why the 1st syntax doesn't work but... who cares. It's IE7. Ciao
P.S.: I'm going to CVS update that file in moodle.org to have it working ok ASAP. It doesn't interfere with anything else.
P.S.: I'm going to CVS update that file in moodle.org to have it working ok ASAP. It doesn't interfere with anything else. 
Wow!
I thought I had tested that. And results are accumulated on the right. And those nulls.. crazy! Looking to it now.