It'd be useful to have a way of removing a facet that only had one item, e.g. only one content type.

CommentFileSizeAuthor
#12 interdiff-2426693-5-12.txt1.11 KBbrockfanning
#12 facetapi_n2426693-12.patch2.57 KBbrockfanning
#10 interdiff-2426693-5-10.txt905 bytesbrockfanning
#10 facetapi_n2426693-10.patch2.34 KBbrockfanning
#5 facetapi-n2426693-4.patch1.8 KBdamienmckenna
#4 facetapi-n2426693-3.patch1.22 KBdamienmckenna
#2 facetapi-n2426693-2.patch1.22 KBdamienmckenna
#1 facetapi-n2426693-1.patch1.19 KBdamienmckenna

Comments

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review
StatusFileSize
new facetapi-n2426693-1.patch1.19 KB

This is the theory, I need to test it.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new facetapi-n2426693-2.patch1.22 KB

Verify that $item['#count'] exists, just to be sure.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

It'd be really nice to be able to hide all facets if there's only one search result, but I suspect it's wishful thinking for now.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new facetapi-n2426693-3.patch1.22 KB

Doh! Forgot to actually check for $count > 1.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new facetapi-n2426693-4.patch1.8 KB

This version adds two filters - one to remove the facet if there's only one total count, one to remove the filter if there's only one matching item for this facet.

criscom’s picture

criscom
Location Graz, Austria
commented

Patch in #5 works for me. Thanks, Damien!

Scott Robertson’s picture

Scott Robertson commented

#5 is working for me as well.

sebastien m.’s picture

sebastien m. commented

#5 is working for me too.
thanks

sebastien m.’s picture

sebastien m. commented

seems to be not working with 'parent_all' taxonomy facets in case of one parent is selected and contains multiple children.
I think that an element shouldn't be hidden if selected.

brockfanning’s picture

brockfanning commented
StatusFileSize
new facetapi_n2426693-10.patch2.34 KB
new interdiff-2426693-5-10.txt905 bytes

I can't test this at the moment but it should fix #9.

brockfanning’s picture

brockfanning commented

To follow up, I did get a chance to test #10, and it seems to work fine.

brockfanning’s picture

brockfanning commented
StatusFileSize
new facetapi_n2426693-12.patch2.57 KB
new interdiff-2426693-5-12.txt1.11 KB

One consequence of filtering out single-item facets is that if a user selects 1 item from a 2-item facet, then that facet will disappear, which prevents the user from de-selecting that facet. (They just have to click the back button.)

That seems undesirable to me, so here is an updated version of the patch to deal with that.

brockfanning’s picture

brockfanning commented
Status: Needs review » Closed (won't fix)

I recently noticed that the dev version of facetapi_bonus has this functionality. I'll close this issue since there's an existing solution for the problem.