Rails嵌套属性错误未传播求助：关联销毁时错误未反馈至主模型

首先，我得帮你拆解下为什么会出现这个矛盾的情况：

核心原因 #

当你通过嵌套属性tasks_attributes设置_destroy: 1时，Rails的处理流程是这样的：

1. 先完成CourseSection自身的属性更新（这里你没修改父对象属性，所以这一步很快结束）
2. 再处理关联的Task对象，标记要销毁的Task并调用其destroy方法
3. Task的dependent: :destroy触发ContentRef的销毁逻辑，进而触发它的before_destroy回调
4. 回调里你用throw(:abort)中止了ContentRef的销毁，但这个中止只会停在ContentRef的销毁流程里，不会向上冒泡影响CourseSection的update结果

另外，你在ContentRef的回调里给course_section.errors加了错误，但这时候CourseSection的保存流程已经走完了——父对象的update方法只关心自身是否保存成功，不会去检查关联对象的销毁是否全部完成，所以它会返回true，导致控制器误以为操作成功。

解决方案 #

这里有几个优雅的解决思路，你可以根据业务场景选择：

方案1：在CourseSection层面添加验证，提前拦截不可销毁的Task

利用Rails给标记为销毁的关联对象提供的marked_for_destruction?方法，在CourseSection的更新验证中检查要删除的Task是否符合条件：

class CourseSection < ApplicationRecord
  has_many :tasks, through: :content_refs, source: :content, source_type: "Task"
  has_many :content_refs

  validate :prevent_deletion_of_completed_tasks, on: :update

  private

  def prevent_deletion_of_completed_tasks
    # 筛选出通过嵌套属性标记为销毁的Task
    tasks_marked_for_destroy = tasks.select { |task| task.marked_for_destruction? }
    
    tasks_marked_for_destroy.each do |task|
      # 找到对应的ContentRef，检查是否有已完成的任务记录
      content_ref = content_refs.find_by(content: task, content_type: "Task")
      if content_ref&.task_completions&.any?
        errors.add(:tasks, "Task '#{task.title}' has associated completions and can't be deleted. Try archiving it instead.")
      end
    end
  end
end

这样一来，当你调用@course_section.update时，验证会先触发，如果发现有不可删除的Task，就会把错误添加到CourseSection的错误集合里，update会返回false，控制器就能正确感知异常了。

方案2：修改Task的销毁回调，把错误关联到父对象

既然Task的销毁会触发ContentRef的检查，我们可以把检查逻辑移到Task模型里，直接给对应的CourseSection添加错误：

class Task < ApplicationRecord
  has_many :referring_content_refs, as: :content, dependent: :destroy, class_name: "ContentRef"

  before_destroy :check_for_completed_task_refs

  private

  def check_for_completed_task_refs
    referring_content_refs.each do |ref|
      if ref.task_completions.any?
        # 直接给关联的CourseSection添加错误
        ref.course_section.errors.add(:base, "Task '#{title}' has associated completions and can't be deleted. Try archiving it instead.")
        throw(:abort)
      end
    end
  end
end

然后在控制器里，你需要手动检查@course_section.errors是否为空，因为即使update返回true，父对象可能已经被添加了错误：

def update
  @course_section = CourseSection.find(params[:id])
  if @course_section.update(update_attributes_params)
    # 额外检查父对象是否有错误
    if @course_section.errors.empty?
      redirect_to @course_section, notice: "Course section updated successfully."
    else
      render :edit
    end
  else
    render :edit
  end
end

方案3：手动处理嵌套属性的销毁逻辑

如果不想依赖Rails的自动嵌套属性处理，可以在控制器里手动筛选要销毁的Task，先检查是否可以删除，再执行操作：

def update
  @course_section = CourseSection.find(params[:id])
  task_attrs = update_attributes_params[:tasks_attributes] || {}
  
  # 提取要销毁的Task ID
  task_ids_to_destroy = task_attrs.values.select { |attrs| attrs[:_destroy] == "1" }.map { |attrs| attrs[:id] }

  # 检查这些Task是否可以被销毁
  task_ids_to_destroy.each do |task_id|
    task = @course_section.tasks.find(task_id)
    content_ref = task.referring_content_refs.first
    if content_ref&.task_completions&.any?
      @course_section.errors.add(:tasks, "Task '#{task.title}' can't be deleted due to existing completions.")
    end
  end

  if @course_section.errors.empty?
    # 移除销毁标记，手动删除符合条件的Task
    filtered_task_attrs = task_attrs.reject { |k, v| v[:_destroy] == "1" }
    if @course_section.update(update_attributes_params.merge(tasks_attributes: filtered_task_attrs))
      @course_section.tasks.where(id: task_ids_to_destroy).destroy_all
      redirect_to @course_section, notice: "Course section updated successfully."
    else
      render :edit
    end
  else
    render :edit
  end
end

这个方案更灵活，完全掌控销毁流程，但代码量会多一些。

总结 #

核心问题就是Rails嵌套属性的销毁错误不会自动冒泡到父对象，所以需要我们主动把错误关联到父对象，或者在父对象层面提前做验证。方案1是最符合Rails约定的方式，推荐优先尝试。

内容的提问来源于stack exchange，提问作者frostblooded