From 725f4acc0c4ffb2b143e9e5b92d374bfbc4c5dea Mon Sep 17 00:00:00 2001 From: Aleisha Amohia Date: Thu, 7 Apr 2022 06:03:20 +0000 Subject: [PATCH] Bug 30358: (follow-up) Fixing tests and including for inner newlines --- C4/AuthoritiesMarc.pm | 4 ++-- C4/Biblio.pm | 4 ++-- t/db_dependent/Biblio/ModBiblioMarc.t | 12 ++++++------ 3 files changed, 10 insertions(+), 10 deletions(-) diff --git a/C4/AuthoritiesMarc.pm b/C4/AuthoritiesMarc.pm index 3cd455bba9..382e952386 100644 --- a/C4/AuthoritiesMarc.pm +++ b/C4/AuthoritiesMarc.pm @@ -643,8 +643,8 @@ sub AddAuthority { foreach my $subfield ( $field->subfields ) { my $key = $subfield->[0]; my $value = $subfield->[1]; - $value =~ s/^\s+|\s+$|^\t+|\t+$|^\n+|\n+$|^\r+|\r+$//g; - $value =~ s/\n+|\r+/ /g; + $value =~ s/[\n\r]+/ /g; + $value =~ s/^\s+|\s+$|^\t+|\t+$//g; $field->update( $key => $value ); } } diff --git a/C4/Biblio.pm b/C4/Biblio.pm index 575ac24701..6e9248c59b 100644 --- a/C4/Biblio.pm +++ b/C4/Biblio.pm @@ -2897,8 +2897,8 @@ sub ModBiblioMarc { foreach my $subfield ( $field->subfields ) { my $key = $subfield->[0]; my $value = $subfield->[1]; - $value =~ s/^\s+|\s+$|^\t+|\t+$|^\n+|\n+$|^\r+|\r+$//g; - $value =~ s/\n+|\r+/ /g; + $value =~ s/[\n\r]+/ /g; + $value =~ s/^\s+|\s+$|^\t+|\t+$//g; $field->update( $key => $value ); } } diff --git a/t/db_dependent/Biblio/ModBiblioMarc.t b/t/db_dependent/Biblio/ModBiblioMarc.t index 2e3e926904..cac3cea171 100755 --- a/t/db_dependent/Biblio/ModBiblioMarc.t +++ b/t/db_dependent/Biblio/ModBiblioMarc.t @@ -49,24 +49,24 @@ subtest "Check MARC field length calculation" => sub { subtest "StripWhitespaceChars tests" => sub { plan tests => 3; - t::lib::Mocks->mock_preference('marcflavour', 'MARC21'); - t::lib::Mocks->mock_preference('StripWhitespaceChars', 0); + t::lib::Mocks::mock_preference('marcflavour', 'MARC21'); + t::lib::Mocks::mock_preference('StripWhitespaceChars', 0); my $biblio = t::lib::TestBuilder->new->build_sample_biblio; my $record = MARC::Record->new; $record->append_fields( - MARC::Field->new( '245', '', '', a => " My title\n" ), + MARC::Field->new( '245', '', '', a => " My\ntitle\n" ), ); my $title = $record->title; - is( $title, " My title\n", 'Title has whitespace characters' ); + is( $title, " My\ntitle\n", 'Title has whitespace characters' ); C4::Biblio::ModBiblioMarc( $record, $biblio->biblionumber ); my $savedrec = C4::Biblio::GetMarcBiblio({ biblionumber => $biblio->biblionumber }); my $savedtitle = $savedrec->title; - is( $savedtitle, " My title\n", "Title still has whitespace characters because StripWhitespaceChars is disabled" ); + is( $savedtitle, " My\ntitle\n", "Title still has whitespace characters because StripWhitespaceChars is disabled" ); - t::lib::Mocks->mock_preference('StripWhitespaceChars', 1); + t::lib::Mocks::mock_preference('StripWhitespaceChars', 1); C4::Biblio::ModBiblioMarc( $record, $biblio->biblionumber ); my $amendedrec = C4::Biblio::GetMarcBiblio({ biblionumber => $biblio->biblionumber }); -- 2.11.0