0

当我单击登录链接时，它应该检查有效的用户名和密码并在登录部分页面上显示消息，我在颜色框中将其作为弹出窗口调用，但我无法显示错误消息。

这是我的登录控制器：

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

用户.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 methodNilclass`的model_name'

4

1 回答 1

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 变量，因此只需避免不必要的数据库查询。不知道用 :back 好不好用，我总是尝试使用固定路径。

于 2012-04-20T11:00:15.477 回答