I see both message(set by username_enumeration_prevention) module and error message on user password reset form.


a_thakur created an issue. See original summary.

a_thakur’s picture

Status: Needs work » Closed (works as designed)

Closing works as designed.

Arnaud Kali’s picture

Status: Closed (works as designed) » Active

Re-opening as I think this is indeed a bug - the module is sending a successfully sent email message to the user even if there is a validation failure upon submission of the forgot password form (when there is a wrong captcha answer for example) this message should rather be sent upon successful submission of the form than always shown in the validation callback.

Arnaud Kali’s picture

Status: Active » Needs review
1.68 KB

Attaching a patch for review.

Arnaud Kali’s picture

Version: 7.x-1.1 » 7.x-1.2
Arnaud Kali’s picture

Arnaud Kali’s picture

Title: Both message and error appear on user password rest form » Both message and error appear on user password reset form
Issue summary: View changes
17.46 KB

Removing outdated image in favour of a more relevant one.

badjava’s picture

This patch no longer cleanly applies to 7.x-1.x. @Arnaud Kali do you mind re-rolling? Thank you.

Arnaud Kali’s picture

Hi @badjava - Thanks for your comment.
Patch re-rolled - see patch attached to this comment.

Arnaud Kali’s picture

arsn’s picture

Status: Needs review » Reviewed & tested by the community


I have had the same issue with validation failure on another element than a Captcha. Thanks for the 2587683-limit-error-message-to-be-shown-only-on-submit-instead-of-validation.patch of comment #10 which is working fine for me against version 7.x-1.2.

I am looking forward to see it merged in the near future!

Best regards

badjava’s picture

RTBC - we are running the patch from #10 and it fixes the issue.