BugFix.

[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 09fa1477db0c8a60d57535b1eeb66c6a586b780d..9b9057a57ad3d9560c62b7ccc229d2c826b8d421 100644 (file)
--- a/sistema/www/permisos/permisos-abm.php
+++ b/sistema/www/permisos/permisos-abm.php
@@ -27,7 +27,7 @@ if (!$SAMURAI_PERM->tiene(SAMURAI_PERM_DEVELOPER)) {
 $MARCO =& new MECON_Marco ('/var/www/sistemas/samurai/sistema/conf/confSecciones.php', $SAMURAI_PERM);
 //}}}
 
-//XXX OBTENGO LA ACCION {{{
+//OBTENGO LA ACCION {{{
 switch ($_REQUEST['accion']) {
     case 'm':
         $b_accion = 'Modificar';
@@ -45,109 +45,117 @@ switch ($_REQUEST['accion']) {
 //}}}
     
 //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';
-    //}}}
+//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');
-    $FORM->renderer->updateAttributes('width="400"');
+$FORM = new MECON_HTML_QuickForm ('permisos_abm','post','permisos-abm');
+$FORM->renderer->updateAttributes('width="400"');
 // }}}
+
 //AGREGO LOS ELEMENTOS DEL FORM {{{
-    $FORM->addElement ('header', 'cabecera', 'ABM Permisos');
-    $FORM->addElement ('hidden', 'id_permiso');
-    $FORM->addElement ('text'  , 'desc_permiso', 'Descripción', array('size'=>'50'));
-    $FORM->addElement ('hidden', 'responsable' , $_SESSION['usuario']);
-    $FORM->addElement ('hidden', 'accion', $accion);
-    $group[] = HTML_QuickForm::createElement('submit', 'aceptar', $b_accion);
-    $group[] = HTML_QuickForm::createElement('button', 'cancelar', 'Cancelar', array ('onClick' => 'javascript:history.go(-1);return true;'));
-    $FORM->addGroup($group,'botones');
+$FORM->addElement ('header', 'cabecera', 'ABM Permisos');
+$FORM->addElement ('hidden', 'id_permiso');
+$FORM->addElement ('text'  , 'desc_permiso', 'Descripción', 
+        array('size'=>'50'));
+$FORM->addElement ('hidden', 'responsable' , $_SESSION['usuario']);
+$FORM->addElement ('hidden', 'accion', $accion);
+$group[] = HTML_QuickForm::createElement('submit', 'aceptar', $b_accion);
+$group[] = HTML_QuickForm::createElement('button', 'cancelar', 'Cancelar', 
+        array ('onClick' => 'javascript:history.go(-1);return true;'));
+$FORM->addGroup($group,'botones');
 // }}}
+
 //AGREGO LAS REGLAS DE VALIDACION DE LOS ELEMENTOS {{{
-    $FORM->addRule('desc_permiso', 'El campo descripción es obligatorio', 'required', '', 'client');
+$FORM->addRule('desc_permiso', 'El campo descripción es obligatorio', 'required', '', 'client');
 // }}}
+
 //CREO EL OBJETO PERMISO {{{
-    //Obtengo el id del permiso.
-    $idPermiso =& $FORM->getElement('id_permiso');
-    if (isset($_GET['accion']) && $_GET['accion'] != '') {
-        //Viene en el get es -> modificar o eliminar, me ocupo yo.
-        $id_permiso = $_GET['idPermiso'];
-        $idPermiso->setValue($id_permiso);
-    }
-    else {
-        //Viene en el post, no me ocupo yo.
-        $id_permiso = $idPermiso->getValue();
-    }
-    $PERMISO = new SAMURAI_Permiso ($DB, $id_permiso);
-// }}}
+//Obtengo el id del permiso.
+$idPermiso =& $FORM->getElement('id_permiso');
+if (isset($_GET['accion']) && $accion != '') {
+    //Viene en el get es -> modificar o eliminar, me ocupo yo.
+    $id_permiso = $_GET['_id_permiso'];
+    $idPermiso->setValue($id_permiso);
+}
+else {
+    //Viene en el post, no me ocupo yo.
+    $id_permiso = $idPermiso->getValue();
+}
+$PERMISO = new SAMURAI_Permiso ($DB, $id_permiso);
+//}}}
+
 //SETEO LOS VALORES DE LOS ELEMENTOS DEL OBJETO FORM {{{
-    if (isset($_GET['accion']) && $_GET['accion'] != '') {
-        //MODIFICACION -> agrego la info a los elementos
-        $desc_permiso =& $FORM->getElement  ('desc_permiso');
-        $responsable  =& $FORM->getElement  ('responsable' );
-        $group        =& $FORM->getElement  ('botones'     );
-        $group        =& $group->getElements('aceptar'     );
-        $aceptar      =& $group[0];
+if (isset($_GET['accion']) && $accion != '') {
+    //MODIFICACION -> agrego la info a los elementos
+    $desc_permiso =& $FORM->getElement  ('desc_permiso');
+    $responsable  =& $FORM->getElement  ('responsable' );
+    $group        =& $FORM->getElement  ('botones'     );
+    $group        =& $group->getElements('aceptar'     );
+    $aceptar      =& $group[0];
 
-        $desc_permiso->setValue ($PERMISO->getDescripcion());
-        
-        if ($PERMISO->getResponsable() != '') {
-            $responsable->setValue($PERMISO->getResponsable());
-        }
-        
-        $aceptar->setValue('Modificar');
-        
-        //ELIMINACION -> modifico el valor del boton
-        if ($_GET['accion'] == 'e') {
-            $aceptar->setValue('Eliminar');
-            $FORM->freeze();
-        }
+    $desc_permiso->setValue ($PERMISO->getDescripcion());
+
+    if ($PERMISO->getResponsable() != '') {
+        $responsable->setValue($PERMISO->getResponsable());
     }
+
+    $aceptar->setValue('Modificar');
+
+    //ELIMINACION -> modifico el valor del boton
+    if ($_GET['accion'] == 'e') {
+        $aceptar->setValue('Eliminar');
+        $FORM->freeze();
+    }
+}
 // }}}
+
 //VALIDO EL FORMULARIO {{{
-    if ($FORM->validate()) {
-       // VEO SI SE CANCELO {{{
-        $botones = $FORM->getSubmitValue('botones');
-        if (@$botones['cancelar']) {
-            header('Location: permisos');
-        }
-        // }}}
-        $desc_permiso =& $FORM->getElement  ('desc_permiso');
-        $responsable  =& $FORM->getElement  ('responsable' );
-        $group        =& $FORM->getElement  ('botones'     );
-        $group        =& $group->getElements('aceptar'     );
-        $aceptar      =& $group[0];
-        
-        $PERMISO->setDescripcion($desc_permiso->getValue());
-        $PERMISO->setResponsable($responsable->getValue());
-        
-        //Verifico si se produjo algun error
-        $res = $PERMISO->guardarDatos($aceptar->getValue());
+if ($FORM->validate()) {
+    // VEO SI SE CANCELO {{{
+    $botones = $FORM->getSubmitValue('botones');
+    if (@$botones['cancelar']) {
+        header('Location: permisos');
+    }
+    // }}}
+    $desc_permiso =& $FORM->getElement  ('desc_permiso');
+    $responsable  =& $FORM->getElement  ('responsable' );
+    $group        =& $FORM->getElement  ('botones'     );
+    $group        =& $group->getElements('aceptar'     );
+    $aceptar      =& $group[0];
 
-        if (PEAR::isError($res)) {
-            $ERROR = new MECON_HTML_Error(traducir($res->getMessage()));
-            if ($aceptar->getValue() == 'Eliminar') {
-                $FORM->freeze();
-            }
-        }
-        else {
+    $PERMISO->setDescripcion($desc_permiso->getValue());
+    $PERMISO->setResponsable($responsable->getValue());
+
+    //Verifico si se produjo algun error
+    $res = $PERMISO->guardarDatos($aceptar->getValue());
+
+    if (PEAR::isError($res)) {
+        $ERROR = new MECON_HTML_Error(traducir($res->getMessage()));
+        if ($aceptar->getValue() == 'Eliminar') {
             $FORM->freeze();
-            header('Location: permisos');
         }
     }
+    else {
+        $FORM->freeze();
+        header('Location: permisos');
+    }
+}
 // }}}
+
 //DIBUJO LA PAGINA {{{
-    $MARCO->addTitle('ABM Permisos');
-    if (isset($ERROR)) {
-        $MARCO->addBody($ERROR);
-    }
-    $MARCO->addBody($FORM);
-    $MARCO->display();
+$MARCO->addTitle('ABM Permisos');
+if (isset($ERROR)) {
+    $MARCO->addBody($ERROR);
+}
+$MARCO->addBody($FORM);
+$MARCO->display();
 // }}}
-//FIN
 ?>