chamilo / chamilo-lms

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

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");
+    }
 
 }

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

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;
+    }
  
- }
+    }
  

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

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");
+    }
 	
 }