Блог

Я получаю ошибку no method при попытке отправить электронное письмо с помощью 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) %>
  

Есть идеи?

Ваша почтовая программа хочет, чтобы в качестве аргумента использовался 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
  #...