Changeset 48068 in spip-zone

Timestamp:

May 24, 2011, 11:56:29 AM (10 years ago)

Author:

eric@…

Message:

On supprime deux passages par référence qui retournaient une erreur en spip 3 (je vois pas pourquoi d'ailleurs).

Location:

_plugins_/boussole

Files:

• base/boussole_installer.php (modified) (1 diff)
• formulaires/ajouter_boussole.php (modified) (2 diffs)
• genie/boussole_spip_actualiser.php (modified) (2 diffs)
• inc/deboussoler.php (modified) (4 diffs)

_plugins_/boussole/base/boussole_installer.php

@@ -21 +21 @@
                 // On ajoute la boussole SPIP par defaut.
                 // Cependant on ne teste ni la validite du fichier xml fourni ni la bonne insertion en bdd
-                $url = boussole_localiser_xml('', 'standard');
-                boussole_ajouter($url, $erreur);
+                $xml = 'http://zone.spip.org/trac/spip-zone/export/HEAD/_galaxie_/boussole.spip.org/boussole_spip.xml';
+                $url = boussole_localiser_xml($xml);
+                list($ok, $message) = boussole_ajouter($url);
         }
 }

_plugins_/boussole/formulaires/ajouter_boussole.php

@@ -32 +32 @@
         $xml = _request('url_boussole');
 
+        // Cas de la boussole SPIP
+        if ($mode == 'standard')
+                $xml = url_absolue('http://zone.spip.org/trac/spip-zone/export/HEAD/_galaxie_/boussole.spip.org/boussole_spip.xml');
+
         // On fait des verifications dans traiter pour renvoyer les resultats dans le message d'erreur global
-        if (!$url = boussole_localiser_xml($xml, $mode)) {
+        if (!$url = boussole_localiser_xml($xml)) {
                 // Le fichier est introuvable
                 $retour['message_erreur'] = _T('boussole:message_nok_xml_introuvable', array('fichier' => $xml));
@@ -47 +51 @@
                         // et on traite le cas d'erreur fichier ($retour['message_erreur']) non conforme
                         // si c'est encore possible apres avoir valide le fichier avec la dtd
-                        $ok = boussole_ajouter($url, $message);
+                        list($ok, $message) = boussole_ajouter($url);
                 
                         // Determination des messages de retour

_plugins_/boussole/genie/boussole_spip_actualiser.php

@@ -11 +11 @@
         if (lire_meta($meta_boussole)) {
                 // On appelle donc la fonction d'actualisation
-                if (!$url = boussole_localiser_xml('', 'standard')) {
+                $xml = 'http://zone.spip.org/trac/spip-zone/export/HEAD/_galaxie_/boussole.spip.org/boussole_spip.xml';
+                if (!$url = boussole_localiser_xml($xml)) {
                         // Le fichier est introuvable
                         spip_log("ERREUR ACTUALISATION CRON : fichier xml introuvable", 'boussole');
@@ -24 +25 @@
                                 // et on traite le cas d'erreur fichier ($retour['message_erreur']) non conforme
                                 // si c'est encore possible apres avoir valide le fichier avec la dtd
-                                $ok = boussole_ajouter($url, $message);
+                                list($ok, $message) = boussole_ajouter($url);
                         
                                 // Determination des messages de retour

_plugins_/boussole/inc/deboussoler.php

@@ -16 +16 @@
 // $url => url ou path du fichier xml de description de la boussole
 // $erreur      => message d'erreur deja traduit
-function boussole_ajouter($url, &$message='') {
-
+function boussole_ajouter($url) {
+
+        // On initialise le message de sortie
+        $message = '';
+        
         // On recupere les infos du fichier xml de description de la balise
         $infos = boussole_parser_xml($url);
         if (!infos OR !$infos['boussole']['alias']){
                 $message = _T('boussole:message_nok_xml_invalide', array('fichier' => $url));
-                return false;
+                return array(false, $message);
         }
 
@@ -58 +61 @@
         if (!$ids = sql_insertq_multi('spip_boussoles', $infos['sites'])) {
                 $message = _T('boussole:message_nok_ecriture_bdd');
-                return false;
+                return array(false, $message);
         }
         // -- consignation des informations de mise a jour de cette boussole dans la table spip_meta
@@ -71 +74 @@
                 $message = _T('boussole:message_ok_boussole_ajoutee', array('fichier' => $url));
         
-        return true;
+        return array(true, $message);
 }
 
@@ -105 +108 @@
  * @return string
  */
-function boussole_localiser_xml(&$xml, $mode) {
+function boussole_localiser_xml($xml) {
 
         include_spip('inc/distant');
         $retour = '';
-
-        // La boussole SPIP
-        if ($mode == 'standard')
-                $xml = url_absolue('http://zone.spip.org/trac/spip-zone/export/HEAD/_galaxie_/boussole.spip.org/boussole_spip.xml');
 
         // On calcul une url absolue dans tous les cas