Bug 6611 - patron barcode removed when editing an attribute
Summary: patron barcode removed when editing an attribute
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Patrons (show other bugs)
Version: Main
Hardware: All All
: PATCH-Sent (DO NOT USE) critical
Assignee: Kyle M Hall
QA Contact: Bugs List
URL:
Keywords:
: 6788 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-07-20 12:15 UTC by Nicole C. Engard
Modified: 2019-06-27 09:24 UTC (History)
6 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

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 (1.83 KB, patch)
2011-07-20 22:22 UTC, Chris Cormack 		Details | Diff | Splinter 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 (1.93 KB, patch)
2011-07-21 13:58 UTC, Liz Rea 		Details | Diff | Splinter Review
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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. ***