From 7fe908c50c8dbc5cc05f571dbe11d66141caacd4 Mon Sep 17 00:00:00 2001 From: Marius Cramer <m.cramer@pixcept.de> Date: Thu, 14 Nov 2013 09:01:22 -0500 Subject: [PATCH] Cleaning up code to match coding guidelines --- server/plugins-available/ftpuser_base_plugin.inc.php | 146 ++++++++++++++++++++++++------------------------ 1 files changed, 73 insertions(+), 73 deletions(-) diff --git a/server/plugins-available/ftpuser_base_plugin.inc.php b/server/plugins-available/ftpuser_base_plugin.inc.php index 66723d2..af33e93 100644 --- a/server/plugins-available/ftpuser_base_plugin.inc.php +++ b/server/plugins-available/ftpuser_base_plugin.inc.php @@ -29,104 +29,104 @@ */ class ftpuser_base_plugin { - + var $plugin_name = 'ftpuser_base_plugin'; var $class_name = 'ftpuser_base_plugin'; - + //* This function is called during ispconfig installation to determine // if a symlink shall be created for this plugin. function onInstall() { global $conf; - + if($conf['services']['web'] == true) { return true; } else { return false; } - + } - - + + /* This function is called when the plugin is loaded */ - + function onLoad() { global $app; - + /* Register for the events */ - - $app->plugins->registerEvent('ftp_user_insert',$this->plugin_name,'insert'); - $app->plugins->registerEvent('ftp_user_update',$this->plugin_name,'update'); - $app->plugins->registerEvent('ftp_user_delete',$this->plugin_name,'delete'); - - } - - - function insert($event_name,$data) { - global $app, $conf; - - $app->uses('system'); - - if(!is_dir($data['new']['dir'])) { - $app->log("FTP User directory '".$data['new']['dir']."' does not exist. Creating it now.",LOGLEVEL_DEBUG); - - $web = $app->db->queryOneRecord("SELECT * FROM web_domain WHERE domain_id = ".intval($data['new']['parent_domain_id'])); - - //* Check if the resulting path is inside the docroot - if(substr($data['new']['dir'],0,strlen($web['document_root'])) != $web['document_root']) { - $app->log('User dir is outside of docroot.',LOGLEVEL_WARN); - return false; - } - - $app->system->web_folder_protection($web['document_root'],false); - exec('mkdir -p '.escapeshellcmd($data['new']['dir'])); - exec('chown '.escapeshellcmd($web["system_user"]).':'.escapeshellcmd($web['system_group']).' '.$data['new']['dir']); - $app->system->web_folder_protection($web['document_root'],true); - - $app->log("Added ftpuser_dir: ".$data['new']['dir'],LOGLEVEL_DEBUG); - } - - } - - function update($event_name,$data) { - global $app, $conf; - - $app->uses('system'); - - if(!is_dir($data['new']['dir'])) { - $app->log("FTP User directory '".$data['new']['dir']."' does not exist. Creating it now.",LOGLEVEL_DEBUG); - - $web = $app->db->queryOneRecord("SELECT * FROM web_domain WHERE domain_id = ".intval($data['new']['parent_domain_id'])); - - //* Check if the resulting path is inside the docroot - if(substr($data['new']['dir'],0,strlen($web['document_root'])) != $web['document_root']) { - $app->log('User dir is outside of docroot.',LOGLEVEL_WARN); - return false; - } + $app->plugins->registerEvent('ftp_user_insert', $this->plugin_name, 'insert'); + $app->plugins->registerEvent('ftp_user_update', $this->plugin_name, 'update'); + $app->plugins->registerEvent('ftp_user_delete', $this->plugin_name, 'delete'); - $app->system->web_folder_protection($web['document_root'],false); - exec('mkdir -p '.escapeshellcmd($data['new']['dir'])); - exec('chown '.escapeshellcmd($web["system_user"]).':'.escapeshellcmd($web['system_group']).' '.$data['new']['dir']); - $app->system->web_folder_protection($web['document_root'],true); - - $app->log("Added ftpuser_dir: ".$data['new']['dir'],LOGLEVEL_DEBUG); - } + } - - function delete($event_name,$data) { + + + function insert($event_name, $data) { global $app, $conf; - - $app->log("Ftpuser:".$data['new']['username']." deleted.",LOGLEVEL_DEBUG); - + + $app->uses('system'); + + if(!is_dir($data['new']['dir'])) { + $app->log("FTP User directory '".$data['new']['dir']."' does not exist. Creating it now.", LOGLEVEL_DEBUG); + + $web = $app->db->queryOneRecord("SELECT * FROM web_domain WHERE domain_id = ".intval($data['new']['parent_domain_id'])); + + //* Check if the resulting path is inside the docroot + if(substr($data['new']['dir'], 0, strlen($web['document_root'])) != $web['document_root']) { + $app->log('User dir is outside of docroot.', LOGLEVEL_WARN); + return false; + } + + $app->system->web_folder_protection($web['document_root'], false); + exec('mkdir -p '.escapeshellcmd($data['new']['dir'])); + exec('chown '.escapeshellcmd($web["system_user"]).':'.escapeshellcmd($web['system_group']).' '.$data['new']['dir']); + $app->system->web_folder_protection($web['document_root'], true); + + $app->log("Added ftpuser_dir: ".$data['new']['dir'], LOGLEVEL_DEBUG); + } + } - - - + + function update($event_name, $data) { + global $app, $conf; + + $app->uses('system'); + + if(!is_dir($data['new']['dir'])) { + $app->log("FTP User directory '".$data['new']['dir']."' does not exist. Creating it now.", LOGLEVEL_DEBUG); + + $web = $app->db->queryOneRecord("SELECT * FROM web_domain WHERE domain_id = ".intval($data['new']['parent_domain_id'])); + + //* Check if the resulting path is inside the docroot + if(substr($data['new']['dir'], 0, strlen($web['document_root'])) != $web['document_root']) { + $app->log('User dir is outside of docroot.', LOGLEVEL_WARN); + return false; + } + + $app->system->web_folder_protection($web['document_root'], false); + exec('mkdir -p '.escapeshellcmd($data['new']['dir'])); + exec('chown '.escapeshellcmd($web["system_user"]).':'.escapeshellcmd($web['system_group']).' '.$data['new']['dir']); + $app->system->web_folder_protection($web['document_root'], true); + + $app->log("Added ftpuser_dir: ".$data['new']['dir'], LOGLEVEL_DEBUG); + } + } + + function delete($event_name, $data) { + global $app, $conf; + + $app->log("Ftpuser:".$data['new']['username']." deleted.", LOGLEVEL_DEBUG); + + } + + + } // end class -?> \ No newline at end of file +?> -- Gitblit v1.9.1