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.
The patch on https://www.drupal.org/node/1395184 checks whether $form_state->getTriggeringElement()['#limit_validation_errors'] is not empty. This will be an empty array if the ajax call doesn't want any validation to occur (https://api.drupal.org/api/drupal/developer!topics!forms_api_reference.h...)
Instead it should check if it is an array.
Comment | File | Size | Author |
---|---|---|---|
#9 | captcha-session-reuse-attack-detected-2814137-9-D8.patch | 999 bytes | B N Pandey |
#4 | captcha-session-reuse-attack-detected-2814137-4-D8.patch | 1.24 KB | B N Pandey |
#2 | ajax_in_forms_still-2814137-2.patch | 853 bytes | arknoll |
|
Comments
Comment #2
arknoll CreditAttribution: arknoll commentedComment #3
sk33lz CreditAttribution: sk33lz at Zivtech commented#2 applies cleanly and fixes the error even on 8.x-1.0-alpha1. I think this is RTBC in my book.
Comment #4
B N Pandey CreditAttribution: B N Pandey at TO THE NEW commentedI fixed my issue in a different way.
Providing patch here.
Comment #6
Sam152 CreditAttribution: Sam152 as a volunteer and at PreviousNext commentedAs #4 points out, it seems strange to me there is a db_update which is never executed. This was cropping up for me when click "Remove" on an inline_entity_form entity on a registration page. #2 does indeed solve the problem for me, so +1 to RTBC.
Comment #7
wundo CreditAttribution: wundo at Chuva Inc. for Chuva Inc. commentedMy impression is that we should commit #2 not #4.
Can we get the test passing before committing this please? Looks like the patch needs a re-roll.
Comment #8
BerdirAgreed, #2 worked for us. It is also still still passing, I'll request another test after posting this to be sure.
Comment #9
B N Pandey CreditAttribution: B N Pandey at TO THE NEW commentedI got this error when I submit registration form. I applied the patch provided in comment #2 but it does not work for me. So I fixed my issue in a new way. Please review it.
Comment #11
elachlan CreditAttribution: elachlan commentedIs this a duplicate of #810534: Fix CAPTCHA session reuse?
Comment #12
B N Pandey CreditAttribution: B N Pandey at TO THE NEW commentedHi @elachlan,
May be both are related because issue in both the case are same.
Comment #13
elachlan CreditAttribution: elachlan commentedCould you re-roll the patch?
Comment #14
B N Pandey CreditAttribution: B N Pandey at TO THE NEW commentedHi @elachlan,
Please explain how to re-roll path and why it's required?
Comment #16
BerdirLooks like this was committed but not closed?