From 6096110fd0e843c1dc9a357d390e709d9fb47465 Mon Sep 17 00:00:00 2001
From: christianbeeznst <christian.fasanando@beeznest.com>
Date: Sat, 23 Mar 2024 22:45:46 -0500
Subject: [PATCH 1/2] Platform: Refactor global events management - refs #5270

---
 assets/vue/router/ccalendarevent.js           |   3 +-
 .../ccalendarevent/CCalendarEventList.vue     |  17 ++
 public/main/inc/ajax/agenda.ajax.php          |  20 ---
 public/main/inc/lib/agenda.lib.php            |  85 ---------
 .../Admin/IndexBlocksController.php           |   2 +-
 .../Entity/Listener/ResourceListener.php      |  49 ++++++
 src/CoreBundle/Entity/SysCalendar.php         | 166 ------------------
 src/CoreBundle/Filter/GlobalEventFilter.php   |  82 +++++++++
 .../Schema/V200/Version20240323181500.php     | 102 +++++++++++
 .../Schema/V200/Version20240323222700.php     |  29 +++
 .../Repository/SysCalendarRepository.php      |  19 --
 src/CourseBundle/Entity/CCalendarEvent.php    |   2 +
 12 files changed, 284 insertions(+), 292 deletions(-)
 delete mode 100644 src/CoreBundle/Entity/SysCalendar.php
 create mode 100644 src/CoreBundle/Filter/GlobalEventFilter.php
 create mode 100644 src/CoreBundle/Migrations/Schema/V200/Version20240323181500.php
 create mode 100644 src/CoreBundle/Migrations/Schema/V200/Version20240323222700.php
 delete mode 100644 src/CoreBundle/Repository/SysCalendarRepository.php

diff --git a/assets/vue/router/ccalendarevent.js b/assets/vue/router/ccalendarevent.js
index 501753bd21..81446f3490 100644
--- a/assets/vue/router/ccalendarevent.js
+++ b/assets/vue/router/ccalendarevent.js
@@ -18,7 +18,8 @@ export default {
     {
       name: 'CCalendarEventList',
       path: '',
-      component: () => import('../views/ccalendarevent/CCalendarEventList.vue')
+      component: () => import('../views/ccalendarevent/CCalendarEventList.vue'),
+      props: (route) => ({ type: route.query.type })
     }
   ]
 };
diff --git a/assets/vue/views/ccalendarevent/CCalendarEventList.vue b/assets/vue/views/ccalendarevent/CCalendarEventList.vue
index 79b5a70d02..0499f88590 100644
--- a/assets/vue/views/ccalendarevent/CCalendarEventList.vue
+++ b/assets/vue/views/ccalendarevent/CCalendarEventList.vue
@@ -23,6 +23,7 @@
         v-if="dialog"
         ref="createForm"
         :values="item"
+        :is-global="isGlobal"
       />
       <template #footer>
         <BaseButton
@@ -128,6 +129,7 @@ import { useStore } from "vuex"
 import { useI18n } from "vue-i18n"
 import { useConfirm } from "primevue/useconfirm"
 import { useFormatDate } from "../../composables/formatDate"
+import { useRoute } from "vue-router"
 
 import Loading from "../../components/Loading.vue"
 import FullCalendar from "@fullcalendar/vue3"
@@ -171,6 +173,8 @@ const allowToUnsubscribe = ref(false)
 const currentUser = computed(() => store.getters["security/getUser"])
 const { t } = useI18n()
 const { appLocale } = useLocale()
+const route = useRoute()
+const isGlobal = ref(route.query.type === 'global')
 
 let currentEvent = null
 
@@ -203,6 +207,10 @@ async function getCalendarEvents({ startStr, endStr }) {
     params.gid = group.value.id
   }
 
+  if (route.query?.type === 'global') {
+    params.type = 'global'
+  }
+
   const calendarEvents = await cCalendarEventService.findAll({ params }).then((response) => response.json())
 
   return calendarEvents["hydra:member"].map((event) => ({
@@ -359,6 +367,10 @@ function onCreateEventForm() {
 
   let itemModel = createForm.value.v$.item.$model
 
+  if (isGlobal.value) {
+    itemModel.isGlobal = true
+  }
+
   if (itemModel["@id"]) {
     store.dispatch("ccalendarevent/update", itemModel)
   } else {
@@ -380,6 +392,11 @@ function onCreateEventForm() {
 
 const toast = useToast()
 
+watch(() => route.query.type, (newType) => {
+  isGlobal.value = newType === 'global'
+  reFetch()
+})
+
 watch(
   () => store.state.ccalendarevent.created,
   (created) => {
diff --git a/public/main/inc/ajax/agenda.ajax.php b/public/main/inc/ajax/agenda.ajax.php
index 72ea7f7848..5874e9fc39 100644
--- a/public/main/inc/ajax/agenda.ajax.php
+++ b/public/main/inc/ajax/agenda.ajax.php
@@ -177,26 +177,6 @@ switch ($action) {
                 $month,
                 $year
             );
-            $agendaitems = Agenda::get_global_agenda_items(
-                $agendaitems,
-                $day,
-                $month,
-                $year,
-                $week,
-                "month_view"
-            );
-
-            if ('true' === api_get_setting('allow_personal_agenda')) {
-                /*$agendaitems = Agenda::get_personal_agenda_items(
-                    $user_id,
-                    $agendaitems,
-                    $day,
-                    $month,
-                    $year,
-                    $week,
-                    "month_view"
-                );*/
-            }
             Agenda::display_mymonthcalendar(
                 $user_id,
                 $agendaitems,
diff --git a/public/main/inc/lib/agenda.lib.php b/public/main/inc/lib/agenda.lib.php
index 0144b6f457..83446afe28 100644
--- a/public/main/inc/lib/agenda.lib.php
+++ b/public/main/inc/lib/agenda.lib.php
@@ -3,8 +3,6 @@
 /* For licensing terms, see /license.txt */
 
 use Chamilo\CoreBundle\Entity\Course;
-//use Chamilo\CoreBundle\Entity\PersonalAgenda;
-use Chamilo\CoreBundle\Entity\SysCalendar;
 use Chamilo\CoreBundle\Framework\Container;
 use Chamilo\CourseBundle\Entity\CCalendarEvent;
 use Chamilo\CourseBundle\Entity\CCalendarEventAttachment;
@@ -57,8 +55,6 @@ class Agenda
         $sessionId = 0
     ) {
         // Table definitions
-        $this->tbl_global_agenda = Database::get_main_table(TABLE_MAIN_SYSTEM_CALENDAR);
-        //$this->tbl_personal_agenda = Database::get_main_table(TABLE_PERSONAL_AGENDA);
         $this->tbl_course_agenda = Database::get_course_table(TABLE_AGENDA);
         $this->table_repeat = Database::get_course_table(TABLE_AGENDA_REPEAT);
 
@@ -365,24 +361,6 @@ class Agenda
                     }
                 }
                 break;
-            case 'admin':
-                if (api_is_platform_admin()) {
-                    $event = new SysCalendar();
-                    $color = SysCalendar::COLOR_SYSTEM_EVENT;
-                    $event
-                        ->setTitle($title)
-                        ->setContent($content)
-                        ->setStartDate($start)
-                        ->setEndDate($end)
-                        ->setAllDay($allDay)
-                        ->setUrl(api_get_url_entity())
-                        ->setColor($color)
-                    ;
-                    $em->persist($event);
-                    $em->flush();
-                    $id = $event->getId();
-                }
-                break;
         }
 
         return $id;
@@ -811,26 +789,6 @@ class Agenda
 
                 return false;
                 break;
-            case 'admin':
-            case 'platform':
-                if (api_is_platform_admin()) {
-                    $attributes = [
-                        'title' => $title,
-                        'start_date' => $start,
-                        'end_date' => $end,
-                        'all_day' => $allDay,
-                    ];
-
-                    if ($updateContent) {
-                        $attributes['content'] = $content;
-                    }
-                    Database::update(
-                        $this->tbl_global_agenda,
-                        $attributes,
-                        ['id = ?' => $id]
-                    );
-                }
-                break;
         }
     }
 
@@ -925,14 +883,6 @@ class Agenda
                     }
                 }
                 break;
-            case 'admin':
-                if (api_is_platform_admin()) {
-                    Database::delete(
-                        $this->tbl_global_agenda,
-                        ['id = ?' => $id]
-                    );
-                }
-                break;
         }
     }
 
@@ -957,9 +907,6 @@ class Agenda
         $format = 'json'
     ) {
         switch ($this->type) {
-            case 'admin':
-                $this->getPlatformEvents($start, $end);
-                break;
             case 'course':
                 $course = api_get_course_entity($courseId);
 
@@ -1001,14 +948,6 @@ class Agenda
                     $sessionFilterActive = true;
                 }
 
-                if (false == $sessionFilterActive) {
-                    // Getting personal events
-                    //$this->getPersonalEvents($start, $end);
-
-                    // Getting platform/admin events
-                    $this->getPlatformEvents($start, $end);
-                }
-
                 $ignoreVisibility = ('true' === api_get_setting('agenda.personal_agenda_show_all_session_events'));
 
                 // Getting course events
@@ -1171,12 +1110,6 @@ class Agenda
 							    id = ".$id;
                     Database::query($sql);
                     break;
-                case 'admin':
-                    $sql = "UPDATE $this->tbl_global_agenda SET
-                            end_date = DATE_ADD(end_date, INTERVAL $delta MINUTE)
-							WHERE id = ".$id;
-                    Database::query($sql);
-                    break;
             }
         }
 
@@ -1222,14 +1155,6 @@ class Agenda
 							    id=".$id;
                     Database::query($sql);
                     break;
-                case 'admin':
-                    $sql = "UPDATE $this->tbl_global_agenda SET
-                            all_day = $allDay,
-                            start_date = DATE_ADD(start_date,INTERVAL $delta MINUTE),
-                            end_date = DATE_ADD(end_date, INTERVAL $delta MINUTE)
-							WHERE id=".$id;
-                    Database::query($sql);
-                    break;
             }
         }
 
@@ -1291,16 +1216,6 @@ class Agenda
                     }
                 }
                 break;
-            case 'admin':
-            case 'platform':
-                $sql = "SELECT * FROM ".$this->tbl_global_agenda."
-                        WHERE id = $id";
-                $result = Database::query($sql);
-                if (Database::num_rows($result)) {
-                    $event = Database::fetch_array($result, 'ASSOC');
-                    $event['description'] = $event['content'];
-                }
-                break;
         }
 
         return $event;
diff --git a/src/CoreBundle/Controller/Admin/IndexBlocksController.php b/src/CoreBundle/Controller/Admin/IndexBlocksController.php
index e6b1306209..afac854f11 100644
--- a/src/CoreBundle/Controller/Admin/IndexBlocksController.php
+++ b/src/CoreBundle/Controller/Admin/IndexBlocksController.php
@@ -339,7 +339,7 @@ class IndexBlocksController extends BaseController
         ];
         $items[] = [
             'class' => 'item-global-agenda',
-            'url' => $this->generateUrl('legacy_main', ['name' => 'calendar/agenda_js.php', 'type' => 'admin']),
+            'url' => '/resources/ccalendarevent?type=global',
             'label' => $this->translator->trans('Global agenda'),
         ];
 
diff --git a/src/CoreBundle/Entity/Listener/ResourceListener.php b/src/CoreBundle/Entity/Listener/ResourceListener.php
index 001efbaf24..39c075e353 100644
--- a/src/CoreBundle/Entity/Listener/ResourceListener.php
+++ b/src/CoreBundle/Entity/Listener/ResourceListener.php
@@ -13,6 +13,7 @@ use Chamilo\CoreBundle\Entity\EntityAccessUrlInterface;
 use Chamilo\CoreBundle\Entity\PersonalFile;
 use Chamilo\CoreBundle\Entity\ResourceFile;
 use Chamilo\CoreBundle\Entity\ResourceFormat;
+use Chamilo\CoreBundle\Entity\ResourceLink;
 use Chamilo\CoreBundle\Entity\ResourceNode;
 use Chamilo\CoreBundle\Entity\ResourceToRootInterface;
 use Chamilo\CoreBundle\Entity\ResourceType;
@@ -20,6 +21,7 @@ use Chamilo\CoreBundle\Entity\ResourceWithAccessUrlInterface;
 use Chamilo\CoreBundle\Entity\User;
 use Chamilo\CoreBundle\Tool\ToolChain;
 use Chamilo\CoreBundle\Traits\AccessUrlListenerTrait;
+use Chamilo\CourseBundle\Entity\CCalendarEvent;
 use Cocur\Slugify\SlugifyInterface;
 use Doctrine\ORM\Event\PostUpdateEventArgs;
 use Doctrine\ORM\Event\PrePersistEventArgs;
@@ -257,6 +259,10 @@ class ResourceListener
 
             throw new InvalidArgumentException($message);
         }
+
+        if ($resource instanceof CCalendarEvent) {
+            $this->addCCalendarEventGlobalLink($resource, $eventArgs);
+        }
     }
 
     /**
@@ -299,4 +305,47 @@ class ResourceListener
         $slug = sprintf('%s.%s', $this->slugify->slugify($originalBasename), $originalExtension);*/
         $resource->getResourceNode()->setTitle($resourceName);
     }
+
+    private function addCCalendarEventGlobalLink(CCalendarEvent $event, PrePersistEventArgs $eventArgs): void
+    {
+        $currentRequest = $this->request->getCurrentRequest();
+
+        if (null === $currentRequest) {
+            return;
+        }
+
+        $type = $currentRequest->query->get('type');
+        if (null === $type) {
+            $content = $currentRequest->getContent();
+            $params = json_decode($content, true);
+            if (isset($params['isGlobal']) && 1 === (int) $params['isGlobal']) {
+                $type = 'global';
+            }
+        }
+
+        if ('global' === $type) {
+            $em = $eventArgs->getObjectManager();
+            $resourceNode = $event->getResourceNode();
+
+            $globalLink = new ResourceLink();
+            $globalLink->setCourse(null)
+                ->setSession(null)
+                ->setGroup(null)
+                ->setUser(null);
+
+            $alreadyHasGlobalLink = false;
+            foreach ($resourceNode->getResourceLinks() as $existingLink) {
+                if (null === $existingLink->getCourse() && null === $existingLink->getSession() &&
+                    null === $existingLink->getGroup() && null === $existingLink->getUser()) {
+                    $alreadyHasGlobalLink = true;
+                    break;
+                }
+            }
+
+            if (!$alreadyHasGlobalLink) {
+                $resourceNode->addResourceLink($globalLink);
+                $em->persist($globalLink);
+            }
+        }
+    }
 }
diff --git a/src/CoreBundle/Entity/SysCalendar.php b/src/CoreBundle/Entity/SysCalendar.php
deleted file mode 100644
index fb394a34ef..0000000000
--- a/src/CoreBundle/Entity/SysCalendar.php
+++ /dev/null
@@ -1,166 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-/* For licensing terms, see /license.txt */
-
-namespace Chamilo\CoreBundle\Entity;
-
-use DateTime;
-use Doctrine\ORM\Mapping as ORM;
-
-/**
- * SysCalendar.
- */
-#[ORM\Table(name: 'sys_calendar')]
-#[ORM\Entity]
-class SysCalendar
-{
-    public const COLOR_SYSTEM_EVENT = '#FF0000';
-
-    #[ORM\Column(name: 'id', type: 'integer')]
-    #[ORM\Id]
-    #[ORM\GeneratedValue(strategy: 'IDENTITY')]
-    protected ?int $id = null;
-
-    #[ORM\Column(name: 'title', type: 'string', length: 255, nullable: false)]
-    protected string $title;
-
-    #[ORM\Column(name: 'content', type: 'text', nullable: true)]
-    protected ?string $content = null;
-
-    #[ORM\Column(name: 'start_date', type: 'datetime', nullable: true)]
-    protected ?DateTime $startDate = null;
-
-    #[ORM\Column(name: 'end_date', type: 'datetime', nullable: true)]
-    protected ?DateTime $endDate = null;
-
-    #[ORM\ManyToOne(targetEntity: AccessUrl::class)]
-    #[ORM\JoinColumn(name: 'access_url_id', referencedColumnName: 'id', onDelete: 'CASCADE')]
-    protected AccessUrl $url;
-
-    #[ORM\Column(name: 'all_day', type: 'integer', nullable: false)]
-    protected int $allDay;
-
-    #[ORM\Column(name: 'color', type: 'string', length: 20, nullable: true)]
-    protected ?string $color = null;
-
-    public function setTitle(string $title): self
-    {
-        $this->title = $title;
-
-        return $this;
-    }
-
-    /**
-     * Get title.
-     *
-     * @return string
-     */
-    public function getTitle()
-    {
-        return $this->title;
-    }
-
-    public function setContent(string $content): self
-    {
-        $this->content = $content;
-
-        return $this;
-    }
-
-    /**
-     * Get content.
-     *
-     * @return string
-     */
-    public function getContent()
-    {
-        return $this->content;
-    }
-
-    public function setStartDate(DateTime $startDate): self
-    {
-        $this->startDate = $startDate;
-
-        return $this;
-    }
-
-    /**
-     * Get startDate.
-     *
-     * @return DateTime
-     */
-    public function getStartDate()
-    {
-        return $this->startDate;
-    }
-
-    public function setEndDate(DateTime $endDate): self
-    {
-        $this->endDate = $endDate;
-
-        return $this;
-    }
-
-    /**
-     * Get endDate.
-     *
-     * @return DateTime
-     */
-    public function getEndDate()
-    {
-        return $this->endDate;
-    }
-
-    public function setAllDay(int $allDay): self
-    {
-        $this->allDay = $allDay;
-
-        return $this;
-    }
-
-    /**
-     * Get allDay.
-     *
-     * @return int
-     */
-    public function getAllDay()
-    {
-        return $this->allDay;
-    }
-
-    /**
-     * Get id.
-     *
-     * @return int
-     */
-    public function getId()
-    {
-        return $this->id;
-    }
-
-    public function getColor(): ?string
-    {
-        return $this->color;
-    }
-
-    public function setColor(string $color): self
-    {
-        $this->color = $color;
-
-        return $this;
-    }
-
-    public function getUrl(): AccessUrl
-    {
-        return $this->url;
-    }
-
-    public function setUrl(AccessUrl $url): self
-    {
-        $this->url = $url;
-
-        return $this;
-    }
-}
diff --git a/src/CoreBundle/Filter/GlobalEventFilter.php b/src/CoreBundle/Filter/GlobalEventFilter.php
new file mode 100644
index 0000000000..5171f70f8a
--- /dev/null
+++ b/src/CoreBundle/Filter/GlobalEventFilter.php
@@ -0,0 +1,82 @@
+<?php
+/* For licensing terms, see /license.txt */
+
+declare(strict_types=1);
+
+namespace Chamilo\CoreBundle\Filter;
+
+use ApiPlatform\Doctrine\Orm\Filter\AbstractFilter;
+use ApiPlatform\Doctrine\Orm\Util\QueryNameGeneratorInterface;
+use ApiPlatform\Metadata\Operation;
+use Chamilo\CoreBundle\Entity\ResourceNode;
+use Doctrine\ORM\QueryBuilder;
+use Doctrine\Persistence\ManagerRegistry;
+use Psr\Log\LoggerInterface;
+use Symfony\Component\Serializer\NameConverter\NameConverterInterface;
+
+class GlobalEventFilter extends AbstractFilter
+{
+    public function __construct(
+        ManagerRegistry $managerRegistry,
+        ?LoggerInterface $logger = null,
+        ?array $properties = null,
+        ?NameConverterInterface $nameConverter = null
+    ) {
+        parent::__construct($managerRegistry, $logger, $properties, $nameConverter);
+    }
+
+    protected function filterProperty(
+        string $property,
+               $value,
+        QueryBuilder $queryBuilder,
+        QueryNameGeneratorInterface $queryNameGenerator,
+        string $resourceClass,
+        ?Operation $operation = null,
+        array $context = []
+    ): void {
+        $isGlobalType = isset($context['filters']['type']) && $context['filters']['type'] === 'global';
+
+        $rootAlias = $queryBuilder->getRootAliases()[0];
+        $resourceNodeAlias = $queryNameGenerator->generateJoinAlias('resourceNode');
+        $resourceLinkAlias = $queryNameGenerator->generateJoinAlias('resourceLink');
+        if ($isGlobalType) {
+            $queryBuilder
+                ->innerJoin("$rootAlias.resourceNode", $resourceNodeAlias)
+                ->innerJoin("$resourceNodeAlias.resourceLinks", $resourceLinkAlias)
+                ->andWhere("$resourceLinkAlias.course IS NULL")
+                ->andWhere("$resourceLinkAlias.session IS NULL")
+                ->andWhere("$resourceLinkAlias.group IS NULL")
+                ->andWhere("$resourceLinkAlias.user IS NULL");
+
+            return;
+        }
+
+        if (!$isGlobalType) {
+            $subQueryBuilder = $queryBuilder->getEntityManager()->createQueryBuilder();
+            $subRN = $queryNameGenerator->generateJoinAlias('subResourceNode');
+            $subRL = $queryNameGenerator->generateJoinAlias('subResourceLink');
+            $subQueryBuilder->select('1')
+                ->from(ResourceNode::class, $subRN)
+                ->innerJoin("$subRN.resourceLinks", $subRL)
+                ->where("$subRL.course IS NULL")
+                ->andWhere("$subRL.session IS NULL")
+                ->andWhere("$subRL.group IS NULL")
+                ->andWhere("$subRL.user IS NULL")
+                ->andWhere("$subRN.id = $rootAlias.resourceNode");
+
+            $queryBuilder->andWhere($queryBuilder->expr()->not($queryBuilder->expr()->exists($subQueryBuilder->getDQL())));
+        }
+    }
+
+    public function getDescription(string $resourceClass): array
+    {
+        return [
+            'type' => [
+                'property' => 'type',
+                'type' => 'string',
+                'required' => false,
+                'description' => 'Filter events by type. Use "global" to get global events.',
+            ],
+        ];
+    }
+}
diff --git a/src/CoreBundle/Migrations/Schema/V200/Version20240323181500.php b/src/CoreBundle/Migrations/Schema/V200/Version20240323181500.php
new file mode 100644
index 0000000000..d5d5cfc74e
--- /dev/null
+++ b/src/CoreBundle/Migrations/Schema/V200/Version20240323181500.php
@@ -0,0 +1,102 @@
+<?php
+
+/* For licensing terms, see /license.txt */
+
+declare(strict_types=1);
+
+namespace Chamilo\CoreBundle\Migrations\Schema\V200;
+
+use Chamilo\CoreBundle\Entity\ResourceLink;
+use Chamilo\CoreBundle\Entity\User;
+use Chamilo\CoreBundle\Migrations\AbstractMigrationChamilo;
+use Chamilo\CourseBundle\Entity\CCalendarEvent;
+use DateTime;
+use DateTimeZone;
+use Doctrine\DBAL\Schema\Schema;
+
+class Version20240323181500 extends AbstractMigrationChamilo
+{
+    public function getDescription(): string
+    {
+        return 'Migrate sys_calendar to c_calendar_event';
+    }
+
+    public function up(Schema $schema): void
+    {
+        $em = $this->getEntityManager();
+        $sysCalendars = $this->connection->fetchAllAssociative('SELECT * FROM sys_calendar');
+
+        $utc = new DateTimeZone('UTC');
+        $admin = $this->getAdmin();
+        foreach ($sysCalendars as $sysCalendar) {
+
+            $calendarEvent = $this->createCCalendarEvent(
+                $sysCalendar['title'] ?: '-',
+                $sysCalendar['content'],
+                $sysCalendar['start_date'] ? new DateTime($sysCalendar['start_date'], $utc) : null,
+                $sysCalendar['end_date'] ? new DateTime($sysCalendar['end_date'], $utc) : null,
+                (bool) $sysCalendar['all_day'],
+                $sysCalendar['color'] ?? '',
+                $admin
+            );
+
+            $em->persist($calendarEvent);
+            $this->addGlobalResourceLinkToNode($em, $calendarEvent->getResourceNode());
+        }
+
+        $em->flush();
+    }
+
+    private function createCCalendarEvent(
+        string $title,
+        string $content,
+        ?DateTime $startDate,
+        ?DateTime $endDate,
+        bool $allDay,
+        string $color,
+        User $creator
+    ): CCalendarEvent {
+        $calendarEvent = new CCalendarEvent();
+        $calendarEvent
+            ->setTitle($title)
+            ->setContent($content)
+            ->setStartDate($startDate)
+            ->setEndDate($endDate)
+            ->setAllDay($allDay)
+            ->setColor($color)
+            ->setCreator($creator)
+            ->setResourceName($title)
+            ->setParentResourceNode($creator->getResourceNode()->getId())
+        ;
+
+        return $calendarEvent;
+    }
+
+    private function addGlobalResourceLinkToNode($em, $resourceNode): void
+    {
+        $globalLink = new ResourceLink();
+        $globalLink->setCourse(null)
+            ->setSession(null)
+            ->setGroup(null)
+            ->setUser(null);
+
+        $alreadyHasGlobalLink = false;
+        foreach ($resourceNode->getResourceLinks() as $existingLink) {
+            if (null === $existingLink->getCourse() && null === $existingLink->getSession() &&
+                null === $existingLink->getGroup() && null === $existingLink->getUser()) {
+                $alreadyHasGlobalLink = true;
+                break;
+            }
+        }
+
+        if (!$alreadyHasGlobalLink) {
+            $resourceNode->addResourceLink($globalLink);
+            $em->persist($globalLink);
+        }
+    }
+
+    public function down(Schema $schema): void
+    {
+        // Down migration is not defined, as data migration cannot be easily reverted
+    }
+}
diff --git a/src/CoreBundle/Migrations/Schema/V200/Version20240323222700.php b/src/CoreBundle/Migrations/Schema/V200/Version20240323222700.php
new file mode 100644
index 0000000000..4971b1f33d
--- /dev/null
+++ b/src/CoreBundle/Migrations/Schema/V200/Version20240323222700.php
@@ -0,0 +1,29 @@
+<?php
+/* For licensing terms, see /license.txt */
+
+declare(strict_types=1);
+
+namespace Chamilo\CoreBundle\Migrations\Schema\V200;
+
+use Chamilo\CoreBundle\Migrations\AbstractMigrationChamilo;
+use Doctrine\DBAL\Schema\Schema;
+
+final class Version20240323222700 extends AbstractMigrationChamilo
+{
+    public function getDescription(): string
+    {
+        return 'Remove the sys_calendar table';
+    }
+
+    public function up(Schema $schema): void
+    {
+        $this->addSql('DROP TABLE IF EXISTS sys_calendar');
+    }
+
+    public function down(Schema $schema): void
+    {
+        if (!$schema->hasTable('sys_calendar')) {
+            $this->addSql('CREATE TABLE sys_calendar (id INT AUTO_INCREMENT NOT NULL, title VARCHAR(255) NOT NULL, content LONGTEXT DEFAULT NULL, start_date DATETIME DEFAULT NULL, end_date DATETIME DEFAULT NULL, access_url_id INT DEFAULT NULL, all_day INT NOT NULL, color VARCHAR(20) DEFAULT NULL, PRIMARY KEY(id)) DEFAULT CHARACTER SET utf8mb4 COLLATE `utf8mb4_unicode_ci` ENGINE = InnoDB');
+        }
+    }
+}
diff --git a/src/CoreBundle/Repository/SysCalendarRepository.php b/src/CoreBundle/Repository/SysCalendarRepository.php
deleted file mode 100644
index edea53c613..0000000000
--- a/src/CoreBundle/Repository/SysCalendarRepository.php
+++ /dev/null
@@ -1,19 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-/* For licensing terms, see /license.txt */
-
-namespace Chamilo\CoreBundle\Repository;
-
-use Chamilo\CoreBundle\Entity\SysCalendar;
-use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
-use Doctrine\Persistence\ManagerRegistry;
-
-class SysCalendarRepository extends ServiceEntityRepository
-{
-    public function __construct(ManagerRegistry $registry)
-    {
-        parent::__construct($registry, SysCalendar::class);
-    }
-}
diff --git a/src/CourseBundle/Entity/CCalendarEvent.php b/src/CourseBundle/Entity/CCalendarEvent.php
index d675b2c09e..0e58e21922 100644
--- a/src/CourseBundle/Entity/CCalendarEvent.php
+++ b/src/CourseBundle/Entity/CCalendarEvent.php
@@ -21,6 +21,7 @@ use Chamilo\CoreBundle\Entity\AbstractResource;
 use Chamilo\CoreBundle\Entity\ResourceInterface;
 use Chamilo\CoreBundle\Entity\Room;
 use Chamilo\CoreBundle\Filter\CidFilter;
+use Chamilo\CoreBundle\Filter\GlobalEventFilter;
 use Chamilo\CoreBundle\Filter\SidFilter;
 use Chamilo\CoreBundle\State\CalendarEventProvider;
 use Chamilo\CoreBundle\State\CCalendarEventProcessor;
@@ -67,6 +68,7 @@ use Symfony\Component\Validator\Constraints as Assert;
 #[ApiFilter(filterClass: DateFilter::class, strategy: 'exclude_null')]
 #[ApiFilter(filterClass: CidFilter::class)]
 #[ApiFilter(filterClass: SidFilter::class)]
+#[ApiFilter(GlobalEventFilter::class, properties: ["type"])]
 class CCalendarEvent extends AbstractResource implements ResourceInterface, Stringable
 {
     public const COLOR_STUDENT_PUBLICATION = '#FF8C00';

From 2de9ba5832d058bea58e73badbc763ba7f58747d Mon Sep 17 00:00:00 2001
From: christianbeeznst <christian.fasanando@beeznest.com>
Date: Sun, 24 Mar 2024 17:34:06 -0500
Subject: [PATCH 2/2] Platform: Improve event management by type in calendar -
 refs #5270

---
 .../ccalendarevent/CCalendarEventList.vue     | 57 ++++++++++++++++---
 src/CoreBundle/ApiResource/CalendarEvent.php  | 13 +++++
 .../Extension/CCalendarEventExtension.php     |  9 ++-
 .../CalendarEventTransformer.php              |  8 ++-
 src/CoreBundle/Filter/GlobalEventFilter.php   | 37 ++++--------
 .../Repository/CCalendarEventRepository.php   | 32 +++++++++++
 6 files changed, 117 insertions(+), 39 deletions(-)

diff --git a/assets/vue/views/ccalendarevent/CCalendarEventList.vue b/assets/vue/views/ccalendarevent/CCalendarEventList.vue
index 0499f88590..408fe2b736 100644
--- a/assets/vue/views/ccalendarevent/CCalendarEventList.vue
+++ b/assets/vue/views/ccalendarevent/CCalendarEventList.vue
@@ -1,10 +1,10 @@
 <template>
   <div class="flex flex-col gap-4">
-    <CalendarSectionHeader
+    <!--CalendarSectionHeader
       @add-click="showAddEventDialog"
       @my-students-schedule-click="goToMyStudentsSchedule"
       @session-planning-click="goToSessionPanning"
-    />
+    /-->
 
     <FullCalendar
       ref="cal"
@@ -73,17 +73,18 @@
         />
 
         <BaseButton
+          v-if="showDeleteButton"
           :label="t('Delete')"
           icon="delete"
           type="danger"
           @click="confirmDelete"
         />
         <BaseButton
-          v-if="allowToEdit"
+          v-if="allowToEdit && showEditButton"
           :label="t('Edit')"
           type="secondary"
           @click="dialog = true"
-        />
+          icon="delete"/>
       </template>
     </Dialog>
 
@@ -213,11 +214,23 @@ async function getCalendarEvents({ startStr, endStr }) {
 
   const calendarEvents = await cCalendarEventService.findAll({ params }).then((response) => response.json())
 
-  return calendarEvents["hydra:member"].map((event) => ({
-    ...event,
-    start: event.startDate,
-    end: event.endDate,
-  }))
+  return calendarEvents["hydra:member"].map((event) => {
+    let color = '#007BFF'
+    if (event.type === 'global') {
+      color = '#FF0000'
+    } else if (event.type === 'course') {
+      color = '#28a745'
+    } else if (event.type === 'session') {
+      color = '#800080'
+    }
+
+    return {
+      ...event,
+      start: event.startDate,
+      end: event.endDate,
+      color,
+    }
+  })
 }
 
 const calendarLocale = allLocales.find(
@@ -299,6 +312,32 @@ const calendarOptions = ref({
   },
 })
 
+const currentContext = computed(() => {
+  if (route.query.type === 'global') {
+    return 'global'
+  } else if (course.value) {
+    return 'course'
+  } else if (session.value) {
+    return 'session'
+  } else {
+    return 'personal'
+  }
+});
+
+const allowAction = (eventType) => {
+  const contextRules = {
+    global: ['global'],
+    course: ['course'],
+    session: ['session'],
+    personal: ['personal']
+  };
+
+  return contextRules[currentContext.value].includes(eventType);
+};
+
+const showEditButton = computed(() => allowAction(item.value.type));
+const showDeleteButton = computed(() => allowAction(item.value.type));
+
 const cal = ref(null)
 
 function reFetch() {
diff --git a/src/CoreBundle/ApiResource/CalendarEvent.php b/src/CoreBundle/ApiResource/CalendarEvent.php
index 0d235f2e98..189e5f2ad6 100644
--- a/src/CoreBundle/ApiResource/CalendarEvent.php
+++ b/src/CoreBundle/ApiResource/CalendarEvent.php
@@ -43,7 +43,20 @@ class CalendarEvent extends AbstractResource
         #[Groups(['calendar_event:read'])]
         public ?ResourceNode $resourceNode = null,
         ?array $resourceLinkListFromEntity = null,
+        #[Groups(['calendar_event:read'])]
+        public ?string $type = null,
     ) {
         $this->resourceLinkListFromEntity = $resourceLinkListFromEntity;
     }
+
+    public function getType(): ?string
+    {
+        return $this->type;
+    }
+
+    public function setType(?string $type): self
+    {
+        $this->type = $type;
+        return $this;
+    }
 }
diff --git a/src/CoreBundle/DataProvider/Extension/CCalendarEventExtension.php b/src/CoreBundle/DataProvider/Extension/CCalendarEventExtension.php
index 12bce35c2c..64541fbbda 100644
--- a/src/CoreBundle/DataProvider/Extension/CCalendarEventExtension.php
+++ b/src/CoreBundle/DataProvider/Extension/CCalendarEventExtension.php
@@ -35,15 +35,20 @@ final class CCalendarEventExtension implements QueryCollectionExtensionInterface
         ?Operation $operation = null,
         array $context = []
     ): void {
-        $this->addWhere($queryBuilder, $resourceClass);
+        $this->addWhere($queryBuilder, $resourceClass, $context);
     }
 
-    private function addWhere(QueryBuilder $qb, string $resourceClass): void
+    private function addWhere(QueryBuilder $qb, string $resourceClass, array $context): void
     {
         if (CCalendarEvent::class !== $resourceClass) {
             return;
         }
 
+        $isGlobalType = isset($context['filters']['type']) && $context['filters']['type'] === 'global';
+        if ($isGlobalType) {
+            return;
+        }
+
         $courseId = $this->cidReqHelper->getCourseId();
         $sessionId = $this->cidReqHelper->getSessionId();
         $groupId = $this->cidReqHelper->getGroupId();
diff --git a/src/CoreBundle/DataTransformer/CalendarEventTransformer.php b/src/CoreBundle/DataTransformer/CalendarEventTransformer.php
index e6145cf33c..2daf88edfe 100644
--- a/src/CoreBundle/DataTransformer/CalendarEventTransformer.php
+++ b/src/CoreBundle/DataTransformer/CalendarEventTransformer.php
@@ -12,6 +12,7 @@ use Chamilo\CoreBundle\Entity\Session;
 use Chamilo\CoreBundle\Entity\SessionRelCourse;
 use Chamilo\CoreBundle\Repository\Node\UsergroupRepository;
 use Chamilo\CourseBundle\Entity\CCalendarEvent;
+use Chamilo\CourseBundle\Repository\CCalendarEventRepository;
 use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
 use Symfony\Component\Routing\RouterInterface;
 
@@ -20,6 +21,7 @@ class CalendarEventTransformer implements DataTransformerInterface
     public function __construct(
         private readonly RouterInterface $router,
         private readonly UsergroupRepository $usergroupRepository,
+        private readonly CCalendarEventRepository $calendarEventRepository
     ) {}
 
     public function transform($object, string $to, array $context = []): object
@@ -48,7 +50,8 @@ class CalendarEventTransformer implements DataTransformerInterface
             $subscriptionItemTitle = $this->usergroupRepository->find($object->getSubscriptionItemId())?->getTitle();
         }
 
-        return new CalendarEvent(
+        $eventType = $this->calendarEventRepository->determineEventType($object);
+        $calendarEvent = new CalendarEvent(
             'calendar_event_'.$object->getIid(),
             $object->getTitle(),
             $object->getContent(),
@@ -65,6 +68,9 @@ class CalendarEventTransformer implements DataTransformerInterface
             $object->getResourceNode(),
             $object->getResourceLinkListFromEntity(),
         );
+        $calendarEvent->setType($eventType);
+
+        return $calendarEvent;
     }
 
     private function mapSessionToDto(object $object): CalendarEvent
diff --git a/src/CoreBundle/Filter/GlobalEventFilter.php b/src/CoreBundle/Filter/GlobalEventFilter.php
index 5171f70f8a..b2454ec01c 100644
--- a/src/CoreBundle/Filter/GlobalEventFilter.php
+++ b/src/CoreBundle/Filter/GlobalEventFilter.php
@@ -35,37 +35,20 @@ class GlobalEventFilter extends AbstractFilter
         array $context = []
     ): void {
         $isGlobalType = isset($context['filters']['type']) && $context['filters']['type'] === 'global';
+        if (!$isGlobalType) {
+            return;
+        }
 
         $rootAlias = $queryBuilder->getRootAliases()[0];
         $resourceNodeAlias = $queryNameGenerator->generateJoinAlias('resourceNode');
         $resourceLinkAlias = $queryNameGenerator->generateJoinAlias('resourceLink');
-        if ($isGlobalType) {
-            $queryBuilder
-                ->innerJoin("$rootAlias.resourceNode", $resourceNodeAlias)
-                ->innerJoin("$resourceNodeAlias.resourceLinks", $resourceLinkAlias)
-                ->andWhere("$resourceLinkAlias.course IS NULL")
-                ->andWhere("$resourceLinkAlias.session IS NULL")
-                ->andWhere("$resourceLinkAlias.group IS NULL")
-                ->andWhere("$resourceLinkAlias.user IS NULL");
-
-            return;
-        }
-
-        if (!$isGlobalType) {
-            $subQueryBuilder = $queryBuilder->getEntityManager()->createQueryBuilder();
-            $subRN = $queryNameGenerator->generateJoinAlias('subResourceNode');
-            $subRL = $queryNameGenerator->generateJoinAlias('subResourceLink');
-            $subQueryBuilder->select('1')
-                ->from(ResourceNode::class, $subRN)
-                ->innerJoin("$subRN.resourceLinks", $subRL)
-                ->where("$subRL.course IS NULL")
-                ->andWhere("$subRL.session IS NULL")
-                ->andWhere("$subRL.group IS NULL")
-                ->andWhere("$subRL.user IS NULL")
-                ->andWhere("$subRN.id = $rootAlias.resourceNode");
-
-            $queryBuilder->andWhere($queryBuilder->expr()->not($queryBuilder->expr()->exists($subQueryBuilder->getDQL())));
-        }
+        $queryBuilder
+            ->innerJoin("$rootAlias.resourceNode", $resourceNodeAlias)
+            ->innerJoin("$resourceNodeAlias.resourceLinks", $resourceLinkAlias)
+            ->andWhere("$resourceLinkAlias.course IS NULL")
+            ->andWhere("$resourceLinkAlias.session IS NULL")
+            ->andWhere("$resourceLinkAlias.group IS NULL")
+            ->andWhere("$resourceLinkAlias.user IS NULL");
     }
 
     public function getDescription(string $resourceClass): array
diff --git a/src/CourseBundle/Repository/CCalendarEventRepository.php b/src/CourseBundle/Repository/CCalendarEventRepository.php
index 47970cf9f8..fa8aa38e6f 100644
--- a/src/CourseBundle/Repository/CCalendarEventRepository.php
+++ b/src/CourseBundle/Repository/CCalendarEventRepository.php
@@ -8,6 +8,7 @@ namespace Chamilo\CourseBundle\Repository;
 
 use Chamilo\CoreBundle\Entity\AbstractResource;
 use Chamilo\CoreBundle\Entity\Course;
+use Chamilo\CoreBundle\Entity\ResourceLink;
 use Chamilo\CoreBundle\Entity\Session;
 use Chamilo\CoreBundle\Entity\User;
 use Chamilo\CoreBundle\Repository\ResourceRepository;
@@ -76,4 +77,35 @@ final class CCalendarEventRepository extends ResourceRepository
 
         return $event;
     }
+
+    public function determineEventType(CCalendarEvent $event): string
+    {
+        $em = $this->getEntityManager();
+        $queryBuilder = $em->createQueryBuilder();
+
+        $queryBuilder
+            ->select('rl')
+            ->from(ResourceLink::class, 'rl')
+            ->innerJoin('rl.resourceNode', 'rn')
+            ->where('rn.id = :resourceNodeId')
+            ->setParameter('resourceNodeId', $event->getResourceNode()->getId());
+
+        $resourceLinks = $queryBuilder->getQuery()->getResult();
+
+        foreach ($resourceLinks as $link) {
+            if (null === $link->getCourse() && null === $link->getSession() && null === $link->getGroup() && null === $link->getUser()) {
+                return 'global';
+            }
+
+            if (null !== $link->getCourse()) {
+                return 'course';
+            }
+
+            if (null !== $link->getSession()) {
+                return 'session';
+            }
+        }
+
+        return 'personal';
+    }
 }