From 00ae9305cac93ba01c379897ac91b2a5528eaa16 Mon Sep 17 00:00:00 2001 From: Etienne Pallier Date: Tue, 26 Feb 2019 16:25:33 +0100 Subject: [PATCH] Bugfix "Categorie sélectionnée ne met plus à jour le domaine" --- README.md | 8 +++----- src/Controller/SurCategoriesController.php | 17 +++++++++++------ src/Model/Table/LdapConnectionsTable.php | 2 +- 3 files changed, 15 insertions(+), 12 deletions(-) diff --git a/README.md b/README.md index 16ccc67..1fbe4c6 100644 --- a/README.md +++ b/README.md @@ -53,12 +53,10 @@ Logiciel testé et validé sur les configurations suivantes : VERSION ACTUELLE -Date: 25/02/2019 -Version: 2.10.19 +Date: 26/02/2019 +Version: 2.10.20 Author: EP - LDAP refactorisation && optimisation (5) - - LdapConnectionsTable.php/getAllLdapUsers() remplacé par getAllLdapUsersNEW - - LDAP refactor progressif pour remettre le mode LDAP authentifié (pour CRAL) + Bugfix "Categorie sélectionnée ne met plus à jour le domaine" IMPORTANT: - Pour connaitre la version actuelle, taper "./VERSION" diff --git a/src/Controller/SurCategoriesController.php b/src/Controller/SurCategoriesController.php index a88055b..6b22b9c 100755 --- a/src/Controller/SurCategoriesController.php +++ b/src/Controller/SurCategoriesController.php @@ -26,6 +26,7 @@ class SurCategoriesController extends AppController */ public function isAuthorized($user) { + $action = $this->getActionPassed(); /* * $configuration = $this->confLabinvent; * $role = TableRegistry::get('Users')->find()->where(['username' => $user[$configuration->authentificationType_ldap][0]])->first()['role']; @@ -34,13 +35,17 @@ class SurCategoriesController extends AppController * // Super-Admin peut accéder à chaque action * if($role == 'Super Administrateur') return true; * - * // Administration peut ajouter, supprimer ou modifier une sur-categorie + * // Administration peut ajouter, supprimer ou modifier une sous categorie * if($role == 'Administration' && in_array($action,['add','delete','edit'])) return true; - * - * if (in_array($action, ['view', 'index'])) { - * return true; - * } - * + */ + + // if (in_array($action, ['getByCategorie', 'view', 'index'])) { + if (in_array($action, [ + 'getFromCategorie' + ])) + return true; + + /* * if($this->userHasRoleAtLeast('Administration Plus')) { * if($action != 'delete') return true; * } diff --git a/src/Model/Table/LdapConnectionsTable.php b/src/Model/Table/LdapConnectionsTable.php index dc11bff..8c89143 100755 --- a/src/Model/Table/LdapConnectionsTable.php +++ b/src/Model/Table/LdapConnectionsTable.php @@ -232,7 +232,6 @@ class LdapConnectionsTable extends AppTable /** * @return $users_fetched or FALSE - */ // REAL or FAKE LDAP public function getAllLdapUsersOLD() { @@ -259,6 +258,7 @@ class LdapConnectionsTable extends AppTable return false; } + */ /** * @return $users_fetched or FALSE -- libgit2 0.21.2