CarrierWave::Uploader::Store#filename should proxy to us.

[richardkmichael]

There appears to be a bug in CarrierWave::Uploader::Store#filename -> @filename is nil, e.g. @filename is not being set by the cache code. Neither cache! nor retrieve_from_cache! are being called.

• Review CarrierWave's Uploader spec re: Store#filename()

Workaround by adding a filename() method to your uploader:

class FooUploader
  def filename
    self.file.filename
  end
end

CarrierWave::Uploader should proxy the #filename() message to the file, as it does for other convenience methods. E.g. https://github.com/jnicklas/carrierwave/blob/master/lib/carrierwave/uploader/store.rb#L27 could do:

  @filename ||= file.filename if file.respond_to?(:filename)