Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
IE Version 6.0.2900.2180.xpsp_sp2_gdr.050301-1519
Clicking on the stars then this error.
Bummer dude.
Mike
Comment | File | Size | Author |
---|---|---|---|
#10 | fivestar-10-fix-ie_voteHook_indent.patch | 12.94 KB | mr700 |
#9 | fivestar-02-fix-voteHook.patch | 1.08 KB | mr700 |
#8 | fivestar-01-fix-ie-fix.patch | 695 bytes | mr700 |
Comments
Comment #1
MichaelCole CreditAttribution: MichaelCole commentedThe Microsoft Script Editor debugger points to one monstrous line of code
Comment #2
MichaelCole CreditAttribution: MichaelCole commentedOne last thing... The vote "stays" so if I click 2 stars, I get the error, but the control remembers my vote.
The cancel rating button works ok.
Comment #3
mkjones CreditAttribution: mkjones commentedI was getting this yesterday with IE6, however the vote was still case, is that correct?
Comment #4
mkjones CreditAttribution: mkjones commentedsorry, I mean: "the vote was still cast"
Comment #5
ivix CreditAttribution: ivix commentedGetting the same error in IE7.
Comment #6
mr700 CreditAttribution: mr700 commentedI can confirm this, I get:
Comment #7
mr700 CreditAttribution: mr700 commentedIn jquery.js 1.0.4 (not the packed version) from node 110935 comment 13 at line 1521(1519 in the unpatched version) in find: function( t, context ) there's:
The error occurs on ret[i].getElementsByTagName(m[2]).
Comment #8
mr700 CreditAttribution: mr700 commentedI think I figured this out (by trial and error but anyway). There are actually two separate errors, but I don't know how to 'clone' this bug here I'll put two patches.
The first problem is that the 'fix ie6 background flicker problem' is for IE=6 only, not 6.x, so
should probably be
or http://www.hedgerwow.com/360/bugs/dom-fix-ie6-background-image-flicker.html is a better idea, haven't tried this one.
NB: This is patch 1/2.
Comment #9
mr700 CreditAttribution: mr700 commentedAnd the second one. I have absolutely no idea what and how this code works (almost), but changing '$(data)' with 'data' in voteHook silences my IE6. I've tested this one (with the previous patch applied too) with FF 1.5, Konqueror 3.5.6, Opera 9.10 and IE 6.0.2800.1106.xpsp1.020828-1920 and all they work without any error. Clicking on the stars shows ..../fivestar/vote/node/[nid]/[rating] in my apache logs and the database changes accordingly, so I assume I got lucky and fixed it. Here are the change and the patch:
old code:
new code:
Would be nice if someone who understands jQuery can verify this...
Comment #10
mr700 CreditAttribution: mr700 commentedIf the previous two patches are ok, here's one that includes them and also removes the extra spaces and makes the code indentation match drupal coding standards (maybe just attaching the new file was a better idea).
Comment #11
jonathan_hunt CreditAttribution: jonathan_hunt commentedI applied these patches manually. Fivestar now works ok on IE6 and IE7.
Comment #12
b_f CreditAttribution: b_f commentedYes, the two patches correct the problem. Thanks, Doncho.
Comment #13
quicksketchThanks mr700! I applied your IE flicker patch and committed to CVS. Unfortunately the second patch (removing $) breaks Firefox 2 (Mac) in my testing. There are like 6 issues open with this same problem, so one of them will probably work :D
Comment #14
quicksketchOkay, both patches were fantastic :)
I combined the second patch with http://drupal.org/node/132666, which properly sets the content type on the returned xml. Now jQuery gets an XMLObject (not html) returned in all browsers. Firefox had been compensating and reading it as XML, but oddly... IE was doing the right thing! First time for that one...
Thanks again mr700. Both are now applied and will be in the 1.5 release.
Comment #15
mr700 CreditAttribution: mr700 commentedThank you too.
Comment #16
Dries CreditAttribution: Dries commented