railt / sdl

src/Reflection/Validation/Inheritance/ScalarValidator.php

Doc Comments +12 added lines, -12 removed lines

@@ -20,8 +20,8 @@ 
 class ScalarValidator extends BaseInheritanceValidator
 {
     /**
-     * @param AllowsTypeIndication|TypeDefinition $child
-     * @param AllowsTypeIndication|TypeDefinition $parent
+     * @param TypeDefinition $child
+     * @param TypeDefinition $parent
      * @return bool
      */
     public function match(TypeDefinition $child, TypeDefinition $parent): bool
@@ -31,8 +31,8 @@ 
     }
 
     /**
-     * @param AllowsTypeIndication|TypeDefinition $child
-     * @param AllowsTypeIndication|TypeDefinition $parent
+     * @param TypeDefinition $child
+     * @param TypeDefinition $parent
      * @return void
      * @throws TypeConflictException
      */
@@ -46,8 +46,8 @@ 
     }
 
     /**
-     * @param AllowsTypeIndication|TypeDefinition $child
-     * @param AllowsTypeIndication|TypeDefinition $parent
+     * @param TypeDefinition $child
+     * @param TypeDefinition $parent
      * @return void
      * @throws TypeConflictException
      */
@@ -60,8 +60,8 @@ 
     }
 
     /**
-     * @param AllowsTypeIndication|TypeDefinition $child
-     * @param AllowsTypeIndication|TypeDefinition $parent
+     * @param TypeDefinition $child
+     * @param TypeDefinition $parent
      * @return void
      * @throws TypeConflictException
      */
@@ -75,8 +75,8 @@ 
     }
 
     /**
-     * @param AllowsTypeIndication|TypeDefinition $child
-     * @param AllowsTypeIndication|TypeDefinition $parent
+     * @param TypeDefinition $child
+     * @param TypeDefinition $parent
      * @return void
      * @throws TypeConflictException
      */
@@ -90,8 +90,8 @@ 
     }
 
     /**
-     * @param AllowsTypeIndication|TypeDefinition $child
-     * @param AllowsTypeIndication|TypeDefinition $parent
+     * @param TypeDefinition $child
+     * @param TypeDefinition $parent
      * @return void
      * @throws TypeConflictException
      */

Spacing +2 added lines, -2 removed lines

@@ -69,7 +69,7 @@ 
     {
         $parentType = $parent->getTypeDefinition();
 
-        if (! ($child->getTypeDefinition() instanceof $parentType)) {
+        if (!($child->getTypeDefinition() instanceof $parentType)) {
             $this->throwScalarIncompatibilityException($child, $parent);
         }
     }
@@ -84,7 +84,7 @@ 
     {
         $childType = $child->getTypeDefinition();
 
-        if (! ($parent->getTypeDefinition() instanceof $childType)) {
+        if (!($parent->getTypeDefinition() instanceof $childType)) {
             $this->throwScalarIncompatibilityException($child, $parent);
         }
     }

src/Reflection/Validation/Inheritance/UnionValidator.php

Doc Comments +6 added lines, -6 removed lines

@@ -21,8 +21,8 @@ 
 class UnionValidator extends BaseInheritanceValidator
 {
     /**
-     * @param AllowsTypeIndication|TypeDefinition $child
-     * @param AllowsTypeIndication|TypeDefinition $parent
+     * @param TypeDefinition $child
+     * @param TypeDefinition $parent
      * @return bool
      */
     public function match(TypeDefinition $child, TypeDefinition $parent): bool
@@ -32,8 +32,8 @@ 
     }
 
     /**
-     * @param AllowsTypeIndication|TypeDefinition $child
-     * @param AllowsTypeIndication|TypeDefinition $parent
+     * @param TypeDefinition $child
+     * @param TypeDefinition $parent
      * @return void
      * @throws TypeConflictException
      */
@@ -49,8 +49,8 @@ 
     }
 
     /**
-     * @param AllowsTypeIndication|TypeDefinition $child
-     * @param AllowsTypeIndication|TypeDefinition $parent
+     * @param TypeDefinition $child
+     * @param TypeDefinition $parent
      * @return bool
      */
     private function isUnionContainsChild(TypeDefinition $child, TypeDefinition $parent): bool

Spacing +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
      */
     private function validateUnionContainsChild(TypeDefinition $child, TypeDefinition $parent): void
     {
-        if (! $this->isUnionContainsChild($child, $parent)) {
+        if (!$this->isUnionContainsChild($child, $parent)) {
             $this->throwIncompatibleTypes($child, $parent);
         }
     }

src/Reflection/Validation/Inheritance/WrapperValidator.php

Doc Comments +8 added lines, -8 removed lines

@@ -20,8 +20,8 @@ 
 class WrapperValidator extends BaseInheritanceValidator
 {
     /**
-     * @param AllowsTypeIndication|TypeDefinition $child
-     * @param AllowsTypeIndication|TypeDefinition $parent
+     * @param TypeDefinition $child
+     * @param TypeDefinition $parent
      * @return bool
      */
     public function match(TypeDefinition $child, TypeDefinition $parent): bool
@@ -31,8 +31,8 @@ 
     }
 
     /**
-     * @param AllowsTypeIndication|TypeDefinition $child
-     * @param AllowsTypeIndication|TypeDefinition $parent
+     * @param TypeDefinition $child
+     * @param TypeDefinition $parent
      * @return void
      * @throws \Railt\SDL\Exceptions\TypeConflictException
      * @throws \OutOfBoundsException
@@ -124,8 +124,8 @@ 
     }
 
     /**
-     * @param AllowsTypeIndication|DependentDefinition $child
-     * @param AllowsTypeIndication|DependentDefinition $parent
+     * @param AllowsTypeIndication $child
+     * @param AllowsTypeIndication $parent
      * @return void
      * @throws TypeConflictException
      */
@@ -145,8 +145,8 @@ 
     }
 
     /**
-     * @param AllowsTypeIndication|DependentDefinition $child
-     * @param AllowsTypeIndication|DependentDefinition $parent
+     * @param AllowsTypeIndication $child
+     * @param AllowsTypeIndication $parent
      * @return void
      * @throws \Railt\SDL\Exceptions\TypeConflictException
      */

Spacing +6 added lines, -6 removed lines

@@ -87,7 +87,7 @@ 
      */
     private function validateListDirectRedefinition(AllowsTypeIndication $child, AllowsTypeIndication $parent): void
     {
-        $isBrokenDirectRedefinition = $parent->isList() && ! $child->isList();
+        $isBrokenDirectRedefinition = $parent->isList() && !$child->isList();
 
         if ($isBrokenDirectRedefinition) {
             $error = \sprintf('The %s cannot be overridden by non-list, but %s given', $parent, $child);
@@ -115,7 +115,7 @@ 
      */
     private function validateListInverseRedefinition(AllowsTypeIndication $child, AllowsTypeIndication $parent): void
     {
-        $isBrokenInverseRedefinition = ! $parent->isList() && $child->isList();
+        $isBrokenInverseRedefinition = !$parent->isList() && $child->isList();
 
         if ($isBrokenInverseRedefinition) {
             $error = \sprintf('The %s cannot be overridden by list, but %s given', $parent, $child);
@@ -131,10 +131,10 @@ 
      */
     private function validatePostconditionalInheritance(AllowsTypeIndication $child, AllowsTypeIndication $parent): void
     {
-        $invalidWrapperInheritance = $parent->isNonNull() && ! $child->isNonNull();
+        $invalidWrapperInheritance = $parent->isNonNull() && !$child->isNonNull();
 
         $invalidListWrapperInheritance = $this->isSameWrapperList($child, $parent) &&
-            $parent->isListOfNonNulls() && ! $child->isListOfNonNulls();
+            $parent->isListOfNonNulls() && !$child->isListOfNonNulls();
 
         if ($invalidWrapperInheritance || $invalidListWrapperInheritance) {
             $error = '%s postcondition of %s can not be weakened by %s of %s';
@@ -152,10 +152,10 @@ 
      */
     private function validatePreconditionalInheritance(AllowsTypeIndication $child, AllowsTypeIndication $parent): void
     {
-        $invalidWrapperInheritance = ! $parent->isNonNull() && $child->isNonNull();
+        $invalidWrapperInheritance = !$parent->isNonNull() && $child->isNonNull();
 
         $invalidListWrapperInheritance = $this->isSameWrapperList($child, $parent) &&
-            ! $parent->isListOfNonNulls() && $child->isListOfNonNulls();
+            !$parent->isListOfNonNulls() && $child->isListOfNonNulls();
 
         if ($invalidWrapperInheritance || $invalidListWrapperInheritance) {
             $error = '%s precondition of %s can not be strengthened by %s of %s';

src/Reflection/Builder/Invocations/ValueBuilder.php

Spacing +4 added lines, -4 removed lines

@@ -123,7 +123,7 @@ 
      */
     private function toFloat(LeafInterface $ast): float
     {
-        return (float)$ast->getValue();
+        return (float) $ast->getValue();
     }
 
     /**
@@ -132,7 +132,7 @@ 
      */
     private function toInt(LeafInterface $ast): int
     {
-        return (int)$ast->getValue();
+        return (int) $ast->getValue();
     }
 
     /**
@@ -168,7 +168,7 @@ 
                 return \substr($ast->getValue(), 3, -3);
         }
 
-        return (string)$ast->getValue();
+        return (string) $ast->getValue();
     }
 
     /**
@@ -201,7 +201,7 @@ 
         // TODO Probably may be escaped by backslash like "\\u0000"
         $pattern = '/\\\\u([0-9a-fA-F]{4})/';
 
-        $callee = function (array $matches): string {
+        $callee = function(array $matches): string {
             [$char, $code] = [$matches[0], $matches[1]];
 
             try {

src/Reflection/Builder/Invocations/InputInvocationBuilder.php

Spacing +2 added lines, -2 removed lines

@@ -65,7 +65,7 @@ 
      */
     protected function onCompile(NodeInterface $ast): bool
     {
-        $key   = (string)$ast->getChild(0)->getChild(0)->getValue();
+        $key   = (string) $ast->getChild(0)->getChild(0)->getValue();
         $value = $ast->getChild(1)->getChild(0);
 
         $this->arguments[$key] = $this->parseValue($value, $this->parentType, \array_merge($this->path, [$key]));
@@ -97,7 +97,7 @@ 
      */
     public function getTypeDefinition(): ?TypeDefinition
     {
-        $reduce = function (?InputDefinition $carry, $item): ?TypeDefinition {
+        $reduce = function(?InputDefinition $carry, $item): ?TypeDefinition {
             /** @var ArgumentDefinition|null $argument */
             $argument = $carry->getArgument($item);
             // TODO $argument can be null. Add validation?

src/Reflection/Builder/Invocations/DirectiveInvocationBuilder.php

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
             /** @var ArgumentDefinition|null $argument */
             $argument   = $definition->getArgument($name);
 
-            if (! $argument) {
+            if (!$argument) {
                 $this->getCompiler()->getCallStack()->push($definition);
 
                 $error = \sprintf('Argument %s not defined in %s', $name, $definition);

src/Reflection/Validation/Base/Factory.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
      */
     public function addValidator(string $item): self
     {
-        if (! \is_subclass_of($item, ValidatorInterface::class)) {
+        if (!\is_subclass_of($item, ValidatorInterface::class)) {
             $error = \sprintf('%s must be instance of %s', $item, ValidatorInterface::class);
             throw new \InvalidArgumentException($error);
         }

Doc Comments +1 added lines

@@ -55,6 +55,7 @@
 
     /**
      * @param array ...$args
+     * @param \Railt\SDL\Contracts\Definitions\Definition $args
      * @return void
      * @internal Delegate
      */

src/Reflection/Validation/Definitions/DirectiveInvocationValidator.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
         /** @var DirectiveDefinition $definition */
         $definition = $directive->getTypeDefinition();
 
-        if (! $definition->isAllowedFor($directive->getParent())) {
+        if (!$definition->isAllowedFor($directive->getParent())) {
             $error = \vsprintf('Trying to define directive %s on %s, but only %s locations allowed.', [
                 $directive,
                 $directive->getParent(),

src/Reflection/Validation/Definitions/PassedArgumentsValidator.php

Spacing +5 added lines, -5 removed lines

@@ -71,7 +71,7 @@ 
 
         $this->getCallStack()->push($type);
 
-        if (! ($type instanceof Inputable)) {
+        if (!($type instanceof Inputable)) {
             $error = \sprintf('%s must be type of Scalar, Enum or Input', $type);
             throw new TypeConflictException($error, $this->getCallStack());
         }
@@ -93,7 +93,7 @@ 
      */
     private function validateSingleArgument(ArgumentDefinition $arg, Inputable $type, $value): void
     {
-        if (! $type->isCompatible($value)) {
+        if (!$type->isCompatible($value)) {
             $error = \vsprintf('The argument %s of %s contain non compatible value %s', [
                 $arg->getName(),
                 $type,
@@ -112,7 +112,7 @@ 
      */
     private function validateListArguments(ArgumentDefinition $arg, Inputable $type, $value): void
     {
-        if (! \is_iterable($value)) {
+        if (!\is_iterable($value)) {
             $error = \vsprintf('The argument %s of %s should contain list value, but %s given', [
                 $arg->getName(),
                 $type,
@@ -135,7 +135,7 @@ 
      */
     private function validateMissingArgument(HasPassedArguments $invocation, ArgumentDefinition $argument): void
     {
-        if (! $invocation->hasPassedArgument($argument->getName())) {
+        if (!$invocation->hasPassedArgument($argument->getName())) {
             $error = \sprintf('Required argument "%s" of %s not specified', $argument, $argument->getParent());
 
             throw new TypeConflictException($error, $this->getCallStack());
@@ -150,7 +150,7 @@ 
      */
     private function validateArgumentExisting(HasArguments $container, string $argument): void
     {
-        if (! $container->hasArgument($argument)) {
+        if (!$container->hasArgument($argument)) {
             $error = \sprintf('In the %s there is no specified argument "%s"', $container, $argument);
 
             throw new TypeConflictException($error, $this->getCallStack());