Is `topic_users.last_visited_at` supposed to be updated on every visit?

sam · July 25, 2016, 6:47am

Well actually…

app/controllers/topics_controller.rb

main


      
          topic_id = params[:topic_id].to_i
          enabled = ActiveModel::Type::Boolean.new.cast(params[:enabled])
          
          check_for_status_presence(:status, status)
          @topic =
            if params[:category_id]
              Topic.find_by(id: topic_id, category_id: params[:category_id].to_i)
            else
              Topic.find_by(id: topic_id)
            end
          
          status_opts = { until: params[:until].presence }
          
          if status == "visible"
            status_opts[:visibility_reason_id] = (
              if enabled
                Topic.visibility_reasons[:manually_relisted]
              else
                Topic.visibility_reasons[:manually_unlisted]
              end
            )

This seems to be working fine for me.

The null rows will usually be users who never really visited and instead got a row there cause they were watching a category or had mailing list mode enabled. As soon as we send a mail out to a user we add a row in topic users.

Topic		Replies	Views
NULL first_visited_at dates in the topic_users table Development	5	2883	July 2, 2016
Last visit doesn't work for fixed topic list Bug	2	730	February 2, 2019
Already seen posts appearing above Last Visit line? Bug	5	883	January 9, 2025
Mysterious entries in topic_users table Bug	22	3969	May 16, 2016
Request from API doesn't update User#last_seen_at Product Support	8	989	April 15, 2019

Is `topic_users.last_visited_at` supposed to be updated on every visit?

Related topics