Changeset 81643 in spip-zone

Timestamp:

Mar 28, 2014, 11:12:28 AM (5 years ago)

Author:

cedric@…

Message:

BugFix? sur les collisions d'URLs propres/arbo qui continuait même après être revenu en URL propre car le décodage des anciennes URLs arbo génére de nouvelles URLs arbos :

Quand on decode une url arbo qui est en fait une ancienne URL car on a changé de schema d'URL, il ne faut pas regenerer de nouveaux segments arbo mais rediriger directement vers l'URL dans le nouveau schema.
Pour cela on se base sur le nom de la fonction qui appelle urls_arbo_dist : si c'est urls_xx et pas urls_decoder_url : c'est un appel de compatibilité depuis un autre schema.

Location:

_core_/branches/spip-3.0/plugins/urls_etendues

Files:

• paquet.xml (modified) (1 diff)
• urls/arbo.php (modified) (1 diff)

_core_/branches/spip-3.0/plugins/urls_etendues/paquet.xml

@@ -2 +2 @@
         prefix="urls"
         categorie="statistique"
-        version="1.4.17"
+        version="1.4.18"
         etat="stable"
         compatibilite="[3.0.0;3.0.*]"

_core_/branches/spip-3.0/plugins/urls_etendues/urls/arbo.php

@@ -597 +597 @@
 
                 if (count($url_arbo_new)){
-                        foreach($url_arbo_new as $k=>$o)
-                                if ($s = declarer_url_arbo($o['objet'],$o['id_objet']))
-                                        $url_arbo_new[$k] = $s;
-                                else
-                                        $url_arbo_new[$k] = implode('/',$o['segment']);
-                        $url_arbo_new = ltrim(implode('/',$url_arbo_new),'/');
-                        
-                        if ($url_arbo_new!==$url_propre){
-                                $url_redirect = $url_arbo_new;
+                        $caller = debug_backtrace();
+                        $caller = $caller[1]['function'];
+                        // si on est appele par un autre module d'url c'est du decodage d'une ancienne URL
+                        // ne pas regenerer des segments arbo, mais rediriger vers la nouvelle URL
+                        // dans la nouvelle forme
+                        if (strncmp($caller,"urls_",5)==0 AND $caller!=="urls_decoder_url"){
                                 // en absolue, car assembler ne gere pas ce cas particulier
                                 include_spip('inc/filtres_mini');
-                                $url_redirect = url_absolue($url_redirect);
+                                $col_id = id_table_objet($entite);
+                                $url_new = generer_url_entite($contexte[$col_id],$entite);
+                                // securite contre redirection infinie
+                                if ($url_new!==$url_propre)
+                                        $url_redirect = url_absolue($url_new);
+                        }
+                        else {
+                                foreach($url_arbo_new as $k=>$o)
+                                        if ($s = declarer_url_arbo($o['objet'],$o['id_objet']))
+                                                $url_arbo_new[$k] = $s;
+                                        else
+                                                $url_arbo_new[$k] = implode('/',$o['segment']);
+                                $url_arbo_new = ltrim(implode('/',$url_arbo_new),'/');
+
+                                if ($url_arbo_new!==$url_propre){
+                                        $url_redirect = $url_arbo_new;
+                                        // en absolue, car assembler ne gere pas ce cas particulier
+                                        include_spip('inc/filtres_mini');
+                                        $url_redirect = url_absolue($url_redirect);
+                                }
                         }
                 }