2010-11-09 3 views
4

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

Я бегу Rails 3.0.1 на Ruby 1.9.2. Вот соответствующая модель, контроллер и представление.Почему 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 

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

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

TypeError в UsersController # обновление

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

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.rb использовать @user.update_attributes params[:user] в update действия.

+0

Как выглядит ваша схема для вашей модели пользователя? – raidfive

+0

Добавлен вопрос. Благодарю. –

+0

Можете ли вы добавить к ним свои параметры, чтобы отправить форму? (из журналов сервера) – monocle

ответ

5

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

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

ActiveRecord :: Base # hash is already defined!

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

@user.hash = password_hash 

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

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

+1

Сохраненный мой день! 'hash' теперь находится в моем ментальном черном списке для любой переменной или поля :) –

+1

Такая же проблема здесь, проголосуйте! –

1

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

Update Ниже:

Можете ли вы пройти те же 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

Конечно. Прости. Я был довольно измучен, когда я разместил этот вопрос. Я должен был включить трассировку стека. Сейчас я обновлю его. –

+0

Я должен был сначала упомянуть, что используя трассировку стека, я смог отслеживать свою проблему для update_attributes, но я не могу получить дальше. Я не знаю, что такое «нуль». Я думаю, что это может иметь отношение к моему 'attr_accessor's, поскольку, если я устанавливаю пароль, он начинает говорить, что он не может преобразовать ** String ** (вместо nil) в Integer. –

+1

Я клонировал ваш источник из github и изменил действие обновления контроллера пользователя, чтобы использовать update_attributes, и он отлично работает для меня. Это с Ruby 1.8.7. – jwarchol

1

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

+0

Я получаю то же самое поведение, если я использую 'update_attributes!'. Я посмотрю, что я могу узнать о обновлении @ user.id. Это был хороший улов. Я не знаю, почему моя форма отправляет @ user.id в качестве параметра. Есть идеи? –

+0

Похоже, что представление идентификатора - это нормальное поведение. Я вижу такое же поведение в новом приложении Rails, используя строительные леса. –

Смежные вопросы