From d5d001d2b4477091a2513dc9799ca4a0b6aea120 Mon Sep 17 00:00:00 2001
From: Fred Tempez <frederic.tempez@outlook.com>
Date: Sat, 21 Oct 2023 11:46:58 +0200
Subject: [PATCH] Statistiques nombre de pages vues okay

---
 core/core.php                                       |  6 +++---
 core/module/course/course.php                       | 10 +++++++---
 core/module/course/view/userHistory/userHistory.php |  2 +-
 3 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/core/core.php b/core/core.php
index ed66536..ea69dcb 100644
--- a/core/core.php
+++ b/core/core.php
@@ -393,7 +393,7 @@ class common
 		setcookie('ZWII_UI', self::$i18nUI, time() + 3600, helper::baseUrl(false, false), '', false, false);
 		setcookie('ZWII_SITE_CONTENT', self::$siteContent, time() + 3600, helper::baseUrl(false, false), '', false, false);
 		setlocale(LC_ALL, self::$i18nUI);
-		
+
 		// Construit la liste des pages parents/enfants
 		if ($this->hierarchy['all'] === []) {
 			$this->buildHierarchy();
@@ -622,6 +622,7 @@ class common
 			'dir' => self::DATA_DIR . $path . '/',
 			'backup' => file_exists('site/data/.backup')
 		]);
+
 	}
 
 	/**
@@ -704,7 +705,6 @@ class common
 
 	/**
 	 * Fonction pour construire le tableau des pages
-	 * Appelée par le core uniquement
 	 */
 
 	private function buildHierarchy()
@@ -1395,7 +1395,7 @@ class common
 	public function saveLog($message = '')
 	{
 		// Journalisation
-		$dataLog = helper::dateUTF8('%Y %m %d', time(),  self::$i18nUI) . ' - ' . helper::dateUTF8('%H:%M', time(), self::$i18nUI);
+		$dataLog = helper::dateUTF8('%Y %m %d', time(), self::$i18nUI) . ' - ' . helper::dateUTF8('%H:%M', time(), self::$i18nUI);
 		$dataLog .= helper::getIp($this->getData(['config', 'connect', 'anonymousIp'])) . ';';
 		$dataLog .= empty($this->getUser('id')) ? 'visitor;' : $this->getUser('id') . ';';
 		$dataLog .= $message ? $this->getUrl() . ';' . $message : $this->getUrl();
diff --git a/core/module/course/course.php b/core/module/course/course.php
index 792005b..ea63dac 100644
--- a/core/module/course/course.php
+++ b/core/module/course/course.php
@@ -419,8 +419,9 @@ class course extends common
         // Statistiques du cours sélectionné calcul du nombre de pages
         $sumPages = 0;
         $data = json_decode(file_get_contents(self::DATA_DIR . $courseId . '/page.json'), true);
+        // Exclure les barres et les pages masquées
         foreach ($data['page'] as $pageId => $pageData) {
-            if ($pageData['block'] !== 'bar') {
+            if ($pageData['position'] > 0) {
                 $sumPages++;
             }
         }
@@ -471,7 +472,8 @@ class course extends common
 
             // Construction du tableau
             self::$courseUsers[] = [
-                $userId,
+                //$userId,
+                $viewPages . '=' . $sumPages,
                 $this->getData(['user', $userId, 'firstname']) . ' ' . $this->getData(['user', $userId, 'lastname']),
                 $pageId,
                 helper::dateUTF8('%d %B %Y - %H:%M', $maxTime),
@@ -673,10 +675,12 @@ class course extends common
         $userId = $this->getUrl(3);
         $history = $this->getData(['enrolment', $courseId, $userId]);
         $pages = json_decode(file_get_contents(self::DATA_DIR . $courseId . '/page.json'), true);
+        $pages = $pages ['page'];
+
         foreach ($history['history'] as $pageId => $time) {
             self::$userHistory[$pageId] = [
                 helper::dateUTF8('%d %B %Y - %H:%M:%S', $time),
-                $pages['page'][$pageId]['title'],
+                $pages[$pageId]['title'],
             ];
         }
 
diff --git a/core/module/course/view/userHistory/userHistory.php b/core/module/course/view/userHistory/userHistory.php
index 0e6d9f4..06ca9ed 100644
--- a/core/module/course/view/userHistory/userHistory.php
+++ b/core/module/course/view/userHistory/userHistory.php
@@ -10,7 +10,7 @@
 <?php if ($module::$userHistory): ?>
     <div class="row textAlignCenter">
         <div class="col8">
-            <?php echo template::table([6, 6], $module::$userHistory, ['Dernière consultation de cette page', 'Titre de la page'], ['id' => 'dataTables']); ?>
+            <?php echo template::table([1, 6, 5], $module::$userHistory, ['Ordre', 'Titre de la page', 'Dernière consultation de cette page'], ['id' => 'dataTables']); ?>
         </div>
     </div>
 <?php else: ?>