ActionMailer 失败

Question

发送电子邮件时，我收到无方法错误

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

当尝试使用 ActionMailer My 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 view

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

有什么想法吗？

Answer 1

您的邮件程序想要一个 Order 作为其参数：

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
  #...