From 59de6bc32f0e54944df14c607d9a03f3570033eb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Tempez?= <frederic.tempez@outlook.com>
Date: Fri, 25 Oct 2024 09:25:16 +0200
Subject: [PATCH] =?UTF-8?q?1.14.05=20Stocke=20la=20progression=20d'un=20pa?=
 =?UTF-8?q?rticipant=20dans=20la=20bdd=20enrolment=20pour=20=C3=A9viter=20?=
 =?UTF-8?q?un=20recalcul=20=C3=A0=20l'avenir.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 core/module/course/course.php | 27 ++++++++++++++++++++++-----
 1 file changed, 22 insertions(+), 5 deletions(-)

diff --git a/core/module/course/course.php b/core/module/course/course.php
index 2307d98..e292c3e 100644
--- a/core/module/course/course.php
+++ b/core/module/course/course.php
@@ -734,8 +734,16 @@ class course extends common
                     $this->getData(['user', $userId, 'tags']),
                     template::button('userReport' . $userId, [
                         'href' => helper::baseUrl() . 'course/userReport/' . $courseId . '/' . $userId,
-                        'value' => $viewPages ? min(round(($viewPages * 100) / $sumPages, 1), 100) . ' %' : '0%',
-                        'disable' => empty($viewPages)
+                        /** La lecture de la progression s'effectue selon la nouvelle méthode (progression dans la base des enrolements)
+                         *  Soit avec l'ancienne méthode qui consiste à recalculer la progression.  
+                         *  TRANSITOIRE A SUPPRIMER EN FIN D'ANNEE
+                         **/
+                        'value' => array_key_exists('progress', $userValue)
+                            ? $userValue['progress']
+                            : ($viewPages ? min(round(($viewPages * 100) / $sumPages, 1), 100) . ' %' : '0%'),
+                        'disable' => empty($userValue['datePageView']),
+                        //'value' => $viewPages ? min(round(($viewPages * 100) / $sumPages, 1), 100) . ' %' : '0%',
+                        //'disable' => empty($viewPages)
                     ]),
                     template::button('userDelete' . $userId, [
                         'class' => 'userDelete buttonRed',
@@ -1405,7 +1413,14 @@ class course extends common
                     : $this->getData(['enrolment', $courseId, $userId, 'lastPageView']) . ' (supprimée)',
                     helper::dateUTF8('%d/%d/%Y', $this->getData(['enrolment', $courseId, $userId, 'datePageView'])),
                     helper::dateUTF8('%H:%M', $this->getData(['enrolment', $courseId, $userId, 'datePageView'])),
-                    number_format(min(round(($viewPages * 100) / $sumPages, 1) / 100, 1), 2, ',')
+                    /** La lecture de la progression s'effectue selon la nouvelle méthode (progression dans la base des enrolements)
+                     *  Soit avec l'ancienne méthode qui consiste à recalculer la progression.  
+                     *  TRANSITOIRE A SUPPRIMER EN FIN D'ANNEE
+                     **/
+                    array_key_exists('progress', $userValue)
+                    ? $userValue['progress']
+                    : ($viewPages ? min(round(($viewPages * 100) / $sumPages, 1), 100) . ' %' : '0%'),
+                    //number_format(min(round(($viewPages * 100) / $sumPages, 1) / 100, 1), 2, ','),
                 ];
 
                 // Synthèse des historiques
@@ -2057,7 +2072,7 @@ class course extends common
 
 
     /**
-     * Méthode pour afficher la progression dans les espaces.
+     * Méthode externe pour afficher la progression dans les espaces.
      * 
      * @param mixed $courseId
      * @param mixed $userId
@@ -2106,7 +2121,9 @@ class course extends common
             $courseId,
             $userId,
             [
-                'history' => [],
+                'lastPageView' => '',
+                'datePageView' => '',
+                'progress' => '',
             ]
         ]);
     }