From 03600cf4ab336d228cf8f0bea092b6cfad7740fd Mon Sep 17 00:00:00 2001
From: Ryan Walberg <generalpf@gmail.com>
Date: Mon, 26 Nov 2012 03:23:33 +0000
Subject: [PATCH] moved the javascript confirms to the head

---
 src/templates/home.tpl  | 26 ++++++++++++++++++--------
 src/templates/users.tpl | 19 ++++++++++++-------
 2 files changed, 30 insertions(+), 15 deletions(-)

diff --git a/src/templates/home.tpl b/src/templates/home.tpl
index 3c8fe39..47d43c9 100644
--- a/src/templates/home.tpl
+++ b/src/templates/home.tpl
@@ -27,12 +27,6 @@ Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 	<script src="lightbox/js/jquery.lightbox-0.5.min.js"></script>
 	
 	<script language="JavaScript" type="text/javascript">
-		function confirmUnshop(fullname) {
-			return window.confirm("Are you sure you no longer wish to shop for " + fullname + "?");
-		}
-		function confirmItemDelete(desc) {
-			return window.confirm("Are you sure you want to delete " + desc + "?");
-		}
 		$(document).ready(function() {
 			$('a[rel=lightbox]').lightBox({
 				imageLoading: 'lightbox/images/lightbox-ico-loading.gif',
@@ -41,6 +35,22 @@ Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 				imageBtnNext: 'lightbox/images/lightbox-btn-next.gif'
 			});
 			$('a[rel=popover]').removeAttr('href').popover();
+			{if $opt.confirm_item_deletes}
+				$('a[rel=confirmitemdelete]').click(function(event) {
+					var desc = $(this).attr('data-content');
+					if (!window.confirm('Are you sure you want to delete "' + desc + '"?')) {
+						event.preventDefault();
+					}
+				});
+			{/if}
+			{if $opt.shop_requires_approval}
+				$('a[rel=confirmunshop]').click(function(event) {
+					var fn = $(this).attr('data-content');
+					if (!window.confirm('Are you sure you no longer wish to shop for ' + fn + '?')) {
+						event.preventDefault();
+					}
+				});
+			{/if}
 		});
 	</script>
 </head>
@@ -107,7 +117,7 @@ Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 						<td align="right">
 							<a href="receive.php?itemid={$row.itemid}"><img alt="Mark Item Received" src="images/refresh_nav.gif" border="0" title="Mark Item Received" /></a>&nbsp;
 							<a href="item.php?action=edit&itemid={$row.itemid}"><img alt="Edit Item" src="images/write_obj.gif" border="0" title="Edit Item" /></a>&nbsp;
-							<a href="item.php?action=delete&itemid={$row.itemid}"><img alt="Delete Item" src="images/remove.gif" border="0" alt="Delete" title="Delete Item" /></a>
+							<a rel="confirmitemdelete" data-content="{$row.description|escape:'htmlall'}" href="item.php?action=delete&itemid={$row.itemid}"><img alt="Delete Item" src="images/remove.gif" border="0" alt="Delete" title="Delete Item" /></a>
 						</td>
 					</tr>
 				{/foreach}
@@ -160,7 +170,7 @@ Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 							{if $row.itemcount > 0}
 								<a href="shop.php?shopfor={$row.userid}"><img alt="Shop for {$row.fullname|escape:'htmlall'}" src="images/tasks_tsk.gif" border="0" alt="Shop" title="Shop"></a>&nbsp;
 							{/if}
-							<a href="index.php?action=cancel&shopfor={$row.userid}" onclick="return confirmUnshop('{$row.fullname|escape:'javascript'}')"><img src="images/remove.gif" border="0" alt="Don't shop for {$row.fullname|escape:'htmlall'} anymore" title="Don't shop for {$row.fullname|escape:'htmlall'} anymore" /></a>
+							<a rel="confirmunshop" data-content="{$row.fullname|escape:'htmlall'}" href="index.php?action=cancel&shopfor={$row.userid}"><img src="images/remove.gif" border="0" alt="Don't shop for {$row.fullname|escape:'htmlall'} anymore" title="Don't shop for {$row.fullname|escape:'htmlall'} anymore" /></a>
 						</td>
 					</tr>
 				{/foreach}
diff --git a/src/templates/users.tpl b/src/templates/users.tpl
index cc50045..1bf9799 100644
--- a/src/templates/users.tpl
+++ b/src/templates/users.tpl
@@ -21,11 +21,18 @@ Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 	<meta name="viewport" content="width=device-width, initial-scale=1.0">
 	<link href="bootstrap/css/bootstrap.min.css" rel="stylesheet">
 	<link href="bootstrap/css/bootstrap-responsive.css" rel="stylesheet">
-
+	<script src="http://ajax.googleapis.com/ajax/libs/jquery/1.7.2/jquery.min.js"></script>
+	<script src="bootstrap/js/bootstrap.min.js"></script>
+	
 	<script language="JavaScript" type="text/javascript">
-		function confirmDelete(fullname) {
-			return confirm("Are you sure you want to delete " + fullname + "?");
-		}
+		$(document).ready(function() {
+			$('a[rel=confirmdeleteuser]').click(function(event) {
+				var u = $(this).attr('data-content');
+				if (!window.confirm('Are you sure you want to delete ' + u + '?')) {
+					event.preventDefault();
+				}
+			});
+		});
 	</script>
 </head>
 <body>
@@ -67,7 +74,7 @@ Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 									<td>{if $row.admin}Yes{else}No{/if}</td>
 									<td align="right">
 										<a href="users.php?action=edit&userid={$row.userid}#userform"><img alt="Edit User" src="images/write_obj.gif" border="0" title="Edit User" /></a> /
-										<a onClick="return confirmDelete('{$row.fullname|escape:'htmlall'}');" href="users.php?action=delete&userid={$row.userid}"><img alt="Delete User" src="images/remove.gif" border="0" title="Delete User" /></a> /
+										<a rel="confirmdeleteuser" data-content="{$row.fullname|escape:'htmlall'}" href="users.php?action=delete&userid={$row.userid}"><img alt="Delete User" src="images/remove.gif" border="0" title="Delete User" /></a> /
 										{if $row.email != ''}
 											<a href="users.php?action=reset&userid={$row.userid}&email={$row.email|escape:'htmlall'}">Reset Pwd</a>
 										{else}
@@ -142,7 +149,5 @@ Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 			</div>
 		</div>
 	</div>
-	<script src="http://ajax.googleapis.com/ajax/libs/jquery/1.7.2/jquery.min.js"></script>
-	<script src="bootstrap/js/bootstrap.min.js"></script>
 </body>
 </html>