I wanted to add a contextual filter on a view using the "Content Type" field. Using a Filter is possible, but using a Contextual Filter seemed to be missing.

Attached a patch, that adds the missing part.

CommentFileSizeAuthor
#3 2844527-3--add_contextual_filter_definitions.patch1.28 KBdrunken monkey
#3 2844527-3--add_contextual_filter_definitions--interdiff.txt988 bytesdrunken monkey
allow-contextual-filter-for-content-type.patch545 bytesayalon

Comments

ayalon created an issue. See original summary.

ayalon’s picture

ayalon commented
drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Component: General code » Views integration
Category: Bug report » Feature request
Status: Active » Needs review
StatusFileSize
new 2844527-3--add_contextual_filter_definitions--interdiff.txt988 bytes
new 2844527-3--add_contextual_filter_definitions.patch1.28 KB

Thanks a lot for reporting this!
You're right, no reason not to just use our default contextual filter plugin for content types, too. Also seems to work fine in practice.
However, looking at that function, it seems we can add that plugin for some other of our types, too. Really, the only one where the normal plugin won't really work is date, I think.
Please test/review the attached patch and I can commit it!

ayalon’s picture

ayalon commented

Thx for the patch, works as expected.

ayalon’s picture

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

  • drunken monkey committed 47500fa on 8.x-1.x authored by ayalon 
    Issue #2844527 by ayalon, drunken monkey: Added some contextual filters...
drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Reviewed & tested by the community » Fixed

Good to hear, thanks for testing!
Committed.
Thanks again for your report and patch!

Status: Fixed » Closed (fixed)

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