From 8392646428c7a0c98cc872def83243a16f201dcb Mon Sep 17 00:00:00 2001
From: Etienne Pallier <epallier@irap.omp.eu>
Date: Wed, 30 Aug 2017 10:10:35 +0200
Subject: [PATCH] renommages de variables et fonctions pour clarifier le code

---
 src/Controller/AppController.php   | 51 ++++++++++++++++++++++-----------------------------
 src/Template/Element/menu_view.ctp | 48 ++++++++++++++++++++++++------------------------
 src/Template/Emprunts/find.ctp     |  4 ++--
 src/Template/Materiels/edit.ctp    |  8 ++++----
 src/Template/Materiels/find.ctp    | 12 ++++++------
 src/Template/Materiels/index.ctp   | 12 ++++++------
 src/Template/Materiels/view.ctp    | 89 +++++++++++++++++++++++++++++++++++++++++++++++------------------------------------------
 src/Template/Pages/home_app.ctp    |  8 ++++----
 src/Template/Suivis/find.ctp       |  4 ++--
 src/Template/Suivis/index.ctp      |  4 ++--
 src/Template/Suivis/view.ctp       |  8 ++++----
 11 files changed, 123 insertions(+), 125 deletions(-)

diff --git a/src/Controller/AppController.php b/src/Controller/AppController.php
index 420bff4..56c5717 100755
--- a/src/Controller/AppController.php
+++ b/src/Controller/AppController.php
@@ -163,7 +163,7 @@ class AppController extends Controller {
         return false;
     }
     
-    public function getUser($user=null) { return $this->getTablePriviledgedUserFromCurrentSessionUser($user); }
+    //public function getUser($user=null) { return $this->getTablePriviledgedUserFromCurrentSessionUser($user); }
     public function getTablePriviledgedUserFromCurrentSessionUser($user=null) {
         if (! $this->CURRENT_USER) {
             $configuration = $this->confLabinvent;
@@ -178,23 +178,14 @@ class AppController extends Controller {
         }
         return $this->CURRENT_USER;
     }
-    public function getUserRole($user=null) { return $this->getPriviledgedUserRole($user); }
-    public function getPriviledgedUserRole($user=null) {
-        $priviledgedUser = $this->getTablePriviledgedUserFromCurrentSessionUser($user);
+    public function getUserRole($user=null) { 
+        $priviledgedUser = $this->getPriviledgedUserRole($user); 
         // default role is "Utilisateur" (for people who are not in the table utilisateurs)
         if (! $priviledgedUser) return 'Utilisateur';
         return $priviledgedUser['role'];
-        /*
-        if (! $this->CURRENT_ROLE) {
-            $configuration = $this->confLabinvent;
-            $user = TableRegistry::get('Users')->find()
-            ->where(['username' => $this->LdapAuth->user($configuration->authentificationType_ldap)[0]])
-            //->where(['username' => $this->LdapAuth->user('cn')[0]])
-            ->first();
-            $this->CURRENT_ROLE = $user['role'];
-        }
-        return $this->CURRENT_ROLE;
-        */
+    }
+    public function getPriviledgedUserRole($user=null) {
+        return $this->getTablePriviledgedUserFromCurrentSessionUser($user);
     }
     
     private function userHasRole($expectedRole, $ORMORE=false) {
@@ -301,9 +292,8 @@ class AppController extends Controller {
         $this->set('configuration', $configuration);
         $this->request->session()->write("authType", $configuration->authentificationType_ldap);
 
-        //TODO: mettre ca au propre, le cas ou l'utilisateur a un role UTILISATEUR doit etre traité AVANT, en amont, et pas ici, bien trop tard
-        $user = $this->getUser();
-        $role = $this->getUserRole();
+        //$user = $this->getUser();
+        $priviledgedUser = $this->getTablePriviledgedUserFromCurrentSessionUser();
         /*
         $user = TableRegistry::get('Users')->find()
         	->where(['username' => $this->LdapAuth->user($configuration->authentificationType_ldap)[0]])
@@ -312,8 +302,11 @@ class AppController extends Controller {
         if ($role == null)
             $role = 'Utilisateur';
         */
+        // Role = 'Utilisateur', 'Responsable", ...
+        $role = $this->getUserRole();
         $this->set('role', $role);
-        
+
+        // Profile = PROFILE_USER (=1), PROFILE_RESPONSABLE (=2), ...
         //$profile = $this->allProfiles["$role"];
         $profile = self::PROFILES["$role"];
         $this->set('profile', $profile);
@@ -336,7 +329,7 @@ class AppController extends Controller {
         $this->set('USER_IS_ADMIN_OR_MORE', $USER_IS_ADMIN_OR_MORE);
         $this->set('USER_IS_ADMINPLUS_OR_MORE', $USER_IS_ADMINPLUS_OR_MORE);
         
-        $this->set('userConnected', $user);
+        $this->set('priviledgedUser', $priviledgedUser);
 
         $this->set('idGmNa', TableRegistry::get('GroupesMetiers')->find()->where(['nom =' => 'N/A'])->first()['id']);
         $this->set('idGtNa', TableRegistry::get('GroupesThematiques')->find()->where(['nom =' => 'N/A'])->first()['id']);
@@ -349,16 +342,16 @@ class AppController extends Controller {
         };
         $this->set('displayElement', $displayElement);
         
+        // Pass this function to all views
+        // @todo : Si cette fonction ne concerne que SuivisController, il faut la déplacer dans ce controleur
         $dateProchainControleVerif = function($t) {
-        	$time = Time::now(); // On récupère la date et l'heure actuelles
-        	$today = new \DateTime((new date("$time->year-$time->month-$time->day"))->format('Y-m-d'));
-        	$time1 = new time($t);
-        	$dateTime1 = new \DateTime((new date("$time1->year-$time1->month-$time1->day"))->format('y-m-d'));
-        
-        	$interval = ($today->diff($dateTime1));
-        	$strInterval = $interval->format('%a');
-        
-        	return (int) $strInterval;
+            	$time = Time::now(); // On récupère la date et l'heure actuelles
+            	$today = new \DateTime((new date("$time->year-$time->month-$time->day"))->format('Y-m-d'));
+            	$time1 = new time($t);
+            	$dateTime1 = new \DateTime((new date("$time1->year-$time1->month-$time1->day"))->format('y-m-d'));
+            	$interval = ($today->diff($dateTime1));
+            	$strInterval = $interval->format('%a');
+            	return (int) $strInterval;
         };
         $this->set('dateProchainControleVerif', $dateProchainControleVerif);
     }
diff --git a/src/Template/Element/menu_view.ctp b/src/Template/Element/menu_view.ctp
index dc1b24c..ffa1f68 100755
--- a/src/Template/Element/menu_view.ctp
+++ b/src/Template/Element/menu_view.ctp
@@ -41,10 +41,10 @@ use Cake\ORM\TableRegistry;
 			if((($role == 'Utilisateur'|| $role == 'Responsable') && (in_array($username, [$materiel->nom_createur, $materiel->nom_responsable]))) || (in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur']))) {
 				$isAuthorized = true;
 			} else if($role == 'Responsable') {
-				if(isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $userConnected->groupes_metier_id) {
+				if(isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id) {
 					$isAuthorized = true;
 				}
-				else if (isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id) {
+				else if (isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id) {
 					$isAuthorized = true;
 				}
 				else {
@@ -66,10 +66,10 @@ use Cake\ORM\TableRegistry;
 					$isAuthorized = true;
 				}
 				else if($role == 'Responsable') {
-					if(isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $userConnected->groupes_metier_id) {
+					if(isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $priviledgedUser->groupes_metier_id) {
 						$isAuthorized = true;
 					}
-					else if (isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $userConnected->groupe_thematique_id) {
+					else if (isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $priviledgedUser->groupe_thematique_id) {
 						$isAuthorized = true;
 					}
 					else {
@@ -110,11 +110,11 @@ use Cake\ORM\TableRegistry;
 				$id = $doc->get('suivi_id');
 				if ($role == 'Responsable') {
 					$suiviTable = TableRegistry::get('Suivis');
-					if(isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa) {
-						$isAuthorized = ($suiviTable->exists(['id' => $id, 'groupes_metier_id' => $userConnected->groupes_metier_id]));
+					if(isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa) {
+						$isAuthorized = ($suiviTable->exists(['id' => $id, 'groupes_metier_id' => $priviledgedUser->groupes_metier_id]));
 					}
-					else if (isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa) {
-						$isAuthorized = ($suiviTable->exists(['id' => $id, 'groupes_thematique_id' => $userConnected->groupe_thematique_id]));
+					else if (isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa) {
+						$isAuthorized = ($suiviTable->exists(['id' => $id, 'groupes_thematique_id' => $priviledgedUser->groupe_thematique_id]));
 					}
 					else {
 						$isAuthorized = false;
@@ -127,11 +127,11 @@ use Cake\ORM\TableRegistry;
 				$materielTable = TableRegistry::get('Materiels');
 				if ($role == 'Responsable') {
 				
-					if(isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa) {
-						$isAuthorized = ($materielTable->exists(['id' => $id, 'groupes_metier_id' => $userConnected->groupes_metier_id]));
+					if(isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa) {
+						$isAuthorized = ($materielTable->exists(['id' => $id, 'groupes_metier_id' => $priviledgedUser->groupes_metier_id]));
 					}
-					else if (isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa) {
-						$isAuthorized = ($materielTable->exists(['id' => $id, 'groupes_thematique_id' => $userConnected->groupe_thematique_id]));
+					else if (isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa) {
+						$isAuthorized = ($materielTable->exists(['id' => $id, 'groupes_thematique_id' => $priviledgedUser->groupe_thematique_id]));
 					}
 					else {
 						$isAuthorized = false;
@@ -185,10 +185,10 @@ use Cake\ORM\TableRegistry;
 					$isAuthorized = true;
 				} 
 				else if($role == 'Responsable') {
-					if(isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $userConnected->groupes_metier_id) {
+					if(isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id) {
 						$isAuthorized = true;
 					}
-					else if (isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id) {
+					else if (isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id) {
 						$isAuthorized = true;
 					}
 					else {
@@ -206,10 +206,10 @@ use Cake\ORM\TableRegistry;
 						$isAuthorized = true;
 					} 
 					else if($role == 'Responsable') {
-						if(isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $userConnected->groupes_metier_id) {
+						if(isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $priviledgedUser->groupes_metier_id) {
 							$isAuthorized = true;
 						}
-						else if (isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $userConnected->groupe_thematique_id) {
+						else if (isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $priviledgedUser->groupe_thematique_id) {
 							$isAuthorized = true;
 						}
 						else {
@@ -238,11 +238,11 @@ use Cake\ORM\TableRegistry;
 				$id = $doc->get('suivi_id');
 				if ($role == 'Responsable') {
 					$suiviTable = TableRegistry::get('Suivis');
-					if(isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa) {
-						$isAuthorized = ($suiviTable->exists(['id' => $id, 'groupes_metier_id' => $userConnected->groupes_metier_id]));
+					if(isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa) {
+						$isAuthorized = ($suiviTable->exists(['id' => $id, 'groupes_metier_id' => $priviledgedUser->groupes_metier_id]));
 					}
-					else if (isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa) {
-						$isAuthorized = ($suiviTable->exists(['id' => $id, 'groupes_thematique_id' => $userConnected->groupe_thematique_id]));
+					else if (isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa) {
+						$isAuthorized = ($suiviTable->exists(['id' => $id, 'groupes_thematique_id' => $priviledgedUser->groupe_thematique_id]));
 					}
 					else {
 						$isAuthorized = false;
@@ -254,11 +254,11 @@ use Cake\ORM\TableRegistry;
 			} else {
 				$materielTable = TableRegistry::get('Materiels');
 				if ($role == 'Responsable') {
-					if(isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa) {
-						$isAuthorized = ($materielTable->exists(['id' => $id, 'groupes_metier_id' => $userConnected->groupes_metier_id]));
+					if(isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa) {
+						$isAuthorized = ($materielTable->exists(['id' => $id, 'groupes_metier_id' => $priviledgedUser->groupes_metier_id]));
 					}
-					else if (isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa) {
-						$isAuthorized = ($materielTable->exists(['id' => $id, 'groupes_thematique_id' => $userConnected->groupe_thematique_id]));
+					else if (isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa) {
+						$isAuthorized = ($materielTable->exists(['id' => $id, 'groupes_thematique_id' => $priviledgedUser->groupe_thematique_id]));
 					}
 					else {
 						$isAuthorized = false;
diff --git a/src/Template/Emprunts/find.ctp b/src/Template/Emprunts/find.ctp
index f1ba1a9..cea7d18 100644
--- a/src/Template/Emprunts/find.ctp
+++ b/src/Template/Emprunts/find.ctp
@@ -142,8 +142,8 @@ $r = isset ($_results);
                     <?php 
                     if(($role == 'Utilisateur' && in_array($username, [$emprunt->nom_createur])) || (in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur']))
                     		|| ($role == 'Responsable' && 
-									((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $emprunt->groupes_metier_id == $userConnected->groupes_metier_id)
-									 ||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $emprunt->groupes_thematique_id == $userConnected->groupe_thematique_id)))) {
+									((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $emprunt->groupes_metier_id == $priviledgedUser->groupes_metier_id)
+									 ||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $emprunt->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) {
 						echo $this->Html->link(__('<i class="icon-pencil"></i>'), ['action' => 'edit', $emprunt->id], ['title' => 'Editer', 'style' => 'margin: 0 2px', 'escape' => false ]);
 						echo $this->Form->postLink(__('<i class="icon-trash"></i>'), ['action' => 'delete', $emprunt->id], ['title' => 'Supprimer', 'style' => 'margin: 0 2px', 'escape' => false, 'confirm' => __('Êtes-vous sur de vouloir supprimer # {0}?', $emprunt->id)]);
 						echo '</td>';
diff --git a/src/Template/Materiels/edit.ctp b/src/Template/Materiels/edit.ctp
index ed7d0d5..b46ad57 100755
--- a/src/Template/Materiels/edit.ctp
+++ b/src/Template/Materiels/edit.ctp
@@ -4,8 +4,8 @@ use Cake\ORM\TableRegistry;
 $USER_IS_UTILISATEUR_AND_CREATOR_OR_OWNER = $USER_IS_UTILISATEUR && in_array($username, [$materiel->nom_createur, $materiel->nom_responsable]);
 
 $USER_IS_RESPONSABLE_AND_SAME_GROUP = $USER_IS_RESPONSABLE && ( 
-		( isset($userConnected->groupes_metier_id) && $materiel->groupes_metier_id == $userConnected->groupes_metier_id ) 
-		|| ( isset($userConnected->groupe_thematique_id) && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id )
+		( isset($priviledgedUser->groupes_metier_id) && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id ) 
+		|| ( isset($priviledgedUser->groupe_thematique_id) && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id )
 		);
 
 $administrationData = array(
@@ -88,8 +88,8 @@ if ($IS_VALIDATED && $materiel->numero_serie)
         ]) || ($role == 'Utilisateur' && (in_array($username, [
             $materiel->nom_createur,
             $materiel->nom_responsable
-        ]))) || ($role == 'Responsable' && ($materiel->groupes_metier_id == $userConnected->groupes_metier_id || 
-        		$materiel->groupes_thematique_id == $userConnected->groupe_thematique_id))) {
+        ]))) || ($role == 'Responsable' && ($materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id || 
+        		$materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id))) {
         */
             echo $this->Form->input('hors_service', [
                 'label' => 'Appareil hors_service'
diff --git a/src/Template/Materiels/find.ctp b/src/Template/Materiels/find.ctp
index 62cb031..d2aba04 100644
--- a/src/Template/Materiels/find.ctp
+++ b/src/Template/Materiels/find.ctp
@@ -281,8 +281,8 @@ $r = isset ($_results);
 				if (in_array($materiel->status, ['CREATED', 'VALIDATED'])) {
 					if(($role == 'Utilisateur' && (in_array($username, [$materiel->nom_createur, $materiel->nom_responsable]))) || (in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur'])) 
 							|| ($role == 'Responsable' && 
-									((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $userConnected->groupes_metier_id)
-									 ||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id)))) {
+									((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id)
+									 ||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) {
 						echo $this->Html->link('<i class="icon-pencil"></i>',
 								['action' => 'edit', $materiel->id],
 								['title' => 'Éditer', 'style' => 'margin: 0 2px', 'escape' => false]
@@ -301,8 +301,8 @@ $r = isset ($_results);
 				
 				if(in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur']) 
 						|| ($role == 'Responsable' && 
-									((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $userConnected->groupes_metier_id)
-									 ||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id)))) {
+									((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id)
+									 ||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) {
 					if(h($materiel->status) == 'CREATED') {
 						echo $this->Html->link('<i class="icon-ok-sign"></i>',
 								['action' => 'statusValidated', $materiel->id],
@@ -330,8 +330,8 @@ $r = isset ($_results);
 				if ($materiel->status == 'CREATED') {
 					if(($role != 'Utilisateur' && $role != 'Responsable') || in_array($username, [$materiel->nom_createur, $materiel->nom_responsable]) 
 							|| ($role == 'Responsable' && 
-									((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $userConnected->groupes_metier_id)
-									 ||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id)))) {
+									((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id)
+									 ||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) {
 						echo $this->Html->link(__('<i class="icon-trash"></i>'),
 								['action' => 'delete', $materiel->id],
 								['title' => 'Supprimer', 'style' => 'margin: 0 2px', 'escape' => false, 'confirm' => __('Êtes-vous sur de vouloir supprimer # {0}?', $materiel->designation)]
diff --git a/src/Template/Materiels/index.ctp b/src/Template/Materiels/index.ctp
index f7d83fa..c1f608b 100755
--- a/src/Template/Materiels/index.ctp
+++ b/src/Template/Materiels/index.ctp
@@ -159,16 +159,16 @@ if ($nbMateriels > 0) {
                 <?php if (in_array($materiel->status, ['CREATED', 'VALIDATED'])): 
                 	  if(($role == 'Utilisateur' && (in_array($username, [$materiel->nom_createur, $materiel->nom_responsable]))) || (in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur'])) 
                 	  		|| ($role == 'Responsable' && 
-									((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $userConnected->groupes_metier_id)
-									 ||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id)))) {
+									((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id)
+									 ||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) {
 				?>
                     <?= $this->Html->link(__('<i class="icon-pencil"></i>'), ['action' => 'edit', $materiel->id], ['title' => 'Editer', 'style' => 'margin: 0 2px', 'escape' => false ]) ?>
                 <?php } endif; ?>
 
                     <?php 
                     if(in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur']) || ($role == 'Responsable'
-                    		&& ((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $userConnected->groupes_metier_id)
-                    				||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id)))) {
+                    		&& ((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id)
+                    				||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) {
                     	
                     					if(h($materiel->status) == 'CREATED'  && $role != 'Responsable') {
 				                    	echo $this->Html->link('<i class="icon-ok-sign"></i>',
@@ -195,8 +195,8 @@ if ($nbMateriels > 0) {
                    if (h($materiel->status) == 'CREATED') {
                    	if(($role != 'Utilisateur' && $role != 'Responsable') || in_array($username, [$materiel->nom_createur, $materiel->nom_responsable]) 
                    			|| ($role == 'Responsable' && 
-									((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $userConnected->groupes_metier_id)
-									 ||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id)))) {
+									((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id)
+									 ||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) {
                    		echo $this->Html->link(__('<i class="icon-trash"></i>'), 
                    				['action' => 'delete', $materiel->id],
                    				['title' => 'Supprimer', 'style' => 'margin: 0 2px', 'escape' => false, 'confirm' => __('Êtes-vous sur de vouloir supprimer # {0}?', $materiel->designation)]);
diff --git a/src/Template/Materiels/view.ctp b/src/Template/Materiels/view.ctp
index 1dd4463..f3a0fca 100755
--- a/src/Template/Materiels/view.ctp
+++ b/src/Template/Materiels/view.ctp
@@ -63,6 +63,9 @@ $USER_IS_ADMIN_OR_MORE = $profile >= $PROFILE_ADMIN;
 // 	return (int) $strInterval;
 // };
 
+//debug("user=".$priviledgedUser);
+//debug("user name = ".$username);
+
 $USER_IS_UTILISATEUR_AND_CREATOR_OR_OWNER = $USER_IS_UTILISATEUR && in_array($username, [$materiel->nom_createur, $materiel->nom_responsable]);
 
 $USER_IS_RESPONSABLE_AND_SAME_GROUP =
@@ -70,19 +73,21 @@ $USER_IS_RESPONSABLE_AND_SAME_GROUP =
 	&&
 	(
 		(
-			isset($userConnected->groupes_metier_id)
-			&& $userConnected->groupes_metier_id != $idGmNa
-			&& $materiel->groupes_metier_id == $userConnected->groupes_metier_id
+			isset($priviledgedUser->groupes_metier_id)
+			&& $priviledgedUser->groupes_metier_id != $idGmNa
+			&& $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id
 		)
 		||
 		(
-			isset($userConnected->groupe_thematique_id)
-			&& $userConnected->groupe_thematique_id != $idGtNa
-			&& $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id
+			isset($priviledgedUser->groupe_thematique_id)
+			&& $priviledgedUser->groupe_thematique_id != $idGtNa
+			&& $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id
 		)
 	);
 ?>
 
+
+
 <div class="materiels view">
 
 	<h2>
@@ -144,15 +149,15 @@ $USER_IS_RESPONSABLE_AND_SAME_GROUP =
 					$role == 'Responsable'
 					&& (
 							(
-								isset($userConnected->groupes_metier_id) 
-								&& $userConnected->groupes_metier_id != $idGmNa 
-								&& $materiel->groupes_metier_id == $userConnected->groupes_metier_id
+								isset($priviledgedUser->groupes_metier_id) 
+								&& $priviledgedUser->groupes_metier_id != $idGmNa 
+								&& $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id
 							) 
 							|| 
 							(
-								isset($userConnected->groupe_thematique_id) 
-								&& $userConnected->groupe_thematique_id != $idGtNa 
-								&& $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id
+								isset($priviledgedUser->groupe_thematique_id) 
+								&& $priviledgedUser->groupe_thematique_id != $idGtNa 
+								&& $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id
 							)
 					)
 				)
@@ -215,15 +220,15 @@ $USER_IS_RESPONSABLE_AND_SAME_GROUP =
 				$role == 'Responsable' 
 				&& (
 					(
-						isset($userConnected->groupes_metier_id) 
-						&& $userConnected->groupes_metier_id != $idGmNa 
-						&& $materiel->groupes_metier_id == $userConnected->groupes_metier_id
+						isset($priviledgedUser->groupes_metier_id) 
+						&& $priviledgedUser->groupes_metier_id != $idGmNa 
+						&& $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id
 					) 
 					|| 
 					(
-						isset($userConnected->groupe_thematique_id) 
-						&& $userConnected->groupe_thematique_id != $idGtNa 
-						&& $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id
+						isset($priviledgedUser->groupe_thematique_id) 
+						&& $priviledgedUser->groupe_thematique_id != $idGtNa 
+						&& $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id
 					)
 				)
 			)
@@ -315,15 +320,15 @@ $USER_IS_RESPONSABLE_AND_SAME_GROUP =
 			$role == 'Responsable' 
 			&& (
 				(
-					isset($userConnected->groupes_metier_id) 
-					&& $userConnected->groupes_metier_id != $idGmNa 
-					&& $materiel->groupes_metier_id == $userConnected->groupes_metier_id
+					isset($priviledgedUser->groupes_metier_id) 
+					&& $priviledgedUser->groupes_metier_id != $idGmNa 
+					&& $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id
 				) 
 				|| 
 				(
-					isset($userConnected->groupe_thematique_id) 
-					&& $userConnected->groupe_thematique_id != $idGtNa 
-					&& $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id
+					isset($priviledgedUser->groupe_thematique_id) 
+					&& $priviledgedUser->groupe_thematique_id != $idGtNa 
+					&& $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id
 				)
 			)
 		)
@@ -613,15 +618,15 @@ $USER_IS_RESPONSABLE_AND_SAME_GROUP =
 							$role == 'Responsable' 
 							&& (
 								(
-									isset($userConnected->groupes_metier_id) 
-									&& $userConnected->groupes_metier_id != $idGmNa 
-									&& $materiel->groupes_metier_id == $userConnected->groupes_metier_id
+									isset($priviledgedUser->groupes_metier_id) 
+									&& $priviledgedUser->groupes_metier_id != $idGmNa 
+									&& $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id
 								) 
 								|| 
 								(
-									isset($userConnected->groupe_thematique_id) 
-									&& $userConnected->groupe_thematique_id != $idGtNa 
-									&& $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id
+									isset($priviledgedUser->groupe_thematique_id) 
+									&& $priviledgedUser->groupe_thematique_id != $idGtNa 
+									&& $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id
 								)
 							)
 						)
@@ -701,15 +706,15 @@ $USER_IS_RESPONSABLE_AND_SAME_GROUP =
 										$role == 'Responsable' 
 										&& (
 											(
-												isset($userConnected->groupes_metier_id) 
-												&& $userConnected->groupes_metier_id != $idGmNa 
-												&& $materiel->groupes_metier_id == $userConnected->groupes_metier_id
+												isset($priviledgedUser->groupes_metier_id) 
+												&& $priviledgedUser->groupes_metier_id != $idGmNa 
+												&& $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id
 											) 
 											|| 
 											(
-												isset($userConnected->groupe_thematique_id) 
-												&& $userConnected->groupe_thematique_id != $idGtNa 
-												&& $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id
+												isset($priviledgedUser->groupe_thematique_id) 
+												&& $priviledgedUser->groupe_thematique_id != $idGtNa 
+												&& $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id
 											)
 										)
 									)
@@ -790,15 +795,15 @@ $USER_IS_RESPONSABLE_AND_SAME_GROUP =
 									$role == 'Responsable' 
 									&& (
 										(
-											isset($userConnected->groupes_metier_id) 
-											&& $userConnected->groupes_metier_id != $idGmNa 
-											&& $materiel->groupes_metier_id == $userConnected->groupes_metier_id
+											isset($priviledgedUser->groupes_metier_id) 
+											&& $priviledgedUser->groupes_metier_id != $idGmNa 
+											&& $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id
 										) 
 										|| 
 										(
-											isset($userConnected->groupe_thematique_id) 
-											&& $userConnected->groupe_thematique_id != $idGtNa 
-											&& $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id
+											isset($priviledgedUser->groupe_thematique_id) 
+											&& $priviledgedUser->groupe_thematique_id != $idGtNa 
+											&& $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id
 										)
 									)
 								)
diff --git a/src/Template/Pages/home_app.ctp b/src/Template/Pages/home_app.ctp
index e8be580..20c2155 100755
--- a/src/Template/Pages/home_app.ctp
+++ b/src/Template/Pages/home_app.ctp
@@ -51,14 +51,14 @@ if ($role == 'Responsable') {
     echo '<tr><td> ' . $this->Html->link('Voir les matériels  dont je suis responsable', [
         'controller' => 'materiels',
         'action' => 'index',
-        'GM' => $userConnected->groupes_metier_id,
-        'GT' => $userConnected->groupe_thematique_id
+        'GM' => $priviledgedUser->groupes_metier_id,
+        'GT' => $priviledgedUser->groupe_thematique_id
     ]) . ' </td></tr>';
     echo '<tr><td> ' . $this->Html->link('Voir les suivis des materiels dont je suis responsable', [
         'controller' => 'suivis',
         'action' => 'index',
-        'GM' => $userConnected->groupes_metier_id,
-        'GT' => $userConnected->groupe_thematique_id
+        'GM' => $priviledgedUser->groupes_metier_id,
+        'GT' => $priviledgedUser->groupe_thematique_id
     ]) . ' </td></tr>';
     echo '</table>';
 }
diff --git a/src/Template/Suivis/find.ctp b/src/Template/Suivis/find.ctp
index 7257175..f38c58f 100755
--- a/src/Template/Suivis/find.ctp
+++ b/src/Template/Suivis/find.ctp
@@ -129,8 +129,8 @@ $r = isset ($_results);
                     <?php 
                     if(($role == 'Utilisateur' && in_array($username, [$suivi->nom_createur])) || (in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur']))
                     		|| ($role == 'Responsable' && 
-									((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $userConnected->groupes_metier_id)
-									 ||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $userConnected->groupe_thematique_id)))) {
+									((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $priviledgedUser->groupes_metier_id)
+									 ||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) {
 						echo $this->Html->link(__('<i class="icon-pencil"></i>'), ['action' => 'edit', $suivi->id], ['title' => 'Editer', 'style' => 'margin: 0 2px', 'escape' => false ]);
 						echo $this->Form->postLink(__('<i class="icon-trash"></i>'), ['action' => 'delete', $suivi->id], ['title' => 'Supprimer', 'style' => 'margin: 0 2px', 'escape' => false, 'confirm' => __('Êtes-vous sur de vouloir supprimer # {0}?', $suivi->id)]);
 						echo '</td>';
diff --git a/src/Template/Suivis/index.ctp b/src/Template/Suivis/index.ctp
index 1f1a60a..8a54764 100755
--- a/src/Template/Suivis/index.ctp
+++ b/src/Template/Suivis/index.ctp
@@ -22,8 +22,8 @@
                 <td class="actions" style="padding: 6px 0; text-align: left;">
                     <?php if(($role == 'Utilisateur' && in_array($username, [$suivi->nom_createur])) || (in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur']))
                     		|| ($role == 'Responsable' && 
-									((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $userConnected->groupes_metier_id)
-									 ||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $userConnected->groupe_thematique_id)))) { ?>
+									((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $priviledgedUser->groupes_metier_id)
+									 ||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) { ?>
                     <?= $this->Html->link(__('<i class="icon-pencil"></i>'), ['action' => 'edit', $suivi->id], ['title' => 'Editer', 'style' => 'margin: 0 2px', 'escape' => false ]) ?>
                     <?= $this->Form->postLink(__('<i class="icon-trash"></i>'), ['action' => 'delete', $suivi->id], ['title' => 'Supprimer', 'style' => 'margin: 0 2px', 'escape' => false, 'confirm' => __('Êtes-vous sur de vouloir supprimer # {0}?', $suivi->id)]) ?>
               		<?php } ?>
diff --git a/src/Template/Suivis/view.ctp b/src/Template/Suivis/view.ctp
index c65051e..59be87e 100755
--- a/src/Template/Suivis/view.ctp
+++ b/src/Template/Suivis/view.ctp
@@ -12,9 +12,9 @@
     	|| ( in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur']) )
     	|| (
     			$role == 'Responsable' && (
-    				( isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $userConnected->groupes_metier_id )
+    				( isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $priviledgedUser->groupes_metier_id )
 					||
-    				( isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $userConnected->groupe_thematique_id )
+    				( isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $priviledgedUser->groupe_thematique_id )
     			)
     		)
     ) {
@@ -132,8 +132,8 @@
 				<?php 
 				if(($role == 'Utilisateur' && in_array($username, [$suivi->nom_createur])) || (in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur']))
 						|| ($role == 'Responsable' && 
-										((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $userConnected->groupes_metier_id)
-										 ||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $userConnected->groupe_thematique_id)))) {
+										((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $priviledgedUser->groupes_metier_id)
+										 ||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) {
 				?>
 	                <?= $this->Html->link(__('<i class="icon-pencil"></i>'), ['controller' => 'Documents', 'action' => 'edit', $documents->id], ['escape' => false, 'style' => 'margin:0'])?>
 	                <?= $this->Form->postLink(__('<i class="icon-trash"></i>'), ['controller' => 'Documents', 'action' => 'delete', $documents->id], ['escape' => false, 'style' => 'margin:0', 'confirm' => __('Êtes-vous sur de vouloir supprimer # {0}?', $documents->id)]) ?>
--
libgit2 0.21.2