Use filter handler that allows for multiple selection

ryo - June 1, 2007 - 12:23
Project:CCK Taxonomy Fields
Version:5.x-1.1
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

It seems that we cannot see multi selectable filtering dropdown list at Filters section in "/admin/build/views/views_name/"
despite we can select multiple taxonomy terms from one vocab when creating/editing a node.

Forgive me if I'm saying the wrong thing as I'm not so familier with Views module for now.

#1

ryo - June 1, 2007 - 12:46

Correction. I meant
In "/admin/build/views/views_name/edit"

#2

robertDouglass - June 1, 2007 - 15:18

I think I know what you're talking about because I found this bug today, too. Will try to improve the views code for the next release.

#3

robertDouglass - June 26, 2007 - 22:04
Status:active» fixed

Yup. This was a bug and I've nailed it for 1.1

#4

ryo - June 27, 2007 - 02:28

Hmm....
I cannot confirm the fix but I might be doing something wrong.
Please give me sometime.

regards,

#5

robertDouglass - June 27, 2007 - 05:58

Make sure to clear your views cache. On fresh installation I'm able to filter on multiple select vocabs.

#6

ryo - June 27, 2007 - 07:57

I deleted my view, fresh installed CCK Taxonomy 1.1, cleared my view's cash and remade my view.
But still, there appears a single term (Value) selection Operator in "/admin/build/views/project/edit".
Please see attached image and give me any suggestion when you have time.

regards,

AttachmentSize
CCK_Taxonomy.jpg115.93 KB

#7

robertDouglass - June 27, 2007 - 10:54
Title:we cannot see multi selectable filtering dropdown list» Use filter handler that allows for multiple selection
Category:bug report» feature request
Status:fixed» active

Aaaah, that's what you mean! Pictures are very handy communication devices. That's by design. Perhaps there is different views filter handler that I should use, so I'm open to suggestions concerning that. But the software is doing what it is programmed to do, so I'm going to change this to a feature request for a different filter handler.

#8

robertDouglass - June 27, 2007 - 10:55
Version:5.x-1.0» 5.x-1.1

#9

ryo - June 27, 2007 - 11:27

OK, I got it.
What is important is not in filtering section itself, but in "Exposing the filter".
You'll get what I want to say by seeing the attached image.
Yeah, pictures are much better communication device than my poor English! :)

AttachmentSize
CCK_Taxonomy_2.jpg212.79 KB

#10

toma - July 4, 2007 - 23:33

The exposed filter doesn't work, you can't filter by taxonomy even if it works with some case, it doesn't filter correctly your result, its a great module and good idea, i change all my cck content to cck taxonomy and finally i discover this bug, so no filter exposed until it get corrected :p

Thanks for your great work

#11

regx - August 3, 2007 - 05:52

Basically to paraphrase, the last update fixed the list display issue in exposed views filters, where the taxonomy tree was being flattened to one level.

dogs
- poodle
- lab

was displayed as
dogs
poodle
lab

but it did not fix the multi-select issue for exposed filters.
If the taxonomy field has multi-select enables and the derived cck field has multi-select enabled, then the exposed filter should allow multiple selections. Currently the multi-select checkbox is selected automatically and greyed-out in views/exposed filters which is the correct behavior since the cck field has multi-select enabled, but the form does not allow multiple selections.

Seems like this would be a simple patch!

#12

regx - August 3, 2007 - 08:22
Category:feature request» bug report
Status:active» patch (code needs review)

Ok, here is a patch that fixes multi-select in exposed filters.

AttachmentSize
cck_taxonomy_multiselect.patch1.07 KB

#13

regx - August 3, 2007 - 08:25

The patch above is for version 5.x-1.1 , I should probably include the version in my patch names!

#14

robertDouglass - August 3, 2007 - 14:01

Thanks. Will evaluate shortly. Putting the version in your patch names isn't necessary because the issue already has a version selector. I just assume since this issue is against 1.1 that the patch will be too.

#15

ryo - August 21, 2007 - 00:59

I'm reporting that I tried regx's patch and it works quite fine.
Thanks regx.

Please apply it and commit an updated version.

regards,

#16

robertDouglass - August 21, 2007 - 10:26
Status:patch (code needs review)» fixed

Committed. Thanks for the patch and the review.

#17

Anonymous - September 4, 2007 - 10:32
Status:fixed» closed
 
 

Drupal is a registered trademark of Dries Buytaert.