feat(Solr): Highlightings keep fields name to allow to display title …

…with haighlighted parts, not only collection_txt BREAKING CHANGE: Solr search with hightlightings will output `highlighting` object with different labelled keys (`title_txt_en`, `collection_txt_en`) according to search `locale`.
roadiz · Sep 25, 2023 · 3ae0379 · 3ae0379
1 parent 75bb3d3
commit 3ae0379
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 16 deletions.
diff --git a/lib/RoadizCoreBundle/src/SearchEngine/AbstractSearchHandler.php b/lib/RoadizCoreBundle/src/SearchEngine/AbstractSearchHandler.php
@@ -98,7 +98,7 @@ protected function getHighlightingOptions(array &$args = []): array
     {
         $tmp = [];
         $tmp["hl"] = true;
-        $tmp["hl.fl"] = $this->getCollectionField($args);
+        $tmp["hl.fl"] = $this->getTitleField($args) . ' ' . $this->getCollectionField($args);
         $tmp["hl.fragsize"] = $this->getHighlightingFragmentSize();
         $tmp["hl.simple.pre"] = '<span class="solr-highlight">';
         $tmp["hl.simple.post"] = '</span>';

diff --git a/lib/RoadizCoreBundle/src/SearchEngine/SolrSearchResults.php b/lib/RoadizCoreBundle/src/SearchEngine/SolrSearchResults.php
@@ -104,24 +104,18 @@ function ($item) {
     }
 
     /**
-     * Merge collection_txt localized fields.
+     * Get highlighting for one field.
+     * This do not merge highlighting for all fields anymore.
      *
      * @param string $id
-     * @return array|array[]|mixed
+     * @return array<string, array>
      */
-    protected function getHighlighting(string $id): mixed
+    protected function getHighlighting(string $id): array
     {
-        $highlights = $this->response['highlighting'][$id];
-        if (!isset($highlights['collection_txt'])) {
-            $collectionTxt = [];
-            foreach ($highlights as $field => $value) {
-                $collectionTxt = array_merge($collectionTxt, $value);
-            }
-            $highlights = array_merge($highlights, [
-                'collection_txt' => $collectionTxt
-            ]);
+        if (isset($this->response['highlighting'][$id]) && \is_array($this->response['highlighting'][$id])) {
+            return $this->response['highlighting'][$id];
         }
-        return $highlights;
+        return [];
     }
 
     /**

diff --git a/...izCoreBundle/src/SearchEngine/Subscriber/DefaultDocumentTranslationIndexingSubscriber.php b/...izCoreBundle/src/SearchEngine/Subscriber/DefaultDocumentTranslationIndexingSubscriber.php
@@ -112,7 +112,7 @@ public function onIndexing(DocumentTranslationIndexingEvent $event): void
          */
         $assoc['collection_txt'] = $collection;
         // Compile all text content into a single localized text field.
-        $assoc['collection_txt_' . $lang] = implode(PHP_EOL, $collection);
+        $assoc['collection_txt_' . $lang] = trim(implode(PHP_EOL, array_filter($collection)));
         $event->setAssociations($assoc);
     }
 }
diff --git a/lib/RoadizCoreBundle/src/SearchEngine/Subscriber/DefaultNodesSourcesIndexingSubscriber.php b/lib/RoadizCoreBundle/src/SearchEngine/Subscriber/DefaultNodesSourcesIndexingSubscriber.php
@@ -152,7 +152,7 @@ function (Tag $tag) {
          */
         $assoc['collection_txt'] = $collection;
         // Compile all text content into a single localized text field.
-        $assoc['collection_txt_' . $lang] = implode(PHP_EOL, $collection);
+        $assoc['collection_txt_' . $lang] = trim(implode(PHP_EOL, array_filter($collection)));
         $event->setAssociations($assoc);
     }