remove unneeded sprintf() call

xabbuh · xabbuh · commit a8252a43b823 · 2021-01-03T12:55:43.000+01:00
diff --git a/src/Symfony/Component/PropertyAccess/PropertyAccessor.php b/src/Symfony/Component/PropertyAccess/PropertyAccessor.php
@@ -398,7 +398,7 @@ private function readProperty(array $zval, string $property, bool $ignoreInvalid
                     if (__FILE__ === $trace['file']
                         && $access[self::ACCESS_NAME] === $trace['function']
                         && $object instanceof $trace['class']
-                        && preg_match((sprintf('/Return value (?:of .*::\w+\(\) )?must be of (?:the )?type (\w+), null returned$/')), $e->getMessage(), $matches)
+                        && preg_match('/Return value (?:of .*::\w+\(\) )?must be of (?:the )?type (\w+), null returned$/', $e->getMessage(), $matches)
                     ) {
                         throw new AccessException(sprintf('The method "%s::%s()" returned "null", but expected type "%3$s". Did you forget to initialize a property or to make the return type nullable using "?%3$s"?', false === strpos(\get_class($object), "@anonymous\0") ? \get_class($object) : (get_parent_class($object) ?: 'class').'@anonymous', $access[self::ACCESS_NAME], $matches[1]), 0, $e);
                     }

Original file line num 72C5 ber	Diff line number	Diff line change
`@@ -398,7 +398,7 @@ private function readProperty(array $zval, string $property, bool $ignoreInvalid`
`398`	`398`	`if (__FILE__ === $trace['file']`
`399`	`399`	`&& $access[self::ACCESS_NAME] === $trace['function']`
`400`	`400`	`&& $object instanceof $trace['class']`
`401`		`- && preg_match((sprintf('/Return value (?:of .*::\w+\(\) )?must be of (?:the )?type (\w+), null returned$/')), $e->getMessage(), $matches)`
	`401`	`+ && preg_match('/Return value (?:of .*::\w+\(\) )?must be of (?:the )?type (\w+), null returned$/', $e->getMessage(), $matches)`
`402`	`402`	`) {`
`403`	`403`	`throw new AccessException(sprintf('The method "%s::%s()" returned "null", but expected type "%3$s". Did you forget to initialize a property or to make the return type nullable using "?%3$s"?', false === strpos(\get_class($object), "@anonymous\0") ? \get_class($object) : (get_parent_class($object) ?: 'class').'@anonymous', $access[self::ACCESS_NAME], $matches[1]), 0, $e);`
`404`	`404`	`}`