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 de5acc3fb3672c1f4e6082154d2e8b8f18b0782a..9b9057a57ad3d9560c62b7ccc229d2c826b8d421 100644 (file)
--- a/sistema/www/permisos/permisos-abm.php
+++ b/sistema/www/permisos/permisos-abm.php
@@ -26,110 +26,136 @@ if (!$SAMURAI_PERM->tiene(SAMURAI_PERM_DEVELOPER)) {
 }
 $MARCO =& new MECON_Marco ('/var/www/sistemas/samurai/sistema/conf/confSecciones.php', $SAMURAI_PERM);
 //}}}
 }
 $MARCO =& new MECON_Marco ('/var/www/sistemas/samurai/sistema/conf/confSecciones.php', $SAMURAI_PERM);
 //}}}

+
+//OBTENGO LA ACCION {{{
+switch ($_REQUEST['accion']) {
+    case 'm':
+        $b_accion = 'Modificar';
+        $accion = 'm';
+        break;
+    case 'e':
+        $b_accion = 'Eliminar';
+        $accion = 'e';
+        break;
+    default:
+        $b_accion = 'Grabar';
+        $accion = '';
+        break;
+}
+//}}}

     
 //REQUIRE_ONCE {{{    
     
 //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 {{{
 //CREO LOS OBJETO NECESARIOS {{{

-    $FORM = new MECON_HTML_QuickForm ('permisos_abm','post','permisos-abm');
+$FORM = new MECON_HTML_QuickForm ('permisos_abm','post','permisos-abm');
+$FORM->renderer->updateAttributes('width="400"');

 // }}}
 // }}}

+

 //AGREGO LOS ELEMENTOS DEL FORM {{{
 //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']);
-    $group[] = HTML_QuickForm::createElement('submit', 'aceptar' , 'Grabar');
-    $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 {{{
 //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 {{{
 //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 {{{
 //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');
-            $aceptar->updateAttributes(array ('onClick' => 'javascript:if (confirm(\'¿Esta Seguro?\')) return true;return false;'));
-            $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 {{{
 //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();
             $FORM->freeze();

-            header('Location: permisos');

         }
     }
         }
     }

+    else {
+        $FORM->freeze();
+        header('Location: permisos');
+    }
+}

 // }}}
 // }}}

+

 //DIBUJO LA PAGINA {{{
 //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

 ?>
 ?>