From 61251ba8599dbe311587421f713ff8bb87c7ca09 Mon Sep 17 00:00:00 2001
From: Ilyassbennanii <ilyass.bennani@beeznest.com>
Date: Thu, 20 Nov 2025 11:10:19 +0100
Subject: [PATCH 1/5] Catalogue: Fix voting in catalogue refs #2860

---
 .../components/course/CatalogueCourseCard.vue | 237 +++++++++++++-----
 .../Controller/CatalogueController.php        |  31 ++-
 .../Helpers/TrackingStatsHelper.php           |  15 ++
 .../Helpers/UserRelCourseVoteHelper.php       |  43 ++++
 4 files changed, 261 insertions(+), 65 deletions(-)
 create mode 100644 src/CoreBundle/Helpers/UserRelCourseVoteHelper.php

diff --git a/assets/vue/components/course/CatalogueCourseCard.vue b/assets/vue/components/course/CatalogueCourseCard.vue
index ef93c423052..9226a5dbf7c 100644
--- a/assets/vue/components/course/CatalogueCourseCard.vue
+++ b/assets/vue/components/course/CatalogueCourseCard.vue
@@ -3,23 +3,23 @@
     class="course-card relative hover:shadow-lg transition duration-300 rounded-2xl overflow-hidden border border-gray-300 bg-white flex flex-col"
   >
     <div
-      v-if="course.categories?.length"
+      v-if="localCourse.categories?.length"
       class="absolute top-2 left-2 flex flex-wrap gap-1 z-30"
     >
-      <span
-        v-for="cat in course.categories"
-        :key="cat.id"
-        class="bg-support-5 text-white text-xs font-bold px-2 py-0.5 rounded"
-      >
-        {{ cat.title }}
-      </span>
+     <span
+       v-for="cat in localCourse.categories"
+       :key="cat.id"
+       class="bg-support-5 text-white text-xs font-bold px-2 py-0.5 rounded"
+     >
+       {{ cat.title }}
+     </span>
     </div>
     <span
-      v-if="course.courseLanguage"
+      v-if="localCourse.courseLanguage"
       class="absolute top-0 right-0 bg-support-4 text-white text-xs px-2 py-0.5 font-semibold rounded-bl-lg z-20"
     >
-      {{ getOriginalLanguageName(course.courseLanguage) }}
-    </span>
+     {{ getOriginalLanguageName(localCourse.courseLanguage) }}
+   </span>
 
     <Button
       v-if="allowDescription && showInfoPopup"
@@ -35,15 +35,15 @@
       :to="imageLink"
     >
       <img
-        :src="course.illustrationUrl"
-        :alt="course.title"
+        :src="localCourse.illustrationUrl"
+        :alt="localCourse.title"
         class="w-full object-cover"
       />
     </router-link>
     <img
       v-else
-      :src="course.illustrationUrl"
-      :alt="course.title"
+      :src="localCourse.illustrationUrl"
+      :alt="localCourse.title"
       class="w-full object-cover"
     />
     <div class="p-4 flex flex-col flex-grow gap-2">
@@ -52,62 +52,65 @@
         :to="titleLink"
         class="text-xl font-semibold"
       >
-        {{ course.title }}
+        {{ localCourse.title }}
       </router-link>
       <h3
         v-else-if="showTitle"
         class="text-xl font-semibold"
       >
-        {{ course.title }}
+        {{ localCourse.title }}
       </h3>
       <div
-        v-if="course.duration"
+        v-if="localCourse.duration"
         class="text-sm text-gray-700"
       >
         <strong>{{ $t("Duration") }}:</strong> {{ durationInHours }}
       </div>
 
       <div
-        v-if="course.dependencies?.length"
+        v-if="localCourse.dependencies?.length"
         class="text-sm text-gray-700"
       >
         <strong>{{ $t("Dependencies") }}:</strong>
-        {{ course.dependencies.map((dep) => dep.title).join(", ") }}
+        {{ localCourse.dependencies.map((dep) => dep.title).join(", ") }}
       </div>
 
       <div
-        v-if="course.price !== undefined"
+        v-if="localCourse.price !== undefined"
         class="text-sm text-gray-700"
       >
         <strong>{{ $t("Price") }}:</strong>
-        {{ course.price > 0 ? "S/. " + course.price.toFixed(2) : $t("Free") }}
+        {{ localCourse.price > 0 ? "S/. " + localCourse.price.toFixed(2) : $t("Free") }}
       </div>
       <div
-        v-if="course.teachers?.length"
+        v-if="localCourse.teachers?.length"
         class="text-sm text-gray-700"
       >
         <strong>{{ $t("Teachers") }}:</strong>
-        {{ course.teachers.map((t) => t.user.fullName).join(", ") }}
+        {{ localCourse.teachers.map((t) => t.user.fullName).join(", ") }}
+      </div>
+      <div class="mt-2 flex items-center">
+        <Rating
+          v-if="props.currentUserId"
+          :key="`rating-${localCourse.id}-${ratingResetKey}`"
+          :modelValue="displayRatingAvg"
+          :stars="5"
+          :cancel="true"
+          class="mt-2"
+          @update:modelValue="onUserRate"
+        />
       </div>
-      <Rating
-        v-if="props.currentUserId"
-        :model-value="course.userVote?.vote || 0"
-        :stars="5"
-        :cancel="false"
-        @change="emitRating"
-        class="mt-2"
-      />
       <div
         class="text-xs text-gray-600 mt-1"
-        v-if="course.popularity || course.userVote?.vote"
+        v-if="localCourse.popularity || localVote"
       >
-        {{ course.popularity || 0 }} Vote<span v-if="course.popularity !== 1">s</span>
+        {{ localCourse.popularity || 0 }} Vote<span v-if="localCourse.popularity !== 1">s</span>
         |
-        {{ course.nbVisits || 0 }} Visite<span v-if="course.nbVisits !== 1">s</span>
-        <span v-if="course.userVote?.vote">
-          |
-          {{ $t("Your vote") }} [{{ course.userVote.vote }}]
-        </span>
+        {{ localCourse.nbVisits || 0 }} Visite<span v-if="localCourse.nbVisits !== 1">s</span>
+        <span v-if="localVote">
+         |
+         {{ $t("Your vote") }} [{{ localVote }}]
+       </span>
       </div>
 
       <div
@@ -116,13 +119,13 @@
         class="text-sm text-gray-700"
       >
         <strong>{{ field.display_text }}:</strong>
-        {{ course.extra_fields?.[field.variable] ?? "-" }}
+        {{ localCourse.extra_fields?.[field.variable] ?? "-" }}
       </div>
 
       <div class="mt-auto pt-2">
         <router-link
-          v-if="course.subscribed"
-          :to="{ name: 'CourseHome', params: { id: course.id } }"
+          v-if="localCourse.subscribed"
+          :to="{ name: 'CourseHome', params: { id: localCourse.id } }"
         >
           <Button
             :label="$t('Go to the course')"
@@ -140,7 +143,7 @@
         />
 
         <Button
-          v-else-if="course.subscribe && props.currentUserId"
+          v-else-if="localCourse.subscribe && props.currentUserId && allowSelfSignup"
           :label="$t('Subscribe')"
           icon="pi pi-sign-in"
           class="w-full"
@@ -148,7 +151,7 @@
         />
 
         <Button
-          v-else-if="course.visibility === 2 && !course.subscribe && props.currentUserId"
+          v-else-if="props.currentUserId && !allowSelfSignup"
           :label="$t('Subscription not allowed')"
           icon="pi pi-ban"
           disabled
@@ -156,7 +159,7 @@
         />
 
         <Button
-          v-else-if="course.visibility === 1"
+          v-else-if="localCourse.visibility === 1"
           :label="$t('Private course')"
           icon="pi pi-lock"
           disabled
@@ -175,19 +178,14 @@
   </div>
   <CatalogueRequirementModal
     v-model="showDependenciesModal"
-    :course-id="course.id"
-    :session-id="course.sessionId || 0"
+    :course-id="localCourse.id"
+    :session-id="localCourse.sessionId || 0"
     :requirements="requirementList"
     :graph-image="graphImage"
   />
-  <Dialog
-    v-model:visible="showDescriptionDialog"
-    :header="course.title"
-    modal
-    class="w-96"
-  >
+  <Dialog v-model:visible="showDescriptionDialog" :header="localCourse.title" modal class="w-96">
     <p class="text-sm text-gray-700 whitespace-pre-line">
-      {{ course.description || $t("No description available") }}
+      {{ localCourse.description || $t("No description available") }}
     </p>
   </Dialog>
 </template>
@@ -195,7 +193,7 @@
 import Rating from "primevue/rating"
 import Button from "primevue/button"
 import Dialog from "primevue/dialog"
-import { computed, ref, onMounted } from "vue"
+import { computed, ref, onMounted, watch } from "vue"
 import { useRoute, useRouter } from "vue-router"
 import { useNotification } from "../../composables/notification"
 import { usePlatformConfig } from "../../store/platformConfig"
@@ -227,19 +225,130 @@ const platformConfigStore = usePlatformConfig()
 
 const showDescriptionDialog = ref(false)
 const showDependenciesModal = ref(false)
+const ratingResetKey = ref(0)
+
+
+// local copy for display / optimistic updates
+const localCourse = ref(JSON.parse(JSON.stringify(props.course || {})))
+// local reference for the vote
+const localVote = ref(props.course?.userVote?.vote || 0)
+// ensure numeric placeholders
+localCourse.value.ratingAvg = Number(localCourse.value.ratingAvg ?? 0)
+
+// --- fetch rating ---
+// adjust fetchRating to tolerate multiple formats returned by the API
+const fetchRating = async () => {
+  if (!localCourse.value?.id) return
+  try {
+    const sessionQuery = localCourse.value?.sessionId ? `?session=${localCourse.value.sessionId}` : ''
+    const res = await fetch(`/catalogue/api/courses/${localCourse.value.id}/rating${sessionQuery}`, {
+      headers: { Accept: 'application/json' },
+      credentials: 'same-origin',
+    })
+    if (!res.ok) return
+    const data = await res.json()
+    // robust fallback: average, avg, ratingAvg, etc.
+    localCourse.value.ratingAvg = Number(data.average ?? data.avg ?? data.ratingAvg ?? 0)
+  } catch (e) {
+    console.error('fetchRating error', e)
+  }
+}
+
+// call on mount
+onMounted(() => {
+  fetchRating()
+})
+
+// watcher on localVote: apply update + emit but do not alter the local average
+watch(
+  localVote,
+  (newVote, oldVote) => {
+    if (newVote === oldVote) return
+
+
+    const prevVote = props.course?.userVote?.vote ?? oldVote ?? 0
+
+
+    if ((localCourse.value.popularity === undefined) && props.course?.popularity !== undefined) {
+      localCourse.value.popularity = props.course.popularity
+    }
+
+
+    if (prevVote === 0 && newVote > 0) {
+      localCourse.value.popularity = (localCourse.value.popularity || 0) + 1
+    } else if (prevVote > 0 && newVote === 0) {
+      localCourse.value.popularity = Math.max((localCourse.value.popularity || 1) - 1, 0)
+    }
+
+
+    localCourse.value.userVote = { vote: newVote }
+
+
+    // Emit to the parent to persist (the parent must call the API)
+    emit("rate", { value: newVote, course: props.course })
+  },
+  { immediate: false },
+)
+
+const allowSelfSignup = computed(() => {
+  if (localCourse.value?.allow_self_signup !== undefined) return Boolean(localCourse.value.allow_self_signup)
+  if (localCourse.value?.allowSelfSignup !== undefined) return Boolean(localCourse.value.allowSelfSignup)
+  return localCourse.value?.visibility === 0
+})
+localCourse.value.nbVisits = Number(localCourse.value.nbVisits ?? 0)
+
+// fetch visits
+const fetchVisits = async () => {
+  if (!localCourse.value?.id) return
+  try {
+    const sessionQuery = localCourse.value?.sessionId ? `?session=${localCourse.value.sessionId}` : ''
+    const res = await fetch(`/catalogue/api/courses/${localCourse.value.id}/visits${sessionQuery}`, {
+      headers: { 'Accept': 'application/json' },
+      credentials: 'same-origin',
+    })
+    if (!res.ok) return
+    const data = await res.json()
+    localCourse.value.nbVisits = Number(data.visits ?? 0)
+  } catch (e) {
+    console.error('fetchVisits error', e)
+  }
+}
+
+onMounted(fetchVisits)
+
+
 
 const allowDescription = computed(
   () => platformConfigStore.getSetting("catalog.show_courses_descriptions_in_catalog") !== "false",
 )
 
 const durationInHours = computed(() => {
-  if (!props.course.duration) return "-"
-  const duration = props.course.duration / 3600
-  return props.course.durationExtra ? `${duration.toFixed(2)}+ h` : `${duration.toFixed(2)} h`
+  if (!localCourse.value.duration) return "-"
+  const duration = localCourse.value.duration / 3600
+  return localCourse.value.durationExtra ? `${duration.toFixed(2)}+ h` : `${duration.toFixed(2)} h`
+})
+
+
+// the display computed to prioritize the local value (optimistic/fetch)
+const displayRatingAvg = computed(() => {
+  return Number(
+    localCourse.value?.ratingAvg ??
+    props.course?.average ??
+    props.course?.avg ??
+    props.course?.avgRating ??
+    localCourse.value?.avg ??
+    0
+  )
 })
 
-const emitRating = (event) => {
-  emit("rate", { value: event.value, course: props.course })
+// computed used by the Rating component: shows the user's vote if available, otherwise the average
+const onUserRate = (val) => {
+  // updates localVote -> watcher handles popularity + emit("rate")
+  localVote.value = Number(val || 0)
+  // force the remount of the Rating component to return to displaying the average
+  setTimeout(() => {
+    ratingResetKey.value++
+  }, 0)
 }
 
 const subscribing = ref(false)
@@ -313,7 +422,7 @@ const imageLink = computed(() => {
         : null
 
   if (routeName && routeExists(routeName)) {
-    return { name: routeName, params: { id: props.course.id } }
+    return { name: routeName, params: { id: localCourse.value.id } }
   }
 
   return null
@@ -323,7 +432,7 @@ const titleLink = computed(() => {
   const routeName = linkSettings.value.title_url === "course_home" ? "CourseHome" : null
 
   if (routeName && routeExists(routeName)) {
-    return { name: routeName, params: { id: props.course.id } }
+    return { name: routeName, params: { id: localCourse.value.id } }
   }
 
   return null
@@ -336,8 +445,8 @@ const showInfoPopup = computed(() => {
 })
 
 const { isLocked, hasRequirements, requirementList, graphImage, fetchStatus } = useCourseRequirementStatus(
-  props.course.id,
-  props.course.sessionId || 0,
+  () => localCourse.value.id,
+  () => localCourse.value.sessionId || 0,
 )
 
 onMounted(() => {
diff --git a/src/CoreBundle/Controller/CatalogueController.php b/src/CoreBundle/Controller/CatalogueController.php
index 10659c074e3..c2c9cacc278 100644
--- a/src/CoreBundle/Controller/CatalogueController.php
+++ b/src/CoreBundle/Controller/CatalogueController.php
@@ -38,9 +38,33 @@ public function __construct(
         private readonly UserHelper $userHelper,
         private readonly AccessUrlHelper $accessUrlHelper,
         private readonly CourseRepository $courseRepository,
-        private readonly SessionRepository $sessionRepository
+        private readonly TrackingStatsHelper $trackingStatsHelper,
+        private readonly SessionRepository $sessionRepository,
+        private readonly UserRelCourseVoteHelper $userRelCourseVoteHelper
     ) {}
 
+    #[Route('/api/courses/{id}/rating', name: 'api_course_rating', methods: ['GET'])]
+    public function courseRating(Course $course, Request $request): JsonResponse
+    {
+        $sessionId = $request->query->getInt('session', 0);
+        $session = $sessionId > 0 ? $this->sessionRepository->find($sessionId) : null;
+        $res = $this->userRelCourseVoteHelper->getCourseRating($course, $session);
+
+        return $this->json([
+            'average' => $res['avg'],
+            'count' => $res['count'],
+        ]);
+    }
+    #[Route('/api/courses/{id}/visits', name: 'api_course_visits', methods: ['GET'])]
+    public function courseVisits(Course $course, Request $request): JsonResponse
+    {
+        $sessionId = $request->query->getInt('session', 0);
+        $session = $sessionId > 0 ? $this->sessionRepository->find($sessionId) : null;
+        $count = $this->trackingStatsHelper->getCourseVisits($course, $session);
+
+        return $this->json(['visits' => $count]);
+    }
+
     #[Route('/courses-list', name: 'chamilo_core_catalogue_courses_list', methods: ['GET'])]
     public function listCourses(): JsonResponse
     {
@@ -534,6 +558,11 @@ public function autoSubscribeCourse(int $courseId, SettingsManager $settings): J
             return $this->json(['error' => 'Course or user not found'], 400);
         }
 
+        $isPrivileged = $this->isGranted('ROLE_ADMIN') || $this->isGranted('ROLE_TEACHER') || $this->isGranted('ROLE_SESSION_ADMIN');
+        if (!$course->getAllowSelfSignup() && !$isPrivileged) {
+            return $this->json(['error' => 'Self sign up not allowed for this course'], 403);
+        }
+
         $useAutoSession = 'true' === $settings->getSetting('catalog.course_subscription_in_user_s_session', true);
 
         if ($useAutoSession) {
diff --git a/src/CoreBundle/Helpers/TrackingStatsHelper.php b/src/CoreBundle/Helpers/TrackingStatsHelper.php
index 4930879ebff..1fbf9dc8db4 100644
--- a/src/CoreBundle/Helpers/TrackingStatsHelper.php
+++ b/src/CoreBundle/Helpers/TrackingStatsHelper.php
@@ -75,6 +75,21 @@ public function getUserAvgLpProgress(User $user, Course $course, ?Session $sessi
 
         return ['avg' => $avg, 'count' => $count];
     }
+    public function getCourseVisits(Course $course, ?Session $session = null): int
+    {
+        $conn = $this->em->getConnection();
+        $sql = 'SELECT COUNT(course_access_id) FROM track_e_course_access WHERE c_id = :cId';
+        $params = ['cId' => $course->getId()];
+        $types = ['cId' => ParameterType::INTEGER];
+
+        if ($session !== null) {
+            $sql .= ' AND session_id = :sessionId';
+            $params['sessionId'] = $session->getId();
+            $types['sessionId'] = ParameterType::INTEGER;
+        }
+        $count = $conn->fetchOne($sql, $params, $types);
+        return (int) $count;
+    }
 
     /**
      * Certificates of a user within a course/session.
diff --git a/src/CoreBundle/Helpers/UserRelCourseVoteHelper.php b/src/CoreBundle/Helpers/UserRelCourseVoteHelper.php
new file mode 100644
index 00000000000..e53341bdb4d
--- /dev/null
+++ b/src/CoreBundle/Helpers/UserRelCourseVoteHelper.php
@@ -0,0 +1,43 @@
+<?php
+declare(strict_types=1);
+
+namespace Chamilo\CoreBundle\Helpers;
+
+
+use Chamilo\CoreBundle\Entity\Course;
+use Chamilo\CoreBundle\Entity\Session;
+use Chamilo\CoreBundle\Entity\UserRelCourseVote;
+use Doctrine\DBAL\ParameterType;
+use Doctrine\ORM\EntityManagerInterface;
+
+class UserRelCourseVoteHelper
+{
+    public function __construct(private readonly EntityManagerInterface $em) {}
+
+
+    public function getCourseRating(Course $course, ?Session $session = null): array
+    {
+        $qb = $this->em->createQueryBuilder()
+            ->select('AVG(v.vote) AS avgVote', 'COUNT(v.id) AS countVotes')
+            ->from(UserRelCourseVote::class, 'v')
+            ->where('v.course = :course')
+            ->setParameter('course', $course->getId(), ParameterType::INTEGER);
+
+
+        if ($session !== null) {
+            $qb->andWhere('v.session = :session')
+                ->setParameter('session', $session->getId(), ParameterType::INTEGER);
+        } else {
+            $qb->andWhere('v.session IS NULL');
+        }
+
+
+        $row = $qb->getQuery()->getSingleResult();
+        $avg = isset($row['avgVote']) && $row['avgVote'] !== null ? round((float) $row['avgVote'], 2) : 0.0;
+        $count = isset($row['countVotes']) ? (int) $row['countVotes'] : 0;
+
+        return ['avg' => $avg, 'average' => $avg, 'count' => $count];
+
+
+    }
+}

From 0da704c70e86502b93a31b72940973c5eb044d35 Mon Sep 17 00:00:00 2001
From: Ilyassbennanii <ilyass.bennani@beeznest.com>
Date: Fri, 21 Nov 2025 15:28:45 +0100
Subject: [PATCH 2/5] Removed privilege check for self sign-up in course.

---
 src/CoreBundle/Controller/CatalogueController.php | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/src/CoreBundle/Controller/CatalogueController.php b/src/CoreBundle/Controller/CatalogueController.php
index c2c9cacc278..a1f0c4685ac 100644
--- a/src/CoreBundle/Controller/CatalogueController.php
+++ b/src/CoreBundle/Controller/CatalogueController.php
@@ -20,6 +20,8 @@
 use Chamilo\CoreBundle\Repository\Node\CourseRepository;
 use Chamilo\CoreBundle\Repository\SessionRepository;
 use Chamilo\CoreBundle\Settings\SettingsManager;
+use Chamilo\CoreBundle\Helpers\TrackingStatsHelper;
+use Chamilo\CoreBundle\Helpers\UserRelCourseVoteHelper;
 use DateTime;
 use Doctrine\ORM\EntityManagerInterface;
 use ExtraField;
@@ -558,11 +560,6 @@ public function autoSubscribeCourse(int $courseId, SettingsManager $settings): J
             return $this->json(['error' => 'Course or user not found'], 400);
         }
 
-        $isPrivileged = $this->isGranted('ROLE_ADMIN') || $this->isGranted('ROLE_TEACHER') || $this->isGranted('ROLE_SESSION_ADMIN');
-        if (!$course->getAllowSelfSignup() && !$isPrivileged) {
-            return $this->json(['error' => 'Self sign up not allowed for this course'], 403);
-        }
-
         $useAutoSession = 'true' === $settings->getSetting('catalog.course_subscription_in_user_s_session', true);
 
         if ($useAutoSession) {

From 160d4c1005e1795e76a74f552e5f62207d921c87 Mon Sep 17 00:00:00 2001
From: Ilyassbennanii <ilyass.bennani@beeznest.com>
Date: Fri, 21 Nov 2025 15:54:01 +0100
Subject: [PATCH 3/5] Initialize popularity and nbVisits as numbers, and use
 oldVote (fallback to props.course) for prevVote so changing a rating doesn't
 alter the vote count.

---
 .../components/course/CatalogueCourseCard.vue | 20 +++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/assets/vue/components/course/CatalogueCourseCard.vue b/assets/vue/components/course/CatalogueCourseCard.vue
index 9226a5dbf7c..1e5e6ef8ba5 100644
--- a/assets/vue/components/course/CatalogueCourseCard.vue
+++ b/assets/vue/components/course/CatalogueCourseCard.vue
@@ -234,6 +234,10 @@ const localCourse = ref(JSON.parse(JSON.stringify(props.course || {})))
 const localVote = ref(props.course?.userVote?.vote || 0)
 // ensure numeric placeholders
 localCourse.value.ratingAvg = Number(localCourse.value.ratingAvg ?? 0)
+// ensure popularity / nbVisits are numbers
+localCourse.value.popularity = Number(localCourse.value.popularity ?? props.course?.popularity ?? 0)
+localCourse.value.nbVisits = Number(localCourse.value.nbVisits ?? 0)
+
 
 // --- fetch rating ---
 // adjust fetchRating to tolerate multiple formats returned by the API
@@ -265,25 +269,23 @@ watch(
   (newVote, oldVote) => {
     if (newVote === oldVote) return
 
+    // use the previous local value first (oldVote), fallback to props (may be undefined/stale)
+    const prevVote = oldVote ?? props.course?.userVote?.vote ?? 0
 
-    const prevVote = props.course?.userVote?.vote ?? oldVote ?? 0
-
-
-    if ((localCourse.value.popularity === undefined) && props.course?.popularity !== undefined) {
-      localCourse.value.popularity = props.course.popularity
+    // ensure popularity numeric
+    if (localCourse.value.popularity === undefined || localCourse.value.popularity === null) {
+      localCourse.value.popularity = Number(props.course?.popularity ?? 0)
     }
 
-
+    // only change popularity when crossing from 0 -> >0 or >0 -> 0
     if (prevVote === 0 && newVote > 0) {
       localCourse.value.popularity = (localCourse.value.popularity || 0) + 1
     } else if (prevVote > 0 && newVote === 0) {
       localCourse.value.popularity = Math.max((localCourse.value.popularity || 1) - 1, 0)
     }
 
-
     localCourse.value.userVote = { vote: newVote }
 
-
     // Emit to the parent to persist (the parent must call the API)
     emit("rate", { value: newVote, course: props.course })
   },
@@ -295,8 +297,6 @@ const allowSelfSignup = computed(() => {
   if (localCourse.value?.allowSelfSignup !== undefined) return Boolean(localCourse.value.allowSelfSignup)
   return localCourse.value?.visibility === 0
 })
-localCourse.value.nbVisits = Number(localCourse.value.nbVisits ?? 0)
-
 // fetch visits
 const fetchVisits = async () => {
   if (!localCourse.value?.id) return

From 9828012a667133c554d665713e06eb6aedd360ea Mon Sep 17 00:00:00 2001
From: Yannick Warnier <ywarnier@users.noreply.github.com>
Date: Sat, 22 Nov 2025 10:41:02 +0100
Subject: [PATCH 4/5] Minor: Use dynamic table name in getCourseVisits method

---
 src/CoreBundle/Helpers/TrackingStatsHelper.php | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/CoreBundle/Helpers/TrackingStatsHelper.php b/src/CoreBundle/Helpers/TrackingStatsHelper.php
index 1fbf9dc8db4..a403ecf407e 100644
--- a/src/CoreBundle/Helpers/TrackingStatsHelper.php
+++ b/src/CoreBundle/Helpers/TrackingStatsHelper.php
@@ -78,7 +78,8 @@ public function getUserAvgLpProgress(User $user, Course $course, ?Session $sessi
     public function getCourseVisits(Course $course, ?Session $session = null): int
     {
         $conn = $this->em->getConnection();
-        $sql = 'SELECT COUNT(course_access_id) FROM track_e_course_access WHERE c_id = :cId';
+        $table = Database::get_main_table(TABLE_STATISTIC_TRACK_E_COURSE_ACCESS);
+        $sql = 'SELECT COUNT(course_access_id) FROM '.$table.' WHERE c_id = :cId';
         $params = ['cId' => $course->getId()];
         $types = ['cId' => ParameterType::INTEGER];
 

From 7a48249803445d42fd9e16f1565344427a391044 Mon Sep 17 00:00:00 2001
From: Ilyassbennanii <ilyass.bennani@beeznest.com>
Date: Mon, 24 Nov 2025 08:44:52 +0100
Subject: [PATCH 5/5] Remove duplicate allowSelfSignup

---
 assets/vue/components/course/CatalogueCourseCard.vue | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/assets/vue/components/course/CatalogueCourseCard.vue b/assets/vue/components/course/CatalogueCourseCard.vue
index 2a20cdb9e05..1e5e6ef8ba5 100644
--- a/assets/vue/components/course/CatalogueCourseCard.vue
+++ b/assets/vue/components/course/CatalogueCourseCard.vue
@@ -452,10 +452,4 @@ const { isLocked, hasRequirements, requirementList, graphImage, fetchStatus } =
 onMounted(() => {
   fetchStatus()
 })
-
-const allowSelfSignup = computed(() => {
-  if (props.course?.allow_self_signup !== undefined) return Boolean(props.course.allow_self_signup)
-  if (props.course?.allowSelfSignup !== undefined) return Boolean(props.course.allowSelfSignup)
-  return props.course?.visibility === 0
-})
 </script>