From 1a14f4d8a05ff1ccc781bf2c2a2569244548f9c6 Mon Sep 17 00:00:00 2001
From: tbrehm <t.brehm@ispconfig.org>
Date: Fri, 14 Nov 2008 11:03:24 -0500
Subject: [PATCH] Fixed #250. Services are only restarted if a startscript is defined in the $config variable during installation or update.

---
 install/update.php |   42 ++++++++++++++++++++++++++----------------
 1 files changed, 26 insertions(+), 16 deletions(-)

diff --git a/install/update.php b/install/update.php
index 3200a6d..fe77547 100644
--- a/install/update.php
+++ b/install/update.php
@@ -52,6 +52,7 @@
 
 //** Installer/updater logfile
 define('ISPC_LOG_FILE', '/var/log/ispconfig_install.log');
+define('ISPC_INSTALL_ROOT', realpath(dirname(__FILE__).'/../'));
 
 //** Check for ISPConfig 2.x versions
 if(is_dir('/root/ispconfig') || is_dir('/home/admispconfig')) {
@@ -143,6 +144,13 @@
 	system("mysql -h ".$conf['mysql']['host']." -u ".$conf['mysql']['admin_user']." ".$conf['mysql']['database']." < existing_db.sql");
 }
 
+// create a backup copy of the ispconfig database in the root folder
+$backup_db_name = '/root/ispconfig_db_backup_'.@date('Y-m-d_h-i').'.sql';
+copy('existing_db.sql',$backup_db_name);
+exec("chmod 700 $backup_db_name");
+exec("chown root:root $backup_db_name");
+
+
 //** Update server ini
 $tmp_server_rec = $inst->db->queryOneRecord("SELECT config FROM server WHERE server_id = ".$conf['server_id']);
 $old_ini_array = ini_to_array(stripslashes($tmp_server_rec['config']));
@@ -150,9 +158,11 @@
 $tpl_ini_array = ini_to_array(rf('tpl/server.ini.master'));
 
 // update the new template with the old values
-foreach($old_ini_array as $tmp_section_name => $tmp_section_content) {
-	foreach($tmp_section_content as $tmp_var_name => $tmp_var_content) {
-		$tpl_ini_array[$tmp_section_name][$tmp_var_name] = $tmp_var_content;
+if(is_array($old_ini_array)) {
+	foreach($old_ini_array as $tmp_section_name => $tmp_section_content) {
+		foreach($tmp_section_content as $tmp_var_name => $tmp_var_content) {
+			$tpl_ini_array[$tmp_section_name][$tmp_var_name] = $tmp_var_content;
+		}
 	}
 }
 
@@ -240,19 +250,19 @@
 //** Restart services:
 if($reconfigure_services_answer == 'yes') {
 	swriteln('Restarting services ...');
-	system($conf['init_scripts'].'/'.$conf['mysql']['init_script'].' restart');
-	system($conf['init_scripts'].'/'.$conf['postfix']['init_script'].' restart');
-	system($conf['init_scripts'].'/'.$conf['saslauthd']['init_script'].' restart');
-	system($conf['init_scripts'].'/'.$conf['amavis']['init_script'].' restart');
-	system($conf['init_scripts'].'/'.$conf['clamav']['init_script'].' restart');
-	system($conf['init_scripts'].'/'.$conf['courier']['courier-authdaemon'].' restart');
-	system($conf['init_scripts'].'/'.$conf['courier']['courier-imap'].' restart');
-	system($conf['init_scripts'].'/'.$conf['courier']['courier-imap-ssl'].' restart');
-	system($conf['init_scripts'].'/'.$conf['courier']['courier-pop'].' restart');
-	system($conf['init_scripts'].'/'.$conf['courier']['courier-pop-ssl'].' restart');
-	system($conf['init_scripts'].'/'.$conf['apache']['init_script'].' restart');
-	system($conf['init_scripts'].'/'.$conf['pureftpd']['init_script'].' restart');
-	system($conf['init_scripts'].'/'.$conf['mydns']['init_script'].' restart &> /dev/null');
+	if($conf['mysql']['init_script'] != '')				system($conf['init_scripts'].'/'.$conf['mysql']['init_script'].' restart');
+	if($conf['postfix']['init_script'] != '')			system($conf['init_scripts'].'/'.$conf['postfix']['init_script'].' restart');
+	if($conf['saslauthd']['init_script'] != '')			system($conf['init_scripts'].'/'.$conf['saslauthd']['init_script'].' restart');
+	if($conf['amavis']['init_script'] != '')			system($conf['init_scripts'].'/'.$conf['amavis']['init_script'].' restart');
+	if($conf['clamav']['init_script'] != '')			system($conf['init_scripts'].'/'.$conf['clamav']['init_script'].' restart');
+	if($conf['courier']['courier-authdaemon'] != '') 	system($conf['init_scripts'].'/'.$conf['courier']['courier-authdaemon'].' restart');
+	if($conf['courier']['courier-imap'] != '') 			system($conf['init_scripts'].'/'.$conf['courier']['courier-imap'].' restart');
+	if($conf['courier']['courier-imap-ssl'] != '') 		system($conf['init_scripts'].'/'.$conf['courier']['courier-imap-ssl'].' restart');
+	if($conf['courier']['courier-pop'] != '') 			system($conf['init_scripts'].'/'.$conf['courier']['courier-pop'].' restart');
+	if($conf['courier']['courier-pop-ssl'] != '') 		system($conf['init_scripts'].'/'.$conf['courier']['courier-pop-ssl'].' restart');
+	if($conf['apache']['init_script'] != '') 			system($conf['init_scripts'].'/'.$conf['apache']['init_script'].' restart');
+	if($conf['pureftpd']['init_script'] != '')			system($conf['init_scripts'].'/'.$conf['pureftpd']['init_script'].' restart');
+	if($conf['mydns']['init_script'] != '')				system($conf['init_scripts'].'/'.$conf['mydns']['init_script'].' restart &> /dev/null');
 }
 
 echo "Update finished.\n";

--
Gitblit v1.9.1