Hello,

maybe you have an idea how to solve the following problem:

- I have installed Google reCAPTCHA and Ajax Comments.
- I have activated Google reCAPTCHA for the comments form.
- Everything works fine - until f.e. i forget to fill a required field.
- The error massage is visible, but the Google reCAPTCHA form element is not shown again.

So maybe the reinitialisation together with the ajax reload is missing?

Should i ask this also in the Ajax Comments issue list?

Thanks and best regards
SB

CommentFileSizeAuthor
#13 google_recaptcha-ajax_support-2865266-13.patch2.89 KBfrakke
#12 google_recaptcha-ajax_support-2865266-12.patch2.59 KBfrakke
#11 google_recaptcha-ajax_support-2865266-11.patch2.24 KBsimeon157
#9 google_recaptch-ajax_support-2865266-9.patch2.22 KBnikolabintev
#4 google_recaptcha_ajax_support.patch2.68 KBJonasSavs
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sachbearbeiter created an issue. See original summary.

sergey-serov’s picture

sergey-serov
Primary language Russian
Location Moscow
CreditAttribution: sergey-serov as a volunteer and commented
Assigned: Unassigned » sergey-serov

Greetings, sachbearbeiter!
I suppose the problem is in the rebuilding process.
This very like the situation with the form with image file upload field (which use ajax for generating image preview).
It is necessary to look in details on the process of form building when #rebuild == TRUE.
I will debug this case with Ajax Comment, but not promise that at the nearest time :)

sachbearbeiter’s picture

sachbearbeiter CreditAttribution: sachbearbeiter commented

Thanks a lot for the feedback ...
If we find some solution i let you know ;)

JonasSavs’s picture

JonasSavs CreditAttribution: JonasSavs as a volunteer and commented
FileSize
google_recaptcha_ajax_support.patch2.68 KB

Hi Guys,

I did get this issue too.

Resolved and uploading a patch.

Hope it helps.

Regards,
Nelson

sergey-serov’s picture

sergey-serov
Primary language Russian
Location Moscow
CreditAttribution: sergey-serov as a volunteer and commented

Hi Nelson!
Great!!
Sure, this way more appropriate for adding js and it may resolve some other troubles too.

sachbearbeiter’s picture

sachbearbeiter CreditAttribution: sachbearbeiter commented

@Nelson: Thanks a lot ...
I will test ...

sachbearbeiter’s picture

sachbearbeiter CreditAttribution: sachbearbeiter commented

Seems to work - super ;)

sachbearbeiter’s picture

sachbearbeiter CreditAttribution: sachbearbeiter commented

@Nelson
I have a list of node in an views teaser list - the captcha is only shown below the first node ...
Could be an Ajax Comments problem - we are investigating ...

nikolabintev’s picture

nikolabintev CreditAttribution: nikolabintev commented
FileSize
google_recaptch-ajax_support-2865266-9.patch2.22 KB

Hello guys,

#4 works for me, but the things is that the patch should be created from the project directory, but this one is created from 'sites' directory and cannot be applied correctly. That's why I've applied it and recreated it from google_recaptcha directory.

nikolabintev’s picture

nikolabintev CreditAttribution: nikolabintev commented
Status: Active » Needs review
simeon157’s picture

simeon157 CreditAttribution: simeon157 at FFW commented
FileSize
google_recaptcha-ajax_support-2865266-11.patch2.24 KB

There is an issue with the recaptcha where sometimes the grecaptcha.render function is undefined. I've added a check for that.

frakke’s picture

frakke CreditAttribution: frakke at Adapt commented
FileSize
google_recaptcha-ajax_support-2865266-12.patch2.59 KB

The recaptcha form can't be rendered once it's already rendered. This results in Js error, so we reset the form instead when it's already rendered after an ajax submit.

frakke’s picture

frakke CreditAttribution: frakke at Adapt commented
FileSize
google_recaptcha-ajax_support-2865266-13.patch2.89 KB

The recaptcha isn’t always rendered, so we wrap the reset in a try/catch and render it again if it throws an error.