Changeset 107575 in spip-zone

Timestamp:

Nov 15, 2017, 3:16:34 PM (3 years ago)

Author:

rastapopoulos@…

Message:

Backport de [107574] : Amélioration de [107572] : comme proposé par cerdic, quand on trouve une jointure ET qu'on détecte que l'objet peut être classé avec polyhier, alors on cherche les deux liaisons avec un OR. Sinon c'est que l'un ou que l'autre.

Location:

_plugins_/polyhierarchie/branches/v2.0

Files:

• paquet.xml (modified) (1 diff)
• polyhier_fonctions.php (modified) (2 diffs)

_plugins_/polyhierarchie/branches/v2.0/paquet.xml

@@ -2 +2 @@
         prefix="polyhier"
         categorie="navigation"
-        version="2.3.7"
+        version="2.3.8"
         etat="stable"
         compatibilite="[3.0.0;3.2.*]"

_plugins_/polyhierarchie/branches/v2.0/polyhier_fonctions.php

@@ -206 +206 @@
                         }
                 }
+                // si le champ id_rubrique est recuperer par jointure, c'est le type et la primary de la table jointe
+                // qu'il faut chercher dans la table spip_rubriques_liens (ie cas des evenements)
+                if ($cle AND $desc) {
+                        $type_jointure = objet_type($boucle->from[$cle]);
+                        $primary_jointure = $cle . "." . id_table_objet($boucle->from[$cle]);
+                }
         }
         else $cle = $boucle->id_table;
@@ -216 +222 @@
         if ($tous!=='directs'
           AND in_array(table_objet_sql($boucle->type_requete),array_keys(lister_tables_objets_sql()))){
-                $sous = "sql_get_select('rl.id_objet','spip_rubriques_liens as rl',sql_in('rl.id_parent',\$b" . ($not ? ", 'NOT'" : '') . ").' AND rl.objet=\'$type\'')";
-                $where[] = "array('IN', '$primary', '(SELECT * FROM('.$sous.') AS subquery)')";
+                // S'il y a une jointure, on cherche toujours les liaisons avec celle-ci
+                if (isset($type_jointure)) {
+                        $sous_jointure = "sql_get_select('rl.id_objet','spip_rubriques_liens as rl',sql_in('rl.id_parent',\$in_rub" . ($not ? ", 'NOT'" : '') . ").' AND rl.objet=\'$type_jointure\'')";
+                        $where_jointure = "array('IN', '$primary_jointure', '(SELECT * FROM('.$sous_jointure.') AS subquery)')";
+                }
+                
+                // S'il n'y a pas de jointure (cas par défaut) ou que l'objet est explicitement configuré pour être classé avec polyhier
+                // on cherche les liaisons sur l'objet
+                if (
+                        !isset($type_jointure)
+                        or (include_spip('inc/config') and in_array(table_objet_sql($type), lire_config('polyhier/lier_objets', array())))
+                ) {
+                        $sous_objet = "sql_get_select('rl.id_objet','spip_rubriques_liens as rl',sql_in('rl.id_parent',\$in_rub" . ($not ? ", 'NOT'" : '') . ").' AND rl.objet=\'$type\'')";
+                        $where_objet = "array('IN', '$primary', '(SELECT * FROM('.$sous_objet.') AS subquery)')";
+                }
+                
+                // S'il y a les deux, on fait un OR
+                if (isset($where_jointure) and isset($where_objet)) {
+                        $where[] = "array('OR', $where_jointure, $where_objet)";
+                }
+                // Sinon s'il n'y a que jointure
+                elseif (isset($where_jointure)) {
+                        $where[] = $where_jointure;
+                }
+                // Sinon que sur l'objet
+                else {
+                        $where[] = $where_objet;
+                }
         }