Changeset 82681 in spip-zone

Timestamp:

May 22, 2014, 4:58:28 PM (5 years ago)

Author:

rastapopoulos@…

Message:

Tout comme pour indexer_sources(), on déporte la récupération d'un indexeur valide dans une fonction indexer_indexer() utilisable plusieurs fois.

On s'insère dans le pipeline "post_edition", et dès qu'il y a une modif sur un objet, on cherche si on a une ou plusieurs de nos sources enregistrées qui correspondent à l'objet en cours d'édition.

Si c'est le cas, on appelle l'indexeur et on lui donne à remplacer (ou ajouter) le document issu de la source en question (ou des sources).

Sans rien toucher, le mécanisme teste avec l'alias donné à la source ("articles", "fiches", etc) MAIS si on veut un truc plus bizarre, ou surtout si on a plusieurs sources différentes travaillant sur le même type d'objet, alors la classe de source peut implémenter une méthode getObjet() qui doit retourner "patate" (par exemple).

Testé aussi sur un objet non-spip, pour l'instant ça marche.

Là on réindexe le doc directement dans le pipeline, et ça ne me parait pas lent. Si on trouve que ça gêne, on pourra déporter l'indexation dans un job. Mais pour l'instant ça m'a l'air d'aller.

Location:

_plugins_/indexer/trunk

Files:

• inc/indexer.php (modified) (3 diffs)
• indexer_pipelines.php (added)
• paquet.xml (modified) (2 diffs)
• plugin.xml (modified) (1 diff)
• prive/squelettes/contenu/indexer.html (modified) (1 diff)

_plugins_/indexer/trunk/inc/indexer.php

@@ -18 +18 @@
 $loader->addPsr4('Spip\\Indexer\\Sources\\',  _DIR_PLUGIN_INDEXER . 'Sources');
 
+$loader->register();
 
-$loader->register();
+/**
+ * Renvoyer un indexeur configuré avec un (et peut-être un jour plusieurs) lieu de stockage des choses à indexer
+ *
+ * Par défaut renvoie l'indexeur avec le stockage Sphinx intégré en interne et les paramètres des define()
+ * 
+ * @pipeline_appel indexer_indexer
+ * @return Retourne un objet Indexer, ayant été configuré avec la méthode registerStorage()
+ */
+function indexer_indexer(){
+        static $indexer = null;
+        
+        if (is_null($indexer)){
+                // On crée un indexeur
+                $indexer = new Indexer\Indexer();
+        
+                // On tente de le configurer avec Sphinx et les define()
+                try {
+                        $indexer->registerStorage(
+                            new Indexer\Storage\Sphinx(
+                                new Sphinx\SphinxQL\SphinxQL(SPHINX_SERVER_HOST, SPHINX_SERVER_PORT), SPHINX_DEFAULT_INDEX)
+                        );
+                } catch( \Exception $e ) {
+                        if (!$message = $e->getMessage())
+                            $message = _L('Erreur inconnue');
+                        die("<p class='erreur'>$message</p>");
+                }
+                
+                // On le fait passer dans un pipeline
+                $indexer = pipeline('indexer_indexer', $indexer);
+        }
+        
+        return $indexer;
+}
 
 /**
@@ -26 +59 @@
  * Un pipeline "indexer_sources" est appelée avec la liste par défaut, permettant de retirer ou d'ajouter des sources.
  *
- * @pipeline_appel insexer_sources
+ * @pipeline_appel indexer_sources
  * @return Sources Retourne un objet Sources listant les sources enregistrées avec la méthode register()
  */
@@ -33 +66 @@
         
         if (is_null($sources)){
+                // On crée la liste des sources
                 $sources = new Indexer\Sources\Sources();
+                // Par défaut on enregistre les articles du SPIP
                 $sources->register('articles', new Spip\Indexer\Sources\Articles());
+                // On passe les sources dans un pipeline
                 $sources = pipeline('indexer_sources', $sources);
         }

_plugins_/indexer/trunk/paquet.xml

@@ -8 +8 @@
         documentation=""
 >
-
         <nom>Indexer</nom>
 
@@ -16 +15 @@
 
         <licence>GNU/GPL</licence>
-
+        
+        <pipeline nom="post_edition" inclure="indexer_pipelines.php" />
 </paquet>

_plugins_/indexer/trunk/plugin.xml

@@ -17 +17 @@
         <necessite id="SPIP" version="[2.1.0;3.1.99]" />
         <categorie>navigation</categorie>
+        
+        <pipeline>
+        <nom>post_edition</nom>
+        <inclure>indexer_pipelines.php</inclure>
+    </pipeline>
 </plugin>
 

_plugins_/indexer/trunk/prive/squelettes/contenu/indexer.html

@@ -5 +5 @@
 if (_request('run') || _request('continue')) {
     include_spip('inc/indexer');
-    $indexer = new Indexer\Indexer();
-
-    try {
-        $indexer->registerStorage(
-            new Indexer\Storage\Sphinx(
-                new Sphinx\SphinxQL\SphinxQL(SPHINX_SERVER_HOST, SPHINX_SERVER_PORT), SPHINX_DEFAULT_INDEX)
-        );
-    } catch( \Exception $e ) {
-        if (!$message = $e->getMessage())
-            $message = _L('Erreur inconnue');
-        die("<p class='erreur'>$message</p>");
-    }
+    
+    // Appeler la fonction qui donne l'indexeur configuré pour ce SPIP
+    $indexer = indexer_indexer();
         
         // Appeler la fonction qui liste les sources et qui comporte un pipeline pour étendre