test_rails/compex_pic_test.rb @@ -170,11 +170,11 @@ module HasImage end def width - minimagick[:width] + self[:width] || minimagick[:width] end def height - minimagick[:height] + self[:height] || minimagick[:height] end def minimagick @@ -210,15 +210,23 @@ module HasImage def update_images return if storage.temp_file.blank? remove_images - update_attribute(has_image_options[:column], storage.install_images(self)) + populate_attributes end # Processes and installs the image and its thumbnails. def install_images return if !storage.temp_file - update_attribute(has_image_options[:column], storage.install_images(self)) + populate_attributes end + def populate_attributes + send("#{has_image_options[:column]}=", storage.install_images(self)) + self[:width] = minimagick[:width] if self.class.column_names.include?('width') + self[:height] = minimagick[:height] if self.class.column_names.include?('height') + save! + end + private :populate_attributes + # Gets an instance of the underlying storage functionality. See # HasImage::Storage. def storage lib/has_image.rb @@ -26,7 +26,9 @@ module HasImage `identify #{arg.respond_to?(:path) ? arg.path : arg.to_s}` $? == 0 end - end + end + + end # The constuctor should be invoked with the options set by has_image. lib/has_image/processor.rb @@ -5,5 +5,11 @@ ActiveRecord::Schema.define(:version => 1) do t.datetime :created_at t.datetime :updated_at end - + + create_table 'complex_pics', :force => true do |t| + t.string :filename + t.integer :width, :height + t.timestamps + end + end test_rails/schema.rb fb9a4d609387f8e33151042bbca0e0d1f812db83 Gerrit Kaiser gerrit@gerritkaiser.de http://github.com/norman/has_image/commit/d7b057604aa0399afd51c89c3199644b860a2aea d7b057604aa0399afd51c89c3199644b860a2aea 2008-10-21T23:55:08-07:00 2008-10-21T23:55:08-07:00 use width/height columns to store dimensions if they’re present in the model 4130a8d2ba3685963f48c5b3fa0696bd782c078b Gerrit Kaiser gerrit@gerritkaiser.de