From: Leandro Lucarella Date: Wed, 4 May 2005 05:22:58 +0000 (+0000) Subject: - Se arregla el redirect (al final queda usando variable de sesión porque si no X-Git-Tag: svn_import~107 X-Git-Url: https://git.llucax.com/z.facultad/75.43/tp1.git/commitdiff_plain/67a8494965128063251f7084fee7d8076d5f19e4?ds=sidebyside - 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 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; } }