railt / sdl

src/Compiler/Builder/Common/ValueBuilder.php

Unused Use Statements -1 removed lines

@@ -24,7 +24,6 @@
 use Railt\SDL\Compiler\Ast\Value\NullValueNode;
 use Railt\SDL\Compiler\Ast\Value\ValueInterface;
 use Railt\SDL\Compiler\Ast\Value\ValueNode;
-use Railt\SDL\Exception\SemanticException;
 use Railt\SDL\Exception\TypeConflictException;
 
 /**

Spacing +5 added lines, -5 removed lines

@@ -104,7 +104,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());
@@ -164,7 +164,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());
@@ -186,7 +186,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());
@@ -209,7 +209,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());
@@ -223,7 +223,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/ValueInvocation.php

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 use Railt\Reflection\Contracts\Definition;
 use Railt\Reflection\Contracts\Definition\InputDefinition;
 use Railt\Reflection\Contracts\Invocation\InputInvocation;
-use Railt\Reflection\Contracts\Invocation\TypeInvocation;
 use Railt\SDL\Exception\SemanticException;
 use Railt\SDL\Exception\TypeConflictException;
 

Spacing +2 added lines, -2 removed lines

@@ -77,7 +77,7 @@ 
         $stack = clone $this->stack;
 
         foreach ($stack as $i) {
-            if ((string)$i === (string)$definition) {
+            if ((string) $i === (string) $definition) {
                 return true;
             }
         }
@@ -115,7 +115,7 @@ 
         foreach ($definition->getFields() as $field) {
             $argument = $input->getArgument($field->getName());
 
-            if ($argument === null && $field->isNonNull() && ! $field->hasDefaultValue()) {
+            if ($argument === null && $field->isNonNull() && !$field->hasDefaultValue()) {
                 $error = \sprintf('Missing value for required argument %s', $field);
                 throw (new TypeConflictException($error))->throwsIn($input->getFile(), $input->getLine(),
                         $input->getColumn());

src/Compiler/Builder/Common/ValueTypeResolver.php

Doc Comments +2 added lines, -2 removed lines

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

Spacing +3 added lines, -3 removed lines

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

src/Compiler/Builder/Definition/DirectiveBuilder.php

Spacing +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
             $location = new DirectiveLocation($directive, $name);
             $location->withOffset($ast->getOffset());
 
-            if (! \in_array($name, $locations, true)) {
+            if (!\in_array($name, $locations, true)) {
                 $error = \sprintf('Invalid directive location %s, only one of {%s} allowed',
                     $location, \implode(', ', $locations));
                 throw (new TypeConflictException($error))->throwsIn($directive->getFile(), $ast->getOffset());

src/Compiler/Builder/Invocation/DirectiveBuilder.php

Spacing +3 added lines, -3 removed lines

@@ -33,7 +33,7 @@ 
         $directive = new Invocation($parent->getDocument(), $rule->getDirectiveName());
         $directive->withOffset($rule->getOffset());
 
-        $this->when->resolving(function () use ($rule, $directive): void {
+        $this->when->resolving(function() use ($rule, $directive): void {
             $definition = $this->loadDefinition($directive, $rule);
 
             $this->buildArguments($directive, $definition, $rule);
@@ -55,7 +55,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());
@@ -101,7 +101,7 @@ 
                 continue;
             }
 
-            if (! $argument->hasDefaultValue()) {
+            if (!$argument->hasDefaultValue()) {
                 $error = 'Missing value for required argument %s of %s';
                 throw (new TypeConflictException(\sprintf($error, $argument,
                     $directive)))->throwsIn($directive->getFile(), $directive->getLine(), $directive->getColumn());

src/Compiler/Builder/Dependent/EnumValueBuilder.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
         $value->withOffset($rule->getOffset());
         $value->withDescription($rule->getDescription());
 
-        $this->when->runtime(function () use ($rule, $value): void {
+        $this->when->runtime(function() use ($rule, $value): void {
             if ($hint = $rule->getTypeHint()) {
                 $virtualTypeHint = $this->virtualTypeHint($value, $hint);
 

Braces +2 added lines, -1 removed lines

@@ -68,7 +68,8 @@
      */
     private function virtualTypeHint(EnumValueDefinition $value, TypeHintNode $ast): TypeHint
     {
-        $virtual = new class($value->getDocument(), $value->getName()) extends TypeHint {
+        $virtual = new class($value->getDocument(), $value->getName()) extends TypeHint
+        {
             /**
              * @var string
              */

src/Compiler/Builder/Dependent/FieldBuilder.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
             ]);
         });
 
-        $this->when->runtime(function () use ($rule, $field): void {
+        $this->when->runtime(function() use ($rule, $field): void {
             foreach ($rule->getDirectives() as $ast) {
                 $field->withDirective($this->dependent($ast, $field));
             }

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

@@ -44,7 +44,7 @@
             $this->shouldBeTypeOf($field, $field->getDefinition(), [
                 Type::SCALAR,
                 Type::OBJECT,
-                Type::INTERFACE,
+                Type::interface,
                 Type::UNION,
                 Type::ENUM,
                 Type::ANY,

src/Compiler/Builder/Dependent/InputFieldBuilder.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
             ]);
         });
 
-        $this->when->runtime(function () use ($rule, $field): void {
+        $this->when->runtime(function() use ($rule, $field): void {
             if ($default = $rule->getDefaultValue()) {
                 $field->withDefaultValue($this->valueOf($field, $default));
             }

src/Compiler/Builder/Dependent/ArgumentBuilder.php

Spacing +2 added lines, -2 removed lines

@@ -44,10 +44,10 @@
             ]);
         });
 
-        $this->when->runtime(function () use ($rule, $argument): void {
+        $this->when->runtime(function() use ($rule, $argument): void {
             if ($default = $rule->getDefaultValue()) {
                 $argument->withDefaultValue($this->valueOf($argument, $default));
-            } elseif (! $argument->isNonNull()) {
+            } elseif (!$argument->isNonNull()) {
                 $argument->withDefaultValue(null);
             }