Closed (fixed)
Project:
Commerce Kickstart
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
10 Aug 2012 at 09:15 UTC
Updated:
14 Nov 2012 at 01:01 UTC
The categories filter is broken on the products view of the no demo store content :
Notice: Undefined index: value in views_handler_filter->exposed_form() (line 760 of /media/nfsdrive/stratus_kickstart_v2/www/commerce_kickstart/profiles/commerce_kickstart/modules/contrib/views/handlers/views_handler_filter.inc).
Notice: Undefined index: term_node_tid_multiple_vocabularies in views_handler_filter_term_node_tid->exposed_validate() (line 258 of /media/nfsdrive/stratus_kickstart_v2/www/commerce_kickstart/profiles/commerce_kickstart/modules/contrib/views/modules/taxonomy/views_handler_filter_term_node_tid.inc).
If I remove the filter, the message disappears.
| Comment | File | Size | Author |
|---|---|---|---|
| Error message | 79.08 KB | GuGuss |
Comments
Comment #1
vasikethis is a normal issue as there are some vocabularies not built (Collection, Gender, etc...)
So probably we'll need (to define) 2 different products pages.
One default to be used for no demo content with less specific filters, fields and one which overwrites the default as it is now.
Comment #2
vasikepossible solutions (AFAICS):
Comment #3
hazaMaybe we need to find a way to alter just those specific vocabularies ?
Comment #4
vasikesolution proposal: to build a new views filter "Content: Product Display Has taxonomy terms (from Multiple Vocabularies)"
- provides only vocabularies defined as term references in the Product Display content types
- if there's no vocabulary selected, use all these vocabularies.
could be built within Drupal Commerce Product Reference, so it could be used without Kickstart distribution.
In Kickstart could be used with no selected vocabulary.
Comment #5
vasikeUsing a the a Views patch #1716826-14: Content: Has taxonomy terms from Multiple Vocabularies Filter
i create a new filter Product Display Has taxonomy terms (from Multiple Vocabularies)
this implements my previous defined solution proposal.
here is the commit : https://code.drupalcommerce.org/#/c/397/
Comment #6
vasikeactually this new filter should be used also for the Products page.
Comment #7
vasikeReBuilt the Products page view.
commit rebased : https://code.drupalcommerce.org/#/c/397/
Comment #8
vasikechange the Views patch : #1716826-15: Content: Has taxonomy terms from Multiple Vocabularies Filter
commit rebased
Comment #9
vasikequestions:
1. should this filter be part of Drupal Commerce?
2, should we change the Issue title: Product Display Has taxonomy terms (from Multiple Vocabularies) Views filter?
Comment #10
vasikecommit rebased https://code.drupalcommerce.org/#/c/397/
all it seems ok
Comment #11
bojanz commentedMerged. Fixes our errors.
Assigning to me and setting "needs work" so that I can look into the Views handler.
Comment #12
bojanz commentedLooked into this. Opened #1827622: Implement a custom category filter for Backoffice, remove the views patch.
Closing this issue.