CToolProvider::provide() - Code Metrics - Inspection of "Display: Add CSS class to section headers" - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( ca77e6...fd95ee )

by Angel Fernando Quiroz

created 2024-01-13 18:04 UTC

CToolProvider::provide() F

↳ Parent: CToolProvider

Complexity

Conditions	18
Paths	792

Size

Total Lines	61
Code Lines	31

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	18
eloc	31
c	1
b	0
f	0
nc	792
nop	3
dl	0
loc	61
rs	0.9888

How to fix Long Method Complexity

<?php

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

declare(strict_types=1);

namespace Chamilo\CoreBundle\State;

use ApiPlatform\Doctrine\Orm\State\CollectionProvider;
use ApiPlatform\Metadata\Operation;
use ApiPlatform\State\Pagination\PartialPaginatorInterface;
use ApiPlatform\State\ProviderInterface;
use Chamilo\CoreBundle\Entity\ResourceLink;
use Chamilo\CoreBundle\Entity\User;
use Chamilo\CoreBundle\Settings\SettingsManager;
use Chamilo\CoreBundle\Tool\ToolChain;
use Chamilo\CoreBundle\Traits\CourseFromRequestTrait;
use Chamilo\CourseBundle\Entity\CTool;
use Doctrine\ORM\EntityManagerInterface;
use Symfony\Component\HttpFoundation\RequestStack;
use Symfony\Component\Security\Core\Security;

/**
 * @template-implements ProviderInterface<CTool>
 */
class CToolProvider implements ProviderInterface
{
    use CourseFromRequestTrait;

    public function __construct(
        private readonly CollectionProvider $provider,
        protected EntityManagerInterface $entityManager,
        private readonly SettingsManager $settingsManager,
        private readonly Security $security,
        private readonly ToolChain $toolChain,
        protected RequestStack $requestStack,
    ) {}

    public function provide(Operation $operation, array $uriVariables = [], array $context = []): array
    {
        /** @var PartialPaginatorInterface $result */
        $result = $this->provider->provide($operation, $uriVariables, $context);

        $request = $this->requestStack->getMainRequest();

        $studentView = $request ? $request->getSession()->get('studentview') : 'studentview';

        /** @var User|null $user */
        $user = $this->security->getUser();

        $isAllowToEdit = $user && ($user->hasRole('ROLE_ADMIN') || $user->hasRole('ROLE_CURRENT_COURSE_TEACHER'));
        $isAllowToEditBack = $user && ($user->hasRole('ROLE_ADMIN') || $user->hasRole('ROLE_CURRENT_COURSE_TEACHER'));
        $isAllowToSessionEdit = $user && ($user->hasRole('ROLE_ADMIN') || $user->hasRole('ROLE_CURRENT_COURSE_TEACHER') || $user->hasRole('ROLE_CURRENT_COURSE_SESSION_TEACHER'));

        $allowVisibilityInSession = $this->settingsManager->getSetting('course.allow_edit_tool_visibility_in_session');
        $session = $this->getSession();

        $results = [];

        /** @var CTool $cTool */
        foreach ($result as $cTool) {
            $toolModel = $this->toolChain->getToolFromName(
                $cTool->getTool()->getName()
            );

            if (!$isAllowToEdit && 'admin' === $toolModel->getCategory()) {
                continue;
            }

            $resourceLinks = $cTool->getResourceNode()->getResourceLinks();

            if ($session && $allowVisibilityInSession) {
                $sessionLink = $resourceLinks->findFirst(
                    fn (int $key, ResourceLink $resourceLink): bool => $resourceLink->getSession()?->getId() === $session->getId()
                );

                if ($sessionLink) {
                    // Set the session link as unique to include in repsonse
                    $resourceLinks->clear();
                    $resourceLinks->add($sessionLink);

                    $isAllowToEdit = $isAllowToSessionEdit;
                } else {
                    $isAllowToEdit = $isAllowToEditBack;
                }
            }

            if (!$isAllowToEdit || 'studentview' === $studentView) {
                $notPublishedLink = ResourceLink::VISIBILITY_PUBLISHED !== $resourceLinks->first()->getVisibility();

                if ($notPublishedLink) {
                    continue;
                }
            }

            $results[] = $cTool;
        }

        return $results;
    }
}


1			<?php
2
3			/* For licensing terms, see /license.txt */
4
5			declare(strict_types=1);
6
7			namespace Chamilo\CoreBundle\State;
8
9			use ApiPlatform\Doctrine\Orm\State\CollectionProvider;
10			use ApiPlatform\Metadata\Operation;
11			use ApiPlatform\State\Pagination\PartialPaginatorInterface;
12			use ApiPlatform\State\ProviderInterface;
13			use Chamilo\CoreBundle\Entity\ResourceLink;
14			use Chamilo\CoreBundle\Entity\User;
15			use Chamilo\CoreBundle\Settings\SettingsManager;
16			use Chamilo\CoreBundle\Tool\ToolChain;
17			use Chamilo\CoreBundle\Traits\CourseFromRequestTrait;
18			use Chamilo\CourseBundle\Entity\CTool;
19			use Doctrine\ORM\EntityManagerInterface;
20			use Symfony\Component\HttpFoundation\RequestStack;
21			use Symfony\Component\Security\Core\Security;
22
23			/**
24			* @template-implements ProviderInterface<CTool>
25			*/
26			class CToolProvider implements ProviderInterface
27			{
28			use CourseFromRequestTrait;
29
30			public function __construct(
31			private readonly CollectionProvider $provider,
32			protected EntityManagerInterface $entityManager,
33			private readonly SettingsManager $settingsManager,
34			private readonly Security $security,
35			private readonly ToolChain $toolChain,
36			protected RequestStack $requestStack,
37			) {}
38
39			public function provide(Operation $operation, array $uriVariables = [], array $context = []): array
40			{
41			/** @var PartialPaginatorInterface $result */
42			$result = $this->provider->provide($operation, $uriVariables, $context);
43
44			$request = $this->requestStack->getMainRequest();
45
46			$studentView = $request ? $request->getSession()->get('studentview') : 'studentview';
47
48			/** @var User\|null $user */
49			$user = $this->security->getUser();
50
51			$isAllowToEdit = $user && ($user->hasRole('ROLE_ADMIN') \|\| $user->hasRole('ROLE_CURRENT_COURSE_TEACHER'));
52			$isAllowToEditBack = $user && ($user->hasRole('ROLE_ADMIN') \|\| $user->hasRole('ROLE_CURRENT_COURSE_TEACHER'));
53			$isAllowToSessionEdit = $user && ($user->hasRole('ROLE_ADMIN') \|\| $user->hasRole('ROLE_CURRENT_COURSE_TEACHER') \|\| $user->hasRole('ROLE_CURRENT_COURSE_SESSION_TEACHER'));
54
55			$allowVisibilityInSession = $this->settingsManager->getSetting('course.allow_edit_tool_visibility_in_session');
56			$session = $this->getSession();
57
58			$results = [];
59
60			/** @var CTool $cTool */
61			foreach ($result as $cTool) {
62			$toolModel = $this->toolChain->getToolFromName(
63			$cTool->getTool()->getName()
64			);
65
66			if (!$isAllowToEdit && 'admin' === $toolModel->getCategory()) {
67			continue;
68			}
69
70			$resourceLinks = $cTool->getResourceNode()->getResourceLinks();
71
72			if ($session && $allowVisibilityInSession) {
73			$sessionLink = $resourceLinks->findFirst(
74			fn (int $key, ResourceLink $resourceLink): bool => $resourceLink->getSession()?->getId() === $session->getId()
75			);
76
77			if ($sessionLink) {
78			// Set the session link as unique to include in repsonse
79			$resourceLinks->clear();
80			$resourceLinks->add($sessionLink);
81
82			$isAllowToEdit = $isAllowToSessionEdit;
83			} else {
84			$isAllowToEdit = $isAllowToEditBack;
85			}
86			}
87
88			if (!$isAllowToEdit \|\| 'studentview' === $studentView) {
89			$notPublishedLink = ResourceLink::VISIBILITY_PUBLISHED !== $resourceLinks->first()->getVisibility();
90
91			if ($notPublishedLink) {
92			continue;
93			}
94			}
95
96			$results[] = $cTool;
97			}
98
99			return $results;
100			}
101			}
102

chamilo / chamilo-lms

Push — master ( ca77e6...fd95ee )

CToolProvider::provide() F

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like