Created attachment 113307 [details] screenshots and irc chat In the Koha Version 20.05.04 the extendedPatronAttributes not showing during selfregistration. I created a PatronAttribute "Matrikelnummer" which is configured to be shown and editable in OPAC. It also is searchable. The attribut is set for all user and bibtypes. PatronSelfRegistrationVerifyByEmail is disabled extendedPatronAttributes is enabled I tested the same configuration on another server with koha version 20.05.01. The field "Matrikelnummer" is shown during the selfregistration process and the entered values are set correctly in the mysql database and userdetails.
Created attachment 113366 [details] [review] Bug 26973: (bug 26119 follow-up) Restore patron's attributes during self-registration The assumption on bug 26119 was wrong. We actually store patron's attributes during self-registration if PatronSelfRegistrationVerifyByEmail is not enabled. Test plan: Test the different combinations of PatronSelfRegistration and PatronSelfRegistrationVerifyByEmail Confirm that the patron's attributes appears during self-registration if PatronSelfRegistrationVerifyByEmail is not set. Confirm that the patron's attributes are stored when they are displayed on the self-registration form Confirm that you can edit the patron's attributes in any cases (if they are marked as editable at the OPAC)
With PatronSelfRegistrationVerifyByEmail set to 'Don't Require': - Can set the patron attributes on the self-registration form - Can edit the patron attributes in 'your personal details' - If the patron attribute is set to display but not edit, you can view it but not change it - If the patron attribute is set to display but not edit, it correctly doesn't show on the self-registration form The only issue I can see is that all Additional attributes and identifiers are submitted, even if they're not changed. To replicate: - Set one or two patron attributes to editable in the OPAC - Log in as a patron on the OPAC - Go to 'your personal details' - opac-memberentry.pl - Change the salutation and do not modify anything else - Submit - Go to the staff client - View 'Patrons requesting modifications' - members/members-update.pl - Note that the changed salutation field is included, but so are the patron attribute fields, despite no edits being made to the Additional attributes and identifiers With PatronSelfRegistrationVerifyByEmail set to 'Require': - The patron attributes are not visible on the self-registration form - Existing users can edit the patron attributes in 'your personal details' - If the patron attribute is set to display but not edit, you can view it but not change it However, trying to sign up a new account gives an error - not sure if this is a sandbox configuration issue - but the error is: Invalid 'from' parameter: root@localhost at /usr/share/perl5/Exception/Class/Base.pm line 88
Thanks for testing, Sally! (In reply to Sally from comment #2) > The only issue I can see is that all Additional attributes and identifiers > are submitted, even if they're not changed. To replicate: > > - Set one or two patron attributes to editable in the OPAC > - Log in as a patron on the OPAC > - Go to 'your personal details' - opac-memberentry.pl > - Change the salutation and do not modify anything else > - Submit > - Go to the staff client > - View 'Patrons requesting modifications' - members/members-update.pl > - Note that the changed salutation field is included, but so are the patron > attribute fields, despite no edits being made to the Additional attributes > and identifiers But then this problem exists without this patch as well, right? > However, trying to sign up a new account gives an error - not sure if this > is a sandbox configuration issue - but the error is: Invalid 'from' > parameter: root@localhost at /usr/share/perl5/Exception/Class/Base.pm line 88 Yes, it's because the syspref KohaAdminEmailAddress is set to an invalid email address (root@localhost), you can try with "test@example.org".
Created attachment 113462 [details] [review] Bug 26973: (bug 26119 follow-up) Restore patron's attributes during self-registration The assumption on bug 26119 was wrong. We actually store patron's attributes during self-registration if PatronSelfRegistrationVerifyByEmail is not enabled. Test plan: Test the different combinations of PatronSelfRegistration and PatronSelfRegistrationVerifyByEmail Confirm that the patron's attributes appears during self-registration if PatronSelfRegistrationVerifyByEmail is not set. Confirm that the patron's attributes are stored when they are displayed on the self-registration form Confirm that you can edit the patron's attributes in any cases (if they are marked as editable at the OPAC) Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk>
> But then this problem exists without this patch as well, right? Ah, so it does - sorry, I don't know how I missed that (doing too much at once again). > Yes, it's because the syspref KohaAdminEmailAddress is set to an invalid > email address (root@localhost), you can try with "test@example.org". Thank you! I've signed the patch off. :)
*** Bug 27020 has been marked as a duplicate of this bug. ***
Created attachment 113663 [details] [review] Bug 26973: (bug 26119 follow-up) Restore patron's attributes during self-registration The assumption on bug 26119 was wrong. We actually store patron's attributes during self-registration if PatronSelfRegistrationVerifyByEmail is not enabled. Test plan: Test the different combinations of PatronSelfRegistration and PatronSelfRegistrationVerifyByEmail Confirm that the patron's attributes appears during self-registration if PatronSelfRegistrationVerifyByEmail is not set. Confirm that the patron's attributes are stored when they are displayed on the self-registration form Confirm that you can edit the patron's attributes in any cases (if they are marked as editable at the OPAC) Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Works as expected, no regressions found. Passing QA
Pushed to master for 20.11, thanks to everybody involved!
this patchset does not apply cleanly to 20.05.x, but it is needed.
Created attachment 113737 [details] [review] Bug 26973: [20.05.x] (bug 26119 follow-up) Restore patron's attributes during self-registration The assumption on bug 26119 was wrong. We actually store patron's attributes during self-registration if PatronSelfRegistrationVerifyByEmail is not enabled. Test plan: Test the different combinations of PatronSelfRegistration and PatronSelfRegistrationVerifyByEmail Confirm that the patron's attributes appears during self-registration if PatronSelfRegistrationVerifyByEmail is not set. Confirm that the patron's attributes are stored when they are displayed on the self-registration form Confirm that you can edit the patron's attributes in any cases (if they are marked as editable at the OPAC)
Thanks Jonathan, backported to 20.05.x for 20.05.06
backported to 19.11.x for 19.11.12
Missing dependencies for 19.05.x so bug shouldn't affect it. No backporting.
I find it a bit weird that patron attributes aren't shown when PatronSelfRegistrationVerifyByEmail is enabled. Surely most Koha instances should have PatronSelfRegistrationVerifyByEmail enabled... I suppose that's more of an enhancement though...
(In reply to David Cook from comment #15) > I find it a bit weird that patron attributes aren't shown when > PatronSelfRegistrationVerifyByEmail is enabled. > > Surely most Koha instances should have PatronSelfRegistrationVerifyByEmail > enabled... > > I suppose that's more of an enhancement though... Although it looks like borrower_modifications can support extended_attributes...
Ah hence bug 30120. Never mind...