From 41d76b8dbc389db93003cb6bc628c741a2455536 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Fri, 13 Nov 2015 16:30:32 +0000 Subject: [PATCH] [SIGNED-OFF] 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 Signed-off-by: Bernardo Gonzalez Kriegel Tested framework is lost using tool without patch Problem gone with patch No qa errors --- 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 60775a9..3aec4f7 100755 --- a/tools/batch_record_modification.pl +++ b/tools/batch_record_modification.pl @@ -192,7 +192,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, { -- 1.9.1