From df62ba4fe039fd39094cfe9218593a1fe6f01464 Mon Sep 17 00:00:00 2001 From: Lucas Gass Date: Tue, 31 Aug 2021 20:36:37 +0000 Subject: [PATCH] Bug 28933: Add subscription class to each subscription in opac-detail To Test: 1. Go to a record with some subscriptions, have some closed and some open. 2. Go to the opac-detail page, click on the subscription tab and use your developer tools to inscept the element. 3. Notice the hierarchy of HTML structure, everything is a sibling of the next. 4. Apply patch 5. Reload the same opac-detail page and again inscept the element. 6. Now every individual subscription should be wrapped in an element with the class name of 'subscription'. --- .../bootstrap/en/modules/opac-detail.tt | 146 +++++++++--------- 1 file changed, 74 insertions(+), 72 deletions(-) diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt index ce9bff3939..fe2b35da26 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt +++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt @@ -628,84 +628,86 @@

This is a serial

There are [% subscriptionsnumber | html %] subscription(s) associated with this title.

[% FOREACH subscription IN subscriptions %] - [% IF ( subscription.branchcode ) %] -

At library: [% Branches.GetName( subscription.branchcode ) | html %]

- [% ELSE %] +
[% IF ( subscription.branchcode ) %] -

At library: [% subscription.branchcode | html %]

+

At library: [% Branches.GetName( subscription.branchcode ) | html %]

+ [% ELSE %] + [% IF ( subscription.branchcode ) %] +

At library: [% subscription.branchcode | html %]

+ [% END %] [% END %] - [% END %] - [% IF ( subscription.closed ) %] -

This subscription is closed.

- [% END %] - [% IF ( subscription.location ) %] -

Location: [% AuthorisedValues.GetByCode( 'LOC', subscription.location, 1 ) | html %] - [% END %] - [% IF ( subscription.callnumber ) %] -

Call number: [% subscription.callnumber | html %]

- [% END %] - [% IF ( subscription.subscriptionnotes ) %] -

[% subscription.subscriptionnotes | html | html_line_break %]

- [% END %] -

Subscription from: [% subscription.histstartdate | $KohaDates %] to:[% IF ( subscription.histenddate ) %] [% subscription.histenddate | $KohaDates %] [% ELSE %] now (current)[% END %]

- [% IF ( subscription.letter ) %] -
[% END # / FOREACH subscriptions %]

More details

-- 2.20.1