February 2017 - Koha-bugs - lists.koha-community.org

[Bug 9988] Leave larger authority merges to merge_authority cronjob ( pref AuthorityMergeLimit)
by bugzilla-daemon＠bugs.koha-community.org 23 Feb '17

23 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9988 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This enhancement replaces release notes| |dontmerge by a limit. The | |Zebra code in merge is | |removed. The cron job has | |been refactored, and is no | |longer optional; it also | |supports merges from one | |authority type to another | |(with a table revision). -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 8579] Need script for updating authority 001s
by bugzilla-daemon＠bugs.koha-community.org 23 Feb '17

23 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8579 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 8579] Need script for updating authority 001s
by bugzilla-daemon＠bugs.koha-community.org 23 Feb '17

23 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8579 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |DUPLICATE Status|NEW |RESOLVED --- Comment #1 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- Closed in favor of bug 18071 *** This bug has been marked as a duplicate of bug 18071 *** -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15108] OAI-PMH provider improvements
by bugzilla-daemon＠bugs.koha-community.org 23 Feb '17

23 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15108 --- Comment #86 from Ere Maijala <ere.maijala(a)helsinki.fi> --- Created attachment 60608 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60608&action=ed… Fixed tests to be more robust and also work with UNIMARC. Added a patch to improve the server tests: - delete issues before biblios - delete oai_sets to avoid having them in the response - create UNIMARC test records when UNIMARC is enabled - include language code in the records to avoid failures caused by minor differences in how a missing language code is handled in OAI_DC transformation of different MARC flavors. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9988] Leave larger authority merges to merge_authority cronjob ( pref AuthorityMergeLimit)
by bugzilla-daemon＠bugs.koha-community.org 23 Feb '17

23 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9988 --- Comment #27 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- If you apply all patches, you may want to use this COMBINED test script: Run the three db revs. Run tests: t/db_dependent/Koha/Authorities.t t/db_dependent/Authorities/Merge.t t/db_dependent/Authorities/MergeRequests.t t/db_dependent/Koha/SearchEngine/Search.t t/db_dependent/UsageStats.t Follow test plan: Check the merge limit in sub merge Follow test plan: Refactor the cron script -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9988] Leave larger authority merges to merge_authority cronjob ( pref AuthorityMergeLimit)
by bugzilla-daemon＠bugs.koha-community.org 23 Feb '17

23 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9988 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Medium patch -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15108] OAI-PMH provider improvements
by bugzilla-daemon＠bugs.koha-community.org 23 Feb '17

23 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15108 --- Comment #85 from Ere Maijala <ere.maijala(a)helsinki.fi> --- I suppose that would have been the case with a clean database. I'll revise the tests a bit to assume it's not clean (delete issues and oai_sets at least). -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9988] Leave larger authority merges to merge_authority cronjob ( pref AuthorityMergeLimit)
by bugzilla-daemon＠bugs.koha-community.org 23 Feb '17

23 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9988 --- Comment #26 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- Created attachment 60607 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60607&action=ed… Bug 9988: Final polishing [1] DelAuthority: There is actually no need to call GetAuthority in this routine. The subfields of the old record are not relevant when removing all references in the linked biblio records. [2] Added a few POD lines to merge. [3] Removed a trailing space in a comment line in merge. Test plan: [1] Run Merge.t Signed-off-by: Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9988] Leave larger authority merges to merge_authority cronjob ( pref AuthorityMergeLimit)
by bugzilla-daemon＠bugs.koha-community.org 23 Feb '17

23 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9988 --- Comment #25 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- Created attachment 60606 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60606&action=ed… Bug 9988: Add the cronjob and print an upgrade message The Debian cron file and the misc crontab example are updated. A message is printed when upgrading. Note: At this moment the merge cron job is run once a day. This is imo a good starting point. The load for this job greatly depends on the value of pref AuthorityMergeLimit. Of course you can schedule the job more often, and if this need is felt more globally, we can adjust it later. Test plan: [1] Run the dbrev and see the message. [2] Read the changes to the cron files. Signed-off-by: Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9988] Leave larger authority merges to merge_authority cronjob ( pref AuthorityMergeLimit)
by bugzilla-daemon＠bugs.koha-community.org 23 Feb '17

23 Feb '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9988 --- Comment #24 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- Created attachment 60605 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60605&action=ed… Bug 9988: Fix a small last minute bug in merge When you postpone an 'authtype crossing' authority merge from A to B, the record for A will be deleted. In that case the condition for tags_new in sub merge does not work well. We should allow an empty authtypefrom here in order to get a new field belonging to authority B. Simply put, if we go from PERSO to CORPO, all linked records should go from X00 to X10 etc. Test plan: [1] Run Merge.t [2] Select two authorities of different type with linked records. [3] Set AuthorityMergeLimit to zero. And merge both auth records. Note: Do not select Default. See also bug 17380. [4] Run the cron job and check the linked records (especially the ones that were linked to the deleted record). Signed-off-by: Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.

1 0