Project

General

Profile

Actions

Feature #82260

closed

Additional value for indexed_search result

Added by Sylvia Moschner about 7 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Indexed Search
Target version:
-
Start date:
2017-08-31
Due date:
% Done:

100%

Estimated time:
PHP Version:
Tags:
Complexity:
easy
Sprint Focus:
On Location Sprint

Description

For Styling of the result of indexed_search it would be nice to have the path of the result in a seperate value.

That would make it possible to enclose the whole result in a Link (e.g. a Bootstrap List Item). That increases usabilty on smartphones extremly.

I added / changed the following lines in the SearchController.php

507 $resultData['path'] = '<a href="' . htmlspecialchars($row['data_filename']) . '"' . $targetAttribute . '>' . htmlspecialchars($row['data_filename']) . '</a>';
508 $resultData['pathraw'] = htmlspecialchars($row['data_filename']);

513 $resultData['path'] = $this->linkPage($pathId, $pathStr, [
514 'cHashParams' => $row['cHashParams'],
515 'data_page_type' => $row['data_page_type'],
516 'data_page_mp' => $pathMP,
517 'sys_language_uid' => $row['sys_language_uid']
518 ]);
519 $resultData['pathraw'] = $this->linkPage($pathId, $pathStr, [
520 'cHashParams' => $row['cHashParams'],
521 'data_page_type' => $row['data_page_type'],
522 'data_page_mp' => $pathMP,
523 'sys_language_uid' => $row['sys_language_uid']
524 ], [], true);

1316 protected function linkPage($pageUid, $linkText, $row = [], $markUpSwParams = [], $rawlink = false)

1349 if ($rawlink) {
1350 return $uri;
1351 }else {
1352 return '<a href="' . htmlspecialchars($uri) . '"' . $target . '>' . $linkText . '</a>';
1353 }

Actions #1

Updated by Gerrit Code Review about 7 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54149

Actions #2

Updated by Gerrit Code Review about 7 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54149

Actions #3

Updated by Gerrit Code Review about 7 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54149

Actions #4

Updated by Gerrit Code Review about 7 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54149

Actions #5

Updated by Gerrit Code Review about 7 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54149

Actions #6

Updated by Gerrit Code Review about 7 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54149

Actions #7

Updated by Susanne Moog about 7 years ago

  • Sprint Focus set to On Location Sprint
Actions #8

Updated by Joerg Boesche almost 7 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #9

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF