Not sure how best to title this issue but the patch provided addresses an issue where one is using Facet API Pretty Paths AND have patched Facet API to constrain a facet to only allow one active item (See patch in issue: #1393928: Add a per-facet setting that allows only one item to be active at a time).

This is quick and dirty so hoping anyone who is using the above patch and also wants to use pretty paths can test.

CommentFileSizeAuthor
facetapi_pretty_paths-fix-for-facetapi-issue-1393928.patch1.26 KBjaydub
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dasjo’s picture

dasjo
he/him
Primary language German
Location Zurich
CreditAttribution: dasjo commented

hi, thanks for your contribution! the patch looks clean to me.

i will wait until the until the facet api patch goes in.

dasjo’s picture

dasjo
he/him
Primary language German
Location Zurich
CreditAttribution: dasjo commented
Category: support » feature
fago’s picture

fago
Primary language German
Location Vienna
CreditAttribution: fago commented

Patch works fine for me.

Sumi’s picture

Sumi CreditAttribution: Sumi commented
Issue summary: View changes

Patch works fine for me to.

khiminrm’s picture

khiminrm CreditAttribution: khiminrm commented

Patch works fine for me

jaydub’s picture

jaydub CreditAttribution: jaydub commented
Status: Needs review » Reviewed & tested by the community
dasjo’s picture

dasjo
he/him
Primary language German
Location Zurich
CreditAttribution: dasjo commented

Thanks everyone for your feedback. Let's make sure, #1393928: Add a per-facet setting that allows only one item to be active at a time gets in and I will be happy to commit this one.

philipz’s picture

philipz CreditAttribution: philipz commented

Looks like it has been committed :)

radimklaska’s picture

radimklaska
Primary language Czech
Location 🇨🇿Czech Republic
CreditAttribution: radimklaska commented

#1393928: Add a per-facet setting that allows only one item to be active at a time was commited and is now included in recommended release. I also tested the patch from this issue and it works. Thanks!

  • jaydub authored 4483e63 on 7.x-1.x 
    Issue #1935782 by jaydub: Added Support pretty paths for Facet API per-...
dasjo’s picture

dasjo
he/him
Primary language German
Location Zurich
CreditAttribution: dasjo commented
Status: Reviewed & tested by the community » Fixed

committed to 7.x-1.x, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

ShaneOnABike’s picture

ShaneOnABike CreditAttribution: ShaneOnABike commented

Sorry to nag but could we release a new version. This bug pretty much halts any proper use of FacetAPI after they repaired it to allow for hidden values. I had _no clue_ that this module was blocking htat from working (~2 hours later) :/

Pretty pleaaaze!