Hello,

I'm working on an upgraded d7 site with a new theme and a dropdown jquery megamenu.... (from the arctica - lumi theme http://drupal.org/node/1913540)

Anyhow, with recaptcha enabled on any page the entire drop down menu shows as expanded (i.e. fills the page - screen shot at other post above)...

With math captcha all is well.

I'm assuming this is some sort of jquery conflict, but unfortunately that is out of my experience level...

Any help would be appreciated!
-Chad.

CommentFileSizeAuthor
#7 Screen Shot 2013-03-10 at 3.37.27 PM.png98.56 KBhswong3i
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Chad_Dupuis’s picture

Chad_Dupuis CreditAttribution: Chad_Dupuis commented

Uploaded the latest dev of recaptcha (which fixed one error I was seeing) and then installed jquery_update and all is well except for some strange conflict with ckeditor....

Pages with ckeditor and recaptcha installed get a blank ckeditor
Pages without recaptcha or with a math captcha work fine...

The error I see is:
TypeError: Drupal.settings.ckeditor is undefined
[Break On This Error]

window.CKEDITOR_BASEPATH = Drupal.settings.ckeditor.editor_path;

But without recaptcha it works fine .... also with js aggregation off it works fine (not the setting in the ckeditor module, but with caching and aggregation shut off within aegir/omega8 hosting)?

Chad_Dupuis’s picture

Chad_Dupuis CreditAttribution: Chad_Dupuis commented

This appears to be just a ckeditor issue... although proper functioning of recaptcha on my site at least seemed to require jquery_update and the latest dev, so I'll leave this open for that reason...

Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland commented
Title: jquery conflict? with dropdown menu... » jQuery update may be required
Version: 7.x-1.9 » 7.x-1.x-dev
Category: support » bug

The only substantive change between reCAPTCHA 1.9 (latest) and -dev is #1826404: ReCaptchaState undefined, skip comment approval. This changes which of Google's JS files is used. That change might be what fixes your main problem.

What remains is a possible bug report. Does anyone else find they need jQuery Update? If so, I will make that a dependency of this module. Everyone: Please let me know whether or not reCAPTCHA works for you with and without jQuery update.

Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland commented
Status: Active » Fixed

I have noted the possible need for jQuery Update on the module page. It does not seem to be required by everyone, so I will not make it a module dependency.

RobLoach’s picture

RobLoach
he/him
CreditAttribution: RobLoach commented

Thanks Liam!

tstermitz’s picture

tstermitz
Location Colorado
CreditAttribution: tstermitz commented

I have seen a similar issue where reCaptcha was conflicting with my javascript validation scrpits on the entityforms module. When I'm logged in as admin, my validation script worked fine, but not for anonymous users. Changing to the math Captcha made my scripts work again.

And, I haven't been using jQuery update.

Ahah. Downloading jQuery Update, and enabling it... now my scripts function.

hswong3i’s picture

hswong3i CreditAttribution: hswong3i commented
Status: Fixed » Active
FileSize
Screen Shot 2013-03-10 at 3.37.27 PM.png98.56 KB

For my case recaptcha-7.x-1.x conflict with http://drupal.org/project/prettify 7.x-1.0-beta1:

  • When activate recaptcha but browsing blog homepage (which without comment box, i.e. no recaptcha box show up, too), node teaser view with syntax highlight
  • When activate recaptcha, no syntax highlight for anonymous during full node view
  • When replace recaptcha with math captcha, syntax highligh works for anonymous during full node view
  • When activate recaptcha + jquery_update-7.x-2.3, syntax highligh again works for anonymous during full node view

Most likely confirm that latest recaptcha REQUIRE jquery_update for complete and expected compatibility with else modules :-(

P.S. I also find that when check with Firebug (recaptcha-7.x-1.x-dev on, prettify on, jquery_update off), following error message show up:

j is undefined

for jquery.js?v=1.4.4 (line 27)

Screenshot:
Firebug error message

Shall this file as another bug report, or we just simply include jquery_update into dependency so get simple fix?

Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland commented

It already says on the project page that jQuery update may be needed. I don't want to make it a hard dependency because there may be people who don't need or want to use it. I will add a note about this to the README.

Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland commented
Status: Active » Fixed

http://drupalcode.org/project/recaptcha.git/commitdiff/2ac2b51

Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Fixed » Patch (to be ported)

  • Liam Morland committed 2ac2b51 on 7.x-1.x, 7.x-2.x 
    Issue #1913556: Note that jQuery update may be required in README.

  • Liam Morland committed 2ac2b51 on 8.x-2.x 
    Issue #1913556: Note that jQuery update may be required in README.

  • Liam Morland committed 2ac2b51 on 6.x-2.x 
    Issue #1913556: Note that jQuery update may be required in README.
hass’s picture

hass CreditAttribution: hass commented
Issue summary: View changes

Can someone of you try 2.x branches and confirm if these also require jquery_update or make this obsolete, please?

hass’s picture

hass CreditAttribution: hass commented
Status: Patch (to be ported) » Fixed

  • hass committed edba33d on 6.x-1.x authored by Liam Morland 
    Issue #1913556 by Liam Morland: jQuery update may be required

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.