Hi, just upgraded from 1.5 to 1.7, and I seem to be getting stuck on "Saving vote..." after the AJAX call. I have seen the problem with jQuery 1.7 and 1.8. The vote gets registered, but the widget no longer refreshes afterwards. in the console I see the following:

"event.returnValue is deprecated. Please use the standard event.preventDefault() instead."

It does not seem to be related to either:
https://drupal.org/node/1865510 or https://drupal.org/node/2232031 (occurs even without nginx)
or
https://drupal.org/node/1809340 (since this was reported fixed in 1.6)

CommentFileSizeAuthor
#8 rate_widget_id_class_mismatch.patch650 bytesetiennechataignier
#4 rate-js-2266371.patch561 bytesmpaler
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

agapito78’s picture

agapito78 CreditAttribution: agapito78 commented

Hi,
Did you get any solution for this issue?

Thanks!

jOksanen’s picture

jOksanen CreditAttribution: jOksanen commented

Having same issue.

mpaler’s picture

mpaler CreditAttribution: mpaler commented

subscribe

mpaler’s picture

mpaler CreditAttribution: mpaler commented
FileSize
rate-js-2266371.patch561 bytes

Try this...

sgoelz’s picture

sgoelz CreditAttribution: sgoelz commented

Thanks for the patch. That worked for me.

maximpodorov’s picture

maximpodorov CreditAttribution: maximpodorov commented

The patch works, but its format is not quite correct (extra directory names).

thomas1977’s picture

thomas1977 CreditAttribution: thomas1977 commented

@ mpaler

Could your patch be committed to a dev release (for a full module download)? Thanks :-)

Just ran into this problem as well.

Best,
Thomas

etiennechataignier’s picture

etiennechataignier CreditAttribution: etiennechataignier commented
Status: Active » Needs review
FileSize
rate_widget_id_class_mismatch.patch650 bytes

The patch posted below is a workaround.
The real problem comes from the misuse of drupal_html_id() to build the DIV widget.
Attached is a patch which is better with rate logic.

newsec’s picture

newsec CreditAttribution: newsec commented

Confirm bug.
#8 solve problem.

newsec’s picture

newsec CreditAttribution: newsec commented
Status: Needs review » Reviewed & tested by the community
thomas1977’s picture

thomas1977 CreditAttribution: thomas1977 commented

Any news on a new release that fixes the issue (prob based on #8)?

rpayanm’s picture

rpayanm
he/him
Primary language Spanish
CreditAttribution: rpayanm at SeeD EM commented

The #8 work for me perfect!

mr.alinaki’s picture

mr.alinaki CreditAttribution: mr.alinaki commented

Met same problem yesterday. Patch from #8 worked for me.