Summary: | Merging Authorities | ||
---|---|---|---|
Product: | Koha | Reporter: | Chris Cormack <chris> |
Component: | MARC Authority data support | Assignee: | Matthias Meusburger <matthias.meusburger> |
Status: | RESOLVED DUPLICATE | QA Contact: | Bugs List <koha-bugs> |
Severity: | enhancement | ||
Priority: | PATCH-Sent (DO NOT USE) | CC: | koha.sekjal, m.de.rooy, martin.renvoize, nengard, stephane.delaune |
Version: | 3.4 | ||
Hardware: | All | ||
OS: | All | ||
See Also: | http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5572 | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Bug Depends on: | |||
Bug Blocks: | 5581 | ||
Attachments: | Proposed patch |
Description
Chris Cormack
2011-03-24 02:18:02 UTC
Created attachment 3419 [details] [review] Proposed patch This patch conflicts with patch bugs 5572 and 5657. It just changes the sub merge C4::AuthoritiesMarc::merge which is only used by function C4::AuthoritiesMarc::ModAuthority if (C4::Context->preference('MergeAuthoritiesOnUpdate') ). This syspref does not exist by default. So it makes me doubt the relevance of this patch (In reply to comment #2) > This patch conflicts with patch bugs 5572 and 5657. It just changes the sub > merge C4::AuthoritiesMarc::merge which is only used by function > C4::AuthoritiesMarc::ModAuthority if > (C4::Context->preference('MergeAuthoritiesOnUpdate') ). This syspref does not > exist by default. So it makes me doubt the relevance of this patch This new pref was not introduced correctly. And should be based on value of the old pref dontmerge. This sounds like you're all saying this patch has failed QA and needs to be resubmitted so I'm updating the status. This is still desired - can we get a new patch? |