Voici le ?bonjour=oui sans faille : l'idée est que, si on constate un echec de...

Voici le ?bonjour=oui sans faille : l'idée est que, si on constate un echec de login malgré le bonjour=oui, on envoie sur spip_cookie, qui pose un cookie et renvoit vers spip_login, lequel teste le cookie qui vient d'être posé. S'il est là, c'est qu'il n'y a pas d'échec de cookie (on vient d'un bookmark), s'il n'est pas là, c'est avéré, et on propose l'auth_http.

+ correction d'un bug de redirection en auth_http.

(et deux accents)

ecrire/inc_auth.php3

@@ -114,10 +114,10 @@ function auth() {
 
 	// Tentative de login echec
 	else if ($GLOBALS['bonjour'] == 'oui') { 
-		if ($GLOBALS['essai_cookie'] == 'oui')
-			@header("Location: ../spip_login.php3?var_echec_cookie=oui");
-		else
-			@header("Location: ../spip_login.php3");
+		$link = new Link("../spip_cookie.php3?test_echec_cookie=oui");
+		$clean_link->delVar('bonjour');
+		$link->addVar('url', 'ecrire/'.$clean_link->getUrl());
+		@header("Location: ".$link->getUrl());
 		exit;
 	}
 

ecrire/statistiques_visites.php3

@@ -359,11 +359,11 @@ if (count($log)>0){
 		echo "<b>total : $total_absolu</b>";
 		
 		if ($id_article) {
-			echo "<br>popularit&nbsp;: $val_popularite%";
+			echo "<br>popularit&eacute;&nbsp;: $val_popularite%";
 		} else {
 			echo "<font size=1>";
-			echo "<br>popularit du site&nbsp;: ";
-			echo round(max(1,lire_meta('popularite_total')));
+			echo "<br>popularit&eacute; du site&nbsp;: ";
+			echo ceil(lire_meta('popularite_total'));
 			echo "</font>";
 		}
 		

inc-login.php3

@@ -55,10 +55,16 @@ function ferme_login() {
 function login($cible = '', $prive = 'prive', $message_login='') {
 	$login = $GLOBALS['var_login'];
 	$erreur = $GLOBALS['var_erreur'];
-	$echec_cookie = $GLOBALS['var_echec_cookie'];
 	$essai_auth_http = $GLOBALS['var_essai_auth_http'];
 	$logout = $GLOBALS['var_logout'];
 
+	// en cas d'echec de cookie, inc_auth a renvoye vers spip_cookie qui
+	// a tente de poser un cookie ; s'il n'est pas la, c'est echec cookie
+	// s'il est la, c'est probablement un bookmark sur bonjour=oui,
+	// et pas un echec cookie.
+	if ($GLOBALS['var_echec_cookie'])
+		$echec_cookie = ($GLOBALS['spip_session'] != 'test_echec_cookie');
+
 	global $auteur_session;
 	global $spip_session, $PHP_AUTH_USER;
 	global $spip_admin;
@@ -200,7 +206,7 @@ function login($cible = '', $prive = 'prive', $message_login='') {
 		echo "de connexion (moins sécurisée) est à votre disposition : \n";
 		echo "<input type='hidden' name='essai_auth_http' value='oui'> ";
 		$url = $cible->getUrl();
-		echo "<input type='hidden' name='var_url' value='$url'>\n";
+		echo "<input type='hidden' name='url' value='$url'>\n";
 		echo "<div align='right'><input type='submit' name='submit' class='spip_bouton' value='Identification sans cookie'></div>\n";
 		echo "</fieldset></form>\n";
 	}

spip_cookie.php3

@@ -56,6 +56,17 @@ if ($logout) {
 }
 
 
+// en cas de login sur bonjour=oui, on tente de poser un cookie
+// puis de passer a spip_login qui diagnostiquera l'echec de cookie
+// le cas echeant.
+if ($test_echec_cookie == 'oui') {
+	setcookie('spip_session', 'test_echec_cookie');
+	$link = new Link("spip_login.php3?var_echec_cookie=oui");
+	$link->addVar("var_url", $cible->getUrl());
+	@header("Location: ".$link->getUrl());
+	exit;
+}
+
 // tentative de login
 unset ($cookie_session);
 if ($essai_login == "oui") {
@@ -102,7 +113,6 @@ if ($essai_login == "oui") {
 		@spip_query($query);
 		if (ereg("ecrire/", $cible->getUrl())) {
 			$cible->addVar('bonjour','oui');
-			$cible->addVar('essai_cookie','oui');
 		}
 	}
 	else {