- Se arregla el redirect (al final queda usando variable de sesión porque si no

  había que arrastrarlo por todos lados, pero se borra cuando no se necesita
  más). Ahora no hace falta pasarle el origen a pagina_check_login(), lo
  averigua solo.
- Algunos bugfixes menores (páginas a las que le faltaba el include de
  lib/pagina.php y hacía que falle el logout).
- Se pone bien el $Id$ de los archivos.

diff --git a/src/admin.login.php b/src/admin.login.php
index 55f505ce34626940c45cbed22db9e56dc7d0299a..e8f24d5a416b474c29dbff2a08d90c656d1a4595 100644 (file)
--- a/src/admin.login.php
+++ b/src/admin.login.php
@@ -22,18 +22,16 @@ if (Usuario::checkAdmins()) // Si est
             // Pongo el usuario en la sesión para que los otros módulos sepan
             // que está logueado
             $_SESSION['user'] = new Usuario($_POST['admin_id']);
-
-                                               // Esta modificacion permitiria despues del login redireccionar a la pagina
-                                               // desde la que fue llamado.
-                                               if ( isset( $_SESSION['origen'] ) )
-                                               {
-                                                $redireccion = $_SESSION['origen'];
-                                               }
-                                               else
-                                               {
-                                                $redireccion = 'admin.logged.php';
-                                               }
-            include $redireccion;
+            // Si nos piden, mostramos la página pedida en vez de la default.
+            if (isset($_SESSION['login_redirect']))
+            {
+                include $_SESSION['login_redirect'];
+                unset($_SESSION['login_redirect']);
+            }
+            else // default
+            {
+                include 'admin.logged.php';
+            }
         }
         else // Si falla el login, muestro error y form de nuevo
         {

diff --git a/src/admin.logout.php b/src/admin.logout.php
index dc7caf8b1a15c387646d8e8c62f2647b4b1bb749..9376e2a48556c54763e41e2d661215c3da975950 100644 (file)
--- a/src/admin.logout.php
+++ b/src/admin.logout.php
@@ -8,6 +8,8 @@
 //
 // $Id$
 
+require_once 'lib/pagina.php';
+
 unset($_SESSION['user']);
 include 'admin.login.php';
 

diff --git a/src/faq.desactivarPregunta.php b/src/faq.desactivarPregunta.php
index a2bc8679185ec0910dfb60514a36983e2c8ac369..677286ce101415d94ede770b5ab55917106703dc 100644 (file)
--- a/src/faq.desactivarPregunta.php
+++ b/src/faq.desactivarPregunta.php
@@ -6,7 +6,7 @@
 //
 // Creado: Jonathan Schein 02/05/2005
 //
-// $Id: faq.desactivarPregunta.php 
+// $Id$
 
 require_once 'lib/Pregunta.php';
 require_once 'lib/faq.functions.php';
@@ -31,4 +31,3 @@ else
 pagina_pie('Jonathan Schein', 'jonathanschein@fibertel.com.ar');
 
 ?>
-

diff --git a/src/faq.ingresarPregunta.php b/src/faq.ingresarPregunta.php
index 310f2e0433be413b1a39c70bb9676e056107ce5a..41590252842c9709878c09a852cf9c55c07652f9 100644 (file)
--- a/src/faq.ingresarPregunta.php
+++ b/src/faq.ingresarPregunta.php
@@ -6,7 +6,7 @@
 //
 // Creado: Jonathan Schein 01/05/2005
 //
-// $Id: faq.ingresarPregunta.php 
+// $Id$
 
 require_once 'lib/Pregunta.php';
 require_once 'lib/faq.functions.php';
@@ -31,4 +31,4 @@ else
 
 pagina_pie('Jonathan Schein', 'jonathanschein@fibertel.com.ar');
 
-?>
+?>
\ No newline at end of file

diff --git a/src/faq.listadoPreguntas.php b/src/faq.listadoPreguntas.php
index 8a8d195e902ef7793ca00eccacdb9f331c68a386..1a99987e4f12b2e0c12d9e1ee6d61055c8095073 100644 (file)
--- a/src/faq.listadoPreguntas.php
+++ b/src/faq.listadoPreguntas.php
@@ -6,14 +6,14 @@
 //
 // Creado: Jonathan Schein 01/05/2005
 //
-// $Id: faq.listadoPreguntas.php 
+// $Id$
 
 require_once 'lib/faq.functions.php';
 require_once 'lib/pagina.php';
 
 
 // Chequea que se este logueado
-pagina_check_login('faq.listadoPreguntas.php');
+pagina_check_login();
 
 pagina_cabecera('Listado de Preguntas');
 
@@ -21,4 +21,4 @@ Faq_ListarPreguntas($_SESSION['user']);
 
 pagina_pie('Jonathan Schein', 'jonathanschein@fibertel.com.ar');
 
-?>
+?>
\ No newline at end of file

diff --git a/src/faq.responderPregunta.php b/src/faq.responderPregunta.php
index e27b163e45168c453b1e390dc409d659cabaf449..a89c39f1ebe458ef9d04eca0a40c5c2dc1d9abcc 100644 (file)
--- a/src/faq.responderPregunta.php
+++ b/src/faq.responderPregunta.php
@@ -6,7 +6,7 @@
 //
 // Creado: Jonathan Schein 01/05/2005
 //
-// $Id: faq.responderPregunta.php 
+// $Id$
 
 require_once 'lib/Pregunta.php';
 require_once 'lib/faq.functions.php';
@@ -31,4 +31,4 @@ else
 
 pagina_pie('Jonathan Schein', 'jonathanschein@fibertel.com.ar');
 
-?>
+?>
\ No newline at end of file

diff --git a/src/index.php b/src/index.php
index a98f50b18b41b4c5f15b5726914311f1240f6d26..9d474416edef27a9f2520ba57581a5e2bf8be415 100644 (file)
--- a/src/index.php
+++ b/src/index.php
@@ -8,6 +8,7 @@
 //
 // $Id$
 
+require_once 'lib/pagina.php';
 require_once 'lib/Usuario.php';
 
 if (Usuario::checkAdmins()) // Están todos los admins

diff --git a/src/lib/faq.forms.php b/src/lib/faq.forms.php
index c018dba46020767a6491aca74df2a6e81a4ce0a8..0784dcb22da40c66f2a566c1ba90867d8ca0fc62 100644 (file)
--- a/src/lib/faq.forms.php
+++ b/src/lib/faq.forms.php
@@ -6,7 +6,7 @@
 //
 // Creado: Jonathan Schein 01/05/2005
 //
-// $Id: faq.forms.php 
+// $Id$
 
 require_once('Pregunta.php');
 require_once('faq.functions.php');
@@ -73,4 +73,4 @@ function faq_form_desactivar_pregunta()
       </form>
 <?php
 }
-?>
+?>
\ No newline at end of file

diff --git a/src/lib/faq.functions.php b/src/lib/faq.functions.php
index b9168e78e0aeedd50bbd8d5e7658e3a03ea5801d..d33b3fe673137db6783b904188318aec9f5deae2 100644 (file)
--- a/src/lib/faq.functions.php
+++ b/src/lib/faq.functions.php
@@ -6,7 +6,7 @@
 //
 // Creado: Jonathan Schein ( 01/05/2005 )
 //
-// $Id: faq.functions.php
+// $Id$
 
 require_once 'Pregunta.php';
 require_once 'Respuesta.php';  
@@ -196,4 +196,4 @@ function Faq_DesactivarPregunta( $PreguntaId )
  fclose($fp);
  return "La pregunta fue dada de baja, no sera listada mas, pero conservada en un historico";   
 }
-?>
+?>
\ No newline at end of file

diff --git a/src/lib/pagina.php b/src/lib/pagina.php
index b0c278f944fbd83bf6ebde628bc58ec69113dd0b..8df27be7f809fbb785d60e9f9e11a1d2a112fdac 100644 (file)
--- a/src/lib/pagina.php
+++ b/src/lib/pagina.php
@@ -12,7 +12,6 @@ require_once 'lib/Usuario.php';
 
 // Arranca la sesión
 session_start();
-//TODO - verificar si existe el id y reconstruirlo
 
 function pagina_cabecera($titulo = '')
 {
@@ -54,16 +53,13 @@ function ok($msg)
     printf('<p style="color: green">%s</p>', $msg);
 }
 
-function pagina_check_login($Origen = '')
+function pagina_check_login($redirect = true)
 {
     if (!@$_SESSION['user'])
     {
-        // Guarda la pagina desde donde es llamado el login, para despues redireccionarse a esa
-                               if ( $Origen != '' )
-                               {
-                                $_SESSION['origen'] = $Origen;
-                               }
-                               include 'admin.login.php';
+        // Avisamos a donde hay que volver
+        if ($redirect) $_SESSION['login_redirect'] = basename($_SERVER['PHP_SELF']);
+        include 'admin.login.php'; // Mostramos el login
         exit;
     }
 }