2010-11-09 84 views
4

原问题

我在Ruby 1.9.2上运行Rails 3.0.1。这里是相关的模型,控制器和视图。为什么update_attributes会破坏我的Rails应用程序?

代码

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? 
    [email protected]_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 users name:string password:string和所有CRUD操作正常。

这让我疯狂!我找到了一个解决方法,但我真的很想理解为什么update_attributes在这里不起作用。当我运行更新的动作,我得到这个:

类型错误在UsersController#更新

不能转换成零整数

Rails.root:/家庭/力量/凸出

应用/controllers/users_controller.rb:34:in'更新”

完整的堆栈跟踪

 
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.rbupdate行动使用@user.update_attributes params[:user]

+0

您的模式对于您的用户模型是什么样的? – raidfive 2010-11-09 00:44:36

+0

添加到问题。谢谢。 – 2010-11-09 01:11:37

+0

您可以添加什么时候提交表单的参数吗? (来自服务器日志) – monocle 2010-11-09 02:37:04

回答

5

好的。我感觉真的很愚蠢。我很惊讶没有人抓到这个。

在我的模型中,我命名了我的一个属性。 hash,所以要访问它,我会说@user.hash

ActiveRecord :: Base#hash是already defined

所以我搞砸了。当ActiveRecord的被试图执行该交易,它试图设置一个值等

@user.hash = password_hash 

ActiveRecord::Base#hash=期望的整数,并且password_hash输出如果密码被改变字符串,和否则无。

所以永远不要命名列哈希!当您选择列名以避免冲突时请检查文档。

+1

保存我的一天! '哈希'现在在我的心理黑名单上的任何类型的变量或字段:) – 2010-12-16 16:14:36

+1

同样问题在这里,投票了! – 2012-02-10 09:53:15

1

如果没有完整的堆栈跟踪,很难说,但错误消息“无法将nil转换为整数”是追踪的关键。同样,我们希望查看失败请求的日志。我认为日志的参数:行可能会说明问题。

更新下面:

你能顺利通过相同PARAMS为@ user.update_attributes在轨控制台?

rails console 

user = User.find(42) # whatever a good test user's id is 
user_params = {"login"=>"local", "new_password"=>"supersecret", "new_password_confirmation"=>"supersecret", "directory_id"=>"", "admin"=>"0"} 
user.update_attributes!(user_params) 

这种行为有什么不同吗?

+0

当然。对不起。当我发布这个问题时,我非常疲惫。我应该包含堆栈跟踪。我现在就更新它。 – 2010-11-09 17:06:42

+0

我应该首先提到使用堆栈跟踪,我能够跟踪update_attributes的问题,但除此之外我无法得到任何进一步的信息。我不知道所谓的'nil'是什么。我想这可能与我的'attr_accessor'有关,因为如果我设置了密码,它开始说它不能将** String **(而不是nil)转换为Integer。 – 2010-11-09 17:18:10

+1

我克隆了github的源代码,并将用户控制器的更新操作更改为使用update_attributes,并且对我来说工作正常。这是与Ruby 1.8.7。 – jwarchol 2010-11-09 21:16:58

1

它看起来像什么可能是问题是,用户ID是一个主键,并以某种方式试图更新。如果您使用'bang version of update_attributes : update_attributes!`,您会得到什么错误消息?

+0

如果我使用'update_attributes!',我会得到完全相同的行为。我会看看我可以找出有关正在更新的@ user.id。这是一个很好的结果。我不知道为什么我的表单将@ user.id作为参数提交。有任何想法吗? – 2010-11-09 17:54:05

+0

看起来像提交id是正常行为。我在使用脚手架的全新Rails应用程序中看到了相同的行为。 – 2010-11-09 18:01:10

相关问题