The reset link appear only when some facet filters are selected. It could be also usefull to make it appear when search keys are submitted.

CommentFileSizeAuthor
#7 add_reset_link_on_search_keys-2376711-7.patch4.13 KBmlanth
#6 add_reset_link_on_search_keys-2376711-4.patch5.11 KBPaulDinelle
#3 add_reset_link_on_search_keys-2376711-3.patch4.95 KBPaulDinelle
#2 add_reset_link_on_search_keys-2376711-2.patch1.92 KBdrikc
#1 add_reset_link_on_search_keys-2376711-1.patch1.72 KBdrikc
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drikc’s picture

drikc CreditAttribution: drikc commented
Status: Active » Needs review
FileSize
add_reset_link_on_search_keys-2376711-1.patch1.72 KB

The attached patch make the reset link appear also when search keys are submitted.

drikc’s picture

drikc CreditAttribution: drikc commented
FileSize
add_reset_link_on_search_keys-2376711-2.patch1.92 KB

Fix compatiblity with php < 5.5.

PaulDinelle’s picture

PaulDinelle CreditAttribution: PaulDinelle at Fenix Solutions Inc commented
FileSize
add_reset_link_on_search_keys-2376711-3.patch4.95 KB

I used this patch for a bit, but after customizing my search key and combining it with current_search_links module (to display reset buttons beside each facet), this module would no longer clear out keywords.

I've updated the patch to add in a custom settings field for the search keyword (mimicking the current_search_links module) so that the keywords could be properly removed. It meant I had to duplicate the CurrentSearchItem settings form, but I couldn't see a way around this.

Hope this helps someone else!

PaulDinelle’s picture

PaulDinelle CreditAttribution: PaulDinelle at Fenix Solutions Inc commented
PaulDinelle’s picture

PaulDinelle CreditAttribution: PaulDinelle at Fenix Solutions Inc commented

Hmm, my patch still doesn't work 100% with current_search_links module.

PaulDinelle’s picture

PaulDinelle CreditAttribution: PaulDinelle at Fenix Solutions Inc commented
FileSize
add_reset_link_on_search_keys-2376711-4.patch5.11 KB

Okay, fixed an issue with the filter intersection list. If the first entry was empty (as is the case if removing the parent term in a hierarchy-enforced list), it was accidentally including the second filter in the reset link query.

mlanth’s picture

mlanth CreditAttribution: mlanth commented
FileSize
add_reset_link_on_search_keys-2376711-7.patch4.13 KB

Patch re-roll to reflect newer module version.