Почему update_attributes ломает мое приложение Rails?

Question

Оригинальный вопрос

Я использую Rails 3.0.1 на Ruby 1.9.2.Вот соответствующая модель, контроллер и представление.

Код

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

У меня есть другая модель / контроллер / представление для каталогов, которая очень похожа, но не имеет виртуальных средств доступа или других идентификаторов модели, и update_attributes прекрасно работает в ней.Я сделал быстрое тестовое приложение с rails g scaffold users name:string password:string, и все действия CRUD работают нормально.

Это сводило меня с ума!Я нашел обходной путь, но я действительно хочу понять, почему update_attributes здесь не работает.Когда я запускаю действие обновления, я получаю следующее:

TypeError в UsersController # update

не может преобразовать ноль в Integer

Rails.root: / home / force /proj

app / controllers / users_controller.rb: 34: in `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] в действии update.

Answer 1

OK.Я чувствую себя действительно глупо.Я удивлен, что никто не поймал это.

В моей модели я назову один из моих атрибутов.hash, чтобы получить к нему доступ, я бы сказал @user.hash.

ActiveRecord :: Base # hash is уже определено !

Итак, я облажался.Когда ActiveRecord пытался выполнить транзакцию, он пытался установить значение, например

@user.hash = password_hash

ActiveRecord::Base#hash= ожидает целое число, и password_hash выводит строку, если пароль изменен, и ноль в противном случае.

Поэтому никогда не называйте хэш столбца! Проверяйте документацию при выборе имен столбцов, чтобы избежать коллизий.

Answer 2

Похоже, что проблема может заключаться в том, что идентификатор пользователя является первичным ключом и каким-то образом его пытаются обновить. Какое сообщение об ошибке вы получите, если используете 'bang version of update_attributes : update_attributes! `?

Answer 3

Без полной трассировки стека трудно сказать, но сообщение об ошибке «невозможно преобразовать ноль в целое число» - это ваша ключевая вещь для отслеживания. Точно так же мы хотим видеть журналы для запроса, который терпит неудачу. Я думаю, что строка параметров: может пролить свет на проблему.

Обновление ниже:

Можете ли вы успешно передать те же самые параметры в @ user.update_attributes в консоли rails?

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)

Это ведет себя по-другому?