Feature #20421

Epic #65814: Make Indexed search extbase plugin shine

Feature #19465: Configuration enhancement for indexed_search

Unflexible Output of Resultbrowser

Added by Andre Dittmar over 10 years ago. Updated about 3 years ago.

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

0%

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)

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

History

#1 Updated by Ralf Hettinger almost 10 years ago

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

#2 Updated by Gerrit Code Review over 6 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

#3 Updated by Gerrit Code Review over 6 years ago

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

#4 Updated by Gerrit Code Review over 6 years ago

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

#5 Updated by Gerrit Code Review over 6 years ago

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

#6 Updated by Gerrit Code Review about 6 years ago

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

#7 Updated by Gerrit Code Review almost 5 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

#8 Updated by Tymoteusz Motylewski over 4 years ago

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

#9 Updated by Gerrit Code Review over 4 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

#10 Updated by Tymoteusz Motylewski about 4 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

#11 Updated by Ralf Hettinger about 4 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.

#12 Updated by Anja Leichsenring about 4 years ago

  • Sprint Focus changed from On Location Sprint to Remote Sprint

#13 Updated by Gerrit Code Review about 4 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

#14 Updated by Gerrit Code Review almost 4 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

#15 Updated by Tymoteusz Motylewski about 3 years ago

  • Status changed from Under Review to Closed

Also available in: Atom PDF