覆盖设计控制器的问题

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 的代码，它本身调用了successfully_sent?(resource)

successfully_sent?方法定义在InternalHelpers2中，这就是我尝试添加include Devise::Controllers::InternalHelpers的原因

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

[编辑] 设计版本为 1.4.9 Rails 是 3.0.10

Answer 1

嗯，感谢Kyle在我问题的cmets中的帮助，我会把这个初学者的错误的正确答案写出来。

我没有查看我自己的 Devise 版本来覆盖控制器，而是查看了 Devise 的 Github 存储库。由于我试图覆盖的控制器在我的版本和上次提交的版本之间发生了变化，所以我试图使用的辅助方法根本没有在我的版本中定义......

正如 Kyle 所说，您可以使用 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? 而不是successfully_sent? ...

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

Answer 2

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

所以，如果你只是想设置闪信，看Filters for ActionControllers

具体看一下周边滤镜：

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

  private

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