bug tri par défaut sur critère IN() (et hierarchie) (Paolo)

+ fichier image mal adressé

ecrire/auteur_infos.php3

@@ -435,7 +435,7 @@ if ($connect_statut == "0minirezo"
 	if (($statut == '6forum') OR (lire_meta('accepter_visiteurs') == 'oui') OR (lire_meta('forums_publics') == 'abo'))
 		echo "<OPTION".mySel("6forum",$statut).">"._T('item_visiteur');
 	echo "<OPTION".mySel("5poubelle",$statut).
-	  " style='background:url(" . _DIR_RESTREINT . "rayures-sup.gif)'>&gt; "._T('texte_statut_poubelle');
+	  " style='background:url(" . _DIR_IMG_PACK . "rayures-sup.gif)'>&gt; "._T('texte_statut_poubelle');
 
 	echo "</SELECT></center>\n";
 	fin_cadre_relief();

inc-boucles.php3

@@ -190,7 +190,7 @@ function boucle_HIERARCHIE_dist($id_boucle, &$boucles) {
 	// $hierarchie sera calculee par une fonction de inc-calcul-outils
 	$boucle->where[] = 'id_rubrique IN ($hierarchie)';
 	$boucle->select[] = 'FIND_IN_SET(id_rubrique, \'$hierarchie\')-1 AS rang';
-	$boucle->order = 'rang';
+	$boucle->default_order = 'rang';
 	$boucle->hierarchie = '$hierarchie = calculer_hierarchie('
 	. calculer_argument_precedent($boucle->id_boucle, 'id_rubrique', $boucles)
 	. ', '

inc-compilo-api.php3

@@ -43,6 +43,7 @@ class Boucle {
 	var $limit;
 	var $group = '';
 	var $order = '';
+	var $default_order = '';
 	var $date = 'date' ;
 	var $hash = false ;
 	var $lien = false;

inc-compilo.php3

@@ -262,14 +262,15 @@ function calculer_boucle($id_boucle, &$boucles) {
 
 	// hack critere recherche : ignorer la requete en cas de hash vide
 	// Recherche : recuperer les hash a partir de la chaine de recherche
-	if ($boucle->hash) {
+	if ($boucle->hash)
 		$init =  '
 	// RECHERCHE
 	list($rech_select, $rech_where) = prepare_recherche($GLOBALS["recherche"], "'.$boucle->primary.'", "'.$boucle->id_table.'");
 	if ($rech_select) ';
-	}
 
-	else $init = '';
+	if (!$order = $boucle->order
+	AND !$order = $boucle->default_order)
+		$order = "''";
 
 	$init .= "\n\n	// REQUETE
 	\$result = spip_abstract_select(\n\t\tarray(\"". 
@@ -288,7 +289,7 @@ function calculer_boucle($id_boucle, &$boucles) {
 		"', $boucle->where) . '"')) .
 		"), # WHERE
 		'".addslashes($boucle->group)."', # GROUP
-		" . ($boucle->order ? $boucle->order : "''") .", # ORDER
+		" . $order .", # ORDER
 		" . (strpos($boucle->limit, 'intval') === false ?
 			"'".$boucle->limit."'" :
 			$boucle->limit). ", # LIMIT

inc-criteres.php3

@@ -265,7 +265,6 @@ function critere_par_dist($idb, &$boucles, $param, $not) {
 		if ($order) {
 		  if ($boucle->order) $boucle->order .= '.",".';
 		  $boucle->order .= $order;
-
 		}
 	}
 }
@@ -517,11 +516,9 @@ function calculer_critere_DEFAUT($idb, &$boucles, $param, $not) {
 				if ($match[4] == '!') {
 					$where = "NOT ($where)";
 				} else {
-					if (!$boucle->order) {
-						$boucle->order = 'rang';
-						$boucle->select[] =
+					$boucle->default_order = 'rang';
+					$boucle->select[] =
 						"FIND_IN_SET($col, \\\"$val\\\") AS rang";
-					}
 				}
 			} else {
 				if ($op == '==') $op = 'REGEXP';