Skip to content
Extraits de code Groupes Projets
Valider ad30c37b rédigé par cerdic's avatar cerdic
Parcourir les fichiers

rejoue_session n'a aucune raison d'etre une globale puisqu'on l'affecte une unique fois.

Autant utiliser une constante, c'est plus clair.

Par ailleurs, on peut se demander pourquoi le hit pour rejouer une session est en <img> statique + js pour inserer l'attribut src, et non directement en statique.
Historiquement, ça vient de r424 qui l'inserait sous forme de <script> puis de r427 qui l'a compliqué avec le montage img+js. Pourquoi ne pas simplifier par une simple img ?
parent 5183dabe
Aucune branche associée trouvée
Aucune étiquette associée trouvée
Aucune requête de fusion associée trouvée
...@@ -100,7 +100,8 @@ function html_tests_js(){ ...@@ -100,7 +100,8 @@ function html_tests_js(){
. generer_url_ecrire('test_ajax', 'js=-1') . generer_url_ecrire('test_ajax', 'js=-1')
. "' width='1' height='1' alt='' /></div></noscript>\n"); . "' width='1' height='1' alt='' /></div></noscript>\n");
} }
return $GLOBALS['rejoue_session'] return
(defined('_SESSION_REJOUER') ? _SESSION_REJOUER : '')
. (defined('_TESTER_NOSCRIPT') ? _TESTER_NOSCRIPT : ''); . (defined('_TESTER_NOSCRIPT') ? _TESTER_NOSCRIPT : '');
} }
......
...@@ -20,7 +20,6 @@ if (!defined('_ECRIRE_INC_VERSION')) return; ...@@ -20,7 +20,6 @@ if (!defined('_ECRIRE_INC_VERSION')) return;
*/ */
$GLOBALS['visiteur_session'] = ''; # globale decrivant l'auteur $GLOBALS['visiteur_session'] = ''; # globale decrivant l'auteur
$GLOBALS['rejoue_session'] = ''; # globale pour insertion de JS en fin de page
/** /**
* 3 actions sur les sessions, selon le type de l'argument: * 3 actions sur les sessions, selon le type de l'argument:
...@@ -197,7 +196,7 @@ function verifier_session($change=false) { ...@@ -197,7 +196,7 @@ function verifier_session($change=false) {
// sa victime, mais se ferait deconnecter par elle. // sa victime, mais se ferait deconnecter par elle.
if (hash_env() != $GLOBALS['visiteur_session']['hash_env']) { if (hash_env() != $GLOBALS['visiteur_session']['hash_env']) {
if (!$GLOBALS['visiteur_session']['ip_change']) { if (!$GLOBALS['visiteur_session']['ip_change']) {
$GLOBALS['rejoue_session'] = rejouer_session(); define('_SESSION_REJOUER',rejouer_session());
$GLOBALS['visiteur_session']['ip_change'] = true; $GLOBALS['visiteur_session']['ip_change'] = true;
ajouter_session($GLOBALS['visiteur_session']); ajouter_session($GLOBALS['visiteur_session']);
} else if ($change) { } else if ($change) {
...@@ -357,6 +356,9 @@ function fichier_session($alea, $tantpis=false) { ...@@ -357,6 +356,9 @@ function fichier_session($alea, $tantpis=false) {
/** /**
* Code a inserer par inc/presentation pour rejouer la session * Code a inserer par inc/presentation pour rejouer la session
* Voir action/cookie qui sera appele. * Voir action/cookie qui sera appele.
* Pourquoi insere-t-on le src par js et non directement en statique dans le HTML ?
* Historiquement, insere par une balise <script> en r424
* puis modifie par <img> statique + js en r427
* *
* http://doc.spip.org/@rejouer_session * http://doc.spip.org/@rejouer_session
* *
......
0% Chargement en cours ou .
You are about to add 0 people to the discussion. Proceed with caution.
Terminez d'abord l'édition de ce message.
Veuillez vous inscrire ou vous pour commenter