Apache Solr Search Integration has a great feature of being able to create additional search pages through the GUI. It would be useful for themers to be able to modify the display of the search pages on a per-page basis. The request is to add a template hint to the search-results.tpl.php template that follows the search_results__MODULE__PAGE-ID nomenclature.
| Comment | File | Size | Author |
|---|---|---|---|
| #14 | 1682004-14.patch | 1.52 KB | nick_vh |
| #12 | 1682004-12.patch | 1.7 KB | nick_vh |
| #11 | 1682004-11.patch | 1.19 KB | nick_vh |
| #9 | 1682004-9.patch | 1.73 KB | nick_vh |
| #1 | search-page-template-hints-1682004-1.patch | 631 bytes | cpliakas |
Comments
Comment #1
cpliakas commentedThe attached patch makes the suggested change.
Comment #2
nick_vhcommitted to D7. Thanks!
Comment #3
killua99 commentedThis part exist:
Need anything else?
Comment #4
drupalninja99 commentedThis is a great idea!
Comment #5
pwolanin commentedrelated bug fix: #1717490: Search result template suggestions relies on non-required fields
Let's also work some more on D7 - feels like this tpl suggestion should also (or insteaD) be based on environment ID?
Same with the search result one.
Comment #6
cpliakas commented-1 to templates only based on environment. You loose a ton of flexibility if you have to clone the environment for every search page you want to override. I have seen too many use cases where this would be problematic. +1 for also adding hints based on environment in case you want to bulk modify the the display of a group of pages.
Comment #7
pwolanin commentedI was thinking rather that that would be an alternative suggestion, not replacing what's already there for pages.
Comment #8
cpliakas commentedPerfect. I misunderstood. Sorry about that. In that case, definitely inline with what you are proposing.
Comment #9
nick_vhSo, let's see if this makes people happy?
Comment #10
nick_vhComment #11
nick_vhfewer checks
Comment #12
nick_vhWhoeps, we forgot to convert it to arrays!
Comment #13
pwolanin commentedcommitted, needs to be ported?
Comment #14
nick_vhported, tested it by checking the $variables array. Looking good
Comment #15
nick_vhcommitted