I am using the 6.2x dev.

I am able to filter taxonomy by any term, even if that term is empty. When I choose to filter by that empty term (because I dont know its empty), I get a blank page back and the advanced catalog filters are gone. (Ie, I cannot just choose to filter by something else).

So either I need to find a way to hide empty terms (so users cannot filter by them), or a way to keep the AC filter on the top (or bottom). Potentially with a message (block in the body?), that says "No items found for your search".

Otherwise we are just having confused customers. Ideas?

CommentFileSizeAuthor
#3 uc_advanced_catalog.patch3.12 KB3cwebdev

Comments

Eli Baskin’s picture

Priority: Normal » Major

I have this issue as well.

Furthermore, after getting this empty page, when hitting 'back' button and trying to add a product to the cart from the displayed list, it is not added to the cart and a blank page is displayed again. Then, after I am trying to click on *any* category, a blank page is displayed, with only category name as header.

3cwebdev’s picture

I've got a working hack that makes it so that only product's taxonomy that are in the current view are listed in the filter. I plan on posting the patch as soon as I tweak it to work more globally and not just in my specific setup. It seems to be working good though and would resolve this issue.

3cwebdev’s picture

StatusFileSize
new3.12 KB

I've created a patch the alters uc_advanced_catalog.pages.inc and uc_advanced_catalog.filters.inc to do the following:

  • Modifies taxonomy filters to show only the terms that are actually contained in the current catalog page
  • Resets the taxonomy filter selection if the current catalog page is empty. This will fix a condition where the user selects a term that is no longer contained in the catalog page.

This has only been tested on one site and works good in its scenario. Further testing on different sites is required.

This is also my first patch file. I hope I've created it correctly. The one file contains patched for both files in the module.

3cwebdev’s picture

Status: Active » Needs review
shotokai’s picture

@setfree

I wanted to confirm that this worked on my setup, say thanks, and chime in for the inclusion of this functionality in the next release

timurek’s picture

Will this fix work also with prefix for drupal tables?

timurek’s picture

Status: Needs review » Needs work

... and the answer is no. Patch uses hardcoded table names, therefore it does not work on installations which use table prefix.

timurek’s picture

You have to use table names like {node} and {term_node}. But still - issue is not fixed. I have two filters on cck fields, when there is no product which should be displayed on current filter setting, no products nor filter is not shown, so I cannot re-set it. Works like before applying the patch :-/ Automatical filter reset on empty page does not work as well :-/

maxneb’s picture

I have the same problem. The Patch does`t help. I use filters by taxonomy and attributes, and I need to get message "No items found for your search" instead of blank page and make the filter fields don`t disappear.

Eli Baskin’s picture

Same here.

TriAdX’s picture

This needs fixed bad... Whole sections of my store dissapear when you switch sections because of this issue.

Going back to the original section to change the advanced filter back to show all is not something you should have to do...

PLEASE fix.

Eli Baskin’s picture

+100.

mogtofu33’s picture

Assigned: Unassigned » mogtofu33
Status: Needs work » Fixed

Fixed on 06-01-2011 dev snapshot.
Thanks for the patch but i use function taxonomy_term_count_nodes() to get taxonomy with nodes and change uc_advanced_catalog_products() to handle "no products" display.
Regards.
Mog.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

timurek’s picture

Works nicely, thanks!