Skip to content

Commit 50f85e7

Browse files
authored
Merge pull request #711 from mcg-web/simplify-fqcn-when-using-class_exists
Simplify FQCN when using class_exists
2 parents 4bc9e70 + 55467be commit 50f85e7

File tree

8 files changed

+16
-9
lines changed

8 files changed

+16
-9
lines changed

src/Config/Parser/AnnotationParser.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -321,8 +321,8 @@ private static function typeAnnotationToGQLConfiguration(
321321
private static function getAnnotationReader(): AnnotationReader
322322
{
323323
if (null === self::$annotationReader) {
324-
if (!class_exists('\\Doctrine\\Common\\Annotations\\AnnotationReader') ||
325-
!class_exists('\\Doctrine\\Common\\Annotations\\AnnotationRegistry')) {
324+
if (!class_exists(AnnotationReader::class) ||
325+
!class_exists(AnnotationRegistry::class)) {
326326
throw new RuntimeException('In order to use graphql annotation, you need to require doctrine annotations');
327327
}
328328

src/DependencyInjection/OverblogGraphQLExtension.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
use Symfony\Component\DependencyInjection\Reference;
3232
use Symfony\Component\EventDispatcher\EventDispatcherInterface;
3333
use Symfony\Component\HttpKernel\DependencyInjection\Extension;
34+
use Symfony\Component\Validator\Validation;
3435
use function array_fill_keys;
3536
use function class_exists;
3637
use function realpath;
@@ -105,7 +106,7 @@ private function registerForAutoconfiguration(ContainerBuilder $container): void
105106

106107
private function registerValidatorFactory(ContainerBuilder $container): void
107108
{
108-
if (class_exists('Symfony\\Component\\Validator\\Validation')) {
109+
if (class_exists(Validation::class)) {
109110
$container->register(ValidatorFactory::class)
110111
->setArguments([
111112
new Reference('validator.validator_factory'),

tests/EventListener/ValidationErrorsListenerTest.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
use PHPUnit\Framework\TestCase;
1313
use Symfony\Component\Validator\ConstraintViolation;
1414
use Symfony\Component\Validator\ConstraintViolationList;
15+
use Symfony\Component\Validator\Validation;
1516
use function class_exists;
1617

1718
class ValidationErrorsListenerTest extends TestCase
@@ -21,7 +22,7 @@ class ValidationErrorsListenerTest extends TestCase
2122

2223
protected function setUp(): void
2324
{
24-
if (!class_exists('Symfony\\Component\\Validator\\Validation')) {
25+
if (!class_exists(Validation::class)) {
2526
$this->markTestSkipped('Symfony validator component is not installed');
2627
}
2728
$this->listener = new ValidationErrorsListener();

tests/ExpressionLanguage/ExpressionFunction/GraphQL/ArgumentsTest.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
use Overblog\GraphQLBundle\Tests\Transformer\InputType1;
1616
use Overblog\GraphQLBundle\Tests\Transformer\InputType2;
1717
use Overblog\GraphQLBundle\Transformer\ArgumentsTransformer;
18+
use Symfony\Component\Validator\Validation;
1819
use Symfony\Component\Validator\Validator\RecursiveValidator;
1920
use function class_exists;
2021
use function count;
@@ -23,7 +24,7 @@ class ArgumentsTest extends TestCase
2324
{
2425
public function setUp(): void
2526
{
26-
if (!class_exists('Symfony\\Component\\Validator\\Validation')) {
27+
if (!class_exists(Validation::class)) {
2728
$this->markTestSkipped('Symfony validator component is not installed');
2829
}
2930
parent::setUp();

tests/Functional/Validator/InputValidatorTest.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
namespace Overblog\GraphQLBundle\Tests\Functional\Validator;
66

77
use Overblog\GraphQLBundle\Tests\Functional\TestCase;
8+
use Symfony\Component\Validator\Validation;
89
use function class_exists;
910
use function json_decode;
1011

@@ -13,7 +14,7 @@ class InputValidatorTest extends TestCase
1314
protected function setUp(): void
1415
{
1516
parent::setUp();
16-
if (!class_exists('Symfony\\Component\\Validator\\Validation')) {
17+
if (!class_exists(Validation::class)) {
1718
$this->markTestSkipped('Symfony validator component is not installed');
1819
}
1920
static::bootKernel(['test_case' => 'validator']);

tests/Transformer/ArgumentsTransformerTest.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
use PHPUnit\Framework\TestCase;
1717
use Symfony\Component\Validator\ConstraintViolation;
1818
use Symfony\Component\Validator\ConstraintViolationList;
19+
use Symfony\Component\Validator\Validation;
1920
use Symfony\Component\Validator\Validator\RecursiveValidator;
2021
use function class_exists;
2122
use function count;
@@ -26,7 +27,7 @@ class ArgumentsTransformerTest extends TestCase
2627
protected function setUp(): void
2728
{
2829
parent::setUp();
29-
if (!class_exists('Symfony\\Component\\Validator\\Validation')) {
30+
if (!class_exists(Validation::class)) {
3031
$this->markTestSkipped('Symfony validator component is not installed');
3132
}
3233
}

tests/Validator/InputValidatorTest.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,14 +9,15 @@
99
use PHPUnit\Framework\TestCase;
1010
use Symfony\Component\DependencyInjection\Exception\ServiceNotFoundException;
1111
use Symfony\Component\Validator\ConstraintValidatorFactory;
12+
use Symfony\Component\Validator\Validation;
1213
use function class_exists;
1314

1415
class InputValidatorTest extends TestCase
1516
{
1617
public function setUp(): void
1718
{
1819
parent::setUp();
19-
if (!class_exists('Symfony\\Component\\Validator\\Validation')) {
20+
if (!class_exists(Validation::class)) {
2021
$this->markTestSkipped('Symfony validator component is not installed');
2122
}
2223
}

tests/Validator/Mapping/MetadataFactoryTest.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,14 @@
1111
use PHPUnit\Framework\TestCase;
1212
use stdClass;
1313
use Symfony\Component\Validator\Exception\NoSuchMetadataException;
14+
use Symfony\Component\Validator\Validation;
1415
use function class_exists;
1516

1617
class MetadataFactoryTest extends TestCase
1718
{
1819
public function setUp(): void
1920
{
20-
if (!class_exists('Symfony\\Component\\Validator\\Validation')) {
21+
if (!class_exists(Validation::class)) {
2122
$this->markTestSkipped('Symfony validator component is not installed');
2223
}
2324
parent::setUp();

0 commit comments

Comments
 (0)