From 805e46f1e2a1b4df5a654202b73b1185b90ec969 Mon Sep 17 00:00:00 2001
From: mcramer <m.cramer@pixcept.de>
Date: Wed, 14 Aug 2013 05:56:36 -0400
Subject: [PATCH] - Fixed: sys_user doesn't use 'y'/'n' but 1/0 - so client_edit set wrong value on canceled y/n switching

---
 interface/web/client/client_edit.php |   18 +++++++++++-------
 1 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/interface/web/client/client_edit.php b/interface/web/client/client_edit.php
index 2becb95..e71b08b 100644
--- a/interface/web/client/client_edit.php
+++ b/interface/web/client/client_edit.php
@@ -228,8 +228,10 @@
 		$sql = "UPDATE client SET default_mailserver = $default_mailserver, default_webserver = $default_webserver, default_dnsserver = $default_dnsserver, default_slave_dnsserver = $default_dnsserver, default_dbserver = $default_dbserver WHERE client_id = ".$this->id;
 		$app->db->query($sql);
 		
-        $app->uses('client_templates');
-        $app->client_templates->update_client_templates($this->id, $this->_template_additional);
+        if(isset($this->dataRecord['template_master'])) {
+            $app->uses('client_templates');
+            $app->client_templates->update_client_templates($this->id, $this->_template_additional);
+        }
 
 		parent::onAfterInsert();
 	}
@@ -351,12 +353,12 @@
         }
         
         if(!isset($this->dataRecord['canceled'])) $this->dataRecord['canceled'] = 'n';
-        if(isset($conf['demo_mode']) && $conf['demo_mode'] != true && isset($this->dataRecord["canceled"]) && $this->dataRecord["canceled"] != $this->oldDataRecord['canceled']) {
+        if(isset($conf['demo_mode']) && $conf['demo_mode'] != true && $this->dataRecord["canceled"] != $this->oldDataRecord['canceled']) {
             if($this->dataRecord['canceled'] == 'y') {
-                $sql = "UPDATE sys_user SET active = 'n' WHERE client_id = " . $this->id;
+                $sql = "UPDATE sys_user SET active = '0' WHERE client_id = " . $this->id;
                 $app->db->query($sql);
             } elseif($this->dataRecord['canceled'] == 'n') {
-                $sql = "UPDATE sys_user SET active = 'y' WHERE client_id = " . $this->id;
+                $sql = "UPDATE sys_user SET active = '1' WHERE client_id = " . $this->id;
                 $app->db->query($sql);
             }
         }
@@ -379,8 +381,10 @@
 			$app->db->query($sql);
 		}
 		
-        $app->uses('client_templates');
-        $app->client_templates->update_client_templates($this->id, $this->_template_additional);
+        if(isset($this->dataRecord['template_master'])) {
+            $app->uses('client_templates');
+            $app->client_templates->update_client_templates($this->id, $this->_template_additional);
+        }
         
 		parent::onAfterUpdate();
 	}

--
Gitblit v1.9.1