patch bug affichage employé par le grand responsable (#857)

libertempo · Oct 28, 2019 · fa9943b · fa9943b
1 parent 5d433dd
commit fa9943b
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/App/ProtoControllers/Responsable.php b/App/ProtoControllers/Responsable.php
@@ -135,6 +135,20 @@ public static function isRespDeUtilisateur($resp, $user)
                 && \App\ProtoControllers\Groupe::isResponsableGroupe($resp, \App\ProtoControllers\Utilisateur::getGroupesId($user), \includes\SQL::singleton());
     }
 
+    /**
+     * Vérifie si un utilisateur est bien le grand responsable d'un employé
+     *
+     * @param string $resp
+     * @param string $user
+     *
+     * @return bool
+     */
+    public static function isGrandRespDeUtilisateur($resp, $user)
+    {
+        return $resp != $user
+                && \App\ProtoControllers\Groupe::isGrandResponsableGroupe($resp, \App\ProtoControllers\Utilisateur::getGroupesId($user), \includes\SQL::singleton());
+    }
+
     /**
      * Vérifie si un utilisateur est responsable par délégation d'un employé
      *

diff --git a/responsable/Fonctions.php b/responsable/Fonctions.php
@@ -888,7 +888,8 @@ public static function traiteUserModule()
         };
         $userLogin = $entities(getpost_variable('user_login'));
 
-        if (!\App\ProtoControllers\Responsable::isRespDeUtilisateur($_SESSION['userlogin'], $userLogin)) {
+        if (!(\App\ProtoControllers\Responsable::isRespDeUtilisateur($_SESSION['userlogin'], $userLogin)
+                || \App\ProtoControllers\Responsable::isGrandRespDeUtilisateur($_SESSION['userlogin'])))  {
             redirect(ROOT_PATH . 'deconnexion');
             exit;
         }