From 4e7af155949b33ce78a8fc489d87b08ff3901772 Mon Sep 17 00:00:00 2001
From: tbrehm <t.brehm@ispconfig.org>
Date: Tue, 30 Oct 2012 07:30:33 -0400
Subject: [PATCH] Fixed: FS#2398 - Setting autoresponder start/end dates for a mailbox fails

---
 interface/web/client/client_del.php |   42 +++++++++++++++++++++++++++---------------
 1 files changed, 27 insertions(+), 15 deletions(-)

diff --git a/interface/web/client/client_del.php b/interface/web/client/client_del.php
index b125ba4..5015a55 100644
--- a/interface/web/client/client_del.php
+++ b/interface/web/client/client_del.php
@@ -44,6 +44,7 @@
 
 //* Check permissions for module
 $app->auth->check_module_permissions('client');
+if($conf['demo_mode'] == true) $app->error('This function is disabled in demo mode.');
 
 $app->uses('tpl,tform');
 $app->load('tform_actions');
@@ -52,6 +53,9 @@
 	
 	function onDelete() {
 		global $app, $conf,$list_def_file,$tform_def_file;
+		
+		// Loading tform framework
+        if(!is_object($app->tform)) $app->uses('tform');
 		
 		if($_POST["confirm"] == 'yes') {
 			parent::onDelete();
@@ -62,25 +66,25 @@
 		$app->tpl->setInclude('content_tpl', 'templates/client_del.htm');
 		
 		include_once($list_def_file);
-
-        // Loading tform framework
-        if(!is_object($app->tform)) $app->uses('tform');
-
-        // Load table definition from file
+		
+		// Load table definition from file
         $app->tform->loadFormDef($tform_def_file);
 		
-		$this->id = intval($_REQUEST["id"]);
+		$this->id = $app->functions->intval($_REQUEST["id"]);
 		
 		$this->dataRecord = $app->tform->getDataRecord($this->id);
-		$client_id = intval($this->dataRecord['client_id']);
-		//$parent_client_id = intval($this->dataRecord['parent_client_id']);
+		$client_id = $app->functions->intval($this->dataRecord['client_id']);
+
+        
+		//$parent_client_id = $app->functions->intval($this->dataRecord['parent_client_id']);
 		//$parent_user = $app->db->queryOneRecord("SELECT userid FROM sys_user WHERE client_id = $parent_client_id");
 		$client_group = $app->db->queryOneRecord("SELECT groupid FROM sys_group WHERE client_id = $client_id");
 		
 		// Get all records (sub-clients, mail, web, etc....)  of this client.
-		$tables = 'client,dns_rr,dns_soa,ftp_user,mail_access,mail_content_filter,mail_domain,mail_forwarding,mail_get,mail_user,mail_user_filter,shell_user,spamfilter_users,support_message,web_database,web_domain,web_traffic';
+		$tables = 'cron,client,dns_rr,dns_soa,dns_slave,ftp_user,mail_access,mail_content_filter,mail_domain,mail_forwarding,mail_get,mail_user,mail_user_filter,shell_user,spamfilter_users,support_message,web_database,web_domain';
 		$tables_array = explode(',',$tables);
-		$client_group_id = intval($client_group['groupid']);
+		$client_group_id = $app->functions->intval($client_group['groupid']);
+		
 		$table_list = array();
 		if($client_group_id > 1) {
 			foreach($tables_array as $table) {
@@ -111,11 +115,11 @@
 	function onAfterDelete() {
 		global $app, $conf;
 		
-		$client_id = intval($this->dataRecord['client_id']);
+		$client_id = $app->functions->intval($this->dataRecord['client_id']);
 		
 		if($client_id > 0) {			
 			// remove the group of the client from the resellers group
-			$parent_client_id = intval($this->dataRecord['parent_client_id']);
+			$parent_client_id = $app->functions->intval($this->dataRecord['parent_client_id']);
 			$parent_user = $app->db->queryOneRecord("SELECT userid FROM sys_user WHERE client_id = $parent_client_id");
 			$client_group = $app->db->queryOneRecord("SELECT groupid FROM sys_group WHERE client_id = $client_id");
 			$app->auth->remove_group_from_user($parent_user['userid'],$client_group['groupid']);
@@ -127,9 +131,9 @@
 			$app->db->query("DELETE FROM sys_user WHERE client_id = $client_id");
 			
 			// Delete all records (sub-clients, mail, web, etc....)  of this client.
-			$tables = 'client,dns_rr,dns_soa,ftp_user,mail_access,mail_content_filter,mail_domain,mail_forwarding,mail_get,mail_user,mail_user_filter,shell_user,spamfilter_users,support_message,web_database,web_domain,web_traffic';
+			$tables = 'client,dns_rr,dns_soa,dns_slave,ftp_user,mail_access,mail_content_filter,mail_domain,mail_forwarding,mail_get,mail_user,mail_user_filter,shell_user,spamfilter_users,support_message,web_database,web_domain,web_folder,web_folder_user,domain';
 			$tables_array = explode(',',$tables);
-			$client_group_id = intval($client_group['groupid']);
+			$client_group_id = $app->functions->intval($client_group['groupid']);
 			if($client_group_id > 1) {
 				foreach($tables_array as $table) {
 					if($table != '') {
@@ -145,6 +149,14 @@
 							if(is_array($records)) {
 								foreach($records as $rec) {
 									$app->db->datalogDelete($table, $index_field, $rec[$index_field]);
+									//* Delete traffic records that dont have a sys_groupid column
+									if($table == 'web_domain') {
+										$app->db->query("DELETE FROM web_traffic WHERE hostname = '".$app->db->quote($rec['domain'])."'");
+									}
+									//* Delete mail_traffic records that dont have a sys_groupid
+									if($table == 'mail_user') {
+										$app->db->query("DELETE FROM mail_traffic WHERE mailuser_id = '".$app->db->quote($rec['mailuser_id'])."'");
+									}
 								}
 							}
 						}
@@ -163,4 +175,4 @@
 $page = new page_action;
 $page->onDelete()
 
-?>
\ No newline at end of file
+?>

--
Gitblit v1.9.1