SessionController::aboutAction() - Code Metrics - Inspection of "Upgrade" - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#5329)

by Angel Fernando Quiroz

created 2024-04-12 17:31 UTC

SessionController::aboutAction() F

↳ Parent: Project

Complexity

Conditions	19
Paths	216

Size

Total Lines	219
Code Lines	128

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	19
eloc	128
c	0
b	0
f	0
nc	216
nop	5
dl	0
loc	219
rs	2.7866

How to fix Long Method Complexity

<?php

declare(strict_types=1);

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

namespace Chamilo\CoreBundle\Controller;

use BuyCoursesPlugin;
use Chamilo\CoreBundle\Entity\ExtraField;
use Chamilo\CoreBundle\Entity\SequenceResource;
use Chamilo\CoreBundle\Entity\Session;
use Chamilo\CoreBundle\Entity\SessionRelCourse;
use Chamilo\CoreBundle\Entity\Tag;
use Chamilo\CoreBundle\Entity\User;
use Chamilo\CoreBundle\Framework\Container;
use Chamilo\CoreBundle\Repository\Node\IllustrationRepository;
use Chamilo\CoreBundle\Repository\Node\UserRepository;
use Chamilo\CoreBundle\Repository\SequenceRepository;
use Chamilo\CoreBundle\Repository\TagRepository;
use Chamilo\CourseBundle\Entity\CCourseDescription;
use CourseDescription;
use Doctrine\ORM\EntityManagerInterface;
use Doctrine\ORM\EntityRepository;
use Essence\Essence;
use ExtraFieldValue;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Entity;
use SessionManager;
use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\Routing\Attribute\Route;
use UserManager;

#[Route('/sessions')]
class SessionController extends AbstractController
{
    /**
     * @Entity("session", expr="repository.find(sid)")
     */
    #[Route(path: '/{sid}/about', name: 'chamilo_core_session_about')]
    public function about(
        Request $request,
        Session $session,
        IllustrationRepository $illustrationRepo,
        UserRepository $userRepo,
        EntityManagerInterface $em
    ): Response {
        $requestSession = $request->getSession();
        $htmlHeadXtra[] = api_get_asset('readmore-js/readmore.js');


        $sessionId = $session->getId();
        $courses = [];
        $sessionCourses = $session->getCourses();

        /** @var EntityRepository $fieldsRepo */
        $fieldsRepo = $em->getRepository(ExtraField::class);

        /** @var TagRepository $tagRepo */
        $tagRepo = $em->getRepository(Tag::class);

        /** @var SequenceRepository $sequenceResourceRepo */
        $sequenceResourceRepo = $em->getRepository(SequenceResource::class);

        /** @var ExtraField $tagField */
        $tagField = $fieldsRepo->findOneBy([
            'itemType' => ExtraField::COURSE_FIELD_TYPE,
            'variable' => 'tags',
        ]);

        $courseValues = new ExtraFieldValue('course');
        $userValues = new ExtraFieldValue('user');
        $sessionValues = new ExtraFieldValue('session');

        /** @var SessionRelCourse $sessionRelCourse */
        foreach ($sessionCourses as $sessionRelCourse) {
            $sessionCourse = $sessionRelCourse->getCourse();
            $courseTags = [];

            if (null !== $tagField) {
                $courseTags = $tagRepo->getTagsByItem($tagField, $sessionCourse->getId());
            }

            $courseCoaches = $userRepo->getCoachesForSessionCourse($session, $sessionCourse);
            $coachesData = [];

            /** @var User $courseCoach */
            foreach ($courseCoaches as $courseCoach) {
                $coachData = [
                    'complete_name' => UserManager::formatUserFullName($courseCoach),
                    'image' => $illustrationRepo->getIllustrationUrl($courseCoach),
                    'diploma' => $courseCoach->getDiplomas(),
                    'openarea' => $courseCoach->getOpenarea(),
                    'extra_fields' => $userValues->getAllValuesForAnItem(
                        $courseCoach->getId(),
                        null,
                        true
                    ),
                ];

                $coachesData[] = $coachData;
            }

            $cd = new CourseDescription();
            $cd->set_course_id($sessionCourse->getId());
            $cd->set_session_id($session->getId());
            $descriptionsData = $cd->get_description_data();

            $courseDescription = [];
            $courseObjectives = [];
            $courseTopics = [];
            $courseMethodology = [];
            $courseMaterial = [];
            $courseResources = [];
            $courseAssessment = [];
            $courseCustom = [];

            if (!empty($descriptionsData)) {
                foreach ($descriptionsData as $descriptionInfo) {
                    $type = $descriptionInfo->getDescriptionType();

                    switch ($type) {
                        case CCourseDescription::TYPE_DESCRIPTION:
                            $courseDescription[] = $descriptionInfo;

                            break;

                        case CCourseDescription::TYPE_OBJECTIVES:
                            $courseObjectives[] = $descriptionInfo;

                            break;

                        case CCourseDescription::TYPE_TOPICS:
                            $courseTopics[] = $descriptionInfo;

                            break;

                        case CCourseDescription::TYPE_METHODOLOGY:
                            $courseMethodology[] = $descriptionInfo;

                            break;

                        case CCourseDescription::TYPE_COURSE_MATERIAL:
                            $courseMaterial[] = $descriptionInfo;

                            break;

                        case CCourseDescription::TYPE_RESOURCES:
                            $courseResources[] = $descriptionInfo;

                            break;

                        case CCourseDescription::TYPE_ASSESSMENT:
                            $courseAssessment[] = $descriptionInfo;

                            break;

                        case CCourseDescription::TYPE_CUSTOM:
                            $courseCustom[] = $descriptionInfo;

                            break;
                    }
                }
            }

            $courses[] = [
                'course' => $sessionCourse,
                'description' => $courseDescription,
                'image' => Container::getIllustrationRepository()->getIllustrationUrl($sessionCourse),
                'tags' => $courseTags,
                'objectives' => $courseObjectives,
                'topics' => $courseTopics,
                'methodology' => $courseMethodology,
                'material' => $courseMaterial,
                'resources' => $courseResources,
                'assessment' => $courseAssessment,
                'custom' => array_reverse($courseCustom),
                'coaches' => $coachesData,
                'extra_fields' => $courseValues->getAllValuesForAnItem(
                    $sessionCourse->getId(),
                    null,
                    true
                ),
            ];
        }

        $sessionDates = SessionManager::parseSessionDates($session, true);

        $hasRequirements = false;
        /*$sessionRequirements = $sequenceResourceRepo->getRequirements(
            $session->getId(),
            SequenceResource::SESSION_TYPE
        );

        foreach ($sessionRequirements as $sequence) {
            if (!empty($sequence['requirements'])) {
                $hasRequirements = true;

                break;
            }
        }*/

        $plugin = BuyCoursesPlugin::create();
        $checker = $plugin->isEnabled();
        $sessionIsPremium = null;
        if ($checker) {
            $sessionIsPremium = $plugin->getItemByProduct(
                $sessionId,
                BuyCoursesPlugin::PRODUCT_TYPE_SESSION
            );
            if ([] !== $sessionIsPremium) {
                $requestSession->set('SessionIsPremium', true);
                $requestSession->set('sessionId', $sessionId);
            }
        }

        $redirectToSession = ('true' === Container::getSettingsManager()->getSetting('session.allow_redirect_to_session_after_inscription_about'));
        $redirectToSession = $redirectToSession ? '?s='.$sessionId : false;

        $coursesInThisSession = SessionManager::get_course_list_by_session_id($sessionId);
        $coursesCount = \count($coursesInThisSession);
        $redirectToSession = 1 === $coursesCount && $redirectToSession
            ? ($redirectToSession.'&cr='.array_values($coursesInThisSession)[0]['directory'])
            : $redirectToSession;

        $essence = new Essence();

        $params = [
            'session' => $session,
            'redirect_to_session' => $redirectToSession,
            'courses' => $courses,
            'essence' => $essence,
            'session_extra_fields' => $sessionValues->getAllValuesForAnItem($session->getId(), null, true),
            'has_requirements' => $hasRequirements,
            // 'sequences' => $sessionRequirements,
            'is_premium' => $sessionIsPremium,
            'show_tutor' => 'true' === api_get_setting('show_session_coach'),
            'page_url' => api_get_path(WEB_PATH).sprintf('sessions/%s/about/', $session->getId()),
            'session_date' => $sessionDates,
            'is_subscribed' => SessionManager::isUserSubscribedAsStudent(
                $session->getId(),
                api_get_user_id()
            ),
            /*'subscribe_button' => \CoursesAndSessionsCatalog::getRegisteredInSessionButton(
                $session->getId(),
                $session->getName(),
                $hasRequirements,
                true,
                true
            ),*/
            'user_session_time' => SessionManager::getDayLeftInSession(
                [
                    'id' => $session->getId(),
                    'duration' => $session->getDuration(),
                ],
                api_get_user_id()
            ),
        ];

        return $this->render('@ChamiloCore/Session/about.html.twig', $params);
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			/* For licensing terms, see /license.txt */
6
7			namespace Chamilo\CoreBundle\Controller;
8
9			use BuyCoursesPlugin;
10			use Chamilo\CoreBundle\Entity\ExtraField;
11			use Chamilo\CoreBundle\Entity\SequenceResource;
12			use Chamilo\CoreBundle\Entity\Session;
13			use Chamilo\CoreBundle\Entity\SessionRelCourse;
14			use Chamilo\CoreBundle\Entity\Tag;
15			use Chamilo\CoreBundle\Entity\User;
16			use Chamilo\CoreBundle\Framework\Container;
17			use Chamilo\CoreBundle\Repository\Node\IllustrationRepository;
18			use Chamilo\CoreBundle\Repository\Node\UserRepository;
19			use Chamilo\CoreBundle\Repository\SequenceRepository;
20			use Chamilo\CoreBundle\Repository\TagRepository;
21			use Chamilo\CourseBundle\Entity\CCourseDescription;
22			use CourseDescription;
23			use Doctrine\ORM\EntityManagerInterface;
24			use Doctrine\ORM\EntityRepository;
25			use Essence\Essence;
26			use ExtraFieldValue;
27			use Sensio\Bundle\FrameworkExtraBundle\Configuration\Entity;
28			use SessionManager;
29			use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
30			use Symfony\Component\HttpFoundation\Request;
31			use Symfony\Component\HttpFoundation\Response;
32			use Symfony\Component\Routing\Attribute\Route;
33			use UserManager;
34
35			#[Route('/sessions')]
36			class SessionController extends AbstractController
37			{
38			/**
39			* @Entity("session", expr="repository.find(sid)")
40			*/
41			#[Route(path: '/{sid}/about', name: 'chamilo_core_session_about')]
42			public function about(
43			Request $request,
44			Session $session,
45			IllustrationRepository $illustrationRepo,
46			UserRepository $userRepo,
47			EntityManagerInterface $em
48			): Response {
49			$requestSession = $request->getSession();
50			$htmlHeadXtra[] = api_get_asset('readmore-js/readmore.js');
			0 ignored issues – show Comprehensibility Best Practice introduced 2019-11-29 12:55 UTC by Report Bug Copy Issue Report `$htmlHeadXtra` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$htmlHeadXtra = array();` before regardless. Loading history...
51
52			$sessionId = $session->getId();
53			$courses = [];
54			$sessionCourses = $session->getCourses();
55
56			/** @var EntityRepository $fieldsRepo */
57			$fieldsRepo = $em->getRepository(ExtraField::class);
58
59			/** @var TagRepository $tagRepo */
60			$tagRepo = $em->getRepository(Tag::class);
61
62			/** @var SequenceRepository $sequenceResourceRepo */
63			$sequenceResourceRepo = $em->getRepository(SequenceResource::class);
64
65			/** @var ExtraField $tagField */
66			$tagField = $fieldsRepo->findOneBy([
67			'itemType' => ExtraField::COURSE_FIELD_TYPE,
68			'variable' => 'tags',
69			]);
70
71			$courseValues = new ExtraFieldValue('course');
72			$userValues = new ExtraFieldValue('user');
73			$sessionValues = new ExtraFieldValue('session');
74
75			/** @var SessionRelCourse $sessionRelCourse */
76			foreach ($sessionCourses as $sessionRelCourse) {
77			$sessionCourse = $sessionRelCourse->getCourse();
78			$courseTags = [];
79
80			if (null !== $tagField) {
81			$courseTags = $tagRepo->getTagsByItem($tagField, $sessionCourse->getId());
82			}
83
84			$courseCoaches = $userRepo->getCoachesForSessionCourse($session, $sessionCourse);
85			$coachesData = [];
86
87			/** @var User $courseCoach */
88			foreach ($courseCoaches as $courseCoach) {
89			$coachData = [
90			'complete_name' => UserManager::formatUserFullName($courseCoach),
91			'image' => $illustrationRepo->getIllustrationUrl($courseCoach),
92			'diploma' => $courseCoach->getDiplomas(),
93			'openarea' => $courseCoach->getOpenarea(),
94			'extra_fields' => $userValues->getAllValuesForAnItem(
95			$courseCoach->getId(),
96			null,
97			true
98			),
99			];
100
101			$coachesData[] = $coachData;
102			}
103
104			$cd = new CourseDescription();
105			$cd->set_course_id($sessionCourse->getId());
106			$cd->set_session_id($session->getId());
107			$descriptionsData = $cd->get_description_data();
108
109			$courseDescription = [];
110			$courseObjectives = [];
111			$courseTopics = [];
112			$courseMethodology = [];
113			$courseMaterial = [];
114			$courseResources = [];
115			$courseAssessment = [];
116			$courseCustom = [];
117
118			if (!empty($descriptionsData)) {
119			foreach ($descriptionsData as $descriptionInfo) {
120			$type = $descriptionInfo->getDescriptionType();
121
122			switch ($type) {
123			case CCourseDescription::TYPE_DESCRIPTION:
124			$courseDescription[] = $descriptionInfo;
125
126			break;
127
128			case CCourseDescription::TYPE_OBJECTIVES:
129			$courseObjectives[] = $descriptionInfo;
130
131			break;
132
133			case CCourseDescription::TYPE_TOPICS:
134			$courseTopics[] = $descriptionInfo;
135
136			break;
137
138			case CCourseDescription::TYPE_METHODOLOGY:
139			$courseMethodology[] = $descriptionInfo;
140
141			break;
142
143			case CCourseDescription::TYPE_COURSE_MATERIAL:
144			$courseMaterial[] = $descriptionInfo;
145
146			break;
147
148			case CCourseDescription::TYPE_RESOURCES:
149			$courseResources[] = $descriptionInfo;
150
151			break;
152
153			case CCourseDescription::TYPE_ASSESSMENT:
154			$courseAssessment[] = $descriptionInfo;
155
156			break;
157
158			case CCourseDescription::TYPE_CUSTOM:
159			$courseCustom[] = $descriptionInfo;
160
161			break;
162			}
163			}
164			}
165
166			$courses[] = [
167			'course' => $sessionCourse,
168			'description' => $courseDescription,
169			'image' => Container::getIllustrationRepository()->getIllustrationUrl($sessionCourse),
170			'tags' => $courseTags,
171			'objectives' => $courseObjectives,
172			'topics' => $courseTopics,
173			'methodology' => $courseMethodology,
174			'material' => $courseMaterial,
175			'resources' => $courseResources,
176			'assessment' => $courseAssessment,
177			'custom' => array_reverse($courseCustom),
178			'coaches' => $coachesData,
179			'extra_fields' => $courseValues->getAllValuesForAnItem(
180			$sessionCourse->getId(),
181			null,
182			true
183			),
184			];
185			}
186
187			$sessionDates = SessionManager::parseSessionDates($session, true);
188
189			$hasRequirements = false;
190			/*$sessionRequirements = $sequenceResourceRepo->getRequirements(
191			$session->getId(),
192			SequenceResource::SESSION_TYPE
193			);
194
195			foreach ($sessionRequirements as $sequence) {
196			if (!empty($sequence['requirements'])) {
197			$hasRequirements = true;
198
199			break;
200			}
201			}*/
202
203			$plugin = BuyCoursesPlugin::create();
204			$checker = $plugin->isEnabled();
205			$sessionIsPremium = null;
206			if ($checker) {
207			$sessionIsPremium = $plugin->getItemByProduct(
208			$sessionId,
209			BuyCoursesPlugin::PRODUCT_TYPE_SESSION
210			);
211			if ([] !== $sessionIsPremium) {
212			$requestSession->set('SessionIsPremium', true);
213			$requestSession->set('sessionId', $sessionId);
214			}
215			}
216
217			$redirectToSession = ('true' === Container::getSettingsManager()->getSetting('session.allow_redirect_to_session_after_inscription_about'));
218			$redirectToSession = $redirectToSession ? '?s='.$sessionId : false;
219
220			$coursesInThisSession = SessionManager::get_course_list_by_session_id($sessionId);
221			$coursesCount = \count($coursesInThisSession);
222			$redirectToSession = 1 === $coursesCount && $redirectToSession
223			? ($redirectToSession.'&cr='.array_values($coursesInThisSession)[0]['directory'])
224			: $redirectToSession;
225
226			$essence = new Essence();
227
228			$params = [
229			'session' => $session,
230			'redirect_to_session' => $redirectToSession,
231			'courses' => $courses,
232			'essence' => $essence,
233			'session_extra_fields' => $sessionValues->getAllValuesForAnItem($session->getId(), null, true),
234			'has_requirements' => $hasRequirements,
235			// 'sequences' => $sessionRequirements,
236			'is_premium' => $sessionIsPremium,
237			'show_tutor' => 'true' === api_get_setting('show_session_coach'),
238			'page_url' => api_get_path(WEB_PATH).sprintf('sessions/%s/about/', $session->getId()),
239			'session_date' => $sessionDates,
240			'is_subscribed' => SessionManager::isUserSubscribedAsStudent(
241			$session->getId(),
242			api_get_user_id()
243			),
244			/*'subscribe_button' => \CoursesAndSessionsCatalog::getRegisteredInSessionButton(
245			$session->getId(),
246			$session->getName(),
247			$hasRequirements,
248			true,
249			true
250			),*/
251			'user_session_time' => SessionManager::getDayLeftInSession(
252			[
253			'id' => $session->getId(),
254			'duration' => $session->getDuration(),
255			],
256			api_get_user_id()
257			),
258			];
259
260			return $this->render('@ChamiloCore/Session/about.html.twig', $params);
261			}
262			}
263

chamilo / chamilo-lms

Pull Request — master (#5329)

SessionController::aboutAction() F

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like