Merge pull request overblog#734 from overblog/revert-656-mo-olsavmic-patch-1

Vincz · web-flow · commit e5d1cab6227c · 2020-08-16T18:29:48.000+02:00
Revert "Allow to use non-empty constructor for input types"
diff --git a/src/Transformer/ArgumentsTransformer.php b/src/Transformer/ArgumentsTransformer.php
@@ -12,7 +12,6 @@
 use GraphQL\Type\Definition\Type;
 use Overblog\GraphQLBundle\Error\InvalidArgumentError;
 use Overblog\GraphQLBundle\Error\InvalidArgumentsError;
-use ReflectionClass;
 use Symfony\Component\PropertyAccess\PropertyAccess;
 use Symfony\Component\PropertyAccess\PropertyAccessor;
 use Symfony\Component\Validator\ConstraintViolationList;
@@ -47,7 +46,7 @@ private function getTypeClassInstance(string $type)
     {
         $classname = isset($this->classesMap[$type]) ? $this->classesMap[$type]['class'] : false;
 
-        return $classname ? (new ReflectionClass($classname))->newInstanceWithoutConstructor() : false;
+        return $classname ? new $classname() : false;
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,6 @@`
`12`	`12`	`use GraphQL\Type\Definition\Type;`
`13`	`13`	`use Overblog\GraphQLBundle\Error\InvalidArgumentError;`
`14`	`14`	`use Overblog\GraphQLBundle\Error\InvalidArgumentsError;`
`15`		`-use ReflectionClass;`
`16`	`15`	`use Symfony\Component\PropertyAccess\PropertyAccess;`
`17`	`16`	`use Symfony\Component\PropertyAccess\PropertyAccessor;`
`18`	`17`	`use Symfony\Component\Validator\ConstraintViolationList;`
`@@ -47,7 +46,7 @@ private function getTypeClassInstance(string $type)`
`47`	`46`	`{`
`48`	`47`	`$classname = isset($this->classesMap[$type]) ? $this->classesMap[$type]['class'] : false;`
`49`	`48`
`50`		`- return $classname ? (new ReflectionClass($classname))->newInstanceWithoutConstructor() : false;`
	`49`	`+ return $classname ? new $classname() : false;`
`51`	`50`	`}`
`52`	`51`
`53`	`52`	`/**`