From 91ac76131620cb062d3128f2fc23d6d174d9e3ba Mon Sep 17 00:00:00 2001
From: tbrehm <t.brehm@ispconfig.org>
Date: Tue, 03 Jan 2012 08:43:32 -0500
Subject: [PATCH] Fixed: FS#1905 - client_add function in remote api adds duplicate user after returining correct dupliate error.

---
 interface/lib/classes/remoting.inc.php |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/interface/lib/classes/remoting.inc.php b/interface/lib/classes/remoting.inc.php
index 304ab11..3330447 100644
--- a/interface/lib/classes/remoting.inc.php
+++ b/interface/lib/classes/remoting.inc.php
@@ -2158,6 +2158,11 @@
 		$sql = $app->remoting_lib->getSQL($params,'INSERT',0);
 		$app->db->query($sql);
 		
+		//* Check if no system user with that username exists
+		$username = $app->db->quote($params["username"]);
+		$tmp = $app->db->queryOneRecord("SELECT count(userid) as number FROm sys_user WHERE username = '$username'");
+		if($tmp['number'] > 0) $app->remoting_lib->errorMessage .= "Duplicate username<br />";
+		
 		if($app->remoting_lib->errorMessage != '') {
 			$this->server->fault('data_processing_error', $app->remoting_lib->errorMessage);
 			return false;

--
Gitblit v1.9.1