Self-registration DOB validation regex is too lax, should only allow '-' separator
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Evergreen |
Confirmed
|
Medium
|
Unassigned | ||
Bug Description
2.11 and 3.0.7
Steps to trigger:
In library settings, enable patron self-registration
Again, in library settings, show the DOB field on the form.
Go to the self-registration URL.
Notice that the example text indicates you can use "-" or "/
Complete the form and enter a birth date with "/" for the separator e.g. 1980/01/01
In the staff client (web or XUL) view the pending patrons
Note here that the date of birth for your patron is as you entered it.
Select your patron and either click Load Patron or double-click to complete the registration process.
Notice when the full registration screen loads so you can finish editing the patron information and save it, the DOB field is blank
To see the expected behavior, repeat the steps but use a DOB of 1980-01-01.
What I expected to happen:
Based on the fact that the DOB field accepts dates with "/", I expected the registration process to carry the birth date all the way through to the patron registration screen, like it does with "-".
It seems either the validation should limit the separator to "-" or if it not, perhaps replace the "/" with "-" so the date will work with the rest of the registration process.
| Changed in evergreen: | |
| status: | New → Confirmed |
| importance: | Undecided → Medium |
| tags: | added: patron |
| Terran McCanna (tmccanna) wrote | #1 |
| Elizabeth Davis (elidavis) wrote | #2 |
| Jason Stephenson (jstephenson) wrote | #3 |
| Stephanie Leary (stephanieleary) wrote | #4 |
| Stephanie Leary (stephanieleary) wrote | #5 |
| Galen Charlton (gmc) wrote | #6 |
See also:
https:/
and
https:/