Preserve stats despite auth

danneu · Jan 15, 2015 · 6cd05d9 · 6cd05d9
1 parent a7b147f
commit 6cd05d9
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 10 deletions.
diff --git a/server/index.js b/server/index.js
@@ -389,9 +389,6 @@ app.use(route.get('/', function*() {
 
   // Get stats
   var stats = cache.get('stats');
-  stats.onlineUsers = stats.onlineUsers.filter(function(user) {
-    return cancan.can(this.currUser, 'READ_USER_ONLINE_STATUS', user);
-  }, this);
   stats.onlineUsers = stats.onlineUsers.map(pre.presentUser);
   if (stats.latestUser)
     stats.latestUser = pre.presentUser(stats.latestUser);

diff --git a/views/homepage.html b/views/homepage.html
@@ -96,13 +96,15 @@ <h5 class="list-group-item-heading forum-item-title">
         <div class="col-xs-12">
           <h4>{{ stats.onlineUsers.length }} Users Online (within 15 min)</h4>
           {% for user in stats.onlineUsers %}
-            {% if user.is_ghost %}
-              <a href="{{ user.url }}" style="color: #000">{{ user.uname }}</a>
-            {% else %}
-              <a href="{{ user.url }}" style="color: #999">{{ user.uname }}</a>
-            {% endif %}
-            {% if !loop.last %}
-              ,
+            {% if ctx.can(ctx.currUser, 'READ_USER_ONLINE_STATUS', user) %}
+              {% if user.is_ghost %}
+                <a href="{{ user.url }}" style="color: #000">{{ user.uname }}</a>
+              {% else %}
+                <a href="{{ user.url }}" style="color: #999">{{ user.uname }}</a>
+              {% endif %}
+              {% if !loop.last %}
+                ,
+              {% endif %}
             {% endif %}
           {% endfor %}
         </div>