Etienne Pallier / labinvent

Commit 8392646428c7a0c98cc872def83243a16f201dcb

Authored by Etienne Pallier
1 parent b6f3fdbf
Exists in master and in 3 other branches dev, dev-IRAP, dev-LATMOS

renommages de variables et fonctions pour clarifier le code

Showing 11 changed files with 123 additions and 125 deletions   Show diff stats
Inline Side-by-side
src/Controller/AppController.php
  Diff comments   View file @8392646
... ... @@ -163,7 +163,7 @@ class AppController extends Controller {
163 163 return false;
164 164 }
165 165  
166   - public function getUser($user=null) { return $this->getTablePriviledgedUserFromCurrentSessionUser($user); }
  166 + //public function getUser($user=null) { return $this->getTablePriviledgedUserFromCurrentSessionUser($user); }
167 167 public function getTablePriviledgedUserFromCurrentSessionUser($user=null) {
168 168 if (! $this->CURRENT_USER) {
169 169 $configuration = $this->confLabinvent;
... ... @@ -178,23 +178,14 @@ class AppController extends Controller {
178 178 }
179 179 return $this->CURRENT_USER;
180 180 }
181   - public function getUserRole($user=null) { return $this->getPriviledgedUserRole($user); }
182   - public function getPriviledgedUserRole($user=null) {
183   - $priviledgedUser = $this->getTablePriviledgedUserFromCurrentSessionUser($user);
  181 + public function getUserRole($user=null) {
  182 + $priviledgedUser = $this->getPriviledgedUserRole($user);
184 183 // default role is "Utilisateur" (for people who are not in the table utilisateurs)
185 184 if (! $priviledgedUser) return 'Utilisateur';
186 185 return $priviledgedUser['role'];
187   - /*
188   - if (! $this->CURRENT_ROLE) {
189   - $configuration = $this->confLabinvent;
190   - $user = TableRegistry::get('Users')->find()
191   - ->where(['username' => $this->LdapAuth->user($configuration->authentificationType_ldap)[0]])
192   - //->where(['username' => $this->LdapAuth->user('cn')[0]])
193   - ->first();
194   - $this->CURRENT_ROLE = $user['role'];
195   - }
196   - return $this->CURRENT_ROLE;
197   - */
  186 + }
  187 + public function getPriviledgedUserRole($user=null) {
  188 + return $this->getTablePriviledgedUserFromCurrentSessionUser($user);
198 189 }
199 190  
200 191 private function userHasRole($expectedRole, $ORMORE=false) {
... ... @@ -301,9 +292,8 @@ class AppController extends Controller {
301 292 $this->set('configuration', $configuration);
302 293 $this->request->session()->write("authType", $configuration->authentificationType_ldap);
303 294  
304   - //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
305   - $user = $this->getUser();
306   - $role = $this->getUserRole();
  295 + //$user = $this->getUser();
  296 + $priviledgedUser = $this->getTablePriviledgedUserFromCurrentSessionUser();
307 297 /*
308 298 $user = TableRegistry::get('Users')->find()
309 299 ->where(['username' => $this->LdapAuth->user($configuration->authentificationType_ldap)[0]])
... ... @@ -312,8 +302,11 @@ class AppController extends Controller {
312 302 if ($role == null)
313 303 $role = 'Utilisateur';
314 304 */
  305 + // Role = 'Utilisateur', 'Responsable", ...
  306 + $role = $this->getUserRole();
315 307 $this->set('role', $role);
316   -
  308 +
  309 + // Profile = PROFILE_USER (=1), PROFILE_RESPONSABLE (=2), ...
317 310 //$profile = $this->allProfiles["$role"];
318 311 $profile = self::PROFILES["$role"];
319 312 $this->set('profile', $profile);
... ... @@ -336,7 +329,7 @@ class AppController extends Controller {
336 329 $this->set('USER_IS_ADMIN_OR_MORE', $USER_IS_ADMIN_OR_MORE);
337 330 $this->set('USER_IS_ADMINPLUS_OR_MORE', $USER_IS_ADMINPLUS_OR_MORE);
338 331  
339   - $this->set('userConnected', $user);
  332 + $this->set('priviledgedUser', $priviledgedUser);
340 333  
341 334 $this->set('idGmNa', TableRegistry::get('GroupesMetiers')->find()->where(['nom =' => 'N/A'])->first()['id']);
342 335 $this->set('idGtNa', TableRegistry::get('GroupesThematiques')->find()->where(['nom =' => 'N/A'])->first()['id']);
... ... @@ -349,16 +342,16 @@ class AppController extends Controller {
349 342 };
350 343 $this->set('displayElement', $displayElement);
351 344  
  345 + // Pass this function to all views
  346 + // @todo : Si cette fonction ne concerne que SuivisController, il faut la déplacer dans ce controleur
352 347 $dateProchainControleVerif = function($t) {
353   - $time = Time::now(); // On récupère la date et l'heure actuelles
354   - $today = new \DateTime((new date("$time->year-$time->month-$time->day"))->format('Y-m-d'));
355   - $time1 = new time($t);
356   - $dateTime1 = new \DateTime((new date("$time1->year-$time1->month-$time1->day"))->format('y-m-d'));
357   -
358   - $interval = ($today->diff($dateTime1));
359   - $strInterval = $interval->format('%a');
360   -
361   - return (int) $strInterval;
  348 + $time = Time::now(); // On récupère la date et l'heure actuelles
  349 + $today = new \DateTime((new date("$time->year-$time->month-$time->day"))->format('Y-m-d'));
  350 + $time1 = new time($t);
  351 + $dateTime1 = new \DateTime((new date("$time1->year-$time1->month-$time1->day"))->format('y-m-d'));
  352 + $interval = ($today->diff($dateTime1));
  353 + $strInterval = $interval->format('%a');
  354 + return (int) $strInterval;
362 355 };
363 356 $this->set('dateProchainControleVerif', $dateProchainControleVerif);
364 357 }
... ...
src/Template/Element/menu_view.ctp
  Diff comments   View file @8392646
... ... @@ -41,10 +41,10 @@ use Cake\ORM\TableRegistry;
41 41 if((($role == 'Utilisateur'|| $role == 'Responsable') && (in_array($username, [$materiel->nom_createur, $materiel->nom_responsable]))) || (in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur']))) {
42 42 $isAuthorized = true;
43 43 } else if($role == 'Responsable') {
44   - if(isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $userConnected->groupes_metier_id) {
  44 + if(isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id) {
45 45 $isAuthorized = true;
46 46 }
47   - else if (isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id) {
  47 + else if (isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id) {
48 48 $isAuthorized = true;
49 49 }
50 50 else {
... ... @@ -66,10 +66,10 @@ use Cake\ORM\TableRegistry;
66 66 $isAuthorized = true;
67 67 }
68 68 else if($role == 'Responsable') {
69   - if(isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $userConnected->groupes_metier_id) {
  69 + if(isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $priviledgedUser->groupes_metier_id) {
70 70 $isAuthorized = true;
71 71 }
72   - else if (isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $userConnected->groupe_thematique_id) {
  72 + else if (isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $priviledgedUser->groupe_thematique_id) {
73 73 $isAuthorized = true;
74 74 }
75 75 else {
... ... @@ -110,11 +110,11 @@ use Cake\ORM\TableRegistry;
110 110 $id = $doc->get('suivi_id');
111 111 if ($role == 'Responsable') {
112 112 $suiviTable = TableRegistry::get('Suivis');
113   - if(isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa) {
114   - $isAuthorized = ($suiviTable->exists(['id' => $id, 'groupes_metier_id' => $userConnected->groupes_metier_id]));
  113 + if(isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa) {
  114 + $isAuthorized = ($suiviTable->exists(['id' => $id, 'groupes_metier_id' => $priviledgedUser->groupes_metier_id]));
115 115 }
116   - else if (isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa) {
117   - $isAuthorized = ($suiviTable->exists(['id' => $id, 'groupes_thematique_id' => $userConnected->groupe_thematique_id]));
  116 + else if (isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa) {
  117 + $isAuthorized = ($suiviTable->exists(['id' => $id, 'groupes_thematique_id' => $priviledgedUser->groupe_thematique_id]));
118 118 }
119 119 else {
120 120 $isAuthorized = false;
... ... @@ -127,11 +127,11 @@ use Cake\ORM\TableRegistry;
127 127 $materielTable = TableRegistry::get('Materiels');
128 128 if ($role == 'Responsable') {
129 129  
130   - if(isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa) {
131   - $isAuthorized = ($materielTable->exists(['id' => $id, 'groupes_metier_id' => $userConnected->groupes_metier_id]));
  130 + if(isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa) {
  131 + $isAuthorized = ($materielTable->exists(['id' => $id, 'groupes_metier_id' => $priviledgedUser->groupes_metier_id]));
132 132 }
133   - else if (isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa) {
134   - $isAuthorized = ($materielTable->exists(['id' => $id, 'groupes_thematique_id' => $userConnected->groupe_thematique_id]));
  133 + else if (isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa) {
  134 + $isAuthorized = ($materielTable->exists(['id' => $id, 'groupes_thematique_id' => $priviledgedUser->groupe_thematique_id]));
135 135 }
136 136 else {
137 137 $isAuthorized = false;
... ... @@ -185,10 +185,10 @@ use Cake\ORM\TableRegistry;
185 185 $isAuthorized = true;
186 186 }
187 187 else if($role == 'Responsable') {
188   - if(isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $userConnected->groupes_metier_id) {
  188 + if(isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id) {
189 189 $isAuthorized = true;
190 190 }
191   - else if (isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id) {
  191 + else if (isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id) {
192 192 $isAuthorized = true;
193 193 }
194 194 else {
... ... @@ -206,10 +206,10 @@ use Cake\ORM\TableRegistry;
206 206 $isAuthorized = true;
207 207 }
208 208 else if($role == 'Responsable') {
209   - if(isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $userConnected->groupes_metier_id) {
  209 + if(isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $priviledgedUser->groupes_metier_id) {
210 210 $isAuthorized = true;
211 211 }
212   - else if (isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $userConnected->groupe_thematique_id) {
  212 + else if (isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $priviledgedUser->groupe_thematique_id) {
213 213 $isAuthorized = true;
214 214 }
215 215 else {
... ... @@ -238,11 +238,11 @@ use Cake\ORM\TableRegistry;
238 238 $id = $doc->get('suivi_id');
239 239 if ($role == 'Responsable') {
240 240 $suiviTable = TableRegistry::get('Suivis');
241   - if(isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa) {
242   - $isAuthorized = ($suiviTable->exists(['id' => $id, 'groupes_metier_id' => $userConnected->groupes_metier_id]));
  241 + if(isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa) {
  242 + $isAuthorized = ($suiviTable->exists(['id' => $id, 'groupes_metier_id' => $priviledgedUser->groupes_metier_id]));
243 243 }
244   - else if (isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa) {
245   - $isAuthorized = ($suiviTable->exists(['id' => $id, 'groupes_thematique_id' => $userConnected->groupe_thematique_id]));
  244 + else if (isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa) {
  245 + $isAuthorized = ($suiviTable->exists(['id' => $id, 'groupes_thematique_id' => $priviledgedUser->groupe_thematique_id]));
246 246 }
247 247 else {
248 248 $isAuthorized = false;
... ... @@ -254,11 +254,11 @@ use Cake\ORM\TableRegistry;
254 254 } else {
255 255 $materielTable = TableRegistry::get('Materiels');
256 256 if ($role == 'Responsable') {
257   - if(isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa) {
258   - $isAuthorized = ($materielTable->exists(['id' => $id, 'groupes_metier_id' => $userConnected->groupes_metier_id]));
  257 + if(isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa) {
  258 + $isAuthorized = ($materielTable->exists(['id' => $id, 'groupes_metier_id' => $priviledgedUser->groupes_metier_id]));
259 259 }
260   - else if (isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa) {
261   - $isAuthorized = ($materielTable->exists(['id' => $id, 'groupes_thematique_id' => $userConnected->groupe_thematique_id]));
  260 + else if (isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa) {
  261 + $isAuthorized = ($materielTable->exists(['id' => $id, 'groupes_thematique_id' => $priviledgedUser->groupe_thematique_id]));
262 262 }
263 263 else {
264 264 $isAuthorized = false;
... ...
src/Template/Emprunts/find.ctp
  Diff comments   View file @8392646
... ... @@ -142,8 +142,8 @@ $r = isset ($_results);
142 142 <?php
143 143 if(($role == 'Utilisateur' && in_array($username, [$emprunt->nom_createur])) || (in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur']))
144 144 || ($role == 'Responsable' &&
145   - ((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $emprunt->groupes_metier_id == $userConnected->groupes_metier_id)
146   - ||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $emprunt->groupes_thematique_id == $userConnected->groupe_thematique_id)))) {
  145 + ((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $emprunt->groupes_metier_id == $priviledgedUser->groupes_metier_id)
  146 + ||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $emprunt->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) {
147 147 echo $this->Html->link(__('<i class="icon-pencil"></i>'), ['action' => 'edit', $emprunt->id], ['title' => 'Editer', 'style' => 'margin: 0 2px', 'escape' => false ]);
148 148 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)]);
149 149 echo '</td>';
... ...
src/Template/Materiels/edit.ctp
  Diff comments   View file @8392646
... ... @@ -4,8 +4,8 @@ use Cake\ORM\TableRegistry;
4 4 $USER_IS_UTILISATEUR_AND_CREATOR_OR_OWNER = $USER_IS_UTILISATEUR && in_array($username, [$materiel->nom_createur, $materiel->nom_responsable]);
5 5  
6 6 $USER_IS_RESPONSABLE_AND_SAME_GROUP = $USER_IS_RESPONSABLE && (
7   - ( isset($userConnected->groupes_metier_id) && $materiel->groupes_metier_id == $userConnected->groupes_metier_id )
8   - || ( isset($userConnected->groupe_thematique_id) && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id )
  7 + ( isset($priviledgedUser->groupes_metier_id) && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id )
  8 + || ( isset($priviledgedUser->groupe_thematique_id) && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id )
9 9 );
10 10  
11 11 $administrationData = array(
... ... @@ -88,8 +88,8 @@ if ($IS_VALIDATED &amp;&amp; $materiel-&gt;numero_serie)
88 88 ]) || ($role == 'Utilisateur' && (in_array($username, [
89 89 $materiel->nom_createur,
90 90 $materiel->nom_responsable
91   - ]))) || ($role == 'Responsable' && ($materiel->groupes_metier_id == $userConnected->groupes_metier_id ||
92   - $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id))) {
  91 + ]))) || ($role == 'Responsable' && ($materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id ||
  92 + $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id))) {
93 93 */
94 94 echo $this->Form->input('hors_service', [
95 95 'label' => 'Appareil hors_service'
... ...
src/Template/Materiels/find.ctp
  Diff comments   View file @8392646
... ... @@ -281,8 +281,8 @@ $r = isset ($_results);
281 281 if (in_array($materiel->status, ['CREATED', 'VALIDATED'])) {
282 282 if(($role == 'Utilisateur' && (in_array($username, [$materiel->nom_createur, $materiel->nom_responsable]))) || (in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur']))
283 283 || ($role == 'Responsable' &&
284   - ((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $userConnected->groupes_metier_id)
285   - ||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id)))) {
  284 + ((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id)
  285 + ||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) {
286 286 echo $this->Html->link('<i class="icon-pencil"></i>',
287 287 ['action' => 'edit', $materiel->id],
288 288 ['title' => 'Éditer', 'style' => 'margin: 0 2px', 'escape' => false]
... ... @@ -301,8 +301,8 @@ $r = isset ($_results);
301 301  
302 302 if(in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur'])
303 303 || ($role == 'Responsable' &&
304   - ((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $userConnected->groupes_metier_id)
305   - ||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id)))) {
  304 + ((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id)
  305 + ||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) {
306 306 if(h($materiel->status) == 'CREATED') {
307 307 echo $this->Html->link('<i class="icon-ok-sign"></i>',
308 308 ['action' => 'statusValidated', $materiel->id],
... ... @@ -330,8 +330,8 @@ $r = isset ($_results);
330 330 if ($materiel->status == 'CREATED') {
331 331 if(($role != 'Utilisateur' && $role != 'Responsable') || in_array($username, [$materiel->nom_createur, $materiel->nom_responsable])
332 332 || ($role == 'Responsable' &&
333   - ((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $userConnected->groupes_metier_id)
334   - ||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id)))) {
  333 + ((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id)
  334 + ||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) {
335 335 echo $this->Html->link(__('<i class="icon-trash"></i>'),
336 336 ['action' => 'delete', $materiel->id],
337 337 ['title' => 'Supprimer', 'style' => 'margin: 0 2px', 'escape' => false, 'confirm' => __('Êtes-vous sur de vouloir supprimer # {0}?', $materiel->designation)]
... ...
src/Template/Materiels/index.ctp
  Diff comments   View file @8392646
... ... @@ -159,16 +159,16 @@ if ($nbMateriels &gt; 0) {
159 159 <?php if (in_array($materiel->status, ['CREATED', 'VALIDATED'])):
160 160 if(($role == 'Utilisateur' && (in_array($username, [$materiel->nom_createur, $materiel->nom_responsable]))) || (in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur']))
161 161 || ($role == 'Responsable' &&
162   - ((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $userConnected->groupes_metier_id)
163   - ||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id)))) {
  162 + ((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id)
  163 + ||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) {
164 164 ?>
165 165 <?= $this->Html->link(__('<i class="icon-pencil"></i>'), ['action' => 'edit', $materiel->id], ['title' => 'Editer', 'style' => 'margin: 0 2px', 'escape' => false ]) ?>
166 166 <?php } endif; ?>
167 167  
168 168 <?php
169 169 if(in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur']) || ($role == 'Responsable'
170   - && ((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $userConnected->groupes_metier_id)
171   - ||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id)))) {
  170 + && ((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id)
  171 + ||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) {
172 172  
173 173 if(h($materiel->status) == 'CREATED' && $role != 'Responsable') {
174 174 echo $this->Html->link('<i class="icon-ok-sign"></i>',
... ... @@ -195,8 +195,8 @@ if ($nbMateriels &gt; 0) {
195 195 if (h($materiel->status) == 'CREATED') {
196 196 if(($role != 'Utilisateur' && $role != 'Responsable') || in_array($username, [$materiel->nom_createur, $materiel->nom_responsable])
197 197 || ($role == 'Responsable' &&
198   - ((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $userConnected->groupes_metier_id)
199   - ||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id)))) {
  198 + ((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id)
  199 + ||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) {
200 200 echo $this->Html->link(__('<i class="icon-trash"></i>'),
201 201 ['action' => 'delete', $materiel->id],
202 202 ['title' => 'Supprimer', 'style' => 'margin: 0 2px', 'escape' => false, 'confirm' => __('Êtes-vous sur de vouloir supprimer # {0}?', $materiel->designation)]);
... ...
src/Template/Materiels/view.ctp
  Diff comments   View file @8392646
... ... @@ -63,6 +63,9 @@ $USER_IS_ADMIN_OR_MORE = $profile &gt;= $PROFILE_ADMIN;
63 63 // return (int) $strInterval;
64 64 // };
65 65  
  66 +//debug("user=".$priviledgedUser);
  67 +//debug("user name = ".$username);
  68 +
66 69 $USER_IS_UTILISATEUR_AND_CREATOR_OR_OWNER = $USER_IS_UTILISATEUR && in_array($username, [$materiel->nom_createur, $materiel->nom_responsable]);
67 70  
68 71 $USER_IS_RESPONSABLE_AND_SAME_GROUP =
... ... @@ -70,19 +73,21 @@ $USER_IS_RESPONSABLE_AND_SAME_GROUP =
70 73 &&
71 74 (
72 75 (
73   - isset($userConnected->groupes_metier_id)
74   - && $userConnected->groupes_metier_id != $idGmNa
75   - && $materiel->groupes_metier_id == $userConnected->groupes_metier_id
  76 + isset($priviledgedUser->groupes_metier_id)
  77 + && $priviledgedUser->groupes_metier_id != $idGmNa
  78 + && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id
76 79 )
77 80 ||
78 81 (
79   - isset($userConnected->groupe_thematique_id)
80   - && $userConnected->groupe_thematique_id != $idGtNa
81   - && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id
  82 + isset($priviledgedUser->groupe_thematique_id)
  83 + && $priviledgedUser->groupe_thematique_id != $idGtNa
  84 + && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id
82 85 )
83 86 );
84 87 ?>
85 88  
  89 +
  90 +
86 91 <div class="materiels view">
87 92  
88 93 <h2>
... ... @@ -144,15 +149,15 @@ $USER_IS_RESPONSABLE_AND_SAME_GROUP =
144 149 $role == 'Responsable'
145 150 && (
146 151 (
147   - isset($userConnected->groupes_metier_id)
148   - && $userConnected->groupes_metier_id != $idGmNa
149   - && $materiel->groupes_metier_id == $userConnected->groupes_metier_id
  152 + isset($priviledgedUser->groupes_metier_id)
  153 + && $priviledgedUser->groupes_metier_id != $idGmNa
  154 + && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id
150 155 )
151 156 ||
152 157 (
153   - isset($userConnected->groupe_thematique_id)
154   - && $userConnected->groupe_thematique_id != $idGtNa
155   - && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id
  158 + isset($priviledgedUser->groupe_thematique_id)
  159 + && $priviledgedUser->groupe_thematique_id != $idGtNa
  160 + && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id
156 161 )
157 162 )
158 163 )
... ... @@ -215,15 +220,15 @@ $USER_IS_RESPONSABLE_AND_SAME_GROUP =
215 220 $role == 'Responsable'
216 221 && (
217 222 (
218   - isset($userConnected->groupes_metier_id)
219   - && $userConnected->groupes_metier_id != $idGmNa
220   - && $materiel->groupes_metier_id == $userConnected->groupes_metier_id
  223 + isset($priviledgedUser->groupes_metier_id)
  224 + && $priviledgedUser->groupes_metier_id != $idGmNa
  225 + && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id
221 226 )
222 227 ||
223 228 (
224   - isset($userConnected->groupe_thematique_id)
225   - && $userConnected->groupe_thematique_id != $idGtNa
226   - && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id
  229 + isset($priviledgedUser->groupe_thematique_id)
  230 + && $priviledgedUser->groupe_thematique_id != $idGtNa
  231 + && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id
227 232 )
228 233 )
229 234 )
... ... @@ -315,15 +320,15 @@ $USER_IS_RESPONSABLE_AND_SAME_GROUP =
315 320 $role == 'Responsable'
316 321 && (
317 322 (
318   - isset($userConnected->groupes_metier_id)
319   - && $userConnected->groupes_metier_id != $idGmNa
320   - && $materiel->groupes_metier_id == $userConnected->groupes_metier_id
  323 + isset($priviledgedUser->groupes_metier_id)
  324 + && $priviledgedUser->groupes_metier_id != $idGmNa
  325 + && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id
321 326 )
322 327 ||
323 328 (
324   - isset($userConnected->groupe_thematique_id)
325   - && $userConnected->groupe_thematique_id != $idGtNa
326   - && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id
  329 + isset($priviledgedUser->groupe_thematique_id)
  330 + && $priviledgedUser->groupe_thematique_id != $idGtNa
  331 + && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id
327 332 )
328 333 )
329 334 )
... ... @@ -613,15 +618,15 @@ $USER_IS_RESPONSABLE_AND_SAME_GROUP =
613 618 $role == 'Responsable'
614 619 && (
615 620 (
616   - isset($userConnected->groupes_metier_id)
617   - && $userConnected->groupes_metier_id != $idGmNa
618   - && $materiel->groupes_metier_id == $userConnected->groupes_metier_id
  621 + isset($priviledgedUser->groupes_metier_id)
  622 + && $priviledgedUser->groupes_metier_id != $idGmNa
  623 + && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id
619 624 )
620 625 ||
621 626 (
622   - isset($userConnected->groupe_thematique_id)
623   - && $userConnected->groupe_thematique_id != $idGtNa
624   - && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id
  627 + isset($priviledgedUser->groupe_thematique_id)
  628 + && $priviledgedUser->groupe_thematique_id != $idGtNa
  629 + && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id
625 630 )
626 631 )
627 632 )
... ... @@ -701,15 +706,15 @@ $USER_IS_RESPONSABLE_AND_SAME_GROUP =
701 706 $role == 'Responsable'
702 707 && (
703 708 (
704   - isset($userConnected->groupes_metier_id)
705   - && $userConnected->groupes_metier_id != $idGmNa
706   - && $materiel->groupes_metier_id == $userConnected->groupes_metier_id
  709 + isset($priviledgedUser->groupes_metier_id)
  710 + && $priviledgedUser->groupes_metier_id != $idGmNa
  711 + && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id
707 712 )
708 713 ||
709 714 (
710   - isset($userConnected->groupe_thematique_id)
711   - && $userConnected->groupe_thematique_id != $idGtNa
712   - && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id
  715 + isset($priviledgedUser->groupe_thematique_id)
  716 + && $priviledgedUser->groupe_thematique_id != $idGtNa
  717 + && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id
713 718 )
714 719 )
715 720 )
... ... @@ -790,15 +795,15 @@ $USER_IS_RESPONSABLE_AND_SAME_GROUP =
790 795 $role == 'Responsable'
791 796 && (
792 797 (
793   - isset($userConnected->groupes_metier_id)
794   - && $userConnected->groupes_metier_id != $idGmNa
795   - && $materiel->groupes_metier_id == $userConnected->groupes_metier_id
  798 + isset($priviledgedUser->groupes_metier_id)
  799 + && $priviledgedUser->groupes_metier_id != $idGmNa
  800 + && $materiel->groupes_metier_id == $priviledgedUser->groupes_metier_id
796 801 )
797 802 ||
798 803 (
799   - isset($userConnected->groupe_thematique_id)
800   - && $userConnected->groupe_thematique_id != $idGtNa
801   - && $materiel->groupes_thematique_id == $userConnected->groupe_thematique_id
  804 + isset($priviledgedUser->groupe_thematique_id)
  805 + && $priviledgedUser->groupe_thematique_id != $idGtNa
  806 + && $materiel->groupes_thematique_id == $priviledgedUser->groupe_thematique_id
802 807 )
803 808 )
804 809 )
... ...
src/Template/Pages/home_app.ctp
  Diff comments   View file @8392646
... ... @@ -51,14 +51,14 @@ if ($role == &#39;Responsable&#39;) {
51 51 echo '<tr><td> ' . $this->Html->link('Voir les matériels dont je suis responsable', [
52 52 'controller' => 'materiels',
53 53 'action' => 'index',
54   - 'GM' => $userConnected->groupes_metier_id,
55   - 'GT' => $userConnected->groupe_thematique_id
  54 + 'GM' => $priviledgedUser->groupes_metier_id,
  55 + 'GT' => $priviledgedUser->groupe_thematique_id
56 56 ]) . ' </td></tr>';
57 57 echo '<tr><td> ' . $this->Html->link('Voir les suivis des materiels dont je suis responsable', [
58 58 'controller' => 'suivis',
59 59 'action' => 'index',
60   - 'GM' => $userConnected->groupes_metier_id,
61   - 'GT' => $userConnected->groupe_thematique_id
  60 + 'GM' => $priviledgedUser->groupes_metier_id,
  61 + 'GT' => $priviledgedUser->groupe_thematique_id
62 62 ]) . ' </td></tr>';
63 63 echo '</table>';
64 64 }
... ...
src/Template/Suivis/find.ctp
  Diff comments   View file @8392646
... ... @@ -129,8 +129,8 @@ $r = isset ($_results);
129 129 <?php
130 130 if(($role == 'Utilisateur' && in_array($username, [$suivi->nom_createur])) || (in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur']))
131 131 || ($role == 'Responsable' &&
132   - ((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $userConnected->groupes_metier_id)
133   - ||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $userConnected->groupe_thematique_id)))) {
  132 + ((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $priviledgedUser->groupes_metier_id)
  133 + ||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) {
134 134 echo $this->Html->link(__('<i class="icon-pencil"></i>'), ['action' => 'edit', $suivi->id], ['title' => 'Editer', 'style' => 'margin: 0 2px', 'escape' => false ]);
135 135 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)]);
136 136 echo '</td>';
... ...
src/Template/Suivis/index.ctp
  Diff comments   View file @8392646
... ... @@ -22,8 +22,8 @@
22 22 <td class="actions" style="padding: 6px 0; text-align: left;">
23 23 <?php if(($role == 'Utilisateur' && in_array($username, [$suivi->nom_createur])) || (in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur']))
24 24 || ($role == 'Responsable' &&
25   - ((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $userConnected->groupes_metier_id)
26   - ||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $userConnected->groupe_thematique_id)))) { ?>
  25 + ((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $priviledgedUser->groupes_metier_id)
  26 + ||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) { ?>
27 27 <?= $this->Html->link(__('<i class="icon-pencil"></i>'), ['action' => 'edit', $suivi->id], ['title' => 'Editer', 'style' => 'margin: 0 2px', 'escape' => false ]) ?>
28 28 <?= $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)]) ?>
29 29 <?php } ?>
... ...
src/Template/Suivis/view.ctp
  Diff comments   View file @8392646
... ... @@ -12,9 +12,9 @@
12 12 || ( in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur']) )
13 13 || (
14 14 $role == 'Responsable' && (
15   - ( isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $userConnected->groupes_metier_id )
  15 + ( isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $priviledgedUser->groupes_metier_id )
16 16 ||
17   - ( isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $userConnected->groupe_thematique_id )
  17 + ( isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $priviledgedUser->groupe_thematique_id )
18 18 )
19 19 )
20 20 ) {
... ... @@ -132,8 +132,8 @@
132 132 <?php
133 133 if(($role == 'Utilisateur' && in_array($username, [$suivi->nom_createur])) || (in_array($role, ['Administration', 'Administration Plus', 'Super Administrateur']))
134 134 || ($role == 'Responsable' &&
135   - ((isset($userConnected->groupes_metier_id) && $userConnected->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $userConnected->groupes_metier_id)
136   - ||(isset($userConnected->groupe_thematique_id) && $userConnected->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $userConnected->groupe_thematique_id)))) {
  135 + ((isset($priviledgedUser->groupes_metier_id) && $priviledgedUser->groupes_metier_id != $idGmNa && $suivi->groupes_metier_id == $priviledgedUser->groupes_metier_id)
  136 + ||(isset($priviledgedUser->groupe_thematique_id) && $priviledgedUser->groupe_thematique_id != $idGtNa && $suivi->groupes_thematique_id == $priviledgedUser->groupe_thematique_id)))) {
137 137 ?>
138 138 <?= $this->Html->link(__('<i class="icon-pencil"></i>'), ['controller' => 'Documents', 'action' => 'edit', $documents->id], ['escape' => false, 'style' => 'margin:0'])?>
139 139 <?= $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)]) ?>
... ...