Попытка установить переменную в before_validation, но она не работает - PullRequest
Новая
       0

Попытка установить переменную в before_validation, но она не работает

0 голосов
/ 20 мая 2011

Так что, на мой взгляд, я использую календарь, чтобы выбрать день, и выпадающий список, чтобы выбрать время. Поэтому я использую метод before_validation, чтобы собрать его вместе:

proposed_time.rb 

before_validation :construct_starting_at

def construct_starting_at
  d = Time.parse(date)
  puts "************** construct_starting_at BEGIN *****************"
  puts "DATE: #{d}"
  puts "Time: #{time}"
  puts "Timezone: #{timezone}"
  puts "construct_starting_at :: #{d.year}-#{d.month}-#{d.day} #{time.hour}:#{time.min}:00 #{timezone}"
  if date.present? && time.present? && timezone.present?
    starting_at = Time.zone.parse("#{d.year}-#{d.month}-#{d.day} #{time.hour}:#{time.min}:00 #{timezone}")
  end
  puts "starting_at: #{starting_at}"
  puts "************** construct_starting_at END *****************"
end

И он прекрасно работает, когда я создаю объект, но не когда я его обновляю.

1012 * журнал *

************** construct_starting_at BEGIN *****************
DATE: Fri Jun 03 00:00:00 -0500 2011
Time: Thu May 19 23:00:00 UTC 2011
Timezone: (GMT-05:00) Eastern Time (US & Canada)
construct_starting_at :: 2011-6-3 23:0:00 (GMT-05:00) Eastern Time (US & Canada)
starting_at: 2011-06-04 00:00:00 -0400
************** construct_starting_at END *****************

Но когда я использую его для обновления, он полностью теряет его и возвращается к тому, что было. Это заставляет меня чувствовать, что это на самом деле не спасается. Итак, чтобы помочь объяснить контекст этого следующего, у меня есть объект ProposedTime, и это ребенок Consultation (в каждой консультации есть 3 предложенных времени), который также имеет accepts_nested_attributes_for :proposed_times:

consultation.rb 

def proposed_times_attributes=(attributes)
  puts "$$$$$$$$$$$$$$ proposed_times_attributes $$$$$$$$$$$$$$$$"
  attributes.each do |key,value|
    value[:timezone] = timezone
    if value[:id]
      puts "Updating #{value[:id]}"
      p = ProposedTime.find(value[:id])
      value.delete(:id)
      unless p.update_attributes(value)
        puts "@@@@@@@@@@@@@@@@@ ERROR @@@@@@@@@@@@@@@"
        error.add(:proposed_times, "something is wrong")
      end
      puts "-- starting_at:  #{p.starting_at}"
    else
      puts "Creating a new proposed time"
      proposed_times << ProposedTime.new(value)
    end
  end
  puts "$$$$$$$$$$$$$$ proposed_times_attributes $$$$$$$$$$$$$$$$"
end

журнал 

...
Updating 18
************** construct_starting_at BEGIN *****************
DATE: Fri Jun 03 00:00:00 -0500 2011
Time: Thu May 19 23:00:00 UTC 2011
Timezone: (GMT-05:00) Eastern Time (US & Canada)
construct_starting_at :: 2011-6-3 23:0:00 (GMT-05:00) Eastern Time (US & Canada)
starting_at: 2011-06-04 00:00:00 -0400
************** construct_starting_at END *****************
-- starting_at:  2011-06-01 06:00:00 -0400

Я думал, что это могло выдать ошибку в update_attributes, но это не так. Есть идеи?

1 Ответ

4 голосов
/ 20 мая 2011

Я не до конца справился с этим, но я просто остановился на простых вещах - я думаю, что вы хотите, чтобы start_at не был локальной переменной, а на самом деле устанавливал атрибут start_at вашего объекта: 

self.starting_at = Time.zone.parse("#{d.year}-#{d.month}-#{d.day} #{time.hour}:#{time.min}:00 #{timezone}")

Ключевым битом является self., чтобы фактически убедиться, что атрибут установлен, а не какая-то локальная переменная с тем же именем, которая существует только в этом методе.

...