SessionAdminController::getCourseForSessionAdmin() - Code Metrics - Inspection of "Course: Unify course cards & image loading across..." - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Passed
Pull Request — master (#6780)

unknown
created 2025-09-20 03:00 UTC
SessionAdminController::getCourseForSessionAdmin() A

↳ Parent: SessionAdminController
Complexity

Conditions	2
Paths	2
Size

Total Lines	34
Code Lines	23
Duplication

Lines	0
Ratio	0 %
Importance

Changes	2
Bugs	0	Features	0
Metric	Value
cc	2
eloc	23
c	2
b	0
f	0
nc	2
nop	1
dl	0
loc	34
rs	9.552
<?php

declare(strict_types=1);

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

namespace Chamilo\CoreBundle\Controller\Admin;

use Chamilo\CoreBundle\Controller\BaseController;
use Chamilo\CoreBundle\Entity\AccessUrlRelUser;
use Chamilo\CoreBundle\Entity\Course;
use Chamilo\CoreBundle\Entity\ExtraField;
use Chamilo\CoreBundle\Entity\GradebookCertificate;
use Chamilo\CoreBundle\Entity\SessionRelCourseRelUser;
use Chamilo\CoreBundle\Entity\SessionRelUser;
use Chamilo\CoreBundle\Helpers\AccessUrlHelper;
use Chamilo\CoreBundle\Helpers\UserHelper;
use Chamilo\CoreBundle\Repository\ExtraFieldRepository;
use Chamilo\CoreBundle\Repository\ExtraFieldValuesRepository;
use Chamilo\CoreBundle\Repository\GradebookCertificateRepository;
use Chamilo\CoreBundle\Repository\Node\CourseRepository;
use Chamilo\CoreBundle\Repository\Node\IllustrationRepository;
use Chamilo\CoreBundle\Repository\Node\UserRepository;
use Chamilo\CoreBundle\Repository\SessionRelCourseRelUserRepository;
use Chamilo\CoreBundle\Settings\SettingsManager;
use Chamilo\CourseBundle\Repository\CCourseDescriptionRepository;
use DateTime;
use Doctrine\ORM\EntityManagerInterface;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Security;
use Symfony\Component\HttpFoundation\JsonResponse;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\Routing\Annotation\Route;

use function pathinfo;

use const PATHINFO_FILENAME;

#[Security("is_granted('ROLE_ADMIN') or is_granted('ROLE_SESSION_MANAGER')")]
#[Route('/admin/sessionadmin')]
class SessionAdminController extends BaseController
{
    public function __construct(
        private readonly CourseRepository $courseRepository,
        private readonly AccessUrlHelper $accessUrlHelper,
        private readonly SettingsManager $settingsManager,
        private readonly UserHelper $userHelper,
        private readonly CCourseDescriptionRepository $courseDescriptionRepository,
        private readonly IllustrationRepository $illustrationRepository
    ) {}

    #[Route('/courses', name: 'chamilo_core_admin_sessionadmin_courses', methods: ['GET'])]
    public function listCourses(): JsonResponse
    {
        $url = $this->accessUrlHelper->getCurrent();
        $courses = $this->courseRepository->getCoursesByAccessUrl($url);

        $filterForCards = '';

        $data = array_map(function (Course $course) use ($filterForCards) {
            $illustrationUrl = $this->illustrationRepository->getIllustrationUrl(
                $course,
                $filterForCards,
                512
            );

            return [
                'id' => $course->getId(),
                'title' => $course->getTitle(),
                'code' => $course->getCode(),
                'description' => $course->getDescription(),
                'visibility' => $course->getVisibility(),
                'illustrationUrl' => $this->normalizePath($illustrationUrl),
            ];
        }, $courses);

        return $this->json($data);
    }

    private function normalizePath(?string $path): string
    {
        if (!\is_string($path) || $path === '') {
            return '/img/session_default.svg';
        }
        $p = trim($path);
        if (\str_starts_with($p, 'http://') || \str_starts_with($p, 'https://') || \str_starts_with($p, '/')) {
            return $p;
        }
        return '/'.ltrim($p, '/');
    }

    #[Route('/courses/completed', name: 'chamilo_core_admin_sessionadmin_courses_completed', methods: ['GET'])]
    public function listCompleted(
        Request $request,
        GradebookCertificateRepository $repo,
        AccessUrlHelper $accessUrlHelper,
    ): JsonResponse {
        // Extract and validate pagination parameters from the query
        $offset = max(0, (int) $request->query->get('offset', 0));
        $limit = max(1, (int) $request->query->get('limit', 50));

        // Determine current access URL context
        $url = $accessUrlHelper->getCurrent();

        // Retrieve certificates with associated session and course context
        $certs = $repo->findCertificatesWithContext($url->getId(), $offset, $limit);

        $user = $this->userHelper->getCurrent();

        $allowPublic = 'true' === $this->settingsManager->getSetting('course.allow_public_certificates', true);
        $allowSessionAdmin = 'true' === $this->settingsManager->getSetting('certificate.session_admin_can_download_all_certificates', true);
        $isSessionAdmin = $user && $user->isSessionAdmin();

        // Transform the certificate entities into a frontend-friendly structure
        $mapCertificate = function (GradebookCertificate $gc) use (
            $allowPublic,
            $allowSessionAdmin,
            $isSessionAdmin,
            $user
        ) {
            $sessionRel = $gc->getCategory()->getCourse()->getSessions()[0] ?? null;
            $session = $sessionRel?->getSession();
            $path = $gc->getPathCertificate();

            $hash = null;
            $downloadUrl = null;
            $isDownloadAllowed = false;

            if (!empty($path)) {
                $hash = pathinfo($path, PATHINFO_FILENAME);
                $downloadUrl = '/certificates/'.$hash.'.pdf';
                $downloadUrl = '/certificates/'./** @scrutinizer ignore-type */ $hash.'.pdf';

                $isPlatformAdmin = $user && $user->isAdmin();
                $isPublic = $allowPublic && $gc->getPublish();

                $isDownloadAllowed = $isPlatformAdmin
                    || $isPublic
                    || ($isSessionAdmin && $allowSessionAdmin);
            }

            return [
                'id' => $gc->getId(),
                'issuedAt' => $gc->getCreatedAt()->format('Y-m-d H:i:s'),
                'user' => [
                    'id' => $gc->getUser()->getId(),
                    'name' => $gc->getUser()->getFullName(),
                ],
                'course' => [
                    'id' => $gc->getCategory()->getCourse()->getId(),
                    'title' => $gc->getCategory()->getCourse()->getTitle(),
                ],
                'session' => $session ? [
                    'id' => $session->getId(),
                    'title' => $session->getTitle(),
                ] : null,
                'downloadUrl' => $downloadUrl,
                'isDownloadAllowed' => $isDownloadAllowed,
            ];
        };

        $items = array_map($mapCertificate, $certs);

        // Return JSON response with pagination metadata and certificate list
        return $this->json([
            'items' => $items,
            'offset' => $offset,
            'limit' => $limit,
            'count' => \count($items),
        ]);
    }

    #[Route('/courses/incomplete', name: 'chamilo_core_admin_sessionadmin_courses_incomplete', methods: ['GET'])]
    public function listIncomplete(
        GradebookCertificateRepository $repo,
        AccessUrlHelper $accessUrlHelper
    ): JsonResponse {
        $url = $accessUrlHelper->getCurrent();
        $results = $repo->findIncompleteCertificates($url->getId());

        $items = array_map(function (SessionRelUser $sru) {
            $user = $sru->getUser();
            $session = $sru->getSession();

            $courses = $session->getCourses();
            $courseItems = [];

            foreach ($courses as $src) {
                $course = $src->getCourse();

                $courseItems[] = [
                    'user' => [
                        'id' => $user->getId(),
                        'name' => $user->getFullName(),
                    ],
                    'course' => [
                        'id' => $course->getId(),
                        'title' => $course->getTitle(),
                    ],
                    'session' => [
                        'id' => $session->getId(),
                        'title' => $session->getTitle(),
                        'startDate' => $session->getAccessStartDate()?->format('Y-m-d'),
                        'endDate' => $session->getAccessEndDate()?->format('Y-m-d'),
                    ],
                ];
            }

            return $courseItems;
        }, $results);

        $flatItems = array_merge(...$items);

        return $this->json([
            'items' => $flatItems,
            'count' => \count($flatItems),
        ]);
    }

    #[Route('/courses/restartable', name: 'chamilo_core_admin_sessionadmin_courses_restartable', methods: ['GET'])]
    public function listRestartables(
        Request $request,
        GradebookCertificateRepository $repo,
        AccessUrlHelper $accessUrlHelper
    ): JsonResponse {
        $offset = max(0, (int) $request->query->get('offset', 0));
        $limit = max(1, (int) $request->query->get('limit', 10));

        $urlId = $accessUrlHelper->getCurrent()->getId();

        /** @var SessionRelCourseRelUser[] $rows */
        $rows = $repo->findRestartableSessions($urlId, $offset, $limit);

        $items = array_map(static function (SessionRelCourseRelUser $srcu) {
            $session = $srcu->getSession();
            $course = $srcu->getCourse();
            $user = $srcu->getUser();

            return [
                'user' => ['id' => $user->getId(), 'name' => $user->getFullName()],
                'course' => ['id' => $course->getId(), 'title' => $course->getTitle()],
                'session' => [
                    'id' => $session->getId(),
                    'title' => $session->getTitle(),
                    'endDate' => $session->getAccessEndDate()?->format('Y-m-d'),
                ],
            ];
        }, $rows);

        return $this->json([
            'items' => $items,
            'offset' => $offset,
            'limit' => $limit,
            'count' => \count($items),
        ]);
    }

    #[Route('/courses/extend_week', name: 'chamilo_core_admin_sessionadmin_session_extend_one_week', methods: ['POST'])]
    public function extendSessionByWeek(
        Request $request,
        SessionRelCourseRelUserRepository $sessionRelCourseRelUserRepository,
        EntityManagerInterface $entityManager
    ): JsonResponse {
        $data = json_decode($request->getContent(), true);
        $sessionId = (int) ($data['sessionId'] ?? 0);
        $userId = (int) ($data['userId'] ?? 0);
        $courseId = (int) ($data['courseId'] ?? 0);

        if (!$sessionId || !$userId || !$courseId) {
            return $this->json(['error' => 'Missing data'], Response::HTTP_BAD_REQUEST);
        }

        $rel = $sessionRelCourseRelUserRepository->findOneBy([
            'session' => $sessionId,
            'user' => $userId,
            'course' => $courseId,
        ]);

        if (!$rel) {
            return $this->json(['error' => 'Relation not found'], Response::HTTP_NOT_FOUND);
        }

        $session = $rel->getSession();

        $now = new DateTime('now');
        $currentEndDate = $session->getAccessEndDate();

        $baseDate = $now > $currentEndDate ? $now : $currentEndDate;

        $newEndDate = clone $baseDate;
        $newEndDate->modify('+1 week');

        $session->setAccessEndDate($newEndDate);
        $entityManager->flush();

        return $this->json([
            'success' => true,
            'message' => 'Session extended by one week.',
            'newEndDate' => $newEndDate->format('Y-m-d'),
        ]);
    }

    #[Route('/courses/{id}', name: 'chamilo_core_admin_sessionadmin_course_view', methods: ['GET'])]
    public function getCourseForSessionAdmin(int $id): JsonResponse
    {
        /** @var Course $course */
        $course = $this->courseRepository->find($id);

        if (!$course) {

            return $this->json(['error' => 'Course not found'], Response::HTTP_NOT_FOUND);
        }

        $qb = $this->courseDescriptionRepository->getResourcesByCourse($course);
        $items = $qb->getQuery()->getResult();

        $descriptions = array_map(static function ($d) {
            return [
                'title' => $d->getTitle(),
                'content' => $d->getContent(),
            ];
        }, $items);

        $filterForCards = '';
        $illustrationUrl = $this->illustrationRepository->getIllustrationUrl(
            $course,
            $filterForCards,
            512
        );

        return $this->json([
            'id' => $course->getId(),
            'title' => $course->getTitle(),
            'code' => $course->getCode(),
            'description' => '',
            'descriptions' => $descriptions,
            'illustrationUrl' => $this->normalizePath($illustrationUrl),
        ]);
    }

    #[Route('/users', name: 'chamilo_core_admin_sessionadmin_search_users', methods: ['GET'])]
    public function searchUsers(
        Request $request,
        UserRepository $userRepo,
        ExtraFieldRepository $extraFieldRepo,
        ExtraFieldValuesRepository $extraFieldValuesRepo,
        AccessUrlHelper $accessUrlHelper,
        SettingsManager $settingsManager,
    ): JsonResponse {
        $lastname = $request->query->get('lastname');
        $firstname = $request->query->get('firstname');
        $extraFilters = $request->query->all('extraFilters');

        $configuredExtraFieldVariable = $settingsManager->getSetting(
            'platform.session_admin_user_subscription_search_extra_field_to_search',
            true
        );

        $allUrlsAllowed = 'true' === $settingsManager->getSetting('platform.session_admin_access_to_all_users_on_all_urls', true);

        $filters = [];
        if ($lastname) {
            $filters['lastname'] = $lastname;
        }
        if ($firstname) {
            $filters['firstname'] = $firstname;
        }
        if ($configuredExtraFieldVariable && !empty($extraFilters[$configuredExtraFieldVariable])) {
            $filters['extraFilters'] = [
                $configuredExtraFieldVariable => $extraFilters[$configuredExtraFieldVariable],
            ];
        }

        $currentUrl = $accessUrlHelper->getCurrent();
        $accessUrl = $allUrlsAllowed ? null : $currentUrl;

        $users = $userRepo->findUsersForSessionAdmin(
            $filters['lastname'] ?? null,
            $filters['firstname'] ?? null,
            $filters['extraFilters'] ?? [],
            $accessUrl
        );

        $data = [];
        foreach ($users as $user) {
            $extra = [];
            $extraValue = $extraFieldValuesRepo->getValueByVariableAndItem(
                $configuredExtraFieldVariable,
                $user->getId(),
                ExtraField::USER_FIELD_TYPE
            );

            $extra[$configuredExtraFieldVariable] = $extraValue?->getFieldValue();

            if (!$allUrlsAllowed) {
                $localAccess = true;
            } else {
                $localAccess = $user->getPortals()->exists(
                    fn ($key, AccessUrlRelUser $rel) => $rel->getUrl() === $currentUrl
                );
            }

            $data[] = [
                'id' => $user->getId(),
                'lastname' => $user->getLastname(),
                'firstname' => $user->getFirstname(),
                'fullname' => trim($user->getFirstname().' '.$user->getLastname()),
                'email' => $user->getEmail(),
                'isActive' => $user->getIsActive(),
                'hasLocalAccess' => $localAccess,
                'extra' => $extra,
            ];
        }

        return $this->json([
            'items' => $data,
        ]);
    }
}

chamilo / chamilo-lms

Pull Request — master (#6780)

SessionAdminController::getCourseForSessionAdmin() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like
