From c5d17d3fd1bbdca95dd9230a2ab54f5aef04aef8 Mon Sep 17 00:00:00 2001
From: tbrehm <t.brehm@ispconfig.org>
Date: Fri, 22 Jul 2011 09:48:47 -0400
Subject: [PATCH] Bugfixes in server list in vm server forms.

---
 server/plugins-available/cron_plugin.inc.php |   22 ++++++++++++++++++----
 1 files changed, 18 insertions(+), 4 deletions(-)

diff --git a/server/plugins-available/cron_plugin.inc.php b/server/plugins-available/cron_plugin.inc.php
index c2b4b3a..388ca65 100644
--- a/server/plugins-available/cron_plugin.inc.php
+++ b/server/plugins-available/cron_plugin.inc.php
@@ -97,7 +97,7 @@
             $app->log("Parent domain not found",LOGLEVEL_WARN);
             return 0;
         } elseif($parent_domain["system_user"] == 'root' or $parent_domain["system_group"] == 'root') {
-			$app->log("Websites (and Crons) can not be owned by the root user or group.",LOGLEVEL_WARN);
+			$app->log("Websites (and Crons) cannot be owned by the root user or group.",LOGLEVEL_WARN);
 			return 0;
 		}
 		
@@ -133,8 +133,16 @@
 			exec("setquota -T -u $username 604800 604800 -a &> /dev/null");
 		}
 		
-		// make temp direcory writable for the apache user and the website user
-		exec("chmod 777 ".escapeshellcmd($parent_domain["document_root"]."/tmp"));
+		//TODO : change this when distribution information has been integrated into server record
+        //* Gentoo requires a user to be part of the crontab group.
+        if (file_exists('/etc/gentoo-release')) {
+        	if (strpos($app->system->get_user_groups($username), 'crontab') === false) {
+        		$app->system->add_user_to_group('crontab', $username);
+        	}
+        }
+		
+		// make temp directory writable for the apache and website users
+		chmod(escapeshellcmd($parent_domain["document_root"].'/tmp'), 0777);
 		
         /** TODO READ CRON MASTER **/
         
@@ -214,6 +222,12 @@
         }
         
         $cron_file = escapeshellcmd($cron_config["crontab_dir"].'/ispc_'.$this->parent_domain["system_user"]);
+        //TODO : change this when distribution information has been integrated into server record
+        //* Gentoo vixie-cron requires files to end with .cron in the cron.d directory
+        if (file_exists('/etc/gentoo-release')) {
+        	$cron_file .= '.cron';
+        }
+        
         if($cmd_count > 0) {
             file_put_contents($cron_file, $cron_content);
             $app->log("Wrote Cron file $cron_file with content:\n$cron_content",LOGLEVEL_DEBUG);
@@ -236,4 +250,4 @@
 
 } // end class
 
-?>
\ No newline at end of file
+?>

--
Gitblit v1.9.1