Bug 6116

Summary: Adding child patron: library not set
Product: Koha Reporter: Julian Maurice <julian.maurice>
Component: PatronsAssignee: Julian Maurice <julian.maurice>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: minor    
Priority: PATCH-Sent (DO NOT USE) CC: chris, gmcharlt, nengard, paul.poulain
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Text to go in the release notes:
Version(s) released in:
Attachments: Patch
Bug 6116: Adding a child patron
Signed-off patch
proposed patch (passed QA)

Description Julian Maurice 2011-04-07 13:46:13 UTC
When adding a patron who is a child, the default library is not set when defining a guarantor. Coming patch will fix that.
Comment 1 Julian Maurice 2011-04-07 13:49:02 UTC Comment hidden (obsolete)
Comment 2 Nicole C. Engard 2011-05-13 19:43:20 UTC
Apply? [y]es/[n]o/[e]dit/[v]iew patch/[a]ccept all y
Applying: Bug 6116: Adding a child patron
Using index info to reconstruct a base tree...
Falling back to patching base and 3-way merge...
Renaming koha-tmpl/intranet-tmpl/prog/en/modules/members/guarantor_search.tmpl => koha-tmpl/intranet-tmpl/prog/en/modules/members/guarantor_search.tt
Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/members/guarantor_search.tt
CONFLICT (rename/modify): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/members/guarantor_search.tt
Auto-merging members/memberentry.pl
CONFLICT (content): Merge conflict in members/memberentry.pl
Failed to merge in the changes.
Patch failed at 0001 Bug 6116: Adding a child patron
When you have resolved this problem run "git am --resolved".
If you would prefer to skip this patch, instead run "git am --skip".
To restore the original branch and stop patching run "git am --abort".
Comment 3 Chris Cormack 2011-07-08 03:25:19 UTC Comment hidden (obsolete)
Comment 4 Chris Cormack 2011-07-08 03:25:43 UTC
Rebased for TT
Comment 5 Owen Leonard 2011-07-08 13:01:54 UTC Comment hidden (obsolete)
Comment 6 Paul Poulain 2011-08-10 10:14:30 UTC
Created attachment 4924 [details] [review]
proposed patch (passed QA)

QA comment

* this patch fixes a bug by adding some code that is similar to other fields managed the same way
* works as expected & announced
* silently fixes a uc/lc in a syspref borrowerRelationship, written BorrowerRelationship previously, that's correct.
* silently add '' in hashes $data{'branchcode'} written $data{branchcode} previously, that's correct too

Marking Passed QA
Comment 7 Chris Cormack 2011-08-16 22:27:29 UTC
Pushed, please test
Comment 8 Julian Maurice 2011-12-02 12:54:24 UTC
Ok in master, marking as resolved