]> git.llucax.com Git - mecon/samurai.git/blobdiff - lib/SAMURAI/Perfil.php
BugFixes despues de tocar los $_SESSION
[mecon/samurai.git] / lib / SAMURAI / Perfil.php
index 776774e0017702eb52796be8ab90d75641dc67af..9d32a5c1caf9bd40621c0989a65609327fa19075 100644 (file)
@@ -26,8 +26,6 @@
 
 #require_once 'PEAR.php';
 
-
-
 // +X2C Class 208 :SAMURAI_Perfil
 /**
  * Clase para el manejo de los perfies.
@@ -83,6 +81,14 @@ class SAMURAI_Perfil {
      */
     var $_permisos;
 
+    /**
+     * Identificador del sistema en el cual se esta trabajando
+     *
+     * @var    int $idSistema
+     * @access protected
+     */
+    var $_idSistema;
+
     /**
      * Gets Id.
      *
@@ -206,13 +212,15 @@ class SAMURAI_Perfil {
      *
      * @param  SAMURAI_DB &$db Objeto conexion
      * @param  int $id Identificador del perfil.
+     * @param  int $idSistema Identificador del sistema en el que se esta trabajando
      *
      * @return void
      * @access public
      */
-    function SAMURAI_Perfil(&$db, $id = null) // ~X2C
+    function SAMURAI_Perfil(&$db, $id = null, $idSistema = null) // ~X2C
     {
         $this->_db = $db;
+        $this->_idSistema = $idSistema;
         if (!is_null($id)) {
             $this->setId($id);
             $this->_obtenerDatosDb();
@@ -258,7 +266,7 @@ class SAMURAI_Perfil {
         //OBTENGO EL TIPO DE PERFIL
         $tmp = $sql['verificar_asociacion'];
         $dbh = $this->_db->prepare($tmp);
-        $tmp = array ($this->getId(), $_SESSION['samurai']['id_sistema']);
+        $tmp = array ($this->getId(), $this->_idSistema);
         $res = $this->_db->execute($dbh,$tmp);        
         $re = $res->fetchrow(DB_FETCHMODE_ASSOC);
         $this->setTipo($re['tipo_perfil']);
@@ -266,7 +274,7 @@ class SAMURAI_Perfil {
         //OBTENGO LOS PERMISOS QUE TIENE ASIGNADO EL PERFIL DESDE PERM_PERFIL_SIST
         $tmp = $sql['obtener_permisos'];
         $dbh = $this->_db->prepare($tmp);
-        $tmp = array ($this->getId(), $_SESSION['samurai']['id_sistema']);
+        $tmp = array ($this->getId(), $this->_idSistema);
         $res = $this->_db->execute($dbh,$tmp);        
         $rta = array ();
         while ($re = $res->fetchrow(DB_FETCHMODE_ASSOC)) {
@@ -312,18 +320,23 @@ class SAMURAI_Perfil {
      */
     function _grabarDb() // ~X2C
     {
-       //GRABO EN PERFIL
+        //Obtengo el id del perfil de ser necesario
+        $nuevo = 0;
         if (!$this->getId()) { 
             //No existe el perfil. Lo cargo por primera vez.
             $idPerfil = $this->_db->nextId('perfil');
             $this->setId($idPerfil);       
-            
-            //GRABO EN PERM_PERFIL_SIST
-            $res = $this->_guardarPermisos();
-            if (PEAR::isError($res)) {         
-                return $res;               
-            }
+            $nuevo = 1;
+        }        
+       
+        //GRABO EN PERM_PERFIL_SIST
+        $res = $this->_guardarPermisos();
+        if (PEAR::isError($res)) {         
+            return $res;               
+        }
 
+        //GRABO EN PERFIL
+        if ($nuevo) { 
             $datos = array (
                         'id_perfil'   => $idPerfil,
                         'desc_perfil' => $this->getDescripcion(),
@@ -336,7 +349,7 @@ class SAMURAI_Perfil {
         }
         //GRABO EN PERFIL_SIST
         $datos = array ('id_perfil'   => $this->getId(),
-                        'id_sistema'  => $_SESSION['samurai']['id_sistema'],
+                        'id_sistema'  => $this->_idSistema,
                         'tipo_perfil' => $this->getTipo(),
                         'responsable' => $this->getResponsable(),
                     );
@@ -357,7 +370,7 @@ class SAMURAI_Perfil {
         //Verifico en perfil_sist_usuario
         $tmp = $sql['verif_perfil_sist_usuario'];
         $dbh = $this->_db->prepare($tmp);
-        $datos = array ($this->getId(), $_SESSION['samurai']['id_sistema']);
+        $datos = array ($this->getId(), $this->_idSistema);
         $res = $this->_db->execute($dbh, $datos);
         if (($re  = $res->fetchRow(DB_FETCHMODE_ASSOC)) && !$re['cuenta'] == 0) {
             return new PEAR_Error("Hay usuarios asociados al perfil seleccionado");
@@ -370,7 +383,7 @@ class SAMURAI_Perfil {
         //Borro perfil_sist
         $tmp = $sql['borrar_perfil_sist'];
         $dbh = $this->_db->prepare($tmp);
-        $datos = array ($this->getId(), $_SESSION['samurai']['id_sistema']);
+        $datos = array ($this->getId(), $this->_idSistema);
         $res = $this->_db->execute($dbh, $datos);
         if (PEAR::isError($res)) {         
             return $res;               
@@ -408,10 +421,18 @@ class SAMURAI_Perfil {
                     'tipo_perfil' => $this->getTipo(),
                     'responsable' => $this->getResponsable(),
         );
-        $res = $this->_db->autoExecute('perfil_sist', $datos, DB_AUTOQUERY_UPDATE, 'id_perfil ='.$this->getId().' AND id_sistema='.$_SESSION['samurai']['id_sistema']);
+        $res = $this->_db->autoExecute('perfil_sist', $datos,
+                DB_AUTOQUERY_UPDATE, 'id_perfil ='.$this->getId().' AND
+                id_sistema='.$this->_idSistema);
         if (PEAR::isError($res)) {         
             return $res;               
         }
+
+        //VERIFICO QUE NO HAYA UN PERFIL CON LOS MISMOS PERMISOS YA ASIGNADO AL SISTEMA
+        $res = $this->_verifPermisos();
+        if (PEAR::isError($res)) {
+            return $res;
+        }
         //Modifico la tabla perm_perfil_sist
         $res = $this->_borrarPermisos();
         if (PEAR::isError($res)) {         
@@ -447,6 +468,7 @@ class SAMURAI_Perfil {
             //Reemplazo el filtro por ##?##
             $consulta = ereg_replace ('##FILTRO##', $filtro, $tmp2);
         }
+        $consulta.= $sql['obtener_id_perfiles4'];
         $dbh = $db->prepare($consulta);
         if ($id_sistema) { 
             $tmp[] = $id_sistema;
@@ -457,7 +479,7 @@ class SAMURAI_Perfil {
         }
         while ($re = $res->fetchrow(DB_FETCHMODE_ASSOC)) {
             array_push($rta,$re['id_perfil']);
-        }        
+        }
         $res->free();
         return $rta;
     }
@@ -465,6 +487,8 @@ class SAMURAI_Perfil {
 
     // +X2C Operation 339
     /**
+     * Devuelve un array de perfiles
+     *
      * @param  SAMURAI_DB &$db Base de datos
      * @param  string $filtro Filtro por nombre del perfil
      * @param  int $id_sistema Identificador del sistema
@@ -477,7 +501,7 @@ class SAMURAI_Perfil {
     {
         $rta = array ();
         foreach (SAMURAI_Perfil::_getIdPerfiles($db, $filtro, $id_sistema) as $id) {
-            $tmp = new SAMURAI_Perfil($db,$id);
+            $tmp = new SAMURAI_Perfil($db,$id, $id_sistema);
             array_push($rta, $tmp);
         }
         return $rta;
@@ -490,16 +514,17 @@ class SAMURAI_Perfil {
      *
      * @param  SAMURAI_DB $db Base de Datos
      * @param  string $filtro Filtro por descripcion del perfil
+     * @param  int $id_sistema Identificador del sistema con el que se esta trabajando
      *
      * @return array()
      * @access public
      * @static
      */
-    function getArrayPerfiles($db, $filtro = null) // ~X2C
+    function getArrayPerfiles($db, $filtro = null, $id_sistema = null) // ~X2C
     {
         //FORECHEO LO QUE ME DEVUELVA GET PERMISOS
         $rta = array ();
-        foreach (SAMURAI_Perfil::getPerfiles($db, $filtro) as $perfil) {
+        foreach (SAMURAI_Perfil::getPerfiles($db, $filtro, $id_sistema) as $perfil) {
             $rta[$perfil->getId()] = $perfil->getDescripcion();
         }
         return $rta;
@@ -572,22 +597,11 @@ class SAMURAI_Perfil {
      */
     function _guardarPermisos() // ~X2C
     {
-        //VERIFICO QUE NO HAYA UN PERFIL CON LOS MISMOS PERMISOS YA ASIGNADO AL SISTEMA
         $sql = parse_ini_file(dirname(__FILE__) . '/Perfil/consultas.ini', true);
-        $tmp = $sql['verif_perm_perfil_sist'];
-        $dbh = $this->_db->prepare($tmp);
-        $tmp = array ($_SESSION['samurai']['id_sistema']);
-        $res = $this->_db->execute($dbh,$tmp);
-        $perm = array();
-        while ($re  = $res->fetchRow(DB_FETCHMODE_ASSOC)) {
-            $perm[$re['id_perfil']][] = $re['id_permiso'].'##'.$re['observaciones'];
-        }
-        foreach ($perm as $p) {
-            $rta1 = array_diff($p, $this->getPermisos());
-            $rta2 = array_diff($this->getPermisos(), $p);
-            if (!$rta1 && !$rta2) {
-                return new PEAR_Error("Ya existe un perfil con esos mismos permisos");
-            }
+        //VERIFICO QUE NO HAYA UN PERFIL CON LOS MISMOS PERMISOS YA ASIGNADO AL SISTEMA
+        $res = $this->_verifPermisos();
+        if (PEAR::isError($res)) {
+            return $res;
         }
 
         //GRABO EN PERM_PERFIL_SIST
@@ -595,7 +609,7 @@ class SAMURAI_Perfil {
         $re = $this->_db->autoPrepare('perm_perfil_sist', $datos, DB_AUTOQUERY_INSERT);
         foreach ($this->getPermisos() as $permiso) {
             list($id, $obs) = split ('##',$permiso);
-            $datos = array ($id, $this->getId(), $_SESSION['samurai']['id_sistema'], $obs, $this->getResponsable());
+            $datos = array ($id, $this->getId(), $this->_idSistema, $obs, $this->getResponsable());
             $res = $this->_db->execute($re, $datos);
             if (PEAR::isError($res)) {
                 return $res;
@@ -616,11 +630,44 @@ class SAMURAI_Perfil {
         $sql = parse_ini_file(dirname(__FILE__) . '/Perfil/consultas.ini', true);
         $tmp = $sql['borrar_permisos'];
         $dbh = $this->_db->prepare($tmp);
-        $tmp = array ($this->getId(), $_SESSION['samurai']['id_sistema']);
+        $tmp = array ($this->getId(), $this->_idSistema);
         return $this->_db->execute($dbh,$tmp);        
     }
     // -X2C
 
+    // +X2C Operation 376
+    /**
+     * Verifica si se puede insertar
+     *
+     * @return mixed
+     * @access protected
+     */
+    function _verifPermisos() // ~X2C
+    {
+        //VERIFICO QUE NO HAYA UN PERFIL CON LOS MISMOS PERMISOS YA ASIGNADO AL SISTEMA
+        $sql = parse_ini_file(dirname(__FILE__) . '/Perfil/consultas.ini', true);
+        $tmp = $sql['verif_perm_perfil_sist'];
+        $dbh = $this->_db->prepare($tmp);
+        $tmp = array ($this->_idSistema);
+        $res = $this->_db->execute($dbh,$tmp);
+        $perm = array();
+        while ($re  = $res->fetchRow(DB_FETCHMODE_ASSOC)) {
+            $perm[$re['id_perfil']][] = $re['id_permiso'].'##'.$re['observaciones'];
+        }
+        foreach ($perm as $key => $p) {
+            $rta1 = array_diff($p, $this->getPermisos());
+            $rta2 = array_diff($this->getPermisos(), $p);
+            if (!$rta1 && !$rta2) {
+                $perf = new SAMURAI_Perfil($this->_db, $key, $this->_idSistema);
+                if ($perf->getDescripcion() != $this->getDescripcion()) {
+                    return new PEAR_Error("El perfil \"".$perf->getDescripcion()."\" contiene los mismos permisos.");
+                }
+            }
+        }
+        return true;
+    }
+    // -X2C
+
 } // -X2C Class :SAMURAI_Perfil
 
 ?>