Testing bug 30120 I noticed that when you lave a mandatory extended patron attribute (or possibly another mandatory field) empty, all values already entered in extended attributes will be removed when clicking submit. To test: - Activate patron self registration without email verification - Create several patron attributes as visible and editable in the OPAC, make one mandatory - Register as a new patron from the OPAC - Fill in all required fields but the extended attribute - Submit - Verify that the contents of the other extended attribute fields have been removed and you are forced to re-enter
Created attachment 167170 [details] [review] Bug 30372: Fix value->attribute Test plan: - Activate patron self registration without email verification - Create several patron attributes as visible and editable in the OPAC, make one mandatory - Register as a new patron from the OPAC - Fill in all required fields but the extended attribute - Fill in at least one of the non-required extended attributes - Submit - Verify that the contents of the other extended attribute fields are still present.
Created attachment 167177 [details] [review] Bug 30372: Fix value->attribute Test plan: - Activate patron self registration without email verification - Create several patron attributes as visible and editable in the OPAC, make one mandatory - Register as a new patron from the OPAC - Fill in all required fields but the extended attribute - Fill in at least one of the non-required extended attributes - Submit - Verify that the contents of the other extended attribute fields are still present. Signed-off-by: David Nind <david@davidnind.com>
Created attachment 168112 [details] [review] Bug 30372: Fix value->attribute Test plan: - Activate patron self registration without email verification - Create several patron attributes as visible and editable in the OPAC, make one mandatory - Register as a new patron from the OPAC - Fill in all required fields but the extended attribute - Fill in at least one of the non-required extended attributes - Submit - Verify that the contents of the other extended attribute fields are still present. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Pushed for 24.11! Well done everyone, thank you!
Backported to 24.05.x for upcoming 24.05.02
Pushed to 23.11.x for 23.11.07
Not backporting to 23.05.x unless requested