I've installed Apachesolr 6.x-3.x and the dev of Facets API. It mostly works, but I've noticed that even when I select "Links with checkboxes" from a facet's settings, it does not display with the checkboxes. Is this a bug or am I missing something?

CommentFileSizeAuthor
#12 1518258-11.patch3.38 KBnick_vh
#11 1518258-10.patch3.21 KBnick_vh
#9 1518258-8.patch3.09 KBnick_vh
#6 facetapi-javascript-1518258-6.patch2.28 KBdandaman
#4 facetapi-javascript-1518258-2.patch1.49 KBdandaman

Comments

dandaman’s picture

dandaman commented

I noticed I did get a JavaScript error in my FireBug when running this with the checkboxes. It seems that the Drupal.behaviors code is in a D7 format and not a D6 format. I'll try to see if I can update it and get it working if I have time.

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented

Sounds good, Would love to have this fixed.

Nick

cpliakas’s picture

cpliakas commented

Piece removed from OP:

Also, I was playing with Facet API on D7 just to figure out how it all worked, and when I turned on checkboxes, I still could not select more than one item from the facets. Isn't that the point of the checkboxes? Or is there another way I'd be able to do something like say "I want to filter by items of Content Type A and Content Type C but not Content Type B"? (This could be broken out into another issue if needed; I'm just a bit new to Facet API and want to better understand how all this works in order to help debug it.)

Let's break this out into a separate issue and cross-post here.

Thanks for reporting,
Chris

dandaman’s picture

dandaman commented
StatusFileSize
new facetapi-javascript-1518258-2.patch1.49 KB

Here's some code that I believe is more in the 6.x JavaScript format.... it no longer gives an error, but it still doesn't make checkboxes show up.

dandaman’s picture

dandaman commented

Created separate issue for #3 at #1524640: AND / OR functionality of the facets (D6).

dandaman’s picture

dandaman commented
StatusFileSize
new facetapi-javascript-1518258-6.patch2.28 KB

Updated patch to include working classes on the links within a facet.

dandaman’s picture

dandaman commented

For some reason my UL IDs are not good. They have an @ symbol in them and I'm pretty sure that's not OK, right?

<ul id="facetapi-facet-apachesolr@apachesolr_search-block-bundle" class="facetapi-facetapi_checkbox_links facetapi-facet-bundle">

Thanks for all your help!

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
Status: Active » Needs work

Cpliakas, how was it solved with the id's in Drupal 7? We're hashing them or?

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
Status: Needs work » Needs review
StatusFileSize
new 1518258-8.patch3.09 KB
nick_vh’s picture

nick_vh
he/him
Location Ghent
commented

@dandeman, I encountered similar problems but I just worked around it, let's see if this passes the tests and I'll commit it right away

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
StatusFileSize
new 1518258-10.patch3.21 KB

some small code standard fixes

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
StatusFileSize
new 1518258-11.patch3.38 KB
killua99’s picture

killua99 commented
Status: Needs review » Reviewed & tested by the community

Awesome, works as expected.

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
Status: Reviewed & tested by the community » Closed (fixed)

and committed! Thanks all for the input!

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
Issue summary: View changes

Updated issue summary.