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.
Hola,
Strange problem here, the 'Submit' button only on the modr8 queue page only appears to work for people visiting our site via Firefox; while the button does show for IE users, and has the little 'click' animation when clicked, nothing actually happens.
Any ideas?
Cheers,
Dinis
Comment | File | Size | Author |
---|---|---|---|
#10 | fivestar-5.x-1.11_modr8_form_teaser.patch | 555 bytes | davej |
Comments
Comment #1
pwolanin CreditAttribution: pwolanin commentedodd- though this could be a core Forms API bug - what version of Drupal do you have running? Do other forms on your site work right in IE7? Is there another form on the same page as the modr8 form?
Comment #2
Dinis CreditAttribution: Dinis commentedThe site is running on Drupal 5.7 though this started a while ago (5.5 iirc). I've built a profile for testing with no blocks or forms visible or running etc. so the only button on the page is the Modr8 submit, and even then it's still the same.
It's the only page / button on the entire site I can find which has this odd bug too so It's quite a mystery tbh.
To be thorough, I've tried the same thing with Safari and it's fine on that browser too.
Comment #3
pwolanin CreditAttribution: pwolanin commentedPerhaps this is the same problem? http://drupal.org/node/189553
Comment #4
Dinis CreditAttribution: Dinis commentedInteresting, they sound like identical symptoms with the only difference being I don't use Activeedit.
The submit button used to work perfectly, so maybe I'll try disabling mod by mod on my test server and see if the functionality returns at some point.
Comment #5
Dinis CreditAttribution: Dinis commentedI've tracked down the conflicting module; it's with the 5 Star mod - http://drupal.org/project/fivestar
I'm unable to work out exactly why it's failing only on IE, but what ever it is, if Fivestar is enabled, the Modr8 submit button ceases to function.
Comment #6
pwolanin CreditAttribution: pwolanin commentedok, well maybe Eaton can tell us if there is a conflicting form_alter or some such.
Comment #7
quicksketchHi Dinis, that is indeed a strange symptom. Fivestar doesn't do anything with buttons on the page other than hide the ones for fivestar widgets, in which case they're not visible at all. Are there fivestar widgets being displayed on the page? Also check if jquery.rating.js is loaded on the page and if the form submits properly if javascript is disabled.
Comment #8
Dinis CreditAttribution: Dinis commentedHowdy, thanks for the quick response.
I've tested as requested and can confirm that the Fivestar widgets display on the Modr8 submission page.
There is a temporary workaround in the Fivestar options in Content Types; Selecting the "hidden" option for teasers for any content which needs to be Modr8ed prevents the Fivestar widget from displaying on the Modr8 page.
Keep up the good work with these modules guys :)
Comment #9
davej CreditAttribution: davej commentedWhen the teaser list at admin/content/modr8 contains Fivestar widgets, this means there are forms nested within the modr8 form. So I suspect that IE is being tripped up by this invalid HTML structure. If this is the cause then the solution involves not displaying Fivestar widgets in contexts where they are within an existing form.
Comment #10
davej CreditAttribution: davej commentedLuckily modr8 sets a flag on nodes displayed in modr8_form, so there's a simple fix to hide fivestar widgets in this form. Patch attached.
Comment #11
Dinis CreditAttribution: Dinis commentedMany thanks,
I'm going with a new production build tonight so too late to make this one, but I'll add it to the test server on Monday (all going well over the weekend) and let you know how it works out.
Cheers again for the support and these terrific mods.
Danielle
Comment #12
pwolanin CreditAttribution: pwolanin commentedI'll have to look - maybe modr8 itself should be making sure that
$node->in_preview == TRUE
Comment #13
pwolanin CreditAttribution: pwolanin commentedoh, wait - it's not in preview, it's a teaser.
Also, perhaps a broader fix would invovle changing the name of the modr8 submit button? How doe Fivestar decide which button(s) to hide?
Comment #14
quicksketchI believe davej was correct in his assessment. The forms inside of forms bug seems to be keeping IE7 from operating correctly. Even with JavaScript disabled the modr8 form still won't submit. Given the options, I think hiding the widget entirely is about as good as we can get. I've applied the patch in #10, slightly tweaked to use isset() so we can avoid unnecessary PHP notices.
Comment #15
pwolanin CreditAttribution: pwolanin commented@quicksketch - maybe we need to think about a core patch? For example, we could at least suppress the building of an additional form if we are in the middle of a different form?
Comment #16
Dinis CreditAttribution: Dinis commented@pwolanin - This is something I would like to see too :)
Comment #17
Anonymous (not verified) CreditAttribution: Anonymous commentedAutomatically closed -- issue fixed for two weeks with no activity.