Bug 6611

Summary: patron barcode removed when editing an attribute
Product: Koha Reporter: Nicole C. Engard <nengard>
Component: PatronsAssignee: Kyle M Hall <kyle.m.hall>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: critical    
Priority: PATCH-Sent (DO NOT USE) CC: chris, gmcharlt, jdeca, koha.sekjal, mcoalwell, wizzyrea
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 6611 : With the change to making cardnumber a nullable field, we need to pass it around when we are updating borrowers, to make sure we don't null it
[Signed Off] Bug 6611 : With the change to making cardnumber a nullable field, we need to pass it around when we are updating borrowers, to make sure we don't null it

Description Nicole C. Engard 2011-07-20 12:15:31 UTC
I am hoping that the below is actually a T:T issue:



This is a major problem- the patron barcode is deleted when an attribute is
edited. This only seems to happen when the attribute is edited by clicking on
the 'edit' link below the attribute box on the patron's 'details' page- not
when editing it through the edit button at the top.

Example- I just created a library card for a student, including entering her
barcode and then after saving the record, noticed that I could add her student
ID number to the attribute field. I clicked on the edit link below the
attribute box, added the ID number and saved the change. When her details page
popped back up, there was no longer a barcode. I double-checked this with my
own account (Student) and another staff account. We aren't sure if the barcode
is the only info that is deleted or if other fields are also deleted.
Comment 1 Liz Rea 2011-07-20 21:56:13 UTC
confirmed, this is happening in our production system at 3.05.00.002.
Comment 2 Chris Cormack 2011-07-20 21:56:41 UTC
Looking now
Comment 3 Chris Cormack 2011-07-20 22:22:26 UTC Comment hidden (obsolete)
Comment 4 Liz Rea 2011-07-21 13:58:47 UTC
Created attachment 4700 [details] [review]
[Signed Off] Bug 6611 : With the change to making cardnumber a nullable field, we need to pass it around when we are updating borrowers, to make sure we don't null it

Tested with every small edit link on moremember.pl.
Comment 5 Ian Walls 2011-07-26 19:15:40 UTC
Template only change, easy to read, marking as Passed QA.
Comment 6 Chris Cormack 2011-07-26 20:54:11 UTC
Pushed, please test
Comment 7 Liz Rea 2011-07-26 21:36:09 UTC
Works! closing as fixed. Thanks for the quick fix!
Comment 8 Ian Walls 2011-08-30 19:43:20 UTC
*** Bug 6788 has been marked as a duplicate of this bug. ***