railt / sdl

src/Compiler/Ast/Dependent/ArgumentDefinitionNode.php

Doc Comments +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
     }
 
     /**
-     * @return null|ValueInterface|NodeInterface
+     * @return null|NodeInterface
      */
     public function getDefaultValue(): ?ValueInterface
     {

src/Compiler/Ast/Dependent/EnumValueDefinitionNode.php

Doc Comments +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
     }
 
     /**
-     * @return null|ValueInterface|NodeInterface
+     * @return null|NodeInterface
      */
     public function getValue(): ?ValueInterface
     {

src/Compiler/Ast/Dependent/InputFieldDefinitionNode.php

Doc Comments +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
     }
 
     /**
-     * @return null|ValueInterface|NodeInterface
+     * @return null|NodeInterface
      */
     public function getDefaultValue(): ?ValueInterface
     {

src/Compiler/Ast/Value/InputValueNode.php

Doc Comments +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
     }
 
     /**
-     * @return iterable|ValueInterface[]
+     * @return \Generator
      */
     public function getValues(): iterable
     {

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
         $result = [];
 
         foreach ($this->getValues() as $key => $value) {
-            $result[] = $key->getValue() . ': ' . $value->toString();
+            $result[] = $key->getValue().': '.$value->toString();
         }
 
         return \sprintf('{%s}', \implode(', ', $result));

src/Compiler/Builder/Common/ValueBuilder.php

Doc Comments +11 added lines, -11 removed lines

@@ -41,7 +41,7 @@ 
     private $type;
 
     /**
-     * @param TypeHint|Definition $type
+     * @param TypeHint $type
      */
     public function __construct(TypeHint $type)
     {
@@ -79,7 +79,7 @@ 
     }
 
     /**
-     * @param TypeHint|TypeDefinition $type
+     * @param TypeHint $type
      * @param ValueInterface|ListValueNode $value
      * @return \Traversable
      * @throws \Railt\Io\Exception\ExternalFileException
@@ -127,8 +127,8 @@ 
     }
 
     /**
-     * @param TypeHint|TypeDefinition $type
-     * @param ValueInterface|RuleInterface $value
+     * @param TypeHint $type
+     * @param ValueInterface $value
      * @return mixed
      * @throws \Railt\Io\Exception\ExternalFileException
      */
@@ -152,8 +152,8 @@ 
     }
 
     /**
-     * @param TypeHint|TypeDefinition $type
-     * @param ValueInterface|RuleInterface|InputValueNode $value
+     * @param TypeHint $type
+     * @param ValueInterface $value
      * @return InputInvocation
      * @throws \Railt\Io\Exception\ExternalFileException
      */
@@ -197,8 +197,8 @@ 
     }
 
     /**
-     * @param TypeHint|TypeDefinition|ConstantValueNode $type
-     * @param ValueInterface|RuleInterface $value
+     * @param TypeHint $type
+     * @param ValueInterface $value
      * @return mixed
      * @throws \Railt\Io\Exception\ExternalFileException
      */
@@ -231,8 +231,8 @@ 
     }
 
     /**
-     * @param TypeHint|TypeDefinition $type
-     * @param ValueInterface|RuleInterface $value
+     * @param TypeHint $type
+     * @param ValueInterface $value
      * @return bool|float|int|mixed|null|string
      * @throws \Railt\Io\Exception\ExternalFileException
      * @throws \Railt\Reflection\Exception\TypeNotFoundException
@@ -251,7 +251,7 @@ 
     }
 
     /**
-     * @param TypeHint|TypeDefinition $type
+     * @param TypeHint $type
      * @param ValueInterface $value
      * @return mixed
      * @throws \Railt\Io\Exception\ExternalFileException

Spacing +5 added lines, -5 removed lines

@@ -102,7 +102,7 @@ 
         /**
          * @validation <name: [Type] = Value>
          */
-        if (! ($value instanceof ListValueNode)) {
+        if (!($value instanceof ListValueNode)) {
             $error = 'Value of %s should be a List, but %s given';
             throw (new TypeConflictException(\sprintf($error, $type, $value->toString())))->throwsIn($type->getFile(),
                 $value->getOffset());
@@ -162,7 +162,7 @@ 
         /**
          * @validation <name: InputType = Value>
          */
-        if (! ($value instanceof InputValueNode)) {
+        if (!($value instanceof InputValueNode)) {
             $error = 'Value of %s should be a %s, but %s given';
             throw (new TypeConflictException(\sprintf($error, $type, $type->getDefinition(),
                 $value->toString())))->throwsIn($type->getFile(), $value->getOffset());
@@ -184,7 +184,7 @@ 
             /**
              * @validation <name: InputType = {nonExistentField: Value}>
              */
-            if (! $definition->hasField($name)) {
+            if (!$definition->hasField($name)) {
                 $error = 'Input field "%s" does not provided by %s, but %s given';
                 throw (new TypeConflictException(\sprintf($error, $name, $type->getDefinition(),
                     $value->toString())))->throwsIn($type->getFile(), $value->getOffset());
@@ -207,7 +207,7 @@ 
         /**
          * @validation <name: Enum = "NotEnumValue">
          */
-        if (! ($value instanceof ConstantValueNode)) {
+        if (!($value instanceof ConstantValueNode)) {
             $error = 'Value of %s can be one of %s value, but %s given';
             throw (new TypeConflictException(\sprintf($error, $type, $type->getDefinition(),
                 $value->toString())))->throwsIn($type->getFile(), $value->getOffset());
@@ -221,7 +221,7 @@ 
         /**
          * @validation <name: Enum = NonExistentValue>
          */
-        if (! $definition->hasValue($name)) {
+        if (!$definition->hasValue($name)) {
             $error = 'Enum %s does not provide value %s';
             throw (new TypeConflictException(\sprintf($error, $type->getDefinition(), $value->toString())))
                 ->throwsIn($type->getFile(), $value->getOffset());

src/Compiler/Builder/Common/ValueTypeResolver.php

Doc Comments +2 added lines, -2 removed lines

@@ -99,7 +99,7 @@ 
 
     /**
      * @param mixed $value
-     * @return iterable|TypeDefinition[]
+     * @return \Generator
      * @throws \Railt\Reflection\Exception\TypeNotFoundException
      */
     public function resolveType($value): iterable
@@ -139,7 +139,7 @@ 
     /**
      * @param TypeDefinition $type
      * @param \Closure $filter
-     * @return \Generator|TypeDefinition[]
+     * @return \Generator
      */
     private function getFilteredChildrenInheritance(TypeDefinition $type, \Closure $filter): \Traversable
     {

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 
 namespace Railt\SDL\Compiler\Builder\Common;
 
-use Railt\Reflection\Contracts\Definition\Behaviour\ProvidesTypeIndication;
 use Railt\Reflection\Contracts\Definition\ScalarDefinition;
 use Railt\Reflection\Contracts\Definition\TypeDefinition;
 use Railt\Reflection\Contracts\Dictionary;

Spacing +3 added lines, -3 removed lines

@@ -122,7 +122,7 @@ 
      */
     private function getFilter(): \Closure
     {
-        return function (TypeDefinition $resolved): bool {
+        return function(TypeDefinition $resolved): bool {
             return $this->shouldBreak($resolved);
         };
     }
@@ -133,7 +133,7 @@ 
      */
     private function shouldBreak(TypeDefinition $resolved): bool
     {
-        return ! isset($this->breakpoints[$resolved->getName()]);
+        return !isset($this->breakpoints[$resolved->getName()]);
     }
 
     /**
@@ -146,7 +146,7 @@ 
         yield $type;
 
         foreach ($type->getChildrenInheritance() as $child) {
-            if (! $filter($child)) {
+            if (!$filter($child)) {
                 continue;
             }
 

src/Compiler/Builder/Invocation/DirectiveBuilder.php

Spacing +2 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
             /** @var DirectiveDefinition $definition */
             $definition = $directive->getDefinition();
 
-            if (! ($definition instanceof Definition\DirectiveDefinition)) {
+            if (!($definition instanceof Definition\DirectiveDefinition)) {
                 $error = '%s should be a Directive, but %s given';
                 throw (new TypeConflictException(\sprintf($error, $directive, $definition)))
                     ->throwsIn($directive->getFile(), $rule->getOffset());
@@ -47,7 +47,7 @@ 
             foreach ($rule->getDirectiveArguments() as $ast) {
                 $name = $ast->getArgumentName();
 
-                if (! $definition->hasArgument($name)) {
+                if (!$definition->hasArgument($name)) {
                     $error = 'Directive %s does not provide argument %s';
                     throw (new TypeConflictException(\sprintf($error, $definition, $name)))
                         ->throwsIn($directive->getFile(), $ast->getOffset());

src/Compiler/Builder/Definition/ScalarBuilder.php

Spacing +3 added lines, -3 removed lines

@@ -32,11 +32,11 @@ 
         $scalar->withOffset($rule->getOffset());
         $scalar->withDescription($rule->getDescription());
 
-        $this->when->resolving(function () use ($rule, $scalar) {
+        $this->when->resolving(function() use ($rule, $scalar) {
             if ($ast = $rule->getExtends()) {
                 $parent = $this->load($ast->getTypeName(), $scalar);
 
-                if (! ($parent instanceof Definition\ScalarDefinition)) {
+                if (!($parent instanceof Definition\ScalarDefinition)) {
                     $error = '%s can extends only Scalar type, but %s given';
                     throw (new TypeConflictException(\sprintf($error, $scalar, $parent)))
                         ->throwsIn($scalar->getFile(), $rule->getOffset());
@@ -46,7 +46,7 @@ 
             }
         });
 
-        $this->when->runtime(function () use ($rule, $scalar) {
+        $this->when->runtime(function() use ($rule, $scalar) {
             foreach ($rule->getDirectives() as $ast) {
                 $scalar->withDirective($this->dependent($ast, $scalar));
             }

src/Compiler/Builder/Definition/InterfaceBuilder.php

Spacing +2 added lines, -2 removed lines

@@ -37,13 +37,13 @@
             $interface->withField($this->dependent($ast, $interface));
         }
 
-        $this->when->runtime(function () use ($rule, $interface) {
+        $this->when->runtime(function() use ($rule, $interface) {
             foreach ($rule->getDirectives() as $ast) {
                 $interface->withDirective($this->dependent($ast, $interface));
             }
         });
 
-        $this->when->resolving(function () use ($rule, $interface) {
+        $this->when->resolving(function() use ($rule, $interface) {
             foreach ($rule->getImplementations() as $child) {
                 $interface->withInterface($child->getTypeName());
             }