chamilo / chamilo-lms

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/pens/lib/tests/test_pens_request.php

Indentation +163 added lines, -163 removed lines

@@ -44,170 +44,170 @@
  */
 class TestPENSRequest extends UnitTestCase {
 	
-	/**
-	 * Valid arguments to be used for tests
-	 * 
-	 * @var array
-	 */
-	private $args = null;
-	
-	public function setUp() {
-		$this->args = array(
-			"pens-version" => "1.0.0",
-			"command" => "collect",
-			"package-type" => "aicc-pkg",
-			"package-type-version" => "1.0",
-			"package-format" => "zip",
-			"package-id" => "http://myurl.com/12345",
-			"package-url" => "http://myurl.com/mypackage.zip",
-			"package-url-expiry" => "2006-04-01T06:51:29Z",
-			"client" => "Authorware7",
-			"receipt" => "mailto:name@domain.com",
-			"package-url-user-id" => "guillaumev",
-			"package-url-account" => "toto",
-			"package-url-password" => "12345",
-			"system-user-id" => "guillaumev",
-			"system-password" => "12345",
-			"alerts" => "http://myurl.com/alerts",
-			"vendor-data" => "here are my data");
-	}
-	
-	public function createObject($myargs) {
-		return PENSRequestFactory::createPENSRequest($myargs);
-	}
-	
-	public function exceptionTestForValue($key, $value, $code) {
-		try {
-			$myargs = $this->args;
-			if($value === null) {
-				unset($myargs[$key]);
-			} else {
-				$myargs[$key] = $value;
-			}
-			$object = $this->createObject($myargs);
-			$this->fail();
-		} catch(PENSException $e) {
-			$this->assertEqual($e->getCode(), $code);
-		}
-	}
-	
-	public function testPensVersionInvalid() {
-		$this->exceptionTestForValue("pens-version", "0.8.0", 2001);
-	}
-	
-	public function testPensVersionNull() {
-		$this->exceptionTestForValue("pens-version", null, 2001);
-	}
-	
-	
-	public function testCommandInvalid() {
-		$this->exceptionTestForValue("command", "testing", 2002);
-	}
-	
-	public function testCommandNull() {
-		$this->exceptionTestForValue("command", null, 2002);
-	}
-	
-	public function testPackageTypeInvalid() {
-		$this->exceptionTestForValue("package-type", "testing", 2003);
-	}
-	
-	public function testPackageTypeNull() {
-		$this->exceptionTestForValue("package-type", null, 2003);
-	}
-	
-	public function testPackageTypeVersionNull() {
-		$this->exceptionTestForValue("package-type-version", null, 2004);
-	}
-	
-	public function testPackageFormatInvalid() {
-		$this->exceptionTestForValue("package-format", "testing", 2005);
-	}
-	
-	public function testPackageFormatNull() {
-		$this->exceptionTestForValue("package-format", null, 2005);
-	}
-	
-	public function testPackageIdInvalid() {
-		$this->exceptionTestForValue("package-id", "testing", 2007);
-	}
-	
-	public function testPackageIdNull() {
-		$this->exceptionTestForValue("package-id", null, 2007);
-	}
-	
-	public function testPackageUrlInvalid() {
-		$this->exceptionTestForValue("package-url", "testing", 2008);
-	}
-	
-	public function testPackageUrlInvalid2() {
-		$this->exceptionTestForValue("package-url", "http://myurl.com/mypackage", 2008);
-	}
-	
-	public function testPackageUrlNull() {
-		$this->exceptionTestForValue("package-url", null, 2008);
-	}
-	
-	public function testPackageUrlExpiryNull() {
-		$this->exceptionTestForValue("package-url-expiry", null, 2009);
-	}
-	
-	public function testPackageUrlExpiryInvalid() {
-		$this->exceptionTestForValue("package-url-expiry", "testing", 2009);
-	}
-	
-	public function testClientNull() {
-		$this->exceptionTestForValue("client", null, 2010);
-	}
-	
-	public function testReceiptNull() {
-		$this->exceptionTestForValue("receipt", null, 2011);
-	}
-	
-	public function testReceiptInvalid() {
-		$this->exceptionTestForValue("receipt", "testing", 2011);
-	}
-	
-	public function testAlertsInvalid() {
-		$this->exceptionTestForValue("alerts", "testing", 1201);
-	}
-	
-	public function testReceiptValid() {
-		try {
-			$myargs = $this->args;
-			$myargs["receipt"] = "mailto:name@domain.com,name2@domain.com";
-			$object = $this->createObject($myargs);
-			$this->pass();
-		} catch(PENSException $e) {
-			$this->fail();
-		}
-	}
-	
-	public function testValid() {
-		try {
-			$object = $this->createObject($this->args);
-			$this->assertIsA($object, "PENSRequestCollect");
-			$this->assertEqual($object->getPensVersion(), "1.0.0");
-			$this->assertEqual($object->getPackageType(), "aicc-pkg");
-			$this->assertEqual($object->getPackageTypeVersion(), "1.0");
-			$this->assertEqual($object->getPackageFormat(), "zip");
-			$this->assertEqual($object->getPackageId(), "http://myurl.com/12345");
-			$this->assertEqual($object->getPackageUrl(), "http://myurl.com/mypackage.zip");
-			$this->assertIsA($object->getPackageUrlExpiry(), "DateTime");
-			$this->assertEqual($object->getClient(), "Authorware7");
-			$this->assertEqual($object->getReceipt(), "mailto:name@domain.com");
-			$this->assertEqual($object->getPackageUrlUserId(), "guillaumev");
-			$this->assertEqual($object->getPackageUrlAccount(), "toto");
-			$this->assertEqual($object->getPackageUrlPassword(), "12345");
-			$this->assertEqual($object->getSystemUserId(), "guillaumev");
-			$this->assertEqual($object->getSystemPassword(), "12345");
-			$this->assertEqual($object->getAlerts(), "http://myurl.com/alerts");
-			$this->assertEqual($object->getVendorData(), "here are my data");
+    /**
+     * Valid arguments to be used for tests
+     * 
+     * @var array
+     */
+    private $args = null;
+	
+    public function setUp() {
+        $this->args = array(
+            "pens-version" => "1.0.0",
+            "command" => "collect",
+            "package-type" => "aicc-pkg",
+            "package-type-version" => "1.0",
+            "package-format" => "zip",
+            "package-id" => "http://myurl.com/12345",
+            "package-url" => "http://myurl.com/mypackage.zip",
+            "package-url-expiry" => "2006-04-01T06:51:29Z",
+            "client" => "Authorware7",
+            "receipt" => "mailto:name@domain.com",
+            "package-url-user-id" => "guillaumev",
+            "package-url-account" => "toto",
+            "package-url-password" => "12345",
+            "system-user-id" => "guillaumev",
+            "system-password" => "12345",
+            "alerts" => "http://myurl.com/alerts",
+            "vendor-data" => "here are my data");
+    }
+	
+    public function createObject($myargs) {
+        return PENSRequestFactory::createPENSRequest($myargs);
+    }
+	
+    public function exceptionTestForValue($key, $value, $code) {
+        try {
+            $myargs = $this->args;
+            if($value === null) {
+                unset($myargs[$key]);
+            } else {
+                $myargs[$key] = $value;
+            }
+            $object = $this->createObject($myargs);
+            $this->fail();
+        } catch(PENSException $e) {
+            $this->assertEqual($e->getCode(), $code);
+        }
+    }
+	
+    public function testPensVersionInvalid() {
+        $this->exceptionTestForValue("pens-version", "0.8.0", 2001);
+    }
+	
+    public function testPensVersionNull() {
+        $this->exceptionTestForValue("pens-version", null, 2001);
+    }
+	
+	
+    public function testCommandInvalid() {
+        $this->exceptionTestForValue("command", "testing", 2002);
+    }
+	
+    public function testCommandNull() {
+        $this->exceptionTestForValue("command", null, 2002);
+    }
+	
+    public function testPackageTypeInvalid() {
+        $this->exceptionTestForValue("package-type", "testing", 2003);
+    }
+	
+    public function testPackageTypeNull() {
+        $this->exceptionTestForValue("package-type", null, 2003);
+    }
+	
+    public function testPackageTypeVersionNull() {
+        $this->exceptionTestForValue("package-type-version", null, 2004);
+    }
+	
+    public function testPackageFormatInvalid() {
+        $this->exceptionTestForValue("package-format", "testing", 2005);
+    }
+	
+    public function testPackageFormatNull() {
+        $this->exceptionTestForValue("package-format", null, 2005);
+    }
+	
+    public function testPackageIdInvalid() {
+        $this->exceptionTestForValue("package-id", "testing", 2007);
+    }
+	
+    public function testPackageIdNull() {
+        $this->exceptionTestForValue("package-id", null, 2007);
+    }
+	
+    public function testPackageUrlInvalid() {
+        $this->exceptionTestForValue("package-url", "testing", 2008);
+    }
+	
+    public function testPackageUrlInvalid2() {
+        $this->exceptionTestForValue("package-url", "http://myurl.com/mypackage", 2008);
+    }
+	
+    public function testPackageUrlNull() {
+        $this->exceptionTestForValue("package-url", null, 2008);
+    }
+	
+    public function testPackageUrlExpiryNull() {
+        $this->exceptionTestForValue("package-url-expiry", null, 2009);
+    }
+	
+    public function testPackageUrlExpiryInvalid() {
+        $this->exceptionTestForValue("package-url-expiry", "testing", 2009);
+    }
+	
+    public function testClientNull() {
+        $this->exceptionTestForValue("client", null, 2010);
+    }
+	
+    public function testReceiptNull() {
+        $this->exceptionTestForValue("receipt", null, 2011);
+    }
+	
+    public function testReceiptInvalid() {
+        $this->exceptionTestForValue("receipt", "testing", 2011);
+    }
+	
+    public function testAlertsInvalid() {
+        $this->exceptionTestForValue("alerts", "testing", 1201);
+    }
+	
+    public function testReceiptValid() {
+        try {
+            $myargs = $this->args;
+            $myargs["receipt"] = "mailto:name@domain.com,name2@domain.com";
+            $object = $this->createObject($myargs);
+            $this->pass();
+        } catch(PENSException $e) {
+            $this->fail();
+        }
+    }
+	
+    public function testValid() {
+        try {
+            $object = $this->createObject($this->args);
+            $this->assertIsA($object, "PENSRequestCollect");
+            $this->assertEqual($object->getPensVersion(), "1.0.0");
+            $this->assertEqual($object->getPackageType(), "aicc-pkg");
+            $this->assertEqual($object->getPackageTypeVersion(), "1.0");
+            $this->assertEqual($object->getPackageFormat(), "zip");
+            $this->assertEqual($object->getPackageId(), "http://myurl.com/12345");
+            $this->assertEqual($object->getPackageUrl(), "http://myurl.com/mypackage.zip");
+            $this->assertIsA($object->getPackageUrlExpiry(), "DateTime");
+            $this->assertEqual($object->getClient(), "Authorware7");
+            $this->assertEqual($object->getReceipt(), "mailto:name@domain.com");
+            $this->assertEqual($object->getPackageUrlUserId(), "guillaumev");
+            $this->assertEqual($object->getPackageUrlAccount(), "toto");
+            $this->assertEqual($object->getPackageUrlPassword(), "12345");
+            $this->assertEqual($object->getSystemUserId(), "guillaumev");
+            $this->assertEqual($object->getSystemPassword(), "12345");
+            $this->assertEqual($object->getAlerts(), "http://myurl.com/alerts");
+            $this->assertEqual($object->getVendorData(), "here are my data");
 			
-		} catch(PENSException $e) {
-			$this->fail();
-		}
-	}
+        } catch(PENSException $e) {
+            $this->fail();
+        }
+    }
 	
 	
 }

Spacing +4 added lines, -4 removed lines

@@ -79,14 +79,14 @@ 
 	public function exceptionTestForValue($key, $value, $code) {
 		try {
 			$myargs = $this->args;
-			if($value === null) {
+			if ($value === null) {
 				unset($myargs[$key]);
 			} else {
 				$myargs[$key] = $value;
 			}
 			$object = $this->createObject($myargs);
 			$this->fail();
-		} catch(PENSException $e) {
+		} catch (PENSException $e) {
 			$this->assertEqual($e->getCode(), $code);
 		}
 	}
@@ -178,7 +178,7 @@ 
 			$myargs["receipt"] = "mailto:name@domain.com,name2@domain.com";
 			$object = $this->createObject($myargs);
 			$this->pass();
-		} catch(PENSException $e) {
+		} catch (PENSException $e) {
 			$this->fail();
 		}
 	}
@@ -204,7 +204,7 @@ 
 			$this->assertEqual($object->getAlerts(), "http://myurl.com/alerts");
 			$this->assertEqual($object->getVendorData(), "here are my data");
 			
-		} catch(PENSException $e) {
+		} catch (PENSException $e) {
 			$this->fail();
 		}
 	}

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