ActionMailer失败

Question

尝试使用ActionMailer发送电子邮件时出现无方法错误

NoMethodError in OrdersController#deliver
undefined method `client' for #<OrdersController:0x007fdc22aa4938>

我的Mailer

 class OrdersMailer < ActionMailer::Base
 def email_copy_of_order_to_client(order)
    @order = order
    @client = @order.client
    mail(:to => @client.email, :subject => "Your order")
  end

当然，通过控制台我得到了我想要的东西

  Order.last.client
    Order Load (0.2ms)  SELECT "orders".* FROM "orders" ORDER BY "orders"."id" DESC LIMIT 1
  => #<Client id: 3, name: "Test Client", address1: "", address2: "", city: ""...`

编辑过的

class OrdersController < InheritedResources::Base
def deliver
@email = OrdersMailer.email_copy_of_order_to_client(self).deliver

respond_to do |format|
  if @email.valid?
    format.html { redirect_to order, notice: 'Order was successfully sent.' }
  else
    format.html { render action: "show", alert: 'An error occured. Please try again.' }
  end
end
end

Order#show视图

<%= pill_button_link_to "Send Order by Email", deliver_order_path(@order) %>

有什么想法吗？

Answer 1

您的邮件发送程序需要一个订单作为它的参数：

class OrdersMailer < ActionMailer::Base
  def email_copy_of_order_to_client(order)
    #...

但是你给了它一个OrderController的实例

class OrdersController < InheritedResources::Base
  def deliver
    @email = OrdersMailer.email_copy_of_order_to_client(self).deliver
    #...

该self是控制器的实例，而不是Order的实例。因此，您将看到关于OrdersController的错误消息：

undefined method `client' for #<OrdersController:0x007fdc22aa4938>

您可能希望在您的deliver中包含以下内容

def deliver
  order = Order.find(params[:id])
  # check access/ownership/... on order
  @email = OrdersMailer.email_copy_of_order_to_client(order).deliver
  #...