simPod / graphql-php

src/Validator/Rules/UniqueInputFieldNames.php

Indentation +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 {
     static function duplicateInputFieldMessage($fieldName)
     {
-      return "There can be only one input field named \"$fieldName\".";
+        return "There can be only one input field named \"$fieldName\".";
     }
 
     public $knownNames;

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
                 if (!empty($this->knownNames[$fieldName])) {
                     $context->reportError(new Error(
                         self::duplicateInputFieldMessage($fieldName),
-                        [ $this->knownNames[$fieldName], $node->name ]
+                        [$this->knownNames[$fieldName], $node->name]
                     ));
                 } else {
                     $this->knownNames[$fieldName] = $node->name;

src/Validator/Rules/QueryComplexity.php

Spacing +5 added lines, -5 removed lines

@@ -78,7 +78,7 @@ 
         return $this->invokeIfNeeded(
             $context,
             [
-                NodeKind::SELECTION_SET => function (SelectionSetNode $selectionSet) use ($context) {
+                NodeKind::SELECTION_SET => function(SelectionSetNode $selectionSet) use ($context) {
                     $this->fieldNodeAndDefs = $this->collectFieldASTsAndDefs(
                         $context,
                         $context->getParentType(),
@@ -87,12 +87,12 @@ 
                         $this->fieldNodeAndDefs
                     );
                 },
-                NodeKind::VARIABLE_DEFINITION => function ($def) {
+                NodeKind::VARIABLE_DEFINITION => function($def) {
                     $this->variableDefs[] = $def;
                     return Visitor::skipNode();
                 },
                 NodeKind::OPERATION_DEFINITION => [
-                    'leave' => function (OperationDefinitionNode $operationDefinition) use ($context, &$complexity) {
+                    'leave' => function(OperationDefinitionNode $operationDefinition) use ($context, &$complexity) {
                         $errors = $context->getErrors();
 
                         if (empty($errors)) {
@@ -210,7 +210,7 @@ 
 
             if ($variableValuesResult['errors']) {
                 throw new Error(implode("\n\n", array_map(
-                    function ($error) {
+                    function($error) {
                         return $error->getMessage();
                     }
                 , $variableValuesResult['errors'])));
@@ -237,7 +237,7 @@ 
 
             if ($variableValuesResult['errors']) {
                 throw new Error(implode("\n\n", array_map(
-                    function ($error) {
+                    function($error) {
                         return $error->getMessage();
                     }
                     , $variableValuesResult['errors'])));

src/Validator/Rules/LoneAnonymousOperation.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
             NodeKind::DOCUMENT => function(DocumentNode $node) use (&$operationCount) {
                 $tmp = Utils::filter(
                     $node->definitions,
-                    function ($definition) {
+                    function($definition) {
                         return $definition->kind === NodeKind::OPERATION_DEFINITION;
                     }
                 );

src/Validator/Rules/VariablesInAllowedPosition.php

Spacing +3 added lines, -3 removed lines

@@ -15,7 +15,7 @@ 
 {
     static function badVarPosMessage($varName, $varType, $expectedType)
     {
-        return "Variable \"\$$varName\" of type \"$varType\" used in position expecting ".
+        return "Variable \"\$$varName\" of type \"$varType\" used in position expecting " .
         "type \"$expectedType\".";
     }
 
@@ -25,7 +25,7 @@ 
     {
         return [
             NodeKind::OPERATION_DEFINITION => [
-                'enter' => function () {
+                'enter' => function() {
                     $this->varDefMap = [];
                 },
                 'leave' => function(OperationDefinitionNode $operation) use ($context) {
@@ -56,7 +56,7 @@ 
                     }
                 }
             ],
-            NodeKind::VARIABLE_DEFINITION => function (VariableDefinitionNode $varDefNode) {
+            NodeKind::VARIABLE_DEFINITION => function(VariableDefinitionNode $varDefNode) {
                 $this->varDefMap[$varDefNode->variable->name->value] = $varDefNode;
             }
         ];

src/Validator/Rules/OverlappingFieldsCanBeMerged.php

Indentation +10 added lines, -10 removed lines

@@ -249,11 +249,11 @@ 
         // (D) First collect any conflicts between the provided collection of fields
         // and the collection of fields represented by the given fragment.
         $this->collectConflictsBetween(
-          $context,
-          $conflicts,
-          $areMutuallyExclusive,
-          $fieldMap,
-          $fieldMap2
+            $context,
+            $conflicts,
+            $areMutuallyExclusive,
+            $fieldMap,
+            $fieldMap2
         );
 
         // (E) Then collect any conflicts between the provided collection of fields
@@ -327,11 +327,11 @@ 
         // (F) First, collect all conflicts between these two collections of fields
         // (not including any nested fragments).
         $this->collectConflictsBetween(
-          $context,
-          $conflicts,
-          $areMutuallyExclusive,
-          $fieldMap1,
-          $fieldMap2
+            $context,
+            $conflicts,
+            $areMutuallyExclusive,
+            $fieldMap1,
+            $fieldMap2
         );
 
         // (G) Then collect conflicts between the first fragment and any nested

Spacing +8 added lines, -12 removed lines

@@ -33,7 +33,7 @@ 
     static function reasonMessage($reason)
     {
         if (is_array($reason)) {
-            $tmp = array_map(function ($tmp) {
+            $tmp = array_map(function($tmp) {
                 list($responseName, $subReason) = $tmp;
                 $reasonMessage = self::reasonMessage($subReason);
                 return "subfields \"$responseName\" conflict because $reasonMessage";
@@ -692,23 +692,19 @@ 
     {
         if ($type1 instanceof ListOfType) {
             return $type2 instanceof ListOfType ?
-                $this->doTypesConflict($type1->getWrappedType(), $type2->getWrappedType()) :
-                true;
+                $this->doTypesConflict($type1->getWrappedType(), $type2->getWrappedType()) : true;
         }
         if ($type2 instanceof ListOfType) {
             return $type1 instanceof ListOfType ?
-                $this->doTypesConflict($type1->getWrappedType(), $type2->getWrappedType()) :
-                true;
+                $this->doTypesConflict($type1->getWrappedType(), $type2->getWrappedType()) : true;
         }
         if ($type1 instanceof NonNull) {
             return $type2 instanceof NonNull ?
-                $this->doTypesConflict($type1->getWrappedType(), $type2->getWrappedType()) :
-                true;
+                $this->doTypesConflict($type1->getWrappedType(), $type2->getWrappedType()) : true;
         }
         if ($type2 instanceof NonNull) {
             return $type1 instanceof NonNull ?
-                $this->doTypesConflict($type1->getWrappedType(), $type2->getWrappedType()) :
-                true;
+                $this->doTypesConflict($type1->getWrappedType(), $type2->getWrappedType()) : true;
         }
         if (Type::isLeafType($type1) || Type::isLeafType($type2)) {
             return $type1 !== $type2;
@@ -858,20 +854,20 @@ 
             return [
                 [
                     $responseName,
-                    array_map(function ($conflict) {
+                    array_map(function($conflict) {
                         return $conflict[0];
                     }, $conflicts),
                 ],
                 array_reduce(
                     $conflicts,
-                    function ($allFields, $conflict) {
+                    function($allFields, $conflict) {
                         return array_merge($allFields, $conflict[1]);
                     },
                     [$ast1]
                 ),
                 array_reduce(
                     $conflicts,
-                    function ($allFields, $conflict) {
+                    function($allFields, $conflict) {
                         return array_merge($allFields, $conflict[2]);
                     },
                     [$ast2]

src/Validator/Rules/UniqueDirectivesPerLocation.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
 {
     static function duplicateDirectiveMessage($directiveName)
     {
-        return 'The directive "'.$directiveName.'" can only be used once at this location.';
+        return 'The directive "' . $directiveName . '" can only be used once at this location.';
     }
 
     public function getVisitor(ValidationContext $context)

src/Validator/Rules/QueryDepth.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
             $context,
             [
                 NodeKind::OPERATION_DEFINITION => [
-                    'leave' => function (OperationDefinitionNode $operationDefinition) use ($context) {
+                    'leave' => function(OperationDefinitionNode $operationDefinition) use ($context) {
                         $maxDepth = $this->fieldDepth($operationDefinition);
 
                         if ($maxDepth > $this->getMaxQueryDepth()) {

src/Validator/Rules/FieldsOnCorrectType.php

Spacing +3 added lines, -3 removed lines

@@ -15,7 +15,7 @@ 
 {
     static function undefinedFieldMessage($fieldName, $type, array $suggestedTypeNames, array $suggestedFieldNames)
     {
-        $message = 'Cannot query field "' . $fieldName . '" on type "' . $type.'".';
+        $message = 'Cannot query field "' . $fieldName . '" on type "' . $type . '".';
 
         if ($suggestedTypeNames) {
             $suggestions = Utils::quotedOrList($suggestedTypeNames);
@@ -86,14 +86,14 @@ 
             $suggestedObjectTypes = [];
             $interfaceUsageCount = [];
 
-            foreach($schema->getPossibleTypes($type) as $possibleType) {
+            foreach ($schema->getPossibleTypes($type) as $possibleType) {
                 $fields = $possibleType->getFields();
                 if (!isset($fields[$fieldName])) {
                     continue;
                 }
                 // This object type defines this field.
                 $suggestedObjectTypes[] = $possibleType->name;
-                foreach($possibleType->getInterfaces() as $possibleInterface) {
+                foreach ($possibleType->getInterfaces() as $possibleInterface) {
                     $fields = $possibleInterface->getFields();
                     if (!isset($fields[$fieldName])) {
                         continue;

src/Validator/Rules/KnownArgumentNames.php

Spacing +2 added lines, -2 removed lines

@@ -51,7 +51,7 @@ 
                                     $parentType->name,
                                     Utils::suggestionList(
                                         $node->name->value,
-                                        array_map(function ($arg) { return $arg->name; }, $fieldDef->args)
+                                        array_map(function($arg) { return $arg->name; }, $fieldDef->args)
                                     )
                                 ),
                                 [$node]
@@ -66,7 +66,7 @@ 
                                     $directive->name,
                                     Utils::suggestionList(
                                         $node->name->value,
-                                        array_map(function ($arg) { return $arg->name; }, $directive->args)
+                                        array_map(function($arg) { return $arg->name; }, $directive->args)
                                     )
                                 ),
                                 [$node]