Странная ошибка при подписании пользователем после перенаправления

Question

Я хочу автоматически подписывать пользователей после создания учетной записи. Моя заявка является многопользовательской, где учетные записи реализуются на уровне субдомена: * account_name.myapp.com *

У меня есть модель Account с has_many :owners, through: :ownerships. Я использую accepts_nested_attributes_for для создания учетной записи и модели пользователя с использованием одной формы.

Я написал before_filter, чтобы обработать вход, но я получаю ошибку wrong number of arguments (1 for 0). Я не уверен, как его отладить.

После создания моделей учетных записей и пользователей я перенаправляюсь на учетные записи # show - где происходит ошибка. Я передаю параметр sign_in_token. Используя before_filter, я загружаю пользователя из токена и вызываю метод sign_in.

class AccountsController < ApplicationController 
    before_filter :sign_in, only: :show 

    def show 
    @account ||= Account.find_by_subdomain!(request.subdomain) 
    end 

    def new 
    @account = Account.new 
    @owner = @account.owners.build 
    end 

    def create 
    @account = Account.new(params[:account]) 
    if @account.save 
     redirect_to root_url(subdomain: @account.subdomain, sign_in_token: @account.owners.first.sign_in_token) 
    else 
     render 'new' 
    end 
    end 

    private 

    def sign_in 
    if params[:sign_in_token] 
     user = User.find_by_sign_in_token(params[:sign_in_token]) 
     sign_in user 
    end 
    end 
end 

Мои sign_in метод работает нормально, и я написал тесты. Это только ломается в этом сценарии.

def sign_in(user) 
    cookies.permanent[:remember_token] = user.remember_token 
    current_user = user 
    end 

Ошибка:

ArgumentError in AccountsController#show 

wrong number of arguments (1 for 0) 

Stack

app/controllers/accounts_controller.rb:31:in `sign_in' 
app/controllers/accounts_controller.rb:34:in `sign_in' 
activesupport (3.2.3) lib/active_support/callbacks.rb:429:in `_run__3150241459892085714__process_action__1436783733275598907__callbacks' 
activesupport (3.2.3) lib/active_support/callbacks.rb:405:in `__run_callback' 
activesupport (3.2.3) lib/active_support/callbacks.rb:385:in `_run_process_action_callbacks' 
activesupport (3.2.3) lib/active_support/callbacks.rb:81:in `run_callbacks' 
actionpack (3.2.3) lib/abstract_controller/callbacks.rb:17:in `process_action' 
actionpack (3.2.3) lib/action_controller/metal/rescue.rb:29:in `process_action' 
actionpack (3.2.3) lib/action_controller/metal/instrumentation.rb:30:in `block in process_action' 
activesupport (3.2.3) lib/active_support/notifications.rb:123:in `block in instrument' 
activesupport (3.2.3) lib/active_support/notifications/instrumenter.rb:20:in `instrument' 
activesupport (3.2.3) lib/active_support/notifications.rb:123:in `instrument' 
actionpack (3.2.3) lib/action_controller/metal/instrumentation.rb:29:in `process_action' 
actionpack (3.2.3) lib/action_controller/metal/params_wrapper.rb:205:in `process_action' 
activerecord (3.2.3) lib/active_record/railties/controller_runtime.rb:18:in `process_action' 
actionpack (3.2.3) lib/abstract_controller/base.rb:121:in `process' 
actionpack (3.2.3) lib/abstract_controller/rendering.rb:45:in `process' 
actionpack (3.2.3) lib/action_controller/metal.rb:203:in `dispatch' 
actionpack (3.2.3) lib/action_controller/metal/rack_delegation.rb:14:in `dispatch' 
actionpack (3.2.3) lib/action_controller/metal.rb:246:in `block in action' 
actionpack (3.2.3) lib/action_dispatch/routing/route_set.rb:73:in `call' 
actionpack (3.2.3) lib/action_dispatch/routing/route_set.rb:73:in `dispatch' 
actionpack (3.2.3) lib/action_dispatch/routing/route_set.rb:36:in `call' 
actionpack (3.2.3) lib/action_dispatch/routing/mapper.rb:40:in `call' 
journey (1.0.3) lib/journey/router.rb:68:in `block in call' 
journey (1.0.3) lib/journey/router.rb:56:in `each' 
journey (1.0.3) lib/journey/router.rb:56:in `call' 
actionpack (3.2.3) lib/action_dispatch/routing/route_set.rb:600:in `call' 
actionpack (3.2.3) lib/action_dispatch/middleware/best_standards_support.rb:17:in `call' 
rack (1.4.1) lib/rack/etag.rb:23:in `call' 
rack (1.4.1) lib/rack/conditionalget.rb:25:in `call' 
actionpack (3.2.3) lib/action_dispatch/middleware/head.rb:14:in `call' 
actionpack (3.2.3) lib/action_dispatch/middleware/params_parser.rb:21:in `call' 
actionpack (3.2.3) lib/action_dispatch/middleware/flash.rb:242:in `call' 
rack (1.4.1) lib/rack/session/abstract/id.rb:205:in `context' 
rack (1.4.1) lib/rack/session/abstract/id.rb:200:in `call' 
actionpack (3.2.3) lib/action_dispatch/middleware/cookies.rb:338:in `call' 
activerecord (3.2.3) lib/active_record/query_cache.rb:64:in `call' 
activerecord (3.2.3) lib/active_record/connection_adapters/abstract/connection_pool.rb:467:in `call' 
actionpack (3.2.3) lib/action_dispatch/middleware/callbacks.rb:28:in `block in call' 
activesupport (3.2.3) lib/active_support/callbacks.rb:405:in `_run__1668768892136358850__call__837556511977231927__callbacks' 
activesupport (3.2.3) lib/active_support/callbacks.rb:405:in `__run_callback' 
activesupport (3.2.3) lib/active_support/callbacks.rb:385:in `_run_call_callbacks' 
activesupport (3.2.3) lib/active_support/callbacks.rb:81:in `run_callbacks' 
actionpack (3.2.3) lib/action_dispatch/middleware/callbacks.rb:27:in `call' 
actionpack (3.2.3) lib/action_dispatch/middleware/reloader.rb:65:in `call' 
actionpack (3.2.3) lib/action_dispatch/middleware/remote_ip.rb:31:in `call' 
actionpack (3.2.3) lib/action_dispatch/middleware/debug_exceptions.rb:16:in `call' 
actionpack (3.2.3) lib/action_dispatch/middleware/show_exceptions.rb:56:in `call' 
railties (3.2.3) lib/rails/rack/logger.rb:26:in `call_app' 
railties (3.2.3) lib/rails/rack/logger.rb:16:in `call' 
actionpack (3.2.3) lib/action_dispatch/middleware/request_id.rb:22:in `call' 
rack (1.4.1) lib/rack/methodoverride.rb:21:in `call' 
rack (1.4.1) lib/rack/runtime.rb:17:in `call' 
activesupport (3.2.3) lib/active_support/cache/strategy/local_cache.rb:72:in `call' 
rack (1.4.1) lib/rack/lock.rb:15:in `call' 
actionpack (3.2.3) lib/action_dispatch/middleware/static.rb:62:in `call' 
railties (3.2.3) lib/rails/engine.rb:479:in `call' 
railties (3.2.3) lib/rails/application.rb:220:in `call' 
rack (1.4.1) lib/rack/content_length.rb:14:in `call' 
railties (3.2.3) lib/rails/rack/log_tailer.rb:14:in `call' 
rack (1.4.1) lib/rack/handler/webrick.rb:59:in `service' 
/home/mohamad/.rvm/rubies/ruby-1.9.3-p0/lib/ruby/1.9.1/webrick/httpserver.rb:138:in `service' 
/home/mohamad/.rvm/rubies/ruby-1.9.3-p0/lib/ruby/1.9.1/webrick/httpserver.rb:94:in `run' 
/home/mohamad/.rvm/rubies/ruby-1.9.3-p0/lib/ruby/1.9.1/webrick/server.rb:191:in `block in start_thread' 

Answer 1

У вас есть такое же имя функции, которую вы звоните

def **sign_in** 
    if params[:sign_in_token] 
    user = User.find_by_sign_in_token(params[:sign_in_token]) 
    **sign_in user** 
    end 
end 

так sign_in пользователь пытается позвонить функция, в которой она определена, но с аргументом. вероятно, не то, что вы имели в виду.

Что касается отладки, как только вы отправили трассировку стека, стало очевидно. sign_in user передавался параметр, когда никто не ожидал (1/0) ... и причина была в трех строках выше него.)

Answer 2

Вы определили метод sign_in в вашем контроллере, который скрывает метод sign_in, который использует библиотека проверки подлинности, поэтому ваш sign_in не принимает параметры, а оригинал. Исправить это - переименовать свой метод в другое:

class AccountsController < ApplicationController 
    before_filter :sign_in_user, only: :show 

    def show 
    @account ||= Account.find_by_subdomain!(request.subdomain) 
    end 

    def new 
    @account = Account.new 
    @owner = @account.owners.build 
    end 

    def create 
    @account = Account.new(params[:account]) 
    if @account.save 
     redirect_to root_url(subdomain: @account.subdomain, sign_in_token: @account.owners.first.sign_in_token) 
    else 
     render 'new' 
    end 
    end 

    private 

    def sign_in_user 
    if params[:sign_in_token] 
     user = User.find_by_sign_in_token(params[:sign_in_token]) 
     sign_in user 
    end 
    end 
end 

Теперь он должен работать.