ActionMailer 失败

Question

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

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

我的邮件

 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

订单#显示视图

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