goaop / parser-reflection

src/NodeVisitor/BuiltinTypeFixer.php

Doc Comments +6 added lines

@@ -189,6 +189,9 @@ 
         $node->returnType = $this->fixType($node->returnType, self::RETURN_TYPES);
     }
 
+    /**
+     * @param integer $contextType
+     */
     private function fixType($node, $contextType)
     {
         $typeAsString = $this->getTypeAsString($node);
@@ -232,6 +235,9 @@ 
         return strval($node);
     }
 
+    /**
+     * @param string $typeName
+     */
     private function isTypeDefinedForContext($typeName, $contextType)
     {
         return

Spacing +20 added lines, -20 removed lines

@@ -43,7 +43,7 @@ 
             }
             $numericIndexCount = count(array_filter(
                 $options['supportedBuiltinTypeHints'],
-                (function ($val) {
+                (function($val) {
                     return preg_match('/^0*(0|[1-9]\\d*)$/', $val);
                 }),
                 ARRAY_FILTER_USE_KEY
@@ -53,24 +53,24 @@ 
                 $typeHintName = $key;
                 $validFor     = $value;
                 if (preg_match('/^0*(0|[1-9]\\d*)$/', $key) &&
-                    (intval($key) >= 0)                     &&
-                    (intval($key) <  $numericIndexCount)
+                    (intval($key) >= 0) &&
+                    (intval($key) < $numericIndexCount)
                 ) {
                     $numericIndex = true;
                     $typeHintName = $value;
-                    $validFor     = self::PARAMETER_TYPES|self::RETURN_TYPES;
+                    $validFor     = self::PARAMETER_TYPES | self::RETURN_TYPES;
                 }
                 if (!is_string($typeHintName) || !preg_match('/^\\w(?<!\\d)\\w*$/', $typeHintName)) {
                     throw new \InvalidArgumentException(
                         sprintf(
-                            "Option 'supportedBuiltinTypeHints's element %s " .
+                            "Option 'supportedBuiltinTypeHints's element %s ".
                                 "isn't a valid typehint string.",
                             var_export($typeHintName, true)
                         )
                     );
-                } elseif (!is_scalar($validFor)                                                                   ||
-                    (strval($validFor) != strval(intval($validFor)))                                        ||
-                    (intval($validFor) != (intval($validFor) & (self::PARAMETER_TYPES|self::RETURN_TYPES))) ||
+                } elseif (!is_scalar($validFor) ||
+                    (strval($validFor) != strval(intval($validFor))) ||
+                    (intval($validFor) != (intval($validFor) & (self::PARAMETER_TYPES | self::RETURN_TYPES))) ||
                     (intval($validFor) == 0)
                 ) {
                     throw new \InvalidArgumentException(
@@ -88,7 +88,7 @@ 
                         )
                     );
                 } else {
-                    $this->supportedBuiltinTypeHints[$typeHintName] = intval($validFor) & (self::PARAMETER_TYPES|self::RETURN_TYPES);
+                    $this->supportedBuiltinTypeHints[$typeHintName] = intval($validFor) & (self::PARAMETER_TYPES | self::RETURN_TYPES);
                 }
             }
         } else {
@@ -126,31 +126,31 @@ 
             $builtInTypeNames = [
                 'array'    => [
                     'introduced_version' => 50100,
-                    'valid_for'          => self::PARAMETER_TYPES|self::RETURN_TYPES,
+                    'valid_for'          => self::PARAMETER_TYPES | self::RETURN_TYPES,
                 ],
                 'callable' => [
                     'introduced_version' => 50400,
-                    'valid_for'          => self::PARAMETER_TYPES|self::RETURN_TYPES,
+                    'valid_for'          => self::PARAMETER_TYPES | self::RETURN_TYPES,
                 ],
                 'bool'     => [
                     'introduced_version' => 70000,
-                    'valid_for'          => self::PARAMETER_TYPES|self::RETURN_TYPES,
+                    'valid_for'          => self::PARAMETER_TYPES | self::RETURN_TYPES,
                 ],
                 'float'    => [
                     'introduced_version' => 70000,
-                    'valid_for'          => self::PARAMETER_TYPES|self::RETURN_TYPES,
+                    'valid_for'          => self::PARAMETER_TYPES | self::RETURN_TYPES,
                 ],
                 'int'      => [
                     'introduced_version' => 70000,
-                    'valid_for'          => self::PARAMETER_TYPES|self::RETURN_TYPES,
+                    'valid_for'          => self::PARAMETER_TYPES | self::RETURN_TYPES,
                 ],
                 'string'   => [
                     'introduced_version' => 70000,
-                    'valid_for'          => self::PARAMETER_TYPES|self::RETURN_TYPES,
+                    'valid_for'          => self::PARAMETER_TYPES | self::RETURN_TYPES,
                 ],
                 'iterable' => [
                     'introduced_version' => 70100,
-                    'valid_for'          => self::PARAMETER_TYPES|self::RETURN_TYPES,
+                    'valid_for'          => self::PARAMETER_TYPES | self::RETURN_TYPES,
                 ],
                 'void'     => [
                     'introduced_version' => 70100,
@@ -158,7 +158,7 @@ 
                 ],
                 'object'   => [
                     'introduced_version' => 70200,
-                    'valid_for'          => self::PARAMETER_TYPES|self::RETURN_TYPES,
+                    'valid_for'          => self::PARAMETER_TYPES | self::RETURN_TYPES,
                 ],
             ];
             foreach ($builtInTypeNames as $typeHintName => $valid) {
@@ -172,7 +172,7 @@ 
     public function enterNode(Node $node)
     {
         if (
-            ($node instanceof Stmt\Function_)   ||
+            ($node instanceof Stmt\Function_) ||
             ($node instanceof Stmt\ClassMethod) ||
             ($node instanceof Expr\Closure)
         ) {
@@ -224,10 +224,10 @@ 
         }
         if ($node instanceof Node\NullableType) {
             // This *SHOULD* never be called, but correct behavior shorter than Exception.
-            return '?' . $this->getTypeAsString($node->type);
+            return '?'.$this->getTypeAsString($node->type);
         }
         if ($node instanceof Name) {
-            return ($node->isFullyQualified() ? '\\' : '') . $node->toString();
+            return ($node->isFullyQualified() ? '\\' : '').$node->toString();
         }
         return strval($node);
     }