simPod / graphql-php

src/Type/Definition/EnumType.php

Spacing +5 added lines, -5 removed lines

@@ -41,7 +41,7 @@ 
 
     public function __construct($config)
     {
-        if (! isset($config['name'])) {
+        if (!isset($config['name'])) {
             $config['name'] = $this->tryInferName();
         }
 
@@ -63,7 +63,7 @@ 
     {
         $lookup = $this->getNameLookup();
 
-        if (! is_string($name)) {
+        if (!is_string($name)) {
             return null;
         }
 
@@ -75,7 +75,7 @@ 
      */
     private function getNameLookup()
     {
-        if (! $this->nameLookup) {
+        if (!$this->nameLookup) {
             $lookup = new ArrayObject();
             foreach ($this->getValues() as $value) {
                 $lookup[$value->name] = $value;
@@ -96,7 +96,7 @@ 
             $config       = $this->config;
 
             if (isset($config['values'])) {
-                if (! is_array($config['values'])) {
+                if (!is_array($config['values'])) {
                     throw new InvariantViolation(sprintf('%s values must be an array', $this->name));
                 }
                 foreach ($config['values'] as $name => $value) {
@@ -213,7 +213,7 @@ 
         $values = $this->getValues();
         foreach ($values as $value) {
             Utils::invariant(
-                ! isset($value->config['isDeprecated']),
+                !isset($value->config['isDeprecated']),
                 sprintf(
                     '%s.%s should provide "deprecationReason" instead of "isDeprecated".',
                     $this->name,

src/Type/Definition/ResolveInfo.php

Spacing +1 added lines, -1 removed lines

@@ -187,7 +187,7 @@
         $fields = [];
         foreach ($selectionSet->selections as $selectionNode) {
             if ($selectionNode instanceof FieldNode) {
-                $fields[$selectionNode->name->value] = $descend > 0 && ! empty($selectionNode->selectionSet)
+                $fields[$selectionNode->name->value] = $descend > 0 && !empty($selectionNode->selectionSet)
                     ? $this->foldSelectionSet($selectionNode->selectionSet, $descend - 1)
                     : true;
             } elseif ($selectionNode instanceof FragmentSpreadNode) {

src/Type/SchemaValidationContext.php

Spacing +22 added lines, -22 removed lines

@@ -64,12 +64,12 @@ 
     public function validateRootTypes()
     {
         $queryType = $this->schema->getQueryType();
-        if (! $queryType) {
+        if (!$queryType) {
             $this->reportError(
                 'Query root type must be provided.',
                 $this->schema->getAstNode()
             );
-        } elseif (! $queryType instanceof ObjectType) {
+        } elseif (!$queryType instanceof ObjectType) {
             $this->reportError(
                 'Query root type must be Object type, it cannot be ' . Utils::printSafe($queryType) . '.',
                 $this->getOperationTypeNode($queryType, 'query')
@@ -77,7 +77,7 @@ 
         }
 
         $mutationType = $this->schema->getMutationType();
-        if ($mutationType && ! $mutationType instanceof ObjectType) {
+        if ($mutationType && !$mutationType instanceof ObjectType) {
             $this->reportError(
                 'Mutation root type must be Object type if provided, it cannot be ' . Utils::printSafe($mutationType) . '.',
                 $this->getOperationTypeNode($mutationType, 'mutation')
@@ -85,7 +85,7 @@ 
         }
 
         $subscriptionType = $this->schema->getSubscriptionType();
-        if (! $subscriptionType || $subscriptionType instanceof ObjectType) {
+        if (!$subscriptionType || $subscriptionType instanceof ObjectType) {
             return;
         }
 
@@ -143,7 +143,7 @@ 
         $directives = $this->schema->getDirectives();
         foreach ($directives as $directive) {
             // Ensure all directives are in fact GraphQL directives.
-            if (! $directive instanceof Directive) {
+            if (!$directive instanceof Directive) {
                 $this->reportError(
                     'Expected directive but got: ' . Utils::printSafe($directive) . '.',
                     is_object($directive) ? $directive->astNode : null
@@ -198,7 +198,7 @@ 
     {
         // Ensure names are valid, however introspection types opt out.
         $error = Utils::isValidNameError($node->name, $node->astNode);
-        if (! $error || Introspection::isIntrospectionType($node)) {
+        if (!$error || Introspection::isIntrospectionType($node)) {
             return;
         }
 
@@ -244,7 +244,7 @@ 
         $typeMap = $this->schema->getTypeMap();
         foreach ($typeMap as $typeName => $type) {
             // Ensure all provided types are in fact GraphQL type.
-            if (! $type instanceof NamedType) {
+            if (!$type instanceof NamedType) {
                 $this->reportError(
                     'Expected GraphQL named type but got: ' . Utils::printSafe($type) . '.',
                     is_object($type) ? $type->astNode : null
@@ -287,7 +287,7 @@ 
         $fieldMap = $type->getFields();
 
         // Objects and Interfaces both must define one or more fields.
-        if (! $fieldMap) {
+        if (!$fieldMap) {
             $this->reportError(
                 sprintf('Type %s must define one or more fields.', $type->name),
                 $this->getAllObjectOrInterfaceNodes($type)
@@ -309,7 +309,7 @@ 
             }
 
             // Ensure the type is an output type
-            if (! Type::isOutputType($field->getType())) {
+            if (!Type::isOutputType($field->getType())) {
                 $this->reportError(
                     sprintf(
                         'The type of %s.%s must be Output Type but got: %s.',
@@ -386,7 +386,7 @@ 
         $fieldNodes = [];
         $astNodes   = $this->getAllObjectOrInterfaceNodes($type);
         foreach ($astNodes as $astNode) {
-            if (! $astNode || ! $astNode->fields) {
+            if (!$astNode || !$astNode->fields) {
                 continue;
             }
 
@@ -484,7 +484,7 @@ 
     {
         $implementedTypeNames = [];
         foreach ($object->getInterfaces() as $iface) {
-            if (! $iface instanceof InterfaceType) {
+            if (!$iface instanceof InterfaceType) {
                 $this->reportError(
                     sprintf(
                         'Type %s must only implement Interface types, it cannot implement %s.',
@@ -547,7 +547,7 @@ 
         $astNodes        = $this->getAllObjectOrInterfaceNodes($type);
 
         foreach ($astNodes as $astNode) {
-            if (! $astNode || ! $astNode->interfaces) {
+            if (!$astNode || !$astNode->interfaces) {
                 continue;
             }
 
@@ -578,7 +578,7 @@ 
                 : null;
 
             // Assert interface field exists on object.
-            if (! $objectField) {
+            if (!$objectField) {
                 $this->reportError(
                     sprintf(
                         'Interface field %s.%s expected but %s does not provide it.',
@@ -593,7 +593,7 @@ 
 
             // Assert interface field type is satisfied by object field type, by being
             // a valid subtype. (covariant)
-            if (! TypeComparators::isTypeSubTypeOf(
+            if (!TypeComparators::isTypeSubTypeOf(
                 $this->schema,
                 $objectField->getType(),
                 $ifaceField->getType()
@@ -629,7 +629,7 @@ 
                 }
 
                 // Assert interface field arg exists on object field.
-                if (! $objectArg) {
+                if (!$objectArg) {
                     $this->reportError(
                         sprintf(
                             'Interface field argument %s.%s(%s:) expected but %s.%s does not provide it.',
@@ -650,7 +650,7 @@ 
                 // Assert interface field arg type matches object field arg type.
                 // (invariant)
                 // TODO: change to contravariant?
-                if (! TypeComparators::isEqualType($ifaceArg->getType(), $objectArg->getType())) {
+                if (!TypeComparators::isEqualType($ifaceArg->getType(), $objectArg->getType())) {
                     $this->reportError(
                         sprintf(
                             'Interface field argument %s.%s(%s:) expects type %s but %s.%s(%s:) is type %s.',
@@ -684,7 +684,7 @@ 
                     }
                 }
 
-                if ($ifaceArg || ! ($objectArg->getType() instanceof NonNull)) {
+                if ($ifaceArg || !($objectArg->getType() instanceof NonNull)) {
                     continue;
                 }
 
@@ -711,7 +711,7 @@ 
     {
         $memberTypes = $union->getTypes();
 
-        if (! $memberTypes) {
+        if (!$memberTypes) {
             $this->reportError(
                 sprintf('Union type %s must define one or more member types.', $union->name),
                 $union->astNode
@@ -754,7 +754,7 @@ 
         if ($union->astNode && $union->astNode->types) {
             return array_filter(
                 $union->astNode->types,
-                static function (NamedTypeNode $value) use ($typeName) {
+                static function(NamedTypeNode $value) use ($typeName) {
                     return $value->name->value === $typeName;
                 }
             );
@@ -768,7 +768,7 @@ 
     {
         $enumValues = $enumType->getValues();
 
-        if (! $enumValues) {
+        if (!$enumValues) {
             $this->reportError(
                 sprintf('Enum type %s must define one or more values.', $enumType->name),
                 $enumType->astNode
@@ -810,7 +810,7 @@ 
         if ($enum->astNode && $enum->astNode->values) {
             return array_filter(
                 iterator_to_array($enum->astNode->values),
-                static function (EnumValueDefinitionNode $value) use ($valueName) {
+                static function(EnumValueDefinitionNode $value) use ($valueName) {
                     return $value->name->value === $valueName;
                 }
             );
@@ -824,7 +824,7 @@ 
     {
         $fieldMap = $inputObj->getFields();
 
-        if (! $fieldMap) {
+        if (!$fieldMap) {
             $this->reportError(
                 sprintf('Input Object type %s must define one or more fields.', $inputObj->name),
                 $inputObj->astNode

benchmarks/Utils/QueryGenerator.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
 
         $totalFields = 0;
         foreach ($schema->getTypeMap() as $type) {
-            if (! ($type instanceof ObjectType)) {
+            if (!($type instanceof ObjectType)) {
                 continue;
             }
 

benchmarks/HugeSchemaBench.php

Spacing +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
         return new Schema(
             SchemaConfig::create()
                 ->setQuery($this->schemaBuilder->buildQueryType())
-                ->setTypeLoader(function ($name) {
+                ->setTypeLoader(function($name) {
                     return $this->schemaBuilder->loadType($name);
                 })
         );