ColonyRepository::getForeignColoniesInBroadcastRange() - Code Metrics - Inspection of "fix" - st-universe/core - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( 87706c...9d01da )

by Nico

created 2023-09-17 11:44 UTC

getForeignColoniesInBroadcastRange() A

↳ Parent: ColonyRepository

Complexity

Conditions	1
Paths	1

Size

Total Lines	25
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
cc	1
eloc	18
nc	1
nop	2
dl	0
loc	25
ccs	0
cts	17
cp	0
crap	2
rs	9.6666
c	0
b	0
f	0

<?php

declare(strict_types=1);

namespace Stu\Orm\Repository;

use Doctrine\ORM\EntityRepository;
use Doctrine\ORM\Query\ResultSetMapping;
use Stu\Module\PlayerSetting\Lib\UserEnum;
use Stu\Orm\Entity\Colony;
use Stu\Orm\Entity\ColonyClass;
use Stu\Orm\Entity\ColonyInterface;
use Stu\Orm\Entity\Map;
use Stu\Orm\Entity\MapRegionSettlement;
use Stu\Orm\Entity\StarSystemMap;
use Stu\Orm\Entity\StarSystemMapInterface;
use Stu\Orm\Entity\UserInterface;

/**
 * @extends EntityRepository<Colony>
 */
final class ColonyRepository extends EntityRepository implements ColonyRepositoryInterface
{
    public function prototype(): ColonyInterface
    {
        return new Colony();
    }

    public function save(ColonyInterface $colony): void
    {
        $em = $this->getEntityManager();

        $em->persist($colony);
    }

    public function delete(ColonyInterface $colony): void
    {
        $em = $this->getEntityManager();

        $em->remove($colony);
        $em->flush();
    }

    public function getAmountByUser(UserInterface $user, int $colonyType): int
    {
        return (int) $this->getEntityManager()
            ->createQuery(
                sprintf(
                    'SELECT count(c.id) from %s c WHERE c.user_id = :userId AND c.colonies_classes_id IN (
                        SELECT cc.id FROM %s cc WHERE cc.type = :type
                    )',
                    Colony::class,
                    ColonyClass::class
                )
            )
            ->setParameters([
                'userId' => $user,
                'type' => $colonyType
            ])
            ->getSingleScalarResult();
    }

    public function getStartingByFaction(int $factionId): array
    {
        return $this->getEntityManager()

            ->createQuery(
                sprintf(
                    'SELECT c FROM %s c INDEX BY c.id
                     JOIN %s sm
                     WITH c.starsystem_map_id = sm.id
                     WHERE c.user_id = :userId AND c.colonies_classes_id IN (
                        SELECT pt.id FROM %s pt WHERE pt.allow_start = :allowStart
                    ) AND sm.systems_id IN (
                        SELECT m.systems_id FROM %s m WHERE m.systems_id > 0 AND m.admin_region_id IN (
                            SELECT mrs.region_id from %s mrs WHERE mrs.faction_id = :factionId
                        )
                    )',
                    Colony::class,
                    StarSystemMap::class,
                    ColonyClass::class,
                    Map::class,
                    MapRegionSettlement::class
                )
            )
            ->setParameters([
                'allowStart' => 1,
                'userId' => UserEnum::USER_NOONE,
                'factionId' => $factionId
            ])
            ->getResult();
    }

    public function getByPosition(StarSystemMapInterface $sysmap): ?ColonyInterface
    {
        return $this->findOneBy([
            'starsystem_map_id' => $sysmap->getId()
        ]);
    }

    public function getForeignColoniesInBroadcastRange(
        StarSystemMapInterface $systemMap,
        UserInterface $user
    ): array {
        return $this->getEntityManager()

            ->createQuery(
                sprintf(
                    'SELECT c FROM %s c
                     JOIN %s sm
                     WITH c.starsystem_map_id = sm.id
                     WHERE c.user_id NOT IN (:ignoreIds)
                     AND sm.systems_id = :systemId
                     AND sm.sx BETWEEN (:sx - 1) AND (:sx + 1)
                     AND sm.sy BETWEEN (:sy - 1) AND (:sy + 1)',
                    Colony::class,
                    StarSystemMap::class
                )
            )
            ->setParameters([
                'ignoreIds' => [$user->getId(), UserEnum::USER_NOONE],
                'systemId' => $systemMap->getSystem()->getId(),
                'sx' => $systemMap->getSx(),
                'sy' => $systemMap->getSy()
            ])
            ->getResult();
    }

    public function getByBatchGroup(int $batchGroup, int $batchGroupCount): iterable
    {
        return $this->getEntityManager()
            ->createQuery(
                sprintf(
                    'SELECT c FROM %s c
                    WHERE MOD(c.user_id, :groupCount) + 1 = :groupId
                    AND c.user_id != :userId',
                    Colony::class
                )
            )
            ->setParameters([
                'groupId' => $batchGroup,
                'groupCount' => $batchGroupCount,
                'userId' => UserEnum::USER_NOONE
            ])
            ->getResult();
    }

    public function getColonized(): iterable
    {
        return $this->getEntityManager()
            ->createQuery(
                sprintf(
                    'SELECT c FROM %s c WHERE c.user_id != :userId',
                    Colony::class
                )
            )
            ->setParameters([
                'userId' => UserEnum::USER_NOONE,
            ])
            ->getResult();
    }

    public function getColonyListForRenderFragment(UserInterface $user): array
    {
        $rsm = new ResultSetMapping();
        $rsm->addScalarResult('colonyid', 'colonyid', 'integer');
        $rsm->addScalarResult('classid', 'classid', 'integer');
        $rsm->addScalarResult('type', 'type', 'integer');
        $rsm->addScalarResult('nameandsector', 'nameandsector', 'string');

        return $this->getEntityManager()
            ->createNativeQuery(
                'SELECT c.id AS colonyid, cc.id AS classid, cc.type AS type,
                            concat(c.name, \' \', sm.sx, \'|\', sm.sy, \' (\', s.name, \'-\',
                                    CASE WHEN s.is_wormhole
                                        THEN \'Wurmloch\'
                                        ELSE \'System\'
                                    END, \')\') as nameandsector
                        FROM stu_colonies c
                        JOIN stu_colonies_classes cc
                            ON c.colonies_classes_id = cc.id
                        JOIN stu_sys_map sm
                            ON c.starsystem_map_id = sm.id
                        JOIN stu_systems s
                            ON sm.systems_id = s.id
                        WHERE c.user_id = :userId
                        ORDER BY cc.id ASC, c.id ASC',
                $rsm
            )
            ->setParameter('userId', $user->getId())
            ->getResult();
    }

    public function getColoniesNetWorth(): array
    {
        $rsm = new ResultSetMapping();
        $rsm->addScalarResult('user_id', 'user_id', 'integer');
        $rsm->addScalarResult('commodity_id', 'commodity_id', 'integer');
        $rsm->addScalarResult('sum', 'sum', 'integer');

        return $this->getEntityManager()
            ->createNativeQuery(
                'SELECT u.id as user_id, bc.commodity_id as commodity_id, sum(bc.count) as sum
                FROM stu_user u
                JOIN stu_colonies c
                ON u.id = c.user_id 
                JOIN stu_colonies_fielddata cf
                ON cf.colonies_id = c.id
                JOIN stu_buildings_cost bc 
                ON cf.buildings_id = bc.buildings_id 
                WHERE u.id >= :firstUserId
                AND cf.buildings_id IS NOT NULL
                AND cf.aktiv = 1
                GROUP BY u.id, bc.commodity_id',
                $rsm
            )
            ->setParameters(['firstUserId' => UserEnum::USER_FIRST_ID])
            ->getResult();
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Stu\Orm\Repository;
6
7			use Doctrine\ORM\EntityRepository;
8			use Doctrine\ORM\Query\ResultSetMapping;
9			use Stu\Module\PlayerSetting\Lib\UserEnum;
10			use Stu\Orm\Entity\Colony;
11			use Stu\Orm\Entity\ColonyClass;
12			use Stu\Orm\Entity\ColonyInterface;
13			use Stu\Orm\Entity\Map;
14			use Stu\Orm\Entity\MapRegionSettlement;
15			use Stu\Orm\Entity\StarSystemMap;
16			use Stu\Orm\Entity\StarSystemMapInterface;
17			use Stu\Orm\Entity\UserInterface;
18
19			/**
20			* @extends EntityRepository<Colony>
21			*/
22			final class ColonyRepository extends EntityRepository implements ColonyRepositoryInterface
23			{
24			public function prototype(): ColonyInterface
25			{
26			return new Colony();
27			}
28
29			public function save(ColonyInterface $colony): void
30			{
31			$em = $this->getEntityManager();
32
33			$em->persist($colony);
34			}
35
36			public function delete(ColonyInterface $colony): void
37			{
38			$em = $this->getEntityManager();
39
40			$em->remove($colony);
41			$em->flush();
42			}
43
44			public function getAmountByUser(UserInterface $user, int $colonyType): int
45			{
46			return (int) $this->getEntityManager()
47			->createQuery(
48			sprintf(
49			'SELECT count(c.id) from %s c WHERE c.user_id = :userId AND c.colonies_classes_id IN (
50			SELECT cc.id FROM %s cc WHERE cc.type = :type
51			)',
52			Colony::class,
53			ColonyClass::class
54			)
55			)
56			->setParameters([
57			'userId' => $user,
58			'type' => $colonyType
59			])
60			->getSingleScalarResult();
61			}
62
63			public function getStartingByFaction(int $factionId): array
64			{
65			return $this->getEntityManager()
			1 ignored issue – show Bug Best Practice introduced 2023-04-26 07:38 UTC by Report Bug Copy Issue Report The expression `return $this->getEntityM...actionId))->getResult()` could return the type `integer` which is incompatible with the type-hinted return `array`. Consider adding an additional type-check to rule them out. Loading history...
66			->createQuery(
67			sprintf(
68			'SELECT c FROM %s c INDEX BY c.id
69			JOIN %s sm
70			WITH c.starsystem_map_id = sm.id
71			WHERE c.user_id = :userId AND c.colonies_classes_id IN (
72			SELECT pt.id FROM %s pt WHERE pt.allow_start = :allowStart
73			) AND sm.systems_id IN (
74			SELECT m.systems_id FROM %s m WHERE m.systems_id > 0 AND m.admin_region_id IN (
75			SELECT mrs.region_id from %s mrs WHERE mrs.faction_id = :factionId
76			)
77			)',
78			Colony::class,
79			StarSystemMap::class,
80			ColonyClass::class,
81			Map::class,
82			MapRegionSettlement::class
83			)
84			)
85			->setParameters([
86			'allowStart' => 1,
87			'userId' => UserEnum::USER_NOONE,
88			'factionId' => $factionId
89			])
90			->getResult();
91			}
92
93			public function getByPosition(StarSystemMapInterface $sysmap): ?ColonyInterface
94			{
95			return $this->findOneBy([
96			'starsystem_map_id' => $sysmap->getId()
97			]);
98			}
99
100			public function getForeignColoniesInBroadcastRange(
101			StarSystemMapInterface $systemMap,
102			UserInterface $user
103			): array {
104			return $this->getEntityManager()
			1 ignored issue – show Bug Best Practice introduced 2023-04-26 07:38 UTC by Report Bug Copy Issue Report The expression `return $this->getEntityM...>getSy()))->getResult()` could return the type `integer` which is incompatible with the type-hinted return `array`. Consider adding an additional type-check to rule them out. Loading history...
105			->createQuery(
106			sprintf(
107			'SELECT c FROM %s c
108			JOIN %s sm
109			WITH c.starsystem_map_id = sm.id
110			WHERE c.user_id NOT IN (:ignoreIds)
111			AND sm.systems_id = :systemId
112			AND sm.sx BETWEEN (:sx - 1) AND (:sx + 1)
113			AND sm.sy BETWEEN (:sy - 1) AND (:sy + 1)',
114			Colony::class,
115			StarSystemMap::class
116			)
117			)
118			->setParameters([
119			'ignoreIds' => [$user->getId(), UserEnum::USER_NOONE],
120			'systemId' => $systemMap->getSystem()->getId(),
121			'sx' => $systemMap->getSx(),
122			'sy' => $systemMap->getSy()
123			])
124			->getResult();
125			}
126
127			public function getByBatchGroup(int $batchGroup, int $batchGroupCount): iterable
128			{
129			return $this->getEntityManager()
130			->createQuery(
131			sprintf(
132			'SELECT c FROM %s c
133			WHERE MOD(c.user_id, :groupCount) + 1 = :groupId
134			AND c.user_id != :userId',
135			Colony::class
136			)
137			)
138			->setParameters([
139			'groupId' => $batchGroup,
140			'groupCount' => $batchGroupCount,
141			'userId' => UserEnum::USER_NOONE
142			])
143			->getResult();
144			}
145
146			public function getColonized(): iterable
147			{
148			return $this->getEntityManager()
149			->createQuery(
150			sprintf(
151			'SELECT c FROM %s c WHERE c.user_id != :userId',
152			Colony::class
153			)
154			)
155			->setParameters([
156			'userId' => UserEnum::USER_NOONE,
157			])
158			->getResult();
159			}
160
161			public function getColonyListForRenderFragment(UserInterface $user): array
162			{
163			$rsm = new ResultSetMapping();
164			$rsm->addScalarResult('colonyid', 'colonyid', 'integer');
165			$rsm->addScalarResult('classid', 'classid', 'integer');
166			$rsm->addScalarResult('type', 'type', 'integer');
167			$rsm->addScalarResult('nameandsector', 'nameandsector', 'string');
168
169			return $this->getEntityManager()
170			->createNativeQuery(
171			'SELECT c.id AS colonyid, cc.id AS classid, cc.type AS type,
172			concat(c.name, \' \', sm.sx, \'\|\', sm.sy, \' (\', s.name, \'-\',
173			CASE WHEN s.is_wormhole
174			THEN \'Wurmloch\'
175			ELSE \'System\'
176			END, \')\') as nameandsector
177			FROM stu_colonies c
178			JOIN stu_colonies_classes cc
179			ON c.colonies_classes_id = cc.id
180			JOIN stu_sys_map sm
181			ON c.starsystem_map_id = sm.id
182			JOIN stu_systems s
183			ON sm.systems_id = s.id
184			WHERE c.user_id = :userId
185			ORDER BY cc.id ASC, c.id ASC',
186			$rsm
187			)
188			->setParameter('userId', $user->getId())
189			->getResult();
190			}
191
192			public function getColoniesNetWorth(): array
193			{
194			$rsm = new ResultSetMapping();
195			$rsm->addScalarResult('user_id', 'user_id', 'integer');
196			$rsm->addScalarResult('commodity_id', 'commodity_id', 'integer');
197			$rsm->addScalarResult('sum', 'sum', 'integer');
198
199			return $this->getEntityManager()
200			->createNativeQuery(
201			'SELECT u.id as user_id, bc.commodity_id as commodity_id, sum(bc.count) as sum
202			FROM stu_user u
203			JOIN stu_colonies c
204			ON u.id = c.user_id
205			JOIN stu_colonies_fielddata cf
206			ON cf.colonies_id = c.id
207			JOIN stu_buildings_cost bc
208			ON cf.buildings_id = bc.buildings_id
209			WHERE u.id >= :firstUserId
210			AND cf.buildings_id IS NOT NULL
211			AND cf.aktiv = 1
212			GROUP BY u.id, bc.commodity_id',
213			$rsm
214			)
215			->setParameters(['firstUserId' => UserEnum::USER_FIRST_ID])
216			->getResult();
217			}
218			}
219

st-universe / core

Push — master ( 87706c...9d01da )

getForeignColoniesInBroadcastRange() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like