From b1a6a5a3991cec5cd08873b01376e45d0b247f18 Mon Sep 17 00:00:00 2001
From: Marius Cramer <m.cramer@pixcept.de>
Date: Thu, 14 Nov 2013 09:05:33 -0500
Subject: [PATCH] Cleaning up code to match coding guidelines

---
 server/lib/classes/cron.d/500-backup.inc.php |  498 +++++++++++++++++++++++++++---------------------------
 1 files changed, 249 insertions(+), 249 deletions(-)

diff --git a/server/lib/classes/cron.d/500-backup.inc.php b/server/lib/classes/cron.d/500-backup.inc.php
index 7231d7f..6010c75 100644
--- a/server/lib/classes/cron.d/500-backup.inc.php
+++ b/server/lib/classes/cron.d/500-backup.inc.php
@@ -29,301 +29,301 @@
 */
 
 class cronjob_backup extends cronjob {
-    
-    // job schedule
-    protected $_schedule = '0 0 * * *';
-    
-    /* this function is optional if it contains no custom code */
-    public function onPrepare() {
-        global $app;
-        
-        parent::onPrepare();
-    }
-    
-    /* this function is optional if it contains no custom code */
-    public function onBeforeRun() {
-        global $app;
-        
-        return parent::onBeforeRun();
-    }
-    
-    public function onRunJob() {
-        global $app, $conf;
-        
-        #######################################################################################################
-        // Create website backups
-        #######################################################################################################
-        function formatBytes($size, $precision = 2) {
-            $base=log($size)/log(1024);
-            $suffixes=array('','k','M','G','T');
-                return round(pow(1024,$base-floor($base)),$precision).$suffixes[floor($base)];
-        }
 
-        $server_config = $app->getconf->get_server_config($conf['server_id'], 'server');
-        $backup_dir = $server_config['backup_dir'];
-        $backup_mode = $server_config['backup_mode'];
-        if($backup_mode == '') $backup_mode = 'userzip';
+	// job schedule
+	protected $_schedule = '0 0 * * *';
 
-        $web_config = $app->getconf->get_server_config($conf['server_id'], 'web');
-        $http_server_user = $web_config['user'];
+	/* this function is optional if it contains no custom code */
+	public function onPrepare() {
+		global $app;
 
-        if($backup_dir != '') {
+		parent::onPrepare();
+	}
 
-            if(isset($server_config['backup_dir_ftpread']) && $server_config['backup_dir_ftpread'] == 'y') {
-                $backup_dir_permissions = 0755;
-            } else {
-                $backup_dir_permissions = 0750;
-            }
+	/* this function is optional if it contains no custom code */
+	public function onBeforeRun() {
+		global $app;
 
-            if(!is_dir($backup_dir)) {
-                mkdir(escapeshellcmd($backup_dir), $backup_dir_permissions, true);
-            } else {
-                chmod(escapeshellcmd($backup_dir), $backup_dir_permissions);
-            }
+		return parent::onBeforeRun();
+	}
 
-            $sql = "SELECT * FROM web_domain WHERE server_id = '".$conf['server_id']."' AND (type = 'vhost' OR type = 'vhostsubdomain') AND backup_interval != 'none'";
-            $records = $app->db->queryAllRecords($sql);
-            if(is_array($records)) {
-                foreach($records as $rec) {
+	public function onRunJob() {
+		global $app, $conf;
 
-                    //* Do the website backup
-                    if($rec['backup_interval'] == 'daily' or ($rec['backup_interval'] == 'weekly' && date('w') == 0) or ($rec['backup_interval'] == 'monthly' && date('d') == '01')) {
+		//######################################################################################################
+		// Create website backups
+		//######################################################################################################
+		function formatBytes($size, $precision = 2) {
+			$base=log($size)/log(1024);
+			$suffixes=array('', 'k', 'M', 'G', 'T');
+			return round(pow(1024, $base-floor($base)), $precision).$suffixes[floor($base)];
+		}
 
-                        $web_path = $rec['document_root'];
-                        $web_user = $rec['system_user'];
-                        $web_group = $rec['system_group'];
-                        $web_id = $rec['domain_id'];
-                        $web_backup_dir = $backup_dir.'/web'.$web_id;
-                        if(!is_dir($web_backup_dir)) mkdir($web_backup_dir, 0750);
-                        chmod($web_backup_dir, 0750);
-                        //if(isset($server_config['backup_dir_ftpread']) && $server_config['backup_dir_ftpread'] == 'y') {
-                        chown($web_backup_dir, $rec['system_user']);
-                        chgrp($web_backup_dir, $rec['system_group']);
-                        /*} else {
+		$server_config = $app->getconf->get_server_config($conf['server_id'], 'server');
+		$backup_dir = $server_config['backup_dir'];
+		$backup_mode = $server_config['backup_mode'];
+		if($backup_mode == '') $backup_mode = 'userzip';
+
+		$web_config = $app->getconf->get_server_config($conf['server_id'], 'web');
+		$http_server_user = $web_config['user'];
+
+		if($backup_dir != '') {
+
+			if(isset($server_config['backup_dir_ftpread']) && $server_config['backup_dir_ftpread'] == 'y') {
+				$backup_dir_permissions = 0755;
+			} else {
+				$backup_dir_permissions = 0750;
+			}
+
+			if(!is_dir($backup_dir)) {
+				mkdir(escapeshellcmd($backup_dir), $backup_dir_permissions, true);
+			} else {
+				chmod(escapeshellcmd($backup_dir), $backup_dir_permissions);
+			}
+
+			$sql = "SELECT * FROM web_domain WHERE server_id = '".$conf['server_id']."' AND (type = 'vhost' OR type = 'vhostsubdomain') AND backup_interval != 'none'";
+			$records = $app->db->queryAllRecords($sql);
+			if(is_array($records)) {
+				foreach($records as $rec) {
+
+					//* Do the website backup
+					if($rec['backup_interval'] == 'daily' or ($rec['backup_interval'] == 'weekly' && date('w') == 0) or ($rec['backup_interval'] == 'monthly' && date('d') == '01')) {
+
+						$web_path = $rec['document_root'];
+						$web_user = $rec['system_user'];
+						$web_group = $rec['system_group'];
+						$web_id = $rec['domain_id'];
+						$web_backup_dir = $backup_dir.'/web'.$web_id;
+						if(!is_dir($web_backup_dir)) mkdir($web_backup_dir, 0750);
+						chmod($web_backup_dir, 0750);
+						//if(isset($server_config['backup_dir_ftpread']) && $server_config['backup_dir_ftpread'] == 'y') {
+						chown($web_backup_dir, $rec['system_user']);
+						chgrp($web_backup_dir, $rec['system_group']);
+						/*} else {
                             chown($web_backup_dir, 'root');
                             chgrp($web_backup_dir, 'root');
                         }*/
-                        if($backup_mode == 'userzip') {
-                            //* Create a .zip backup as web user and include also files owned by apache / nginx user
-                            $web_backup_file = 'web'.$web_id.'_'.date('Y-m-d_H-i').'.zip';
-                            exec('cd '.escapeshellarg($web_path).' && sudo -u '.escapeshellarg($web_user).' find . -group '.escapeshellarg($web_group).' -print 2> /dev/null | zip -b /tmp --exclude=backup\* --symlinks '.escapeshellarg($web_backup_dir.'/'.$web_backup_file).' -@', $tmp_output, $retval);
-                            if($retval == 0) exec('cd '.escapeshellarg($web_path).' && sudo -u '.escapeshellarg($web_user).' find . -user '.escapeshellarg($http_server_user).' -print 2> /dev/null | zip -b /tmp --exclude=backup\* --update --symlinks '.escapeshellarg($web_backup_dir.'/'.$web_backup_file).' -@', $tmp_output, $retval);
-                        } else {
-                            //* Create a tar.gz backup as root user
-                            $web_backup_file = 'web'.$web_id.'_'.date('Y-m-d_H-i').'.tar.gz';
-                            exec('tar pczf '.escapeshellarg($web_backup_dir.'/'.$web_backup_file).' --exclude=backup\* --directory '.escapeshellarg($web_path).' .', $tmp_output, $retval);
-                        }
-                        if($retval == 0){
-                            chown($web_backup_dir.'/'.$web_backup_file, 'root');
-                            chgrp($web_backup_dir.'/'.$web_backup_file, 'root');
-                            chmod($web_backup_dir.'/'.$web_backup_file, 0750);
+						if($backup_mode == 'userzip') {
+							//* Create a .zip backup as web user and include also files owned by apache / nginx user
+							$web_backup_file = 'web'.$web_id.'_'.date('Y-m-d_H-i').'.zip';
+							exec('cd '.escapeshellarg($web_path).' && sudo -u '.escapeshellarg($web_user).' find . -group '.escapeshellarg($web_group).' -print 2> /dev/null | zip -b /tmp --exclude=backup\* --symlinks '.escapeshellarg($web_backup_dir.'/'.$web_backup_file).' -@', $tmp_output, $retval);
+							if($retval == 0) exec('cd '.escapeshellarg($web_path).' && sudo -u '.escapeshellarg($web_user).' find . -user '.escapeshellarg($http_server_user).' -print 2> /dev/null | zip -b /tmp --exclude=backup\* --update --symlinks '.escapeshellarg($web_backup_dir.'/'.$web_backup_file).' -@', $tmp_output, $retval);
+						} else {
+							//* Create a tar.gz backup as root user
+							$web_backup_file = 'web'.$web_id.'_'.date('Y-m-d_H-i').'.tar.gz';
+							exec('tar pczf '.escapeshellarg($web_backup_dir.'/'.$web_backup_file).' --exclude=backup\* --directory '.escapeshellarg($web_path).' .', $tmp_output, $retval);
+						}
+						if($retval == 0){
+							chown($web_backup_dir.'/'.$web_backup_file, 'root');
+							chgrp($web_backup_dir.'/'.$web_backup_file, 'root');
+							chmod($web_backup_dir.'/'.$web_backup_file, 0750);
 
-                            //* Insert web backup record in database
-                            //$insert_data = "(server_id,parent_domain_id,backup_type,backup_mode,tstamp,filename) VALUES (".$conf['server_id'].",".$web_id.",'web','".$backup_mode."',".time().",'".$app->db->quote($web_backup_file)."')";
-                            //$app->dbmaster->datalogInsert('web_backup', $insert_data, 'backup_id');
-			    $sql = "INSERT INTO web_backup (server_id,parent_domain_id,backup_type,backup_mode,tstamp,filename,filesize) VALUES (".$conf['server_id'].",".$web_id.",'web','".$backup_mode."',".time().",'".$app->db->quote($web_backup_file)."','".formatBytes(filesize($web_backup_dir.'/'.$web_backup_file))."')";
-                            $app->db->query($sql);
-                            if($app->db->dbHost != $app->dbmaster->dbHost) $app->dbmaster->query($sql);
-                        } else {
-                            if(is_file($web_backup_dir.'/'.$web_backup_file)) unlink($web_backup_dir.'/'.$web_backup_file);
-                        }
+							//* Insert web backup record in database
+							//$insert_data = "(server_id,parent_domain_id,backup_type,backup_mode,tstamp,filename) VALUES (".$conf['server_id'].",".$web_id.",'web','".$backup_mode."',".time().",'".$app->db->quote($web_backup_file)."')";
+							//$app->dbmaster->datalogInsert('web_backup', $insert_data, 'backup_id');
+							$sql = "INSERT INTO web_backup (server_id,parent_domain_id,backup_type,backup_mode,tstamp,filename,filesize) VALUES (".$conf['server_id'].",".$web_id.",'web','".$backup_mode."',".time().",'".$app->db->quote($web_backup_file)."','".formatBytes(filesize($web_backup_dir.'/'.$web_backup_file))."')";
+							$app->db->query($sql);
+							if($app->db->dbHost != $app->dbmaster->dbHost) $app->dbmaster->query($sql);
+						} else {
+							if(is_file($web_backup_dir.'/'.$web_backup_file)) unlink($web_backup_dir.'/'.$web_backup_file);
+						}
 
-                        //* Remove old backups
-                        $backup_copies = intval($rec['backup_copies']);
+						//* Remove old backups
+						$backup_copies = intval($rec['backup_copies']);
 
-                        $dir_handle = dir($web_backup_dir);
-                        $files = array();
-                        while (false !== ($entry = $dir_handle->read())) {
-                            if($entry != '.' && $entry != '..' && substr($entry,0,3) == 'web' && is_file($web_backup_dir.'/'.$entry)) {
-                                $files[] = $entry;
-                            }
-                        }
-                        $dir_handle->close();
+						$dir_handle = dir($web_backup_dir);
+						$files = array();
+						while (false !== ($entry = $dir_handle->read())) {
+							if($entry != '.' && $entry != '..' && substr($entry, 0, 3) == 'web' && is_file($web_backup_dir.'/'.$entry)) {
+								$files[] = $entry;
+							}
+						}
+						$dir_handle->close();
 
-                        rsort($files);
+						rsort($files);
 
-                        for ($n = $backup_copies; $n <= 10; $n++) {
-                            if(isset($files[$n]) && is_file($web_backup_dir.'/'.$files[$n])) {
-                                unlink($web_backup_dir.'/'.$files[$n]);
-                                //$sql = "SELECT backup_id FROM web_backup WHERE server_id = ".$conf['server_id']." AND parent_domain_id = $web_id AND filename = '".$app->db->quote($files[$n])."'";
-                                //$tmp = $app->dbmaster->queryOneRecord($sql);
-                                //$app->dbmaster->datalogDelete('web_backup', 'backup_id', $tmp['backup_id']);
-                                //$sql = "DELETE FROM web_backup WHERE backup_id = ".intval($tmp['backup_id']);
-                                $sql = "DELETE FROM web_backup WHERE server_id = ".$conf['server_id']." AND parent_domain_id = $web_id AND filename = '".$app->db->quote($files[$n])."'";
-                                $app->db->query($sql);
-                                if($app->db->dbHost != $app->dbmaster->dbHost) $app->dbmaster->query($sql);
-                            }
-                        }
+						for ($n = $backup_copies; $n <= 10; $n++) {
+							if(isset($files[$n]) && is_file($web_backup_dir.'/'.$files[$n])) {
+								unlink($web_backup_dir.'/'.$files[$n]);
+								//$sql = "SELECT backup_id FROM web_backup WHERE server_id = ".$conf['server_id']." AND parent_domain_id = $web_id AND filename = '".$app->db->quote($files[$n])."'";
+								//$tmp = $app->dbmaster->queryOneRecord($sql);
+								//$app->dbmaster->datalogDelete('web_backup', 'backup_id', $tmp['backup_id']);
+								//$sql = "DELETE FROM web_backup WHERE backup_id = ".intval($tmp['backup_id']);
+								$sql = "DELETE FROM web_backup WHERE server_id = ".$conf['server_id']." AND parent_domain_id = $web_id AND filename = '".$app->db->quote($files[$n])."'";
+								$app->db->query($sql);
+								if($app->db->dbHost != $app->dbmaster->dbHost) $app->dbmaster->query($sql);
+							}
+						}
 
-                        unset($files);
-                        unset($dir_handle);
+						unset($files);
+						unset($dir_handle);
 
-                        //* Remove backupdir symlink and create as directory instead
-                        $app->uses('system');
-                        $app->system->web_folder_protection($web_path,false);
+						//* Remove backupdir symlink and create as directory instead
+						$app->uses('system');
+						$app->system->web_folder_protection($web_path, false);
 
-                        if(is_link($web_path.'/backup')) {
-                            unlink($web_path.'/backup');
-                        }
-                        if(!is_dir($web_path.'/backup')) {
-                            mkdir($web_path.'/backup');
-                            chown($web_path.'/backup', $rec['system_user']);
-                            chgrp($web_path.'/backup', $rec['system_group']);
-                        }
+						if(is_link($web_path.'/backup')) {
+							unlink($web_path.'/backup');
+						}
+						if(!is_dir($web_path.'/backup')) {
+							mkdir($web_path.'/backup');
+							chown($web_path.'/backup', $rec['system_user']);
+							chgrp($web_path.'/backup', $rec['system_group']);
+						}
 
-                        $app->system->web_folder_protection($web_path,true);
+						$app->system->web_folder_protection($web_path, true);
 
-                    }
+					}
 
-                    /* If backup_interval is set to none and we have a
+					/* If backup_interval is set to none and we have a
                     backup directory for the website, then remove the backups */
-                    if($rec['backup_interval'] == 'none') {
-                        $web_id = $rec['domain_id'];
-                        $web_user = $rec['system_user'];
-                        $web_backup_dir = realpath($backup_dir.'/web'.$web_id);
-                        if(is_dir($web_backup_dir)) {
-                            exec('sudo -u '.escapeshellarg($web_user).' rm -f '.escapeshellarg($web_backup_dir.'/*'));
-                        }
-                    }
-                }
-            }
+					if($rec['backup_interval'] == 'none') {
+						$web_id = $rec['domain_id'];
+						$web_user = $rec['system_user'];
+						$web_backup_dir = realpath($backup_dir.'/web'.$web_id);
+						if(is_dir($web_backup_dir)) {
+							exec('sudo -u '.escapeshellarg($web_user).' rm -f '.escapeshellarg($web_backup_dir.'/*'));
+						}
+					}
+				}
+			}
 
-            $sql = "SELECT * FROM web_database WHERE server_id = '".$conf['server_id']."' AND backup_interval != 'none'";
-            $records = $app->db->queryAllRecords($sql);
-            if(is_array($records)) {
+			$sql = "SELECT * FROM web_database WHERE server_id = '".$conf['server_id']."' AND backup_interval != 'none'";
+			$records = $app->db->queryAllRecords($sql);
+			if(is_array($records)) {
 
-                include('lib/mysql_clientdb.conf');
+				include 'lib/mysql_clientdb.conf';
 
-                foreach($records as $rec) {
+				foreach($records as $rec) {
 
-                    //* Do the database backup
-                    if($rec['backup_interval'] == 'daily' or ($rec['backup_interval'] == 'weekly' && date('w') == 0) or ($rec['backup_interval'] == 'monthly' && date('d') == '01')) {
+					//* Do the database backup
+					if($rec['backup_interval'] == 'daily' or ($rec['backup_interval'] == 'weekly' && date('w') == 0) or ($rec['backup_interval'] == 'monthly' && date('d') == '01')) {
 
-                        $web_id = $rec['parent_domain_id'];
-                        $db_backup_dir = $backup_dir.'/web'.$web_id;
-                        if(!is_dir($db_backup_dir)) mkdir($db_backup_dir, 0750);
-                        chmod($db_backup_dir, 0750);
-                        chown($db_backup_dir, 'root');
-                        chgrp($db_backup_dir, 'root');
+						$web_id = $rec['parent_domain_id'];
+						$db_backup_dir = $backup_dir.'/web'.$web_id;
+						if(!is_dir($db_backup_dir)) mkdir($db_backup_dir, 0750);
+						chmod($db_backup_dir, 0750);
+						chown($db_backup_dir, 'root');
+						chgrp($db_backup_dir, 'root');
 
-                        //* Do the mysql database backup with mysqldump or mongodump
-                        $db_id = $rec['database_id'];
-                        $db_name = $rec['database_name'];
+						//* Do the mysql database backup with mysqldump or mongodump
+						$db_id = $rec['database_id'];
+						$db_name = $rec['database_name'];
 
-                        if ($rec['type'] == 'mysql') {
-                            $db_backup_file = 'db_'.$db_name.'_'.date('Y-m-d_H-i').'.sql';
-                            //$command = "mysqldump -h '".escapeshellcmd($clientdb_host)."' -u '".escapeshellcmd($clientdb_user)."' -p'".escapeshellcmd($clientdb_password)."' -c --add-drop-table --create-options --quick --result-file='".$db_backup_dir.'/'.$db_backup_file."' '".$db_name."'";
-                            $command = "mysqldump -h ".escapeshellarg($clientdb_host)." -u ".escapeshellarg($clientdb_user)." -p".escapeshellarg($clientdb_password)." -c --add-drop-table --quote-names --routines --events --triggers --hex-blob --create-options --quick --result-file='".$db_backup_dir.'/'.$db_backup_file."' '".$db_name."'";
-                            exec($command, $tmp_output, $retval);
+						if ($rec['type'] == 'mysql') {
+							$db_backup_file = 'db_'.$db_name.'_'.date('Y-m-d_H-i').'.sql';
+							//$command = "mysqldump -h '".escapeshellcmd($clientdb_host)."' -u '".escapeshellcmd($clientdb_user)."' -p'".escapeshellcmd($clientdb_password)."' -c --add-drop-table --create-options --quick --result-file='".$db_backup_dir.'/'.$db_backup_file."' '".$db_name."'";
+							$command = "mysqldump -h ".escapeshellarg($clientdb_host)." -u ".escapeshellarg($clientdb_user)." -p".escapeshellarg($clientdb_password)." -c --add-drop-table --quote-names --routines --events --triggers --hex-blob --create-options --quick --result-file='".$db_backup_dir.'/'.$db_backup_file."' '".$db_name."'";
+							exec($command, $tmp_output, $retval);
 
-                            //* Compress the backup with gzip
-                            if($retval == 0) exec("gzip -c '".escapeshellcmd($db_backup_dir.'/'.$db_backup_file)."' > '".escapeshellcmd($db_backup_dir.'/'.$db_backup_file).".gz'", $tmp_output, $retval);
+							//* Compress the backup with gzip
+							if($retval == 0) exec("gzip -c '".escapeshellcmd($db_backup_dir.'/'.$db_backup_file)."' > '".escapeshellcmd($db_backup_dir.'/'.$db_backup_file).".gz'", $tmp_output, $retval);
 
-                            if($retval == 0){
-                                chmod($db_backup_dir.'/'.$db_backup_file.'.gz', 0750);
-                                chown($db_backup_dir.'/'.$db_backup_file.'.gz', fileowner($db_backup_dir));
-                                chgrp($db_backup_dir.'/'.$db_backup_file.'.gz', filegroup($db_backup_dir));
+							if($retval == 0){
+								chmod($db_backup_dir.'/'.$db_backup_file.'.gz', 0750);
+								chown($db_backup_dir.'/'.$db_backup_file.'.gz', fileowner($db_backup_dir));
+								chgrp($db_backup_dir.'/'.$db_backup_file.'.gz', filegroup($db_backup_dir));
 
-                                //* Insert web backup record in database
-                                //$insert_data = "(server_id,parent_domain_id,backup_type,backup_mode,tstamp,filename) VALUES (".$conf['server_id'].",$web_id,'mysql','sqlgz',".time().",'".$app->db->quote($db_backup_file).".gz')";
-                                //$app->dbmaster->datalogInsert('web_backup', $insert_data, 'backup_id');
-                                $sql = "INSERT INTO web_backup (server_id,parent_domain_id,backup_type,backup_mode,tstamp,filename,filesize) VALUES (".$conf['server_id'].",$web_id,'mysql','sqlgz',".time().",'".$app->db->quote($db_backup_file).".gz','".formatBytes(filesize($db_backup_dir.'/'.$db_backup_file))."')";
-                                $app->db->query($sql);
-                                if($app->db->dbHost != $app->dbmaster->dbHost) $app->dbmaster->query($sql);
+								//* Insert web backup record in database
+								//$insert_data = "(server_id,parent_domain_id,backup_type,backup_mode,tstamp,filename) VALUES (".$conf['server_id'].",$web_id,'mysql','sqlgz',".time().",'".$app->db->quote($db_backup_file).".gz')";
+								//$app->dbmaster->datalogInsert('web_backup', $insert_data, 'backup_id');
+								$sql = "INSERT INTO web_backup (server_id,parent_domain_id,backup_type,backup_mode,tstamp,filename,filesize) VALUES (".$conf['server_id'].",$web_id,'mysql','sqlgz',".time().",'".$app->db->quote($db_backup_file).".gz','".formatBytes(filesize($db_backup_dir.'/'.$db_backup_file))."')";
+								$app->db->query($sql);
+								if($app->db->dbHost != $app->dbmaster->dbHost) $app->dbmaster->query($sql);
 
-                            } else {
-                                if(is_file($db_backup_dir.'/'.$db_backup_file.'.gz')) unlink($db_backup_dir.'/'.$db_backup_file.'.gz');
-                            }
-                            //* Remove the uncompressed file
-                            if(is_file($db_backup_dir.'/'.$db_backup_file)) unlink($db_backup_dir.'/'.$db_backup_file);
-                        } else if ($rec['type'] == 'mongo') {
-                            $db_backup_file = 'db_'.$db_name.'_'.date('Y-m-d_H-i');
+							} else {
+								if(is_file($db_backup_dir.'/'.$db_backup_file.'.gz')) unlink($db_backup_dir.'/'.$db_backup_file.'.gz');
+							}
+							//* Remove the uncompressed file
+							if(is_file($db_backup_dir.'/'.$db_backup_file)) unlink($db_backup_dir.'/'.$db_backup_file);
+						} else if ($rec['type'] == 'mongo') {
+								$db_backup_file = 'db_'.$db_name.'_'.date('Y-m-d_H-i');
 
-                            try {
-                                $connection = new MongoClient("mongodb://root:123456@127.0.0.1:27017/admin");
-                                $db = $connection->selectDB($db_name);
-                                // exclude not supported by mongodump, only get user collections
-                                $collections = $db->getCollectionNames(false);
+								try {
+									$connection = new MongoClient("mongodb://root:123456@127.0.0.1:27017/admin");
+									$db = $connection->selectDB($db_name);
+									// exclude not supported by mongodump, only get user collections
+									$collections = $db->getCollectionNames(false);
 
-                                foreach ($collections as $collection) {
-                                    // mongodump -h 127.0.0.1 --port 27017 -u root -p 123456 --authenticationDatabase admin -d <db> -c <table> -o /tmp/test
-                                    $command = "mongodump -h 127.0.0.1 --port 27017 -u root -p 123456 --authenticationDatabase admin -d ".escapeshellcmd($db_name)." -c ".escapeshellcmd($collection)." -o ".escapeshellcmd($db_backup_dir.'/'.$db_backup_file);
-                                    exec($command);
-                                }
+									foreach ($collections as $collection) {
+										// mongodump -h 127.0.0.1 --port 27017 -u root -p 123456 --authenticationDatabase admin -d <db> -c <table> -o /tmp/test
+										$command = "mongodump -h 127.0.0.1 --port 27017 -u root -p 123456 --authenticationDatabase admin -d ".escapeshellcmd($db_name)." -c ".escapeshellcmd($collection)." -o ".escapeshellcmd($db_backup_dir.'/'.$db_backup_file);
+										exec($command);
+									}
 
-                                if (is_dir(escapeshellcmd($db_backup_dir.'/'.$db_backup_file))) {
-                                    //* Compress the backup with gzip
-                                    exec("cd ".escapeshellcmd($db_backup_dir)." && tar -pczf ".escapeshellcmd($db_backup_dir.'/'.$db_backup_file).".tar.gz ".escapeshellcmd($db_backup_file));
-                                    chmod($db_backup_dir.'/'.$db_backup_file.'.tar.gz', 0750);
-                                    chown($db_backup_dir.'/'.$db_backup_file.'.tar.gz', fileowner($db_backup_dir));
-                                    chgrp($db_backup_dir.'/'.$db_backup_file.'.tar.gz', filegroup($db_backup_dir));
+									if (is_dir(escapeshellcmd($db_backup_dir.'/'.$db_backup_file))) {
+										//* Compress the backup with gzip
+										exec("cd ".escapeshellcmd($db_backup_dir)." && tar -pczf ".escapeshellcmd($db_backup_dir.'/'.$db_backup_file).".tar.gz ".escapeshellcmd($db_backup_file));
+										chmod($db_backup_dir.'/'.$db_backup_file.'.tar.gz', 0750);
+										chown($db_backup_dir.'/'.$db_backup_file.'.tar.gz', fileowner($db_backup_dir));
+										chgrp($db_backup_dir.'/'.$db_backup_file.'.tar.gz', filegroup($db_backup_dir));
 
-                                    //* Insert web backup record in database
-                                    $sql = "INSERT INTO web_backup (server_id,parent_domain_id,backup_type,backup_mode,tstamp,filename) VALUES (".$conf['server_id'].",$web_id,'mongodb','rootgz',".time().",'".$app->db->quote($db_backup_file).".tar.gz')";
-                                    $app->db->query($sql);
+										//* Insert web backup record in database
+										$sql = "INSERT INTO web_backup (server_id,parent_domain_id,backup_type,backup_mode,tstamp,filename) VALUES (".$conf['server_id'].",$web_id,'mongodb','rootgz',".time().",'".$app->db->quote($db_backup_file).".tar.gz')";
+										$app->db->query($sql);
 
-                                    if ($app->db->dbHost != $app->dbmaster->dbHost) {
-                                        $app->dbmaster->query($sql);
-                                    }
+										if ($app->db->dbHost != $app->dbmaster->dbHost) {
+											$app->dbmaster->query($sql);
+										}
 
-                                    //* Remove the uncompressed file
-                                    exec("rm -rf ".escapeshellcmd($db_backup_dir.'/'.$db_backup_file));
-                                }
-                            } catch (MongoConnnectionException $e) {
-                                // connecting to MongoDB failed - cannot create backup
-                            }
-                        }
+										//* Remove the uncompressed file
+										exec("rm -rf ".escapeshellcmd($db_backup_dir.'/'.$db_backup_file));
+									}
+								} catch (MongoConnnectionException $e) {
+									// connecting to MongoDB failed - cannot create backup
+								}
+							}
 
-                        //* Remove old backups
-                        $backup_copies = intval($rec['backup_copies']);
+						//* Remove old backups
+						$backup_copies = intval($rec['backup_copies']);
 
-                        $dir_handle = dir($db_backup_dir);
-                        $files = array();
-                        while (false !== ($entry = $dir_handle->read())) {
-                            if($entry != '.' && $entry != '..' && (preg_match('/^db_(.*?)_\d{4}-\d{2}-\d{2}_\d{2}-\d{2}\.sql.gz$/', $entry, $matches) || preg_match('/^db_(.*?)_\d{4}-\d{2}-\d{2}_\d{2}-\d{2}\.tar.gz$/', $entry, $matches)) && is_file($db_backup_dir.'/'.$entry)) {
-                                if(array_key_exists($matches[1], $files) == false) $files[$matches[1]] = array();
-                                $files[$matches[1]][] = $entry;
-                            }
-                        }
-                        $dir_handle->close();
+						$dir_handle = dir($db_backup_dir);
+						$files = array();
+						while (false !== ($entry = $dir_handle->read())) {
+							if($entry != '.' && $entry != '..' && (preg_match('/^db_(.*?)_\d{4}-\d{2}-\d{2}_\d{2}-\d{2}\.sql.gz$/', $entry, $matches) || preg_match('/^db_(.*?)_\d{4}-\d{2}-\d{2}_\d{2}-\d{2}\.tar.gz$/', $entry, $matches)) && is_file($db_backup_dir.'/'.$entry)) {
+								if(array_key_exists($matches[1], $files) == false) $files[$matches[1]] = array();
+								$files[$matches[1]][] = $entry;
+							}
+						}
+						$dir_handle->close();
 
-                        reset($files);
-                        foreach($files as $db_name => $filelist) {
-                            rsort($filelist);
-                            for ($n = $backup_copies; $n <= 10; $n++) {
-                                if(isset($filelist[$n]) && is_file($db_backup_dir.'/'.$filelist[$n])) {
-                                    unlink($db_backup_dir.'/'.$filelist[$n]);
-                                    //$sql = "SELECT backup_id FROM web_backup WHERE server_id = ".$conf['server_id']." AND parent_domain_id = $web_id AND filename = '".$app->db->quote($filelist[$n])."'";
-                                    //$tmp = $app->dbmaster->queryOneRecord($sql);
-                                    //$sql = "DELETE FROM web_backup WHERE backup_id = ".intval($tmp['backup_id']);
-                                    $sql = "DELETE FROM web_backup WHERE server_id = ".$conf['server_id']." AND parent_domain_id = $web_id AND filename = '".$app->db->quote($filelist[$n])."'";
-                                    $app->db->query($sql);
-                                    if($app->db->dbHost != $app->dbmaster->dbHost) $app->dbmaster->query($sql);
-                                }
-                            }
-                        }
+						reset($files);
+						foreach($files as $db_name => $filelist) {
+							rsort($filelist);
+							for ($n = $backup_copies; $n <= 10; $n++) {
+								if(isset($filelist[$n]) && is_file($db_backup_dir.'/'.$filelist[$n])) {
+									unlink($db_backup_dir.'/'.$filelist[$n]);
+									//$sql = "SELECT backup_id FROM web_backup WHERE server_id = ".$conf['server_id']." AND parent_domain_id = $web_id AND filename = '".$app->db->quote($filelist[$n])."'";
+									//$tmp = $app->dbmaster->queryOneRecord($sql);
+									//$sql = "DELETE FROM web_backup WHERE backup_id = ".intval($tmp['backup_id']);
+									$sql = "DELETE FROM web_backup WHERE server_id = ".$conf['server_id']." AND parent_domain_id = $web_id AND filename = '".$app->db->quote($filelist[$n])."'";
+									$app->db->query($sql);
+									if($app->db->dbHost != $app->dbmaster->dbHost) $app->dbmaster->query($sql);
+								}
+							}
+						}
 
-                        unset($files);
-                        unset($dir_handle);
-                    }
-                }
+						unset($files);
+						unset($dir_handle);
+					}
+				}
 
-                unset($clientdb_host);
-                unset($clientdb_user);
-                unset($clientdb_password);
+				unset($clientdb_host);
+				unset($clientdb_user);
+				unset($clientdb_password);
 
-            }
-        }
-        
-        parent::onRunJob();
-    }
-    
-    /* this function is optional if it contains no custom code */
-    public function onAfterRun() {
-        global $app;
-        
-        parent::onAfterRun();
-    }
+			}
+		}
+
+		parent::onRunJob();
+	}
+
+	/* this function is optional if it contains no custom code */
+	public function onAfterRun() {
+		global $app;
+
+		parent::onAfterRun();
+	}
 
 }
 

--
Gitblit v1.9.1