Bug Fixes

[mecon/samurai.git] / sistema / www / permisos / permisos-abm.php

diff --git a/sistema/www/permisos/permisos-abm.php b/sistema/www/permisos/permisos-abm.php
index eb7e95fbc47a24af6fec486da002b1a9c1c70884..5767f15f82d53aee3e22125ce5e47314fa7065f9 100644 (file)
--- a/sistema/www/permisos/permisos-abm.php
+++ b/sistema/www/permisos/permisos-abm.php
@@ -20,10 +20,16 @@
 // $Id$
 // 
     
+//REQUIRE_ONCE {{{    
+    //MECON {{{
     require_once 'MECON/HTML/QuickForm.php';
     require_once 'MECON/HTML/Tabla.php';
+    require_once 'MECON/HTML/Error.php';
+    // }}}
+    //SAMURAI {{{
     require_once 'SAMURAI/Permiso.php';
-
+    //}}}
+//}}}
 //CREO LOS OBJETO NECESARIOS {{{
     $FORM = new MECON_HTML_QuickForm ('permisos_abm','post','permisos-abm');
 // }}}
@@ -31,8 +37,9 @@
     $FORM->addElement ('header', 'cabecera', 'ABM Permisos');
     $FORM->addElement ('hidden', 'id_permiso');
     $FORM->addElement ('text'  , 'desc_permiso', 'Descripcion', array('size'=>'50'));
-    $FORM->addElement ('static', 'responsable' , 'Responsable', $_SESSION['samurai']['login']);
-    $group[] = HTML_QuickForm::createElement('submit', 'aceptar' , 'Grabar');
+    $FORM->addElement ('hidden', 'responsable' , $_SESSION['samurai']['login']);
+    $group[] = HTML_QuickForm::createElement('submit', 'aceptar' , 'Grabar', array ('onClick' => 'javascript:if (confirm(\'¿Esta Seguro?\')) return true;return false;'));
+    $group[] = HTML_QuickForm::createElement('submit', 'cancelar', 'Cancelar', array ('onClick' => 'javascript:history.go(-1);return true;'));
     $FORM->addGroup($group,'botones');
 // }}}
 //AGREGO LAS REGLAS DE VALIDACION DE LOS ELEMENTOS {{{
@@ -78,6 +85,12 @@
 // }}}
 //VALIDO EL FORMULARIO {{{
     if ($FORM->validate()) {
+       // VEO SI SE CANCELO {{{
+        $botones = $FORM->getSubmitValue('botones');
+        if (@$botones['cancelar']) {
+            header('Location: perfiles');
+        }
+        // }}}
         $desc_permiso =& $FORM->getElement  ('desc_permiso');
         $responsable  =& $FORM->getElement  ('responsable' );
         $group        =& $FORM->getElement  ('botones'     );
@@ -85,15 +98,13 @@
         $aceptar      =& $group[0];
         
         $PERMISO->setDescripcion($desc_permiso->getValue());
-        $PERMISO->setResponsable($responsable->_text); //TODO URGENTE Modificar esto. No tiene metodo static::getValue()
+        $PERMISO->setResponsable($responsable->getValue());
         
         //Verifico si se produjo algun error
         $res = $PERMISO->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();
             }
@@ -104,15 +115,14 @@
         }
     }
 // }}}
-        
 //DIBUJO LA PAGINA {{{
     $MARCO = new Marco ('../../conf/confSecciones.php');
     $MARCO->addTitle('ABM Permisos');
-    if (isset($TABLA)) {
-        $MARCO->addBody($TABLA->toHtml(1));
+    if (isset($ERROR)) {
+        $MARCO->addBody($ERROR);
     }
     $MARCO->addBody($FORM);
     $MARCO->display();
-//
+// }}}
 //FIN
 ?>