Provide a backport to Drupal 7 of the base_form_id targeting functionality introduced in #2839891: Allow to target the base_form_id

CommentFileSizeAuthor
#5 captcha-allow_base_form_id_captcha_point-2955343-5.patch2.69 KBpvdjay
#2 captcha-allow_base_form_id_captcha_point-2955343-2.patch2.62 KBpvdjay

Comments

jayusa123 created an issue. See original summary.

pvdjay’s picture

pvdjay commented
Status: Active » Needs review
StatusFileSize
new captcha-allow_base_form_id_captcha_point-2955343-2.patch2.62 KB

Status: Needs review » Needs work

The last submitted patch, 2: captcha-allow_base_form_id_captcha_point-2955343-2.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

alexgreyhead’s picture

alexgreyhead commented

Subscribing - this would be very helpful to have :)

pvdjay’s picture

pvdjay commented
Status: Needs work » Needs review
StatusFileSize
new captcha-allow_base_form_id_captcha_point-2955343-5.patch2.69 KB

Fixed bug identified in automated test.

chris matthews’s picture

chris matthews commented
Issue summary: View changes
chris matthews’s picture

chris matthews commented
Assigned: pvdjay » Unassigned
Status: Needs review » Reviewed & tested by the community

The patch in #5 applied cleanly to the latest captcha 7.x-1.x-dev and works great for me.

chris matthews’s picture

chris matthews commented
Parent issue: » #3032542: Plan for CAPTCHA 7.x-1.6 release
wundo’s picture

wundo commented
Parent issue: #3032542: Plan for CAPTCHA 7.x-1.6 release » #3086055: Plan for CAPTCHA 7.x-1.7 release
wundo’s picture

wundo commented
wundo’s picture

wundo commented
Status: Reviewed & tested by the community » Fixed

  • wundo committed 5f1b2ca on 7.x-1.x authored by pvdjay 
    Issue #2955343 by pvdjay, Chris Matthews, wundo, alexharries: Backport...

Status: Fixed » Closed (fixed)

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