Add default option when facet is active

Maybe if a value is selected, the label "--Choose--" should be changed to something like "--View All--". This could help the user to understand the UI. What do you think?

Attached patch is another way to do it.

Tried #4, got

Notice: Undefined index: default_value in facetapi_select_facet_form() (line 10 of /******/facetapi_select/facetapi_select.module).
Notice: Undefined index: default_value in facetapi_select_facet_form() (line 11 of /******/facetapi_select/facetapi_select.module).

Also, when I go back to no selection, it sends me to the homepage...

Applied #1 and that worked beautifully. Also changed the text from Choose to view all as suggested by #2. Works perfectly! Thanks!

Duplicate of #1412442: Selected value ?

I think the approach in comment 4 is cleaner, but it needs work to resolve the issues in comments 5 (easy) and 6.

Hi, everyone! I've found this issue and want to share my idea. In one of my projects I need this feature https://drupal.org/node/2284099 and also to show default element for reseting facets. I've written the code in case when facetapi_pretty_paths installed.

I think my patch from #11 is usefull after applying patch from issue https://drupal.org/node/2283201, when active element is shown.

Hi.

I'm attaching a patch that rewrites the code pretty heavily to allow:
- Customizable reset label with a reset link for the current facet only
- Option to pre-select active facet. (this was inspired and incorporated from https://drupal.org/node/2283201 , thanks mparker17)

In conjunction with the facet api patch here https://drupal.org/node/1393928 , you can create for example 3 select field facets, you can only choose one value from each, and easily reset one of the values chosen.

@Placinta's patch in #13 includes the functionality of #2283201: Provide an option to pre-select the active facet, which I've closed as a duplicate of this patch.

The patch works great for me.

The code looks good, except for two coding-standards problems:

1. +++ b/facetapi_select.module
   @@ -54,36 +59,84 @@ class FacetapiSelectDropdowns extends FacetapiWidgetLinks {
   +      $select_options[$url] = $item['#markup'].' ('.$item['#count'].')';
   

   Always use a space between the dot and the concatenated parts to improve readability.

2. +++ b/facetapi_select.module
   @@ -54,36 +59,84 @@ class FacetapiSelectDropdowns extends FacetapiWidgetLinks {
   +        // Store the key of the active facet and the reset url that is attached to it.
   ...
   +    // The reset URL that needs to be clicked to remove the currently active facet value.
   ...
   +    // Otherwise add reset label and set the active item as the default option, if the settings flag was set.
   ...
   +      // The actual active facet will have the reset url, we need to change it to be the current URL instead.
   ...
   +      // First we get the active item label, then we ask for the current URL and query params.
   +      // Finally we append it to the new options array, which makes it appear directly after the reset label, and before the rest of the items.
   ...
   +      // Remove the old active facet item from the select options, which is keyed by the reset URL.
   

   All comments should wrap at 80 characters.

... once these have been fixed, I'd say the patch in #13 is RTBC!

Installed coder module, and fixed all the warnings that it gave me.
Attaching updated patch and inter diff.

Looks good to me! @Placinta, thanks for all your help and hard work!

it works!

Attaching an updated patch and interdiff, which adds the #states FAPI key, to make the Facet API select settings appear only when it is the active widget chosen. Without this change, if any other widget was selected as active, the Facet API select settings were still shown.

Is this related to #2336857: Unable to de-select a value?
Possible duplicate?

I'm going based on just the original post of both issues so I could be way off.

The patch in #20 works for me, and the change makes sense.

The patch #20 also worked for me! Thanks a bunch!

This does not work as I would expect.

When the facet is not active I have an 'Display all' option (which I added under Default Option Label)

When I choose a facet the 'Display all' is missing. Is my configuration amiss?

Ok - I got it working with some additional configuration. Apparently you have to choose OR for this to work. I didn't want that, I want that it just displays the one that you chose, I think it should work that way, but now the options get filtered out.

EDIT

Oh my gosh how confusing. I think I got it working the way I want it, by selecting OR but 'Limit to one active item' checked. Not sure if this patch should take into account different ways of enabling this option.

Re-roll against 7.x-1.x.

Still testing and going over logic before committing.

#27 failed to patch on 2 out of 3 hunks, so attempting to remedy to it with the attached patch #28. Let me know if that applies and works as intended.

#29 replaces #28 by addressing a coding standard issue on one line.

Patch #29 works for me.

I realized a lot of work has happened here, but I believe another ticket that resolves this better, which also has had a lot of work, resolves this issue nicely. #2336857: Unable to de-select a value

With that I am actually going to mark this one as a dupe.