Se corrige bug.

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

diff --git a/lib/SAMURAI/Perfil.php b/lib/SAMURAI/Perfil.php
index b5742f84a00b76ce33c0429502335a9b3df01f59..acbce4ed588ec9fcb4ed85a3225295c6ca8eba48 100644 (file)
--- a/lib/SAMURAI/Perfil.php
+++ b/lib/SAMURAI/Perfil.php
@@ -415,6 +415,12 @@ class SAMURAI_Perfil {
         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)) {         
@@ -576,24 +582,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 $key => $p) {
-            $rta1 = array_diff($p, $this->getPermisos());
-            $rta2 = array_diff($this->getPermisos(), $p);
-            if (!$rta1 && !$rta2) {
-            echo 1;
-                $perf = new SAMURAI_Perfil($this->_db, $key);
-                return new PEAR_Error("El perfil \"".$perf->getDescripcion()."\" contiene los 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
@@ -627,6 +620,39 @@ class SAMURAI_Perfil {
     }
     // -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 ($_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 $key => $p) {
+            $rta1 = array_diff($p, $this->getPermisos());
+            $rta2 = array_diff($this->getPermisos(), $p);
+            if (!$rta1 && !$rta2) {
+                $perf = new SAMURAI_Perfil($this->_db, $key);
+                if ($perf->getDescripcion() != $this->getDescripcion()) {
+                    return new PEAR_Error("El perfil \"".$perf->getDescripcion()."\" contiene los mismos permisos.");
+                }
+            }
+        }
+        return true;
+    }
+    // -X2C
+
 } // -X2C Class :SAMURAI_Perfil
 
 ?>
\ No newline at end of file