chamilo / chamilo-lms

plugin/pens/lib/pens/pens_response.php

Indentation +119 added lines, -119 removed lines

@@ -39,137 +39,137 @@
  * @author Guillaume Viguier-Just <guillaume@viguierjust.com>
  * @licence http://www.gnu.org/licenses/gpl.txt
  */
- class PENSResponse extends PENSMessage {
-	 /**
-	  * Error code
-	  * @var int
-	  */
-	 protected $_error = null;
+    class PENSResponse extends PENSMessage {
+        /**
+         * Error code
+         * @var int
+         */
+        protected $_error = null;
 	 
-	 /**
-	  * Descriptive text of the error
-	  * @var string
-	  */
-	 protected $_error_text = null;
+        /**
+         * Descriptive text of the error
+         * @var string
+         */
+        protected $_error_text = null;
 	 
-	 /**
-	  * Data linked to the error
-	  * @var string
-	  */
-	 protected $_pens_data = null;
+        /**
+         * Data linked to the error
+         * @var string
+         */
+        protected $_pens_data = null;
 	 
-	 /**
-	  * Version
-	  * @var string
-	  */
-	 protected $_pens_version = null;
+        /**
+         * Version
+         * @var string
+         */
+        protected $_pens_version = null;
 	 
-	 /**
-	  * Constructor
-	  * 
-	  * @param mixed Can be a PENSException or an error code or a string representing an HTTP response
-	  * @param string error text
-	  * @param string PENS data
-	  */
-	public function __construct($error, $error_text = null, $pens_data = null) {
-		$this->_pens_version = PENSConfig::$version;
-		if($error instanceof PENSException) {
-			$this->_error = $error->getCode();
-			$this->_error_text = $error->getMessage();
-		} else if(is_string($error)){
-			// Parse the string
-			$this->parseResponse($error);
-		} else if(is_array($error)) {
-			// Try to build from array
-			$this->_error = $error["error"];
-			$this->_error_text = $error["error-text"];
-			$this->_pens_data = $error["pens-data"];
-		} else {
-			$this->_error = $error;
-			$this->_error_text = $error_text;
-			$this->_pens_data = $pens_data;
-		}
+        /**
+         * Constructor
+         * 
+         * @param mixed Can be a PENSException or an error code or a string representing an HTTP response
+         * @param string error text
+         * @param string PENS data
+         */
+    public function __construct($error, $error_text = null, $pens_data = null) {
+        $this->_pens_version = PENSConfig::$version;
+        if($error instanceof PENSException) {
+            $this->_error = $error->getCode();
+            $this->_error_text = $error->getMessage();
+        } else if(is_string($error)){
+            // Parse the string
+            $this->parseResponse($error);
+        } else if(is_array($error)) {
+            // Try to build from array
+            $this->_error = $error["error"];
+            $this->_error_text = $error["error-text"];
+            $this->_pens_data = $error["pens-data"];
+        } else {
+            $this->_error = $error;
+            $this->_error_text = $error_text;
+            $this->_pens_data = $pens_data;
+        }
 		
-	}
+    }
 	
-	/**
-	 * Parses an HTTP response and assigns the attributes of the object
-	 * 
-	 * @param string HTTP response
-	 */
-	protected function parseResponse($response) {
-		$lines = explode(PENSConfig::$eol, $response);
-		$i = 1;
-		foreach($lines as $line) {
-			if($i < 5) {
-				$pair = explode("=", $line);
-				if($pair[0] == "error") {
-					$this->_error = intval($pair[1]);
-				} else if($pair[0] == "error-text") {
-					$this->_error_text = $pair[1];
-				} else if($pair[0] == "version") {
-					$this->_pens_version = $pair[1];
-				} else if($pair[0] == "pens-data") {
-					if(!empty($pair[1])) {
-						$this->_pens_data = $pair[1].PENSConfig::$eol;
-					}
-				}
-			} else {
-				if(!empty($line)) {
-					$this->_pens_data .= $line.PENSConfig::$eol;
-				}
-			}
-			$i++;
-		}
-	}
+    /**
+     * Parses an HTTP response and assigns the attributes of the object
+     * 
+     * @param string HTTP response
+     */
+    protected function parseResponse($response) {
+        $lines = explode(PENSConfig::$eol, $response);
+        $i = 1;
+        foreach($lines as $line) {
+            if($i < 5) {
+                $pair = explode("=", $line);
+                if($pair[0] == "error") {
+                    $this->_error = intval($pair[1]);
+                } else if($pair[0] == "error-text") {
+                    $this->_error_text = $pair[1];
+                } else if($pair[0] == "version") {
+                    $this->_pens_version = $pair[1];
+                } else if($pair[0] == "pens-data") {
+                    if(!empty($pair[1])) {
+                        $this->_pens_data = $pair[1].PENSConfig::$eol;
+                    }
+                }
+            } else {
+                if(!empty($line)) {
+                    $this->_pens_data .= $line.PENSConfig::$eol;
+                }
+            }
+            $i++;
+        }
+    }
 	
-	public function getError() {
-		return $this->_error;
-	}
+    public function getError() {
+        return $this->_error;
+    }
 	
-	public function getErrorText() {
-		return $this->_error_text;
-	}
+    public function getErrorText() {
+        return $this->_error_text;
+    }
 	
-	public function getPensData() {
-		return $this->_pens_data;
-	}
+    public function getPensData() {
+        return $this->_pens_data;
+    }
 	
-	public function getVersion() {
-		return $this->_pens_version;
-	}
+    public function getVersion() {
+        return $this->_pens_version;
+    }
 	
-	/**
-	 * Sends the response content through HTTP headers
-	 */
-	public function send() {
-		header("Content-Type: text/plain");
-		print $this;
-	}
+    /**
+     * Sends the response content through HTTP headers
+     */
+    public function send() {
+        header("Content-Type: text/plain");
+        print $this;
+    }
 	
-	/**
-	 * Returns an associative array of the response
-	 * 
-	 * @return array Associative array
-	 */
-	public function getArray() {
-		return array("error" => $this->_error,
-			"error-text" => $this->_error_text,
-			"version" => PENSConfig::$version,
-			"pens-data" => $this->_pens_data);
-	}
+    /**
+     * Returns an associative array of the response
+     * 
+     * @return array Associative array
+     */
+    public function getArray() {
+        return array("error" => $this->_error,
+            "error-text" => $this->_error_text,
+            "version" => PENSConfig::$version,
+            "pens-data" => $this->_pens_data);
+    }
 	
-	/**
-	 * Transforms the object into a string
-	 */
-	public function __toString() {
-		$eol = PENSConfig::$eol;
-		$return = "error=".$this->_error.$eol;
-		$return .= "error-text=".$this->_error_text.$eol;
-		$return .= "version=".PENSConfig::$version.$eol;
-		$return .= "pens-data=".$this->_pens_data.$eol;
-		return $return;
-	}
+    /**
+     * Transforms the object into a string
+     */
+    public function __toString() {
+        $eol = PENSConfig::$eol;
+        $return = "error=".$this->_error.$eol;
+        $return .= "error-text=".$this->_error_text.$eol;
+        $return .= "version=".PENSConfig::$version.$eol;
+        $return .= "pens-data=".$this->_pens_data.$eol;
+        return $return;
+    }
  
- }
+    }
  

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

Indentation +3 added lines, -3 removed lines

@@ -26,9 +26,9 @@
 require_once __DIR__ . '/pens.php';
 
 class MyRequestHandler extends PENSRequestHandler {
-	public function processRequest($request, $response) {
-		// Do the processing of the alert or the receipt here
-	}
+    public function processRequest($request, $response) {
+        // Do the processing of the alert or the receipt here
+    }
 }
 
 $handler = new MyRequestHandler();

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

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

Indentation +3 added lines, -3 removed lines

@@ -26,9 +26,9 @@
 require_once __DIR__ . '/pens.php';
 
 class MyPackageHandler extends PENSPackageHandler {
-	public function processPackage($request, $path_to_package) {
-		// Do the package processing here
-	}
+    public function processPackage($request, $path_to_package) {
+        // Do the package processing here
+    }
 }
 
 $handler = new MyPackageHandler();

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

plugin/pens/lib/tests/test_pens_response.php

Indentation +19 added lines, -19 removed lines

@@ -44,25 +44,25 @@
  */
 class TestPENSResponse extends UnitTestCase {
 	
-	public function testCreationFromPENSException() {
-		$object = new PENSResponse(new PENSException(1101));
-		$this->assertEqual($object->getError(), 1101);
-		$this->assertNotNull($object->getErrorText());
-	}
+    public function testCreationFromPENSException() {
+        $object = new PENSResponse(new PENSException(1101));
+        $this->assertEqual($object->getError(), 1101);
+        $this->assertNotNull($object->getErrorText());
+    }
 	
-	public function testCreationFromArguments() {
-		$object = new PENSResponse(0, "collect command received and understood");
-		$this->assertIdentical($object->getError(), 0);
-		$this->assertEqual($object->getErrorText(), "collect command received and understood");
-	}
+    public function testCreationFromArguments() {
+        $object = new PENSResponse(0, "collect command received and understood");
+        $this->assertIdentical($object->getError(), 0);
+        $this->assertEqual($object->getErrorText(), "collect command received and understood");
+    }
 	
-	public function testCreationFromHTTPResponse() {
-		$eol = PENSConfig::$eol;
-		$response = "error=1101".$eol."error-text=unable to parse PENS command".$eol."version=1.0.0".$eol."pens-data=".$eol;
-		$object = new PENSResponse($response);
-		$this->assertIdentical($object->getError(), 1101);
-		$this->assertIdentical($object->getErrorText(), "unable to parse PENS command");
-		$this->assertIdentical($object->getVersion(), "1.0.0");
-		$this->assertNull($object->getPensData());
-	}
+    public function testCreationFromHTTPResponse() {
+        $eol = PENSConfig::$eol;
+        $response = "error=1101".$eol."error-text=unable to parse PENS command".$eol."version=1.0.0".$eol."pens-data=".$eol;
+        $object = new PENSResponse($response);
+        $this->assertIdentical($object->getError(), 1101);
+        $this->assertIdentical($object->getErrorText(), "unable to parse PENS command");
+        $this->assertIdentical($object->getVersion(), "1.0.0");
+        $this->assertNull($object->getPensData());
+    }
 }

plugin/pens/lib/tests/test_pens_package_handler.php

Indentation +32 added lines, -32 removed lines

@@ -33,8 +33,8 @@ 
 require_once(dirname(__FILE__)."/../pens.php");
 
 class MyPackageHandler extends PENSPackageHandler {
-	public function processPackage($request, $path_to_package) {
-	}
+    public function processPackage($request, $path_to_package) {
+    }
 }
 
 /**
@@ -49,39 +49,39 @@ 
  */
 class TestPENSPackageHandler extends UnitTestCase {
 	
-	public function testSupportedPackageTypesInvalid() {
-		$object = new MyPackageHandler();
-		$object->setSupportedPackageTypes("testing");
-		$this->assertEqual($object->getSupportedPackageTypes(), null);
-	}
+    public function testSupportedPackageTypesInvalid() {
+        $object = new MyPackageHandler();
+        $object->setSupportedPackageTypes("testing");
+        $this->assertEqual($object->getSupportedPackageTypes(), null);
+    }
 	
-	public function testSupportedPackageTypesInvalid2() {
-		$object = new MyPackageHandler();
-		$object->setSupportedPackageTypes(array("aicc-pkg", "testing"));
-		$this->assertEqual($object->getSupportedPackageTypes(), array("aicc-pkg"));
-	}
+    public function testSupportedPackageTypesInvalid2() {
+        $object = new MyPackageHandler();
+        $object->setSupportedPackageTypes(array("aicc-pkg", "testing"));
+        $this->assertEqual($object->getSupportedPackageTypes(), array("aicc-pkg"));
+    }
 	
-	public function testSupportedPackageTypesValid() {
-		$object = new MyPackageHandler();
-		$object->setSupportedPackageTypes(array("aicc-pkg", "lms-qti"));
-		$this->assertEqual($object->getSupportedPackageTypes(), array("aicc-pkg", "lms-qti"));
-	}
+    public function testSupportedPackageTypesValid() {
+        $object = new MyPackageHandler();
+        $object->setSupportedPackageTypes(array("aicc-pkg", "lms-qti"));
+        $this->assertEqual($object->getSupportedPackageTypes(), array("aicc-pkg", "lms-qti"));
+    }
 	
-	public function testSupportedPackageFormatsInvalid() {
-		$object = new MyPackageHandler();
-		$object->setSupportedPackageFormats("testing");
-		$this->assertEqual($object->getSupportedPackageFormats(), null);
-	}
+    public function testSupportedPackageFormatsInvalid() {
+        $object = new MyPackageHandler();
+        $object->setSupportedPackageFormats("testing");
+        $this->assertEqual($object->getSupportedPackageFormats(), null);
+    }
 	
-	public function testSupportedPackageFormatsInvalid2() {
-		$object = new MyPackageHandler();
-		$object->setSupportedPackageFormats(array("zip", "testing"));
-		$this->assertEqual($object->getSupportedPackageFormats(), array("zip"));
-	}
+    public function testSupportedPackageFormatsInvalid2() {
+        $object = new MyPackageHandler();
+        $object->setSupportedPackageFormats(array("zip", "testing"));
+        $this->assertEqual($object->getSupportedPackageFormats(), array("zip"));
+    }
 	
-	public function testSupportedPackageFormatsValid() {
-		$object = new MyPackageHandler();
-		$object->setSupportedPackageFormats(array("zip", "jar"));
-		$this->assertEqual($object->getSupportedPackageFormats(), array("zip", "jar"));
-	}
+    public function testSupportedPackageFormatsValid() {
+        $object = new MyPackageHandler();
+        $object->setSupportedPackageFormats(array("zip", "jar"));
+        $this->assertEqual($object->getSupportedPackageFormats(), array("zip", "jar"));
+    }
 }

plugin/pens/lib/tests/test_pens_server.php

Indentation +58 added lines, -58 removed lines

@@ -46,77 +46,77 @@
  */
 class TestPENSServer extends WebTestCase {
 	
-	private $_valid_params = array(
-		"command" => "collect",
-		"pens-version" => "1.0.0",
-		"package-type" => "aicc-pkg",
-		"package-type-version" => "1.0",
-		"package-format" => "zip",
-		"package-id" => "http://myurl.com/12345",
-		"package-url" => "http://www.aicc.org/pens/sample/captivatequiz.zip",
-		"package-url-expiry" => "2099-04-09T14:17:43Z",
-		"client" => "test-pens-server",
-		"receipt" => PENS_TEST_RECEIPT_MAILTO);
+    private $_valid_params = array(
+        "command" => "collect",
+        "pens-version" => "1.0.0",
+        "package-type" => "aicc-pkg",
+        "package-type-version" => "1.0",
+        "package-format" => "zip",
+        "package-id" => "http://myurl.com/12345",
+        "package-url" => "http://www.aicc.org/pens/sample/captivatequiz.zip",
+        "package-url-expiry" => "2099-04-09T14:17:43Z",
+        "client" => "test-pens-server",
+        "receipt" => PENS_TEST_RECEIPT_MAILTO);
 		
-	public function testEmptyQuery() {
-		$this->get(PENS_TEST_SERVER_URL);
-		$this->assertText("error=2002");
-	}
+    public function testEmptyQuery() {
+        $this->get(PENS_TEST_SERVER_URL);
+        $this->assertText("error=2002");
+    }
 	
-	public function testValidQuery() {
-		$this->get(PENS_TEST_SERVER_URL, $this->_valid_params);
-		$this->assertText("error=0");
-	}
+    public function testValidQuery() {
+        $this->get(PENS_TEST_SERVER_URL, $this->_valid_params);
+        $this->assertText("error=0");
+    }
 	
-	private function invalidParameterTest($name, $value, $error) {
-		$params = $this->_valid_params;
-		$params[$name] = $value;
-		$this->get(PENS_TEST_SERVER_URL, $params);
-		$this->assertText("error=".$error);
-	}
+    private function invalidParameterTest($name, $value, $error) {
+        $params = $this->_valid_params;
+        $params[$name] = $value;
+        $this->get(PENS_TEST_SERVER_URL, $params);
+        $this->assertText("error=".$error);
+    }
 	
-	private function validParameterTest($name, $value) {
-		$params = $this->_valid_params;
-		$params[$name] = $value;
-		$this->get(PENS_TEST_SERVER_URL, $params);
-		$this->assertText("error=0");
-	}
+    private function validParameterTest($name, $value) {
+        $params = $this->_valid_params;
+        $params[$name] = $value;
+        $this->get(PENS_TEST_SERVER_URL, $params);
+        $this->assertText("error=0");
+    }
 	
-	public function testInvalidCommand() {
-		$this->invalidParameterTest("command", "invalid", 2002);
-	}
+    public function testInvalidCommand() {
+        $this->invalidParameterTest("command", "invalid", 2002);
+    }
 	
-	public function testInvalidVersion() {
-		$this->invalidParameterTest("pens-version", "0.8.0", 2001);
-	}
+    public function testInvalidVersion() {
+        $this->invalidParameterTest("pens-version", "0.8.0", 2001);
+    }
 	
-	public function testInvalidPackageType() {
-		$this->invalidParameterTest("package-type", "invalid", 2003);
-	}
+    public function testInvalidPackageType() {
+        $this->invalidParameterTest("package-type", "invalid", 2003);
+    }
 	
-	public function testEmptyPackageTypeVersion() {
-		$this->invalidParameterTest("package-type-version", null, 2004);
-	}
+    public function testEmptyPackageTypeVersion() {
+        $this->invalidParameterTest("package-type-version", null, 2004);
+    }
 	
-	public function testInvalidPackageFormat() {
-		$this->invalidParameterTest("package-format", "invalid", 2005);
-	}
+    public function testInvalidPackageFormat() {
+        $this->invalidParameterTest("package-format", "invalid", 2005);
+    }
 	
-	public function testInvalidPackageId() {
-		$this->invalidParameterTest("package-id", "invalid", 2007);
-	}
+    public function testInvalidPackageId() {
+        $this->invalidParameterTest("package-id", "invalid", 2007);
+    }
 	
-	public function testInvalidPackageUrl() {
-		$this->invalidParameterTest("package-url", "invalid", 2008);
-	}
+    public function testInvalidPackageUrl() {
+        $this->invalidParameterTest("package-url", "invalid", 2008);
+    }
 	
-	public function testInvalidPackageUrlExpiry() {
-		$this->invalidParameterTest("package-url-expiry", "invalid", 2009);
-	}
+    public function testInvalidPackageUrlExpiry() {
+        $this->invalidParameterTest("package-url-expiry", "invalid", 2009);
+    }
 	
-	public function testValidPackageUrlExpiry() {
-		$this->validParameterTest("package-url-expiry", "2099-01-01");
-	}
+    public function testValidPackageUrlExpiry() {
+        $this->validParameterTest("package-url-expiry", "2099-01-01");
+    }
 	
 }
 

plugin/tour/src/tour_plugin.class.php

Spacing +2 added lines, -2 removed lines

@@ -129,9 +129,9 @@
      */
     public function getTourConfig()
     {
-        $pluginPath = api_get_path(PLUGIN_PATH) . 'tour/';
+        $pluginPath = api_get_path(PLUGIN_PATH).'tour/';
 
-        $jsonContent = file_get_contents($pluginPath . 'config/tour.json');
+        $jsonContent = file_get_contents($pluginPath.'config/tour.json');
 
         $jsonData = json_decode($jsonContent, true);
 

plugin/tour/index.php

Spacing +3 added lines, -3 removed lines

@@ -5,10 +5,10 @@
  * @author Angel Fernando Quiroz Campos <angel.quiroz@beeznest.com>
  * @package chamilo.plugin.tour
  */
-require_once __DIR__ . '/config.php';
+require_once __DIR__.'/config.php';
 
-$pluginPath = api_get_path(PLUGIN_PATH) . 'tour/';
-$pluginWebPath = api_get_path(WEB_PLUGIN_PATH) . 'tour/';
+$pluginPath = api_get_path(PLUGIN_PATH).'tour/';
+$pluginWebPath = api_get_path(WEB_PLUGIN_PATH).'tour/';
 
 $userId = api_get_user_id();