If an attribute has 'OPAC mandatory' selected but does not have 'Editable in OPAC' checked, the attribute will not appear on the self-registration form. Patrons can still successfully self-register, however. To make this clearer, the description for 'OPAC mandatory' should be updated to remove 'creating or' from 'Check to make this attribute mandatory when creating or editing a patron from the OPAC', *or* a note should be added indicating that 'Editable in OPAC' must also be checked for the attribute to be included on the self-reg form.
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