From: Martín Marrese <marrese@gmail.com>
Date: Wed, 27 Aug 2003 23:09:19 +0000 (+0000)
Subject: Cambios varios para que funcione bien con Netscape (Aunque ya no se si es asi)
X-Git-Tag: svn_import~81
X-Git-Url: https://git.llucax.com/mecon/samurai.git/commitdiff_plain/e96f92763e52997990f65c34e1a44c4deeb56e17?hp=3c1eb8124730b709c84512e3af3a5c361cdd1182

Cambios varios para que funcione bien con Netscape (Aunque ya no se si es asi)
---

diff --git a/lib/SAMURAI/HTML/Perfil.php b/lib/SAMURAI/HTML/Perfil.php
index 2a1bf95..a1e1e5a 100644
--- a/lib/SAMURAI/HTML/Perfil.php
+++ b/lib/SAMURAI/HTML/Perfil.php
@@ -20,8 +20,6 @@
 // $Id$
 //
 
-//require_once 'MECON/general.php';
-//prepend_include_path('/home/mmarrese/public_html/meconlib/lib/');
 
 //REQUIRE ONCE GENERALES {{{
 require_once 'SAMURAI/DB.php';
@@ -67,13 +65,24 @@ function SAMURAI_HTML_Perfil(&$MARCO, $id_sistema) {
     $SAMURAI_PERM = new SAMURAI_Perm ($_SESSION['usuario'], $id_sistema, $DB);
     //}}}
 
+    
+    if (@$_POST['samurai_accion'] == 'abm' || @$_GET['samurai_perfiles'] == 'abm') {
+        $OPCION = 'abm';
+    }
+    elseif (@$_POST['samurai_accion'] == 'nuevo' || @$_GET['samurai_perfiles'] == 'nuevo') {
+        $OPCION = 'nuevo';
+    }
+    else {
+        $OPCION = 'listado';
+    }
+    
     //POSIBLES ACCIONES
     // listado -> Listado de los perfiles ya creados en el sistema.
     // nuevo   -> Pantalla de seleccion o de ingreso de un nuevo nombre.
     // abm     -> Formulario para la carga de la info del perfil.
 
     //LISTADO {{{
-    if ((!@$_GET['samurai_perfiles'] && !@$_POST['samurai_accion'])|| @$_GET['samurai_perfiles'] == 'listado') {
+    if ($OPCION == 'listado') {
         //VEO SI PUEDE ACCEDER{{{
         $SAMURAI_PERM->chequear(SAMURAI_ALTA_PERFIL,SAMURAI_BAJA_PERFIL,SAMURAI_MODI_PERFIL);
         //}}}
@@ -142,7 +151,7 @@ function SAMURAI_HTML_Perfil(&$MARCO, $id_sistema) {
     }
     //}}}
     //NUEVO {{{
-    elseif (@$_GET['samurai_perfiles'] == 'nuevo' || @$_POST['samurai_accion'] == 'nuevo') {
+    elseif ($OPCION == 'nuevo') {
         //VEO SI PUEDE ACCEDER{{{
         $SAMURAI_PERM->chequear(SAMURAI_ALTA_PERFIL);
         //}}}
@@ -169,7 +178,7 @@ function SAMURAI_HTML_Perfil(&$MARCO, $id_sistema) {
             $perfiles    =& $FORM->getElement('perfiles');
             $descripcion =& $FORM->getElement('descripcion');
             if (@$botones['cancelar']) {
-                header('Location: '.$_SERVER['PHP_SELF']);
+                header('Location: '.$_SERVER['PHP_SELF'].'/');
             }
             if (!isset($botones['filtrar'])) {
                 $PERFILES = SAMURAI_Perfil::getArrayPerfiles($DB);
@@ -248,7 +257,7 @@ function SAMURAI_HTML_Perfil(&$MARCO, $id_sistema) {
     }
     //}}}
     //ABM {{{
-    elseif (@$_GET['samurai_perfiles'] == 'abm' || @$_POST['samurai_accion'] == 'abm') {
+    elseif ($OPCION == 'abm') {
         //VEO SI PUEDE ACCEDER{{{
         $SAMURAI_PERM->chequear(SAMURAI_ALTA_PERFIL,SAMURAI_BAJA_PERFIL,SAMURAI_MODI_PERFIL);
         //}}}
@@ -360,6 +369,8 @@ function SAMURAI_HTML_Perfil(&$MARCO, $id_sistema) {
                 }
                 // }}}
                 else {
+                    $samurai_accion =& $FORM->getElement('samurai_accion');
+                    $samurai_accion->setValue('listado');
                     $id_perfil    =& $FORM->getElement  ('id_perfil'  );
                     $desc_perfil  =& $FORM->getElement  ('desc_perfil');
                     $permisos     =& $FORM->getElement  ('permisos'   );
@@ -378,7 +389,7 @@ function SAMURAI_HTML_Perfil(&$MARCO, $id_sistema) {
                     $PERFIL->setPermisos   ($permisos->getSelected()); //Le asigno al perfil los permisos seleccionados
                     
                     if ($SAMURAI_PERM->tiene(SAMURAI_TIPO_PERFIL)) {
-                        $PERFIL->setTipo       ($tipo_perfil_tmp['0']   );
+                        $PERFIL->setTipo($tipo_perfil_tmp['0']);
                     }
                     
                     //Grabo y verifico si se produjo algun error
@@ -392,7 +403,8 @@ function SAMURAI_HTML_Perfil(&$MARCO, $id_sistema) {
                     }
                     else {
                         $FORM->freeze();
-                        header('Location: '.$_SERVER['PHP_SELF']);
+                        header('Location: '.$_SERVER ['PHP_SELF'].'/');
+                        exit;
                     }
                 }
             }
diff --git a/lib/SAMURAI/HTML/Usuario.php b/lib/SAMURAI/HTML/Usuario.php
index bf322a6..51eaf69 100644
--- a/lib/SAMURAI/HTML/Usuario.php
+++ b/lib/SAMURAI/HTML/Usuario.php
@@ -25,6 +25,7 @@
     require_once 'MECON/HTML/Error.php';
     require_once 'MECON/HTML/Tabla.php';
     require_once 'MECON/HTML/Image.php';
+    require_once 'SAMURAI/Perm.php';
     require_once 'SAMURAI/Perfil.php';
     require_once 'SAMURAI/Usuario.php';
     require_once 'SAMURAI/Sistema.php';
@@ -62,14 +63,21 @@ function SAMURAI_HTML_Usuario(&$MARCO, $id_sistema) {
     //VERIFICO SI SE TIENE ACCESO A LA PAGINA{{{
     $SAMURAI_PERM->chequear(SAMURAI_ALTA_USUARIO, SAMURAI_BAJA_USUARIO, SAMURAI_MODI_USUARIO);
     //}}}
+    
+    if (@$_POST['samurai_accion'] == 'abm' || @$_GET['samurai_usuarios'] == 'abm') {
+        $OPCION = 'abm';
+    }
+    else {
+        $OPCION = 'listado';
+    }
 
     //POSIBLES ACCIONES
     // listado -> Listado de los usuarios ya creados en el sistema.
     // abm     -> Formulario para la carga de la info del usuario.
     //LISTADO {{{
-    if ((!@$_GET['samurai_usuarios'] && !@$_POST['samurai_accion'])|| @$_GET['samurai_usuarios'] == 'listado') {
+    if ($OPCION == 'listado') {
         //LINKS {{{
-            $aHref      = '<a href="'.$_SERVER['PHP_SELF'].'?accion=##ACCION##&login=##NUEVO##&samurai_usuarios=abm">';
+            $aHref      = '<a href="'.$_SERVER['PHP_SELF'].'?samurai_usuarios=abm&accion=##ACCION##&login=##NUEVO##">';
             $aHrefModif = $aHref.'<img src="/MECON/images/general_modificar.gif" border="0" alt="Modificar Usuario"></a>';
             $aHrefElim  = $aHref.'<img src="/MECON/images/general_eliminar.gif"  border="0" alt="Eliminar Usuario" ></a>';
         // }}}
@@ -121,12 +129,11 @@ function SAMURAI_HTML_Usuario(&$MARCO, $id_sistema) {
             $MARCO->addBody($TABLA3);
             $MARCO->addBody($TABLA2->toHtml(1));
             $MARCO->addBody($TABLA);
-            
         // }}}
     }
     //}}}
     //ABM {{{
-    elseif (@$_GET['samurai_usuarios'] == 'abm' || @$_POST['samurai_accion'] == 'abm') {
+    elseif ($OPCION == 'abm') {
         //CREO LOS OBJETOS NECESARIOS {{{
             if (@$_GET['login']) {
                 $login = $_GET['login'];
@@ -207,7 +214,7 @@ function SAMURAI_HTML_Usuario(&$MARCO, $id_sistema) {
                 // VEO SI SE CANCELO {{{
                 $botones = $FORM->getSubmitValue('botones');
                 if (@$botones['cancelar']) {
-                    header('Location: '.$_SERVER['PHP_SELF']);
+                    header('Location: '.$_SERVER['PHP_SELF'].'/');
                 }
                 // }}}
                 else {
@@ -248,9 +255,9 @@ function SAMURAI_HTML_Usuario(&$MARCO, $id_sistema) {
                             }
                         }
                         else {
-                            $FORM->removeElement();
+//                            $FORM->removeElement();
                             $FORM->freeze();
-                            header('Location: '.$_SERVER['PHP_SELF']);
+                            header('Location: '.$_SERVER['PHP_SELF'].'/');
                         }
                     }
                 }