Summary: | Adding child patron: library not set | ||
---|---|---|---|
Product: | Koha | Reporter: | Julian Maurice <julian.maurice> |
Component: | Patrons | Assignee: | 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: | 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: |
Patch
Bug 6116: Adding a child patron Signed-off patch proposed patch (passed QA) |
Description
Julian Maurice
2011-04-07 13:46:13 UTC
Created attachment 3796 [details] [review] Patch 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". nengard@debian:~/kohaclone$ Created attachment 4591 [details] [review] Bug 6116: Adding a child patron When adding a patron who is a child, the default library is not set when defining a guarantor, this patch corrects the problem. More, for children with no guarantors, the default branch is the editing branch. BibLibre MT5795 Rebased for TT Created attachment 4603 [details] [review] Signed-off patch Before applying this patch using the "Set to Patron" button to add a guarantor would not change the patron's home library to that of the guarantor when filling in the entry form with the guarantor's information. After applying the patch the library is selected correctly. 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 Pushed, please test Ok in master, marking as resolved |