Changeset 44052 in spip-zone

Timestamp:

Jan 31, 2011, 11:43:22 PM (9 years ago)

Author:

joseph@…

Message:

Autant pour moi, il ne faut lever l'ambiguité que s'il s'agit d'un champ de la table principal, afin que que le critère tri soit compatible avec compteur_objet, somme, etc.

Location:

_plugins_/spip-bonux-2

Files:

• plugin.xml (modified) (1 diff)
• public/spip_bonux_criteres.php (modified) (1 diff)
• spip_bonux_fonctions.php (modified) (2 diffs)

_plugins_/spip-bonux-2/plugin.xml

@@ -32 +32 @@
   <icon>img_pack/spip-bonux.png</icon>
   <licence>(c) 2008-2010 GPL</licence>
-  <version>2.2.16</version>
+  <version>2.2.17</version>
   <etat>stable</etat>
   <description>Le plugin qui lave plus SPIP que SPIP, avec des chouettes cadeaux dedans ! Interface douce, boucles POUR et CONDITION, tableaux, compteur, etc.</description>

_plugins_/spip-bonux-2/public/spip_bonux_criteres.php

@@ -200 +200 @@
         };
         ";
+        $field = serialize(array_keys($boucle->show['field']));
         $boucle->select[] = "\".tri_champ_select(\$tri).\"";
-        $boucle->order[] = "tri_champ_order(\$tri,'$id_table').\$senstri";
+        $boucle->order[] = "tri_champ_order(\$tri,'$id_table','$field').\$senstri";
 }
 

_plugins_/spip-bonux-2/spip_bonux_fonctions.php

@@ -193 +193 @@
  * @return string
  */
-function tri_champ_order($t,$table=NULL){
-        if (!is_null($table))
-                $table = $table.'.';
+function tri_champ_order($t,$table=NULL,$field=NULL){
         if (strncmp($t,'num ',4)==0){
                 $t = substr($t,4);
                 $t = preg_replace(',\s,','',$t);
-                $t = "0+$table$t";
+                // Lever une ambiguité possible si le champs fait partie de la table (pour compatibilité de la balise tri avec compteur, somme, etc.)
+                if (!is_null($table) && !is_null($field) && in_array($t,unserialize($field)))
+                        $t = "0+$table.$t";
+                else
+                        $t = "0+$t";
                 return $t;
         }
@@ -205 +207 @@
                 return "multi";
         }
-        return $table.preg_replace(',\s,','',$t);
+        else {
+                $t = preg_replace(',\s,','',$t);
+                // Lever une ambiguité possible si le champs fait partie de la table (pour compatibilité de la balise tri avec compteur, somme, etc.)
+                if (!is_null($table) && !is_null($field) && in_array($t,unserialize($field)))
+                        return $table.'.'.$t;
+                else
+                        return $t;
+        }
 }