- BugFix. No grababa los permisos asociados a un perfil si se seleccionaba este del...

diff --git a/lib/SAMURAI/Perfil.php b/lib/SAMURAI/Perfil.php
index 776774e0017702eb52796be8ab90d75641dc67af..a74a16fd8b8da7bdf6b5556f56b0f3d97c8edc6e 100644 (file)
--- a/lib/SAMURAI/Perfil.php
+++ b/lib/SAMURAI/Perfil.php
@@ -312,18 +312,21 @@ class SAMURAI_Perfil {
      */
     function _grabarDb() // ~X2C
     {
      */
     function _grabarDb() // ~X2C
     {

-       //GRABO EN PERFIL
+        //Obtengo el id del perfil de ser necesario

         if (!$this->getId()) { 
             //No existe el perfil. Lo cargo por primera vez.
             $idPerfil = $this->_db->nextId('perfil');
             $this->setId($idPerfil);       
         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;               
-            }
+        }        
+       
+        //GRABO EN PERM_PERFIL_SIST
+        $res = $this->_guardarPermisos();
+        if (PEAR::isError($res)) {         
+            return $res;               
+        }

 
 

+        //GRABO EN PERFIL
+        if (!$this->getId()) { 

             $datos = array (
                         'id_perfil'   => $idPerfil,
                         'desc_perfil' => $this->getDescripcion(),
             $datos = array (
                         'id_perfil'   => $idPerfil,
                         'desc_perfil' => $this->getDescripcion(),

diff --git a/sistema/www/perfiles/perfiles-abm.php b/sistema/www/perfiles/perfiles-abm.php
index cf7fb6637bd6cb6a9a5da16f194756614024bc8e..42dcacbd576eade8b130614555db4676ebcccc03 100644 (file)
--- a/sistema/www/perfiles/perfiles-abm.php
+++ b/sistema/www/perfiles/perfiles-abm.php
@@ -23,6 +23,7 @@
 // REQUIRE_ONCE {{{
     require_once 'MECON/HTML/QuickForm.php';
     require_once 'MECON/HTML/Tabla.php';
 // REQUIRE_ONCE {{{
     require_once 'MECON/HTML/QuickForm.php';
     require_once 'MECON/HTML/Tabla.php';

+    require_once 'MECON/HTML/Error.php';

     require_once 'SAMURAI/Perfil.php';
     require_once 'SAMURAI/Permiso.php';
     require_once 'SAMURAI/Sistema.php';
     require_once 'SAMURAI/Perfil.php';
     require_once 'SAMURAI/Permiso.php';
     require_once 'SAMURAI/Sistema.php';


@@ -140,9 +141,7 @@
             $res = $PERFIL->guardarDatos($aceptar->getValue());
 
             if (PEAR::isError($res)) {
             $res = $PERFIL->guardarDatos($aceptar->getValue());
 
             if (PEAR::isError($res)) {

-                $TABLA = new Tabla ('cellspacing=0');
-                $row = array ('<font color="red"><b>'.$res->getMessage().'</b></font>');
-                $TABLA->addRow($row,'align=left');
+                $ERROR = new MECON_HTML_Error($res->getMessage());

                 if ($aceptar->getValue() == 'Eliminar') {
                     $FORM->freeze();
                 }
                 if ($aceptar->getValue() == 'Eliminar') {
                     $FORM->freeze();
                 }


@@ -162,8 +161,8 @@
     $MARCO = new Marco ('../../conf/confSecciones.php');
     $MARCO->addTitle('ABM Perfil');
     $MARCO->addBody($TABLA3);
     $MARCO = new Marco ('../../conf/confSecciones.php');
     $MARCO->addTitle('ABM Perfil');
     $MARCO->addBody($TABLA3);

-    if (isset($TABLA)) {
-        $MARCO->addBody($TABLA);
+    if (isset($ERROR)) {
+        $MARCO->addBody($ERROR);

     }
     $MARCO->addBody($FORM);
     $MARCO->display();
     }
     $MARCO->addBody($FORM);
     $MARCO->display();