See the screenshot for what I mean.

If I choose "only nodes of type blog" I should not be allowed to select "And of type book". No node is an image AND a book page.

CommentFileSizeAuthor
#2 node_filter_by_type_once_0.patch653 bytesKjartan
#1 node_filter_by_type_once.patch587 bytesedmund.kwok
node_filter_oddity.png31.95 KBBèr Kessels

Comments

edmund.kwok’s picture

edmund.kwok commented
Status: Active » Needs review
StatusFileSize
new node_filter_by_type_once.patch587 bytes

In that sense, any node can't be any two types at the same time yeah? This patch checks to see if nodes are already filtered by type, if so, the type filter is unset.

Kjartan’s picture

Kjartan commented
Version: x.y.z » 5.0-beta1
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new node_filter_by_type_once_0.patch653 bytes

Tested the patch and confirmed that it doesn't let you select multiple node types for filtering.

Attached is a clean diff against current HEAD.

dries’s picture

dries commented

Looks good. I'd add a code comment though. Not necessary but recommended.

drumm’s picture

drumm
he/him
Location NY, US
commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD with a comment.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)