[TASK] Improve output to something useful

- Adds summary from the fulltext using highlighting - Adds pagination of results - Adds breadcrumb for each result - Wraps output in container for styling This change has originally been based upon mocdk@c326730.
Flowpack · Mar 20, 2015 · 8c832c4 · 8c832c4
1 parent 7167c20
commit 8c832c4
Show file tree

Hide file tree

Showing 7 changed files with 87 additions and 28 deletions.
diff --git a/Classes/Flowpack/SearchPlugin/EelHelper/SearchArrayHelper.php b/Classes/Flowpack/SearchPlugin/EelHelper/SearchArrayHelper.php
@@ -0,0 +1,47 @@
+<?php
+namespace Flowpack\SearchPlugin\EelHelper;
+
+/*                                                                                                  *
+ * This script belongs to the TYPO3 Flow package "Flowpack.SearchPlugin".                           *
+ *                                                                                                  *
+ * It is free software; you can redistribute it and/or modify it under                              *
+ * the terms of the GNU Lesser General Public License, either version 3                             *
+ *  of the License, or (at your option) any later version.                                          *
+ *                                                                                                  *
+ * The TYPO3 project - inspiring people to share!                                                   *
+ *                                                                                                  */
+use TYPO3\Flow\Annotations as Flow;
+use TYPO3\Eel\ProtectedContextAwareInterface;
+
+/**
+ * Additional Array Helpers which might once
+ *
+ * @Flow\Proxy(false)
+ */
+class SearchArrayHelper implements ProtectedContextAwareInterface {
+
+	/**
+	 * Concatenate arrays or values to a new array
+	 *
+	 * @param array|mixed $array1 First array or value
+	 * @param array|mixed $array2 Second array or value
+	 * @param array|mixed $array_ Optional variable list of additional arrays / values
+	 * @return array The array with concatenated arrays or values
+	 */
+	public function flatten($arrays) {
+		$return = array();
+		array_walk_recursive($arrays, function($a) use (&$return) { $return[] = $a; });
+		return $return;
+	}
+
+	/**
+	 * All methods are considered safe
+	 *
+	 * @param string $methodName
+	 * @return boolean
+	 */
+	public function allowsCallOfMethod($methodName) {
+		return TRUE;
+	}
+
+}
diff --git a/Classes/Flowpack/SearchPlugin/TypoScriptObjects/CanRenderImplementation.php b/Classes/Flowpack/SearchPlugin/TypoScriptObjects/CanRenderImplementation.php
@@ -3,7 +3,7 @@
 
 
 /*                                                                                                  *
- * This script belongs to the TYPO3 Flow package "Flowpack.SearchPlugin.TypoScriptObjects".         *
+ * This script belongs to the TYPO3 Flow package "Flowpack.SearchPlugin".                           *
  *                                                                                                  *
  * It is free software; you can redistribute it and/or modify it under                              *
  * the terms of the GNU Lesser General Public License, either version 3                             *

diff --git a/Configuration/Settings.yaml b/Configuration/Settings.yaml
@@ -2,4 +2,8 @@ TYPO3:
   Neos:
     typoScript:
       autoInclude:
-        'Flowpack.SearchPlugin': TRUE
+        'Flowpack.SearchPlugin': TRUE
+
+  TypoScript:
+    defaultContext:
+      Flowpack.SearchPlugin.Array: 'Flowpack\SearchPlugin\EelHelper\SearchArrayHelper'
diff --git a/Resources/Private/Templates/NodeTypes/Search.html b/Resources/Private/Templates/NodeTypes/Search.html
@@ -1,8 +1,13 @@
 {namespace neos=TYPO3\Neos\ViewHelpers}
 {namespace ts=TYPO3\TypoScript\ViewHelpers}
+{namespace search=TYPO3\TYPO3CR\Search\ViewHelpers}
+<div{attributes -> f:format.raw()}>
+	{searchForm -> f:format.raw()}
 
-{searchForm -> f:format.raw()}
-
-Showing {searchResults -> f:count()} of {totalSearchResults} results
-
-{searchResultRenderer -> f:format.raw()}
+	<f:if condition="{searchQuery}">
+		<search:widget.paginate query="{searchQuery}" as="results">
+			Showing {results.accessibleCount} of {totalSearchResults} results
+			<ts:render path="searchResultRenderer" context="{searchResults: results}" />
+		</search:widget.paginate>
+	</f:if>
+</div>
diff --git a/Resources/Private/Templates/SearchResult/DocumentSearchResult.html b/Resources/Private/Templates/SearchResult/DocumentSearchResult.html
@@ -1,9 +1,18 @@
 {namespace neos=TYPO3\Neos\ViewHelpers}
+<neos:link.node node="{node}">{title}</neos:link.node>
 
-<b><neos:link.node node="{node}">
-	{title}
-</neos:link.node></b><br />
+<f:if condition="{highlight}">
+	<p>
+		<f:for each="{highlight}" as="fragment" iteration="fragmentIterator">
+			{fragment -> f:format.raw()}{f:if(condition: fragmentIterator.isLast, else: '...')}
+		</f:for>
+	</p>
 
-<f:if condition="{description}">
-	{description}
+	<div class="breadcrumb">
+		<span>Path:</span>
+		<f:for each="{parents}" as="parent">
+			<neos:link.node node="{parent}" />
+		</f:for>
+		<neos:link.node node="{node}" />
+	</div>
 </f:if>
diff --git a/Resources/Private/TypoScript/ResultRendering.ts2 b/Resources/Private/TypoScript/ResultRendering.ts2
@@ -15,14 +15,11 @@ prototype(Flowpack.SearchPlugin:SingleResult) < prototype(TYPO3.TypoScript:Case)
 prototype(TYPO3.Neos:DocumentSearchResult) < prototype(TYPO3.TypoScript:Template) {
 	templatePath = 'resource://Flowpack.SearchPlugin/Private/Templates/SearchResult/DocumentSearchResult.html'
 	node = ${node}
+	highlight = ${Flowpack.SearchPlugin.Array.flatten(searchHit.highlight)}
 
 	title = ${q(node).property('title')}
 	description = ''
+	parents = ${Array.reverse(q(node).parents('[instanceof TYPO3.Neos:Document]').get())}
 }
 
-prototype(TYPO3.Neos.NodeTypes:PageSearchResult) < prototype(TYPO3.Neos:DocumentSearchResult)
-
-
-prototype(TYPO3.NeosDemoTypo3Org:ChapterSearchResult) < prototype(TYPO3.Neos:DocumentSearchResult) {
-	description = ${q(node).property('chapterDescription')}
-}
+prototype(TYPO3.Neos.NodeTypes:PageSearchResult) < prototype(TYPO3.Neos:DocumentSearchResult)
diff --git a/Resources/Private/TypoScript/SearchPlugin.ts2 b/Resources/Private/TypoScript/SearchPlugin.ts2
@@ -1,24 +1,22 @@
-
 prototype(Flowpack.SearchPlugin:Search) {
 
-	// we store the search results in the TypoScript context, so that they bubble through the nested elements like the searchResultRenderer
-	@override.searchResults = ${this.searchQuery.execute()}
-
-
-	searchQuery = ${Search.query(site).fulltext(request.arguments.search || '')}
+	searchTerm = ${request.arguments.search}
+	searchQuery = ${this.searchTerm ? Search.query(site).fulltext(this.searchTerm) : null}
 
 	totalSearchResults = ${this.searchQuery.count()}
-	searchResults = ${searchResults}
 
 	searchForm = Flowpack.SearchPlugin:Search.Form
 
 	searchResultRenderer = TYPO3.TypoScript:Collection {
+		// the "searchResults" are filled from the <search:widget.paginate> ViewHelper.
 		collection = ${searchResults}
 		itemRenderer = Flowpack.SearchPlugin:SingleResult
 		itemName = 'node'
+		// we also make the full ElasticSearch hit available to nested rendering, if possible
+		itemRenderer.@override.searchHit = ${searchResults.searchHitForNode(node)}
 	}
 
-	searchResultRenderer.@process.wrap = ${'<ul>' + value + '</ul>'}
+	searchResultRenderer.@process.wrap = ${'<ol>' + value + '</ol>'}
 	prototype(Flowpack.SearchPlugin:SingleResult).@process.wrap = ${'<li>' + value + '</li>'}
 
 	@cache {
@@ -34,5 +32,4 @@ prototype(Flowpack.SearchPlugin:Search) {
 prototype(Flowpack.SearchPlugin:Search.Form) < prototype(TYPO3.TypoScript:Template) {
 	templatePath = 'resource://Flowpack.SearchPlugin/Private/Templates/NodeTypes/Search.Form.html'
 	searchWord = ${request.arguments.search}
-}
-
+}