Attachment 52476 Details for Bug 16754 – Bug 16754 - Use validation plugin in budgets, planning, and contracts

[patch] Bug 16754 - Use validation plugin in budgets, planning, and contracts

Bug-16754---Use-validation-plugin-in-budgets-plann.patch (text/plain), 22.14 KB, created by Owen Leonard on 2016-06-17 11:57:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2016-06-17 11:57:04 UTC

Size: 22.14 KB

patch

obsolete

>From 560d9bd620bac7b9c5e82827be2b18b12a07125f 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
>Content-Type: text/plain; charset="utf-8"
>
>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.
>---
> .../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">
>     
>     
>@@ -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">
>         
>         
>-        <input type="button" value="Save"  onclick="Check(this.form)" />
>-        
>+        <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 ) %]
>         
>     [% 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.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 16754: 52463 | 52476 | 54057 | 79896 | 79940 | 80169