[Koha-bugs] [Bug 17908] Authority merge omnibus

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Sat Jan 14 14:08:29 CET 2017


https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17908

--- Comment #5 from Marcel de Rooy <m.de.rooy at rijksmuseum.nl> ---
This subject is touched in many reports and we have a lot of duplicated code.
The solutions under 5572/11315/11700 are more comprehensive and make merge
behavior pref controlled. Somehow we need to *merge* the best of code from
11315, 11700 and 5572 in order to get further.
I would suggest to copy tests from 11315/11700 and my own extended test
originally aimed for 9988 to report 17909 as a starting point to verify the
current merge routine.
On that we should build the core changes from 5572/11315/11700. (Include the
the $field->replace fix from 5572.) My suggestion for the pref name is
AuthorityMergeMode btw.

I checked the following reports and propose the following order to get some
progress:

[CLOSED] 03343 Duplicates
[CLOSED] 05821 Enable cron job
[CLOSED] 16802 Clearing subfields
[CLOSED] 12791 dontmerge is confusing
[CLOSED] 14583 Deleting subfields
[CLOSED] 13760 What about two 700s linking to the same authority

[START] 17909 Add tests for authority merge
[PHASE2] 05572 Improve C4::AuthoritiesMarc->merge() + syspref AuthCleanBiblios
[PHASE2] 11315 Subfield deletion (overwriteSubfieldsOnMerge)
[PHASE2] 11700 C4::AuthoritiesMarc::merge enhancements/fixes 
[PHASE3] 14769 Correct indicators in merge
[PHASE4/BLOCKED] 16018 Not properly merged (authorities/merge.pl) 
[PHASE5] 09988 AuthorityMergeLimit (instead of dontmerge) 
[PHASE6] 14026 Deleting an authority

Did I forget a report, or would you suggest a different order, please tell.

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list