For this version the filter block display all content when the node without the terms filtered. Is it possible the block display nothing and not to show this block in this case?

CommentFileSizeAuthor
#5 1169366-patch-add-require-terms-on-node-option-3.patch1.83 KBjec006
#4 1169366-patch-add-require-terms-on-node-option-2.patch1.84 KBjec006
#3 1169366-patch-add-require-terms-on-node-option.patch1.47 KBjec006
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tirdadc’s picture

tirdadc CreditAttribution: tirdadc commented
Assigned: Unassigned » tirdadc

Sounds like a good idea, I'll look into it.

astra’s picture

astra CreditAttribution: astra commented

Just an idea: when the node without the terms filtered, maybe options can be configured, something like

1) display all other content
2) display nothing

jec006’s picture

jec006 CreditAttribution: jec006 commented
Status: Active » Needs review
FileSize
1169366-patch-add-require-terms-on-node-option.patch1.47 KB

I've made a patch that will simply show nothing if the node has no terms for valid vocabs.

The display all content is the what current happens, so I simply added a checkbox for showing nothing.

Attached Patch

jec006’s picture

jec006 CreditAttribution: jec006 commented
FileSize
1169366-patch-add-require-terms-on-node-option-2.patch1.84 KB

Cleaned up some Admin stuff - added a default - rerolled patch

jec006’s picture

jec006 CreditAttribution: jec006 commented
FileSize
1169366-patch-add-require-terms-on-node-option-3.patch1.83 KB

Hey,

Adding a version that will work with older versions of drush make (no a/ b/ prefixes) and removing a couple whitespace errors.

This patch seems to be working well after about a month of use.

tirdadc’s picture

tirdadc CreditAttribution: tirdadc commented
Status: Needs review » Fixed

Works well, thanks for that. Committed to dev.

Status: Fixed » Closed (fixed)

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