Project

General

Profile

Actions

Feature #20421

closed

Epic #65814: Make Indexed search extbase plugin shine

Feature #19465: Configuration enhancement for indexed_search

Unflexible Output of Resultbrowser

Added by Andre Dittmar almost 15 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Indexed Search
Target version:
-
Start date:
2009-05-14
Due date:
% Done:

0%

Estimated time:
PHP Version:
Tags:
Complexity:
Sprint Focus:
Remote Sprint

Description

Indexed search is still sometimes unflexible if you try to implement some fancy layouts. For example the resultbrowser is still hardcoded and it is not possible to show the browser only at the bottom of search results and the search summery (result count) at the top for example.

For a running project I introduced some more Typoscript-Parameters to handle this more flexible. To try this feature out, I provided a patch to extend the indexed_search class.

I'm happy about any feedback or suggestions to improve this approach...

Note: We also added a first/last page link to the resultbrowser as a new feature, which might be helpful if you have plenty of results. This feature is only xclassed (and therefore hardcoded) at the moment, but probably also interesting to implement this more sophisticated :-)

Best regards,
Andre

plugin.tx_indexedsearch{
show {
// Configure Resultbrowser at top
resultBrowserTop {
showResultCount = 1
showResultBrowser = 0
}
// Configure Resultbrowser at bottom
resultBrowserBottom {
showResultCount = 0
showResultBrowser = 1
}
}
}
(issue imported from #M11085)


Files

class.tx_indexedsearch.patch (2.67 KB) class.tx_indexedsearch.patch Administrator Admin, 2009-05-14 15:43
Actions #1

Updated by Ralf Hettinger over 14 years ago

partially related to #9574 (patch 0009574_v7.patch)

Actions #2

Updated by Gerrit Code Review almost 11 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20266

Actions #3

Updated by Gerrit Code Review almost 11 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20266

Actions #4

Updated by Gerrit Code Review almost 11 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20266

Actions #5

Updated by Gerrit Code Review almost 11 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20266

Actions #6

Updated by Gerrit Code Review over 10 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20266

Actions #7

Updated by Gerrit Code Review over 9 years ago

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

Actions #8

Updated by Tymoteusz Motylewski over 9 years ago

  • Description updated (diff)
  • Target version deleted (0)
  • Parent task set to #19465
Actions #9

Updated by Gerrit Code Review about 9 years ago

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

Actions #10

Updated by Tymoteusz Motylewski almost 9 years ago

  • Sprint Focus set to On Location Sprint

TODO:
1. check if these changes are also needed in the extbase plugin, if yes, implement it there too (or maybe we can use pagination widget for extbase?)
2. drop showTopBrowser and showBottomBrowser settings (you can easily modify template to remove browser if you don't need it.
3. review the rst file if it contain all needed information and if the markup is correct

Actions #11

Updated by Ralf Hettinger almost 9 years ago

Tymoteusz Motylewski wrote:

2. drop showTopBrowser and showBottomBrowser settings (you can easily modify template to remove browser if you don't need it.

nitpick: modifying template has (minimal) negative impact on runtime compared to configuration ... and imo is (minimally) more work if the default template is fine apart from that.

Actions #12

Updated by Anja Leichsenring over 8 years ago

  • Sprint Focus changed from On Location Sprint to Remote Sprint
Actions #13

Updated by Gerrit Code Review over 8 years ago

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

Actions #14

Updated by Gerrit Code Review over 8 years ago

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

Actions #15

Updated by Tymoteusz Motylewski over 7 years ago

  • Status changed from Under Review to Closed
Actions

Also available in: Atom PDF