vanderlee / PHPSwaggerGen

tests/Swagger/Type/ObjectTypeTest.php

Indentation +11 added lines, -11 removed lines

@@ -337,23 +337,23 @@
 				. ',"tags":[{"name":"Test"}]}', json_encode($array, JSON_NUMERIC_CHECK));
 	}
 
-    public function testObjectPropertiesReadOnly()
-    {
-        $object = new \SwaggerGen\SwaggerGen();
-        $array = $object->getSwagger(array('
+	public function testObjectPropertiesReadOnly()
+	{
+		$object = new \SwaggerGen\SwaggerGen();
+		$array = $object->getSwagger(array('
 			api Test
 			endpoint /test
 			method GET something
 			response 200 object(a!:array(A),b:array(B))
 		'));
 
-        $this->assertSame('{"swagger":2,"info":{"title":"undefined","version":0}'
-            . ',"paths":{"\/test":{"get":{"tags":["Test"],"summary":"something"'
-            . ',"responses":{"200":{"description":"OK","schema":{"type":"object","required":["b"]'
-            . ',"properties":{"a":{"type":"array","items":{"$ref":"#\/definitions\/A"}}'
-            . ',"b":{"type":"array","items":{"$ref":"#\/definitions\/B"}}}}}}}}}'
-            . ',"tags":[{"name":"Test"}]}', json_encode($array, JSON_NUMERIC_CHECK));
-    }
+		$this->assertSame('{"swagger":2,"info":{"title":"undefined","version":0}'
+			. ',"paths":{"\/test":{"get":{"tags":["Test"],"summary":"something"'
+			. ',"responses":{"200":{"description":"OK","schema":{"type":"object","required":["b"]'
+			. ',"properties":{"a":{"type":"array","items":{"$ref":"#\/definitions\/A"}}'
+			. ',"b":{"type":"array","items":{"$ref":"#\/definitions\/B"}}}}}}}}}'
+			. ',"tags":[{"name":"Test"}]}', json_encode($array, JSON_NUMERIC_CHECK));
+	}
 
 	public function testDeepObjectProperties()
 	{

tests/Swagger/Type/PropertyTest.php

Indentation +14 added lines, -14 removed lines

@@ -81,20 +81,20 @@
 				), $object->toArray());
 	}
 
-    /**
-     * @covers \SwaggerGen\Swagger\Type\PropertyType::__construct
-     */
-    public function testConstructReadOnly()
-    {
-        $object = new SwaggerGen\Swagger\Type\Property($this->parent, 'string', 'Some words here', true);
-        $this->assertInstanceOf('\SwaggerGen\Swagger\Type\Property', $object);
-
-        $this->assertSame(array(
-            'type' => 'string',
-            'description' => 'Some words here',
-            'readOnly' => true
-        ), $object->toArray());
-    }
+	/**
+	 * @covers \SwaggerGen\Swagger\Type\PropertyType::__construct
+	 */
+	public function testConstructReadOnly()
+	{
+		$object = new SwaggerGen\Swagger\Type\Property($this->parent, 'string', 'Some words here', true);
+		$this->assertInstanceOf('\SwaggerGen\Swagger\Type\Property', $object);
+
+		$this->assertSame(array(
+			'type' => 'string',
+			'description' => 'Some words here',
+			'readOnly' => true
+		), $object->toArray());
+	}
 
 	/**
 	 * @covers \SwaggerGen\Swagger\Type\PropertyType::__construct

example/api/Example.class.php

Braces +2 added lines, -1 removed lines

@@ -29,7 +29,8 @@
  *
  * @rest\api Users Get some useful information on users
  */
-class Example {
+class Example
+{
 
 	private $data = array();
 

tests/output/OutputTest.php

Braces +3 added lines, -2 removed lines

@@ -30,10 +30,11 @@
 		);
 	}
 	
-	public function provideAllCases() {
+	public function provideAllCases()
+	{
 		$cases = array();
 		
-		foreach (glob(__DIR__ . '/*', GLOB_ONLYDIR) as $dir) {					
+		foreach (glob(__DIR__ . '/*', GLOB_ONLYDIR) as $dir) {
 			$path = realpath($dir);					
 			$json = $this->normalizeJson(file_get_contents($path . '/expected.json'));
 			

SwaggerGen/Swagger/Type/Property.php

Indentation +6 added lines, -6 removed lines

@@ -48,10 +48,10 @@ 
 	 */
 	private $description;
 
-    /**
-     * Whether property is read only
-     * @var bool
-     */
+	/**
+	 * Whether property is read only
+	 * @var bool
+	 */
 	private $readOnly;
 
 	/**
@@ -65,7 +65,7 @@ 
 	 * @param \SwaggerGen\Swagger\AbstractObject $parent
 	 * @param string $definition Either a built-in type or a definition name
 	 * @param string $description description of the property
-     * @param bool $readOnly Whether the property is read only
+	 * @param bool $readOnly Whether the property is read only
 	 * @throws \SwaggerGen\Exception
 	 */
 	public function __construct(\SwaggerGen\Swagger\AbstractObject $parent, $definition, $description = null, $readOnly = null)
@@ -130,7 +130,7 @@ 
 
 		return self::arrayFilterNull(array_merge($valueType, array(
 					'description' => empty($this->description) ? null : $this->description,
-                    'readOnly' => $this->readOnly
+					'readOnly' => $this->readOnly
 								), parent::toArray()));
 	}
 

SwaggerGen/Swagger/Swagger.php

Indentation +4 added lines, -4 removed lines

@@ -142,9 +142,9 @@ 
 				return $this;
 
 			case 'model':
-            case 'model!':
+			case 'model!':
 			case 'definition':
-            case 'definition!':
+			case 'definition!':
 				$name = self::wordShift($data);
 				if (empty($name)) {
 					throw new \SwaggerGen\Exception('Missing definition name');
@@ -156,8 +156,8 @@ 
 
 				$definition = new Schema($this, $typeDef);
 				if(substr($command, -1) === '!') {
-				    $definition->setReadOnly();
-                }
+					$definition->setReadOnly();
+				}
 				$this->definitions[$name] = $definition;
 				return $definition;
 

Spacing +1 added lines, -1 removed lines

@@ -155,7 +155,7 @@
 				}
 
 				$definition = new Schema($this, $typeDef);
-				if(substr($command, -1) === '!') {
+				if (substr($command, -1) === '!') {
 				    $definition->setReadOnly();
                 }
 				$this->definitions[$name] = $definition;

SwaggerGen/Swagger/Type/AbstractType.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -120,7 +120,7 @@
 		// Internal type if type known and not overwritten by definition
 		if (isset(self::$classTypes[$format])) {
 			$type = self::$classTypes[$format];
-			$class = "SwaggerGen\\Swagger\\Type\\{$type}Type";
+			$class = "SwaggerGen\\Swagger\\Type\\{$type}type";
 			return new $class($parent, $definition);
 		} else {
 			return new ReferenceObjectType($parent, $definition);

SwaggerGen/Swagger/Type/ObjectType.php

Doc Comments +15 added lines

@@ -55,6 +55,10 @@ 
 		$this->parseRange($definition, $match);
 	}
 
+	/**
+	 * @param string $definition
+	 * @param string[] $match
+	 */
 	private function parseFormat($definition, $match)
 	{
 		if (strtolower($match[1]) !== 'object') {
@@ -62,6 +66,10 @@ 
 		}
 	}
 
+	/**
+	 * @param string $definition
+	 * @param string[] $match
+	 */
 	private function parseProperties($definition, $match)
 	{
 		if (!empty($match[2])) {
@@ -80,6 +88,10 @@ 
 		}
 	}
 
+	/**
+	 * @param string $definition
+	 * @param string[] $match
+	 */
 	private function parseRange($definition, $match)
 	{
 		if (!empty($match[3])) {
@@ -100,6 +112,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string|false $discriminator
+	 */
 	private function setDiscriminator($discriminator)
 	{
 		if (!empty($this->discriminator)) {

Indentation +5 added lines, -5 removed lines

@@ -104,13 +104,13 @@ 
 	{
 		if (!empty($this->discriminator)) {
 			throw new \SwaggerGen\Exception("Discriminator may only be set once, "
-			                                . "trying to change it "
-			                                . "from '{$this->discriminator}' "
-			                                . "to '{$discriminator}'");
+											. "trying to change it "
+											. "from '{$this->discriminator}' "
+											. "to '{$discriminator}'");
 		}
 		if (isset($this->properties[$discriminator]) && empty($this->required[$discriminator])) {
 			throw new \SwaggerGen\Exception("Discriminator must be a required property, "
-			                                . "property '{$discriminator}' is not required");
+											. "property '{$discriminator}' is not required");
 		}
 		$this->discriminator = $discriminator;
 	}
@@ -152,7 +152,7 @@ 
 
 				if (($name === $this->discriminator) && !$required) {
 					throw new \SwaggerGen\Exception("Discriminator must be a required property, "
-					                                . "property '{$name}' is not required");
+													. "property '{$name}' is not required");
 				}
 
 				unset($this->required[$name]);