CarrierWave gem：重新生成图片版本后如何重命名文件？

针对你的场景，核心矛盾是在recreate_versions!后需要保存模型，但直接调用save!会触发after_save钩子导致循环执行crop_photo。这里有两种通用解决方案，完美适配你用Concern共享逻辑的场景：

方案一：使用update_columns跳过所有回调 #

update_columns是ActiveRecord提供的轻量更新方法，它直接修改数据库字段，不会触发任何回调（包括after_save），从根源避免循环问题。

修改你的PhotoConcern：

module PhotoConcern
  extend ActiveSupport::Concern
  included do
    mount_uploader :photo, PhotoUploader
    attr_accessor :photo_crop_x, :photo_crop_y, :photo_crop_w, :photo_crop_h
    after_save :crop_photo

    def crop_photo
      return unless photo_crop_x.present?

      photo.recreate_versions!
      # 仅更新photo字段，不触发任何回调
      update_columns(photo: photo) if photo.present?
    end
  end
end

方案二：用标志位跳过特定回调 #

如果你需要在保存时触发其他回调（除了crop_photo），可以添加一个临时内存标志位，让第二次保存时跳过crop_photo的执行：

修改PhotoConcern：

module PhotoConcern
  extend ActiveSupport::Concern
  included do
    mount_uploader :photo, PhotoUploader
    attr_accessor :photo_crop_x, :photo_crop_y, :photo_crop_w, :photo_crop_h, :skip_crop_callback
    # 当skip_crop_callback为true时，不执行crop_photo
    after_save :crop_photo, unless: -> { skip_crop_callback }

    def crop_photo
      return unless photo_crop_x.present?

      photo.recreate_versions!
      # 设置标志位避免循环
      self.skip_crop_callback = true
      save!
      # 重置标志位，确保后续保存能正常触发回调
      self.skip_crop_callback = false
    end
  end
end

重命名版本文件（彻底解决缓存） #

接下来修改PhotoUploader，给版本文件添加唯一标识（比如时间戳），确保每次裁剪后的版本URL完全不同，浏览器不会缓存旧文件：

class PhotoUploader < CarrierWave::Uploader::Base
  include CarrierWave::MiniMagick
  storage :file

  def store_dir
    "uploads/#{model.class.to_s.underscore}/#{mounted_as}/#{model.id}"
  end

  # 给版本文件添加时间戳，主文件保持原文件名
  def filename
    return original_filename unless original_filename.present? && version_name.present?
    "#{version_name}_#{Time.now.to_i}_#{original_filename}"
  end

  version :cropped do
    process :crop
  end

  version :thumb, from_version: :cropped do
    process resize_to_fill: [100, 100]
  end

  version :avatar, from_version: :cropped do
    process resize_to_fill: [200, 200]
  end

  def crop
    return if model.photo_crop_x.blank?
    resize_to_limit(500, nil)
    resize_to_fit(500, nil)
    manipulate! do |img|
      size = model.photo_crop_w << 'x' << model.photo_crop_h
      offset = '+' << model.photo_crop_x << '+' << model.photo_crop_y
      img.crop("#{size}#{offset}")
      img
    end
  end
end

方案对比 #

• 方案一：更轻量、性能更好，适合只需要更新photo字段的场景，不会干扰其他模型的钩子逻辑。
• 方案二：更灵活，若需要触发其他回调逻辑可以选择，但要注意重置标志位避免影响后续保存操作。

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