recreate_versions! prepends and appends version to filename

[synoptase]

Issue is quite similar to #1164, with some other side effect.

With this model

require 'carrierwave/processing/rmagick'

class Uploader < CarrierWave::Uploader::Base
  include CarrierWave::RMagick

  def cache_dir
    "/tmp/cache"
  end

  def process_uri(uri)
    URI.parse(uri)
  end

end

class PictureUploader < Uploader

  def reprocess_file!
    cache_stored_file!
    retrieve_from_cache!(cache_name)
    recreate_versions!(:small)
  end

  version :small, :if => :is_a_small_pic? do
     ...
  end
end

Everything works fine on the first file generation.
It also works when i call recreate_versions! without any parameters. But when i call recreate_versions!(:small) my newly generated file will look like this small_<filename>_small.jpg.

Overriding full_filename method doesn't change anything.

[ghost]

Is the only work around here to recreate all versions? or is there a fix for this?

[thomasfedb]

@synoptase Is this bug still present in the latest version of carrierwave?

[synoptase]

Haven't updated my carrierwave gem lately. Don't plan to until at least january.

[thomasfedb]

@synoptase what version are you using?

[synoptase]

Something like this:

carrierwave (0.10.0)
      activemodel (>= 3.2.0)
      activesupport (>= 3.2.0)
      json (>= 1.7)
      mime-types (>= 1.16)
    carrierwave-mongoid (0.7.1)
      carrierwave (>= 0.8.0, < 0.11.0)
      mongoid (>= 3.0, < 5.0)
      mongoid-grid_fs (>= 1.3, < 3.0)
    carrierwave_direct (0.0.14)
      carrierwave
      fog
      uuidtools

[thomasfedb]

Thanks @synoptase, you're using the latest version.

[Grafikart]

It looks like filename is called when doing a recreate_versions! (I'm using 1.2.1). I don't thinks it's the desired behaviour since it should use the name in the database and add the version name (like it does when calling model.image.{version_name}.url)

[mshibuya]

Could not reproduce, maybe because #1879 have changed the version recreation much.
Please let us know if this is still an issue.