Changeset 113737 in spip-zone

Timestamp:

Feb 4, 2019, 4:55:03 PM (2 weeks ago)

Author:

root

Message:

fix fatal quand indexer_job_indexer_source est appele avec un nom d'objet au lieu du nom de table, ce qui se produit dans indexer_redindex_objet() quand on veut reindexer les articles lies a un mot ou un auteur
on couvre ici tous les cas d'appel de maniere plus robuste

Location:

_plugins_/indexer/trunk

Files:

• inc/indexer.php (modified) (1 diff)
• paquet.xml (modified) (1 diff)

_plugins_/indexer/trunk/inc/indexer.php

@@ -119 +119 @@
         $sources = indexer_sources();
         // On récupère celle demandée en paramètre
-        $source = $sources->getSource($alias_de_sources);
-        // On va chercher les documents à indexer
-        $documents = $source->getDocuments($start, $end);
-        // Et on le remplace (ou ajoute) dans l'indexation
-        $res = $indexer->replaceDocuments($documents);
-
-        // en cas d'erreur, on se reprogramme pour une autre fois
-        if (!$res) {
-                job_queue_add(
-                        'indexer_job_indexer_source',
-                        "Replan indexation $alias_de_sources $start $end",
-                        array($alias_de_sources, $start, $end, uniqid()),
-                        'inc/indexer',
-                        false, // duplication possible car ce job-ci n'est pas encore nettoyé
-                        time() + 15*60, // attendre 15 minutes
-                        -10 // priorite basse
-                );
+        // les alias de source correpondent en principe a la table (ex 'articles'), donc si $alias_de_sources est un objet ('article') on ne va pas trouver la source
+        if (!function_exists('table_objet')) {
+                include_spip('base/objets');
+        }
+        if ($source = $sources->getSource($alias_de_sources)
+          or $source = $sources->getSource(table_objet($alias_de_sources))) {
+                // On va chercher les documents à indexer
+                $documents = $source->getDocuments($start, $end);
+                // Et on le remplace (ou ajoute) dans l'indexation
+                $res = $indexer->replaceDocuments($documents);
+
+                // en cas d'erreur, on se reprogramme pour une autre fois
+                if (!$res) {
+                        job_queue_add(
+                                'indexer_job_indexer_source',
+                                "Replan indexation $alias_de_sources $start $end",
+                                array($alias_de_sources, $start, $end, uniqid()),
+                                'inc/indexer',
+                                false, // duplication possible car ce job-ci n'est pas encore nettoyé
+                                time() + 15*60, // attendre 15 minutes
+                                -10 // priorite basse
+                        );
+                }
+        }
+        else {
+                spip_log("indexer_job_indexer_source: Source $alias_de_sources inconnue", 'indexer'._LOG_ERREUR);
         }
 }

_plugins_/indexer/trunk/paquet.xml

@@ -2 +2 @@
         prefix="indexer"
         categorie="navigation"
-        version="2.16.3"
+        version="2.16.4"
         etat="test"
         compatibilite="[3.0.0;3.2.*]"