From: Dan Wells <dbw2@calvin.edu>
Date: Mon, 16 Sep 2013 20:50:53 +0000 (-0400)
Subject: Move list delete check from onclick to onsubmit
X-Git-Tag: sprint4-merge-nov22~2747
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=8b23255595364dcce3fb0b9dbef64f10959ad234;p=working%2FEvergreen.git

Move list delete check from onclick to onsubmit

Cause Brendan Eich says it's better :)

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Remington Steed <rjs7@calvin.edu>
---

diff --git a/Open-ILS/src/templates/opac/myopac/lists.tt2 b/Open-ILS/src/templates/opac/myopac/lists.tt2
index 8672c6e6e9..ccfafb15a4 100644
--- a/Open-ILS/src/templates/opac/myopac/lists.tt2
+++ b/Open-ILS/src/templates/opac/myopac/lists.tt2
@@ -153,14 +153,12 @@
                     [% END %]
                 </div>
             </form>
-            <form action="[% mkurl(ctx.opac_root _ '/myopac/list/update') %]" method="post">
+            <form action="[% mkurl(ctx.opac_root _ '/myopac/list/update') %]" method="post" onsubmit="return confirm('[% l('Do you really want to delete this list?') %]')">
                 <div class="bookbag-controls">
                     <input type="hidden" name="list" value="[% bbag.id %]" />
                     <input type="hidden" name="action" value="delete" />
                     [%- INCLUDE "opac/parts/preserve_params.tt2"; %]
-                    <input type="submit" value="[% l('Delete List') %]"
-                        onClick="if(confirm('[% l('Do you really want to delete this list?') %]')) return true; return false"
-                    />
+                    <input type="submit" value="[% l('Delete List') %]" />
                 </div>
             </form>
             <form action="[% mkurl(ctx.opac_root _ '/myopac/list/print') %]" method="post">