@@ -, +, @@ correctly - Add a subscription - Add comments including line breaks in internal and OPAC note - Check the manual history checkbox - Go to the subscription detail page, tab 'Planning' - Use the 'Edit history' link to edit the history - Enter content with line breaks in all the input fields where it's possible - Go to the OPAC detail page for your subscription record - Go to tab 'Subscriptions' > More details > Brief history - Verify the line breaks show there as
- Apply patch - Relaod page, verify all information displays correctly now - Go back to the detail page, subscription tab - verify same there - Go back to the subscription detail page in staff - Check editing the history works as expected - Verify information on all tabs displays correctly - Receive a serial issue - verify note on top displays right - Go to the serial collection page of the subscription - same there --- C4/Serials.pm | 3 --- .../intranet-tmpl/prog/en/modules/serials/serials-collection.tt | 2 +- koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-edit.tt | 2 +- .../intranet-tmpl/prog/en/modules/serials/subscription-detail.tt | 4 ++-- koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt | 2 +- koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-serial-issues.tt | 4 ++-- opac/opac-serial-issues.pl | 6 ------ 7 files changed, 7 insertions(+), 16 deletions(-) --- a/C4/Serials.pm +++ a/C4/Serials.pm @@ -429,9 +429,6 @@ sub GetSubscriptionsFromBiblionumber { $subs->{histenddate} = ""; } $subs->{opacnote} //= ""; - $subs->{opacnote} =~ s/\n/\/g; - $subs->{missinglist} =~ s/\n/\/g; - $subs->{recievedlist} =~ s/\n/\/g; $subs->{ "periodicity" . $subs->{periodicity} } = 1; $subs->{ "numberpattern" . $subs->{numberpattern} } = 1; $subs->{ "status" . $subs->{'status'} } = 1; --- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt @@ -53,7 +53,7 @@ [% subscription.numberpattern.label | html %] [% IF subscription.branchcode %][% Branches.GetName( subscription.branchcode ) | html %][% END %] [% subscription.callnumber | html %] - [% subscription.notes | html %] + [% subscription.notes | html | html_line_break %] [% UNLESS subscription.closed %] [% IF ( subscription.subscriptionexpired ) %]
Subscription expired --- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-edit.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-edit.tt @@ -95,7 +95,7 @@ $(document).ready(function() {

Serial edition [% bibliotitle | html %] [% IF location %] ([% AuthorisedValues.GetByCode('LOC', location) | html %])[% END %] [% IF ( callnumber ) %] callnumber: [% callnumber | html %][% END %]

-[% IF internalnotes %]

Nonpublic note: [% internalnotes | html %]

[% END %] +[% IF internalnotes %]

Nonpublic note: [% internalnotes | html | html_line_break %]

[% END %]
[% IF ( Errors ) %] --- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-detail.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-detail.tt @@ -332,8 +332,8 @@
  • End date: [% enddate | html %]
  • History start date: [% histstartdate | html %]
  • History end date: [% histenddate | html %]
  • -
  • Received issues:[% recievedlist | html %]
  • -
  • Missing issues:[% missinglist | html %]
  • +
  • Received issues:[% recievedlist | html | html_line_break %]
  • +
  • Missing issues:[% missinglist | html | html_line_break %]
  • Nonpublic note:[% internalnotes | html | html_line_break %]
  • Public note:[% notes | html | html_line_break %]
  • History staff note:[% librariannote | html | html_line_break %]
  • --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt +++ a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt @@ -832,7 +832,7 @@ [% END %] [% END %] [% IF ( subscription.missinglist ) %] -

    Missing issues: [% subscription.missinglist | html %]

    +

    Missing issues: [% subscription.missinglist | html | html_line_break %]

    [% END %] [% IF ( subscription.opacnote ) %]

    [% subscription.opacnote | html | html_line_break %]

    --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-serial-issues.tt +++ a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-serial-issues.tt @@ -120,12 +120,12 @@ [% IF ( subscription_LOO.recievedlist ) %]
    Available issues
    -

    [% subscription_LOO.recievedlist | html %]

    +

    [% subscription_LOO.recievedlist | html | html_line_break %]

    [% END %] [% IF ( subscription_LOO.missinglist ) %]
    Unavailable issues
    -

    [% subscription_LOO.missinglist | html %]

    +

    [% subscription_LOO.missinglist | html | html_line_break %]

    [% END %] [% IF ( subscription_LOO.opacnote ) %] --- a/opac/opac-serial-issues.pl +++ a/opac/opac-serial-issues.pl @@ -71,9 +71,6 @@ if ( $selectview eq "full" ) { my $yearmin = $subscriptions->[0]->{year}; my $yearmax = $subscriptions->[ -1 ]->{year}; - # replace CR by
    in librarian note - # $subscription->{opacnote} =~ s/\n/\/g; - $template->param( biblionumber => scalar $query->param('biblionumber'), years => $subscriptioninformation, @@ -106,9 +103,6 @@ else { } } - # replace CR by
    in librarian note - # $subscription->{opacnote} =~ s/\n/\/g; - my $title = $subscriptions->[0]->{bibliotitle}; $template->param( --