chamilo / chamilo-lms

plugin/pens/plugin.php

Spacing +5 added lines, -5 removed lines

@@ -24,13 +24,13 @@
  */
 
 //the plugin title
-$plugin_info['title']='PENS';
+$plugin_info['title'] = 'PENS';
 //the comments that go with the plugin
-$plugin_info['comment']="PENS implementation for Chamilo";
+$plugin_info['comment'] = "PENS implementation for Chamilo";
 //the locations where this plugin can be shown
-$plugin_info['location']=array();
+$plugin_info['location'] = array();
 //the plugin version
-$plugin_info['version']='1.1';
+$plugin_info['version'] = '1.1';
 //the plugin author
-$plugin_info['author']='Guillaume Viguier-Just, Yannick Warnier';
+$plugin_info['author'] = 'Guillaume Viguier-Just, Yannick Warnier';
 $plugin_info = PENSPlugin::create()->get_info();
\ No newline at end of file

plugin/pens/chamilo_pens.php

Spacing +17 added lines, -17 removed lines

@@ -25,8 +25,8 @@ 
  * @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__.'/../../main/inc/global.inc.php';
+require_once __DIR__.'/lib/pens.php';
 
 /**
  * ChamiloPens
@@ -114,7 +114,7 @@ 
 	 * @param PENSRequest Request
 	 */
 	public function __construct($request) {
-		if($request instanceof PENSRequest) {
+		if ($request instanceof PENSRequest) {
 			$this->_id = 0;
 			$this->_pens_version = $request->getPensVersion();
 			$this->_package_type = $request->getPackageType();
@@ -124,7 +124,7 @@ 
 			$this->_client = $request->getClient();
 			$this->_vendor_data = $request->getVendorData();
 			$this->_package_name = $request->getFilename();
-		} else if(is_array($request)) {
+		} else if (is_array($request)) {
 			$this->_id = $request['id'];
 			$this->_pens_version = $request['pens_version'];
 			$this->_package_type = $request['package_type'];
@@ -135,7 +135,7 @@ 
 			$this->_vendor_data = $request['vendor_data'];
 			$this->_package_name = $request['package_name'];
 			$this->_created_at = new DateTime($request['created_at'], new DateTimeZone('UTC'));
-			if(!empty($request['updated_at'])) {
+			if (!empty($request['updated_at'])) {
 				$this->_updated_at = new DateTime($request['updated_at'], new DateTimeZone('UTC'));
 			}
 		}
@@ -151,16 +151,16 @@ 
 		$clean_vendor_data = Database::escape_string($this->_vendor_data);
 		$created_at = api_get_utc_datetime();
 		$table = Database::get_main_table(ChamiloPens::TABLE_NAME);
-		$sql_query = "INSERT INTO $table (pens_version, package_type, package_type_version, package_format, package_id, client, vendor_data, package_name, created_at) VALUES (" .
-			"'". $this->_pens_version. "', " .
-			"'".$this->_package_type. "', " .
-			"'".$clean_package_type_version. "', " .
-			"'".$this->_package_format. "', " .
-			"'".$clean_package_id. "', " .
-			"'".$clean_client."', " .
-			"'".$clean_vendor_data."', " .
-			"'".$this->_package_name."', " .
-			"'".$created_at."') ON DUPLICATE KEY UPDATE " .
+		$sql_query = "INSERT INTO $table (pens_version, package_type, package_type_version, package_format, package_id, client, vendor_data, package_name, created_at) VALUES (".
+			"'".$this->_pens_version."', ".
+			"'".$this->_package_type."', ".
+			"'".$clean_package_type_version."', ".
+			"'".$this->_package_format."', ".
+			"'".$clean_package_id."', ".
+			"'".$clean_client."', ".
+			"'".$clean_vendor_data."', ".
+			"'".$this->_package_name."', ".
+			"'".$created_at."') ON DUPLICATE KEY UPDATE ".
 			"pens_version = VALUES(pens_version), ".
 			"package_type = VALUES(package_type), ".
 			"package_type_version = VALUES(package_type_version), ".
@@ -182,7 +182,7 @@ 
 		$sql_query = "SELECT * FROM $table WHERE package_id = '".$package_id."';";
 		$results = Database::query($sql_query);
 		$number = Database::num_rows($results);
-		if($number == 1) {
+		if ($number == 1) {
 			$obj = Database::fetch_assoc($results);
 			return new ChamiloPens($obj);
 		} else {
@@ -200,7 +200,7 @@ 
 		$sql_query = "SELECT * FROM $table ORDER BY created_at;";
 		$results = Database::query($sql_query);
 		$return = array();
-		while($assoc = Database::fetch_assoc($results)) {
+		while ($assoc = Database::fetch_assoc($results)) {
 			$return[] = new ChamiloPens($assoc);
 		}
 		return $return;

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

plugin/pens/lib/pens/pens_client.php

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