CarrierWave:为所有版本化文件创建相同的唯一文件名

时间:2011-08-03 02:12:13

标签: ruby-on-rails ruby ruby-on-rails-3 filenames carrierwave

在我详细介绍之前,我会明确指出:有没有人想办法让Carrierwave保存文件,其名称为时间戳或每个文件唯一的任意字符串?

默认情况下,Carrierwave会将每个文件及其备用版本保存在自己的目录中(以型号ID号命名)。我不是这个的粉丝,因为为了使用大的圆形数字而不是一个目录1000,为了使用大文件(在我的情况下是图片),我们得到一个目录,其中包含1000个子目录,每个子目录有一个或两个文件。呸。

现在,当您覆盖上传者的store_dir方法时,如下所示:

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

你最终得到了我想要的确切行为。所有文件(图片)都进入一个大的快乐文件夹。当对象被删除时,不再有子文件夹。

只有一个问题。文件冲突。如果你上传delicious_cake.jpg两次,那么即使它们是美味蛋糕的两张不同的照片,它也会覆盖第一个!这显然是为什么store_dir方法在返回的值的末尾添加了额外的/#{model.id}

那么,该怎么办?在阅读了一下后,我发现在生成的上传器文件中有一个明显的解决方案被注释掉了。

# Override the filename of the uploaded files:
# Avoid using model.id or version_name here, see uploader/store.rb for details.
# def filename
#   "something.jpg" if original_filename
# end

经过一番搜索,我找到了一个做过以下事情的人

def filename
  @name ||= "#{secure_token}.#{file.extension}" if original_filename
end

这让我思考,为什么不这样做呢

def filename
  @name ||= "#{(Time.now.to_i.to_s + Time.now.usec.to_s).ljust(16, '0')}#{File.extname(original_filename)}"
end

当事情被严重破坏时。这个问题是filename显然是为文件的每个版本调用,所以我们最终得到文件名,如1312335603175322.jpg和thumb_1312335603195323.jpg。请注意细微差别?每个文件名都基于为该特定版本调用filename的时间。那根本不会做。

我接下来使用model.created_at作为时间戳的基础感到厌倦。只有一个问题,即第一个版本返回nil,因为它还没有被放入数据库。

经过一番思考后,我决定在我的照片控制器中尝试以下操作。

def create
  if params[:picture] and params[:picture][:image]
    params[:picture][:image].original_filename = "#{(Time.now.to_i.to_s + Time.now.usec.to_s).ljust(16, '0')}#{File.extname(params[:picture][:image].original_filename)}"
  end
  ...

这会覆盖原始文件名属性,然后Carrierwave会将其作为时间戳。它完全符合我的要求。该文件的原始版本最终使用1312332906940106.jpg这样的名称,缩略图版本(或任何其他版本)的最终名称为thumb_1312332906940106.jpg。

但是,这似乎是一个可怕的黑客。这应该是模型的一部分,或者更好的是安装到模型上的上传器的一部分。

所以,我的问题是,有没有更好的方法来实现这一目标?我是否错过了Carrierwave至关重要的一些内容?是否有一个不那么明显但更清洁的方式来解决这个问题?工作代码很好,但是工作代码没有嗅到坏的更好。

3 个答案:

答案 0 :(得分:21)

您可以在uploader文件中执行此类操作,它也适用于版本化文件(即,如果您有一个图像,然后创建同一文件的3个其他缩略图版本,则它们都将具有相同的名称,只需将大小信息附加到名称上):

  # Set the filename for versioned files
  def filename
    random_token = Digest::SHA2.hexdigest("#{Time.now.utc}--#{model.id.to_s}").first(20)
    ivar = "@#{mounted_as}_secure_token"    
    token = model.instance_variable_get(ivar)
    token ||= model.instance_variable_set(ivar, random_token)
    "#{model.id}_#{token}.jpg" if original_filename
  end

这将创建一个这样的文件名,例如:76_a9snx8b81js8kx81kx92.jpg其中76是模型的id,另一位是随机SHA十六进制。

答案 1 :(得分:2)

另请查看现在提供的来自carrierwave wiki的解决方案https://github.com/carrierwaveuploader/carrierwave/wiki/How-to:-Use-a-timestamp-in-file-names

您可以在文件名中包含一个时间戳,覆盖文件名,您可以在Carrierwave文档中阅读:

 class PhotoUploader < CarrierWave::Uploader::Base
     def filename
       @name ||= "#{timestamp}-#{super}" if original_filename.present? and 
       super.present?
    end

   def timestamp
     var = :"@#{mounted_as}_timestamp"
     model.instance_variable_get(var) or model.instance_variable_set(var, Time.now.to_i)
   end
 end

不要忘记在实例变量中记住结果,否则可能会将不同的时间戳写入数据库和文件存储区。

答案 2 :(得分:0)

解决方案与the official documentation

中所述的相同

但是它总是将original_filename作为nil返回。因此,只需将其更改为实例变量为@original_filename.present?