AccessUrlHelper::getCurrent() - Code Metrics - Inspection of "Multi url disabled" - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#5678)

by Angel Fernando Quiroz

created 2024-07-18 21:24 UTC

AccessUrlHelper::getCurrent() A

↳ Parent: AccessUrlHelper

Complexity

Conditions	3
Paths	3

Size

Total Lines	18
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
eloc	8
nc	3
nop	0
dl	0
loc	18
rs	10
c	0
b	0
f	0

<?php

/* For licensing terms, see /license.txt */

declare(strict_types=1);

namespace Chamilo\CoreBundle\ServiceHelper;

use Chamilo\CoreBundle\Entity\AccessUrl;
use Chamilo\CoreBundle\Repository\Node\AccessUrlRepository;
use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
use Symfony\Component\Routing\RouterInterface;

class AccessUrlHelper
{
    public function __construct(
        private readonly AccessUrlRepository $accessUrlRepository,
        private readonly RouterInterface $router,
    ) {}

    public function getFirstAccessUrl(): AccessUrl
    {
        $urlId = $this->accessUrlRepository->getFirstId();

        return $this->accessUrlRepository->find($urlId);
    }

    public function getCurrent(): AccessUrl
    {
        static $accessUrl;

        if (!empty($accessUrl)) {
            return $accessUrl;
        }

        $url = $this->router->generate('index', [], UrlGeneratorInterface::ABSOLUTE_URL);

        /** @var AccessUrl $accessUrl */
        $accessUrl = $this->accessUrlRepository->findOneBy(['url' => $url]);

        if (!$accessUrl) {

            $accessUrl = $this->getFirstAccessUrl();
        }

        return $accessUrl;
    }
}


1			<?php
2
3			/* For licensing terms, see /license.txt */
4
5			declare(strict_types=1);
6
7			namespace Chamilo\CoreBundle\ServiceHelper;
8
9			use Chamilo\CoreBundle\Entity\AccessUrl;
10			use Chamilo\CoreBundle\Repository\Node\AccessUrlRepository;
11			use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
12			use Symfony\Component\Routing\RouterInterface;
13
14			class AccessUrlHelper
15			{
16			public function __construct(
17			private readonly AccessUrlRepository $accessUrlRepository,
18			private readonly RouterInterface $router,
19			) {}
20
21			public function getFirstAccessUrl(): AccessUrl
22			{
23			$urlId = $this->accessUrlRepository->getFirstId();
24
25			return $this->accessUrlRepository->find($urlId);
26			}
27
28			public function getCurrent(): AccessUrl
29			{
30			static $accessUrl;
31
32			if (!empty($accessUrl)) {
33			return $accessUrl;
34			}
35
36			$url = $this->router->generate('index', [], UrlGeneratorInterface::ABSOLUTE_URL);
37
38			/** @var AccessUrl $accessUrl */
39			$accessUrl = $this->accessUrlRepository->findOneBy(['url' => $url]);
40
41			if (!$accessUrl) {
			0 ignored issues – show introduced 2024-07-18 18:44 UTC by Report Bug Copy Issue Report `$accessUrl` is of type `Chamilo\CoreBundle\Entity\AccessUrl`, thus it always evaluated to `true`. Loading history...
42			$accessUrl = $this->getFirstAccessUrl();
43			}
44
45			return $accessUrl;
46			}
47			}
48

chamilo / chamilo-lms

Pull Request — master (#5678)

AccessUrlHelper::getCurrent() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like