Changeset 32222 in spip-zone

Timestamp:

Oct 19, 2009, 3:21:40 PM (11 years ago)

Author:

marcimat@…

Message:

Corriger le bug de non installation des Champs Extras de plugin lorsqu'on active les 2 plugins en même temps (CE2 et le plugin dépendant).

Location:

_plugins_/champs_extras2/core

Files:

• base/cextras.php (modified) (2 diffs)
• inc/cextras_gerer.php (modified) (3 diffs)
• plugin.xml (modified) (1 diff)

_plugins_/champs_extras2/core/base/cextras.php

@@ -3 +3 @@
 
 function cextras_declarer_tables_principales($tables_principales){
+        // declarer que le pipeline de declaration est bien actif
+        define('_CHAMPS_EXTRAS_DECLARES', true);
+        
         // pouvoir utiliser la class ChampExtra
         include_spip('inc/cextras');
-        
+
         // lors du renouvellement de l'alea, au demarrage de SPIP
         // les chemins de plugins ne sont pas encore connus.
@@ -15 +18 @@
         // recuperer les champs crees par les plugins
         $champs = pipeline('declarer_champs_extras', array());
+
         // ajouter les champs au tableau spip
         return declarer_champs_extras($champs, $tables_principales);

_plugins_/champs_extras2/core/inc/cextras_gerer.php

@@ -73 +73 @@
  */
 function creer_champs_extras($champs) {
+        if (!$champs) {
+                return;
+        }
+        
         if (!is_array($champs)) 
                 $champs = array($champs);
-                
+                                
         // on recupere juste les differentes tables a mettre a jour
         $tables = array();
@@ -83 +87 @@
                 }
         }       
-                
+
         // on met a jour les tables trouvees
-        if ($tables) {  
+        if ($tables) {
                 // recharger les tables principales et auxiliaires
                 include_spip('base/serial');
@@ -91 +95 @@
                 global $tables_principales, $tables_auxiliaires;
                 base_serial($tables_principales);
-                base_auxiliaires($tables_auxiliaires);          
+                base_auxiliaires($tables_auxiliaires);
                 
+                // inclure les champs extras declares ALORS que le pipeline
+                // n'est pas encore actif : important lorsqu'on active
+                // en meme temps CE2 et un plugin dependant
+                // et non l'un apres l'autre
+                if (!defined('_CHAMPS_EXTRAS_DECLARES')) {
+                        include_spip('base/cextras');
+                        $tables_principales = cextras_declarer_tables_principales($tables_principales);
+                }
+
                 // executer la mise a jour
                 include_spip('base/create');
-                foreach($tables as $table) {
-                        maj_tables($table);
-                }
+                maj_tables($tables);
         }
 }

_plugins_/champs_extras2/core/plugin.xml

@@ -3 +3 @@
         <auteur>Matthieu Marcillaud, Fil</auteur>
         <licence>GNU/GPL</licence>
-        <version>1.2.0</version>
+        <version>1.3.0</version>
         <description>
         Creer de nouveaux champs aux objets d'&eacute;dition de SPIP