|
Description
Sara Brown
2025-10-21 20:24:24 UTC
On further testing, it appears that 'OPAC mandatory' only works if 'Editable in OPAC' is also selected, even for account modifications. Example: -Create an attribute with 'OPAC mandatory' selected and with 'Editable in OPAC' not selected -Create a self-registered account - note that the attribute doesn't show on the self-reg form, but the account is created successfully -View the patron account from the staff interface and confirm that the attribute is null -Log into your new OPAC account - note that attribute doesn't show for editing -Change some non-attribute information in your OPAC account, and submit changes. Note that the changes save normally. Given this, I'd amend my first comment to say that the 'OPAC mandatory' attribute setting should have an added note indicating that it's only relevant if the attribute is editable in the OPAC. Created attachment 188324 [details] [review] Bug 41067: Disable OPAC mandatory when not editable If a patron attribute is not editable at the OPAC it should not be possible to make it mandatory. Test plan: Create a new patron attribute and confirm that checkboxes behaviour is consistent. Created attachment 188420 [details] [review] Bug 41067: Disable OPAC mandatory when not editable If a patron attribute is not editable at the OPAC it should not be possible to make it mandatory. Test plan: Create a new patron attribute and confirm that checkboxes behaviour is consistent. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Created attachment 188421 [details] [review] Bug 41067: (follow-up) Change hint to include 'Editable' requirement Created attachment 188657 [details] [review] Bug 41067: Disable OPAC mandatory when not editable If a patron attribute is not editable at the OPAC it should not be possible to make it mandatory. Test plan: Create a new patron attribute and confirm that checkboxes behaviour is consistent. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Laura_Escamilla <laura.escamilla@bywatersolutions.com> Created attachment 188658 [details] [review] Bug 41067: (follow-up) Change hint to include 'Editable' requirement Signed-off-by: Laura_Escamilla <laura.escamilla@bywatersolutions.com> Nice work everyone! Pushed to main for 25.11 |