覆盖 Devise 控制器的问题

Question

我正在尝试覆盖 Devise 控制器以进行一些细微的更改，例如在请求未注册电子邮件地址的确认电子邮件时添加一条闪存消息。

我尝试覆盖 Devise::ConfirmationsController1这样：

# app/controllers/confirmations_controller.rb
class ConfirmationsController < Devise::ConfirmationsController

  include Devise::Controllers::InternalHelpers # tried to add this, no success

  def create
    self.resource = resource_class.send_confirmation_instructions(params[resource_name])

    if successfully_sent?(resource)
      respond_with({}, :location => after_resending_confirmation_instructions_path_for(resource_name))
    else
      respond_with(resource)
    end
  end

end

我想我添加了路线正确：

devise_for :users, :controllers => { :confirmations => "confirmations" }

我的控制器方法被调用，但是它引发了此异常：

NoMethodError in ConfirmationsController#create

undefined method `successfully_sent?' for #<ConfirmationsController:0x007fa49e229030>

在我重写的控制器中，我刚刚复制了 Devise::ConfirmationsController#create 的代码，它本身调用 successively_sent?(resource)

successively_sent? 方法在 InternalHelpers 中定义 2，这就是我尝试添加 include Devise::Controllers::InternalHelpers

这不是我第一次尝试覆盖 Devise 控制器，这不是我第一次失败。我总是设法找到解决方法，但我想了解我缺少什么......提前感谢您的帮助！

[编辑] Devise 版本为 1.4.9 Rails 是 3.0.10

Answer 1

好吧，感谢凯尔在我的问题评论中的帮助，我将为这个初学者的错误写出正确的答案。

我没有查看我自己的 Devise 版本来覆盖控制器，而是只是查看 Devise 的 Github 存储库。由于我尝试覆盖的控制器在我的版本和最后提交的版本之间发生了更改，因此我尝试使用的帮助器方法根本没有在我的版本中定义...

正如凯尔所指出的，您可以使用 < code>bundle open devise 查看您实际使用的 gem 的代码，或者您可以使用 gem list devise 查看其版本号，并在Github（对于 Devise，他们为每个版本设置了标签这样您就可以通过选择相应的标签来浏览1.4.9版本的代码）。

这样做，我会使用以下代码覆盖控制器的 create 方法：

def create
  self.resource = resource_class.send_confirmation_instructions(params[resource_name])

  if successful_and_sane?(resource)
    set_flash_message(:notice, :send_instructions) if is_navigational_format?
    respond_with({}, :location => after_resending_confirmation_instructions_path_for(resource_name))
  else
    respond_with_navigational(resource){ render_with_scope :new }
  end
end

使用 successful_and_sane? 而不是 successively_sent? ...

它 总结这个答案，可能有一种更好的方法来向此方法添加闪现消息，而不是覆盖它。 jarrad 建议使用 around_filter，但我还无法让它工作，而且我不确定在从过滤器方法中生成渲染视图后是否仍然可以更改渲染视图...欢迎评论！

Answer 2

这可能无法帮助您理解为什么重写 Devise 控制器失败，但它会使您的代码保持干燥，因为您不需要从 Devise::ConfirmationsController#crete 复制代码

所以，如果您只是想要设置 Flash 消息，请查看 过滤器ActionControllers

具体来说，看看周围过滤器：

class ConfirmationsController < Devise::ConfirmationsController
  around_filter :my_custom_stuff, :only => :create

  private

  def my_custom_stuff
    # do your thing here...
  end
end