Bug Fix Varios

[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 1110161660b586802fd02965988f45426f29e4d5..95f28be82396c99cc05966c64e1a56aaa53e74f9 100644 (file)
--- a/sistema/www/sistemas/sistemas-abm.php
+++ b/sistema/www/sistemas/sistemas-abm.php
@@ -48,8 +48,8 @@
     $FORM->addElement ('textarea', 'contacto'            , 'Contacto'            , array('rows' => '2','cols'=>'50'));
     $FORM->addElement ('select'  , 'permisos'            , 'Permisos'            , $PERMISOS, array('size' => '5', 'multiple' => 'multiple'));
     $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;'));
+    $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 {{{
@@ -58,12 +58,12 @@
         $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');
+//        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');
     }
 // }}}
 //CREO EL OBJETO SISTEMA {{{
@@ -114,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();
         }
     } 
@@ -185,8 +207,11 @@
         } 
         else {
             $res = $SISTEMA->guardarDatos($aceptar->getValue());
+            if ($aceptar->getValue() != 'Eliminar') {
+                $_SESSION['samurai']['id_sistema'] = $SISTEMA->getId();
+            }
             if (PEAR::isError($res)) {
-                $ERROR = new MECON_HTML_Error($res->getMessage());
+                $ERROR = new MECON_HTML_Error(traducir($res->getMessage()));
             }
             else {
                 $FORM->freeze();