ruby-on-rails – 为什么update_attributes会破坏我的Rails应用

我在Ruby 1.9.2上运行Rails 3.0.1.以下是相关的模型,控制器和视图.

码

user.rb：

class User < ActiveRecord::Base
  belongs_to :directory

  attr_accessor :new_password,:new_password_confirmation

  validates_confirmation_of :new_password,:if => :password_changed?

  before_save :hash_password,:if => :password_changed?

  def self.authenticate(login,password)

    # Check to see if the user exists
    if user = find_by_login(login)

      # If this is an directory user,authenticate them against their directory
      if user.directory
        return directory_auth user,password

      # Otherwise,authenticate them against the local database
      elsif user.hash == Digest::SHA2.hexdigest(user.salt + password)
        return user
      end
    end
    return nil
  end

  def password_changed?
    !@new_password.blank?
  end

  private

  def hash_password
    self.salt = ActiveSupport::SecureRandom.base64 8
    self.hash = Digest::SHA2.hexdigest(self.salt + @new_password)
  end

  def self.directory_auth(user,password)

    directory = user.directory
    directory.bind['%s'] = user.login

    ldap = Net::LDAP.new
    if directory.use_simple_tls?
      ldap.encryption :simple_tls
    end
    ldap.host = directory.host
    ldap.port = directory.port
    ldap.auth directory.bind,password

    return user if ldap.bind
    return nil
  end
end

users_controller.rb：

class UsersController < ApplicationController

  def index
    @users = User.all
  end

  def show
    @user = User.find params[:id]
  end

  def new
    @user = User.new
  end

  def create
    @user = User.new params[:user]

    if @user.save
      flash[:notice] = "#{@user.login} was created"
      redirect_to @user
    else
      flash[:notice] = 'The user could not be created' 
      render :action => 'new'
    end
  end

  def edit
    @user = User.find params[:id]
  end

  def update
    @user = User.find params[:id]

    @user.attributes = params[:user] # this works
    # @user.update_attributes params[:user] # this does NOT work

    if @user.save # I realize this is redundant if update_attributes is working
      flash[:notice] = "#{@user.login} was updated" 
      redirect_to @user
    else
      flash[:notice] = 'The user could not be updated' 
      render :action => 'edit'
    end
  end

  def destroy
    @user = User.find params[:id]
    @user.destroy

    flash[:notice] = "#{@user.login} was deleted"
    redirect_to users_url
  end
end

users.html.erb：

<%= form_for @user do |f| %>
  <%= f.label :login %>:
  <%= f.text_field :login %>
  <br>
  <%= f.label :new_password %>:
  <%= f.password_field :new_password %>
  <br>
  <%= f.label :new_password_confirmation %>:
  <%= f.password_field :new_password_confirmation %>
  <br>
  <% if directories = Directory.all.empty? %>
    No directories defined. You can <%= link_to 'add a directory',new_directory_path %>.
  <% else %>
    <%= f.label :directory_id %>:
    <%= f.collection_select :directory_id,Directory.all,:id,:name,{ :include_blank => 'None' } %>
  <% end %>
  <br>
  <%= f.label :admin,'Administrator?' %>:
  <%= f.check_box :admin %>
  <br>
  <%= f.submit %>
<% end %>

schema.rb：

ActiveRecord::Schema.define(:version => 20101107005603) do

  create_table "directories",:force => true do |t|
    t.string   "host"
    t.string   "bind"
    t.boolean  "use_simple_tls"
    t.integer  "port"
    t.string   "name"
    t.datetime "created_at"
    t.datetime "updated_at"
  end

  create_table "users",:force => true do |t|
    t.string   "login"
    t.string   "hash"
    t.string   "salt"
    t.boolean  "admin"
    t.integer  "directory_id"
    t.datetime "created_at"
    t.datetime "updated_at"
  end

end

我有另一个模型/控制器/视图的目录非常相似,但没有虚拟访问器或其他模型ID,并且update_attributes工作正常.我使用rails g scaffold用户名做了一个快速测试应用程序：字符串密码：字符串和所有CRUD操作都正常.

这让我疯了！我找到了一个解决方法,但我真的想了解为什么update_attributes在这里不起作用.当我运行更新操作时,我得到了这个：

UsersController#update中的TypeError

无法将nil转换为Integer

Rails.root：/ home / force / proj

app / controllers / users_controller.rb：34：在`update’中

全栈跟踪

activerecord (3.0.1) lib/active_record/connection_adapters/abstract/database_statements.rb:318:in `uniq'
activerecord (3.0.1) lib/active_record/connection_adapters/abstract/database_statements.rb:318:in `commit_transaction_records'
activerecord (3.0.1) lib/active_record/connection_adapters/abstract/database_statements.rb:165:in `transaction'
activerecord (3.0.1) lib/active_record/transactions.rb:204:in `transaction'
activerecord (3.0.1) lib/active_record/transactions.rb:287:in `with_transaction_returning_status'
activerecord (3.0.1) lib/active_record/persistence.rb:126:in `update_attributes'
app/controllers/users_controller.rb:34:in `update'
actionpack (3.0.1) lib/action_controller/metal/implicit_render.rb:4:in `send_action'
actionpack (3.0.1) lib/abstract_controller/base.rb:150:in `process_action'
actionpack (3.0.1) lib/action_controller/metal/rendering.rb:11:in `process_action'
actionpack (3.0.1) lib/abstract_controller/callbacks.rb:18:in `block in process_action'
activesupport (3.0.1) lib/active_support/callbacks.rb:435:in `_run__805567340__process_action__482539529__callbacks'
activesupport (3.0.1) lib/active_support/callbacks.rb:409:in `_run_process_action_callbacks'
activesupport (3.0.1) lib/active_support/callbacks.rb:93:in `run_callbacks'
actionpack (3.0.1) lib/abstract_controller/callbacks.rb:17:in `process_action'
actionpack (3.0.1) lib/action_controller/metal/instrumentation.rb:30:in `block in process_action'
activesupport (3.0.1) lib/active_support/notifications.rb:52:in `block in instrument'
activesupport (3.0.1) lib/active_support/notifications/instrumenter.rb:21:in `instrument'
activesupport (3.0.1) lib/active_support/notifications.rb:52:in `instrument'
actionpack (3.0.1) lib/action_controller/metal/instrumentation.rb:29:in `process_action'
actionpack (3.0.1) lib/action_controller/metal/rescue.rb:17:in `process_action'
actionpack (3.0.1) lib/abstract_controller/base.rb:119:in `process'
actionpack (3.0.1) lib/abstract_controller/rendering.rb:40:in `process'
actionpack (3.0.1) lib/action_controller/metal.rb:133:in `dispatch'
actionpack (3.0.1) lib/action_controller/metal/rack_delegation.rb:14:in `dispatch'
actionpack (3.0.1) lib/action_controller/metal.rb:173:in `block in action'
actionpack (3.0.1) lib/action_dispatch/routing/route_set.rb:62:in `call'
actionpack (3.0.1) lib/action_dispatch/routing/route_set.rb:62:in `dispatch'
actionpack (3.0.1) lib/action_dispatch/routing/route_set.rb:27:in `call'
rack-mount (0.6.13) lib/rack/mount/route_set.rb:148:in `block in call'
rack-mount (0.6.13) lib/rack/mount/code_generation.rb:93:in `block in recognize'
rack-mount (0.6.13) lib/rack/mount/code_generation.rb:103:in `optimized_each'
rack-mount (0.6.13) lib/rack/mount/code_generation.rb:92:in `recognize'
rack-mount (0.6.13) lib/rack/mount/route_set.rb:139:in `call'
actionpack (3.0.1) lib/action_dispatch/routing/route_set.rb:492:in `call'
actionpack (3.0.1) lib/action_dispatch/middleware/best_standards_support.rb:17:in `call'
actionpack (3.0.1) lib/action_dispatch/middleware/head.rb:14:in `call'
rack (1.2.1) lib/rack/methodoverride.rb:24:in `call'
actionpack (3.0.1) lib/action_dispatch/middleware/params_parser.rb:21:in `call'
actionpack (3.0.1) lib/action_dispatch/middleware/flash.rb:182:in `call'
actionpack (3.0.1) lib/action_dispatch/middleware/session/abstract_store.rb:149:in `call'
actionpack (3.0.1) lib/action_dispatch/middleware/cookies.rb:287:in `call'
activerecord (3.0.1) lib/active_record/query_cache.rb:32:in `block in call'
activerecord (3.0.1) lib/active_record/connection_adapters/abstract/query_cache.rb:28:in `cache'
activerecord (3.0.1) lib/active_record/query_cache.rb:12:in `cache'
activerecord (3.0.1) lib/active_record/query_cache.rb:31:in `call'
activerecord (3.0.1) lib/active_record/connection_adapters/abstract/connection_pool.rb:355:in `call'
actionpack (3.0.1) lib/action_dispatch/middleware/callbacks.rb:46:in `block in call'
activesupport (3.0.1) lib/active_support/callbacks.rb:415:in `_run_call_callbacks'
actionpack (3.0.1) lib/action_dispatch/middleware/callbacks.rb:44:in `call'
rack (1.2.1) lib/rack/sendfile.rb:107:in `call'
actionpack (3.0.1) lib/action_dispatch/middleware/remote_ip.rb:48:in `call'
actionpack (3.0.1) lib/action_dispatch/middleware/show_exceptions.rb:46:in `call'
railties (3.0.1) lib/rails/rack/logger.rb:13:in `call'
rack (1.2.1) lib/rack/runtime.rb:17:in `call'
activesupport (3.0.1) lib/active_support/cache/strategy/local_cache.rb:72:in `call'
rack (1.2.1) lib/rack/lock.rb:11:in `block in call'
:10:in `synchronize'
rack (1.2.1) lib/rack/lock.rb:11:in `call'
actionpack (3.0.1) lib/action_dispatch/middleware/static.rb:30:in `call'
railties (3.0.1) lib/rails/application.rb:168:in `call'
railties (3.0.1) lib/rails/application.rb:77:in `method_missing'
railties (3.0.1) lib/rails/rack/log_tailer.rb:14:in `call'
rack (1.2.1) lib/rack/content_length.rb:13:in `call'
rack (1.2.1) lib/rack/handler/webrick.rb:52:in `service'
/home/force/.rvm/rubies/ruby-1.9.2-p0/lib/ruby/1.9.1/webrick/httpserver.rb:111:in `service'
/home/force/.rvm/rubies/ruby-1.9.2-p0/lib/ruby/1.9.1/webrick/httpserver.rb:70:in `run'
/home/force/.rvm/rubies/ruby-1.9.2-p0/lib/ruby/1.9.1/webrick/server.rb:183:in `block in start_thread'

请求参数

{"utf8"=>"?","_method"=>"put","authenticity_token"=>"OvMUeM9hqfPASC0NS+Th07GELu6B+dQCCTtm3gWdJE4=","user"=>{"login"=>"local","new_password"=>"[FILTERED]","new_password_confirmation"=>"[FILTERED]","directory_id"=>"","admin"=>"0"},"commit"=>"Update User","id"=>"13"}

完整源代码

如果您想尝试一下,可以在http://github.com/sidewaysmilk/deezy下载完整的源代码并编辑app / controllers / users_controller.rb以在更新操作中使用@ user.update_attributes params [：user].