Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Currently this module doesn't play nice with Search API Ranges
It would be great if it did. Not sure if this requires changes in this module, SAR or both.
It currently receives no 'pretty paths' and when one range facet is applied it causes all the facet links to act as if no facet is applied (remove links link to the base search, add links link to the base search + just that one facet).
Comments
Comment #1
Shadlington CreditAttribution: Shadlington commentedCross-posted at search api ranges: #1511144: Please make compatible with Facet API Pretty Paths
Comment #2
dasjosounds like a valid request. i would support such a patch
Comment #3
dasjowith #1431682: Plan & implement pluggability in place that should be pretty straight forward
Comment #4
Anonymous (not verified) CreditAttribution: Anonymous commentedOk, I started working on this.
I have setup a coder override:
And an info alter:
I copied
facetapi_pretty_paths_coder_default.inc
and made it fit my module (added it to search_api_ranges.info file & flushed caches / rebuild registry). So far, this works, my custom coder is in fact called for my range facets!But now,... I am stuck. I want to achieve this type of URL:
<alias>/<from-value>/<to-value>
...any help? Where should I define this "format"?
Comment #5
Anonymous (not verified) CreditAttribution: Anonymous commentedA temporary solution is
<alias>/[<from> TO <to>]
which works, it's being picked up.
I ll have a patch soon.
Comment #6
dasjohi morningtime, thanks for working on this!
yes, as you might have seen, facetapi pretty paths currently doesn't support values separated by multiple slashes.
this would require a change to
fetchParams
of theFacetapiUrlProcessorPrettyPaths
, as it currently does two "hard-coded" array_pops(), see heregreat!, keep us updated
Comment #7
Anonymous (not verified) CreditAttribution: Anonymous commentedI've made a patch here.
http://drupal.org/node/1511144#comment-6171112
Closing this one, marking as duplicate of #1511144: Please make compatible with Facet API Pretty Paths
Comment #7.0
Anonymous (not verified) CreditAttribution: Anonymous commentedExtra info