The Staff Online Now Block

[Brandon_R]

Is that statically coded in there or is it a *widget*. Also what about the online block. Can we add out or blocks?

 

[Michael Biddle]

Judging by the html if you change:

<xen:if is="{$user.is_moderator} OR {$user.is_admin}">

to

<xen:if is="{$user.is_admin}">

you should be set

 

[Kier]

100% correct.

 

[Michael Biddle]

YES!

 

[Erik]

<xen:if hascontent="true">
    <div class="section staffOnline avatarList">
        <div class="secondaryContent">
            <h3>Staff Online Now</h3>
            <ul>
                <xen:contentcheck>
                    <xen:foreach loop="$onlineUsers.records" value="$user">
                        <xen:if is="{$user.is_moderator} OR {$user.is_admin}">
                            <li>
                                <xen:avatar user="$user" size="s" img="true" />
                                <a href="{xen:link members, $user}" class="username">{$user.username}</a>
                                <div class="muted">{xen:helper userTitle, $user}</div>
                            </li>
                        </xen:if>
                    </xen:foreach>
                </xen:contentcheck>
            </ul>
        </div>
    </div>
</xen:if>

A first glimpse of XenForo template syntax for you...

Correct me if I'm wrong, but from the code it looks like it's looping through the entire online users table and checking if each user is a mod or admin.

Doesn't this seem like quite a lot of overhead to be running on each pageload? What happens when you get into the thousands of users (online)?

I should be clear that I do love the template syntax here, I'm just worried about the scalability of this particular piece.

 

[Brandon_R]

I think i saw in another post around the forums that they intend on adding caches so this could be one of them.

 

[Kier]

It's looping through users online ($onlineUsers.records) not the complete user table.

And yes, this is a prime candidate to be cached.

 

[Mike]

It's also a micro optimization. You really wouldn't see any difference, even if there were 1000 users online.

 

[Brandon_R]

Judging from the fact that you named your variables $onlineUsers, you guys have adopted some kind of coding standard like camelcase?

Which one is better

$isModerator();

or

$is_moderator();

 

[Kier]

Answered here:

http://xenforo.com/community/threads/random-questions-answered.114/#post-6838

 

[lazer]

We have 31 staff, of which maybe 10 are online at any one time. Just looking at the code that Kier posted, it seems it's possible to remove the avatars - that would enable the info to be displayed in more of a list format, which would work for us

 

[Martyn]

why not make it pop up a little window like the contact card? showing all online staff??