From 8b23255595364dcce3fb0b9dbef64f10959ad234 Mon Sep 17 00:00:00 2001
From: Dan Wells <dbw2@calvin.edu>
Date: Mon, 16 Sep 2013 16:50:53 -0400
Subject: [PATCH] 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>
---
 Open-ILS/src/templates/opac/myopac/lists.tt2 | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

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">
-- 
2.11.0