Services::__construct() - Code Metrics - Inspection of "Hotfix/scrutinizer 1" - API-Skeletons/doctrine-orm-graphql - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — 10.1.x (#96)

by Tom

created 2024-03-30 09:29 UTC

Services::__construct() B

↳ Parent: Project

Complexity

Conditions	2
Paths	1

Size

Total Lines	114
Code Lines	80

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
cc	2
eloc	80
c	2
b	0
f	0
nc	1
nop	3
dl	0
loc	114
rs	8.4362

How to fix Long Method

<?php

declare(strict_types=1);

namespace ApiSkeletons\Doctrine\ORM\GraphQL;

use ApiSkeletons\Doctrine\ORM\GraphQL\Metadata\GlobalEnable;
use ArrayObject;
use Doctrine\ORM\EntityManager;
use League\Event\EventDispatcher;

/**
 * This trait is used to remove complexity from the Driver class.
 * It doesn't change what the Driver does.  It just separates the container work
 * from the Driver.
 */
trait Services
{
    /** @param mixed[] $metadata */
    public function __construct(
        EntityManager $entityManager,
        Config|null $config = null,
        array $metadata = [],
    ) {
        $metadata = new ArrayObject($metadata);

        $this
            ->set(EntityManager::class, $entityManager)
            ->set(
                Config::class,
                static function () use ($config) {
                    if (! $config) {
                        $config = new Config();
                    }

                    return $config;
                },
            )
            ->set(EventDispatcher::class, static fn () => new EventDispatcher())
            ->set(Type\TypeContainer::class, static fn () => new Type\TypeContainer())
            )

            ->set(
                Type\Entity\EntityTypeContainer::class,
                static fn (Container $container) => new Type\Entity\EntityTypeContainer($container),
            )
            ->set(
                'metadata',
                static function (Container $container) use ($metadata) {
                    return (new Metadata\MetadataFactory(
                        $metadata,
                        $container->get(EntityManager::class),
                        $container->get(Config::class),
                        $container->get(GlobalEnable::class),
                        $container->get(EventDispatcher::class),
                    ))();
                },
            )
            ->set(
                Metadata\GlobalEnable::class,
                static function (Container $container) {
                    return new Metadata\GlobalEnable(
                        $container->get(EntityManager::class),
                        $container->get(Config::class),
                        $container->get(EventDispatcher::class),
                    );
                },
            )
            ->set(
                Resolve\FieldResolver::class,
                static function (Container $container) {
                    return new Resolve\FieldResolver(
                        $container->get(Config::class),
                        $container->get(Type\Entity\EntityTypeContainer::class),
                    );
                },
            )
            ->set(
                Resolve\ResolveCollectionFactory::class,
                static function (Container $container) {
                    return new Resolve\ResolveCollectionFactory(
                        $container->get(EntityManager::class),
                        $container->get(Config::class),
                        $container->get(Resolve\FieldResolver::class),
                        $container->get(Type\TypeContainer::class),
                        $container->get(EventDispatcher::class),
                        $container->get('metadata'),
                    );
                },
            )
            ->set(
                Resolve\ResolveEntityFactory::class,
                static function (Container $container) {
                    return new Resolve\ResolveEntityFactory(
                        $container->get(Config::class),
                        $container->get(EntityManager::class),
                        $container->get(EventDispatcher::class),
                        $container->get('metadata'),
                    );
                },
            )
            ->set(
                Filter\FilterFactory::class,
                static function (Container $container) {
                    return new Filter\FilterFactory(
                        $container->get(Config::class),
                        $container->get(EntityManager::class),
                        $container->get(Type\TypeContainer::class),
                        $container->get(EventDispatcher::class),
                    );
                },
            )
            ->set(
                Hydrator\HydratorContainer::class,
                static function (Container $container) {
                    return new Hydrator\HydratorContainer(
                        $container->get(EntityManager::class),
                        $container->get(Type\Entity\EntityTypeContainer::class),
                    );
                },
            )
            ->set(
                Input\InputFactory::class,
                static function (Container $container) {
                    return new Input\InputFactory(
                        $container->get(Config::class),
                        $container->get(EntityManager::class),
                        $container->get(Type\Entity\EntityTypeContainer::class),
                        $container->get(Type\TypeContainer::class),
                    );
                },
            );
    }

    abstract public function set(string $id, mixed $value): mixed;
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace ApiSkeletons\Doctrine\ORM\GraphQL;
6
7			use ApiSkeletons\Doctrine\ORM\GraphQL\Metadata\GlobalEnable;
8			use ArrayObject;
9			use Doctrine\ORM\EntityManager;
10			use League\Event\EventDispatcher;
11
12			/**
13			* This trait is used to remove complexity from the Driver class.
14			* It doesn't change what the Driver does. It just separates the container work
15			* from the Driver.
16			*/
17			trait Services
18			{
19			/** @param mixed[] $metadata */
20			public function __construct(
21			EntityManager $entityManager,
22			Config\|null $config = null,
23			array $metadata = [],
24			) {
25			$metadata = new ArrayObject($metadata);
26
27			$this
28			->set(EntityManager::class, $entityManager)
29			->set(
30			Config::class,
31			static function () use ($config) {
32			if (! $config) {
33			$config = new Config();
34			}
35
36			return $config;
37			},
38			)
39			->set(EventDispatcher::class, static fn () => new EventDispatcher())
40			->set(Type\TypeContainer::class, static fn () => new Type\TypeContainer())
41			)
			0 ignored issues – show Bug introduced 2024-03-30 09:32 UTC by Report Bug Copy Issue Report A parse error occurred: Syntax error, unexpected ')' on line 41 at column 12 Loading history...
42			->set(
43			Type\Entity\EntityTypeContainer::class,
44			static fn (Container $container) => new Type\Entity\EntityTypeContainer($container),
45			)
46			->set(
47			'metadata',
48			static function (Container $container) use ($metadata) {
49			return (new Metadata\MetadataFactory(
50			$metadata,
51			$container->get(EntityManager::class),
52			$container->get(Config::class),
53			$container->get(GlobalEnable::class),
54			$container->get(EventDispatcher::class),
55			))();
56			},
57			)
58			->set(
59			Metadata\GlobalEnable::class,
60			static function (Container $container) {
61			return new Metadata\GlobalEnable(
62			$container->get(EntityManager::class),
63			$container->get(Config::class),
64			$container->get(EventDispatcher::class),
65			);
66			},
67			)
68			->set(
69			Resolve\FieldResolver::class,
70			static function (Container $container) {
71			return new Resolve\FieldResolver(
72			$container->get(Config::class),
73			$container->get(Type\Entity\EntityTypeContainer::class),
74			);
75			},
76			)
77			->set(
78			Resolve\ResolveCollectionFactory::class,
79			static function (Container $container) {
80			return new Resolve\ResolveCollectionFactory(
81			$container->get(EntityManager::class),
82			$container->get(Config::class),
83			$container->get(Resolve\FieldResolver::class),
84			$container->get(Type\TypeContainer::class),
85			$container->get(EventDispatcher::class),
86			$container->get('metadata'),
87			);
88			},
89			)
90			->set(
91			Resolve\ResolveEntityFactory::class,
92			static function (Container $container) {
93			return new Resolve\ResolveEntityFactory(
94			$container->get(Config::class),
95			$container->get(EntityManager::class),
96			$container->get(EventDispatcher::class),
97			$container->get('metadata'),
98			);
99			},
100			)
101			->set(
102			Filter\FilterFactory::class,
103			static function (Container $container) {
104			return new Filter\FilterFactory(
105			$container->get(Config::class),
106			$container->get(EntityManager::class),
107			$container->get(Type\TypeContainer::class),
108			$container->get(EventDispatcher::class),
109			);
110			},
111			)
112			->set(
113			Hydrator\HydratorContainer::class,
114			static function (Container $container) {
115			return new Hydrator\HydratorContainer(
116			$container->get(EntityManager::class),
117			$container->get(Type\Entity\EntityTypeContainer::class),
118			);
119			},
120			)
121			->set(
122			Input\InputFactory::class,
123			static function (Container $container) {
124			return new Input\InputFactory(
125			$container->get(Config::class),
126			$container->get(EntityManager::class),
127			$container->get(Type\Entity\EntityTypeContainer::class),
128			$container->get(Type\TypeContainer::class),
129			);
130			},
131			);
132			}
133
134			abstract public function set(string $id, mixed $value): mixed;
135			}
136

API-Skeletons / doctrine-orm-graphql

Pull Request — 10.1.x (#96)

Services::__construct() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like