From 1b221b2ff1c66c373c2b35bb9a9c07db9924453c Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Fri, 13 Nov 2015 16:30:32 +0000 Subject: [PATCH] Bug 14795: Prevent modifying a record without using the original framework The origin framework is not passed on modifying the records and so the default one is used. This can caused data lost. Test plan: - Create a bibliographic record and specify a framework (not the default one). - Use the Batch records modification tools to modify it. => Without this patch, the default framework (framework code "") is used and can deteriorate the record => With this patch, the record is saved using the original framework --- tools/batch_record_modification.pl | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/batch_record_modification.pl b/tools/batch_record_modification.pl index 5b46823..f96ec5d 100755 --- a/tools/batch_record_modification.pl +++ b/tools/batch_record_modification.pl @@ -191,7 +191,8 @@ if ( $op eq 'form' ) { my $error = eval { my $record = GetMarcBiblio( $biblionumber ); ModifyRecordWithTemplate( $mmtid, $record ); - ModBiblio( $record, $biblionumber ); + my $frameworkcode = C4::Biblio::GetFrameworkCode( $biblionumber ); + ModBiblio( $record, $biblionumber, $frameworkcode ); }; if ( $error and $error != 1 or $@ ) { # ModBiblio returns 1 if everything as gone well push @messages, { -- 2.1.0