chamilo / chamilo-lms

plugin/pens/lib/pens/pens_exception.php

Spacing +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
 	 * Sets the message based on the code
 	 */
 	protected function setMessageBasedOnCode() {
-		if(empty($this->message) && !empty(self::$_code_to_messages[$this->code])) {
+		if (empty($this->message) && !empty(self::$_code_to_messages[$this->code])) {
 			$this->message = self::$_code_to_messages[$this->code];
 		}
 	}

plugin/pens/lib/pens/pens_package_handler.php

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_receipt.php

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_server.php

Spacing +28 added lines, -28 removed lines

@@ -26,10 +26,10 @@ 
  * @licence http://www.gnu.org/licenses/gpl.txt
  */
 
-require_once __DIR__ . '/pens_controller.php';
-require_once __DIR__ . '/pens_package_handler.php';
-require_once __DIR__ . '/pens_exception.php';
-require_once __DIR__ . '/pens_response.php';
+require_once __DIR__.'/pens_controller.php';
+require_once __DIR__.'/pens_package_handler.php';
+require_once __DIR__.'/pens_exception.php';
+require_once __DIR__.'/pens_response.php';
 
 /**
  * PENSServer
@@ -64,7 +64,7 @@ 
 	 * Singleton method
 	 */
 	public static function singleton() {
-		if(!isset(self::$_instance)) {
+		if (!isset(self::$_instance)) {
 			$c = __CLASS__;
 			self::$_instance = new $c;
 		}
@@ -88,7 +88,7 @@ 
 	 * @param PENSPackageHandler Package handler
 	 */
 	public function setPackageHandler($package_handler) {
-		if($package_handler instanceof PENSPackageHandler) {
+		if ($package_handler instanceof PENSPackageHandler) {
 			$this->_package_handler = $package_handler;
 		}
 	}
@@ -101,8 +101,8 @@ 
 		try {
 			// First, try to parse the request
 			$request = $this->parseRequest();
-			if($request->getCommand() == "collect") {
-				if(isset($_REQUEST['process'])) {
+			if ($request->getCommand() == "collect") {
+				if (isset($_REQUEST['process'])) {
 					// Collect the package and process it
 					$receipt = null;
 					$path_to_package = null;
@@ -110,13 +110,13 @@ 
 						// Collect the package
 						$path_to_package = $this->collectPackage($request);
 						$receipt = new PENSResponse(0, "package successfully collected");
-					} catch(PENSException $e) {
+					} catch (PENSException $e) {
 						$receipt = new PENSResponse($e);
 					}
 					// Send receipt
 					$response = $this->sendReceipt($request, $receipt);
-					if(!is_null($response) && !is_null($path_to_package)) {
-						if($response->getError() === 0) {
+					if (!is_null($response) && !is_null($path_to_package)) {
+						if ($response->getError() === 0) {
 							// Process package
 							$this->processPackage($request, $path_to_package);
 						}
@@ -129,7 +129,7 @@ 
 					$params = $_REQUEST;
 					$params['process'] = 1;
 					$scheme = "http";
-					if(!empty($_SERVER['HTTPS'])) {
+					if (!empty($_SERVER['HTTPS'])) {
 						$scheme = "https";
 					}
 					$ch = curl_init();
@@ -141,7 +141,7 @@ 
 				}
 			}
 				
-		} catch(PENSException $e) {
+		} catch (PENSException $e) {
 			// If we could not parse the request, send the error back to the client
 			$this->sendResponse(new PENSResponse($e));
 		}
@@ -156,20 +156,20 @@ 
 	 */
 	protected function collectPackage($request) {
 		$supported_package_types = $this->_package_handler->getSupportedPackageTypes();
-		if(!in_array($request->getPackageType(), $supported_package_types)) {
+		if (!in_array($request->getPackageType(), $supported_package_types)) {
 			throw new PENSException(1430);
 		}
 		$supported_package_formats = $this->_package_handler->getSupportedPackageFormats();
-		if(!in_array($request->getPackageFormat(), $supported_package_formats)) {
+		if (!in_array($request->getPackageFormat(), $supported_package_formats)) {
 			throw new PENSException(1431);
 		}
-		if(!$this->isExpiryDateValid($request->getPackageUrlExpiry())) {
+		if (!$this->isExpiryDateValid($request->getPackageUrlExpiry())) {
 			throw new PENSException(1322);
 		}
 		
 		// Try to download the package in the temporary directory
 		$tmp = null;
-		if(function_exists("sys_get_temp_dir")) {
+		if (function_exists("sys_get_temp_dir")) {
 			$tmp = sys_get_temp_dir();
 		} else {
 			$tmp = "/tmp";
@@ -180,14 +180,14 @@ 
 		curl_setopt($ch, CURLOPT_URL, $request->getPackageUrl());
 		curl_setopt($ch, CURLOPT_HEADER, false);	
 		curl_setopt($ch, CURLOPT_FILE, $fp);
-		if(!is_null($request->getPackageUrlUserId())) {
+		if (!is_null($request->getPackageUrlUserId())) {
 			curl_setopt($ch, CURLOPT_USERPWD, $request->getPackageUrlUserId().":".$request->getPackageUrlPassword());
 		}
-		if(curl_exec($ch) === false) {
+		if (curl_exec($ch) === false) {
 			$errno = curl_errno($ch);
 			curl_close($ch);
 			// Error occured. Throw an exception
-			switch($errno) {
+			switch ($errno) {
 				case CURLE_UNSUPPORTED_PROTOCOL:
 					throw new PENSException(1301);
 					break;
@@ -223,7 +223,7 @@ 
 		date_default_timezone_set('UTC');
 		$current_time = time();
 		$expiry_time = strtotime($expiry->format(DateTime::ISO8601));
-		if($current_time > $expiry_time) {
+		if ($current_time > $expiry_time) {
 			return false;
 		} else {
 			return true;
@@ -239,17 +239,17 @@ 
 	 * @return PENSResponse Response
 	 */
 	protected function sendAlertOrReceipt($request, $response, $mode) {
-		if($mode == "alert") {
+		if ($mode == "alert") {
 			$url = $request->getAlerts();
 		} else {
 			$url = $request->getReceipt();
 		}
-		if(!empty($url)) {
+		if (!empty($url)) {
 			$url_components = parse_url($url);
 			$scheme = $url_components["scheme"];
-			if($scheme == "mailto") {
+			if ($scheme == "mailto") {
 				$to = $url_components["path"];
-				if($mode == "alert") {
+				if ($mode == "alert") {
 					$subject = "PENS Alert for ".$request->getPackageId();
 				} else {
 					$subject = "PENS Receipt for ".$request->getPackageId();
@@ -257,8 +257,8 @@ 
 				$message = $response->__toString();
 				mail($to, $subject, $message);
 				return new PENSResponse(0, "");
-			} else if($scheme == "http" || $scheme == "https") {
-				if($mode == "alert") {
+			} else if ($scheme == "http" || $scheme == "https") {
+				if ($mode == "alert") {
 					$params = array_merge($request->getSendAlertArray(), $response->getArray());
 				} else {
 					$params = array_merge($request->getSendReceiptArray(), $response->getArray());
@@ -269,7 +269,7 @@ 
 				curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
 				$data = curl_exec($ch);
 				curl_close($ch);
-				if($data === false) {
+				if ($data === false) {
 					return null;
 				} else {
 					return new PENSResponse($data);

plugin/pens/lib/pens/pens_request_factory.php

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);

plugin/pens/lib/pens/pens_request.php

Spacing +17 added lines, -17 removed lines

@@ -186,7 +186,7 @@ 
 	 * @throws PENSException with code 2001 if invalid
 	 */
 	public function setPensVersion($pens_version) {
-		if($pens_version == PENSConfig::$version) {
+		if ($pens_version == PENSConfig::$version) {
 			$this->_pens_version = $pens_version;
 		} else {
 			throw new PENSException(2001);
@@ -205,7 +205,7 @@ 
 	 * @throws PENSException with code 2002 if invalid
 	 */
 	protected function setCommand($command) {
-		if(in_array($command, PENSConfig::$allowed_commands)) {
+		if (in_array($command, PENSConfig::$allowed_commands)) {
 			$this->_command = $command;
 		} else {
 			throw new PENSException(2002);
@@ -224,7 +224,7 @@ 
 	 * @throws PENSException with code 2003 if invalid
 	 */
 	public function setPackageType($package_type) {
-		if(in_array($package_type, PENSConfig::$allowed_package_types)) {
+		if (in_array($package_type, PENSConfig::$allowed_package_types)) {
 			$this->_package_type = $package_type;
 		} else {
 			throw new PENSException(2003);
@@ -243,7 +243,7 @@ 
 	 * @throws PENSException with code 2004 if invalid
 	 */
 	public function setPackageTypeVersion($package_type_version) {
-		if(empty($package_type_version)) {
+		if (empty($package_type_version)) {
 			throw new PENSException(2004);
 		} else {
 			$this->_package_type_version = $package_type_version;
@@ -262,7 +262,7 @@ 
 	 * @throws PENSException with code 2005 if invalid
 	 */
 	public function setPackageFormat($package_format) {
-		if(in_array($package_format, PENSConfig::$allowed_package_formats)) {
+		if (in_array($package_format, PENSConfig::$allowed_package_formats)) {
 			$this->_package_format = $package_format;
 		} else {
 			throw new PENSException(2005);
@@ -316,7 +316,7 @@ 
 	}
 	
 	public function setPackageUrlUserId($package_url_user_id) {
-		if(!empty($package_url_user_id)) {
+		if (!empty($package_url_user_id)) {
 			$this->_package_url_user_id = $package_url_user_id;
 		}
 	}
@@ -326,7 +326,7 @@ 
 	}
 	
 	public function setPackageUrlAccount($package_url_account) {
-		if(!empty($package_url_account)) {
+		if (!empty($package_url_account)) {
 			$this->_package_url_account = $package_url_account;
 		}
 	}
@@ -336,7 +336,7 @@ 
 	}
 	
 	public function setPackageUrlPassword($package_url_password) {
-		if(!empty($package_url_password)) {
+		if (!empty($package_url_password)) {
 			$this->_package_url_password = $package_url_password;
 		}
 	}
@@ -354,13 +354,13 @@ 
 	 * @todo Perform a better validation of the date
 	 */
 	public function setPackageUrlExpiry($package_url_expiry) {
-		if(empty($package_url_expiry)) {
+		if (empty($package_url_expiry)) {
 			throw new PENSException(2009);
 		} else {
 			try {
 				$expiry = new DateTime($package_url_expiry, new DateTimeZone('UTC'));
 				$this->_package_url_expiry = $expiry;
-			} catch(Exception $e) {
+			} catch (Exception $e) {
 				throw new PENSException(2009);
 			}
 		}
@@ -378,7 +378,7 @@ 
 	 * @throws PENSException with code 2010 if invalid
 	 */
 	public function setClient($client) {
-		if(!empty($client)) {
+		if (!empty($client)) {
 			$this->_client = $client;
 		} else {
 			throw new PENSException(2010);
@@ -390,7 +390,7 @@ 
 	}
 	
 	public function setSystemUserId($system_user_id) {
-		if(!empty($system_user_id)) {
+		if (!empty($system_user_id)) {
 			$this->_system_user_id = $system_user_id;
 		}
 	}
@@ -400,7 +400,7 @@ 
 	}
 	
 	public function setSystemPassword($system_password) {
-		if(!empty($system_password)) {
+		if (!empty($system_password)) {
 			$this->_system_password = $system_password;
 		}
 	}
@@ -417,7 +417,7 @@ 
 	 * @throws PENSException with code 2011 if invalid
 	 */
 	public function setReceipt($receipt) {
-		if($this instanceof PENSRequestCollect) {
+		if ($this instanceof PENSRequestCollect) {
 			if (preg_match('/'.ABSOLUTEURI_2396.'/', $receipt)) {
 				$this->_receipt = $receipt;
 			} else {
@@ -431,8 +431,8 @@ 
 	}
 	
 	public function setAlerts($alerts) {
-		if(!empty($alerts)) {
-			if(preg_match('/'.ABSOLUTEURI_2396.'/', $alerts)) {
+		if (!empty($alerts)) {
+			if (preg_match('/'.ABSOLUTEURI_2396.'/', $alerts)) {
 				$this->_alerts = $alerts;
 			} else {
 				throw new PENSException(1201);
@@ -445,7 +445,7 @@ 
 	}
 	
 	public function setVendorData($vendor_data) {
-		if(!empty($vendor_data)) {
+		if (!empty($vendor_data)) {
 			$this->_vendor_data = $vendor_data;
 		}
 	}

plugin/pens/lib/pens/pens_response.php

Spacing +14 added lines, -14 removed lines

@@ -26,9 +26,9 @@ 
  * @licence http://www.gnu.org/licenses/gpl.txt
  */
 
-require_once __DIR__ . '/pens_config.php';
-require_once __DIR__ . '/pens_message.php';
-require_once __DIR__ . '/pens_exception.php';
+require_once __DIR__.'/pens_config.php';
+require_once __DIR__.'/pens_message.php';
+require_once __DIR__.'/pens_exception.php';
 
 /**
  * PENSResponse
@@ -73,13 +73,13 @@ 
 	  */
 	public function __construct($error, $error_text = null, $pens_data = null) {
 		$this->_pens_version = PENSConfig::$version;
-		if($error instanceof PENSException) {
+		if ($error instanceof PENSException) {
 			$this->_error = $error->getCode();
 			$this->_error_text = $error->getMessage();
-		} else if(is_string($error)){
+		} else if (is_string($error)) {
 			// Parse the string
 			$this->parseResponse($error);
-		} else if(is_array($error)) {
+		} else if (is_array($error)) {
 			// Try to build from array
 			$this->_error = $error["error"];
 			$this->_error_text = $error["error-text"];
@@ -100,22 +100,22 @@ 
 	protected function parseResponse($response) {
 		$lines = explode(PENSConfig::$eol, $response);
 		$i = 1;
-		foreach($lines as $line) {
-			if($i < 5) {
+		foreach ($lines as $line) {
+			if ($i < 5) {
 				$pair = explode("=", $line);
-				if($pair[0] == "error") {
+				if ($pair[0] == "error") {
 					$this->_error = intval($pair[1]);
-				} else if($pair[0] == "error-text") {
+				} else if ($pair[0] == "error-text") {
 					$this->_error_text = $pair[1];
-				} else if($pair[0] == "version") {
+				} else if ($pair[0] == "version") {
 					$this->_pens_version = $pair[1];
-				} else if($pair[0] == "pens-data") {
-					if(!empty($pair[1])) {
+				} else if ($pair[0] == "pens-data") {
+					if (!empty($pair[1])) {
 						$this->_pens_data = $pair[1].PENSConfig::$eol;
 					}
 				}
 			} else {
-				if(!empty($line)) {
+				if (!empty($line)) {
 					$this->_pens_data .= $line.PENSConfig::$eol;
 				}
 			}

plugin/pens/lib/pens.php

Spacing +5 added lines, -5 removed lines

@@ -1,7 +1,7 @@
 <?php
 
-require_once __DIR__ . '/pens/pens_request_factory.php';
-require_once __DIR__ . '/pens/pens_response.php';
-require_once __DIR__ . '/pens/pens_package_handler.php';
-require_once __DIR__ . '/pens/pens_server.php';
-require_once __DIR__ . '/pens/pens_client.php';
+require_once __DIR__.'/pens/pens_request_factory.php';
+require_once __DIR__.'/pens/pens_response.php';
+require_once __DIR__.'/pens/pens_package_handler.php';
+require_once __DIR__.'/pens/pens_server.php';
+require_once __DIR__.'/pens/pens_client.php';

plugin/pens/lib/pens_server_test.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
  * @licence http://www.gnu.org/licenses/gpl.txt
  */
 
-require_once __DIR__ . '/pens.php';
+require_once __DIR__.'/pens.php';
 
 class MyPackageHandler extends PENSPackageHandler {
 	public function processPackage($request, $path_to_package) {