From 582769e5b0c0cc1a921f6e28e1628322beb0249b Mon Sep 17 00:00:00 2001
From: zarzuelo <noreply@pandorafms.org>
Date: Fri, 18 Jun 2010 09:56:49 +0000
Subject: [PATCH] 2010-06-18  Sergio Martin <sergio.martin@artica.es>

	* godmode/users/user_list.php: Fixed the bug of
	delete users in Internet Explorer and added the
	deleting confirmation javascript for bug: 3017261



git-svn-id: https://svn.code.sf.net/p/pandora/code/trunk@2913 c3f86ba8-e40f-0410-aaad-9ba5e7f4b01f
---
 pandora_console/ChangeLog                   | 6 ++++++
 pandora_console/godmode/users/user_list.php | 8 ++++----
 2 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/pandora_console/ChangeLog b/pandora_console/ChangeLog
index adc2c2b6b8..502f49d86c 100644
--- a/pandora_console/ChangeLog
+++ b/pandora_console/ChangeLog
@@ -1,3 +1,9 @@
+2010-06-18  Sergio Martin <sergio.martin@artica.es>
+
+	* godmode/users/user_list.php: Fixed the bug of 
+	delete users in Internet Explorer and added the 
+	deleting confirmation javascript for bug: 3017261
+
 2010-06-17  Sergio Martin <sergio.martin@artica.es>
 
 	* godmode/users/configure_user.php: Fixed the edition
diff --git a/pandora_console/godmode/users/user_list.php b/pandora_console/godmode/users/user_list.php
index c3688533f2..f6158aaa46 100644
--- a/pandora_console/godmode/users/user_list.php
+++ b/pandora_console/godmode/users/user_list.php
@@ -29,7 +29,8 @@ if (! give_acl ($config['id_user'], 0, "UM")) {
 print_page_header (__('User management').' &raquo; '.__('Users defined in Pandora'), "images/god3.png", false, "", true);
 
 if (isset ($_GET["user_del"])) { //delete user
-	$id_user = get_parameter_post ("delete_user");
+	$id_user = get_parameter ("delete_user", 0);
+	//debugPrint($_POST);
 	// Only allow delete user if is not the actual user
 	if($id_user != $config['id_user']){
 		$result = delete_user ($id_user);
@@ -123,16 +124,15 @@ foreach ($info as $user_id => $user_info) {
 	$data[4] = print_string_substr ($user_info["comments"], 24, true);
 
 	if ($config["admin_can_delete_user"] && $user_info['id_user'] != $config['id_user']) {
-		$data[5] = print_input_image ("delete_user", "images/cross.png", $user_info["id_user"], 'border:0px;', true); //Delete user button
+		$data[5] = "<a href='index.php?sec=gusuarios&sec2=godmode/users/user_list&user_del=1&delete_user=".$user_info['id_user']."'>".print_image('images/cross.png', true, array ('title' => __('Delete'), 'onclick' => "if (! confirm ('" .__('Deleting User'). " ". $user_info['id_user'] . ". " . __('Are you sure?') ."')) return false"))."</a>";
 	} else {
 		$data[5] = ''; //Delete button not in this mode
 	}
 	array_push ($table->data, $data);
 }
 
-echo '<form method="post" action="index.php?sec=gusuarios&amp;sec2=godmode/users/user_list&amp;user_del=1">';
 print_table ($table);
-echo '</form>';
+
 unset ($table);