Hi!
I use facet api with apachesolr. I have set the facets to display in all node pages (node/*)
Some of my node pages have a path alias set and some do not. The facets are displayed *only* for the pages that do not have the path alias set.
The expected dehaviour would be to display for all node pages no matter the path alias.
This happens because only the alias path is checked and not the original path.
| Comment | File | Size | Author |
|---|---|---|---|
| #13 | 1719302-13.patch | 8.31 KB | nick_vh |
| #11 | 1719302-11.patch | 8.27 KB | nick_vh |
| #10 | 1719302-10.patch | 8.29 KB | nick_vh |
| #6 | 1719302-6.patch | 8.22 KB | nick_vh |
| #4 | 1719302-3.patch | 6.08 KB | nick_vh |
Comments
Comment #1
mstiThis is a patch to fix it
Comment #2
nick_vhSome improvement, but it certainly makes sense what you try to do there.
moved the check to not load this function if facetapi is not available to the beginning of the process. No need to load a search page if that is not available.
perhaps we want to remove the load search pages thing or make that more performant. Looking into it
Comment #4
nick_vhGreat, so this improves the loading of search pages from three times per page load to once max per page load.
We probably do want to reduce this, because loading it on every page load is not something I find very acceptable if the checkbox for getting facets on every search page is not selected.
Comment #6
nick_vhComment #7
nick_vhComment #8
nick_vhCommitted (fixed the end of file error before committing) to D7
Thanks for reporting!
Comment #9
mstiI am glad to help!
Comment #10
nick_vhBackported
Comment #11
nick_vhnewline added
Comment #13
nick_vhComment #14
nick_vhcommitted