chamilo / chamilo-lms

plugin/pens/pens.php

Spacing +6 added lines, -6 removed lines

@@ -24,19 +24,19 @@
  * @licence http://www.gnu.org/licenses/gpl.txt
  */
 
-require_once __DIR__ . '/../../main/inc/global.inc.php';
-require_once __DIR__ . '/lib/pens.php';
-require_once __DIR__ . '/chamilo_pens.php';
+require_once __DIR__.'/../../main/inc/global.inc.php';
+require_once __DIR__.'/lib/pens.php';
+require_once __DIR__.'/chamilo_pens.php';
 
 class ChamiloPackageHandler extends PENSPackageHandler {
 	public function processPackage($request, $path_to_package) {
 		$server = PENSServer::singleton();
 		// Moves the package to archive/pens
-		$path_to_archives = api_get_path(SYS_ARCHIVE_PATH) . 'pens';
-		if(!is_dir($path_to_archives)) {
+		$path_to_archives = api_get_path(SYS_ARCHIVE_PATH).'pens';
+		if (!is_dir($path_to_archives)) {
 			mkdir($path_to_archives, 0777, true);
 		}
-		rename($path_to_package, $path_to_archives . '/' . $request->getFilename());
+		rename($path_to_package, $path_to_archives.'/'.$request->getFilename());
 		// Insert the request in the database
 		$chamilo_pens = new ChamiloPens($request);
 		$chamilo_pens->save();

Indentation +13 added lines, -13 removed lines

@@ -29,19 +29,19 @@
 require_once __DIR__ . '/chamilo_pens.php';
 
 class ChamiloPackageHandler extends PENSPackageHandler {
-	public function processPackage($request, $path_to_package) {
-		$server = PENSServer::singleton();
-		// Moves the package to archive/pens
-		$path_to_archives = api_get_path(SYS_ARCHIVE_PATH) . 'pens';
-		if(!is_dir($path_to_archives)) {
-			mkdir($path_to_archives, 0777, true);
-		}
-		rename($path_to_package, $path_to_archives . '/' . $request->getFilename());
-		// Insert the request in the database
-		$chamilo_pens = new ChamiloPens($request);
-		$chamilo_pens->save();
-		$server->sendAlert($request, new PENSResponse(0, 'Package successfully processed'));
-	}
+    public function processPackage($request, $path_to_package) {
+        $server = PENSServer::singleton();
+        // Moves the package to archive/pens
+        $path_to_archives = api_get_path(SYS_ARCHIVE_PATH) . 'pens';
+        if(!is_dir($path_to_archives)) {
+            mkdir($path_to_archives, 0777, true);
+        }
+        rename($path_to_package, $path_to_archives . '/' . $request->getFilename());
+        // Insert the request in the database
+        $chamilo_pens = new ChamiloPens($request);
+        $chamilo_pens->save();
+        $server->sendAlert($request, new PENSResponse(0, 'Package successfully processed'));
+    }
 }
 
 $handler = new ChamiloPackageHandler();

plugin/pens/lib/pens/pens_request_alert.php

Indentation +4 added lines, -4 removed lines

@@ -40,9 +40,9 @@
 
 class PENSRequestAlert extends PENSRequest {
 	
-	public function __construct($arguments) {
-		parent::__construct($arguments);
-		$this->setCommand("alert");
-	}
+    public function __construct($arguments) {
+        parent::__construct($arguments);
+        $this->setCommand("alert");
+    }
 
 }

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
  * @licence http://www.gnu.org/licenses/gpl.txt
  */
 
-require_once __DIR__ . '/pens_request.php';
+require_once __DIR__.'/pens_request.php';
 
 /**
  * PENSRequestReceipt

plugin/pens/lib/pens/pens_controller.php

Indentation +27 added lines, -27 removed lines

@@ -16,15 +16,15 @@ 
  * along with php-pens.  If not, see <http://www.gnu.org/licenses/>.
  */
  
- /**
- * PENSController
- * 
- * Provides the PENSController class
- * 
- * @package PENS
- * @author Guillaume Viguier-Just <guillaume@viguierjust.com>
- * @licence http://www.gnu.org/licenses/gpl.txt
- */
+    /**
+     * PENSController
+     * 
+     * Provides the PENSController class
+     * 
+     * @package PENS
+     * @author Guillaume Viguier-Just <guillaume@viguierjust.com>
+     * @licence http://www.gnu.org/licenses/gpl.txt
+     */
 
 require_once(dirname(__FILE__)."/pens_request_factory.php");
 
@@ -39,24 +39,24 @@ 
  */
 abstract class PENSController {
 	
-	/**
-	 * Parses a request based on the values present in $_REQUEST
-	 * 
-	 * @return PENSRequest Request generated
-	 * @throws PENSException if the request could not be parsed
-	 */
-	protected function parseRequest() {
-		$request = PENSRequestFactory::createPENSRequest($_REQUEST);
-		return $request;
-	}
+    /**
+     * Parses a request based on the values present in $_REQUEST
+     * 
+     * @return PENSRequest Request generated
+     * @throws PENSException if the request could not be parsed
+     */
+    protected function parseRequest() {
+        $request = PENSRequestFactory::createPENSRequest($_REQUEST);
+        return $request;
+    }
 	
-	/**
-	 * Sends the HTTP Response to the client
-	 * 
-	 * @param PENSResponse Response to be sent
-	 */
-	protected function sendResponse($response) {
-		$response->send();
-	}
+    /**
+     * Sends the HTTP Response to the client
+     * 
+     * @param PENSResponse Response to be sent
+     */
+    protected function sendResponse($response) {
+        $response->send();
+    }
 	
 }

plugin/pens/lib/pens/pens_client.php

Indentation +73 added lines, -73 removed lines

@@ -16,15 +16,15 @@ 
  * along with php-pens.  If not, see <http://www.gnu.org/licenses/>.
  */
  
- /**
- * PENSClient
- * 
- * Provides the PENSClient class
- * 
- * @package PENS
- * @author Guillaume Viguier-Just <guillaume@viguierjust.com>
- * @licence http://www.gnu.org/licenses/gpl.txt
- */
+    /**
+     * PENSClient
+     * 
+     * Provides the PENSClient class
+     * 
+     * @package PENS
+     * @author Guillaume Viguier-Just <guillaume@viguierjust.com>
+     * @licence http://www.gnu.org/licenses/gpl.txt
+     */
  
 require_once(dirname(__FILE__)."/pens_controller.php");
 require_once(dirname(__FILE__)."/pens_request_handler.php");
@@ -39,75 +39,75 @@ 
  * @licence http://www.gnu.org/licenses/gpl.txt
  */
 class PENSClient extends PENSController {
-	/**
-	 * Instance of the PENSClient
-	 * @var PENSServer
-	 */
-	private static $_instance;
+    /**
+     * Instance of the PENSClient
+     * @var PENSServer
+     */
+    private static $_instance;
 	
-	/**
-	 * Request handler
-	 * @var PENSRequestHandler
-	 */
-	protected $_request_handler = null;
+    /**
+     * Request handler
+     * @var PENSRequestHandler
+     */
+    protected $_request_handler = null;
 	
-	/**
-	 * Private constructor
-	 */
-	private function __construct() {
-	}
+    /**
+     * Private constructor
+     */
+    private function __construct() {
+    }
 	
-	/**
-	 * Singleton method
-	 */
-	public static function singleton() {
-		if(!isset(self::$_instance)) {
-			$c = __CLASS__;
-			self::$_instance = new $c;
-		}
-		return self::$_instance;
-	}
+    /**
+     * Singleton method
+     */
+    public static function singleton() {
+        if(!isset(self::$_instance)) {
+            $c = __CLASS__;
+            self::$_instance = new $c;
+        }
+        return self::$_instance;
+    }
 	
-	/**
-	 * Prevent users to clone the instance
-	 */
-	public function __clone() {
-		trigger_error('Clone is not allowed', E_USER_ERROR);
-	}
+    /**
+     * Prevent users to clone the instance
+     */
+    public function __clone() {
+        trigger_error('Clone is not allowed', E_USER_ERROR);
+    }
 	
-	public function getRequestHandler() {
-		return $this->_request_handler;
-	}
+    public function getRequestHandler() {
+        return $this->_request_handler;
+    }
 	
-	/**
-	 * Sets the request handler. Does nothing if the argument is not an instance of PENSRequestHandler
-	 * 
-	 * @param PENSRequestHandler Request handler
-	 */
-	public function setRequestHandler($request_handler) {
-		if($request_handler instanceof PENSRequestHandler) {
-			$this->_request_handler = $request_handler;
-		}
-	}
+    /**
+     * Sets the request handler. Does nothing if the argument is not an instance of PENSRequestHandler
+     * 
+     * @param PENSRequestHandler Request handler
+     */
+    public function setRequestHandler($request_handler) {
+        if($request_handler instanceof PENSRequestHandler) {
+            $this->_request_handler = $request_handler;
+        }
+    }
 	
-	/**
-	 * Receives a request, calls the handler and displays the response
-	 */
-	public function receiveRequest() {
-		$request = null;
-		try {
-			$request = $this->parseRequest();
-			$command = $request->getCommand();
-			if($command == "alert" || $command == "receipt") {
-				if(!is_null($this->_request_handler)) {
-					// Call the handler
-					$this->_request_handler->processRequest($request, new PENSResponse($_REQUEST));
-				}
-				$this->sendResponse(new PENSResponse(0, $command." received and understood"));
-			}
-		} catch(PENSException $e) {
-			// If we could not parse the request, send the error
-			$this->sendResponse(new PENSResponse($e));
-		}
-	}
+    /**
+     * Receives a request, calls the handler and displays the response
+     */
+    public function receiveRequest() {
+        $request = null;
+        try {
+            $request = $this->parseRequest();
+            $command = $request->getCommand();
+            if($command == "alert" || $command == "receipt") {
+                if(!is_null($this->_request_handler)) {
+                    // Call the handler
+                    $this->_request_handler->processRequest($request, new PENSResponse($_REQUEST));
+                }
+                $this->sendResponse(new PENSResponse(0, $command." received and understood"));
+            }
+        } catch(PENSException $e) {
+            // If we could not parse the request, send the error
+            $this->sendResponse(new PENSResponse($e));
+        }
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -61,7 +61,7 @@ 
 	 * Singleton method
 	 */
 	public static function singleton() {
-		if(!isset(self::$_instance)) {
+		if (!isset(self::$_instance)) {
 			$c = __CLASS__;
 			self::$_instance = new $c;
 		}
@@ -85,7 +85,7 @@ 
 	 * @param PENSRequestHandler Request handler
 	 */
 	public function setRequestHandler($request_handler) {
-		if($request_handler instanceof PENSRequestHandler) {
+		if ($request_handler instanceof PENSRequestHandler) {
 			$this->_request_handler = $request_handler;
 		}
 	}
@@ -98,14 +98,14 @@ 
 		try {
 			$request = $this->parseRequest();
 			$command = $request->getCommand();
-			if($command == "alert" || $command == "receipt") {
-				if(!is_null($this->_request_handler)) {
+			if ($command == "alert" || $command == "receipt") {
+				if (!is_null($this->_request_handler)) {
 					// Call the handler
 					$this->_request_handler->processRequest($request, new PENSResponse($_REQUEST));
 				}
 				$this->sendResponse(new PENSResponse(0, $command." received and understood"));
 			}
-		} catch(PENSException $e) {
+		} catch (PENSException $e) {
 			// If we could not parse the request, send the error
 			$this->sendResponse(new PENSResponse($e));
 		}

plugin/pens/lib/pens/pens_request_collect.php

Indentation +4 added lines, -4 removed lines

@@ -40,9 +40,9 @@
 
 class PENSRequestCollect extends PENSRequest {
 	
-	public function __construct($arguments) {
-		parent::__construct($arguments);
-		$this->setCommand("collect");
-	}
+    public function __construct($arguments) {
+        parent::__construct($arguments);
+        $this->setCommand("collect");
+    }
 
 }

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
  * @licence http://www.gnu.org/licenses/gpl.txt
  */
 
-require_once __DIR__ . '/pens_request.php';
+require_once __DIR__.'/pens_request.php';
 
 /**
  * PENSRequestReceipt

plugin/pens/lib/pens/pens_package_handler.php

Indentation +73 added lines, -73 removed lines

@@ -16,15 +16,15 @@ 
  * along with php-pens.  If not, see <http://www.gnu.org/licenses/>.
  */
  
- /**
- * PENSPackageHandler
- * 
- * Provides the PENSPackageHandler abstract class
- * 
- * @package PENS
- * @author Guillaume Viguier-Just <guillaume@viguierjust.com>
- * @licence http://www.gnu.org/licenses/gpl.txt
- */
+    /**
+     * PENSPackageHandler
+     * 
+     * Provides the PENSPackageHandler abstract class
+     * 
+     * @package PENS
+     * @author Guillaume Viguier-Just <guillaume@viguierjust.com>
+     * @licence http://www.gnu.org/licenses/gpl.txt
+     */
 
 require_once(dirname(__FILE__)."/pens_config.php");
 
@@ -39,76 +39,76 @@ 
  * @licence http://www.gnu.org/licenses/gpl.txt
  */
 abstract class PENSPackageHandler {
-	/**
-	 * Array of supported package types for this handler. Set by default to PENSConfig::$allowed_package_types
-	 * @var array
-	 */
-	protected $_supported_package_types = null;
+    /**
+     * Array of supported package types for this handler. Set by default to PENSConfig::$allowed_package_types
+     * @var array
+     */
+    protected $_supported_package_types = null;
 	
-	/**
-	 * Array of supported package formats for this handler. Set by default to PENSConfig::$allowed_package_formats
-	 * @var array
-	 */
-	protected $_supported_package_formats = null;
+    /**
+     * Array of supported package formats for this handler. Set by default to PENSConfig::$allowed_package_formats
+     * @var array
+     */
+    protected $_supported_package_formats = null;
 	
-	/**
-	 * Constructor. Sets the default values of supported_package_types and supported_package_formats
-	 */
-	public function __construct() {
-		$this->_supported_package_types = PENSConfig::$allowed_package_types;
-		$this->_supported_package_formats = PENSConfig::$allowed_package_formats;
-	}
+    /**
+     * Constructor. Sets the default values of supported_package_types and supported_package_formats
+     */
+    public function __construct() {
+        $this->_supported_package_types = PENSConfig::$allowed_package_types;
+        $this->_supported_package_formats = PENSConfig::$allowed_package_formats;
+    }
 	
-	public function getSupportedPackageTypes() {
-		return $this->_supported_package_types;
-	}
+    public function getSupportedPackageTypes() {
+        return $this->_supported_package_types;
+    }
 	
-	/**
-	 * Sets the supported package types. All package types in the array provided as an argument
-	 * not present in PENSConfig::$allowed_package_types will be ignored. The supported_package_types
-	 * will be set to null if the argument given is not an array
-	 * 
-	 * @param array Supported package types
-	 */
-	public function setSupportedPackageTypes($supported_package_types) {
-		$this->_supported_package_types = null;
-		if(is_array($supported_package_types)) {
-			foreach($supported_package_types as $package_type) {
-				if(in_array($package_type, PENSConfig::$allowed_package_types)) {
-					$this->_supported_package_types[] = $package_type;
-				}
-			}
-		}
-	}
+    /**
+     * Sets the supported package types. All package types in the array provided as an argument
+     * not present in PENSConfig::$allowed_package_types will be ignored. The supported_package_types
+     * will be set to null if the argument given is not an array
+     * 
+     * @param array Supported package types
+     */
+    public function setSupportedPackageTypes($supported_package_types) {
+        $this->_supported_package_types = null;
+        if(is_array($supported_package_types)) {
+            foreach($supported_package_types as $package_type) {
+                if(in_array($package_type, PENSConfig::$allowed_package_types)) {
+                    $this->_supported_package_types[] = $package_type;
+                }
+            }
+        }
+    }
 	
-	public function getSupportedPackageFormats() {
-		return $this->_supported_package_formats;
-	}
+    public function getSupportedPackageFormats() {
+        return $this->_supported_package_formats;
+    }
 	
-	/**
-	 * Sets the supported package formats. All package types in the array provided as an argument
-	 * not present in PENSConfig::$allowed_package_formats will be ignored. The supported_package_formats
-	 * will be set to null if the argument given is not an array.
-	 * 
-	 * @param array Supported package formats
-	 */
-	public function setSupportedPackageFormats($supported_package_formats) {
-		$this->_supported_package_formats = null;
-		if(is_array($supported_package_formats)) {
-			foreach($supported_package_formats as $package_format) {
-				if(in_array($package_format, PENSConfig::$allowed_package_formats)) {
-					$this->_supported_package_formats[] = $package_format;
-				}
-			}
-		}
-	}
+    /**
+     * Sets the supported package formats. All package types in the array provided as an argument
+     * not present in PENSConfig::$allowed_package_formats will be ignored. The supported_package_formats
+     * will be set to null if the argument given is not an array.
+     * 
+     * @param array Supported package formats
+     */
+    public function setSupportedPackageFormats($supported_package_formats) {
+        $this->_supported_package_formats = null;
+        if(is_array($supported_package_formats)) {
+            foreach($supported_package_formats as $package_format) {
+                if(in_array($package_format, PENSConfig::$allowed_package_formats)) {
+                    $this->_supported_package_formats[] = $package_format;
+                }
+            }
+        }
+    }
 	
-	/**
-	 * Processes the package. Must be implemented by subclasses
-	 * 
-	 * @param PENSRequestCollect Collect request sent by the client
-	 * @param string Path to the collected package on the hard drive
-	 */
-	abstract public function processPackage($request, $path_to_package);
+    /**
+     * Processes the package. Must be implemented by subclasses
+     * 
+     * @param PENSRequestCollect Collect request sent by the client
+     * @param string Path to the collected package on the hard drive
+     */
+    abstract public function processPackage($request, $path_to_package);
 	
 }

Spacing +6 added lines, -6 removed lines

@@ -72,9 +72,9 @@ 
 	 */
 	public function setSupportedPackageTypes($supported_package_types) {
 		$this->_supported_package_types = null;
-		if(is_array($supported_package_types)) {
-			foreach($supported_package_types as $package_type) {
-				if(in_array($package_type, PENSConfig::$allowed_package_types)) {
+		if (is_array($supported_package_types)) {
+			foreach ($supported_package_types as $package_type) {
+				if (in_array($package_type, PENSConfig::$allowed_package_types)) {
 					$this->_supported_package_types[] = $package_type;
 				}
 			}
@@ -94,9 +94,9 @@ 
 	 */
 	public function setSupportedPackageFormats($supported_package_formats) {
 		$this->_supported_package_formats = null;
-		if(is_array($supported_package_formats)) {
-			foreach($supported_package_formats as $package_format) {
-				if(in_array($package_format, PENSConfig::$allowed_package_formats)) {
+		if (is_array($supported_package_formats)) {
+			foreach ($supported_package_formats as $package_format) {
+				if (in_array($package_format, PENSConfig::$allowed_package_formats)) {
 					$this->_supported_package_formats[] = $package_format;
 				}
 			}

plugin/pens/lib/pens/pens_request_handler.php

Indentation +16 added lines, -16 removed lines

@@ -16,15 +16,15 @@ 
  * along with php-pens.  If not, see <http://www.gnu.org/licenses/>.
  */
  
- /**
- * PENSRequestHandler
- * 
- * Provides the PENSRequestHandler abstract class
- * 
- * @package PENS
- * @author Guillaume Viguier-Just <guillaume@viguierjust.com>
- * @licence http://www.gnu.org/licenses/gpl.txt
- */
+    /**
+     * PENSRequestHandler
+     * 
+     * Provides the PENSRequestHandler abstract class
+     * 
+     * @package PENS
+     * @author Guillaume Viguier-Just <guillaume@viguierjust.com>
+     * @licence http://www.gnu.org/licenses/gpl.txt
+     */
 
 /**
  * PENSRequestHandler
@@ -37,12 +37,12 @@ 
  * @licence http://www.gnu.org/licenses/gpl.txt
  */
 abstract class PENSRequestHandler {
-	/**
-	 * Processes the request. Must be implemented by subclasses
-	 * 
-	 * @param PENSRequest Alert or Receipt request sent by the server
-	 * @param PENSResponse Response sent by the server (error, error-text etc...)
-	 */
-	abstract public function processRequest($request, $response);
+    /**
+     * Processes the request. Must be implemented by subclasses
+     * 
+     * @param PENSRequest Alert or Receipt request sent by the server
+     * @param PENSResponse Response sent by the server (error, error-text etc...)
+     */
+    abstract public function processRequest($request, $response);
 }
 

plugin/pens/lib/pens/pens_request_receipt.php

Indentation +4 added lines, -4 removed lines

@@ -40,9 +40,9 @@
 
 class PENSRequestReceipt extends PENSRequest {
 	
-	public function __construct($arguments) {
-		parent::__construct($arguments);
-		$this->setCommand("receipt");
-	}
+    public function __construct($arguments) {
+        parent::__construct($arguments);
+        $this->setCommand("receipt");
+    }
 
 }

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
  * @licence http://www.gnu.org/licenses/gpl.txt
  */
 
-require_once __DIR__ . '/pens_request.php';
+require_once __DIR__.'/pens_request.php';
 
 /**
  * PENSRequestReceipt

plugin/pens/lib/pens/pens_request_factory.php

Indentation +22 added lines, -22 removed lines

@@ -43,26 +43,26 @@
  */
 class PENSRequestFactory {
 
-	/**
-	 * Factory method
-	 * 
-	 * Creates the right PENSRequest object, based on the command inside the arguments
-	 * 
-	 * @param array Associative array of arguments
-	 * @return mixed PENSRequestAlert or PENSRequestCollect or PENSRequestReceipt
-	 * 
-	 * @throws PENSException with code 2002 if command is invalid
-	 */
-	public static function createPENSRequest($arguments) {
-		$command = $arguments["command"];
-		if($command == "alert") {
-			return new PENSRequestAlert($arguments);
-		} else if($command == "collect") {
-			return new PENSRequestCollect($arguments);
-		} else if($command == "receipt") {
-			return new PENSRequestReceipt($arguments);
-		} else {
-			throw new PENSException(2002);
-		}
-	}
+    /**
+     * Factory method
+     * 
+     * Creates the right PENSRequest object, based on the command inside the arguments
+     * 
+     * @param array Associative array of arguments
+     * @return mixed PENSRequestAlert or PENSRequestCollect or PENSRequestReceipt
+     * 
+     * @throws PENSException with code 2002 if command is invalid
+     */
+    public static function createPENSRequest($arguments) {
+        $command = $arguments["command"];
+        if($command == "alert") {
+            return new PENSRequestAlert($arguments);
+        } else if($command == "collect") {
+            return new PENSRequestCollect($arguments);
+        } else if($command == "receipt") {
+            return new PENSRequestReceipt($arguments);
+        } else {
+            throw new PENSException(2002);
+        }
+    }
 }

Spacing +7 added lines, -7 removed lines

@@ -27,10 +27,10 @@ 
  * @licence http://www.gnu.org/licenses/gpl.txt
  */
 
-require_once __DIR__ . '/pens_exception.php';
-require_once __DIR__ . '/pens_request_receipt.php';
-require_once __DIR__ . '/pens_request_collect.php';
-require_once __DIR__ . '/pens_request_alert.php';
+require_once __DIR__.'/pens_exception.php';
+require_once __DIR__.'/pens_request_receipt.php';
+require_once __DIR__.'/pens_request_collect.php';
+require_once __DIR__.'/pens_request_alert.php';
 
 /**
  * PENSRequestFactory
@@ -55,11 +55,11 @@ 
 	 */
 	public static function createPENSRequest($arguments) {
 		$command = $arguments["command"];
-		if($command == "alert") {
+		if ($command == "alert") {
 			return new PENSRequestAlert($arguments);
-		} else if($command == "collect") {
+		} else if ($command == "collect") {
 			return new PENSRequestCollect($arguments);
-		} else if($command == "receipt") {
+		} else if ($command == "receipt") {
 			return new PENSRequestReceipt($arguments);
 		} else {
 			throw new PENSException(2002);