Bugs getUser getUrl

README.md

@@ -1,4 +1,4 @@
-# ZwiiCampus 1.21.02
+# ZwiiCampus 1.21.03
 
 ZwiiCampus (Learning Management System) est logiciel auteur destiné à mettre en ligne des tutoriels. Il dispose de plusieurs modalités d'ouverture et d'accès des contenus. Basé sur la version 13 du CMS Zwii, la structure logicielle est solide, le framework de Zwii est éprouvé.
 

core/core.php

@@ -51,7 +51,7 @@ class common
 	const ACCESS_TIMER = 1800;
 
 	// Numéro de version
-	const ZWII_VERSION = '1.21.02';
+	const ZWII_VERSION = '1.21.03';
 
 	// URL autoupdate
 	const ZWII_UPDATE_URL = 'https://forge.chapril.org/ZwiiCMS-Team/campus-update/raw/branch/master/';
@@ -641,7 +641,7 @@ class common
 	/**
 	 * Ecrire les données de la page
 	 * @param string pageId
-	 * @param array contenu de la page
+	 * @param string contenu de la page
 	 * @return int nombre d'octets écrits ou erreur
 	 */
 	public function setPage($page, $value, $path)
@@ -1663,7 +1663,7 @@ function makeThumb($src, $dest, $desired_width)
 				}
 				return $filter;
 			default:
-				return null;
+				return [];
 		}
 	}
 

core/module/course/course.php

@@ -231,8 +231,8 @@ class course extends common
         // Liste des enseignants pour le sélecteur d'auteurs
         $teachers = $this->getData(['user']);
         foreach ($teachers as $teacherId => $teacherInfo) {
-            if ($teacherInfo["group"] >= 2) {
-                self::$courseTeachers[$teacherId] = $teacherInfo["firstname"] . ' ' . $teacherInfo["lastname"];
+            if ($teacherInfo['role'] >= 2) {
+                self::$courseTeachers[$teacherId] = $teacherInfo['firstname'] . ' ' . $teacherInfo['lastname'];
             }
         }
 
@@ -307,8 +307,8 @@ class course extends common
         // Liste des enseignants pour le sélecteur d'auteurs
         $teachers = $this->getData(['user']);
         foreach ($teachers as $teacherId => $teacherInfo) {
-            if ($teacherInfo["group"] >= 2) {
-                self::$courseTeachers[$teacherId] = $teacherInfo["firstname"] . ' ' . $teacherInfo["lastname"];
+            if ($teacherInfo['role'] >= 2) {
+                self::$courseTeachers[$teacherId] = $teacherInfo['firstname'] . ' ' . $teacherInfo['lastname'];
             }
         }
 
@@ -366,8 +366,8 @@ class course extends common
         // Liste des enseignants pour le sélecteur d'auteurs
         $teachers = $this->getData(['user']);
         foreach ($teachers as $teacherId => $teacherInfo) {
-            if ($teacherInfo["group"] >= 2) {
-                self::$courseTeachers[$teacherId] = $teacherInfo["firstname"] . ' ' . $teacherInfo["lastname"];
+            if ($teacherInfo['role'] >= 2) {
+                self::$courseTeachers[$teacherId] = $teacherInfo['firstname'] . ' ' . $teacherInfo['lastname'];
             }
         }
 
@@ -414,7 +414,7 @@ class course extends common
 
         // Accès limité aux admins, à l'auteur ou éditeurs inscrits
         if (
-            $this->getUser('role') !== self::$actions[__FUNCTION__]
+            $this->getUser('permission', __CLASS__, __FUNCTION__) !== true
         ) {
             // Valeurs en sortie
             $this->addOutput([
@@ -447,7 +447,7 @@ class course extends common
 
         if (
             // Accès limité aux admins
-            $this->getUser('role') !== self::$actions[__FUNCTION__]
+            $this->getUser('permission', __CLASS__, __FUNCTION__) !== true
             // Le contenu n'existe pas
             || $this->getData(['course', $courseId]) === null
         ) {
@@ -487,7 +487,7 @@ class course extends common
 
         if (
             // Accès limité aux admins
-            $this->getUser('role') !== self::$actions[__FUNCTION__]
+            $this->getUser('permission', __CLASS__, __FUNCTION__) !== true
         ) {
             // Valeurs en sortie
             $this->addOutput([
@@ -526,7 +526,7 @@ class course extends common
 
         if (
             // Accès limité aux admins
-            $this->getUser('role') !== self::$actions[__FUNCTION__]
+            $this->getUser('permission', __CLASS__, __FUNCTION__) !== true
         ) {
             // Valeurs en sortie
             $this->addOutput([
@@ -557,7 +557,7 @@ class course extends common
     {
         if (
             // Accès limité aux admins
-            $this->getUser('role') !== self::$actions[__FUNCTION__]
+            $this->getUser('permission', __CLASS__, __FUNCTION__) !== true
         ) {
             // Valeurs en sortie
             $this->addOutput([
@@ -592,7 +592,7 @@ class course extends common
 
         if (
             // Accès limité aux admins
-            $this->getUser('role') !== self::$actions[__FUNCTION__]
+            $this->getUser('permission', __CLASS__, __FUNCTION__) !== true
         ) {
             // Valeurs en sortie
             $this->addOutput([

core/module/user/user.php

@@ -209,7 +209,7 @@ class user extends common
 			// L'utilisateur n'existe pas
 			$this->getData(['user', $this->getUrl(2)]) === null
 			// Groupe insuffisant
-			and ($this->getUrl('role') < self::ROLE_EDITOR)
+			and ($this->getUser('role') < self::ROLE_EDITOR)
 		) {
 			// Valeurs en sortie
 			$this->addOutput([
@@ -411,10 +411,10 @@ class user extends common
 				and (
 						// Impossible de s'auto-éditer
 					($this->getUser('id') === $this->getUrl(2)
-						and $this->getUrl('role') <= self::ROLE_VISITOR
+						and $this->getUser('role') <= self::ROLE_VISITOR
 					)
 					// Impossible d'éditer un autre utilisateur
-					or ($this->getUrl('role') < self::ROLE_EDITOR)
+					or ($this->getUser('role') < self::ROLE_EDITOR)
 				)
 			) {
 				// Valeurs en sortie
@@ -1423,8 +1423,8 @@ class user extends common
 					$notification = helper::translate('Captcha, identifiant ou mot de passe incorrects');
 					$logStatus = $captcha === true ? helper::translate('Erreur de mot de passe') : helper::translate('Erreur de captcha');
 					// Cas 1 le nombre de connexions est inférieur aux tentatives autorisées : incrément compteur d'échec
-					if ($this->getData(['user', $userId, 'connectFail']) < $this->getData(['config', 'connect', 'attempt'], false)) {
-						$this->setData(['user', $userId, 'connectFail', $this->getdata(['user', $userId, 'connectFail']) + 1], false);
+					if ($this->getData(['user', $userId, 'connectFail']) < $this->getData(['config', 'connect', 'attempt'])) {
+						$this->setData(['user', $userId, 'connectFail', $this->getdata(['user', $userId, 'connectFail']) + 1]);
 					}
 					// Cas 2 la limite du nombre de connexion est atteinte : placer le timer
 					if ($this->getdata(['user', $userId, 'connectFail']) == $this->getData(['config', 'connect', 'attempt'])) {