From bfed8054d6670de71478a158bfb671fd938e1e38 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 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 Signed-off-by: mehdi --- 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 ) ); -- 1.9.1