From 0826010a9d70a5f726b3e989cecab744406fe7ef Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Thu, 16 Jun 2016 09:28:01 -0400
Subject: [PATCH] Bug 16754 - Use validation plugin in budgets, planning, and
 contracts
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This patch removes the custom javascript form validation from three
templates: Budgets, budget planning, and contracts. Validation via the
validation plugin is done instead.

This patch also removes use of event attributes like onclick and
onchange. Also changed: I added the "Required" text next to required
fields as necessary.

To test, apply the patch and go to Administration -> Budgets.

- Test the budget creation process for adding, editing, and duplicating
  budgets. In all cases:
  - The start date and end date should be required
  - You should be warned if you enter an end date which falls before the
    start date.
  - The description should be required
  - The budget amount should be required to be a number

To test the budget planning page:

Note that much of the JS removed from this template was unused. It
looked for inputs by the name 'est_total' which don't exist.

- When viewing the table of funds for a budget, select a choice from the
  'planning' toolbar menu.
  - Verify that after entering a numeric value in any cell, the
    'remaning' value at the end of the row is correctly updated.
  - Verify that after entering non-numeric data an error is displayed.
  - Verify that clicking the 'auto-fill' button at the end of the row

To test the vendor contracts page:

- Go to Acquisitions -> Vendor -> Contracts
- Add or edit a contract
  - Confirm that name, start date, and end date are required.
  - Confirm that you are warned if you enter an end date which falls
    before the start date.

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
---
 .../intranet-tmpl/prog/en/includes/calendar.inc    |  26 +++++
 .../prog/en/modules/admin/aqbudgetperiods.tt       | 112 ++++++---------------
 .../prog/en/modules/admin/aqcontract.tt            |  67 +++++-------
 .../intranet-tmpl/prog/en/modules/admin/aqplan.tt  |  51 +++-------
 4 files changed, 100 insertions(+), 156 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/calendar.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/calendar.inc
index 9b6dd52..55ebbd7 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/calendar.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/calendar.inc
@@ -76,6 +76,32 @@ jQuery(function($){
     $.datepicker.setDefaults($.datepicker.regional['']);
 });
 
+/*  jQuery Validator plugin custom method
+    This allows you to check that a given date falls after another.
+    It is required that a message be defined.
+
+   Syntax:
+       $("#form_id").validate({
+        rules: {
+            input_name_of_later_date_field: {
+                is_date_after: "#input_id_of_earlier_date_field"
+            },
+        },
+        messages: {
+            input_name_of_later_date_field: {
+                is_date_after: _("Validation error to be shown, i.e. End date must come after start date")
+            }
+        }
+    });
+*/
+
+jQuery.validator.addMethod("is_date_after",
+function(value, element, params) {
+    var from = Date_from_syspref( $(params).val() );
+    var to = Date_from_syspref(value);
+    return to > from;
+});
+
 $(document).ready(function(){
 
 $.datepicker.setDefaults({
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgetperiods.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgetperiods.tt
index ecedbea..92b6011 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgetperiods.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqbudgetperiods.tt
@@ -37,75 +37,6 @@
 [% END %]
 <script type="text/javascript" src="[% interface %]/[% theme %]/js/acq.js"></script>
 <script type="text/javascript">
-// #################################################################################
-// Javascript
-// #################################################################################
-    function CheckDuplicateForm(f){
-            var ok=1;
-            var _alertString="";
-            var alertString="";
-            if(!(isNotNull(f.budget_period_startdate,1))){
-              _alertString += "\n- " + _("Start date missing");
-            }
-            if (!(isNotNull(f.budget_period_enddate,1))){
-              _alertString += "\n- " + _("End date missing");
-            }
-            if( f.budget_period_startdate > f.budget_period_enddate ) {
-              _alertString += "\n- " + _("Start date must be before end date");
-            }
-            if (!(isNotNull(f.budget_period_description,1))){
-              _alertString += "\n- " + _("Budget description missing");
-            }
-
-            if(_alertString.length==0){
-              f.submit();
-            } else {
-              alertString += _("Form not submitted because of the following problem(s)");
-              alertString += "\n-----------------------------------------\n";
-              alertString += _alertString;
-              alert(alertString);
-            }
-    }
-    function Check(f) {
-            var ok=1;
-            var _alertString="";
-            var alertString2;
-
-            if (!(isNotNull(f.budget_period_startdate,1))) {
-                    _alertString += "\n- " + _("Start date missing");
-            }
-
-            if (!(isNotNull(f.budget_period_enddate,1))) {
-                    _alertString += "\n- " + _("End date missing");
-            }
-
-            if ( f.budget_period_startdate >   f.budget_period_enddate )  {
-                    _alertString += "\n- " + _("Start date must be before end date");
-            }
-
-            if (!(isNotNull(f.budget_period_description,1))) {
-                    _alertString += "\n- " + _("Description missing");
-            }
-
-            if (!(isNum(f.budget_period_total))) {
-                    _alertString += "\n- " + _("Amount must be a valid number, or empty");
-            }
-
-/*
-			checkBudgetTotal(f) {
-			}
-*/
-
-            if (_alertString.length==0) {
-                    f.submit();
-            } else {
-                    alertString2  = _("Form not submitted because of the following problem(s)");
-                    alertString2 += "\n------------------------------------------------------------------------------------\n";
-                    alertString2 += _alertString;
-                    alert(alertString2);
-            }
-    }
-
     $(document).ready(function() {
         var tabs = $('#budgetsTabs').tabs();
         [% IF ( tab ) %]
@@ -171,6 +102,24 @@
             "sPaginationType": "full_numbers"
           }));
         [% END %]
+        $("#add_modify_budget").validate({
+            rules: {
+                budget_period_startdate: "required",
+                budget_period_enddate: {
+                    required: true,
+                    is_date_after: "#from"
+                },
+                budget_period_description: "required",
+                budget_period_total: {
+                    number: true
+                }
+            },
+            messages: {
+                budget_period_enddate: {
+                    is_date_after: _("Budget end date must be after budget start date")
+                }
+            }
+        });
     });
 </script>
 
@@ -252,13 +201,13 @@
 <div id="yui-main">
 <div class="yui-b">
 
-[% UNLESS close_form OR closed%]
+[% UNLESS close_form OR closed OR duplicate_form %]
   [% INCLUDE 'budgets-admin-toolbar.inc' %]
 [% END %]
 
 [% IF ( duplicate_form ) %]
 <h3>Duplicate budget</h3>
-<form action="/cgi-bin/koha/admin/aqbudgetperiods.pl" name="f" method="post">
+<form action="/cgi-bin/koha/admin/aqbudgetperiods.pl" id="add_modify_budget" name="f" method="post">
     <fieldset class="rows">
     <input type="hidden" name="op" value="duplicate_budget" />
     <input type="hidden" name="budget_period_id" value="[% budget_period_id %]" />
@@ -268,18 +217,21 @@
     <li>
     <label class="required" for="from">Start date: </label>
     <input type="text" size="10" id="from" name="budget_period_startdate" value="[% budget_period_startdate | $KohaDates %]" class="datepickerfrom" />
-				<div class="hint">[% INCLUDE 'date-format.inc' %]</div>
+            <span class="required">Required</span>
+                <div class="hint">[% INCLUDE 'date-format.inc' %]</div>
     </li>
     <li>
 
     <label class="required" for="to">End date: </label>
     <input type="text" size="10" id="to" name="budget_period_enddate" value="[% budget_period_enddate | $KohaDates %]" class="datepickerto" />
-				<div class="hint">[% INCLUDE 'date-format.inc' %]</div>
+            <span class="required">Required</span>
+                <div class="hint">[% INCLUDE 'date-format.inc' %]</div>
     </li>
 
     <li>
     <label class="required" for="budget_period_description">Description</label>
     <input type="text" id="budget_period_description" name="budget_period_description" value="[% budgetperiod.budget_period_description %]" />
+        <span class="required">Required</span>
     </li>
 
     <li>
@@ -308,7 +260,7 @@
     </fieldset>
 
     <fieldset class="action">
-        <input type="button" value="Save Changes"  onclick="CheckDuplicateForm(this.form)" /> <a class="cancel" href="/cgi-bin/koha/admin/aqbudgets.pl?budget_period_id=[% budget_period_id %]">Cancel</a>
+        <input type="submit" value="Save" /> <a class="cancel" href="/cgi-bin/koha/admin/aqbudgets.pl?budget_period_id=[% budget_period_id %]">Cancel</a>
     </fieldset>
 
 </form>
@@ -323,7 +275,7 @@
     [% ELSE %]
         <h3>Add budget</h3>
 	[% END %]
-<form action="/cgi-bin/koha/admin/aqbudgetperiods.pl" name="f" method="post">
+<form action="/cgi-bin/koha/admin/aqbudgetperiods.pl" id="add_modify_budget" name="f" method="post">
     <fieldset class="rows">
     <!-- ################################################################################# -->
     <!-- display information about the budget period that must be added or modified -->
@@ -334,13 +286,15 @@
     <li>
     <label class="required" for="from">Start date: </label>
     <input type="text" size="10" id="from" name="budget_period_startdate" value="[% budget_period_startdate | $KohaDates %]" class="datepickerfrom" />
-				<div class="hint">[% INCLUDE 'date-format.inc' %]</div>
+            <span class="required">Required</span>
+                <div class="hint">[% INCLUDE 'date-format.inc' %]</div>
     </li>
     <li>
 
     <label class="required" for="to">End date: </label>
     <input type="text" size="10" id="to" name="budget_period_enddate" value="[% budget_period_enddate | $KohaDates %]" class="datepickerto" />
-				<div class="hint">[% INCLUDE 'date-format.inc' %]</div>
+            <span class="required">Required</span>
+                <div class="hint">[% INCLUDE 'date-format.inc' %]</div>
     </li>
 
     <li>
@@ -349,6 +303,7 @@
         <label class="required" for="budget_period_description">Description: </label>
         <input type="text" id="budget_period_description" name="budget_period_description"
         size="48" maxlength="80" value="[% budget_period_description %]" />
+            <span class="required">Required</span>
     </li>
 
     <li>
@@ -378,8 +333,7 @@
     <fieldset class="action">
         <!-- "save changes" button -->
         <!-- ###################################### -->
-        <input type="button" value="Save"  onclick="Check(this.form)" />
-        <!--             <input type="submit" value="Save Changes"  />  -->
+        <input type="submit" value="Save" />
         [% IF ( budget_period_id ) %]
             <a href="/cgi-bin/koha/admin/aqbudgets.pl?budget_period_id=[% budget_period_id %]" class="cancel">Cancel</a>
         [% ELSE %]
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqcontract.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqcontract.tt
index 304688a..432a139 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqcontract.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqcontract.tt
@@ -14,41 +14,23 @@
 <script type="text/javascript" src="[% interface %]/[% theme %]/js/acq.js"></script>
 <script type="text/javascript">
 //<![CDATA[
-// to check if the data are correctly entered.
-function Check(ff) {
-    var ok=0;
-    var _alertString=_("Form not submitted because of the following problem(s)");
-    _alertString +="\n-------------------------------------------------------------------\n\n";
-    if (!(isNotNull(ff.contractname,0))){
-        ok=1;
-        _alertString += _("- Name missing") + "\n";
-    }
-    var startDate = Date_from_syspref($("#from").val());
-    var endDate   = Date_from_syspref($("#to").val());
-    if (!parseInt(startDate.getTime())) {
-        ok=1;
-        _alertString += _("- Start date missing or invalid.") + "\n";
-    }
-    if (!parseInt(endDate.getTime())) {
-        ok=1;
-        _alertString += _("- End date missing or invalid.") + "\n";
-    }
-    
-    if (startDate > endDate) {
-        ok=1;
-        _alertString += _("Wrong date! start date cannot be after end date.") + "\n";
-    }
-    if (endDate < (new Date)) {
-        ok=1;
-        _alertString += _("End date before today, Invalid end date!") + "\n";
-    }
-    if (ok) { // if there is a problem
-        alert(_alertString);
-    return false;
-}
-// if all is good
-    ff.submit();
-}
+$(document).ready(function(){
+    $("#add_modify_contract").validate({
+        rules: {
+            contractstartdate: "required",
+            contractname: "required",
+            contractenddate: {
+                required: true,
+                is_date_after: "#from"
+            }
+        },
+        messages: {
+            contractenddate: {
+                is_date_after: _("Contract end date must be after contract start date")
+            }
+        }
+    });
+});
 //]]>
 </script>
 </head>
@@ -97,7 +79,7 @@ function Check(ff) {
 [% END %]
 
                 [% IF ( add_form ) %]
-                    <form name="Aform" action="[% script_name %]" method="post">
+                    <form name="Aform" action="/cgi-bin/koha/admin/aqcontract.pl" id="add_modify_contract" method="post">
                         <input type="hidden" name="op" value="add_validate" />
                         <input type="hidden" name="booksellerid" value="[% booksellerid %]" />
                         <input type="hidden" name="checked" value="0" />
@@ -109,29 +91,32 @@ function Check(ff) {
                         <fieldset class="rows">
                             <ol>
                                 [% IF ( contractnumber ) %]
-                                        <li><span class="label">Contract id </span>[% contractnumber %]
+                                        <li><span class="label">Contract id:</span>[% contractnumber %]
                                             <input type="hidden" name="contractnumber" value="[% contractnumber %]" />
                                             <input type="hidden" name="is_a_modif" value="1" />
                                         </li>
                                 [% END %]
-                                <li><label for="contractname" class="required">Name: *</label> &nbsp;
+                                <li><label for="contractname" class="required">Name:</label> &nbsp;
                                         <input type="text" name="contractname" id="contractname" size="40" maxlength="80" value="[% contractname %]" />
+                                        <span class="required">Required</span>
                                 </li>
                                 <li><label for="contractdescription">Description: </label> &nbsp;
                                         <input type="text" name="contractdescription" id="contractdescription" size="40" maxlength="80" value="[% contractdescription %]" />
                                 </li>
-                                <li><label for="from" class="required">Start date: *</label> &nbsp;
+                                <li><label for="from" class="required">Start date:</label> &nbsp;
                                         <input type="text" name="contractstartdate" id="from" value="[% contractstartdate | $KohaDates %]" maxlength="10" size="10" class="datepickerfrom" />
+                                        <span class="required">Required</span>
 				<div class="hint">[% INCLUDE 'date-format.inc' %]</div>
                                 </li>
-                                <li><label for="to" class="required">End date: *</label> &nbsp;
+                                <li><label for="to" class="required">End date:</label> &nbsp;
                                         <input type="text" name="contractenddate" id="to" value="[% contractenddate | $KohaDates %]" maxlength="10"  size="10" class="datepickerto" />
+                                        <span class="required">Required</span>
 				<div class="hint">[% INCLUDE 'date-format.inc' %]</div>
                                 </li>
                             </ol>
                         </fieldset>
                         <fieldset class="action">
-                            <input type="button" value="Save" onclick="Check(this.form);" /> <a class="cancel" href="/cgi-bin/koha/admin/aqcontract.pl?booksellerid=[% booksellerid %]&amp;contractnumber=[% contractnumber %]">Cancel</a>
+                            <input type="submit" value="Save" /> <a class="cancel" href="/cgi-bin/koha/admin/aqcontract.pl?booksellerid=[% booksellerid %]&amp;contractnumber=[% contractnumber %]">Cancel</a>
                         </fieldset>
                     </form>
             [% END %]
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqplan.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqplan.tt
index 66f6b9f..3856271 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqplan.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/aqplan.tt
@@ -2,41 +2,12 @@
 [% INCLUDE 'doc-head-open.inc' %]
 <title>Koha &rsaquo; Administration &rsaquo; Budgets &rsaquo; Funds &rsaquo; Planning for [% budget_period_description %] by [% authcat %]</title>
 [% INCLUDE 'doc-head-close.inc' %]
-<script type="text/javascript" src="[% interface %]/lib/jquery/plugins/jquery.checkboxes.min.js"></script>
 <script type="text/javascript" src="[% interface %]/[% theme %]/js/acq.js"></script>
 <script type="text/javascript">
 //<![CDATA[
-    function Check(f) {
-            var ok=1;
-            var _alertString="";
-            var alertString2;
-            var arr = document.getElementsByName('est_total')
-
-            for ( var i=0, len=arr.length; i<len; ++i ){
-                    var tot = arr[i].innerHTML;
-
-                    if (tot == 'NaN') {
-                        _alertString += "\n- " + _("One or more cell values is non-numeric");
-                    }
-            }
-
-            if (_alertString.length==0) {
-
-                var op  = document.createElement('input');
-                op.setAttribute("type","hidden");
-                op.setAttribute("name","op");
-                op.setAttribute("value","save");   //ohh the pain...
-
-                document.Aform.appendChild(op);
-                document.Aform.submit()
-
-            } else {
-                    alertString2  = _("Form not submitted because of the following problem(s)");
-                    alertString2 += "\n------------------------------------------------------------------------------------\n";
-                    alertString2 += _alertString;
-                    alert(alertString2);
-            }
-    }
+    jQuery.validator.addClassRules("plan_entry", {
+        number: true // require that input be a number in fields with the class 'plan_entry'
+    });
     $(document).ready(function() {
         $("#selections input").on("change",function(e){
             var num = $(this).attr("id");
@@ -54,10 +25,18 @@
                 }
             }
         });
+        $(".plan_entry").on("change",function(){
+            calcTotalRow( this );
+        });
+        $("#Aform").validate();
+        $(".auto_fill_row").on("click",function(){
+            var budget_id = $(this).data("budget-id");
+            autoFillRow( budget_id );
+        });
     });
 //]]>
 </script>
-<style type="text/css">td.locked { background-image: url('[% interface %]/[% theme %]/img/locked.png'); padding-left : 20px; background-repeat: no-repeat; background-position: 5px 5px; } a.control { font-size:85%;text-decoration:none; }</style>
+<style type="text/css">td.locked { background-image: url('[% interface %]/[% theme %]/img/locked.png'); padding-left : 20px; background-repeat: no-repeat; background-position: 5px 5px; } a.control { font-size:85%;text-decoration:none; } .plan_entry { text-align: right;  width:90%; } </style>
 </head>
 <body id="admin_aqplan" class="admin">
 [% INCLUDE 'header.inc' %]
@@ -150,7 +129,7 @@
                     [% line.estimated_amount %]&nbsp;
                     <input type="hidden" style="text-align: right;"  name="[% line.cell_name %]" value="[% line.estimated_amount %]"   />
                 [% ELSE %]
-                    <input type="text" style="text-align: right;  width:90%; " size="6" name="[% line.cell_name %]"  value="[% line.estimated_amount %]"  id="budget_[% line.budget_id %][% line.colnum %]" class="plan_entry_[% line.budget_id %]" onchange="calcTotalRow(this);" />
+                    <input type="text" size="6" name="[% line.cell_name %]"  value="[% line.estimated_amount %]"  id="budget_[% line.budget_id %][% line.colnum %]" class="plan_entry plan_entry_[% line.budget_id %]" />
                 [% END %]
 
                 </td></tr>
@@ -188,7 +167,7 @@
 
         <td>
              [% UNLESS ( budget_line.budget_lock ) %]
-                    <input type="button"  onclick="autoFillRow('[% budget_line.budget_id %]')" value="Auto-fill row"/>
+                    <input type="button" class="auto_fill_row" data-budget-id="[% budget_line.budget_id %]" value="Auto-fill row"/>
             [% ELSE %]
              <div style="color:red;">not owned</div>
             [% END %]
@@ -202,7 +181,7 @@
     [% IF ( budget_period_locked ) %]
         <!-- <input STYLE="background: gray;"   type="submit" value="Save" disabled="disabled"/> -->
     [% ELSE %]
-        <fieldset class="action"><input type="button"  onclick="Check(this.form)" value="Save"/></fieldset>
+        <fieldset class="action"><input type="submit" value="Save"/></fieldset>
     [% END %]
 
 
-- 
2.9.2