From 67a8494965128063251f7084fee7d8076d5f19e4 Mon Sep 17 00:00:00 2001 From: Leandro Lucarella Date: Wed, 4 May 2005 05:22:58 +0000 Subject: [PATCH] =?utf8?q?-=20Se=20arregla=20el=20redirect=20(al=20final?= =?utf8?q?=20queda=20usando=20variable=20de=20sesi=C3=B3n=20porque=20si=20?= =?utf8?q?no=20=20=20hab=C3=ADa=20que=20arrastrarlo=20por=20todos=20lados,?= =?utf8?q?=20pero=20se=20borra=20cuando=20no=20se=20necesita=20=20=20m?= =?utf8?q?=C3=A1s).=20Ahora=20no=20hace=20falta=20pasarle=20el=20origen=20?= =?utf8?q?a=20pagina=5Fcheck=5Flogin(),=20lo=20=20=20averigua=20solo.=20-?= =?utf8?q?=20Algunos=20bugfixes=20menores=20(p=C3=A1ginas=20a=20las=20que?= =?utf8?q?=20le=20faltaba=20el=20include=20de=20=20=20lib/pagina.php=20y?= =?utf8?q?=20hac=C3=ADa=20que=20falle=20el=20logout).=20-=20Se=20pone=20bi?= =?utf8?q?en=20el=20$Id$=20de=20los=20archivos.?= MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit --- src/admin.login.php | 22 ++++++++++------------ src/admin.logout.php | 2 ++ src/faq.desactivarPregunta.php | 3 +-- src/faq.ingresarPregunta.php | 4 ++-- src/faq.listadoPreguntas.php | 6 +++--- src/faq.responderPregunta.php | 4 ++-- src/index.php | 1 + src/lib/faq.forms.php | 4 ++-- src/lib/faq.functions.php | 4 ++-- src/lib/pagina.php | 12 ++++-------- 10 files changed, 29 insertions(+), 33 deletions(-) diff --git a/src/admin.login.php b/src/admin.login.php index 55f505c..e8f24d5 100644 --- 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 dc7caf8..9376e2a 100644 --- 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 a2bc867..677286c 100644 --- 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 310f2e0..4159025 100644 --- 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 8a8d195..1a99987 100644 --- 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 e27b163..a89c39f 100644 --- 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 a98f50b..9d47441 100644 --- 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 c018dba..0784dcb 100644 --- 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() +?> \ No newline at end of file diff --git a/src/lib/faq.functions.php b/src/lib/faq.functions.php index b9168e7..d33b3fe 100644 --- 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 b0c278f..8df27be 100644 --- 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('

%s

', $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; } } -- 2.43.0