When the captcha has been added to the form by Captcha Configuration it produces a error by undefined jQuery preventing the click to get new captcha.

Proposed solution add the core/jquery library as dependency of library image_captcha.image-captcha-refresh.

Issue fork captcha-3110283

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joao.ramos created an issue. See original summary.

Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody at DROWL.de commented
Version: 8.x-1.0-beta4 » 8.x-1.x-dev
Status: Active » Needs work

Indeed this is still valid.

Anybody opened merge request !14

Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody at DROWL.de commented
Status: Needs work » Needs review
Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody at DROWL.de commented
Assigned: joao.ramos.costa » Grevil

@Grevil could you please review this? Afterwards we should merge this into 8.x-1.x and 2.x

Grevil’s picture

Grevil CreditAttribution: Grevil as a volunteer and at DROWL.de commented
Status: Needs review » Reviewed & tested by the community

LGTM! Although we should remove the jquery dependency in the future at some point! 😊

  • Anybody committed 8cf66b5 on 8.x-1.x 
    Issue #3110283: Add jQuery as dependency to image-captcha-refresh...
Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody at DROWL.de commented
Version: 8.x-1.x-dev » 2.x-dev
Assigned: Grevil » Unassigned
Status: Reviewed & tested by the community » Needs work

  • Anybody committed 0fee34b on 2.x 
    Issue #3110283 by Anybody, Grevil: Add jQuery as dependency to image-...
Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody at DROWL.de commented
Status: Needs work » Fixed
Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody at DROWL.de commented

I created #3318944: [2.x] Remove jQuery dependencies to remove jQuery from 2.x. Until that, the definition is clean now :)

Status: Fixed » Closed (fixed)

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