0

別のメソッドで割り当てられた値を検証する必要がある検証メソッドがありますが、検証前にそれらの値を認識させるにはどうすればよいですか？pay_must_be_same_to_amountメソッドには、create_items_from_readingsメソッドからのいくつかの値が必要です

class Invoice < ActiveRecord::Base
  attr_accessible :approved_by, :due_date, :invoice_date, :reading_ids, :terms, :customer_id, :customer, :status, :reference_no, :payment_method, :amount, :payment_date

  has_many :invoice_items, :dependent => :destroy
  belongs_to :customer, :inverse_of => :invoices

  validate :pay_must_be_same_to_amount

  def create_item_from_readings
    item = invoice_items.new
    item.rate = customer.unit_cost

    readings_in_this_period = customer.unbilled_readings.where('date_of_reading <= ?',  invoice_date).order(:date_of_reading)
    return nil if readings_in_this_period.empty?

    self.reading_ids = readings_in_this_period.collect(&:id).join(',')

    total_units = 0
    readings_in_this_period.each do |reading|
      total_units = total_units + reading.units_used1 + reading.units_used2 + reading.units_used3
    end

    item.amount = total_units * customer.unit_cost * customer.ct_ratio
    item.tax_amount = (item.amount * Settings.vat) if customer.pays_vat

    invoice_from_reading = readings_in_this_period.first.previous_reading
    invoice_from_reading ||= readings_in_this_period.first
    invoice_to_reading = readings_in_this_period.last


  #Select Item description based on Phase type
    if customer.phase_type == 'Single Phase'
      item.description = "Electricity used from #{invoice_from_reading.date_of_reading.strftime('%d/%m/%Y')} with readings #{invoice_from_reading.reading1} to #{invoice_to_reading.date_of_reading.strftime('%d/%m/%Y')} with   reading #{invoice_to_reading.reading1} - #{total_units.to_i} total units"
    else
      item.description = "Electricity used from #{invoice_from_reading.date_of_reading.strftime('%d/%m/%Y')} with readings, R1: #{invoice_from_reading.reading1}, R2: #{invoice_from_reading.reading2}, R3: #{invoice_from_reading.reading3} to #{invoice_to_reading.date_of_reading.strftime('%d/%m/%Y')} with  readings, R1: #{invoice_to_reading.reading1}, R2:#{invoice_to_reading.reading2}, R3: # {invoice_to_reading.reading3}- #{total_units.to_i} total units"
    end
  end 
end

検証方法は以下のとおりです。上記のitem.amountをクラスInvoiceの金額と比較する必要があります。

 def pay_must_be_same_to_amount
  if item.amount < self.amount && item.amount != self.amount
    self.errors.add :amount, 'The payment amount should be equal to amount on invoice' 
  end
 end
end
4

2 に答える 2

0

質問の解決策は、

def pay_must_be_same_to_amount
 sum = 0
 self.invoice_items.each do |invoice_item|
  sum = sum + invoice_item.amount
end
 if sum != self.amount
  self.errors.add :amount, 'The payment amount should be equal to amount on invoice' 
 end
end
于 2013-02-01T16:46:55.360 に答える