Bug Fix

[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..95f28be82396c99cc05966c64e1a56aaa53e74f9 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']); 
+    $FORM->addElement ('hidden'  , 'responsable'         , $_SESSION['samurai']['login']); 
     $group[] = HTML_QuickForm::createElement('submit', 'aceptar' , 'Grabar');
+    $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.
@@ -103,24 +114,46 @@
                                                     'Y' => $tmp->format("%Y"),
                                            ));
         }
+        else {
+            $fecha_inicio->setSelectedDate(array(   'd' => '',
+                                                    'F' => '',
+                                                    'Y' => '',
+                                            ));
+        }
+
+        
         if ($tmp = $SISTEMA->getFechaFin()) { 
             $fecha_fin->setSelectedDate(array(  'd' => $tmp->format("%d"),
                                                 'F' => $tmp->format("%m"),
                                                 'Y' => $tmp->format("%Y"),
                                         ));
+        } 
+        else {
+            $fecha_fin->setSelectedDate(array(  'd' => '',
+                                                'F' => '',
+                                                'Y' => '',
+                                        ));
         }
+
         if ($tmp = $SISTEMA->getFechaImplementacion()) { 
             $fecha_implementacion->setSelectedDate(array(   'd' => $tmp->format("%d"),
                                                             'F' => $tmp->format("%m"),
                                                             'Y' => $tmp->format("%Y"),
                                                     ));
         } 
+        else {
+            $fecha_implementacion->setSelectedDate(array(   'd' => '',
+                                                            'F' => '',
+                                                            'Y' => '',
+                                                    ));
+        }
 
         $aceptar->setValue('Modificar');
         
         //ELIMINACION -> modifico el valor del boton
         if ($_GET['accion'] == 'e') {
             $aceptar->setValue('Eliminar');
+            $aceptar->updateAttributes(array ('onClick' => 'javascript:if (confirm(\'¿Esta Seguro?\')) return true;return false;'));
             $FORM->freeze();
         }
     } 
@@ -167,17 +200,23 @@
         }
         $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 ($aceptar->getValue() != 'Eliminar') {
+                $_SESSION['samurai']['id_sistema'] = $SISTEMA->getId();
+            }
+            if (PEAR::isError($res)) {
+                $ERROR = new MECON_HTML_Error(traducir($res->getMessage()));
+            }
+            else {
+                $FORM->freeze();
+                header('Location: sistemas');
+            }
         }
     }
 // }}}
@@ -185,8 +224,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();