Funcionalidad terminada, salvo consultas. Solo falta ver bien si hay errores

[mecon/samurai.git] / lib / SAMURAI / Perfil.php

diff --git a/lib/SAMURAI/Perfil.php b/lib/SAMURAI/Perfil.php
index 776774e0017702eb52796be8ab90d75641dc67af..b5742f84a00b76ce33c0429502335a9b3df01f59 100644 (file)
--- a/lib/SAMURAI/Perfil.php
+++ b/lib/SAMURAI/Perfil.php
@@ -26,8 +26,6 @@
 
 #require_once 'PEAR.php';
 
-
-
 // +X2C Class 208 :SAMURAI_Perfil
 /**
  * Clase para el manejo de los perfies.
@@ -312,18 +310,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(),
@@ -447,6 +450,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 +461,7 @@ class SAMURAI_Perfil {
         }
         while ($re = $res->fetchrow(DB_FETCHMODE_ASSOC)) {
             array_push($rta,$re['id_perfil']);
-        }        
+        }
         $res->free();
         return $rta;
     }
@@ -582,11 +586,13 @@ class SAMURAI_Perfil {
         while ($re  = $res->fetchRow(DB_FETCHMODE_ASSOC)) {
             $perm[$re['id_perfil']][] = $re['id_permiso'].'##'.$re['observaciones'];
         }
-        foreach ($perm as $p) {
+        foreach ($perm as $key => $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");
+            echo 1;
+                $perf = new SAMURAI_Perfil($this->_db, $key);
+                return new PEAR_Error("El perfil \"".$perf->getDescripcion()."\" contiene los mismos permisos.");
             }
         }
 
@@ -623,4 +629,4 @@ class SAMURAI_Perfil {
 
 } // -X2C Class :SAMURAI_Perfil
 
-?>
+?>
\ No newline at end of file