Question

我在轨道上学习ruby并且在aasm回调和动作管理器方面遇到麻烦。我有酒店模特。下面是代码：

class Hotel < ActiveRecord::Base
  include AASM

  scope :approved_hotels, -> { where(aasm_state: "approved") }

  has_many :comments
    belongs_to :user, :counter_cache => true
    has_many :ratings
  belongs_to :address

  aasm do
    state :pending, initial: true
    state :approved
    state :rejected

    event :approve, :after => :send_email do
      transitions from: :pending, to: :approved 
    end
    event :reject, :after => :send_email do
      transitions from: :pending, to: :rejected
    end
  end

  def send_email

  end
end

如您所见，当他添加的酒店状态发生变化时，用户必须收到电子邮件。下面是我写的但不是解决方案cos用户每次管理员更新酒店时都会收到电子邮件处于“待定”状态。

class HotelsController < ApplicationController
  before_filter :authenticate_user!, except: [:index, :show, :top5hotels]

  def update
    @hotel = Hotel.find(params[:id])

    if @hotel.aasm_state == "pending"
      @hotel.aasm_state = params[:state]
      UserMailer.changed_state_email(current_user, @hotel.name, 
      @hotel.aasm_state).deliver
    end

    if @hotel.update_attributes!(params[:hotel])
      redirect_to admin_hotel_path(@hotel), notice: "Hotel was successfully updated."
    else
      render "edit"
    end
  end
end

所以我认为我需要使用回调，但我不知道如何调用

UserMailer.changed_state_email(current_user, @hotel.name, 
        @hotel.aasm_state).deliver

来自模特。我试过了

UserMailer.changed_state_email(User.find(:id), Hotel.find(:name), 
        Hotel.find(aasm_state)).deliver

但这不起作用。我真的没有选择，并寻求任何帮助。谢谢！

Answer 1

您不能使用回调，因为您有current_user是控制器上下文的一部分，并且您无法在模型上下文中访问请求信息。

无论如何，即使您可以使用回调，在这种情况下，我强烈建议您遵循不同的路径。应该很少使用ActiveRecord回调，特别是对于涉及与其他对象或资源（如邮件程序或级联更新）交互的任何代码，应避免使用它们。风险在于，即使您不需要它（例如测试）增加开销，也会触发回调，或者当项目的复杂性增加时，它将与其他回调冲突。

在这种情况下，解决方案非常简单。在模型中定义一个新方法（我现在不介绍您的服务对象......），用于更改状态和发送电子邮件。

class Hotel
  def user_state_change(user, new_state)
    return unless pending? && new_state.present?

    if update_attribute(:aasm_state, new_state)
      UserMailer.changed_state_email(user, name, aasm_state).deliver
    end
  end
end

您的控制器将成为

class HotelsController < ApplicationController
  before_filter :authenticate_user!, except: [:index, :show, :top5hotels]

  def update
    @hotel = Hotel.find(params[:id])
    @hotel.user_state_change(current_user, params[:state])

    if @hotel.update_attributes!(params[:hotel])
      redirect_to admin_hotel_path(@hotel), notice: "Hotel was successfully updated."
    else
      render "edit"
    end
  end
end

作为旁注，您可能希望使用状态机转换方法，而不是更改状态属性。实际上，使用状态机转换将确保触发转换验证。

如何在aasm回调中调用ActionMailer方法？

1 个答案: