在模型更新后,Carrierwave不会重新创建版本

问题描述 投票:4回答:3

我在Carrierwave Uploader上引入了一个新版本。当我创建一个新的Event时,它会正确创建两个版本。但是当我更新它时,只有我附加的文件被上传,但版本不会被重新创建。

我正在使用CarrierWave 1.2.2,并查看更改日志,它似乎不是一个在较新版本中修复的错误

class CoverUploader < CarrierWave::Uploader::Base
  include CarrierWave::MiniMagick

  if Rails.env.development? || Rails.env.test?
    storage :file
  elsif Rails.env.production?
    storage :fog
  end

  # Override the directory where uploaded files will be stored.
  # This is a sensible default for uploaders that are meant to be mounted:
  def store_dir
    if ENV['HEROKU_APP_NAME'].to_s.include?('-pr-')
      "review_apps/#{model.class.to_s.underscore}/#{model.id}"
    else
      "#{Rails.env}/#{model.class.to_s.underscore}/#{model.id}"
    end
  end

  # Provide a default URL as a default if there hasn't been a file uploaded:
  def default_url(*args)
    ActionController::Base.helpers.asset_path('test.jpg')
  end

  # Create different versions of your uploaded files:
  version :optimised do
    process convert: 'webp'
    process :set_content_type_to_webp

    def full_filename(_for_file = model.cover.file)
      "cover_#{model.id}.webp"
    end

    def exists?
      file&.exists?
    end
  end

  def extension_blacklist
    %w(webp)
  end

  private

  # Required to actually force Amazon S3 to treat it like an image
  def set_content_type_to_webp
    file.instance_variable_set(:@content_type, 'image/webp')
  end
end
ruby-on-rails ruby ruby-on-rails-5 carrierwave
3个回答
2
投票

@ogelacinyc在full_filename发现了这个错误时,部分是正确的。我回去测试正常功能,创建另一个版本,只需更改一个维度。然后我可以看到更新会自行重新创建版本,就像我预期的那样。

这让我觉得我的version :optimised块可能有问题。因此,在逐一评论之后,我发现full_filename是罪魁祸首。它可能是model.cover.file默默失败,但我认为它是model.id,可以在filename method in Carrierwave的描述中看到

所以相反,我直接获取文件名,提取扩展名并用webp替换它:

  def full_filename(for_file = model.file_name.file)
    extension = File.extname(for_file)
    "cover_#{for_file.sub(extension, '.webp')}"
  end

哪个工作没有问题!


0
投票

您需要向Event添加一个after_save回调,然后在已安装的上传器上调用recreate_versions!

假设您有一个具有以下内容的事件模型,这将解决您的问题。

class Event < ApplicationRecord
  mount_uploader :cover_image, CoverUploader
  after_save :recreate_versions!
  delegate :recreate_versions!, to: :cover_image, allow_nil: true
end

另见CarrierWave's README


-1
投票

我测试了你的代码并找到了一个bug。

它无法在full_filename方法中初始化变量_for_file

def full_filename(_for_file = model.cover.file)
  "cover_#{model.id}.webp"
end

model.cover在创建优化版本时调用错误'undefined method cover'。

© www.soinside.com 2019 - 2024. All rights reserved.