Bug 11752: display the correct frequency for serial subscriptions in OPAC details
authorBlou <philippe.blouin@inlibro.com>
Wed, 12 Feb 2014 17:03:08 +0000 (12:03 -0500)
committerKyle M Hall <kyle@bywatersolutions.com>
Wed, 13 Aug 2014 14:36:03 +0000 (10:36 -0400)
This fixes bootstrap and prog by modifying the description displayed
in the OPAC's detail of serials.

RM NOTE: this patch does not cover the case where custom serial
frequencies have been defined.

TESTING to reproduce
- create/find a serial with a 1/week periodicity (4 in the database)
- Find it in the opac-detail.pl, click "more details" at the bottom
- validate the string.  Before the patch, it will say:
"The current subscription began on 2013-12-06 and is issued every 3
 weeks for 26 issues"

The "every 3 weeks" is clearly wrong.
In fact any periodicity chosen would display a wrong description, not
matching the staff interface.

After the patch, the display is corrected.

As a bonus, the "every 2 years" now has a description, where it had
none before.

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
(cherry picked from commit 7f1e949ea0e8d05b641ddbcb4582a3e1bc913ecd)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

koha-tmpl/opac-tmpl/prog/en/modules/opac-serial-issues.tt

index 5d590a9..3c1574a 100644 (file)
 [% FOREACH subscription_LOO IN subscription_LOOP %]
 <div id="subscriptionline[% subscription_LOO.subscriptionid %]" style="border-bottom:1px solid #EEE;">
 <h3>Subscription information for [% subscription_LOO.bibliotitle %]</h3>
-               <p>The current subscription began on [% subscription_LOO.startdate %] and is issued every
+            <p>The current subscription began on [% subscription_LOO.startdate %] and is issued
                [% IF ( subscription_LOO.periodicity1 ) %]
-                               day
+                               twice per day
                [% END %]
                [% IF ( subscription_LOO.periodicity2 ) %]
-                               week
+                         every day
                [% END %]
                [% IF ( subscription_LOO.periodicity3 ) %]
-                               2 weeks
+                             three times per week
                [% END %]
                [% IF ( subscription_LOO.periodicity4 ) %]
-                               3 weeks
+                          every week
                [% END %]
                [% IF ( subscription_LOO.periodicity5 ) %]
-                               month
+                            every 2 weeks
                [% END %]
                [% IF ( subscription_LOO.periodicity6 ) %]
-                               2 months
+                         every 3 weeks
                [% END %]
                [% IF ( subscription_LOO.periodicity7 ) %]
-                               3 months
-               [% END %]
-               [% IF ( subscription_LOO.periodicity13 ) %]
-                               4 months
+                         every month
                [% END %]
                [% IF ( subscription_LOO.periodicity8 ) %]
-                               quarter
+                           every 2 months
                [% END %]
                [% IF ( subscription_LOO.periodicity9 ) %]
-                               2 quarters
+                                every quarter
                [% END %]
                [% IF ( subscription_LOO.periodicity10 ) %]
-                               year
+                                twice per year
                [% END %]
                [% IF ( subscription_LOO.periodicity11 ) %]
-                               2 years
+                               every year
+             [% END %]
+              [% IF ( subscription_LOO.periodicity12 ) %]
+                            every 2 years
+          [% END %]
+              [% IF ( subscription_LOO.periodicity13 ) %]
+                            irregularly
                [% END %]
                [% IF ( subscription_LOO.arrival1 ) %]
                        on Monday