[BUGFIX] Indexed Search: Display links to external files 42/45142/4
authorTymoteusz Motylewski <t.motylewski@gmail.com>
Sat, 5 Dec 2015 13:29:06 +0000 (14:29 +0100)
committerTymoteusz Motylewski <t.motylewski@gmail.com>
Fri, 11 Dec 2015 20:47:29 +0000 (21:47 +0100)
Indexed search is rendering links to external files now.
Indexed search will not check if the file exists before displaying
search results. As a side effect this change also improves performance.

This change is affecting only the AbstractPlugin based plugin.
Extbase version do not have this check.

Resolves: #44381
Releases: master
Change-Id: Iae4e5b2f2cc575853f25c674cbb4307bdf3efa17
Reviewed-on: https://review.typo3.org/45142
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Tested-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
typo3/sysext/indexed_search/Classes/Controller/SearchFormController.php

index 536875e..0672a1b 100755 (executable)
@@ -566,13 +566,6 @@ class SearchFormController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlugin
             // Now, traverse result and put the rows to be displayed into an array
             // Each row should contain the fields from 'ISEC.*, IP.*' combined + artificial fields "show_resume" (bool) and "result_number" (counter)
             while (false !== ($row = $this->databaseConnection->sql_fetch_assoc($res))) {
-                if (!$this->checkExistence($row)) {
-                    // Check if the record is still available or if it has been deleted meanwhile.
-                    // Currently this works for files only, since extending it to content elements would cause a lot of overhead...
-                    // Otherwise, skip the row.
-                    $totalSearchResultCount--;
-                    continue;
-                }
                 // Set first row:
                 if ($positionInSearchResults === 0) {
                     $firstRow = $row;
@@ -1266,7 +1259,6 @@ class SearchFormController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlugin
      */
     public function checkExistance($row)
     {
-        GeneralUtility::logDeprecatedFunction();
         return $this->checkExistence($row);
     }
 
@@ -1276,9 +1268,11 @@ class SearchFormController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlugin
      *
      * @param array $row Result row array
      * @return bool Returns TRUE if record is still available
+     * @deprecated since TYPO3 CMS 7, will be removed in TYPO3 CMS 8 (it is not used in the core any more, see #44381)
      */
     protected function checkExistence($row)
     {
+        GeneralUtility::logDeprecatedFunction();
         $recordExists = true;
         // Always expect that page content exists
         if ($row['item_type']) {