From b276d938220c12101e2d825228ddc6e8ff030114 Mon Sep 17 00:00:00 2001
From: jomo <github@jomo.tv>
Date: Sun, 29 Jun 2014 06:45:00 +0200
Subject: [PATCH] better online user list

---
 app/models/user.rb                 | 6 +++++-
 app/views/layouts/_footer.html.erb | 2 +-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/app/models/user.rb b/app/models/user.rb
index c969a08..1026eb0 100644
--- a/app/models/user.rb
+++ b/app/models/user.rb
@@ -39,8 +39,12 @@ class User < ActiveRecord::Base
     !!self.confirmed
   end
 
+  def self.seen(time)
+    # when you change this, change footer.html.erb as well?
+    User.where("last_seen >= ?", time.ago).order("last_seen, desc")
+  end
+
   def online?
-    # when you change this, change footer.html.erb as well!
     last_seen && last_seen > 2.minutes.ago
   end
 
diff --git a/app/views/layouts/_footer.html.erb b/app/views/layouts/_footer.html.erb
index d8eb269..02e30c9 100644
--- a/app/views/layouts/_footer.html.erb
+++ b/app/views/layouts/_footer.html.erb
@@ -1,4 +1,4 @@
-<% users = User.select{|u| u.online?} %>
+<% users = User.seen(2.minutes) %>
 <div id="footer">
   <div id="online-users">
     <p>Users seen within the last two minutes: <%= users.length %></p>