Minor: Format code

src/CoreBundle/ChamiloCoreBundle.php

@@ -8,6 +8,4 @@ namespace Chamilo\CoreBundle;
 
 use Symfony\Component\HttpKernel\Bundle\Bundle;
 
-class ChamiloCoreBundle extends Bundle
+class ChamiloCoreBundle extends Bundle {}
-{
-}

src/CoreBundle/Command/UpdateVueTranslations.php

@@ -14,6 +14,8 @@ use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\DependencyInjection\ParameterBag\ParameterBagInterface;
 use Symfony\Contracts\Translation\TranslatorInterface;
 
+use const JSON_PRETTY_PRINT;
+
 class UpdateVueTranslations extends Command
 {
     /**
@@ -88,12 +90,13 @@ class UpdateVueTranslations extends Command
     /**
      * Gets the translation for a given variable with fallbacks to parent language and base language.
      *
-     * @param string $variable The variable to be translated.
+     * @param string   $variable the variable to be translated
-     * @param Language $language The Language entity for the current language.
+     * @param Language $language the Language entity for the current language
      *
-     * @return string The translated string.
+     * @return string the translated string
      */
-    private function getTranslationWithFallback(string $variable, Language $language): string {
+    private function getTranslationWithFallback(string $variable, Language $language): string
+    {
         // Get the ISO code of the current language
         $iso = $language->getIsocode();
         // Try to translate the variable in the current language

src/CoreBundle/Component/Editor/CkEditor/Toolbar/TestFreeAnswerStrict.php

@@ -39,6 +39,7 @@ class TestFreeAnswerStrict extends Basic
         ];
 
         $config['removePlugins'] = 'elementspath';
+
         // $config['height'] = '200';
         return $config;
     }

src/CoreBundle/Component/Editor/CkEditor/Toolbar/ToolbarStartExpanded.php

@@ -6,6 +6,4 @@ declare(strict_types=1);
 
 namespace Chamilo\CoreBundle\Component\Editor\CkEditor\Toolbar;
 
-class ToolbarStartExpanded extends Basic
+class ToolbarStartExpanded extends Basic {}
-{
-}

src/CoreBundle/Component/Editor/Editor.php

@@ -87,6 +87,7 @@ class Editor
 
     /**
      * @param string $key
+     * @param mixed  $value
      */
     public function setConfigAttribute($key, $value): void
     {
@@ -113,25 +114,30 @@ class Editor
                         $this->toolbarSet = $value;
 
                         break;
+
                     case 'Config':
                         $this->processConfig($value);
 
                         break;
+
                     case 'width':
                     case 'Width':
                         $this->setConfigAttribute('width', $value);
 
                         break;
+
                     case 'height':
                     case 'Height':
                         $this->setConfigAttribute('height', $value);
 
                         break;
+
                     case 'FullPage':
                     case 'fullPage':
                         $this->setConfigAttribute('fullPage', $value);
 
                         break;
+
                     default:
                         $this->setConfigAttribute($key, $value);
 
@@ -158,9 +164,11 @@ class Editor
         switch (\gettype($var)) {
             case 'boolean':
                 return $var ? 'true' : 'false'; // Lowercase necessary!
+
             case 'integer':
             case 'double':
                 return (string) $var;
+
             case 'resource':
             case 'string':
                 return '"'.str_replace(
@@ -170,6 +178,7 @@ class Editor
                 ).'"';
 
                 break;
+
             case 'array':
                 // Arrays in JSON can't be associative. If the array is empty or if it
                 // has sequential whole number keys starting with 0, it's not associative
@@ -182,6 +191,7 @@ class Editor
 
                     return '[ '.implode(', ', $output).' ]';
                 }
+
                 // no break
             case 'object':
                 // Otherwise, fall through to convert the array as an object.
@@ -193,6 +203,7 @@ class Editor
                 return '{ '.implode(', '."\n", $output).' } '."\n";
 
                 break;
+
             default:
                 return 'null';
         }

src/CoreBundle/Component/Utils/ChamiloApi.php

@@ -13,9 +13,10 @@ use DateTime;
 use DateTimeZone;
 use Display;
 use Exception;
-use const PHP_SAPI;
 use Template;
 
+use const PHP_SAPI;
+
 class ChamiloApi
 {
     public const COURSE_MANAGER = 1;
@@ -245,7 +246,7 @@ class ChamiloApi
      *
      * @throws Exception
      */
-    public static function getCourseIdByDirectory(?string $directory = null): int
+    public static function getCourseIdByDirectory(string $directory = null): int
     {
         if (!empty($directory)) {
             $directory = Database::escape_string($directory);
@@ -339,7 +340,7 @@ class ChamiloApi
     public static function getColorPalette(
         bool $decimalOpacity = false,
         bool $wrapInRGBA = false,
-        ?int $fillUpTo = null
+        int $fillUpTo = null
     ): array {
         // Get the common colors from the palette used for pchart
         $paletteFile = api_get_path(SYS_CODE_PATH).'palettes/pchart/default.color';
@@ -379,7 +380,7 @@ class ChamiloApi
      *
      * @throws Exception
      */
-    public static function getServerMidnightTime(?string $utcTime = null): DateTime
+    public static function getServerMidnightTime(string $utcTime = null): DateTime
     {
         $localTime = api_get_local_time($utcTime);
         $localTimeZone = api_get_timezone();

src/CoreBundle/Component/Utils/GlideAsset.php

@@ -6,6 +6,4 @@ declare(strict_types=1);
 
 namespace Chamilo\CoreBundle\Component\Utils;
 
-class GlideAsset extends Glide
+class GlideAsset extends Glide {}
-{
-}

src/CoreBundle/Controller/AbstractResourceController.php

@@ -6,6 +6,4 @@ declare(strict_types=1);
 
 namespace Chamilo\CoreBundle\Controller;
 
-abstract class AbstractResourceController extends BaseController
+abstract class AbstractResourceController extends BaseController {}
-{
-}

src/CoreBundle/Controller/Admin/IndexBlocksController.php

@@ -19,7 +19,7 @@ class IndexBlocksController extends BaseController
 {
     private bool $isAdmin = false;
     private bool $isSessionAdmin = false;
-    private $extAuthSource = [];
+    private array $extAuthSource = [];
 
     public function __construct(
         private readonly TranslatorInterface $translator,

src/CoreBundle/Controller/Api/BaseResourceFileAction.php

@@ -150,7 +150,7 @@ class BaseResourceFileAction
             $parentResourceNodeId = (int) $request->get('parentResourceNodeId');
             $resourceLinkList = $request->get('resourceLinkList', []);
             if (!empty($resourceLinkList)) {
-                $resourceLinkList = false === strpos($resourceLinkList, '[') ? json_decode('['.$resourceLinkList.']', true) : json_decode($resourceLinkList, true);
+                $resourceLinkList = !str_contains($resourceLinkList, '[') ? json_decode('['.$resourceLinkList.']', true) : json_decode($resourceLinkList, true);
                 if (empty($resourceLinkList)) {
                     $message = 'resourceLinkList is not a valid json. Use for example: [{"cid":1, "visibility":1}]';
 
@@ -200,7 +200,7 @@ class BaseResourceFileAction
             $fileType = $request->get('filetype');
             $resourceLinkList = $request->get('resourceLinkList', []);
             if (!empty($resourceLinkList)) {
-                $resourceLinkList = false === strpos($resourceLinkList, '[') ? json_decode('['.$resourceLinkList.']', true) : json_decode($resourceLinkList, true);
+                $resourceLinkList = !str_contains($resourceLinkList, '[') ? json_decode('['.$resourceLinkList.']', true) : json_decode($resourceLinkList, true);
                 if (empty($resourceLinkList)) {
                     $message = 'resourceLinkList is not a valid json. Use for example: [{"cid":1, "visibility":1}]';
 
@@ -251,6 +251,7 @@ class BaseResourceFileAction
                 }
 
                 break;
+
             case 'folder':
                 break;
         }

src/CoreBundle/Controller/Api/ExportCGlossaryAction.php

@@ -62,10 +62,13 @@ class ExportCGlossaryAction
         switch ($format) {
             case 'csv':
                 return $this->generateCsvFile($glossaryItems, $exportPath);
+
             case 'xls':
                 return $this->generateExcelFile($glossaryItems, $exportPath);
+
             case 'pdf':
                 return $this->generatePdfFile($glossaryItems, $exportPath, $translator);
+
             default:
                 throw new NotSupported('Export format not supported');
         }
@@ -75,6 +78,7 @@ class ExportCGlossaryAction
     {
         $csvFilePath = $exportPath.'/glossary.csv';
         $csvContent = '';
+
         /** @var CGlossary $item */
         foreach ($glossaryItems as $item) {
             $csvContent .= $item->getName().','.$item->getDescription()."\n";
@@ -89,6 +93,7 @@ class ExportCGlossaryAction
         $excelFilePath = $exportPath.'/glossary.xlsx';
         $spreadsheet = new Spreadsheet();
         $sheet = $spreadsheet->getActiveSheet();
+
         /** @var CGlossary $item */
         foreach ($glossaryItems as $index => $item) {
             $row = $index + 1;
@@ -111,6 +116,7 @@ class ExportCGlossaryAction
         $html = '<h1>'.$translator->trans('Glossary').'</h1>';
         $html .= '<table>';
         $html .= '<tr><th>'.$translator->trans('Term').'</th><th>'.$translator->trans('Definition').'</th></tr>';
+
         /** @var CGlossary $item */
         foreach ($glossaryItems as $item) {
             $html .= '<tr>';

src/CoreBundle/Controller/Api/ExportGlossaryToDocumentsAction.php

@@ -68,6 +68,7 @@ class ExportGlossaryToDocumentsAction
         $html = '<h1>'.$translator->trans('Glossary').'</h1>';
         $html .= '<table>';
         $html .= '<tr><th>'.$translator->trans('Term').'</th><th>'.$translator->trans('Definition').'</th></tr>';
+
         /** @var CGlossary $item */
         foreach ($glossaryItems as $item) {
             $html .= '<tr>';

src/CoreBundle/Controller/Api/GetLinksCollectionController.php

@@ -75,6 +75,7 @@ class GetLinksCollectionController extends BaseResourceFileAction
                 $dataResponse['categories'][$categoryId]['info'] = $categoryInfo;
                 if ($links) {
                     $items = [];
+
                     /** @var CLink $link */
                     foreach ($links as $link) {
                         $items[] = [

src/CoreBundle/Controller/Api/ImportCGlossaryAction.php

@@ -107,6 +107,7 @@ class ImportCGlossaryAction
                     ->andWhere('resource.name = :name')
                     ->setParameter('name', $termToUpdate)
                 ;
+
                 /** @var CGlossary $existingGlossaryTerm */
                 $existingGlossaryTerm = $qb->getQuery()->getOneOrNullResult();
                 if (null !== $existingGlossaryTerm) {
@@ -122,6 +123,7 @@ class ImportCGlossaryAction
                 ->andWhere('resource.name = :name')
                 ->setParameter('name', $term)
             ;
+
             /** @var CGlossary $existingNewGlossaryTerm */
             $existingNewGlossaryTerm = $qb->getQuery()->getOneOrNullResult();
             if (!$existingNewGlossaryTerm) {

src/CoreBundle/Controller/BaseController.php

@@ -10,9 +10,5 @@ use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
 
 /**
  * Each entity controller must extend this class.
- *
- * @abstract
  */
-abstract class BaseController extends AbstractController
+abstract class BaseController extends AbstractController {}
-{
-}

src/CoreBundle/Controller/ChatController.php

@@ -22,8 +22,8 @@ use Symfony\Component\Routing\Annotation\Route;
 class ChatController extends AbstractResourceController implements CourseControllerInterface
 {
     use ControllerTrait;
-    use ResourceControllerTrait;
     use CourseControllerTrait;
+    use ResourceControllerTrait;
 
     /**
      * @Route("/resources/chat/", name="chat_home", options={"expose"=true})
@@ -90,6 +90,7 @@ class ChatController extends AbstractResourceController implements CourseControl
                 Event::registerLog($logInfo);
 
                 break;
+
             case 'track':
                 $courseChatUtils->keepUserAsConnected();
                 $courseChatUtils->disconnectInactiveUsers();
@@ -114,6 +115,7 @@ class ChatController extends AbstractResourceController implements CourseControl
                 ];
 
                 break;
+
             case 'preview':
                 $json = [
                     'status' => true,
@@ -123,6 +125,7 @@ class ChatController extends AbstractResourceController implements CourseControl
                 ];
 
                 break;
+
             case 'reset':
                 $friend = isset($_REQUEST['friend']) ? (int) $_REQUEST['friend'] : 0;
 
@@ -132,6 +135,7 @@ class ChatController extends AbstractResourceController implements CourseControl
                 ];
 
                 break;
+
             case 'write':
                 $friend = isset($_REQUEST['friend']) ? (int) $_REQUEST['friend'] : 0;
                 $status = $courseChatUtils->saveMessage($_REQUEST['message'], $friend);

src/CoreBundle/Controller/CourseController.php

@@ -59,8 +59,7 @@ class CourseController extends ToolBaseController
 {
     public function __construct(
         private readonly SerializerInterface $serializer
-    ) {
+    ) {}
-    }
 
     #[Route('/{cid}/checkLegal.json', name: 'chamilo_core_course_check_legal_json')]
     public function checkTermsAndConditionJson(
@@ -78,9 +77,9 @@ class CourseController extends ToolBaseController
             'url' => '#',
         ];
 
-        if ($user && $user->hasRole('ROLE_STUDENT') &&
+        if ($user && $user->hasRole('ROLE_STUDENT')
-            'true' === $settingsManager->getSetting('allow_terms_conditions') &&
+            && 'true' === $settingsManager->getSetting('allow_terms_conditions')
-            'course' === $settingsManager->getSetting('load_term_conditions_section')
+            && 'course' === $settingsManager->getSetting('load_term_conditions_section')
         ) {
             $termAndConditionStatus = false;
             $extraValue = $extraFieldValuesRepository->findLegalAcceptByItemId($user->getId());
@@ -106,9 +105,9 @@ class CourseController extends ToolBaseController
                     ->getSetting('course.allow_public_course_with_no_terms_conditions')
                 ;
 
-                if (true === $allow &&
+                if (true === $allow
-                    null !== $course->getVisibility() &&
+                    && null !== $course->getVisibility()
-                    Course::OPEN_WORLD === $course->getVisibility()
+                    && Course::OPEN_WORLD === $course->getVisibility()
                 ) {
                     $redirect = false;
                 }
@@ -179,6 +178,7 @@ class CourseController extends ToolBaseController
         $session = $request->getSession();
 
         $userId = 0;
+
         /** @var ?User $user */
         $user = $this->getUser();
         if (null !== $user) {
@@ -206,8 +206,8 @@ class CourseController extends ToolBaseController
 
         $isSpecialCourse = CourseManager::isSpecialCourse($courseId);
 
-        if ($user && $isSpecialCourse && (isset($_GET['autoreg']) && 1 === (int) $_GET['autoreg']) &&
+        if ($user && $isSpecialCourse && (isset($_GET['autoreg']) && 1 === (int) $_GET['autoreg'])
-            CourseManager::subscribeUser($userId, $courseId, STUDENT)
+            && CourseManager::subscribeUser($userId, $courseId, STUDENT)
         ) {
             $session->set('is_allowed_in_course', true);
         }
@@ -404,10 +404,12 @@ class CourseController extends ToolBaseController
         EntityManagerInterface $em
     ): Response {
         $courseId = $course->getId();
+
         /** @var ?User $user */
         $user = $this->getUser();
 
         $fieldsRepo = $em->getRepository(ExtraField::class);
+
         /** @var TagRepository $tagRepo */
         $tagRepo = $em->getRepository(Tag::class);
 
@@ -430,6 +432,7 @@ class CourseController extends ToolBaseController
 
             $teachersData[] = $userData;
         }
+
         /** @var ExtraField $tagField */
         $tagField = $fieldsRepo->findOneBy([
             'itemType' => ExtraField::COURSE_FIELD_TYPE,
@@ -450,30 +453,37 @@ class CourseController extends ToolBaseController
                     $courseDescription = $descriptionTool->getContent();
 
                     break;
+
                 case CCourseDescription::TYPE_OBJECTIVES:
                     $courseObjectives = $descriptionTool;
 
                     break;
+
                 case CCourseDescription::TYPE_TOPICS:
                     $courseTopics = $descriptionTool;
 
                     break;
+
                 case CCourseDescription::TYPE_METHODOLOGY:
                     $courseMethodology = $descriptionTool;
 
                     break;
+
                 case CCourseDescription::TYPE_COURSE_MATERIAL:
                     $courseMaterial = $descriptionTool;
 
                     break;
+
                 case CCourseDescription::TYPE_RESOURCES:
                     $courseResources = $descriptionTool;
 
                     break;
+
                 case CCourseDescription::TYPE_ASSESSMENT:
                     $courseAssessment = $descriptionTool;
 
                     break;
+
                 case CCourseDescription::TYPE_CUSTOM:
                     $courseCustom[] = $descriptionTool;
 
@@ -600,6 +610,7 @@ class CourseController extends ToolBaseController
 
         if ($ctool) {
             $ctoolintroRepo = $em->getRepository(CToolIntro::class);
+
             /** @var CToolIntro $ctoolintro */
             $ctoolintro = $ctoolintroRepo->findOneBy(['courseTool' => $ctool]);
             if ($ctoolintro) {
@@ -654,6 +665,7 @@ class CourseController extends ToolBaseController
             $em->flush();
             if ($courseTool && !empty($ctoolintroId)) {
                 $ctoolintroRepo = Container::getToolIntroRepository();
+
                 /** @var CToolIntro $ctoolintro */
                 $ctoolintro = $ctoolintroRepo->find($ctoolintroId);
                 $ctoolintro->setCourseTool($courseTool);
@@ -705,9 +717,9 @@ class CourseController extends ToolBaseController
         $lpAutoLaunch = api_get_course_setting('enable_lp_auto_launch');
         $session_id = api_get_session_id();
         $allowAutoLaunchForCourseAdmins =
-            api_is_platform_admin() ||
+            api_is_platform_admin()
-            api_is_allowed_to_edit(true, true) ||
+            || api_is_allowed_to_edit(true, true)
-            api_is_coach();
+            || api_is_coach();
 
         if (!empty($lpAutoLaunch)) {
             if (2 === $lpAutoLaunch) {
@@ -721,6 +733,7 @@ class CourseController extends ToolBaseController
                         $url = api_get_path(WEB_CODE_PATH).'lp/lp_controller.php?'.api_get_cidreq();
                         $_SESSION[$session_key] = true;
                         header(sprintf('Location: %s', $url));
+
                         exit;
                     }
                 }
@@ -757,6 +770,7 @@ class CourseController extends ToolBaseController
 
                                 $_SESSION[$session_key] = true;
                                 header(sprintf('Location: %s', $url));
+
                                 exit;
                             }
                         }
@@ -782,11 +796,12 @@ class CourseController extends ToolBaseController
             } else {
                 $url = api_get_path(WEB_CODE_PATH).'forum/index.php?'.api_get_cidreq();
                 header(sprintf('Location: %s', $url));
+
                 exit;
             }
         }
 
-        if (('true' === api_get_setting('exercise.allow_exercise_auto_launch'))) {
+        if ('true' === api_get_setting('exercise.allow_exercise_auto_launch')) {
             $exerciseAutoLaunch = (int) api_get_course_setting('enable_exercise_auto_launch');
             if (2 === $exerciseAutoLaunch) {
                 if ($allowAutoLaunchForCourseAdmins) {
@@ -799,6 +814,7 @@ class CourseController extends ToolBaseController
                     // Redirecting to the document
                     $url = api_get_path(WEB_CODE_PATH).'exercise/exercise.php?'.api_get_cidreq();
                     header(sprintf('Location: %s', $url));
+
                     exit;
                 }
             } elseif (1 === $exerciseAutoLaunch) {
@@ -834,6 +850,7 @@ class CourseController extends ToolBaseController
                         $url = api_get_path(WEB_CODE_PATH).
                             'exercise/overview.php?exerciseId='.$exerciseId.'&'.api_get_cidreq();
                         header(sprintf('Location: %s', $url));
+
                         exit;
                     }
                 }
@@ -852,6 +869,7 @@ class CourseController extends ToolBaseController
                 // Redirecting to the document
                 $url = api_get_path(WEB_CODE_PATH).'document/document.php?'.api_get_cidreq();
                 header("Location: $url");
+
                 exit;
             }
         }

src/CoreBundle/Controller/EditorController.php

@@ -19,8 +19,8 @@ use Symfony\Contracts\Translation\TranslatorInterface;
 class EditorController extends BaseController
 {
     use ControllerTrait;
-    use ResourceControllerTrait;
     use CourseControllerTrait;
+    use ResourceControllerTrait;
 
     /**
      * Get templates (left column when creating a document).

src/CoreBundle/Controller/IndexController.php

@@ -20,16 +20,12 @@ class IndexController extends BaseController
      * @Route("/login", name="login", methods={"GET", "POST"}, options={"expose"=true})
      * @Route("/faq", name="faq", methods={"GET", "POST"}, options={"expose"=true})
      * @Route("/demo", name="demo", methods={"GET", "POST"}, options={"expose"=true})
-     *
      * @Route("/course/{cid}/home", name="chamilo_core_course_home")
      * @Route("/courses", name="courses", methods={"GET", "POST"}, options={"expose"=true})
-     *
      * @Route("/catalog/{slug}", name="catalog", methods={"GET", "POST"}, options={"expose"=true})
      * @Route("/resources/document/{nodeId}/manager", methods={"GET"}, name="resources_filemanager")
      * @Route("/account/home", name="account", options={"expose"=true}, name="chamilo_core_account_home")
-     *
      * @Route("/social", name="social", options={"expose"=true}, name="chamilo_core_socialnetwork")
-     *
      * @Route("/admin", name="admin", options={"expose"=true})
      */
     #[Route('/sessions', name: 'sessions')]
@@ -41,6 +37,8 @@ class IndexController extends BaseController
 
     /**
      * Use only in PHPUnit tests.
+     *
+     * @param mixed $name
      */
     public function classic($name): Response
     {
@@ -54,6 +52,7 @@ class IndexController extends BaseController
         $fileToLoad = $mainPath.$name;
 
         ob_start();
+
         require_once $fileToLoad;
         $content = ob_get_contents();
         ob_end_clean();

src/CoreBundle/Controller/ResourceController.php

@@ -42,9 +42,9 @@ use ZipStream\ZipStream;
 #[Route('/r')]
 class ResourceController extends AbstractResourceController implements CourseControllerInterface
 {
+    use ControllerTrait;
     use CourseControllerTrait;
     use ResourceControllerTrait;
-    use ControllerTrait;
 
     /**
      * @Route("/{tool}/{type}/{id}/disk_space", methods={"GET", "POST"}, name="chamilo_core_resource_disk_space")
@@ -372,6 +372,7 @@ class ResourceController extends AbstractResourceController implements CourseCon
                 $forceDownload = true;
 
                 break;
+
             case 'show':
             default:
                 $forceDownload = false;

src/CoreBundle/Controller/SecurityController.php

@@ -27,8 +27,7 @@ class SecurityController extends AbstractController
     public function __construct(
         private readonly SerializerInterface $serializer,
         private readonly TrackELoginRecordRepository $trackELoginRecordRepository
-    ) {
+    ) {}
-    }
 
     #[Route('/login_json', name: 'login_json', methods: ['POST'])]
     public function loginJson(Request $request, EntityManager $entityManager, SettingsManager $settingsManager, TokenStorageInterface $tokenStorage): Response
@@ -46,9 +45,9 @@ class SecurityController extends AbstractController
         $user = $this->getUser();
         $extraFieldValuesRepository = $entityManager->getRepository(ExtraFieldValues::class);
         $legalTermsRepo = $entityManager->getRepository(Legal::class);
-        if ($user->hasRole('ROLE_STUDENT') &&
+        if ($user->hasRole('ROLE_STUDENT')
-            'true' === $settingsManager->getSetting('allow_terms_conditions') &&
+            && 'true' === $settingsManager->getSetting('allow_terms_conditions')
-            'login' === $settingsManager->getSetting('load_term_conditions_section')
+            && 'login' === $settingsManager->getSetting('load_term_conditions_section')
         ) {
             $termAndConditionStatus = false;
             $extraValue = $extraFieldValuesRepository->findLegalAcceptByItemId($user->getId());

src/CoreBundle/Controller/SessionController.php

@@ -56,6 +56,7 @@ class SessionController extends AbstractController
 
         /** @var EntityRepository $fieldsRepo */
         $fieldsRepo = $em->getRepository(ExtraField::class);
+
         /** @var TagRepository $tagRepo */
         $tagRepo = $em->getRepository(Tag::class);
 
@@ -83,6 +84,7 @@ class SessionController extends AbstractController
 
             $courseCoaches = $userRepo->getCoachesForSessionCourse($session, $sessionCourse);
             $coachesData = [];
+
             /** @var User $courseCoach */
             foreach ($courseCoaches as $courseCoach) {
                 $coachData = [
@@ -117,35 +119,43 @@ class SessionController extends AbstractController
             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;
 

src/CoreBundle/Controller/ToolController.php

@@ -19,6 +19,7 @@ class ToolController extends AbstractController
      * Updates the table tool and resource_type with the content of tools.yml.
      *
      * @Security("is_granted('ROLE_ADMIN')")
+     *
      * @Route("/update", methods={"GET"})
      */
     public function profileAction(ToolChain $toolChain): Response

src/CoreBundle/DataFixtures/AccessUserUrlFixtures.php

@@ -27,6 +27,7 @@ class AccessUserUrlFixtures extends Fixture implements ContainerAwareInterface
     public function load(ObjectManager $manager): void
     {
         $container = $this->container;
+
         /** @var User $admin */
         $admin = $this->getReference(AccessUserFixtures::ADMIN_USER_REFERENCE);
         $anon = $this->getReference(AccessUserFixtures::ANON_USER_REFERENCE);

src/CoreBundle/DataProvider/Extension/CCalendarEventExtension.php

@@ -24,8 +24,7 @@ final class CCalendarEventExtension implements QueryCollectionExtensionInterface
     public function __construct(
         private readonly Security $security,
         private readonly RequestStack $requestStack
-    ) {
+    ) {}
-    }
 
     public function applyToCollection(
         QueryBuilder $queryBuilder,

src/CoreBundle/DataProvider/Extension/CDocumentExtension.php

@@ -26,8 +26,7 @@ final class CDocumentExtension implements QueryCollectionExtensionInterface //,
     public function __construct(
         private readonly Security $security,
         private readonly RequestStack $requestStack
-    ) {
+    ) {}
-    }
 
     public function applyToCollection(
         QueryBuilder $queryBuilder,

src/CoreBundle/DataProvider/Extension/CStudentPublicationExtension.php

@@ -22,8 +22,7 @@ class CStudentPublicationExtension implements QueryCollectionExtensionInterface
     public function __construct(
         private readonly Security $security,
         private readonly RequestStack $requestStack
-    ) {
+    ) {}
-    }
 
     public function applyToCollection(
         QueryBuilder $queryBuilder,

src/CoreBundle/DataProvider/Extension/CToolIntroExtension.php

@@ -24,8 +24,7 @@ final class CToolIntroExtension implements QueryCollectionExtensionInterface
     public function __construct(
         private readonly Security $security,
         private readonly RequestStack $requestStack
-    ) {
+    ) {}
-    }
 
     public function applyToCollection(
         QueryBuilder $queryBuilder,

src/CoreBundle/DataProvider/Extension/CourseExtension.php

@@ -23,8 +23,7 @@ final class CourseExtension implements QueryCollectionExtensionInterface
 {
     public function __construct(
         private readonly Security $security
-    ) {
+    ) {}
-    }
 
     public function applyToCollection(
         QueryBuilder $queryBuilder,

src/CoreBundle/DataProvider/Extension/CourseLinkExtensionTrait.php

@@ -12,8 +12,7 @@ trait CourseLinkExtensionTrait
 {
     public function __construct(
         private readonly Security $security
-    ) {
+    ) {}
-    }
 
     protected function addCourseLinkWithVisibilityConditions(
         QueryBuilder $queryBuilder,
@@ -45,8 +44,8 @@ trait CourseLinkExtensionTrait
         ;
 
         $allowDraft =
-            $this->security->isGranted('ROLE_ADMIN') ||
+            $this->security->isGranted('ROLE_ADMIN')
-            $this->security->isGranted('ROLE_CURRENT_COURSE_TEACHER');
+            || $this->security->isGranted('ROLE_CURRENT_COURSE_TEACHER');
 
         if (!$allowDraft) {
             $queryBuilder

src/CoreBundle/DataProvider/Extension/CourseRelUserExtension.php

@@ -21,8 +21,7 @@ final class CourseRelUserExtension implements QueryCollectionExtensionInterface
 {
     public function __construct(
         private readonly Security $security
-    ) {
+    ) {}
-    }
 
     public function applyToCollection(
         QueryBuilder $queryBuilder,

src/CoreBundle/DataProvider/Extension/MessageExtension.php

@@ -22,8 +22,7 @@ final class MessageExtension implements QueryCollectionExtensionInterface //, Qu
 {
     public function __construct(
         private readonly Security $security
-    ) {
+    ) {}
-    }
 
     public function applyToCollection(
         QueryBuilder $queryBuilder,

src/CoreBundle/DataProvider/Extension/MessageRelUserExtension.php

@@ -18,8 +18,7 @@ class MessageRelUserExtension implements QueryCollectionExtensionInterface
 {
     public function __construct(
         private readonly Security $security
-    ) {
+    ) {}
-    }
 
     public function applyToCollection(
         QueryBuilder $queryBuilder,

src/CoreBundle/DataProvider/Extension/MessageTagExtension.php

@@ -19,8 +19,7 @@ final class MessageTagExtension implements QueryCollectionExtensionInterface //,
 {
     public function __construct(
         private readonly Security $security
-    ) {
+    ) {}
-    }
 
     public function applyToCollection(
         QueryBuilder $queryBuilder,

src/CoreBundle/DataProvider/Extension/PageExtension.php

@@ -22,8 +22,7 @@ final class PageExtension implements QueryCollectionExtensionInterface //, Query
     public function __construct(
         private readonly Security $security,
         private readonly RequestStack $requestStack
-    ) {
+    ) {}
-    }
 
     public function applyToCollection(
         QueryBuilder $queryBuilder,

src/CoreBundle/DataProvider/Extension/PersonalFileExtension.php

@@ -26,8 +26,7 @@ final class PersonalFileExtension implements QueryCollectionExtensionInterface /
     public function __construct(
         private readonly Security $security,
         private readonly RequestStack $requestStack
-    ) {
+    ) {}
-    }
 
     public function applyToCollection(
         QueryBuilder $queryBuilder,

src/CoreBundle/DataProvider/Extension/SessionRelUserExtension.php

@@ -25,8 +25,7 @@ final class SessionRelUserExtension implements QueryCollectionExtensionInterface
     public function __construct(
         private readonly Security $security,
         private readonly RequestStack $requestStack
-    ) {
+    ) {}
-    }
 
     public function applyToCollection(
         QueryBuilder $queryBuilder,

src/CoreBundle/DataProvider/Extension/SocialPostExtension.php

@@ -17,8 +17,7 @@ class SocialPostExtension implements QueryCollectionExtensionInterface
 {
     public function __construct(
         private readonly SettingsManager $settingsManager
-    ) {
+    ) {}
-    }
 
     public function applyToCollection(
         QueryBuilder $queryBuilder,

src/CoreBundle/DataProvider/Extension/TrackEExerciseExtension.php

@@ -19,8 +19,7 @@ final class TrackEExerciseExtension implements QueryCollectionExtensionInterface
 {
     public function __construct(
         private readonly Security $security
-    ) {
+    ) {}
-    }
 
     public function applyToCollection(
         QueryBuilder $queryBuilder,

src/CoreBundle/DataProvider/UserItemDataProvider.php

@@ -17,8 +17,7 @@ class UserItemDataProvider implements ItemDataProviderInterface, RestrictedDataP
     public function __construct(
         private readonly UserRepository $repository,
         private readonly NameConvention $nameConvention
-    ) {
+    ) {}
-    }
 
     public function getItem(string $resourceClass, $id, string $operationName = null, array $context = [])
     {

src/CoreBundle/Entity/AbstractResource.php

@@ -136,6 +136,7 @@ abstract class AbstractResource
                     $sendTo['groups'][] = (int) $id;
 
                     break;
+
                 case 'USER':
                     $sendTo['users'][] = (int) $id;
 
@@ -176,6 +177,7 @@ abstract class AbstractResource
         ;
 
         $rights = [];
+
         switch ($visibility) {
             case ResourceLink::VISIBILITY_PENDING:
             case ResourceLink::VISIBILITY_DRAFT:
@@ -198,9 +200,9 @@ abstract class AbstractResource
         if ($this->hasResourceNode()) {
             $resourceNode = $this->getResourceNode();
             $exists = $resourceNode->getResourceLinks()->exists(
-                fn ($key, $element) => $course === $element->getCourse() &&
+                fn ($key, $element) => $course === $element->getCourse()
-                    $session === $element->getSession() &&
+                    && $session === $element->getSession()
-                    $group === $element->getGroup()
+                    && $group === $element->getGroup()
             );
 
             if ($exists) {

src/CoreBundle/Entity/AgendaEventSubscriber.php

@@ -9,6 +9,4 @@ namespace Chamilo\CoreBundle\Entity;
 use Doctrine\ORM\Mapping as ORM;
 
 #[ORM\Entity]
-class AgendaEventSubscriber extends AgendaEventInvitee
+class AgendaEventSubscriber extends AgendaEventInvitee {}
-{
-}

src/CoreBundle/Entity/Course.php

@@ -553,7 +553,7 @@ class Course extends AbstractResource implements ResourceInterface, ResourceWith
         $teacherSubscriptions = new ArrayCollection();
 
         foreach ($this->users as $subscription) {
-            if ($subscription->getStatus() === CourseRelUser::TEACHER) {
+            if (CourseRelUser::TEACHER === $subscription->getStatus()) {
                 $teacherSubscriptions->add($subscription);
             }
         }
@@ -923,6 +923,7 @@ class Course extends AbstractResource implements ResourceInterface, ResourceWith
               $this->currentSession = $session;
           }*/
         $list = $this->getSessions();
+
         /** @var SessionRelCourse $item */
         foreach ($list as $item) {
             if ($item->getSession()->getId() === $session->getId()) {
@@ -951,6 +952,7 @@ class Course extends AbstractResource implements ResourceInterface, ResourceWith
     public function setCurrentUrl(AccessUrl $url): self
     {
         $urlList = $this->getUrls();
+
         /** @var AccessUrlRelCourse $item */
         foreach ($urlList as $item) {
             if ($item->getUrl()->getId() === $url->getId()) {

src/CoreBundle/Entity/CourseCategory.php

@@ -63,6 +63,7 @@ class CourseCategory implements Stringable
     #[ORM\Id]
     #[ORM\GeneratedValue]
     protected ?int $id = null;
+
     /**
      * @var Collection|CourseCategory[]
      */
@@ -93,6 +94,7 @@ class CourseCategory implements Stringable
     #[Groups(['course_category:read', 'course_category:write'])]
     #[ORM\Column(name: 'description', type: 'text', nullable: true)]
     protected ?string $description = null;
+
     /**
      * @var Collection<int, AccessUrlRelCourseCategory>
      */
@@ -103,6 +105,7 @@ class CourseCategory implements Stringable
         orphanRemoval: true
     )]
     protected Collection $urls;
+
     /**
      * @var Collection<int, Course>
      */

src/CoreBundle/Entity/CourseRelUserCatalogue.php

@@ -34,9 +34,7 @@ class CourseRelUserCatalogue implements Stringable
     #[ORM\Column(name: 'visible', type: 'integer')]
     protected int $visible;
 
-    public function __construct()
+    public function __construct() {}
-    {
-    }
 
     public function __toString(): string
     {

src/CoreBundle/Entity/ExtraField.php

@@ -87,12 +87,14 @@ class ExtraField
     protected ?bool $changeable = null;
     #[ORM\Column(name: 'filter', type: 'boolean', nullable: true, unique: false)]
     protected ?bool $filter = null;
+
     /**
      * @var Collection<int, ExtraFieldOptions>
      */
     #[Groups(['extra_field:read'])]
     #[ORM\OneToMany(targetEntity: ExtraFieldOptions::class, mappedBy: 'field')]
     protected Collection $options;
+
     /**
      * @var Tag[]|Collection
      */
@@ -111,6 +113,7 @@ class ExtraField
         $this->changeable = false;
         $this->filter = false;
     }
+
     /**
      * Get id.
      *
@@ -140,6 +143,7 @@ class ExtraField
 
         return $this;
     }
+
     /**
      * @return string
      */
@@ -153,6 +157,7 @@ class ExtraField
 
         return $this;
     }
+
     /**
      * @return string
      */
@@ -166,6 +171,7 @@ class ExtraField
 
         return $this;
     }
+
     /**
      * @return string
      */
@@ -179,6 +185,7 @@ class ExtraField
 
         return $this;
     }
+
     /**
      * @return int
      */
@@ -192,6 +199,7 @@ class ExtraField
 
         return $this;
     }
+
     /**
      * @return bool
      */
@@ -245,6 +253,7 @@ class ExtraField
 
         return $this;
     }
+
     /**
      * @return Collection<int, ExtraFieldOptions>
      */
@@ -258,6 +267,7 @@ class ExtraField
 
         return $this;
     }
+
     /**
      * @return Tag[]|Collection
      */

src/CoreBundle/Entity/ExtraFieldValues.php

@@ -45,6 +45,7 @@ class ExtraFieldValues
     #[ORM\ManyToOne(targetEntity: Asset::class)]
     #[ORM\JoinColumn(name: 'asset_id', referencedColumnName: 'id', onDelete: 'CASCADE')]
     protected ?Asset $asset = null;
+
     /**
      * Item id can be: userId, courseId, sessionId, etc.
      */
@@ -89,6 +90,7 @@ class ExtraFieldValues
     {
         return $this->comment;
     }
+
     /**
      * Get id.
      *

src/CoreBundle/Entity/GradebookCategory.php

@@ -17,8 +17,8 @@ use Symfony\Component\Validator\Constraints as Assert;
 #[ORM\Entity]
 class GradebookCategory
 {
-    use UserTrait;
     use CourseTrait;
+    use UserTrait;
 
     #[ORM\Column(name: 'id', type: 'integer')]
     #[ORM\Id]

src/CoreBundle/Entity/GradebookComment.php

@@ -14,8 +14,8 @@ use Gedmo\Timestampable\Traits\TimestampableEntity;
 #[ORM\Entity]
 class GradebookComment
 {
-    use UserTrait;
     use TimestampableEntity;
+    use UserTrait;
 
     #[ORM\Column(name: 'id', type: 'bigint')]
     #[ORM\Id]

src/CoreBundle/Entity/Listener/CourseListener.php

@@ -22,8 +22,7 @@ class CourseListener
     public function __construct(
         protected ToolChain $toolChain,
         protected SettingsManager $settingsManager
-    ) {
+    ) {}
-    }
 
     public function prePersist(Course $course, PrePersistEventArgs $args): void
     {

src/CoreBundle/Entity/Listener/MessageListener.php

@@ -14,8 +14,7 @@ class MessageListener
 {
     public function __construct(
         private MessageBusInterface $bus
-    ) {
+    ) {}
-    }
 
     public function postPersist(Message $message, PostPersistEventArgs $args): void
     {

src/CoreBundle/Entity/Listener/ResourceListener.php

@@ -31,6 +31,9 @@ use Symfony\Component\HttpFoundation\RequestStack;
 use Symfony\Component\Security\Core\Exception\UserNotFoundException;
 use Symfony\Component\Security\Core\Security;
 
+use const JSON_THROW_ON_ERROR;
+use const PATHINFO_EXTENSION;
+
 class ResourceListener
 {
     use AccessUrlListenerTrait;
@@ -40,8 +43,7 @@ class ResourceListener
         protected ToolChain $toolChain,
         protected RequestStack $request,
         protected Security $security
-    ) {
+    ) {}
-    }
 
     /**
      * Only in creation.
@@ -132,7 +134,7 @@ class ResourceListener
             }
 
             if (null === $parentUrl) {
-                throw new InvalidArgumentException(('The resource needs an AccessUrl: use $resource->addAccessUrl()'));
+                throw new InvalidArgumentException('The resource needs an AccessUrl: use $resource->addAccessUrl()');
             }
             $parentNode = $parentUrl;
         }
@@ -184,8 +186,8 @@ class ResourceListener
             if (null === $currentUser) {
                 $currentUser = $parentNode->getCreator();
             }
-            $valid = $parentNode->getCreator()->getUsername() === $currentUser->getUsername() ||
+            $valid = $parentNode->getCreator()->getUsername() === $currentUser->getUsername()
-                     $parentNode->getId() === $currentUser->getResourceNode()->getId();
+                     || $parentNode->getId() === $currentUser->getResourceNode()->getId();
 
             if (!$valid) {
                 $msg = sprintf('User %s cannot add a file to another user', $currentUser->getUsername());

src/CoreBundle/Entity/Listener/ResourceNodeListener.php

@@ -24,8 +24,7 @@ class ResourceNodeListener
         protected RequestStack $request,
         protected Security $security,
         protected ResourceNodeRepository $resourceNodeRepository
-    ) {
+    ) {}
-    }
 
     /*public function prePersist(ResourceNode $resourceNode, LifecycleEventArgs $event)
     {

src/CoreBundle/Entity/Listener/SessionListener.php

@@ -25,8 +25,7 @@ class SessionListener
     public function __construct(
         protected RequestStack $request,
         protected Security $security
-    ) {
+    ) {}
-    }
 
     /**
      * This code is executed when a new session is created.
@@ -51,9 +50,7 @@ class SessionListener
     /**
      * This code is executed when a session is updated.
      */
-    public function preUpdate(Session $session, PreUpdateEventArgs $args): void
+    public function preUpdate(Session $session, PreUpdateEventArgs $args): void {}
-    {
-    }
 
     /*protected function checkLimit(SessionRepository $repo, AccessUrl $url): void
     {

src/CoreBundle/Entity/Listener/SkillRelUserListener.php

@@ -23,8 +23,7 @@ class SkillRelUserListener
         private readonly RouterInterface $router,
         private readonly TranslatorInterface $translator,
         protected Security $security
-    ) {
+    ) {}
-    }
 
     public function postPersist(SkillRelUser $skillRelUser, PostPersistEventArgs $event): void
     {

src/CoreBundle/Entity/Listener/UserListener.php

@@ -22,8 +22,7 @@ class UserListener
         private UserRepository $userRepository,
         private Security $security,
         private readonly TranslatorInterface $translator
-    ) {
+    ) {}
-    }
 
     /**
      * This code is executed when a new user is created.

src/CoreBundle/Entity/Listener/UserRelUserListener.php

@@ -15,8 +15,7 @@ use Exception;
 class UserRelUserListener
 {
     public function __construct(
-    ) {
+    ) {}
-    }
 
     public function prePersist(UserRelUser $userRelUser, PrePersistEventArgs $args): void
     {

src/CoreBundle/Entity/Message.php

@@ -180,7 +180,8 @@ class Message
         return $this->receivers
             ->filter(
                 fn (MessageRelUser $messageRelUser) => MessageRelUser::TYPE_TO === $messageRelUser->getReceiverType()
-            )->getValues();
+            )->getValues()
+        ;
     }
 
     #[Groups(['message:read'])]

src/CoreBundle/Entity/PageCategory.php

@@ -21,11 +21,11 @@ use Symfony\Component\Validator\Constraints as Assert;
 
 #[ApiResource(
     operations: [
-        new Get(security: 'is_granted(\'ROLE_USER\')'),
+        new Get(security: "is_granted('ROLE_USER')"),
-        new Put(security: 'is_granted(\'ROLE_ADMIN\')'),
+        new Put(security: "is_granted('ROLE_ADMIN')"),
-        new Delete(security: 'is_granted(\'ROLE_ADMIN\')'),
+        new Delete(security: "is_granted('ROLE_ADMIN')"),
-        new GetCollection(security: 'is_granted(\'ROLE_USER\')'),
+        new GetCollection(security: "is_granted('ROLE_USER')"),
-        new Post(security: 'is_granted(\'ROLE_ADMIN\')'),
+        new Post(security: "is_granted('ROLE_ADMIN')"),
     ],
     normalizationContext: [
         'groups' => ['page_category:read'],
@@ -56,8 +56,9 @@ class PageCategory
     #[Assert\NotBlank]
     #[ORM\Column(name: 'type', type: 'string')]
     protected string $type;
+
     /**
-     * @var Collection|Page[]
+     * @var Collection<int, Page>
      */
     #[ORM\OneToMany(mappedBy: 'category', targetEntity: Page::class, cascade: ['persist'])]
     protected Collection $pages;
@@ -99,7 +100,11 @@ class PageCategory
 
         return $this;
     }
-    public function getPages()
+
+    /**
+     * @return ArrayCollection<int, Page>
+     */
+    public function getPages(): ArrayCollection
     {
         return $this->pages;
     }

src/CoreBundle/Entity/PersonalFile.php

@@ -45,9 +45,7 @@ class PersonalFile extends AbstractResource implements ResourceInterface, String
     #[Groups(['personal_file:read'])]
     #[ORM\Column(name: 'title', type: 'string', length: 255, nullable: false)]
     protected string $title;
-    public function __construct()
+    public function __construct() {}
-    {
-    }
     public function __toString(): string
     {
         return $this->getTitle();

src/CoreBundle/Entity/ResourceComment.php

@@ -30,9 +30,9 @@ use Symfony\Component\Validator\Constraints as Assert;
 #[ORM\Entity(repositoryClass: NestedTreeRepository::class)]
 class ResourceComment
 {
-    use TimestampableTypedEntity;
-    use TimestampableAgoTrait;
     use NestedSetEntity;
+    use TimestampableAgoTrait;
+    use TimestampableTypedEntity;
 
     #[ORM\Id]
     #[ORM\Column(type: 'bigint')]

src/CoreBundle/Entity/ResourceFile.php

@@ -102,6 +102,7 @@ class ResourceFile implements Stringable
     #[Groups(['resource_file:read', 'resource_node:read', 'document:read', 'message:read'])]
     #[ORM\Column(type: 'integer')]
     protected ?int $size = 0;
+
     /**
      * @Vich\UploadableField(
      *     mapping="resources",
@@ -125,6 +126,7 @@ class ResourceFile implements Stringable
     protected ?string $crop = null;
     #[ORM\OneToOne(mappedBy: 'resourceFile', targetEntity: ResourceNode::class)]
     protected ResourceNode $resourceNode;
+
     /**
      * @var string[]
      */
@@ -140,6 +142,7 @@ class ResourceFile implements Stringable
     protected ?bool $text = null;
     #[ORM\Column(name: 'description', type: 'text', nullable: true)]
     protected ?string $description = null;
+
     /**
      * @var DateTime|DateTimeImmutable
      */
@@ -194,6 +197,7 @@ class ResourceFile implements Stringable
     {
         return $this->crop;
     }
+
     /**
      * $crop example: 100,100,100,100 = width,height,x,y.
      */
@@ -324,7 +328,7 @@ class ResourceFile implements Stringable
         return $this->file;
     }
 
-    public function setFile(File|UploadedFile|null $file = null): self
+    public function setFile(File|UploadedFile $file = null): self
     {
         $this->file = $file;
         if (null !== $file) {

src/CoreBundle/Entity/ResourceNode.php

@@ -71,8 +71,8 @@ use Symfony\Component\Validator\Constraints as Assert;
 #[ApiFilter(filterClass: SearchFilter::class, properties: ['title' => 'partial'])]
 class ResourceNode implements Stringable
 {
-    use TimestampableTypedEntity;
     use TimestampableAgoTrait;
+    use TimestampableTypedEntity;
 
     public const PATH_SEPARATOR = '/';
 
@@ -110,6 +110,7 @@ class ResourceNode implements Stringable
     #[Groups(['ctool:read', 'c_tool_intro:read'])]
     #[ORM\OneToMany(mappedBy: 'resourceNode', targetEntity: ResourceLink::class, cascade: ['persist', 'remove'])]
     protected Collection $resourceLinks;
+
     /**
      * ResourceFile available file for this node.
      */
@@ -290,7 +291,7 @@ class ResourceNode implements Stringable
     /**
      * @return Collection|ResourceComment[]
      */
-    public function getComments(): Collection|array
+    public function getComments(): array|Collection
     {
         return $this->comments;
     }
@@ -303,7 +304,7 @@ class ResourceNode implements Stringable
         return $this;
     }
 
-    public function getPathForDisplayToArray(?int $baseRoot = null): array
+    public function getPathForDisplayToArray(int $baseRoot = null): array
     {
         $parts = explode(self::PATH_SEPARATOR, $this->path);
         $list = [];
@@ -440,7 +441,7 @@ class ResourceNode implements Stringable
         return $this->resourceFile;
     }
 
-    public function setResourceFile(?ResourceFile $resourceFile = null): self
+    public function setResourceFile(ResourceFile $resourceFile = null): self
     {
         $this->resourceFile = $resourceFile;
 

src/CoreBundle/Entity/ResourceShowCourseResourcesInSessionInterface.php

@@ -9,6 +9,4 @@ namespace Chamilo\CoreBundle\Entity;
 /**
  * Show base course resources inside a session.
  */
-interface ResourceShowCourseResourcesInSessionInterface
+interface ResourceShowCourseResourcesInSessionInterface {}
-{
-}

src/CoreBundle/Entity/ResourceToRootInterface.php

@@ -9,6 +9,4 @@ namespace Chamilo\CoreBundle\Entity;
 /**
  * This will attach the resource to the main resource node root (For example a Course).
  */
-interface ResourceToRootInterface
+interface ResourceToRootInterface {}
-{
-}

src/CoreBundle/Entity/ScheduledAnnouncement.php

@@ -39,9 +39,7 @@ class ScheduledAnnouncement
     #[ORM\Column(name: 'c_id', type: 'integer', nullable: true)]
     protected ?int $cId = null;
 
-    public function __construct()
+    public function __construct() {}
-    {
-    }
 
     public function getId(): int
     {

src/CoreBundle/Entity/Session.php

@@ -578,7 +578,7 @@ class Session implements ResourceWithAccessUrlInterface, Stringable
         return false;
     }
 
-    public function getSessionRelCourseByUser(User $user, ?int $status = null): Collection
+    public function getSessionRelCourseByUser(User $user, int $status = null): Collection
     {
         $criteria = Criteria::create()->where(Criteria::expr()->eq('user', $user));
         if (null !== $status) {
@@ -1124,7 +1124,7 @@ class Session implements ResourceWithAccessUrlInterface, Stringable
         return $relation->count() > 0;
     }
 
-    public function getUserInCourse(User $user, Course $course, ?int $status = null): Collection
+    public function getUserInCourse(User $user, Course $course, int $status = null): Collection
     {
         $criteria = Criteria::create()
             ->where(
@@ -1161,7 +1161,7 @@ class Session implements ResourceWithAccessUrlInterface, Stringable
         return $this->hasUserInCourse($user, $course, self::STUDENT);
     }
 
-    protected function compareDates(?DateTime $start, ?DateTime $end = null): bool
+    protected function compareDates(?DateTime $start, DateTime $end = null): bool
     {
         $now = new Datetime('now');
 

src/CoreBundle/Entity/SessionCategory.php

@@ -62,6 +62,7 @@ class SessionCategory implements Stringable
     {
         return $this->url;
     }
+
     /**
      * Get id.
      *
@@ -87,6 +88,7 @@ class SessionCategory implements Stringable
 
         return $this;
     }
+
     /**
      * Get dateStart.
      *
@@ -102,6 +104,7 @@ class SessionCategory implements Stringable
 
         return $this;
     }
+
     /**
      * Get dateEnd.
      *

src/CoreBundle/Entity/Skill.php

@@ -32,26 +32,31 @@ class Skill implements Stringable
     #[ORM\ManyToOne(targetEntity: Profile::class, inversedBy: 'skills')]
     #[ORM\JoinColumn(name: 'profile_id', referencedColumnName: 'id')]
     protected ?Profile $profile = null;
+
     /**
      * @var SkillRelUser[]|Collection
      */
     #[ORM\OneToMany(targetEntity: SkillRelUser::class, mappedBy: 'skill', cascade: ['persist'])]
     protected Collection $issuedSkills;
+
     /**
      * @var Collection|SkillRelItem[]
      */
     #[ORM\OneToMany(targetEntity: SkillRelItem::class, mappedBy: 'skill', cascade: ['persist'])]
     protected Collection $items;
+
     /**
      * @var Collection|SkillRelSkill[]
      */
     #[ORM\OneToMany(targetEntity: SkillRelSkill::class, mappedBy: 'skill', cascade: ['persist'])]
     protected Collection $skills;
+
     /**
      * @var Collection|SkillRelCourse[]
      */
     #[ORM\OneToMany(targetEntity: SkillRelCourse::class, mappedBy: 'skill', cascade: ['persist'])]
     protected Collection $courses;
+
     /**
      * @var Collection|SkillRelGradebook[]
      */
@@ -128,6 +133,7 @@ class Skill implements Stringable
     {
         return $this->description;
     }
+
     /**
      * Set accessUrlId.
      *
@@ -139,6 +145,7 @@ class Skill implements Stringable
 
         return $this;
     }
+
     /**
      * Get accessUrlId.
      *
@@ -154,6 +161,7 @@ class Skill implements Stringable
 
         return $this;
     }
+
     /**
      * Get icon.
      *
@@ -169,6 +177,7 @@ class Skill implements Stringable
 
         return $this;
     }
+
     /**
      * Get criteria.
      *
@@ -184,6 +193,7 @@ class Skill implements Stringable
 
         return $this;
     }
+
     /**
      * Get status.
      *
@@ -193,6 +203,7 @@ class Skill implements Stringable
     {
         return $this->status;
     }
+
     /**
      * Set updatedAt.
      *
@@ -206,6 +217,7 @@ class Skill implements Stringable
 
         return $this;
     }
+
     /**
      * Get updatedAt.
      *
@@ -215,6 +227,7 @@ class Skill implements Stringable
     {
         return $this->updatedAt;
     }
+
     /**
      * Get id.
      *
@@ -224,6 +237,7 @@ class Skill implements Stringable
     {
         return $this->id;
     }
+
     /**
      * @return Profile
      */
@@ -237,6 +251,7 @@ class Skill implements Stringable
 
         return $this;
     }
+
     /**
      * Get issuedSkills.
      *
@@ -246,6 +261,7 @@ class Skill implements Stringable
     {
         return $this->issuedSkills;
     }
+
     /**
      * @return Collection
      */
@@ -263,6 +279,7 @@ class Skill implements Stringable
     {
         if (0 !== $this->getItems()->count()) {
             $found = false;
+
             /** @var SkillRelItem $item */
             foreach ($this->getItems() as $item) {
                 if ($item->getItemId() === $itemId && $item->getItemType() === $typeId) {
@@ -282,6 +299,7 @@ class Skill implements Stringable
         $skillRelItem->setSkill($this);
         $this->items[] = $skillRelItem;
     }
+
     /**
      * @return Collection
      */
@@ -295,6 +313,7 @@ class Skill implements Stringable
 
         return $this;
     }
+
     /**
      * @return SkillRelSkill[]|Collection
      */
@@ -302,6 +321,7 @@ class Skill implements Stringable
     {
         return $this->skills;
     }
+
     /**
      * @param SkillRelSkill[]|Collection $skills
      */
@@ -311,6 +331,7 @@ class Skill implements Stringable
 
         return $this;
     }
+
     /**
      * @return SkillRelGradebook[]|Collection
      */
@@ -318,6 +339,7 @@ class Skill implements Stringable
     {
         return $this->gradeBookCategories;
     }
+
     /**
      * @param SkillRelGradebook[]|Collection $gradeBookCategories
      */
@@ -345,6 +367,7 @@ class Skill implements Stringable
     {
         if (0 !== $this->getCourses()->count()) {
             $found = false;
+
             /** @var SkillRelCourse $item */
             foreach ($this->getCourses() as $item) {
                 $sessionPassFilter = false;

src/CoreBundle/Entity/SkillRelItem.php

@@ -268,24 +268,22 @@ class SkillRelItem
     public function getItemResultUrl(string $cidReq): string
     {
         $url = '';
-        $url = match ($this->getItemType()) {
+
+        return match ($this->getItemType()) {
             ITEM_TYPE_EXERCISE => 'exercise/exercise_show.php?action=qualify&'.$cidReq,
             ITEM_TYPE_STUDENT_PUBLICATION => 'work/view.php?'.$cidReq,
             default => $url,
         };
-
-        return $url;
     }
 
     public function getItemResultList(string $cidReq): string
     {
         $url = '';
-        $url = match ($this->getItemType()) {
+
+        return match ($this->getItemType()) {
             ITEM_TYPE_EXERCISE => 'exercise/exercise_report.php?'.$cidReq.'&id='.$this->getItemId(),
             ITEM_TYPE_STUDENT_PUBLICATION => 'work/work_list_all.php?'.$cidReq.'&id='.$this->getItemId(),
             default => $url,
         };
-
-        return $url;
     }
 }

src/CoreBundle/Entity/SkillRelItemRelUser.php

@@ -42,9 +42,7 @@ class SkillRelItemRelUser
     #[ORM\Column(name: 'updated_by', type: 'integer', nullable: false)]
     protected int $updatedBy;
 
-    public function __construct()
+    public function __construct() {}
-    {
-    }
 
     /**
      * @return int

src/CoreBundle/Entity/User.php

@@ -950,6 +950,7 @@ class User implements UserInterface, EquatableInterface, ResourceInterface, Reso
     {
         $classSubscription = $this->getClasses();
         $classList = [];
+
         /** @var UsergroupRelUser $subscription */
         foreach ($classSubscription as $subscription) {
             $class = $subscription->getUsergroup();
@@ -2203,9 +2204,7 @@ class User implements UserInterface, EquatableInterface, ResourceInterface, Reso
         $this->setUsername($name);
     }
 
-    public function setParent(AbstractResource $parent): void
+    public function setParent(AbstractResource $parent): void {}
-    {
-    }
 
     public function getDefaultIllustration(int $size): string
     {
@@ -2339,7 +2338,7 @@ class User implements UserInterface, EquatableInterface, ResourceInterface, Reso
      * Used to be implemented in global function \api_max_sort_value.
      * Reimplemented using the ORM cache.
      */
-    public function getMaxSortValue(?UserCourseCategory $userCourseCategory = null): int
+    public function getMaxSortValue(UserCourseCategory $userCourseCategory = null): int
     {
         $categoryCourses = $this->courses->matching(
             Criteria::create()->where(Criteria::expr()->neq('relationType', COURSE_RELATION_TYPE_RRHH))->andWhere(

src/CoreBundle/Entity/UserRelUser.php

@@ -67,8 +67,8 @@ use Symfony\Component\Validator\Constraints as Assert;
 )]
 class UserRelUser
 {
-    use UserTrait;
     use TimestampableTypedEntity;
+    use UserTrait;
 
     public const USER_UNKNOWN = 0;
     // public const USER_RELATION_TYPE_UNKNOWN = 1;

src/CoreBundle/Entity/Usergroup.php

@@ -8,6 +8,7 @@ namespace Chamilo\CoreBundle\Entity;
 
 use ApiPlatform\Metadata\ApiResource;
 use ApiPlatform\Metadata\Get;
+use Chamilo\CoreBundle\Repository\Node\UsergroupRepository;
 use Doctrine\Common\Collections\ArrayCollection;
 use Doctrine\Common\Collections\Collection;
 use Doctrine\ORM\Mapping as ORM;
@@ -20,7 +21,7 @@ use Symfony\Component\Validator\Constraints as Assert;
  */
 #[ApiResource(security: 'is_granted(\'ROLE_ADMIN\')', normalizationContext: ['groups' => ['usergroup:read']])]
 #[ORM\Table(name: 'usergroup')]
-#[ORM\Entity(repositoryClass: \Chamilo\CoreBundle\Repository\Node\UsergroupRepository::class)]
+#[ORM\Entity(repositoryClass: UsergroupRepository::class)]
 class Usergroup extends AbstractResource implements ResourceInterface, ResourceIllustrationInterface, ResourceWithAccessUrlInterface, Stringable
 {
     use TimestampableEntity;
@@ -50,30 +51,35 @@ class Usergroup extends AbstractResource implements ResourceInterface, ResourceI
     #[Assert\NotBlank]
     #[ORM\Column(name: 'allow_members_leave_group', type: 'integer')]
     protected int $allowMembersToLeaveGroup;
+
     /**
-     * @var Collection|UsergroupRelUser[]
+     * @var Collection<int, UsergroupRelUser>
      */
-    #[ORM\OneToMany(targetEntity: \Chamilo\CoreBundle\Entity\UsergroupRelUser::class, mappedBy: 'usergroup', cascade: ['persist'])]
+    #[ORM\OneToMany(mappedBy: 'usergroup', targetEntity: UsergroupRelUser::class, cascade: ['persist'])]
     protected Collection $users;
+
     /**
-     * @var Collection|UsergroupRelCourse[]
+     * @var Collection<int, UsergroupRelCourse>
      */
-    #[ORM\OneToMany(targetEntity: \Chamilo\CoreBundle\Entity\UsergroupRelCourse::class, mappedBy: 'usergroup', cascade: ['persist'])]
+    #[ORM\OneToMany(mappedBy: 'usergroup', targetEntity: UsergroupRelCourse::class, cascade: ['persist'])]
     protected Collection $courses;
+
     /**
-     * @var Collection|UsergroupRelSession[]
+     * @var Collection<int, UsergroupRelSession>
      */
-    #[ORM\OneToMany(targetEntity: \Chamilo\CoreBundle\Entity\UsergroupRelSession::class, mappedBy: 'usergroup', cascade: ['persist'])]
+    #[ORM\OneToMany(mappedBy: 'usergroup', targetEntity: UsergroupRelSession::class, cascade: ['persist'])]
     protected Collection $sessions;
+
     /**
-     * @var Collection|UsergroupRelQuestion[]
+     * @var Collection<int, UsergroupRelQuestion>
      */
-    #[ORM\OneToMany(targetEntity: \Chamilo\CoreBundle\Entity\UsergroupRelQuestion::class, mappedBy: 'usergroup', cascade: ['persist'])]
+    #[ORM\OneToMany(mappedBy: 'usergroup', targetEntity: UsergroupRelQuestion::class, cascade: ['persist'])]
     protected Collection $questions;
+
     /**
-     * @var AccessUrlRelUserGroup[]|Collection
+     * @var Collection<int, AccessUrlRelUserGroup>
      */
-    #[ORM\OneToMany(targetEntity: \Chamilo\CoreBundle\Entity\AccessUrlRelUserGroup::class, mappedBy: 'userGroup', cascade: ['persist', 'remove'], orphanRemoval: true)]
+    #[ORM\OneToMany(mappedBy: 'userGroup', targetEntity: AccessUrlRelUserGroup::class, cascade: ['persist', 'remove'], orphanRemoval: true)]
     protected Collection $urls;
     public function __construct()
     {
@@ -90,10 +96,11 @@ class Usergroup extends AbstractResource implements ResourceInterface, ResourceI
     {
         return $this->getName();
     }
+
     /**
-     * @return UsergroupRelUser[]|Collection
+     * @return ArrayCollection<int, UsergroupRelUser>
      */
-    public function getUsers(): array|Collection
+    public function getUsers(): ArrayCollection
     {
         return $this->users;
     }
@@ -139,12 +146,11 @@ class Usergroup extends AbstractResource implements ResourceInterface, ResourceI
             }
         }
     }
+
     /**
      * Get id.
-     *
-     * @return int
      */
-    public function getId()
+    public function getId(): ?int
     {
         return $this->id;
     }
@@ -168,10 +174,8 @@ class Usergroup extends AbstractResource implements ResourceInterface, ResourceI
     {
         return $this->description;
     }
-    /**
+
-     * @return int
+    public function getGroupType(): int
-     */
-    public function getGroupType()
     {
         return $this->groupType;
     }
@@ -221,10 +225,11 @@ class Usergroup extends AbstractResource implements ResourceInterface, ResourceI
 
         return $this;
     }
+
     /**
-     * @return UsergroupRelCourse[]|Collection
+     * @return ArrayCollection<int, UsergroupRelCourse>
      */
-    public function getCourses(): array|Collection
+    public function getCourses(): ArrayCollection
     {
         return $this->courses;
     }
@@ -234,10 +239,11 @@ class Usergroup extends AbstractResource implements ResourceInterface, ResourceI
 
         return $this;
     }
+
     /**
-     * @return UsergroupRelSession[]|Collection
+     * @return ArrayCollection<int, UsergroupRelSession>
      */
-    public function getSessions(): array|Collection
+    public function getSessions(): ArrayCollection
     {
         return $this->sessions;
     }
@@ -247,10 +253,11 @@ class Usergroup extends AbstractResource implements ResourceInterface, ResourceI
 
         return $this;
     }
+
     /**
-     * @return UsergroupRelQuestion[]|Collection
+     * @return ArrayCollection<int, UsergroupRelQuestion>
      */
-    public function getQuestions(): array|Collection
+    public function getQuestions(): ArrayCollection
     {
         return $this->questions;
     }

src/CoreBundle/EventListener/CourseListener.php

@@ -41,8 +41,7 @@ class CourseListener
         private readonly AuthorizationCheckerInterface $authorizationChecker,
         private readonly TranslatorInterface $translator,
         private readonly EntityManagerInterface $entityManager,
-    ) {
+    ) {}
-    }
 
     /**
      * Get request from the URL cidReq, c_id or the "ABC" in the courses url (courses/ABC/index.php).
@@ -193,9 +192,7 @@ class CourseListener
         }
     }
 
-    public function onKernelResponse(ResponseEvent $event): void
+    public function onKernelResponse(ResponseEvent $event): void {}
-    {
-    }
 
     /**
      * Once the onKernelRequest was fired, we check if the course/session object were set and we inject them in the controller.
@@ -231,10 +228,10 @@ class CourseListener
         // $cidReset = $sessionHandler->get('cid_reset', false);
 
         // This controller implements ToolInterface? Then set the course/session
-        if (\is_array($controllerList) &&
+        if (\is_array($controllerList)
-            (
+            && (
-                $controllerList[0] instanceof CourseControllerInterface ||
+                $controllerList[0] instanceof CourseControllerInterface
-                $controllerList[0] instanceof EditorController
+                || $controllerList[0] instanceof EditorController
                 // $controllerList[0] instanceof ResourceController
                 // || $controllerList[0] instanceof LegacyController
             )

src/CoreBundle/EventListener/HTTPExceptionListener.php

@@ -15,8 +15,8 @@ final class HTTPExceptionListener
     public function onKernelException(ExceptionEvent $event): void
     {
         $exception = $event->getThrowable();
-        if (!($exception instanceof HttpException) ||
+        if (!($exception instanceof HttpException)
-            false === strpos($event->getRequest()->getRequestUri(), '/api/')
+              || !str_contains($event->getRequest()->getRequestUri(), '/api/')
         ) {
             return;
         }

src/CoreBundle/EventListener/LegacyListener.php

@@ -37,8 +37,7 @@ class LegacyListener
         private readonly RouterInterface $router,
         private readonly ParameterBagInterface $parameterBag,
         private readonly SettingsManager $settingsManager
-    ) {
+    ) {}
-    }
 
     public function onKernelRequest(RequestEvent $event): void
     {
@@ -163,11 +162,7 @@ class LegacyListener
         $session->set('access_url_id', $urlId);
     }
 
-    public function onKernelResponse(ResponseEvent $event): void
+    public function onKernelResponse(ResponseEvent $event): void {}
-    {
-    }
 
-    public function onKernelController(ControllerEvent $event): void
+    public function onKernelController(ControllerEvent $event): void {}
-    {
-    }
 }

src/CoreBundle/EventListener/LoginSuccessHandler.php

@@ -37,6 +37,7 @@ class LoginSuccessHandler
     public function onSecurityInteractiveLogin(InteractiveLoginEvent $event)
     {
         $request = $event->getRequest();
+
         /** @var User $user */
         $user = $event->getAuthenticationToken()->getUser();
         $userId = $user->getId();
@@ -79,10 +80,12 @@ class LoginSuccessHandler
                     $url = $legacyIndex;
 
                     break;
+
                 case 'user_portal.php':
                     $url = $legacyIndex.'user_portal.php';
 
                     break;
+
                 case 'main/auth/courses.php':
                     $url = $legacyIndex.'/'.$pageAfterLogin;
 

src/CoreBundle/EventListener/SettingListener.php

@@ -11,9 +11,7 @@ use Symfony\Component\HttpFoundation\Request;
 
 class SettingListener
 {
-    public function __construct()
+    public function __construct() {}
-    {
-    }
 
     public function onSettingPreSave(SettingsEvent $event): void
     {

src/CoreBundle/EventSubscriber/AnonymousUserSubscriber.php

@@ -113,7 +113,6 @@ class AnonymousUserSubscriber implements EventSubscriberInterface
         foreach ($anonymousUsers as $user) {
             $loginRecord = $trackLoginRepository->findOneBy(['userIp' => $userIp, 'user' => $user]);
             if ($loginRecord) {
-
                 return $user->getId();
             }
         }

src/CoreBundle/EventSubscriber/LoginFailureSubscriber.php

@@ -17,8 +17,7 @@ class LoginFailureSubscriber implements EventSubscriberInterface
 {
     public function __construct(
         private readonly TrackELoginRecordRepository $trackELoginRecordingRepository
-    ) {
+    ) {}
-    }
 
     public static function getSubscribedEvents(): array
     {
@@ -30,6 +29,7 @@ class LoginFailureSubscriber implements EventSubscriberInterface
     public function onFailureEvent(LoginFailureEvent $event): void
     {
         $passport = $event->getPassport();
+
         /** @var UserBadge $userBadge */
         $userBadge = $passport->getBadge(UserBadge::class);
         $username = $userBadge->getUserIdentifier();

src/CoreBundle/Filter/UserExtraFieldFilter.php

@@ -72,6 +72,7 @@ class UserExtraFieldFilter extends AbstractFilter
                 ;
 
                 break;
+
             case 'userExtraFieldValue':
                 $queryBuilder->andWhere('efv.field_value = :value');
 

src/CoreBundle/Form/ContactType.php

@@ -24,12 +24,9 @@ use Symfony\Contracts\Translation\TranslatorInterface;
  */
 class ContactType extends AbstractType
 {
-    private $translator;
+    public function __construct(
-
+        private readonly TranslatorInterface $translator
-    public function __construct(TranslatorInterface $translator)
+    ) {}
-    {
-        $this->translator = $translator;
-    }
 
     public function buildForm(FormBuilderInterface $builder, array $options): void
     {

src/CoreBundle/Form/DataTransformer/ResourceToIdentifierTransformer.php

@@ -19,7 +19,7 @@ final class ResourceToIdentifierTransformer implements DataTransformerInterface
 
     private string $identifier;
 
-    public function __construct(ObjectRepository $repository, ?string $identifier = null)
+    public function __construct(ObjectRepository $repository, string $identifier = null)
     {
         $this->repository = $repository;
         $this->identifier = $identifier ?? 'id';

src/CoreBundle/Form/ExtraFieldType.php

@@ -100,6 +100,7 @@ class ExtraFieldType extends AbstractType
                 case \ExtraField::FIELD_TYPE_TRIPLE_SELECT:
                     // @todo
                     break;
+
                 case \ExtraField::FIELD_TYPE_GEOLOCALIZATION_COORDINATES:
                 case \ExtraField::FIELD_TYPE_GEOLOCALIZATION:
                     if (!$geolocalization) {
@@ -121,6 +122,7 @@ class ExtraFieldType extends AbstractType
                     $builder->add($variable, GoogleMapType::class, $defaultOptions);
 
                     break;
+
                 case \ExtraField::FIELD_TYPE_TAG:
                     $defaultOptions['expanded'] = false;
                     $defaultOptions['multiple'] = true;
@@ -156,15 +158,18 @@ class ExtraFieldType extends AbstractType
                     $builder->add($variable, ChoiceType::class, $defaultOptions);
 
                     break;
+
                 case \ExtraField::FIELD_TYPE_VIDEO_URL:
                 case \ExtraField::FIELD_TYPE_SOCIAL_PROFILE:
                     $builder->add($variable, UrlType::class, $defaultOptions);
 
                     break;
+
                 case \ExtraField::FIELD_TYPE_MOBILE_PHONE_NUMBER:
                     $builder->add($variable, TelType::class, $defaultOptions);
 
                     break;
+
                 case \ExtraField::FIELD_TYPE_DATE:
                     $defaultOptions['data'] = null;
                     if (!empty($value)) {
@@ -174,6 +179,7 @@ class ExtraFieldType extends AbstractType
                     $builder->add($variable, DateType::class, $defaultOptions);
 
                     break;
+
                 case \ExtraField::FIELD_TYPE_DATETIME:
                     $defaultOptions['data'] = null;
                     if (!empty($value)) {
@@ -183,30 +189,36 @@ class ExtraFieldType extends AbstractType
                     $builder->add($variable, DateTimeType::class, $defaultOptions);
 
                     break;
+
                 case \ExtraField::FIELD_TYPE_TEXTAREA:
                     $builder->add($variable, TextareaType::class, $defaultOptions);
 
                     break;
+
                 case \ExtraField::FIELD_TYPE_FLOAT:
                     $builder->add($variable, NumberType::class, $defaultOptions);
 
                     break;
+
                 case \ExtraField::FIELD_TYPE_INTEGER:
                     $builder->add($variable, IntegerType::class, $defaultOptions);
 
                     break;
+
                 case \ExtraField::FIELD_TYPE_LETTERS_ONLY:
                 case \ExtraField::FIELD_TYPE_ALPHANUMERIC:
                 case \ExtraField::FIELD_TYPE_TEXT:
                     $builder->add($variable, TextType::class, $defaultOptions);
 
                     break;
+
                 case \ExtraField::FIELD_TYPE_CHECKBOX:
                     $defaultOptions['data'] = 1 === (int) $value;
 
                     $builder->add($variable, CheckboxType::class, $defaultOptions);
 
                     break;
+
                 case \ExtraField::FIELD_TYPE_RADIO:
                 case \ExtraField::FIELD_TYPE_SELECT:
                 case \ExtraField::FIELD_TYPE_SELECT_MULTIPLE:
@@ -268,6 +280,7 @@ class ExtraFieldType extends AbstractType
                             $this->extraFieldValuesRepository->updateItemData($extraField, $item, $newValue);
 
                             break;
+
                         case \ExtraField::FIELD_TYPE_TAG:
                             $formItem = $event->getForm()->get($extraField->getVariable());
                             $options = $formItem->getConfig()->getOptions();
@@ -281,6 +294,7 @@ class ExtraFieldType extends AbstractType
                             }
 
                             break;
+
                         default:
                             $this->extraFieldValuesRepository->updateItemData($extraField, $item, $newValue);
 

src/CoreBundle/Form/Type/IllustrationType.php

@@ -16,9 +16,7 @@ use Symfony\Component\OptionsResolver\OptionsResolver;
  */
 class IllustrationType extends AbstractType
 {
-    public function configureOptions(OptionsResolver $resolver): void
+    public function configureOptions(OptionsResolver $resolver): void {}
-    {
-    }
 
     public function getParent(): string
     {

src/CoreBundle/Framework/Container.php

@@ -166,9 +166,6 @@ class Container
         return self::$container->get('messenger.bus.default');
     }
 
-    /**
-     * @return Environment
-     */
     public static function getTwig()
     {
         return self::$twig;
@@ -203,7 +200,7 @@ class Container
         self::$request = $request;
     }
 
-    public static function getSession(): Session|HttpSessionInterface|bool|null
+    public static function getSession(): null|bool|HttpSessionInterface|Session
     {
         if (null !== self::$session) {
             return self::$session;
@@ -640,6 +637,7 @@ class Container
     {
         $doctrine = $container->get('doctrine');
         Database::setConnection($doctrine->getConnection());
+
         /** @var EntityManager $em */
         $em = $doctrine->getManager();
         Database::setManager($em);

src/CoreBundle/Framework/Exporter.php

@@ -18,9 +18,9 @@ use Symfony\Component\HttpFoundation\StreamedResponse;
 class Exporter
 {
     /**
-     * @throws RuntimeException
-     *
      * @return StreamedResponse
+     *
+     * @throws RuntimeException
      */
     public function getResponse(string $format, string $filename, SourceIteratorInterface $source)
     {
@@ -30,21 +30,25 @@ class Exporter
                 $contentType = 'application/vnd.ms-excel';
 
                 break;
+
             case 'xml':
                 $writer = new XmlWriter('php://output');
                 $contentType = 'text/xml';
 
                 break;
+
             case 'json':
                 $writer = new JsonWriter('php://output');
                 $contentType = 'application/json';
 
                 break;
+
             case 'csv':
                 $writer = new CsvWriter('php://output', ',', '"', '', true, true);
                 $contentType = 'text/csv';
 
                 break;
+
             default:
                 throw new RuntimeException('Invalid format');
         }

src/CoreBundle/MessageHandler/MessageHandler.php

@@ -17,8 +17,7 @@ class MessageHandler implements MessageHandlerInterface
 {
     public function __construct(
         private readonly Mailer $mailer
-    ) {
+    ) {}
-    }
 
     public function __invoke(Message $message): void
     {

src/CoreBundle/Migrations/AbstractMigrationChamilo.php

@@ -199,7 +199,7 @@ abstract class AbstractMigrationChamilo extends AbstractMigration implements Con
         $fileName = '',
         $description = ''
     ): bool {
-        $class = \get_class($resource);
+        $class = $resource::class;
         $documentPath = basename($filePath);
 
         if (is_dir($filePath) || (!is_dir($filePath) && !file_exists($filePath))) {
@@ -262,16 +262,19 @@ abstract class AbstractMigrationChamilo extends AbstractMigration implements Con
             $groupId = $item['to_group_id'] ?? 0;
 
             $newVisibility = ResourceLink::VISIBILITY_DRAFT;
+
             // Old 1.11.x visibility (item property) is based in this switch:
             switch ($visibility) {
                 case 0:
                     $newVisibility = ResourceLink::VISIBILITY_DRAFT;
 
                     break;
+
                 case 1:
                     $newVisibility = ResourceLink::VISIBILITY_PUBLISHED;
 
                     break;
+
                 case 2:
                     $newVisibility = ResourceLink::VISIBILITY_DELETED;
 

src/CoreBundle/Migrations/Schema/V200/Version20.php

@@ -553,7 +553,5 @@ class Version20 extends AbstractMigrationChamilo
         }
     }
 
-    public function down(Schema $schema): void
+    public function down(Schema $schema): void {}
-    {
-    }
 }

src/CoreBundle/Migrations/Schema/V200/Version20170525122900.php

@@ -179,7 +179,5 @@ class Version20170525122900 extends AbstractMigrationChamilo
         }
     }
 
-    public function down(Schema $schema): void
+    public function down(Schema $schema): void {}
-    {
-    }
 }

src/CoreBundle/Migrations/Schema/V200/Version20170525123900.php

@@ -119,7 +119,5 @@ class Version20170525123900 extends AbstractMigrationChamilo
         );
     }
 
-    public function down(Schema $schema): void
+    public function down(Schema $schema): void {}
-    {
-    }
 }

src/CoreBundle/Migrations/Schema/V200/Version20170625122900.php

@@ -93,7 +93,5 @@ class Version20170625122900 extends AbstractMigrationChamilo
         }
     }
 
-    public function down(Schema $schema): void
+    public function down(Schema $schema): void {}
-    {
-    }
 }

src/CoreBundle/Migrations/Schema/V200/Version20170625123000.php

@@ -189,7 +189,5 @@ class Version20170625123000 extends AbstractMigrationChamilo
         }
     }
 
-    public function down(Schema $schema): void
+    public function down(Schema $schema): void {}
-    {
-    }
 }

src/CoreBundle/Migrations/Schema/V200/Version20170625143000.php

@@ -179,7 +179,5 @@ class Version20170625143000 extends AbstractMigrationChamilo
         }
     }
 
-    public function down(Schema $schema): void
+    public function down(Schema $schema): void {}
-    {
-    }
 }

src/CoreBundle/Migrations/Schema/V200/Version20170625144000.php

@@ -239,7 +239,5 @@ class Version20170625144000 extends AbstractMigrationChamilo
         }
     }
 
-    public function down(Schema $schema): void
+    public function down(Schema $schema): void {}
-    {
-    }
 }

src/CoreBundle/Migrations/Schema/V200/Version20170625145000.php

@@ -123,7 +123,5 @@ class Version20170625145000 extends AbstractMigrationChamilo
         }
     }
 
-    public function down(Schema $schema): void
+    public function down(Schema $schema): void {}
-    {
-    }
 }

src/CoreBundle/Migrations/Schema/V200/Version20170626122900.php

@@ -276,7 +276,5 @@ class Version20170626122900 extends AbstractMigrationChamilo
         }
     }
 
-    public function down(Schema $schema): void
+    public function down(Schema $schema): void {}
-    {
-    }
 }

src/CoreBundle/Migrations/Schema/V200/Version20170627122900.php

@@ -9,6 +9,8 @@ namespace Chamilo\CoreBundle\Migrations\Schema\V200;
 use Chamilo\CoreBundle\Migrations\AbstractMigrationChamilo;
 use Doctrine\DBAL\Schema\Schema;
 
+use const PHP_EOL;
+
 class Version20170627122900 extends AbstractMigrationChamilo
 {
     public function getDescription(): string
@@ -362,7 +364,5 @@ class Version20170627122900 extends AbstractMigrationChamilo
         }
     }
 
-    public function down(Schema $schema): void
+    public function down(Schema $schema): void {}
-    {
-    }
 }