Highlight Content per Usergroup 2023-04-19

[Deleted member 184953]

Thanks @Ivancas
You've pointed something out, for a few posts now, that helps avoid these repeated errors.

This is what I understand now: my conditional statement is not wrong, it works fine and does what it is told. However, it triggers template errors in a special case, when an user concerned by one of the selected user group is deleted (or a guest but this parameter is not taken into account here).

This is also why your error message states:

[*]Generated by: Unknown account

To avoid this you have to add a add a second condition element in the conditional statement $post.User. This is confirmed by this message from @Xon, which is almost 5 years old! As you wrote previously the right code should be:

{{ $post.User AND $post.User.isMemberOf('X') ? 'class-name' : '' }}

I will test this locally and I will publish a new version as soon as I have validated this fix.

 

[Deleted member 184953]

This doesn't seem to work, the class is not applied with this syntax.
@Xon is there anything that would have changed since those years? Thanks.

 

[Xon]

You need to wrap the conditions in a pair of brackets like this:

{{ ($post.User AND $post.User.isMemberOf('X')) ? 'class-name' : '' }}

 

[Deleted member 184953]

It was

{{ $post.User AND $post.User.isMemberOf('X') ? 'class-name' : '' }}

Even like this (as you wrote) it doesn't work

{{ ($post.User AND $post.User.isMemberOf('X')) ? 'class-name' : '' }}

---

This how i used the template modification (post_macros):

Find:

<div class="message-cell message-cell--user">

Replace:

<div class="message-cell message-cell--user {{ $post.User.isMemberOf($xf.options.userGroup1) ? 'ucc-usergroup1Posts' : '' }}{{ $post.User.isMemberOf($xf.options.userGroup2) ? ' ucc-usergroup2Posts' : '' }}{{ $post.User.isMemberOf($xf.options.userGroup3) ? ' ucc-usergroup3Posts' : '' }}{{ $post.User.isMemberOf($xf.options.userGroup4) ? ' ucc-usergroup4Posts' : '' }}{{ ($post.USer AND $post.User.is_staff) ? ' ucc-staffPosts' : '' }}{{ $post.User.is_banned ? ' ucc-bannedPosts' : '' }}">

It works fine except that generates template errors when a deleted user is involved (i think) (regarding what you posted)

If the Replace code is:

<div class="message-cell message-cell--user {{ $post.User AND $post.User.isMemberOf($xf.options.userGroup1) ? 'ucc-usergroup1Posts' : '' }}{{ $post.User AND $post.User.isMemberOf($xf.options.userGroup2) ? ' ucc-usergroup2Posts' : '' }}{{ $post.User AND $post.User.isMemberOf($xf.options.userGroup3) ? ' ucc-usergroup3Posts' : '' }}{{ $post.User AND $post.User.isMemberOf($xf.options.userGroup4) ? ' ucc-usergroup4Posts' : '' }}{{ ($post.User AND $post.User.is_staff) ? ' ucc-staffPosts' : '' }}{{ $post.User AND $post.User.is_banned ? ' ucc-bannedPosts' : '' }}">

It doesn't work at all.

 

[Xon]

The XF templates behaves very oddly with the ? : operator.

When you have;
{{ $post.User AND $post.User.isMemberOf('X') ? 'class-name' : '' }}
What XenForo renders it as is;
{{ $post.User AND ($post.User.isMemberOf('X') ? 'class-name' : '') }}
Which absolutely isn't what you want

 

[Deleted member 184953]

You need to wrap the conditions in a pair of brackets like this:

{{ ($post.User AND $post.User.isMemberOf('X')) ? 'class-name' : '' }}

I must have made a typo because this code actually works perfectly. Thank you so much.

When you have;
{{ $post.User AND $post.User.isMemberOf('X') ? 'class-name' : '' }}
What XenForo renders it as is;
{{ $post.User AND ($post.User.isMemberOf('X') ? 'class-name' : '') }}
Which absolutely isn't what you want

I fully understand the difference, thank you for this explanation, really.

 

[Deleted member 184953]

Nicolas FR updated Highlight Content per Usergroup with a new update entry:

Bug fix

Fixed a bug in conditional statements that triggered template errors when a highlighted message belonged to a deleted user.

Thanks to @Ivancas and @Xon for their help.

Read the rest of this update entry...

 

[Wing]

Giving this mod a try, any way to add more than 4 usergroup options?

 

[Deleted member 184953]

How much is right for you? Basically I didn't want to offer too many different user groups so that the discussion pages wouldn't turn into curious colored Christmas trees . Would 6 groups be a good compromise?

 

[Wing]

Haha yes! That would be perfect imo

 

[KSA]

Thanks a lot Nicolas, a must have add-on really. Is there a chance to make this user-optional where user can choose if they want to highlight or not. I need this for my moderator group so they can highlight certain moderation-related posts when needed.

Thanks again!

 

[Ivancas]

Nicolas FR updated Highlight Content per Usergroup with a new update entry:

Bug fix



Read the rest of this update entry...

Thanks,
However, the addThreadsOptionsColor still has the old syntax:

    <div class="structItem structItem--thread{{ $thread.prefix_id ? ' is-prefix' . $thread.prefix_id : '' }}{{ $thread.isIgnored() ? ' is-ignored' : '' }}{{ ($thread.isUnread() AND !$forceRead) ? ' is-unread' : '' }}{{ $thread.discussion_state == 'moderated' ? ' is-moderated' : '' }}{{ $thread.discussion_state == 'deleted' ? ' is-deleted' : '' }} js-inlineModContainer js-threadListItem-{$thread.thread_id} {{ $thread.User.isMemberOf($xf.options.userGroup1) ? 'ucc-usergroup1Threads' : '' }}{{ $thread.User.isMemberOf($xf.options.userGroup2) ? ' ucc-usergroup2Threads' : '' }}{{ $thread.User.isMemberOf($xf.options.userGroup3) ? ' ucc-usergroup3Threads' : '' }}{{ $thread.User.isMemberOf($xf.options.userGroup4) ? ' ucc-usergroup4Threads' : '' }}{{ $thread.User.is_staff ? ' ucc-staffThreads' : '' }}{{ $thread.User.is_banned ? ' ucc-bannedThreads' : '' }}" data-author="{{ $thread.User.username ?: $thread.username }}">

It is still needing

$thread.User AND

at the beginning of each conditional

 

[beerForo]

Nice idea to be able to highlight paid groups especially but useful for staff and other as well.

 

[Deleted member 184953]

Nicolas FR updated Highlight Content per Usergroup with a new update entry:

Adding a conditional statement:

Adding a conditional statement:

• The content of banned members is now visible only to staff members
• Modifying conditional statements in the thread_list_macros template

Read the rest of this update entry...

 

[D.O.A.]

• Usergroup highlighting can now be enabled/disabled according to forums (nodes)

is this function available? I cant seem to find the option for it. ty.

 

[Deleted member 184953]

Hello, this function is no more available.

 

[beerForo]

To avoid confusion you could add feature removals to your release notes.

 

[Deleted member 184953]

@beerForo that's what i did here

 

[beerForo]

I see what you mean but it's confusing. Instead of saying "Remove node conditional option causing error." You could say "Removed the function of highlighting usergroup content per node." Otherwise it sounds like a bug fix.

 

[Deleted member 184953]

Done. Thanks.