Bug 5943

Summary: Merging Authorities
Product: Koha Reporter: Chris Cormack <chris>
Component: MARC Authority data supportAssignee: 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:
Bug Depends on:    
Bug Blocks: 5581    
Attachments: Proposed patch

Description Chris Cormack 2011-03-24 02:18:02 UTC

    
Comment 1 Chris Cormack 2011-03-24 02:23:05 UTC
Created attachment 3419 [details] [review]
Proposed patch
Comment 2 Stéphane Delaune 2011-04-06 13:12:51 UTC
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
Comment 3 Marcel de Rooy 2011-04-06 13:38:07 UTC
(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.
Comment 4 Nicole C. Engard 2011-04-18 18:06:54 UTC
This sounds like you're all saying this patch has failed QA and needs to be resubmitted so I'm updating the status.
Comment 5 Nicole C. Engard 2013-05-21 15:31:39 UTC
This is still desired - can we get a new patch?
Comment 6 Martin Renvoize 2014-05-08 13:46:50 UTC

*** This bug has been marked as a duplicate of bug 5202 ***