Rails習作 ログイン一覧作った

create table logon_users (
id                  int             not null    auto_increment,
user_id             int             null,
updated_at          datetime        not null,
constraint          fk_user_id      foreign key(user_id) references users(id),
primary key(id)
);

ruby script/generate model LogonUser

class User < ActiveRecord::Base
has_one :logon_user
(略)

class LogonUser < ActiveRecord::Base
belongs_to :user
end

  def login
if request.get?
session[:user_id] = nil
@user = User.new
else
@user = User.new(params[:user])
logged_in_user = @user.try_to_login
if logged_in_user
session[:user_id] = logged_in_user.id
session[:name] = logged_in_user.name
        logon_user = LogonUser.find_by_user_id( logged_in_user.id ) ||
LogonUser.new( :user => logged_in_user  )
logged_in_user.logon_user = logon_user
redirect_to(:controller => "admin", :action => "index" )
else
flash[:notice] = "ユーザ/パスワードの組み合わせが無効です"
end
end
end

    LogonUser.destroy_all( ["user_id = ?", session[:user_id]] )
session[:user_id] = nil
flash[:notice] = "ログアウトしました"
redirect_to(:action => "login")

  def checklogonuser
    logon_user = LogonUser.find_by_user_id( session[:user_id] )
logon_user.save
@users = LogonUser.find( :all,
:order => "user_id" )
render(:layout => false)
end

<div id="userlist">
<%= render :partial => 'userlist' %>
</div>
<%= periodically_call_remote(:update => 'userlist',
:url => { :action => :checklogonuser },
:frequency => 30 ) %>

<%= render :partial => 'userlist' %>

参加者:
<% for user in @users %>
<%=h user.user.name %>　
<% end %>

  def checklogonuser
logon_user = LogonUser.find_by_user_id( session[:user_id] )
logon_user.save
    @users = LogonUser.find( :all,
:order => "user_id" )
render(:layout => false)
end