Merge pull request #11 from michaelkubina-subhh/5.0.6_subhh_fix_highlighting

[BUGFIX] Fix search in document filtering and highlighting

Author: chrizzor

Classes/Common/Solr/SolrSearch.php

@@ -537,7 +537,7 @@ public function submit($start, $rows, $processResults = true)
                                 $searchResult['highlight'] = $doc['highlight'];
                                 $searchResult['highlight_word'] = preg_replace('/^;|;$/', '',       // remove ; at beginning or end
                                                                   preg_replace('/;+/', ';',         // replace any multiple of ; with a single ;
-                                                                  preg_replace('/[{~\d*}{\s+}{^=*\d+.*\d*}`~!@#$%\^&*()_|+-=?;:\'",.<>\{\}\[\]\\\]/', ';', $this->searchParams['query']))); // replace search operators and special characters with ;
+                                                                  preg_replace('/[{~\d*}{\s+}{^=*\d+.*\d*}{\sAND\s}{\sOR\s}{\sNOT\s}`~!@#$%\^&*()_|+-=?;:\'",.<>\{\}\[\]\\\]/', ';', $this->searchParams['query']))); // replace search operators and special characters with ;
                             }
                             $documents[$doc['uid']]['searchResults'][] = $searchResult;
                         }

Classes/Domain/Repository/DocumentRepository.php

@@ -683,7 +683,10 @@ public function getPreviousDocumentUid($uid)
                 return $prevDocument['uid'];
             }
 
-            return $this->getLastChild($this->getPreviousDocumentUid($parentId));
+            $previousDocumentId = $this->getPreviousDocumentUid($parentId);
+            if ($previousDocumentId) {
+                return $this->getLastChild($previousDocumentId);
+            }
         }
     }
 
@@ -727,7 +730,10 @@ public function getNextDocumentUid($uid)
                 return $nextDocument['uid'];
             }
 
-            return $this->getFirstChild($this->getNextDocumentUid($parentId));
+            $nextDocumentId = $this->getNextDocumentUid($parentId);
+            if ($nextDocumentId) {
+                return $this->getFirstChild($nextDocumentId);
+            }
         }
     }
 

Classes/Middleware/SearchInDocument.php

@@ -104,7 +104,9 @@ public function process(ServerRequestInterface $request, RequestHandlerInterface
                     [
                         'tx_dlf[id]' => !empty($resultDocument->getUid()) ? $resultDocument->getUid() : $parameters['uid'],
                         'tx_dlf[page]' => $resultDocument->getPage(),
-                        'tx_dlf[highlight_word]' => $parameters['q']
+                        'tx_dlf[highlight_word]' => preg_replace('/^;|;$/', '',       // remove ; at beginning or end
+                                                    preg_replace('/;+/', ';',         // replace any multiple of ; with a single ;
+                                                    preg_replace('/[{~\d*}{\s+}{^=*\d+.*\d*}{\sAND\s}{\sOR\s}{\sNOT\s}`~!@#$%\^&*()_|+-=?;:\'",.<>\{\}\[\]\\\]/', ';', $parameters['q']))) // replace search operators and special characters with ;
                     ]
                 );
 

Resources/Public/JavaScript/PageView/SearchInDocument.js

@@ -212,7 +212,7 @@ function triggerSearchAfterHitLoad() {
 
         if(searchedQueryParam && decodeURIComponent(queryParam[0]).indexOf(searchedQueryParam) !== -1) {
             $("input[id='tx-dlf-search-in-document-query']").val(decodeURIComponent(queryParam[1]));
-            $("#tx-dlf-search-in-document-form").submit();
+            //$("#tx-dlf-search-in-document-form").submit();
             break;
         }
     }