To allow multiple task handlers for a vocabulary context it looks to me like an access plug-in similar to ctools/plugins/access/term_vocabulary.inc is needed. The required context would be vocabulary.

Similar functionality was available in panels 1 and 2.

It would also help to have an argument placeholder in ctools/plugin/arguments/vid.inc for vocabulary preview pages.

CommentFileSizeAuthor
#6 467948_vocabulary_context_access_1.patch1.2 KBjolidog
#5 467948_vocabulary_context_access.patch731 byteshefox
#4 467948_vocabulary_context_access.patch730 byteshefox

Comments

merlinofchaos’s picture

merlinofchaos commented

I feel like you started this issue in the middle of a thought, because I can't quite figure out what you're asking for.

chaps2’s picture

chaps2 commented

My requirement is a custom page with a path like taxonomy/vocabulary/%vocabulary and multiple panels handling 1 or more vocabularies each.

I couldn't see that this was supported and looking at the code it looks to me like an extra vocabulary access plug-in is what is required. I may very well be wrong.

I can post a patch if this is a valid feature request.

merlinofchaos’s picture

merlinofchaos commented

Oh ok. That makes sense. Yes, it should not be too difficult of a patch if you understand the access plugins. I would welcome that.

hefox’s picture

hefox commented
StatusFileSize
new 467948_vocabulary_context_access.patch730 bytes

There's 2 way to do this as far as I've seen to do this

  • Add a new vocabulary.inc for access, near exact copy of term_vocabulary. This would look like this http://github.com/hefox/vocabulary/blob/master/plugins/access/vocabulary... (I can't figure out how to do a diff that includes new files in git yet)
  • Add vocabulary to the choices of term_vocabulary.inc. The problems with this is the text do not make sense since it refers to terms in the vocabulary.. but less repeative code, which I like (I realized this could be done after doing the above).

Patch attached is for the second way.

(Btw to the original poster, my new vocabulary module provides taxonomy/vid; contemplated doing it taxonomy/vocabulary/vid but for whatever reason, mostly cause I thought vocab pages url should be shorter, went with taxonomy/vid).

hefox’s picture

hefox commented
StatusFileSize
new 467948_vocabulary_context_access.patch731 bytes

Forgot a space after comma

(Went to view the patch to make sure it was the right one (managed to post 2 wrong patches today))

jolidog’s picture

jolidog commented
Status: Active » Needs review
StatusFileSize
new 467948_vocabulary_context_access_1.patch1.2 KB

I've tested the attached patch and I think it's working. When using the vocabulary module from hefox, I'm now able to use the taxonomy:vocabulary in the selection rules of panels.

The text in the popup window should be referring to the vocabularies and not the terms in them, even though the code is dealing with the terms. Am I wrong in assuming that?
I've attached a patch that reflects this assumption and also includes hefox patch.

Thank you!

merlinofchaos’s picture

merlinofchaos commented
Status: Needs review » Reviewed & tested by the community

I agree with #6.

merlinofchaos’s picture

merlinofchaos commented
Status: Reviewed & tested by the community » Fixed

Thanks. #6 committed!

jolidog’s picture

jolidog commented

Credits go to hefox, I just changed the sentences!

And thank you merlinofchaos for another amazing module!

Status: Fixed » Closed (fixed)

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