Closed (fixed)
Project:
Apache Solr Search
Version:
6.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
13 Jul 2012 at 02:48 UTC
Updated:
2 Oct 2012 at 10:11 UTC
Jump to comment: Most recent file
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