]> git.koha-community.org Git - koha.git/commit
Bug 13602 - Username/password already exists when editing borrower attributes and...
authorOlli-Antti Kivilahti <olli-antti.kivilahti@jns.fi>
Tue, 20 Jan 2015 14:48:13 +0000 (16:48 +0200)
committerChris Cormack <chrisc@catalyst.net.nz>
Thu, 12 Feb 2015 21:42:29 +0000 (10:42 +1300)
commit6f51d3b31b690415f9fc9bff6477d93701628b83
tree62964c928bd2bd906bde998f7e17cda40dee0d64
parentd31c4264552fd1fe94ca97737e0fe6365a09cf09
Bug 13602 - Username/password already exists when editing borrower attributes and messaging preferences

When editing a Patron's messaging preferences or borrower attributes, one cannot
save changes because 'userid' is not given as a parameter.

It shouldn't, but the
Check_Userid($newdata{'userid'},$borrowernumber) -check must have it.

REPLICATE THE ISSUE:
1. Find a Borrower.
2. Go to the details-tab.
3. Under "Patron messaging preferences" click edit to edit just the messaging
   preferences.
4. Just save, no need to make any modifications.
5. Observe the error "Username/password already exists"
   This happens to all/most of the sub-forms in the Borrower's Details-view.

AFTER THIS PATCH:
Check_Userid() get the $userid from the new parameters, or if not present, from
the koha.borrowers-table.

TEST FOR REGRESSION:
1. Edit a borrower normally. Notice no errors.
2. Edit a sub-form (Alternate address/Patron messaging preferences). Notice no
   errors.

DISCLAIMER: You are free to steal/rip/modify/hack this patch to your soul's
extent without my permission. As long as it makes to Koha I am happy.

Works as expected.
Signed-off-by: Marc VĂ©ron <veron@veron.ch>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
(cherry picked from commit 58f7f56b8664810335f60f3fe5e43faa079f33bc)
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
members/memberentry.pl