From 551989a9bd9b134858f6709a2756ca9dcec82fd4 Mon Sep 17 00:00:00 2001
From: Marius Cramer <m.cramer@pixcept.de>
Date: Mon, 05 Jan 2015 08:34:48 -0500
Subject: [PATCH] Merge branch 'aps_remote' into 'master'

---
 interface/lib/classes/aps_guicontroller.inc.php |  220 ++++++++++++++++++++++++++-----------------------------
 1 files changed, 104 insertions(+), 116 deletions(-)

diff --git a/interface/lib/classes/aps_guicontroller.inc.php b/interface/lib/classes/aps_guicontroller.inc.php
index f6a0ff4..1f18628 100644
--- a/interface/lib/classes/aps_guicontroller.inc.php
+++ b/interface/lib/classes/aps_guicontroller.inc.php
@@ -211,8 +211,93 @@
 		return true;
 	}
 
-
-
+	public function createDatabaseForPackageInstance(&$settings, $websrv) {
+		global $app;
+	
+		$app->uses('tools_sites');
+	
+		$global_config = $app->getconf->get_global_config('sites');
+	
+		$tmp = array();
+		$tmp['parent_domain_id'] = $websrv['domain_id'];
+		$tmp['sys_groupid'] = $websrv['sys_groupid'];
+		$dbname_prefix = $app->tools_sites->replacePrefix($global_config['dbname_prefix'], $tmp);
+		$dbuser_prefix = $app->tools_sites->replacePrefix($global_config['dbuser_prefix'], $tmp);
+		unset($tmp);
+	
+		// get information if the webserver is a db server, too
+		$web_server = $app->db->queryOneRecord("SELECT server_id,server_name,db_server FROM server WHERE server_id  = ".$app->functions->intval($websrv['server_id']));
+		if($web_server['db_server'] == 1) {
+			// create database on "localhost" (webserver)
+			$mysql_db_server_id = $app->functions->intval($websrv['server_id']);
+			$settings['main_database_host'] = 'localhost';
+			$mysql_db_remote_access = 'n';
+			$mysql_db_remote_ips = '';
+		} else {
+			//* get the default database server of the client
+			$client = $app->db->queryOneRecord("SELECT default_dbserver FROM sys_group, client WHERE sys_group.client_id = client.client_id and sys_group.groupid = ".$app->functions->intval($websrv['sys_groupid']));
+			if(is_array($client) && $client['default_dbserver'] > 0 && $client['default_dbserver'] != $websrv['server_id']) {
+				$mysql_db_server_id =  $app->functions->intval($client['default_dbserver']);
+				$dbserver_config = $web_config = $app->getconf->get_server_config($app->functions->intval($mysql_db_server_id), 'server');
+				$settings['main_database_host'] = $dbserver_config['ip_address'];
+				$mysql_db_remote_access = 'y';
+				$webserver_config = $app->getconf->get_server_config($app->functions->intval($websrv['server_id']), 'server');
+				$mysql_db_remote_ips = $webserver_config['ip_address'];
+			} else {
+				/* I left this in place for a fallback that should NEVER! happen.
+				 * if we reach this point it means that there is NO default db server for the client
+				* AND the webserver has NO db service enabled.
+				* We have to abort the aps installation here... so I added a return false
+				* although this does not present any error message to the user.
+				*/
+				return false;
+	
+				/*$mysql_db_server_id = $websrv['server_id'];
+				 $settings['main_database_host'] = 'localhost';
+				$mysql_db_remote_access = 'n';
+				$mysql_db_remote_ips = '';*/
+			}
+		}
+		
+		if (empty($settings['main_database_name'])) {
+			//* Find a free db name for the app
+			for($n = 1; $n <= 1000; $n++) {
+				$mysql_db_name = $app->db->quote(($dbname_prefix != '' ? $dbname_prefix.'aps'.$n : uniqid('aps')));
+				$tmp = $app->db->queryOneRecord("SELECT count(database_id) as number FROM web_database WHERE database_name = '".$app->db->quote($mysql_db_name)."'");
+				if($tmp['number'] == 0) break;
+			}
+			$settings['main_database_name'] = $mysql_db_name;
+		}
+		if (empty($settings['main_database_login'])) {
+			//* Find a free db username for the app
+			for($n = 1; $n <= 1000; $n++) {
+				$mysql_db_user = $app->db->quote(($dbuser_prefix != '' ? $dbuser_prefix.'aps'.$n : uniqid('aps')));
+				$tmp = $app->db->queryOneRecord("SELECT count(database_user_id) as number FROM web_database_user WHERE database_user = '".$app->db->quote($mysql_db_user)."'");
+				if($tmp['number'] == 0) break;
+			}
+			$settings['main_database_login'] = $mysql_db_user;
+		}
+		
+		//* Create the mysql database user if not existing
+		$tmp = $app->db->queryOneRecord("SELECT database_user_id FROM web_database_user WHERE database_user = '".$app->db->quote($settings['main_database_login'])."'");
+		if(!$tmp) {
+			$insert_data = "(`sys_userid`, `sys_groupid`, `sys_perm_user`, `sys_perm_group`, `sys_perm_other`, `server_id`, `database_user`, `database_user_prefix`, `database_password`)
+                                          VALUES( ".$app->functions->intval($websrv['sys_userid']).", ".$app->functions->intval($websrv['sys_groupid']).", 'riud', '".$app->functions->intval($websrv['sys_perm_group'])."', '', 0, '".$settings['main_database_login']."', '".$app->db->quote($dbuser_prefix) . "', PASSWORD('".$settings['main_database_password']."'))";
+			$mysql_db_user_id = $app->db->datalogInsert('web_database_user', $insert_data, 'database_user_id');
+		}
+		else $mysql_db_user_id = $tmp['database_user_id'];
+		
+		//* Create the mysql database if not existing
+		$tmp = $app->db->queryOneRecord("SELECT count(database_id) as number FROM web_database WHERE database_name = '".$app->db->quote($settings['main_database_name'])."'");
+		if($tmp['number'] == 0) {
+			$insert_data = "(`sys_userid`, `sys_groupid`, `sys_perm_user`, `sys_perm_group`, `sys_perm_other`, `server_id`, `parent_domain_id`, `type`, `database_name`, `database_name_prefix`, `database_user_id`, `database_ro_user_id`, `database_charset`, `remote_access`, `remote_ips`, `backup_copies`, `active`, `backup_interval`)
+                                          VALUES( ".$app->functions->intval($websrv['sys_userid']).", ".$app->functions->intval($websrv['sys_groupid']).", 'riud', '".$app->functions->intval($websrv['sys_perm_group'])."', '', $mysql_db_server_id, ".$app->functions->intval($websrv['domain_id']).", 'mysql', '".$settings['main_database_name']."', '" . $app->db->quote($dbname_prefix) . "', '$mysql_db_user_id', 0, '', '$mysql_db_remote_access', '$mysql_db_remote_ips', ".$app->functions->intval($websrv['backup_copies']).", 'y', '".$app->functions->intval($websrv['backup_interval'])."')";
+			$app->db->datalogInsert('web_database', $insert_data, 'database_id');
+		}
+		
+		return true;
+	}
+	
 	/**
 	 * Creates a new database record for the package instance and
 	 * an install task
@@ -250,86 +335,15 @@
 		}
 
 
-		//* Create the MySQL database for the application
+		//* Create the MySQL database for the application if necessary
 		$pkg = $app->db->queryOneRecord('SELECT * FROM aps_packages WHERE id = '.$app->db->quote($packageid).';');
 		$metafile = $this->interface_pkg_dir.'/'.$pkg['path'].'/APP-META.xml';
 		$sxe = $this->readInMetaFile($metafile);
 
 		$db_id = parent::getXPathValue($sxe, '//db:id');
 		if (!empty($db_id)) {
-			$global_config = $app->getconf->get_global_config('sites');
-
-			$tmp = array();
-			$tmp['parent_domain_id'] = $websrv['domain_id'];
-			$tmp['sys_groupid'] = $websrv['sys_groupid'];
-			$dbname_prefix = $app->tools_sites->replacePrefix($global_config['dbname_prefix'], $tmp);
-			$dbuser_prefix = $app->tools_sites->replacePrefix($global_config['dbuser_prefix'], $tmp);
-			unset($tmp);
-
-			// get information if the webserver is a db server, too
-			$web_server = $app->db->queryOneRecord("SELECT server_id,server_name,db_server FROM server WHERE server_id  = ".$app->functions->intval($websrv['server_id']));
-			if($web_server['db_server'] == 1) {
-				// create database on "localhost" (webserver)
-				$mysql_db_server_id = $app->functions->intval($websrv['server_id']);
-				$mysql_db_host = 'localhost';
-				$mysql_db_remote_access = 'n';
-				$mysql_db_remote_ips = '';
-			} else {
-				//* get the default database server of the client
-				$client = $app->db->queryOneRecord("SELECT default_dbserver FROM sys_group, client WHERE sys_group.client_id = client.client_id and sys_group.groupid = ".$app->functions->intval($websrv['sys_groupid']));
-				if(is_array($client) && $client['default_dbserver'] > 0 && $client['default_dbserver'] != $websrv['server_id']) {
-					$mysql_db_server_id =  $app->functions->intval($client['default_dbserver']);
-					$dbserver_config = $web_config = $app->getconf->get_server_config($app->functions->intval($mysql_db_server_id), 'server');
-					$mysql_db_host = $dbserver_config['ip_address'];
-					$mysql_db_remote_access = 'y';
-					$webserver_config = $app->getconf->get_server_config($app->functions->intval($websrv['server_id']), 'server');
-					$mysql_db_remote_ips = $webserver_config['ip_address'];
-				} else {
-					/* I left this in place for a fallback that should NEVER! happen.
-                     * if we reach this point it means that there is NO default db server for the client
-                     * AND the webserver has NO db service enabled.
-                     * We have to abort the aps installation here... so I added a return false
-                     * although this does not present any error message to the user.
-                     */
-					return false;
-
-					/*$mysql_db_server_id = $websrv['server_id'];
-                    $mysql_db_host = 'localhost';
-                    $mysql_db_remote_access = 'n';
-                    $mysql_db_remote_ips = '';*/
-				}
-			}
-
-			//* Find a free db name for the app
-			for($n = 1; $n <= 1000; $n++) {
-				$mysql_db_name = $app->db->quote(($dbname_prefix != '' ? $dbname_prefix.'aps'.$n : uniqid('aps')));
-				$tmp = $app->db->queryOneRecord("SELECT count(database_id) as number FROM web_database WHERE database_name = '".$app->db->quote($mysql_db_name)."'");
-				if($tmp['number'] == 0) break;
-			}
-			//* Find a free db username for the app
-			for($n = 1; $n <= 1000; $n++) {
-				$mysql_db_user = $app->db->quote(($dbuser_prefix != '' ? $dbuser_prefix.'aps'.$n : uniqid('aps')));
-				$tmp = $app->db->queryOneRecord("SELECT count(database_user_id) as number FROM web_database_user WHERE database_user = '".$app->db->quote($mysql_db_user)."'");
-				if($tmp['number'] == 0) break;
-			}
-
-			$mysql_db_password = $settings['main_database_password'];
-
-			//* Create the mysql database user
-			$insert_data = "(`sys_userid`, `sys_groupid`, `sys_perm_user`, `sys_perm_group`, `sys_perm_other`, `server_id`, `database_user`, `database_user_prefix`, `database_password`)
-					  VALUES( ".$app->functions->intval($websrv['sys_userid']).", ".$app->functions->intval($websrv['sys_groupid']).", 'riud', '".$app->functions->intval($websrv['sys_perm_group'])."', '', 0, '$mysql_db_user', '".$app->db->quote($dbuser_prefix) . "', PASSWORD('$mysql_db_password'))";
-			$mysql_db_user_id = $app->db->datalogInsert('web_database_user', $insert_data, 'database_user_id');
-
-			//* Create the mysql database
-			$insert_data = "(`sys_userid`, `sys_groupid`, `sys_perm_user`, `sys_perm_group`, `sys_perm_other`, `server_id`, `parent_domain_id`, `type`, `database_name`, `database_name_prefix`, `database_user_id`, `database_ro_user_id`, `database_charset`, `remote_access`, `remote_ips`, `backup_copies`, `active`, `backup_interval`)
-					  VALUES( ".$app->functions->intval($websrv['sys_userid']).", ".$app->functions->intval($websrv['sys_groupid']).", 'riud', '".$app->functions->intval($websrv['sys_perm_group'])."', '', $mysql_db_server_id, ".$app->functions->intval($websrv['domain_id']).", 'mysql', '$mysql_db_name', '" . $app->db->quote($dbname_prefix) . "', '$mysql_db_user_id', 0, '', '$mysql_db_remote_access', '$mysql_db_remote_ips', ".$app->functions->intval($websrv['backup_copies']).", 'y', '".$app->functions->intval($websrv['backup_interval'])."')";
-			$app->db->datalogInsert('web_database', $insert_data, 'database_id');
-
-			//* Add db details to package settings
-			$settings['main_database_host'] = $mysql_db_host;
-			$settings['main_database_name'] = $mysql_db_name;
-			$settings['main_database_login'] = $mysql_db_user;
-
+			// mysql-database-name is updated inside if not set already
+			if (!$this->createDatabaseForPackageInstance($settings, $websrv)) return false;
 		}
 
 		//* Insert new package instance
@@ -348,15 +362,13 @@
 		$app->db->datalogUpdate('aps_instances', "instance_status = ".INSTANCE_INSTALL, 'id', $InstanceID);
 	}
 
-
-
 	/**
 	 * Sets the status of an instance to "should be removed" and creates a
 	 * datalog entry to give the ISPConfig server a real removal advice
 	 *
 	 * @param $instanceid the instance to delete
 	 */
-	public function deleteInstance($instanceid)
+	public function deleteInstance($instanceid, $keepdatabase = false)
 	{
 		global $app;
 		/*
@@ -370,46 +382,18 @@
         $app->db->datalogSave('aps', 'DELETE', 'id', $instanceid, array(), $datalog);
 		*/
 
-		$sql = "SELECT web_database.database_id as database_id, web_database.database_user_id as `database_user_id` FROM aps_instances_settings, web_database WHERE aps_instances_settings.value = web_database.database_name AND aps_instances_settings.name = 'main_database_name' AND aps_instances_settings.instance_id = ".$instanceid." LIMIT 0,1";
-		$tmp = $app->db->queryOneRecord($sql);
-		if($tmp['database_id'] > 0) $app->db->datalogDelete('web_database', 'database_id', $tmp['database_id']);
-
-		$database_user = $tmp['database_user_id'];
-		$tmp = $app->db->queryOneRecord("SELECT COUNT(*) as `cnt` FROM `web_database` WHERE `database_user_id` = '" . $app->functions->intval($database_user) . "' OR `database_ro_user_id` = '" . $app->functions->intval($database_user) . "'");
-		if($tmp['cnt'] < 1) $app->db->datalogDelete('web_database_user', 'database_user_id', $database_user);
+		if (!$keepdatabase) {
+			$sql = "SELECT web_database.database_id as database_id, web_database.database_user_id as `database_user_id` FROM aps_instances_settings, web_database WHERE aps_instances_settings.value = web_database.database_name AND aps_instances_settings.name = 'main_database_name' AND aps_instances_settings.instance_id = ".$instanceid." LIMIT 0,1";
+			$tmp = $app->db->queryOneRecord($sql);
+			if($tmp['database_id'] > 0) $app->db->datalogDelete('web_database', 'database_id', $tmp['database_id']);
+	
+			$database_user = $tmp['database_user_id'];
+			$tmp = $app->db->queryOneRecord("SELECT COUNT(*) as `cnt` FROM `web_database` WHERE `database_user_id` = '" . $app->functions->intval($database_user) . "' OR `database_ro_user_id` = '" . $app->functions->intval($database_user) . "'");
+			if($tmp['cnt'] < 1) $app->db->datalogDelete('web_database_user', 'database_user_id', $database_user);
+		}
 
 		$app->db->datalogUpdate('aps_instances', "instance_status = ".INSTANCE_REMOVE, 'id', $instanceid);
 
-	}
-
-
-
-	/**
-	 * Sets the status of an instance to "installation planned" and creates a
-	 * datalog entry to re-install the package. The existing package is simply overwritten.
-	 *
-	 * @param $instanceid the instance to delete
-	 */
-	public function reinstallInstance($instanceid)
-	{
-		global $app;
-
-		/*
-		$app->db->query("UPDATE aps_instances SET instance_status = ".INSTANCE_INSTALL." WHERE id = ".$instanceid.";");
-
-        $webserver_id = $this->getInstanceDataForDatalog($instanceid);
-        if($webserver_id == '') return;
-
-        // Create a sys_datalog entry for re-installation
-        $datalog = array('instance_id' => $instanceid, 'server_id' => $webserver_id);
-        $app->db->datalogSave('aps', 'INSERT', 'id', $instanceid, array(), $datalog);
-		*/
-
-		$sql = "SELECT web_database.database_id as database_id FROM aps_instances_settings, web_database WHERE aps_instances_settings.value = web_database.database_name AND aps_instances_settings.value =  aps_instances_settings.name = 'main_database_name' AND aps_instances_settings.instance_id = ".$app->db->quote($instanceid)." LIMIT 0,1";
-		$tmp = $app->db->queryOneRecord($sql);
-		if($tmp['database_id'] > 0) $app->db->datalogDelete('web_database', 'database_id', $tmp['database_id']);
-
-		$app->db->datalogUpdate('aps_instances', "instance_status = ".INSTANCE_INSTALL, 'id', $instanceid);
 	}
 
 	/**
@@ -608,6 +592,10 @@
 		if(isset($pkg_details['Requirements Database'])
 			&& $pkg_details['Requirements Database'] != '')
 		{
+			if (isset($postinput['main_database_host'])) $input['main_database_host'] = $postinput['main_database_host'];
+			if (isset($postinput['main_database_name'])) $input['main_database_name'] = $postinput['main_database_name'];
+			if (isset($postinput['main_database_login'])) $input['main_database_login'] = $postinput['main_database_login'];
+			
 			if(isset($postinput['main_database_password']))
 			{
 				if($postinput['main_database_password'] == '') $error[] = $app->lng('error_no_database_pw');

--
Gitblit v1.9.1