Bug Fixes

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

diff --git a/sistema/www/sistemas/sistemas-abm.php b/sistema/www/sistemas/sistemas-abm.php
index 7a02ab9ea149d3fa185cf8312a173bdc1085aa95..1110161660b586802fd02965988f45426f29e4d5 100644 (file)
--- a/sistema/www/sistemas/sistemas-abm.php
+++ b/sistema/www/sistemas/sistemas-abm.php
@@ -19,11 +19,18 @@
 //
 // $Id$
 //
+
+//REQUIRE ONCE {{{
+    //MECON {{{
     require_once 'MECON/HTML/QuickForm.php';
+    require_once 'MECON/HTML/Error.php';
     require_once 'MECON/HTML/Tabla.php';
+    // }}}
+    //SAMURAI {{{
     require_once 'SAMURAI/Sistema.php';
     require_once 'SAMURAI/Permiso.php';
-
+    // }}}
+// }}}
 //CREO LOS OBJETO NECESARIOS {{{
     $FORM = new MECON_HTML_QuickForm ('sistemas_abm','post','sistemas-abm');
 // }}}
@@ -40,20 +47,24 @@
     $FORM->addElement ('mdate'   , 'fecha_implementacion', 'Fecha Implementacion');
     $FORM->addElement ('textarea', 'contacto'            , 'Contacto'            , array('rows' => '2','cols'=>'50'));
     $FORM->addElement ('select'  , 'permisos'            , 'Permisos'            , $PERMISOS, array('size' => '5', 'multiple' => 'multiple'));
-    $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:window.location =             ->\'sistemas\';return false;'));
     $FORM->addGroup($group,'botones');
 // }}}
 //AGREGO LAS REGLAS DE VALIDACION DE LOS ELEMENTOS {{{
-    $FORM->addRule('nombre_sistema'      , 'El campo nombre es obligatorio'      , 'required');
-    $FORM->addRule('desc_sistema'        , 'El campo descripcion es obligatorio' , 'required');
-    $FORM->addRule('fecha_inicio'        , 'El campo fecha inicio es obligatorio', 'required');
-    if (isset($_GET['accion']) && $_GET['accion'] != 'e') {
-        $FORM->addRule('fecha_inicio'        , 'La fecha no es válida'               , 'fecha'   );
-        $FORM->addRule('fecha_fin'           , 'La fecha no es válida'               , 'fecha'   );
-        $FORM->addRule('fecha_implementacion', 'La fecha no es válida'               , 'fecha'   );
+    $botones = $FORM->getSubmitValue('botones');
+    if (!@$botones['cancelar']) {
+        $FORM->addRule('nombre_sistema'      , 'El campo nombre es obligatorio'      , 'required');
+        $FORM->addRule('desc_sistema'        , 'El campo descripcion es obligatorio' , 'required');
+        $FORM->addRule('fecha_inicio'        , 'El campo fecha inicio es obligatorio', 'required');
+        if (isset($_GET['accion']) && $_GET['accion'] != 'e') {
+            $FORM->addRule('fecha_inicio'        , 'La fecha no es válida'               , 'fecha'   );
+            $FORM->addRule('fecha_fin'           , 'La fecha no es válida'               , 'fecha'   );
+            $FORM->addRule('fecha_implementacion', 'La fecha no es válida'               , 'fecha'   );
+        }
+        $FORM->addRule('contacto'            , 'El campo contacto es obligatorio'    , 'required');
     }
-    $FORM->addRule('contacto'            , 'El campo contacto es obligatorio'    , 'required');
 // }}}
 //CREO EL OBJETO SISTEMA {{{
     //Obtengo el id del sistema.
@@ -167,17 +178,20 @@
         }
         $SISTEMA->setContacto($contacto->getValue());        
         $SISTEMA->setPermisos($permisos->getSelected());
-        $SISTEMA->setResponsable($responsable->_text); //TODO URGENTE Modificar esto. No tiene hay metodo static::getValue()
-        $res = $SISTEMA->guardarDatos($aceptar->getValue());
-        print '<PRE>';var_dump($res);print '</PRE>';
-        if (PEAR::isError($res)) {
-            $TABLA = new Tabla ('cellspacing=0');
-            $row = array ('<font color="red"><b>'.$res->getMessage().'</b></font>');
-            $TABLA->addRow($row,'align=left');
-        }
+        $SISTEMA->setResponsable($responsable->getValue());
+        $botones = $FORM->getSubmitValue('botones');
+        if (@$botones['cancelar']) {
+            header ('Location: sistemas');
+        } 
         else {
-            $FORM->freeze();
-            header('Location: sistemas');
+            $res = $SISTEMA->guardarDatos($aceptar->getValue());
+            if (PEAR::isError($res)) {
+                $ERROR = new MECON_HTML_Error($res->getMessage());
+            }
+            else {
+                $FORM->freeze();
+                header('Location: sistemas');
+            }
         }
     }
 // }}}
@@ -185,8 +199,8 @@
 //DIBUJO LA PAGINA {{{
     $MARCO = new Marco ('../../conf/confSecciones.php');
     $MARCO->addTitle('ABM Sistema');
-    if (isset($TABLA)) {
-        $MARCO->addBody($TABLA->toHtml(1));
+    if (isset($ERROR)) {
+        $MARCO->addBody($ERROR);
     }
     $MARCO->addBody($FORM);
     $MARCO->display();