implementation for #1469

app/controllers/forum_topics_controller.rb

@@ -2,7 +2,6 @@ class ForumTopicsController < ApplicationController
   respond_to :html, :xml, :json
   before_filter :member_only, :except => [:index, :show]
   before_filter :normalize_search, :only => :index
-  after_filter :update_last_forum_read_at, :only => [:show]
 
   def new
     @forum_topic = ForumTopic.new
@@ -19,6 +18,7 @@ class ForumTopicsController < ApplicationController
   def index
     @query = ForumTopic.active.search(params[:search])
     @forum_topics = @query.order("is_sticky DESC, updated_at DESC").paginate(params[:page], :limit => params[:limit], :search_count => params[:search])
+    @read_forum_topic_ids = read_forum_topic_ids
     respond_with(@forum_topics) do |format|
       format.xml do
         render :xml => @forum_topics.to_xml(:root => "forum-topics")
@@ -31,6 +31,7 @@ class ForumTopicsController < ApplicationController
     @forum_posts = ForumPost.search(:topic_id => @forum_topic.id).order("forum_posts.id").paginate(params[:page])
     @forum_posts.all
     respond_with(@forum_topic)
+    session[:read_forum_topics] = @forum_topic.mark_as_read(read_forum_topic_ids)
   end
 
   def create
@@ -67,14 +68,6 @@ class ForumTopicsController < ApplicationController
   end
 
 private
-  def update_last_forum_read_at
-    return if CurrentUser.is_anonymous?
-
-    if CurrentUser.last_forum_read_at.nil? || CurrentUser.last_forum_read_at < @forum_topic.updated_at
-      CurrentUser.update_column(:last_forum_read_at, @forum_topic.updated_at)
-    end
-  end
-
   def normalize_search
     if params[:title_matches]
       params[:search] ||= {}
@@ -87,6 +80,14 @@ private
     end
   end
 
+  def read_forum_topics
+    session[:read_forum_topics].to_s
+  end
+
+  def read_forum_topic_ids
+    read_forum_topics.scan(/\S+/)
+  end
+
   def check_privilege(forum_topic)
     if !forum_topic.editable_by?(CurrentUser.user)
       raise User::PrivilegeError