From: Kyle Tomita <ktomita@catalystitservices.com>
Date: Fri, 24 May 2013 17:50:05 +0000 (-0700)
Subject: LP1164720 - Do not allow a list to be created with no name
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=refs%2Fheads%2Fuser%2Fcatalystit%2Flp1164720;p=working%2FEvergreen.git

LP1164720 - Do not allow a list to be created with no name

Added a Javascript function to validate the list name field.
A valid name must be entered and cannot contain only spaces.

Signed-off-by: Kyle Tomita <ktomita@catalystitservices.com>
---

diff --git a/Open-ILS/src/templates/opac/myopac/lists.tt2 b/Open-ILS/src/templates/opac/myopac/lists.tt2
index 0b2977dc93..1abfac1974 100644
--- a/Open-ILS/src/templates/opac/myopac/lists.tt2
+++ b/Open-ILS/src/templates/opac/myopac/lists.tt2
@@ -35,11 +35,24 @@
                     <a href="javascript:void(0);" onclick="alert(document.getElementById('bb_publish_text').innerHTML);"><img alt="[% l('Sharing Help') %]"
                         src="[% ctx.media_prefix %]/images/question-mark.png" /></a>
                 </td>
+                <script type='text/javascript'>
+                    function notEmpty(elem, msg){
+                        var removedWhiteSpace = elem.value.replace(/\s/g, '');
+                        if(0 == removedWhiteSpace.length){
+                            alert(msg);
+                            elem.value = '';
+                            return false;
+                        }
+                        return true;
+                    }
+                </script>
                 <td class="list-create-table-buttons">
                     <input type="submit"
                         value="[% l('Submit') %]"
                         alt="[% l('Submit') %]"
-                        class="opac-button"/>
+                        class="opac-button"
+                        onClick="if(notEmpty(document.getElementById('list_create_name'), '[% l('Please enter a valid name for your list.') %]')) return true; return false"
+                        />
                     &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
                     <input type="reset"
                         value="[% l('Cancel') %]"