Changeset 107574 in spip-zone

Timestamp:

Nov 15, 2017, 3:10:11 PM (16 months ago)

Author:

rastapopoulos@…

Message:

Amélioration de [107571] : 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/trunk

Files:

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

_plugins_/polyhierarchie/trunk/paquet.xml

@@ -2 +2 @@
         prefix="polyhier"
         categorie="navigation"
-        version="3.6.8"
+        version="3.6.9"
         etat="test"
         compatibilite="[3.0.0;3.2.*]"

_plugins_/polyhierarchie/trunk/polyhier_fonctions.php

@@ -218 +218 @@
                         }
                 }
+                // 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 {
@@ -245 +251 @@
         // Si c'est tout ou que indirects, on ajoute le critère de branche secondaire, avec la table de liens
         if ($tous !== 'directs') {
+                // 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)')";
+                }
                 
-                $sous = "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[] = "array('IN', '$primary', '(SELECT * FROM('.$sous.') 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;
+                }
         }