当我点击登录链接时,它应该检查有效的用户名和密码,并在登录部分页面上显示该消息,我在彩盒中作为弹出窗口调用,但我无法显示错误消息。
这是我的loginController:
def login
if request.post?
@user = User.new(params[:user])
user=User.authenticate(params[:username], params[:password])
if user.nil?
flash[:error] = 'Login unsuccessful'
redirect_to :back
else
flash[:notice] = 'Login successful'
session[:user] = user.id
redirect_to :controller=>'users', :action => 'show',:username=>user.username
end
end
end
User.rb:
def self.authenticate(username, pass)
user = find(:first, :conditions=>["username = ?", username])
return nil if user.nil?
return user if User.hash_password(pass) == user.password && username == user.username
nil
end
在我看来page_sign_in.html.erb
页面
<%= form_for @user, :url=>{:controller=>"logins",:action=>"login"},:remote=>true,:html=>{ :onSubmit => "return checkSignupValidation()",:id=>"signin_form", :multipart=>true} do |f| %>
@user
给出错误:undefined method
model_name'代表Nilclass`
答案 0 :(得分:0)
您的问题是登录没有模型 - 没有必要。您不想使用form_for
,而是使用form_tag
:
<%= form_tag controller: "logins", action: "login", etc.. do %>
<%= text_field_tag :username %>
<%= password_field_tag :password %>
<% end %>
然后我会向你的控制器推荐一些改进:
def login
if request.post?
user = User.authenticate(params[:username], params[:password])
if user.nil?
flash[:error] = 'Login unsuccessful'
redirect_to :back #I try to avoid :back, it feels better to use a fixed path
else
flash[:notice] = 'Login successful'
session[:user] = user.id
redirect_to :controller=>'users', :action => 'show', :username=>user.username
end
end
end
您不需要@user变量,因此只需避免不必要的数据库查询。我不知道它是否如此好用:回来,我总是尝试使用固定的路径。