X-Git-Url: https://git.llucax.com/mecon/samurai.git/blobdiff_plain/2ba023cd1e829c20ad139ca6c2c738738b4f15fe..3c702b40a1806798e475c48abb456e5a9c2dac72:/sistema/www/perfiles/perfiles-abm.php?ds=sidebyside

diff --git a/sistema/www/perfiles/perfiles-abm.php b/sistema/www/perfiles/perfiles-abm.php
index 42082d4..485a13b 100644
--- a/sistema/www/perfiles/perfiles-abm.php
+++ b/sistema/www/perfiles/perfiles-abm.php
@@ -23,9 +23,11 @@
 // 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 'HTML/Image.php';
 // }}}
 //CREO LOS OBJETO NECESARIOS {{{
     if (@$_GET['id_perfil']) {
@@ -74,7 +76,7 @@
     $FORM->addElement ('static', 'desc_perfil', 'Descripcion', $descripcion);
     $FORM->addElement ('select', 'permisos'   , 'Permisos'   , $PERMISOS, array('size' => '5', 'multiple' => 'true'));
     $FORM->addElement ('select', 'tipo_perfil', 'Tipo Perfil', array('E' => 'Externo', 'I' => 'Interno', 'D' => 'Dios'), array ('size' => '1'));
-    $FORM->addElement ('static', 'responsable', 'Responsable', $responsable);
+    $FORM->addElement ('hidden', 'responsable', $responsable);
     $group[] = HTML_QuickForm::createElement('submit', 'aceptar' , 'Grabar');
     $group[] = HTML_QuickForm::createElement('submit', 'cancelar', 'Cancelar', array ('onClick' => 'javascript:history.go(-1);return true;'));
     $FORM->addGroup($group,'botones');
@@ -83,7 +85,6 @@
     $FORM->addRule ('permisos', 'Se debe seleccionar al menos un permiso', 'required');
 // }}}
 //CARGO LOS DATOS SI YA EXISTEN EN BASE {{{
-// TODO Terminar de armar la parte de modificacion y eliminacion
     if (isset($_GET['accion']) && $_GET['accion'] != '') {
         //MODIFICACION
         $id_perfil    =& $FORM->getElement  ('id_perfil'  );
@@ -132,7 +133,7 @@
             $aceptar      =& $group[0];
       
             $PERFIL->setDescripcion($desc_perfil->_text     );
-            $PERFIL->setResponsable($responsable->_text     ); //TODO URGENTE Modificar esto. No tiene metodo static::getValue()
+            $PERFIL->setResponsable($responsable->getValue());
             $PERFIL->setPermisos   ($permisos->getSelected()); //Le asigno al perfil los permisos seleccionados
             $PERFIL->setTipo       ($tipo_perfil_tmp['0']   );
             
@@ -140,9 +141,7 @@
             $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();
                 }
@@ -155,15 +154,16 @@
     }
 // }}}
 //DIBUJO LA PAGINA {{{
+    $imagen = new HTML_Image('/MECON/images/vinetas_flecha_doble.gif');
+    $row    = array ($imagen->toHtml().'&nbsp;<b>SISTEMA: '.$SISTEMA->getNombre().'</b>');
     $TABLA3 = new Tabla ('cellspacing=0');
-    $row   = array ('<b>SISTEMA: '.$SISTEMA->getNombre().'</b>');
-    $TABLA3->addRow($row,'titulo align=left');    
+    $TABLA3->addRow($row,'align=left');    
 
     $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();