From 0837954cdda85549f5552dacefd3a5bc3e58292e Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Wed, 9 Apr 2014 09:08:27 -0400
Subject: [PATCH] Bug 12056: fix untranslatable strings in calendar

In the calendar there are some strings in a JavaScript function which
are not properly wrapped in a function for translation. This patch
corrects this.

This patch also corrects some minor validation issues and spelling and
grammar issues, including those covered by Bug 12055.

To test, apply the patch and view the calendar in Tools -> Calendar.
When you hover your mouse over a day in the calendar you should see a
title tooltip indicating what kind of day/holiday it is and showing the
title of the holiday, if any.

To test that the strings are now being picked up for translation,
run translate update on a po file and confirm that the affected strings
are now present: "Weekly holiday," "Yearly holiday," etc.

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>

Works well. New strings on translation file. No koha-qa errors.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, passes all tests and QA script.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
---
 .../prog/en/modules/tools/holidays.tt         | 35 ++++++++++---------
 1 file changed, 18 insertions(+), 17 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/holidays.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/holidays.tt
index 850c59bd8c..ea001e4ed8 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/holidays.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/holidays.tt
@@ -4,11 +4,11 @@
 [% INCLUDE 'calendar.inc' %]
 <link rel="stylesheet" type="text/css" href="[% themelang %]/css/datatables.css" />
 [% INCLUDE 'datatables.inc' %]
-<script language="JavaScript" type="text/javascript">
+<script type="text/javascript">
 //<![CDATA[
     var weekdays = new Array(_("Sundays"),_("Mondays"),_("Tuesdays"),_("Wednesdays"),_("Thursdays"),_("Fridays"),_("Saturdays"));
 
-    /* Creates all the structures to deal with all diferents kinds of holidays */
+    /* Creates all the structures to deal with all different kinds of holidays */
     var week_days = new Array();
     var holidays = new Array();
     var holidates = new Array();
@@ -116,15 +116,15 @@
         var dayMonth = month + '/' + day;
         var dateString = year + '/' + month + '/' + day;
         if (exception_holidays[dateString] != null) {
-            return [true, "exception", "Exception: "+exception_holidays[dateString].title];
+            return [true, "exception", _("Exception") + ": "+exception_holidays[dateString].title];
         } else if ( week_days[weekDay] != null ){
-            return [true, "repeatableweekly", "Weekly holdiay: "+week_days[weekDay].title];
+            return [true, "repeatableweekly", _("Weekly holiday") + ": "+week_days[weekDay].title];
         } else if ( day_month_holidays[dayMonth] != null ) {
-            return [true, "repeatableyearly", "Yearly holdiay: "+day_month_holidays[dayMonth].title];
+            return [true, "repeatableyearly", _("Yearly holiday") + ": "+day_month_holidays[dayMonth].title];
         } else if (holidays[dateString] != null) {
-            return [true, "holiday", "Single holiday: "+holidays[dateString].title];
+            return [true, "holiday", _("Single holiday") + ": "+holidays[dateString].title];
         } else {
-            return [true, "normalday", "Normal day"];
+            return [true, "normalday", _("Normal day")];
         }
     }
 
@@ -269,9 +269,10 @@ td.repeatableyearly a.ui-state-default {  background:  #FFFF99 none; color :  Bl
                 <input type="text" id="datecancelrange" name="datecancelrange" size="20" value="[% datecancelrange %]" class="datepicker"/>
             </li>
             <li><label for="showTitle">Title: </label><input type="text" name="showTitle" id="showTitle" size="35" /></li>
-            <!-- showTitle is necessary for exception radio button to work properly --> 
+            <!-- showTitle is necessary for exception radio button to work properly -->
+            <li>
                 <label for="showDescription">Description:</label>
-                <textarea rows="2" cols="40" id="showDescription" name="showDescription"></textarea>    
+                <textarea rows="2" cols="40" id="showDescription" name="showDescription"></textarea>
             </li>
             <li class="radio"><div id="exceptionPosibility" style="position:static">
                 <input type="radio" name="showOperation" id="showOperationExc" value="exception" /> <label for="showOperationExc">Generate an exception for this repeated holiday.</label>
@@ -279,7 +280,7 @@ td.repeatableyearly a.ui-state-default {  background:  #FFFF99 none; color :  Bl
                 <div class="hint">You can make an exception for this holiday rule. This means that you will be able to say that for a repeatable holiday there is one day which is going to be an exception.</div>
             </div></li>
             <li class="radio"><input type="radio" name="showOperation" id="showOperationExcRange" value="exceptionrange" />
-                <label for="newOperationFieldException">Generate exceptions on a range of dates.</label>
+                <label for="showOperationExcRange">Generate exceptions on a range of dates.</label>
                 <a href="#" class="helptext">[?]</a>
                 <div class="hint">You can make an exception on a range of dates repeated yearly.</div>
             </li>
@@ -304,7 +305,7 @@ td.repeatableyearly a.ui-state-default {  background:  #FFFF99 none; color :  Bl
             </ol>
             <fieldset class="action">
                 <input type="submit" name="submit" value="Save" />
-                <a href="#" class="cancel" name="cancel2" onclick=" hidePanel('showHoliday');">Cancel</a>
+                <a href="#" class="cancel" onclick=" hidePanel('showHoliday');">Cancel</a>
             </fieldset>
             </fieldset>
         </form>
@@ -345,7 +346,7 @@ td.repeatableyearly a.ui-state-default {  background:  #FFFF99 none; color :  Bl
             <li class="radio"><input type="radio" name="newOperation" id="newOperationOnce" value="holiday" checked="checked" />
             <label for="newOperationOnce">Holiday only on this day</label>.
             <a href="#" class="helptext">[?]</a>
-            <div class="hint">Make a single holiday. For example, selecting August 1st, 2012 will make it a holiday, but will not affect August 1st in other years.</div>
+            <div class="hint">Make a single holiday. For example, selecting August 1, 2012 will make it a holiday, but will not affect August 1 in other years.</div>
             </li>
             <li class="radio"><input type="radio" name="newOperation" id="newOperationDay" value="weekday" />
                             <label for="newOperationDay">Holiday repeated every same day of the week</label>.
@@ -355,17 +356,17 @@ td.repeatableyearly a.ui-state-default {  background:  #FFFF99 none; color :  Bl
             <li class="radio"><input type="radio" name="newOperation" id="newOperationYear" value="repeatable" />
                             <label for="newOperationYear">Holiday repeated yearly on the same date</label>.
                             <a href="#" class="helptext">[?]</a>
-                            <div class="hint">This will take this day and month as a reference to make it a holiday. Through this option, you can repeat this rule for every year. For example, selecting August 1st will make August 1st a holiday every year.</div>
+                            <div class="hint">This will take this day and month as a reference to make it a holiday. Through this option, you can repeat this rule for every year. For example, selecting August 1 will make August 1 a holiday every year.</div>
                             </li>
             <li class="radio"><input type="radio" name="newOperation" id="newOperationField" value="holidayrange" />
                             <label for="newOperationField">Holidays on a range</label>.
                             <a href="#" class="helptext">[?]</a>
-                            <div class="hint">Make a single holiday on a range. For example, selecting August 1st, 2012  and August 10st, 2012 will make all days between 1st and 10st holiday, but will not affect August 1-10 in other years.</div>
+                            <div class="hint">Make a single holiday on a range. For example, selecting August 1, 2012  and August 10, 2012 will make all days between August 1 and 10 a holiday, but will not affect August 1-10 in other years.</div>
                             </li>
             <li class="radio"><input type="radio" name="newOperation" id="newOperationFieldyear" value="holidayrangerepeat" />
                             <label for="newOperationFieldyear">Holidays repeated yearly on a range</label>.
                             <a href="#" class="helptext">[?]</a>
-                            <div class="hint">Make a single holiday on a range repeated yearly. For example, selecting August 1st, 2012  and August 10st, 2012 will make all days between 1st and 10st holiday, and will affect August 1-10 in other years.</div>
+                            <div class="hint">Make a single holiday on a range repeated yearly. For example, selecting August 1, 2012  and August 10, 2012 will make all days between August 1 and 10 a holiday, and will affect August 1-10 in other years.</div>
                             </li>
                 <li class="radio">
                 <input type="checkbox" name="allBranches" id="allBranches" />
@@ -375,7 +376,7 @@ td.repeatableyearly a.ui-state-default {  background:  #FFFF99 none; color :  Bl
                 </li></ol>
                 <fieldset class="action">
                     <input type="submit" name="submit" value="Save" />
-                    <a href="#" class="cancel" name="cancel2" onclick=" hidePanel('newHoliday');">Cancel</a>
+                    <a href="#" class="cancel" onclick=" hidePanel('newHoliday');">Cancel</a>
                 </fieldset>
                 </fieldset>
          </form>
@@ -412,7 +413,7 @@ td.repeatableyearly a.ui-state-default {  background:  #FFFF99 none; color :  Bl
     <ul>
         <li>Search in the calendar the day you want to set as holiday.</li>
         <li>Click the date to add or edit a holiday.</li>
-        <li>Enter a title and description for the holdiay.</li>
+        <li>Enter a title and description for the holiday.</li>
         <li>Specify how the holiday should repeat.</li>
         <li>Click Save to finish.</li>
     </ul>
-- 
2.39.5