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/lib/classes/plugins.inc.php |   92 +++++++++++++++++++++++-----------------------
 1 files changed, 46 insertions(+), 46 deletions(-)

diff --git a/server/lib/classes/plugins.inc.php b/server/lib/classes/plugins.inc.php
index d13317e..4808a56 100644
--- a/server/lib/classes/plugins.inc.php
+++ b/server/lib/classes/plugins.inc.php
@@ -29,96 +29,96 @@
 */
 
 class plugins {
-	
+
 	var $available_events = array();
 	var $subscribed_events = array();
 	var $subscribed_actions = array();
 	var $debug = false;
-	
+
 	/*
 	 This function is called to load the plugins from the plugins-enabled or the plugins-core folder
 	*/
-	
+
 	function loadPlugins($type) {
-		global $app,$conf;
+		global $app, $conf;
 
 		$subPath = 'plugins-enabled';
 		//if ($type == 'core') $subPath = 'plugins-core';
-		
+
 		$plugins_dir = $conf['rootpath'].$conf['fs_div'].$subPath.$conf['fs_div'];
 		$tmp_plugins = array();
-		
+
 		if (is_dir($plugins_dir)) {
 			if ($dh = opendir($plugins_dir)) {
 				//** Go trough all files in the plugin dir
 				while (($file = readdir($dh)) !== false) {
-					if($file != '.' && $file != '..' && substr($file,-8,8) == '.inc.php') {
-						$plugin_name = substr($file,0,-8);
+					if($file != '.' && $file != '..' && substr($file, -8, 8) == '.inc.php') {
+						$plugin_name = substr($file, 0, -8);
 						$tmp_plugins[$plugin_name] = $file;
 					}
 				}
 				//** sort the plugins by name
 				ksort($tmp_plugins);
-				
+
 				//** load the plugins
 				foreach($tmp_plugins as $plugin_name => $file) {
-					include_once($plugins_dir.$file);
-					if($this->debug) $app->log('Loading plugin: '.$plugin_name,LOGLEVEL_DEBUG);
+					include_once $plugins_dir.$file;
+					if($this->debug) $app->log('Loading plugin: '.$plugin_name, LOGLEVEL_DEBUG);
 					$app->loaded_plugins[$plugin_name] = new $plugin_name;
 					$app->loaded_plugins[$plugin_name]->onLoad();
 				}
 			} else {
-				$app->log('Unable to open the plugins directory: '.$plugins_dir,LOGLEVEL_ERROR);
+				$app->log('Unable to open the plugins directory: '.$plugins_dir, LOGLEVEL_ERROR);
 			}
 		} else {
-			$app->log('Plugins directory missing: '.$plugins_dir,LOGLEVEL_ERROR);
+			$app->log('Plugins directory missing: '.$plugins_dir, LOGLEVEL_ERROR);
 		}
-		
+
 	}
-	
+
 	/*
 		This function is used by the modules to announce which events they provide
 	*/
-	
-	function announceEvents($module_name,$events) {
+
+	function announceEvents($module_name, $events) {
 		global $app;
 		foreach($events as $event_name) {
 			$this->available_events[$event_name] = $module_name;
-			if($this->debug) $app->log('Announced event: '.$event_name,LOGLEVEL_DEBUG);
+			if($this->debug) $app->log('Announced event: '.$event_name, LOGLEVEL_DEBUG);
 		}
 	}
-	
-	
+
+
 	/*
 	 This function is called by the plugin to register for an event
 	*/
-	
-	function registerEvent($event_name,$plugin_name,$function_name) {
+
+	function registerEvent($event_name, $plugin_name, $function_name) {
 		global $app;
 		if(!isset($this->available_events[$event_name])) {
-			$app->log("Unable to register function '$function_name' from plugin '$plugin_name' for event '$event_name'",LOGLEVEL_DEBUG);
+			$app->log("Unable to register function '$function_name' from plugin '$plugin_name' for event '$event_name'", LOGLEVEL_DEBUG);
 		} else {
 			$this->subscribed_events[$event_name][] = array('plugin' => $plugin_name, 'function' => $function_name);
-			if($this->debug)  $app->log("Registered function '$function_name' from plugin '$plugin_name' for event '$event_name'.",LOGLEVEL_DEBUG);
+			if($this->debug)  $app->log("Registered function '$function_name' from plugin '$plugin_name' for event '$event_name'.", LOGLEVEL_DEBUG);
 		}
 	}
-	
-	
-	function raiseEvent($event_name,$data) {
+
+
+	function raiseEvent($event_name, $data) {
 		global $app;
-		
+
 		// Get the subscriptions for this event
 		$events = (isset($this->subscribed_events[$event_name]))?$this->subscribed_events[$event_name]:'';
-		if($this->debug) $app->log('Raised event: '.$event_name,LOGLEVEL_DEBUG);
-		
+		if($this->debug) $app->log('Raised event: '.$event_name, LOGLEVEL_DEBUG);
+
 		if(is_array($events)) {
 			foreach($events as $event) {
 				$plugin_name = $event['plugin'];
 				$function_name = $event['function'];
 				// Call the processing function of the plugin
-				$app->log("Calling function '$function_name' from plugin '$plugin_name' raised by event '$event_name'.",LOGLEVEL_DEBUG);
+				$app->log("Calling function '$function_name' from plugin '$plugin_name' raised by event '$event_name'.", LOGLEVEL_DEBUG);
 				// call_user_method($function_name,$app->loaded_plugins[$plugin_name],$event_name,$data);
-				call_user_func(array($app->loaded_plugins[$plugin_name],$function_name),$event_name,$data);
+				call_user_func(array($app->loaded_plugins[$plugin_name], $function_name), $event_name, $data);
 				unset($plugin_name);
 				unset($function_name);
 			}
@@ -126,33 +126,33 @@
 		unset($event);
 		unset($events);
 	}
-	
+
 	/*
 	 This function is called by the plugin to register for an action
 	*/
-	
-	function registerAction($action_name,$plugin_name,$function_name) {
+
+	function registerAction($action_name, $plugin_name, $function_name) {
 		global $app;
 		$this->subscribed_actions[$action_name][] = array('plugin' => $plugin_name, 'function' => $function_name);
-		if($this->debug)  $app->log("Registered function '$function_name' from plugin '$plugin_name' for action '$event_name'.",LOGLEVEL_DEBUG);
+		if($this->debug)  $app->log("Registered function '$function_name' from plugin '$plugin_name' for action '$event_name'.", LOGLEVEL_DEBUG);
 	}
-	
-	
-	function raiseAction($action_name,$data) {
+
+
+	function raiseAction($action_name, $data) {
 		global $app;
-		
+
 		//* Get the subscriptions for this action
 		$actions = (isset($this->subscribed_actions[$action_name]))?$this->subscribed_actions[$action_name]:'';
-		if($this->debug) $app->log('Raised action: '.$action_name,LOGLEVEL_DEBUG);
-		
+		if($this->debug) $app->log('Raised action: '.$action_name, LOGLEVEL_DEBUG);
+
 		if(is_array($actions)) {
 			foreach($actions as $action) {
 				$plugin_name = $action['plugin'];
 				$function_name = $action['function'];
 				$state_out = 'ok';
 				//* Call the processing function of the plugin
-				$app->log("Calling function '$function_name' from plugin '$plugin_name' raised by action '$action_name'.",LOGLEVEL_DEBUG);
-				$state = call_user_func(array($app->loaded_plugins[$plugin_name],$function_name),$action_name,$data);
+				$app->log("Calling function '$function_name' from plugin '$plugin_name' raised by action '$action_name'.", LOGLEVEL_DEBUG);
+				$state = call_user_func(array($app->loaded_plugins[$plugin_name], $function_name), $action_name, $data);
 				//* ensure that we return the highest warning / error level if a error occured in one of the functions
 				if($state == 'warning' && $state_out != 'error') $state_out = 'warning';
 				if($state == 'error') $state_out = 'error';
@@ -162,10 +162,10 @@
 		}
 		unset($action);
 		unset($actions);
-		
+
 		return $state_out;
 	}
-	
+
 }
 
 ?>

--
Gitblit v1.9.1