From OPAC, a patron cannot change twice his personal details, an error occurs : DBIx::Class::Storage::DBI::_dbh_execute(): Duplicate entry '51' for key 'PRIMARY' at /home/katrin/kohaclone/Koha/Borrower/Modifications.pm line 63 To reproduce : 1/ Login with a user in the opac 2/ Request some modification (surname for example) 3/ Repeat 2/
Created attachment 30972 [details] [review] Bug 12784: Patron cannot change his personal details This patch fixes the regression introduced by the bug 12623 : a patron cannot change his personal details from OPAC. Test plans : In order to test this patch, first you have to apply the patch of bug 12781 a1/ Login with a user in OPAC a2/ Request some modification (surname for example) a3/ Repeat 2/ a4/ There are no errors b1/ Register a new user from OPAC with the syspref : PatronSelfRegistrationVerifyByEmail with the value "Require" b2/ There are no errors
Created attachment 30992 [details] [review] [Signed-off] Bug 12784: Patron cannot change his personal details This patch fixes the regression introduced by the bug 12623 : a patron cannot change his personal details from OPAC. Test plans : In order to test this patch, first you have to apply the patch of bug 12781 a1/ Login with a user in OPAC a2/ Request some modification (surname for example) a3/ Repeat 2/ a4/ There are no errors b1/ Register a new user from OPAC with the syspref : PatronSelfRegistrationVerifyByEmail with the value "Require" b2/ There are no errors Followed test plan. Patch behaves as expected. Signed-off-by: Marc Véron <veron@veron.ch>
Created attachment 31036 [details] [review] [PASSED QA] Bug 12784: Patron cannot change his personal details This patch fixes the regression introduced by the bug 12623 : a patron cannot change his personal details from OPAC. Test plans : In order to test this patch, first you have to apply the patch of bug 12781 a1/ Login with a user in OPAC a2/ Request some modification (surname for example) a3/ Repeat 2/ a4/ There are no errors b1/ Register a new user from OPAC with the syspref : PatronSelfRegistrationVerifyByEmail with the value "Require" b2/ There are no errors Followed test plan. Patch behaves as expected. Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Patch works as expected, passes tests and QA script. Note: The second patron modification request of one patron will overwrite his/her first request.
Patch pushed to master. Thanks Yohann!