XF 2.0 Adding class to thread item based on node ID and thread count

[Case]

Trying to add a class to an existing div in the thread_list_macros template by using {{ $forum.node_id == 34 AND $thread.reply_count > 1 ? 'extra-class' : '' }}

The node_id part isn't working though. Is this possible?

 

[Chris D]

The thread_list_macros template and item macro is called in a few places -- where are you testing this condition? In most places, the $forum arg is passed in so it should be available.

Does this work any better?

{{ ($forum.node_id == 34 AND $thread.reply_count > 1) ? 'extra-class' : '' }}

 

[Case]

Yep, that's sorted it. I was missing the brackets. Thanks, Chris.

 

[Chris D]

FWIW you'd probably might as well switch $forum.node_id with $thread.node_id. The node_id value is always available on the thread entity and the thread arg is always passed into this macro. However, there are some lists of threads where we don't pass the $forum arg in.

 

[Case]

FWIW you'd probably might as well switch $forum.node_id with $thread.node_id. The node_id value is always available on the thread entity and the thread arg is always passed into this macro. However, there are some lists of threads where we don't pass the $forum arg in.

I seem to be getting server errors with this code now.

ErrorException: Template error: Illegal string offset 'node_id'
internal_data/code_cache/templates/l1/s24/public/thread_list_macros.php:302

Did something change?

 

[Chris D]

Nope should still work.

 

[Joe Link]

@Chris D is there a way to do this based on category?