Summary: | Patron password update validation broken | ||
---|---|---|---|
Product: | Koha | Reporter: | Owen Leonard <oleonard> |
Component: | Templates | Assignee: | Owen Leonard <oleonard> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | major | ||
Priority: | P5 - low | CC: | martin.renvoize, nick |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Bug Depends on: | 21964 | ||
Bug Blocks: | 22973 | ||
Attachments: |
Bug 22974: Patron password update validation broken
Bug 22974: Patron password update validation broken Bug 22974: Patron password update validation broken Bug 22974: Patron password update validation broken |
Description
Owen Leonard
2019-05-23 13:25:27 UTC
Created attachment 90030 [details] [review] Bug 22974: Patron password update validation broken This patch adds back an include file to the change password template which was inadvertently removed by Bug 21964. This broke Javascript validation of the new password fields. To test, apply the patch and enable the RequireStrongPassword system preference. - Go to Patrons -> Patron details -> Change password. - Try entering a weak password in the "New password" field. You should get a validation error: "Password must contain at least 3 characters, including UPPERCASE, lowercase and numbers." - Try entering a password in the second password field which doesn't match the first. You should get a validation error: "Please enter the same password as above." Created attachment 90043 [details] [review] Bug 22974: Patron password update validation broken This patch adds back an include file to the change password template which was inadvertently removed by Bug 21964. This broke Javascript validation of the new password fields. To test, apply the patch and enable the RequireStrongPassword system preference. - Go to Patrons -> Patron details -> Change password. - Try entering a weak password in the "New password" field. You should get a validation error: "Password must contain at least 3 characters, including UPPERCASE, lowercase and numbers." - Try entering a password in the second password field which doesn't match the first. You should get a validation error: "Please enter the same password as above." Signed-off-by: Liz Rea <wizzyrea@gmail.com> Created attachment 90080 [details] [review] Bug 22974: Patron password update validation broken This patch adds back an include file to the change password template which was inadvertently removed by Bug 21964. This broke Javascript validation of the new password fields. To test, apply the patch and enable the RequireStrongPassword system preference. - Go to Patrons -> Patron details -> Change password. - Try entering a weak password in the "New password" field. You should get a validation error: "Password must contain at least 3 characters, including UPPERCASE, lowercase and numbers." - Try entering a password in the second password field which doesn't match the first. You should get a validation error: "Please enter the same password as above." Signed-off-by: Liz Rea <wizzyrea@gmail.com> Signed-off-by: Bin Wen <bin.wen@inlibro.com> Created attachment 90099 [details] [review] Bug 22974: Patron password update validation broken This patch adds back an include file to the change password template which was inadvertently removed by Bug 21964. This broke Javascript validation of the new password fields. To test, apply the patch and enable the RequireStrongPassword system preference. - Go to Patrons -> Patron details -> Change password. - Try entering a weak password in the "New password" field. You should get a validation error: "Password must contain at least 3 characters, including UPPERCASE, lowercase and numbers." - Try entering a password in the second password field which doesn't match the first. You should get a validation error: "Please enter the same password as above." Signed-off-by: Liz Rea <wizzyrea@gmail.com> Signed-off-by: Bin Wen <bin.wen@inlibro.com> Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Awesome work all! Pushed to master for 19.05 Bug with enhancement not in 18.11.x series. |