gossi / php-code-generator

src/generator/builder/parts/TypeBuilderPart.php

Doc Comments +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
 
 	/**
 	 *
-	 * @param AbstractModel|TypePart $model
+	 * @param AbstractModel $model
 	 * @param bool $allowed
 	 *
 	 * @return string|null

Indentation +6 added lines, -6 removed lines

@@ -12,7 +12,7 @@ 
 		'string', 'int', 'integer', 'bool', 'boolean', 'float', 'double', 'object', 'mixed', 'resource'
 	];
 
-    protected static $php7typeHints = [
+	protected static $php7typeHints = [
 		'string', 'int', 'integer', 'bool', 'boolean', 'float', 'double'
 	];
 
@@ -36,15 +36,15 @@ 
 	private function getType(AbstractModel $model, bool $allowed, bool $nullable): ?string {
 		$types = $model->getTypes();
 		if (!$types || $types->size() !== 1) {
-		    return null;
-        }
+			return null;
+		}
 		$type = (string)$types->values()->toArray()[0];
 		if (!in_array($type, self::$noTypeHints, true)
-            || ($allowed && in_array($type, self::$php7typeHints, true))) {
+			|| ($allowed && in_array($type, self::$php7typeHints, true))) {
 
-            $type = self::$typeHintMap[$type] ?? $type;
+			$type = self::$typeHintMap[$type] ?? $type;
 
-            if ($nullable && $model->getNullable()) {
+			if ($nullable && $model->getNullable()) {
 				$type = '?' . $type;
 			}
 

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
 		if (!$types || $types->size() !== 1) {
 		    return null;
         }
-		$type = (string)$types->values()->toArray()[0];
+		$type = (string) $types->values()->toArray()[0];
 		if (!in_array($type, self::$noTypeHints, true)
             || ($allowed && in_array($type, self::$php7typeHints, true))) {
 

src/model/PhpClass.php

Doc Comments -1 removed lines

@@ -79,7 +79,6 @@
 	/**
 	 * Sets the parent class name
 	 *
-	 * @param PhpClass|string|null $name the new parent
 	 * @return $this
 	 */
 	public function setParentClassName($parent) {

Indentation +9 added lines, -9 removed lines

@@ -71,10 +71,10 @@ 
 		return $this->parentClassName;
 	}
 
-    public function getParentClass(): PhpClass {
-	    return class_exists($this->parentClassName) ?
-            self::fromName($this->parentClassName) : PhpClass::create($this->parentClassName);
-    }
+	public function getParentClass(): PhpClass {
+		return class_exists($this->parentClassName) ?
+			self::fromName($this->parentClassName) : PhpClass::create($this->parentClassName);
+	}
 
 	/**
 	 * Sets the parent class name
@@ -84,12 +84,12 @@ 
 	 */
 	public function setParentClassName($parent) {
 		if ($parent instanceof PhpClass) {
-		    $this->addUseStatement($parent->getQualifiedName());
-		    $parent = $parent->getName();
-        }
-        $this->parentClassName = $parent;
+			$this->addUseStatement($parent->getQualifiedName());
+			$parent = $parent->getName();
+		}
+		$this->parentClassName = $parent;
 
-        return $this;
+		return $this;
 	}
 
 	public function generateDocblock(): void {

src/model/RoutineInterface.php

Doc Comments +3 added lines, -3 removed lines

@@ -35,7 +35,7 @@ 
 	 * A quick way to add a parameter which is created from the given parameters
 	 *
 	 * @param string      $name
-	 * @param string[]|PhpTypeInterface[] $types
+	 * @param string[]|PhpTypeInterface[] $type
 	 * @param mixed       $defaultValue omit the argument to define no default value
 	 *
 	 * @return $this
@@ -46,7 +46,7 @@ 
 	 * A quick way to add a parameter with description which is created from the given parameters
 	 *
 	 * @param string      $name
-	 * @param string[]|PhpTypeInterface[] $types
+	 * @param string[]|PhpTypeInterface[] $type
 	 * @param null|string $typeDescription
 	 * @param mixed       $defaultValue omit the argument to define no default value
 	 *
@@ -84,7 +84,7 @@ 
 	/**
 	 * Returns a collection of parameters
 	 *
-	 * @return array
+	 * @return PhpParameter[]
 	 */
 	public function getParameters();
 

src/utils/TypeUtils.php

Indentation +59 added lines, -59 removed lines

@@ -7,75 +7,75 @@
 
 class TypeUtils
 {
-    public static $phpTypes = [
-        'string',
-        'int',
-        'integer',
-        'bool',
-        'boolean',
-        'float',
-        'double',
-        'object',
-        'mixed',
-        'resource',
-        'iterable',
-        'array',
-        'callable',
-    ];
+	public static $phpTypes = [
+		'string',
+		'int',
+		'integer',
+		'bool',
+		'boolean',
+		'float',
+		'double',
+		'object',
+		'mixed',
+		'resource',
+		'iterable',
+		'array',
+		'callable',
+	];
 
-    public static function expressionToTypes(?string $typeExpression): array {
-        if (!$typeExpression) {
-            return [];
-        }
+	public static function expressionToTypes(?string $typeExpression): array {
+		if (!$typeExpression) {
+			return [];
+		}
 
-        return explode('|', $typeExpression);
-    }
+		return explode('|', $typeExpression);
+	}
 
-    public static function guessQualifiedName(AbstractPhpStruct $stuct, string $type): string {
-        if (in_array($type, self::$phpTypes, true)) {
-            return $type;
-        }
+	public static function guessQualifiedName(AbstractPhpStruct $stuct, string $type): string {
+		if (in_array($type, self::$phpTypes, true)) {
+			return $type;
+		}
 
-        $suffix = '';
-        if (strpos($type, '[]') !== false) {
-            $type = str_replace('[]', '', $type);
-            $suffix = '[]';
-        }
+		$suffix = '';
+		if (strpos($type, '[]') !== false) {
+			$type = str_replace('[]', '', $type);
+			$suffix = '[]';
+		}
 
-        $uses = $stuct->getUseStatements();
-        foreach ($uses as $use) {
-            $regexp = '/\\\\' . preg_quote($type, '/') . '$/';
-            if (preg_match($regexp, $use)) {
-                return $use . $suffix;
-            }
-        }
+		$uses = $stuct->getUseStatements();
+		foreach ($uses as $use) {
+			$regexp = '/\\\\' . preg_quote($type, '/') . '$/';
+			if (preg_match($regexp, $use)) {
+				return $use . $suffix;
+			}
+		}
 
-        $sameNamespace = $stuct->getNamespace() . '\\' . $type;
-        if (class_exists($sameNamespace)) {
-            return $sameNamespace . $suffix;
-        }
+		$sameNamespace = $stuct->getNamespace() . '\\' . $type;
+		if (class_exists($sameNamespace)) {
+			return $sameNamespace . $suffix;
+		}
 
-        return $type . $suffix;
-    }
+		return $type . $suffix;
+	}
 
-    public static function isGlobalQualifiedName(string $name): bool {
-        return $name[0] === '\\' && substr_count($name, '\\') === 1;
-    }
+	public static function isGlobalQualifiedName(string $name): bool {
+		return $name[0] === '\\' && substr_count($name, '\\') === 1;
+	}
 
-    public static function isNativeType(string $type): bool {
-        return in_array($type, self::$phpTypes, true);
-    }
+	public static function isNativeType(string $type): bool {
+		return in_array($type, self::$phpTypes, true);
+	}
 
-    public static function typesToExpression(iterable $types): ?string {
-        $typeExpr = '';
-        foreach ($types as $type) {
-            $typeExpr .= '|' . $type;
-        }
+	public static function typesToExpression(iterable $types): ?string {
+		$typeExpr = '';
+		foreach ($types as $type) {
+			$typeExpr .= '|' . $type;
+		}
 
-        if (!$typeExpr) {
-            return null;
-        }
+		if (!$typeExpr) {
+			return null;
+		}
 
-        return trim($typeExpr, '|');
-    }
+		return trim($typeExpr, '|');
+	}
 }

Braces +1 added lines, -2 removed lines

@@ -5,8 +5,7 @@
 
 use gossi\codegen\model\AbstractPhpStruct;
 
-class TypeUtils
-{
+class TypeUtils {
     public static $phpTypes = [
         'string',
         'int',

src/model/AbstractPhpStruct.php

Indentation +32 added lines, -32 removed lines

@@ -57,12 +57,12 @@ 
 		return new static($name);
 	}
 
-    public static function fromName(string $name): self
-    {
-        $ref = new \ReflectionClass($name);
+	public static function fromName(string $name): self
+	{
+		$ref = new \ReflectionClass($name);
 
-        return static::fromFile($ref->getFileName());
-    }
+		return static::fromFile($ref->getFileName());
+	}
 
 	/**
 	 * Creates a new struct
@@ -135,17 +135,17 @@ 
 	 * @return $this
 	 */
 	public function addUseStatement($qualifiedName, string $alias = null) {
-        if ($qualifiedName instanceof PhpTypeInterface) {
-            $qualifiedName = $qualifiedName->getQualifiedName();
-        }
+		if ($qualifiedName instanceof PhpTypeInterface) {
+			$qualifiedName = $qualifiedName->getQualifiedName();
+		}
 
-        if (TypeUtils::isGlobalQualifiedName($qualifiedName) || TypeUtils::isNativeType($qualifiedName)) {
-            return $this;
-        }
+		if (TypeUtils::isGlobalQualifiedName($qualifiedName) || TypeUtils::isNativeType($qualifiedName)) {
+			return $this;
+		}
 
-	    if ($qualifiedName === $this->getNamespace()) {
-	        return $this;
-        }
+		if ($qualifiedName === $this->getNamespace()) {
+			return $this;
+		}
 		if (!is_string($alias)) {
 			if (false === strpos($qualifiedName, '\\')) {
 				$alias = $qualifiedName;
@@ -271,24 +271,24 @@ 
 		$method->setParent($this);
 		$this->methods->set($method->getName(), $method);
 		$types = $method->getTypes();
-        if ($types) {
-            foreach ($types as $type) {
-                $this->addUseStatement($type);
-                $method->addType($type);
-            }
-        }
-
-        foreach ($method->getParameters() as $parameter) {
-            $types = $parameter->getTypes();
-            if ($types) {
-                foreach ($types as $type) {
-                    $this->addUseStatement($type);
-                    $parameter->addType($type);
-                }
-            }
-        }
-
-        return $this;
+		if ($types) {
+			foreach ($types as $type) {
+				$this->addUseStatement($type);
+				$method->addType($type);
+			}
+		}
+
+		foreach ($method->getParameters() as $parameter) {
+			$types = $parameter->getTypes();
+			if ($types) {
+				foreach ($types as $type) {
+					$this->addUseStatement($type);
+					$parameter->addType($type);
+				}
+			}
+		}
+
+		return $this;
 	}
 
 	/**

src/model/PhpFunction.php

Indentation +2 added lines, -2 removed lines

@@ -93,8 +93,8 @@
 		$this->setQualifiedName($name);
 		$this->docblock = new Docblock();
 		$this->initParameters();
-        $this->initTypes();
-    }
+		$this->initTypes();
+	}
 
 	/**
 	 * @inheritDoc

src/model/AbstractModel.php

Indentation +2 added lines, -2 removed lines

@@ -33,8 +33,8 @@
 			$description = implode("\n", $description);
 		}
 		if ($description) {
-            $this->description = $description;
-        }
+			$this->description = $description;
+		}
 		return $this;
 	}
 

src/model/PhpTypeInterface.php

Braces +1 added lines, -2 removed lines

@@ -2,8 +2,7 @@
 
 namespace gossi\codegen\model;
 
-interface PhpTypeInterface extends NamespaceInterface
-{
+interface PhpTypeInterface extends NamespaceInterface {
     public function getName(): ?string;
 
     public function getQualifiedName(): ?string;

Doc Comments +12 added lines

@@ -4,12 +4,24 @@
 
 interface PhpTypeInterface extends NamespaceInterface
 {
+    /**
+     * @return string
+     */
     public function getName(): ?string;
 
+    /**
+     * @return string
+     */
     public function getQualifiedName(): ?string;
 
+    /**
+     * @return parts\NamePart
+     */
     public function setName(?string $name);
 
+    /**
+     * @return parts\QualifiedNamePart
+     */
     public function setQualifiedName(?string $qualifiedName);
 
     public function __toString(): string;

Indentation +5 added lines, -5 removed lines

@@ -4,13 +4,13 @@
 
 interface PhpTypeInterface extends NamespaceInterface
 {
-    public function getName(): ?string;
+	public function getName(): ?string;
 
-    public function getQualifiedName(): ?string;
+	public function getQualifiedName(): ?string;
 
-    public function setName(?string $name);
+	public function setName(?string $name);
 
-    public function setQualifiedName(?string $qualifiedName);
+	public function setQualifiedName(?string $qualifiedName);
 
-    public function __toString(): string;
+	public function __toString(): string;
 }

src/model/PhpParameter.php

Indentation +12 added lines, -12 removed lines

@@ -55,8 +55,8 @@ 
 	 */
 	public function __construct($name = null) {
 		$this->setName($name);
-        $this->initTypes();
-    }
+		$this->initTypes();
+	}
 
 	/**
 	 * Sets whether this parameter is passed by reference
@@ -85,16 +85,16 @@ 
 	 * @return ParamTag
 	 */
 	public function getDocblockTag(): ParamTag {
-        $type = '';
-        if ($this->getNullable()) {
-            $type = 'null';
-        }
-        if ($this->getTypes()) {
-            if ($type) {
-                $type .= '|';
-            }
-            $type .= TypeUtils::typesToExpression($this->getTypes());
-        }
+		$type = '';
+		if ($this->getNullable()) {
+			$type = 'null';
+		}
+		if ($this->getTypes()) {
+			if ($type) {
+				$type .= '|';
+			}
+			$type .= TypeUtils::typesToExpression($this->getTypes());
+		}
 		return ParamTag::create()
 			->setType($type)
 			->setVariable($this->getName())