Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fago’s picture

FileSize
4.47 KB

Looks like the patch contains shell control characters?

Re-rolled patch. It probably makes sense to commit this one once #1393928: Add a per-facet setting that allows only one item to be active at a time landed.

maximilianmikus’s picture

ok fixed the patch, the patch from fago was missing changes in the .info file.

vintorg’s picture

Issue summary: View changes

I just rolled patch #2. Are there supposed to be 2 facetapi_bonus_facetapi_widgets() functions? I assume not.

vintorg’s picture

Also, issue with the .info file. I had to add the line by hand.

reject file:

--- facetapi_bonus.info
+++ facetapi_bonus.info
@@ -4,5 +4,6 @@
 files[] = "plugins/facetapi/dependency_facet.inc"
 files[] = "plugins/facetapi/filter_rewrite_items.inc"
 files[] = "plugins/facetapi/filter_exclude_specified_items.inc"
+files[] = plugins/facetapi/widget_links_radio.inc
 name = Facet API Bonus
 package = Search Toolkit
vintorg’s picture

I added the array elements together in facetapi_bonus_facetapi_widgets. The radio buttons are now displaying!

1. On the filters tab, I keep getting the following errors, even if I don't select any options:

Contained fields field is required.
Contained fields field is required.

2. On global settings, I have the the "Limit to one active item" checked, and I have "Hide active items" unchecked in the facet's active filters. However, when I select an option (these are taxonomy terms), the rest of the items hide. I want all the items to display, even if not active. How do I configure that?

nlambert’s picture

For others looking, this patch required #1393928: Add a per-facet setting that allows only one item to be active at a time which has been committed and is now included in the current facetapi version 1.5

In the display settings for the given facet, you must check "Limit to one active item"

@vintorg

I am not experiencing the errors mentioned in #1, but I believe you must change the "Minimum facet count" setting.

Johann Wagner’s picture

Patch #2 works great with version 7011.

Thank you.

Johann Wagner’s picture

I needed "real" radio buttons and also reduce JS to only auto-submit the form.

Though the patch #2 is working, without Javascript the link widget is displayed.

Please find attached a new widget that uses the FAPI's radios element instead, and that could be included to FacetAPI Bonus.

Unfortunately I've no time to make a patch, but the module file is short and self-explanatory.

jsacksick’s picture

Patch rerolled against latest dev, when you download the module via drush, the patch won't apply because of the .info modification, a possible workaround would be to add the class on top of the others...

selva8187’s picture

Patch #2 working great. Thank you fago.

But One correction in that patch js file call : /js/facetapi_bonus_widget_links_radio.js

filsterjisah’s picture

I've tried to use patch from #2 via drush make but the patch failed to apply.
Therefor attached a "valid" one.

filsterjisah’s picture

FileSize
4.91 KB
filsterjisah’s picture

FileSize
4.93 KB

Third ftw... This patch applies to version 1.1.

nareshbw’s picture

Thanks Fago ,

Patch#2 is working for me.

thewilkybarkid’s picture

Same as #13, but have fixed CSS selector for fields through entity references.