Ok, quite alot of patches have been combined to give me this functionality and I think it's a reasonable patch to submit it and combine those patches.
What does this patch?
Creates 2 new settings :
- Apache Solr Base Path: Specify here what the base path of the apache solr search index is
- Apache Solr Default Filter: Specify here if you need a default search to start with
This enables a user to put apache_solr as a frontpage and give the user a default search but without disabling the full search functionality.
This way you can start with the facets blocks and some default search. In my example I gave these variables the following values :
base path : apachesolr_search
default filter : type:product (ubercart)
Also it includes the following patches.
Search for just facet(s) (http://drupal.org/node/358166)
On empty search, show enabled filters to start a search (http://drupal.org/node/457826)
In my opinion these 2 patches are ready to be combined in a bigger patch like this one and will be easier to maintain and progress
Alot of the codestyle has been rearranged as well so sorry for the sometimes useless diff lines
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | apachesolr.patch | 22.85 KB | nick_vh |
Comments
Comment #1
nick_vhAnd as a comment the patch, appearently it didn't upload!
Maybe I would also specify why I didn't make an seperate patch. Mainly because I altered too much to have 3 different contributions the the patches they belong to. And some of these changes are a global change and without combining it doesn't make any sense..!
Some guidance can be found here on how to put apache solr with a default or custom filter on the frontpage. With the current patch you don't need custom menu's anymore like describes in the comment below, but the custom menu can be used if you want to be able to have custom solr startpages.
http://drupal.org/node/358166#comment-1655332
Comment #2
janusman commented@pwolanin just committed #358166: Search for just facet(s) so you might want to re-roll from there.
I'd also suggest--for sanity's sake--to not include other patches in yours, rather give instructions as to which patch to apply before yours =)
Comment #3
nick_vhI'll follow the sanity's sake instead of the simplicity's sake ;-) Thanks for the comment.
Will close this one (and will take a look if I have some more changes than the commited patch)
Comment #4
nick_vhClosed