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 6d5db38baab7ca5ee9adbac3303ce5884cbbd32e..1110161660b586802fd02965988f45426f29e4d5 100644 (file)
--- a/sistema/www/sistemas/sistemas-abm.php
+++ b/sistema/www/sistemas/sistemas-abm.php
@@ -18,19 +18,24 @@
 // +----------------------------------------------------------------------+
 //
 // $Id$
-// $Author$
-// $URL$
-// $Date$
-// $Rev$
 //
-    require_once 'MECON/HTML/QuickForm.php';
 
+//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 {{{
-    $SAMURAI = new Samurai($DB);
-    $FORM    = new MECON_HTML_QuickForm ('sistemas_abm','post','sistemas-abm');
+    $FORM = new MECON_HTML_QuickForm ('sistemas_abm','post','sistemas-abm');
 // }}}
 //OBTENGO LA LISTA DE PERMISOS {{{
-    $PERMISOS = $SAMURAI->getSelectPermisos();
+    $PERMISOS = SAMURAI_Permiso::getArrayPermisos($DB);
 // }}}
 //AGREGO LOS ELEMENTOS DEL FORM {{{
     $FORM->addElement ('header'  , 'cabecera'            , 'ABM Sistemas');
@@ -42,18 +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');
-    $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');
+    $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');
+    }
 // }}}
 //CREO EL OBJETO SISTEMA {{{
     //Obtengo el id del sistema.
@@ -69,7 +80,7 @@
         $id_sistema = $idSistema->getValue();
     }
 
-    $SISTEMA = new Sistema ($DB, $id_sistema);
+    $SISTEMA = new SAMURAI_Sistema ($DB, $id_sistema);
 // }}}
 //SETEO LOS VALORES DE LOS ELEMENTOS DEL OBJETO FORM {{{
     if (isset($_GET['accion']) && $_GET['accion'] != '') {
@@ -167,20 +178,20 @@
         }
         $SISTEMA->setContacto($contacto->getValue());        
         $SISTEMA->setPermisos($permisos->getSelected());
-        $SISTEMA->setResponsable($responsable->_text); //TODO URGENTE Modificar esto. No tiene hay metodo static::getValue()
-        $FORM->freeze();
-        $res = $SISTEMA->guardarDatos($aceptar->getValue());
-        
-        if (PEAR::isError($res)) {
-            $group = array ();
-            $group[] = HTML_QuickForm::createElement('static', 'error' , 'Error:', $res->getMessage());
-            $group[] = HTML_QuickForm::createElement('submit', 'aceptar' , $aceptar->getValue());
-            $FORM->removeElement('botones');
-            $FORM->addGroup($group,'botones', '', '<br>');
-        }
+        $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');
+            }
         }
     }
 // }}}
@@ -188,6 +199,9 @@
 //DIBUJO LA PAGINA {{{
     $MARCO = new Marco ('../../conf/confSecciones.php');
     $MARCO->addTitle('ABM Sistema');
+    if (isset($ERROR)) {
+        $MARCO->addBody($ERROR);
+    }
     $MARCO->addBody($FORM);
     $MARCO->display();
 // }}}