rails - 在更新前操作日期时间数据

Question

我有两个模型：计划和任务。 Task belongs_to Schedule 和 Schedule has_many 任务。任务的形式是计划的嵌套形式。我正在尝试编写控制器代码或模型方法，将用户输入的日期时间（称为：time_frame）用于任务，如果该日期时间已经发生（过去），将发出通知并重定向。我已经尝试过几种方法，但没有一种方法有效。我尝试为 schedules#update 编写此控制器代码：

schedule_params[:tasks_attributes].each do |task|
    if task[:time_frame] < DateTime.now
         render 'update', :notice => 'You must pick a future time.'
    end 
end

这里是 schedule_params：

def schedule_params
      params.require(:schedule).permit(:emp_accepts, 
        tasks_attributes: [:title, :content, :_destroy, :time_frame, 
        :complete_time])    
end

但我收到了错误：

no implicit conversion of symbol to integer

我尝试在 Schedule 模型中编写这样的模型方法：

before_update :compare_datetimes

def compare_datetimes
    puts 'before task is found'
    self.tasks.each do |task|
        puts 'here is the task'
        if task.time_frame < DateTime.now
            puts 'It is in the past'
        end
    end
end

'before task is found' 被放到服务器上，但是其他两个 put 都没有被执行。我该怎么做？

Answer 1

您想在保存之前向任务模型添加验证以检查 time_frame：

class Task < ActiveRecord::Base
  validate :time_frame, presence: true # if you require time_frame to always be present
  validate :datetime_in_future

  private

  def datetime_in_future
    # if time_frame is optional, check its presence before comparing
    if !self.time_frame.blank? && self.time_frame < DateTime.now
      errors.add :time_frame, 'must be a future time.'
    end
  end
end

当您保存或更新它时，这应该会使您的关联和计划无效。

关于您的控制器：不起作用的原因是您在将 time_frame 与 datetime 进行比较之前没有解析它，您应该这样做：

if DateTime.parse(task[:time_frame]) < DateTime.now
  # code here ...

这是因为控制器看到的是原始字符串形式的 time_frame 值。当您将其分配给您的任务时，activerecord 会将其转换为日期时间列的数据类型（我假设您在迁移中以这种方式设置）。

更新： 关于你得到no implicit conversion of symbol to integer的控制器错误tasks_attributes是一个索引哈希，所以你需要像这样迭代它：

schedule_params[:tasks_attributes].each_pair do |index, task|
    if task[:time_frame] < DateTime.now
         render 'update', :notice => 'You must pick a future time.'
    end 
end

当然，任务模型的验证是解决这个问题的更好方法。