gossi / php-code-generator

src/utils/GeneratorUtils.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
 
 	public static function callMethod(\ReflectionMethod $method, array $params = null) {
 		if (null === $params) {
-			$params = array_map(function ($p) {
+			$params = array_map(function($p) {
 				return '$' . $p->name;
 			}, $method->getParameters());
 		}

src/utils/ReflectionUtils.php

Spacing +2 added lines, -2 removed lines

@@ -32,7 +32,7 @@ 
 			$filter |= \ReflectionMethod::IS_PROTECTED;
 		}
 		
-		return array_filter($class->getMethods($filter), function ($method) {
+		return array_filter($class->getMethods($filter), function($method) {
 			return !$method->isFinal() && !$method->isStatic();
 		});
 	}
@@ -81,7 +81,7 @@ 
 		}
 		$tokenizer = new PhpTokenizer();
 		$tokens = $tokenizer->tokenize($content);
-		$tokens = $tokens->filter(function ($token) {
+		$tokens = $tokens->filter(function($token) {
 			return $token->type !== T_WHITESPACE && $token->type !== T_COMMENT && $token->type !== T_DOC_COMMENT;
 		});
 		$statements = [];

src/model/AbstractPhpStruct.php

Indentation +5 added lines, -5 removed lines

@@ -131,11 +131,11 @@
 	 * 
 	 * If the class has already been declared you get only the set alias.
 	 * 
- 	 * @param string      $fullClassName
- 	 * @param null|string $alias
- 	 *
- 	 * @return string
- 	 */
+	 * @param string      $fullClassName
+	 * @param null|string $alias
+	 *
+	 * @return string
+	 */
  	public function declareUse($fullClassName, $alias = null)
 	{
  		$fullClassName = trim($fullClassName, '\\');

Braces +2 added lines, -4 removed lines

@@ -117,8 +117,7 @@ 
 	/**
 	 *  Declares multiple use statements at once.
 	 */
-	public function declareUses()
- 	{
+	public function declareUses() {
  		foreach (func_get_args() as $name) {
  			$this->declareUse($name);
  		}
@@ -136,8 +135,7 @@ 
  	 *
  	 * @return string
  	 */
- 	public function declareUse($fullClassName, $alias = null)
-	{
+ 	public function declareUse($fullClassName, $alias = null) {
  		$fullClassName = trim($fullClassName, '\\');
  		if (!$this->hasUseStatement($fullClassName)) {
  			$this->addUseStatement($fullClassName, $alias);

src/model/PhpParameter.php

Spacing +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
 	 * @return ParamTag
 	 */
 	public function getDocblockTag() {
-		return ParamTag::create()->setType($this->getType() ?  : 'mixed')->setVariable($this->getName())->setDescription($this->getTypeDescription());
+		return ParamTag::create()->setType($this->getType() ?: 'mixed')->setVariable($this->getName())->setDescription($this->getTypeDescription());
 	}
 
 	/**

src/model/parts/ConstantsTrait.php

Spacing +1 added lines, -1 removed lines

@@ -92,7 +92,7 @@
 			return $this->constants;
 		}
 		
-		return array_map(function (PhpConstant $constant) {
+		return array_map(function(PhpConstant $constant) {
 			return $constant->getValue();
 		}, $this->constants);
 	}

src/visitor/DefaultNavigator.php

Spacing +3 added lines, -3 removed lines

@@ -143,7 +143,7 @@ 
 	}
 
 	private function getConstantSortFunc() {
-		return $this->constantSortFunc ?  : 'strcasecmp';
+		return $this->constantSortFunc ?: 'strcasecmp';
 	}
 
 	private function getMethodSortFunc() {
@@ -152,7 +152,7 @@ 
 		}
 		
 		if (empty(self::$defaultMethodSortFunc)) {
-			self::$defaultMethodSortFunc = function ($a, $b) {
+			self::$defaultMethodSortFunc = function($a, $b) {
 				if ($a->isStatic() !== $isStatic = $b->isStatic()) {
 					return $isStatic ? 1 : -1;
 				}
@@ -177,7 +177,7 @@ 
 		}
 		
 		if (empty(self::$defaultPropertySortFunc)) {
-			self::$defaultPropertySortFunc = function ($a, $b) {
+			self::$defaultPropertySortFunc = function($a, $b) {
 				if (($aV = $a->getVisibility()) !== $bV = $b->getVisibility()) {
 					$aV = 'public' === $aV ? 3 : ('protected' === $aV ? 2 : 1);
 					$bV = 'public' === $bV ? 3 : ('protected' === $bV ? 2 : 1);

src/visitor/DefaultVisitor.php

Switch Indentation +8 added lines, -8 removed lines

@@ -343,14 +343,14 @@
 				$defaultValue = $parameter->getDefaultValue();
 
 				switch (true) {
-					case is_array($defaultValue) && empty($defaultValue):
-						$this->writer->write('array()');
-						break;
-					case ($defaultValue instanceof PhpConstant):
-						$this->writer->write($defaultValue->getName());
-						break;
-					default:
-					$this->writer->write($this->getPhpExport($defaultValue));
+				case is_array($defaultValue) && empty($defaultValue):
+					$this->writer->write('array()');
+					break;
+				case ($defaultValue instanceof PhpConstant):
+					$this->writer->write($defaultValue->getName());
+					break;
+				default:
+				$this->writer->write($this->getPhpExport($defaultValue));
 
 				}
 			}

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
 
 	public function __construct(CodeGeneratorConfig $config = null) {
 		// Make sure we retain the old default behavior for this class
-		$this->config = $config?: new CodeGeneratorConfig(['generateEmptyDocblock' => false]);
+		$this->config = $config ?: new CodeGeneratorConfig(['generateEmptyDocblock' => false]);
 		$this->writer = new Writer();
 	}
 

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

@@ -358,7 +358,7 @@
 	}
 
 	protected function writeFunctionReturnType($type) {
-		if ($this->config->getGenerateReturnTypeHints() && $type != NULL && false === strpos($type, '|')) {
+		if ($this->config->getGenerateReturnTypeHints() && $type != null && false === strpos($type, '|')) {
 			$this->writer->write(': ')->write($type);
 		}
 	}

src/parser/visitor/AbstractPhpStructVisitor.php

Unused Use Statements -1 removed lines

@@ -19,7 +19,6 @@
 use PhpParser\Node\Stmt\Property;
 use PhpParser\Node\Stmt\TraitUse;
 use PhpParser\Node\Stmt\UseUse;
-use PhpParser\NodeTraverser;
 use PhpParser\NodeVisitorAbstract;
 use PhpParser\PrettyPrinter\Standard;
 use PhpParser\Node\Stmt\Class_;

Switch Indentation +30 added lines, -30 removed lines

@@ -48,44 +48,44 @@
 	
 	public function enterNode(Node $node) {
 		switch ($node->getType()) {
-			case 'Stmt_Namespace':
-				$this->visitNamespace($node);
-				break;
+		case 'Stmt_Namespace':
+			$this->visitNamespace($node);
+			break;
 				
-			case 'Stmt_UseUse':
-				$this->visitUseStatement($node);
-				break;
+		case 'Stmt_UseUse':
+			$this->visitUseStatement($node);
+			break;
 				
-			case 'Stmt_Class':
-				$this->visitStruct($node);
-				$this->visitClass($node);
-				break;
+		case 'Stmt_Class':
+			$this->visitStruct($node);
+			$this->visitClass($node);
+			break;
 				
-			case 'Stmt_Interface':
-				$this->visitStruct($node);
-				$this->visitInterface($node);
-				break;
+		case 'Stmt_Interface':
+			$this->visitStruct($node);
+			$this->visitInterface($node);
+			break;
 				
-			case 'Stmt_Trait':
-				$this->visitStruct($node);
-				$this->visitTrait($node);
-				break;
+		case 'Stmt_Trait':
+			$this->visitStruct($node);
+			$this->visitTrait($node);
+			break;
 				
-			case 'Stmt_TraitUse':
-				$this->visitTraitUse($node);
-				break;
+		case 'Stmt_TraitUse':
+			$this->visitTraitUse($node);
+			break;
 				
-			case 'Const':
-				$this->visitConstant($node);
-				break;
+		case 'Const':
+			$this->visitConstant($node);
+			break;
 				
-			case 'Stmt_Property':
-				$this->visitProperty($node);
-				break;
+		case 'Stmt_Property':
+			$this->visitProperty($node);
+			break;
 				
-			case 'Stmt_ClassMethod':
-				$this->visitMethod($node);
-				break;
+		case 'Stmt_ClassMethod':
+			$this->visitMethod($node);
+			break;
 		}
 // 		echo $node->getType() . "\n";