From e3b5900c87b78d4af8332ae578304691fb7d2ad0 Mon Sep 17 00:00:00 2001 From: Marcel de Rooy Date: Tue, 24 Jan 2017 14:53:33 +0100 Subject: [PATCH] Bug 16018: Add a TODO for need_merge_authorities in merge.pl Content-Type: text/plain; charset=utf-8 It is not useful to postpone a merge to the cron job here, since the old authority record is deleted and merge_authority cannot yet handle that correctly. But no harm either to leave it here. Note: This will be addressed shortly on bug 9988. So please help! Signed-off-by: Marcel de Rooy --- authorities/merge.pl | 3 +++ 1 file changed, 3 insertions(+) diff --git a/authorities/merge.pl b/authorities/merge.pl index 60bc6db..3996131 100755 --- a/authorities/merge.pl +++ b/authorities/merge.pl @@ -66,6 +66,9 @@ if ($merge) { my $MARCfrom = GetAuthority( $recordid2 ); &merge( $recordid2, $MARCfrom, $recordid1, $record ); } else { + # What follows, is actually useless. Because our cron job cannot handle + # it adequately. But it does not harm either. Will just be ignored. + # FIXME Restructure need_merge_authorities on bug 9988 my $dbh = C4::Context->dbh; my $sqlinsert = "INSERT INTO need_merge_authorities (authid, done) VALUES (?,?)"; $dbh->do( $sqlinsert, undef, ( $recordid1, 0 ) ); -- 2.1.4