Ajax star rating bar

[eluser]Silvrbak[/eluser]
That would be great if you could up the revised versions of the Jscripts. I hooked up Firebug and the error I am getting is this,

Code:

viewer has no properties
changeText(" unit_long66334", "unit_long66334")rating.js (line 94)
handleResponse()rating.js (line 79)
sndReq("3", "66334", "<IP ADDRESS>", "10")rating.js (line 67)
onclick()rating.js (line 120)
[Break on this error] viewer[removed] = text;

This is in the ratings.js script.


[quote author="debian4tw" date="1212742865"]Hi guys!

@Silvrbak
Looks to me that at somepoint javascript is getting problems, probably when showing the response once the vote has been comited, have you tried to check what happens if you disable javascript? or are you getting any javascript errors? firebug says something? let me know and maybe we can figure out togheter whats going on there.

anyway i slightly modified this library to work with query strings off and use uri segments, also i replaced the whole js work (behavior.js and rating.js ) with a few lines of jquery code. let me clean it up a little bit tomorrow and i'll post mi modifications here (if thats ok with wiredesignz) hopefully you'll have better luck with it, or someone will find it usefull

btw:
@wiredesignz
thank you very much sir =) this lib was very helpfull.
One thing that i've noticed is that you're making 2 queries. One to get the current rating and another to check if the user has voted or not(logically to allow the user or not to vote). the last one:

Code:

SELECT used_ips FROM ratings WHERE used_ips LIKE '#7.0.0.1%' AND id = '2'

Imho it's unnecesary to make that query if you call for a static rating from the view.
for instance in a blog, probably you'll want to show the static rating (no rating submit allowed ) when you list and paginate all the articles with the short description and the "read more link". And only let them vote when they are on the "full article" view.
So in the first case by just checking the current rating youre done, and you have 10 or 20 unnecesary queries that are trying to find an ip on a longtext field, i think that's not very cool.

i dont know if it is the best way to solve it, but i've fixed by modifying ratings_model.php, replace:

Code:

$voted = (bool)$this->countBy_ip($ip, $id);

with:

Code:

$voted = '';
if (!$static){
  $voted = (bool)$this->countBy_ip($ip, $id);
}

hope it helps
thanks again[/quote]