Comment on 'of Mogrify, Ruby Tempfile & Dynamic Class Definitions' by silverdr

2009-08-08T18:21:00Z

Steve - thanks. Simple solution to my problem.

Comment on 'of Mogrify, Ruby Tempfile & Dynamic Class Definitions' by Matt

2008-04-28T16:57:28Z

Do you know if the newest version fixes this? == 1.2.2 / 2007-06-01

Comment on 'of Mogrify, Ruby Tempfile & Dynamic Class Definitions' by Steve

2007-07-15T17:48:23Z

I had the same exact problem and extended your idea a little. This modification to make_tmpname allows you to create temp files with whatever extension you like (but if you don't specify an extension, it works like normal):

Module MiniMagick
  class ImgTempFile < Tempfile
    def make_tmpname(basename, n)
      # force tempfile to use basename's extension if provided
      ext = File::extname(basename)
      # force hyphens instead of periods in name
      sprintf('%s%d-%d%s', File::basename(basename, ext), $$, n, ext)
    end
  end # class
end # module

Also, I chose to subclass Tempfile rather than overwrite, which required a small alteration to mini_magick.rb:

          tmp = ImgTempFile.new("minimagic")

Finally, here's a short test that shows it's working (put it in any convenient test file that gets run on related stuff):

  def test_image_temp_file
    tmp = MiniMagick::ImgTempFile.new('test')
    assert_match %r{^test}, File::basename(tmp.path)
    tmp = MiniMagick::ImgTempFile.new('test.jpg')
    assert_match %r{^test}, File::basename(tmp.path)
    assert_match %r{\.jpg$}, File::basename(tmp.path)
  end

Comment on 'of Mogrify, Ruby Tempfile & Dynamic Class Definitions' by mzpn

2007-07-15T17:48:23Z

fun

Martian Intuition - of Mogrify, Ruby Tempfile & Dynamic Class Definitions Comments

Comment on 'of Mogrify, Ruby Tempfile & Dynamic Class Definitions' by silverdr

Comment on 'of Mogrify, Ruby Tempfile & Dynamic Class Definitions' by Matt

Comment on 'of Mogrify, Ruby Tempfile & Dynamic Class Definitions' by Steve

Comment on 'of Mogrify, Ruby Tempfile & Dynamic Class Definitions' by mzpn