Bug 10186

Summary: Typo in subscription-add.tt (years vs year)
Product: Koha Reporter: Mathieu Saby <mathsabypro>
Component: TemplatesAssignee: Mathieu Saby <mathsabypro>
Status: CLOSED FIXED QA Contact:
Severity: trivial    
Priority: P5 - low CC: bgkriegel, chrish, dcook, gmcharlt, m.de.rooy, tomascohen
Version: 3.14   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: String patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: [PATCH] Bug 10186: Fix a typo in subscription-add.tt : 2/years should be 2/year
Bug 10186: Fix a typo in subscription-add.tt : 2/years should be 2/year
Bug 10186: Fix a typo in subscription-add.tt : 2/years should be 2/year
Bug 10186: Fix a typo in subscription-add.tt : 2/years should be 2/year

Description Mathieu Saby 2013-05-03 15:20:38 UTC
Typo to fix in koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt

replace "2/years" with "2/year" in this block : 

[% IF ( periodicity9 ) %]
                    <option value="9" selected="selected">2/years</option>
                [% ELSE %]
                    <option value="9">2/year</option>
                [% END %]



M. Saby
Comment 1 Mathieu Saby 2013-05-04 09:16:37 UTC Comment hidden (obsolete)
Comment 2 David Cook 2013-05-08 08:17:43 UTC Comment hidden (obsolete)
Comment 3 David Cook 2013-05-08 08:18:09 UTC Comment hidden (obsolete)
Comment 4 Marcel de Rooy 2013-05-08 11:46:09 UTC
Created attachment 18018 [details] [review]
Bug 10186: Fix a typo in subscription-add.tt : 2/years should be 2/year

In subscription-add.tt dropdown list for periodiciy, the description for 9 value should be "2/year" (2 issues each year)
At present, if the option value is selected, it is "2/years".
This patch supresses the "s" after "year".

To test :
- in a subscription, change periodicity to "2/year"
- save the subscription
- reopen it and check the value is labelled "2/year"

Signed-off-by: David Cook <dcook@prosentient.com.au>

Trivial change. Works as described.
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 5 Marcel de Rooy 2013-05-08 11:47:00 UTC
Passed QA
Comment 6 Galen Charlton 2013-05-20 14:56:02 UTC
Pushed to master.  Thanks, Mathieu!
Comment 7 Tomás Cohen Arazi 2013-05-20 18:01:07 UTC
This patch has been pushed to 3.12.x, will be in 3.12.1.
Comment 8 Bernardo Gonzalez Kriegel 2013-05-28 01:13:15 UTC
This patch has been pushed to 3.10.x, will be in 3.10.7.
Comment 9 Chris Hall 2013-05-29 09:41:47 UTC
Pushed to 3.8.x, will be in 3.8.14.