Added portraits importing and resizing so that portraits are served l…

…ocally with optimal performance.

.gitignore

@@ -1,3 +1,5 @@
 .bundle
+db/rubycommitters_development.db
+db/rubycommitters_test.db
 log/*.log
 tmp/**/*

Gemfile

@@ -3,17 +3,19 @@ source :rubygems
 # Project requirements
 gem 'rake'
 gem 'rack-flash'
-gem 'thin' # or mongrel
+gem 'thin',                     :group => :development
 
 # Component requirements
-gem 'slim', '>= 0.8.3'
-gem 'sqlite3-ruby', :require => 'sqlite3'
+gem 'slim',         '>= 0.8.3'
+gem 'sqlite3-ruby',             :require => 'sqlite3'
 gem 'activerecord', '~> 3.0.3', :require => 'active_record'
-gem 'rack-cache', '~> 1.0', :require => 'rack/cache'
+gem 'rack-cache',   '~> 1.0',   :require => 'rack/cache'
+gem 'mini_magick'
+gem 'carrierwave'
 
 # Test requirements
-gem 'rack-test', :group => :test, :require => 'rack/test'
+gem 'rack-test',                :require => 'rack/test', :group => :test
 
 # Padrino
-gem 'sinatra', '1.1.0', :require => 'sinatra/base'
-gem 'padrino', '0.9.19'
+gem 'sinatra',      '1.1.0',    :require => 'sinatra/base'
+gem 'padrino',      '0.9.19'

Gemfile.lock

@@ -13,6 +13,8 @@ GEM
     activesupport (3.0.3)
     arel (2.0.6)
     builder (2.1.2)
+    carrierwave (0.5.1)
+      activesupport (~> 3.0)
     daemons (1.1.0)
     eventmachine (0.12.10)
     git (1.2.5)
@@ -26,6 +28,8 @@ GEM
       mime-types (~> 1.16)
       treetop (~> 1.4.8)
     mime-types (1.16)
+    mini_magick (3.1)
+      subexec (~> 0.0.4)
     padrino (0.9.19)
       padrino-admin (= 0.9.19)
       padrino-core (= 0.9.19)
@@ -67,6 +71,7 @@ GEM
       temple (~> 0.1.6)
       tilt (~> 1.1)
     sqlite3-ruby (1.3.2)
+    subexec (0.0.4)
     temple (0.1.6)
     thin (1.2.7)
       daemons (>= 1.0.9)
@@ -85,6 +90,8 @@ PLATFORMS
 
 DEPENDENCIES
   activerecord (~> 3.0.3)
+  carrierwave
+  mini_magick
   padrino (= 0.9.19)
   rack-cache (~> 1.0)
   rack-flash

app/models/hero.rb

@@ -8,6 +8,7 @@ class Hero < ActiveRecord::Base
 
   def self.import(io)
     require 'yaml'
+    require 'open-uri'
 
     entries = YAML.load(io)
     entries.each do |entry|
@@ -30,8 +31,20 @@ def self.import(io)
         )
       end
 
-      (entry['portraits'] || []).each do |portrait|
-        hero.portraits.create!(:url => portrait)
+      if entry['portraits']
+        entry['portraits'].each do |url|
+          portrait = hero.portraits.create!(:url => url)
+          begin
+            portrait.remote_file_url = URI.unescape(url) # caters for utf-8 strings
+            portrait.save!
+          rescue OpenURI::HTTPError => e
+            puts e.message
+            puts "Error retrieving #{url}"
+          end
+        end
+      else
+        # creates a dummy portrait entry so that Carrierwave could fallback
+        hero.portraits.create!(:url => "")
       end
 
       (entry['services'] || []).each do |name, key|

app/models/portrait.rb

@@ -1,3 +1,4 @@
 class Portrait < ActiveRecord::Base
+  mount_uploader :file, PortraitUploader
   belongs_to :hero
 end

app/views/heroes/_portrait.html.slim

@@ -1 +1 @@
-li== image_tag portrait.url
+li== image_tag portrait.file.url

db/migrate/005_create_portraits.rb

@@ -2,6 +2,7 @@ class CreatePortraits < ActiveRecord::Migration
   def self.up
     create_table :portraits do |t|
       t.string :url
+      t.text :file
       t.references :hero
       t.timestamps
     end

db/schema.rb

@@ -41,6 +41,7 @@
 
   create_table "portraits", :force => true do |t|
     t.string   "url"
+    t.text     "file"
     t.integer  "hero_id"
     t.datetime "created_at"
     t.datetime "updated_at"

lib/portrait_uploader.rb

@@ -0,0 +1,86 @@
+class PortraitUploader < CarrierWave::Uploader::Base
+
+  ##
+  # Image manipulator library:
+  #
+  # include CarrierWave::RMagick
+  # include CarrierWave::ImageScience
+  include CarrierWave::MiniMagick
+
+  ##
+  # Storage type
+  #
+  storage :file
+  #
+  # configure do |config|
+  #   config.s3_access_key_id = 'xxxx'
+  #   config.s3_secret_access_key = 'xxxx'
+  #   config.s3_bucket = 'assets-web'
+  # end
+  #
+  # storage :right_s3
+
+
+  ## Manually set root
+  def root; File.join(Padrino.root,"public/"); end
+
+  ##
+  # Directory where uploaded files will be stored (default is /public/uploads)
+  #
+  def store_dir
+    'images/portraits/'
+  end
+
+  ##
+  # Directory where uploaded temp files will be stored (default is [root]/tmp)
+  #
+  def cache_dir
+    Padrino.root("tmp")
+  end
+
+  ##
+  # Default URL as a default if there hasn't been a file uploaded
+  #
+  def default_url
+    'http://placehold.it/100x100'
+  end
+
+  ##
+  # Process files as they are uploaded.
+  #
+  process :resize_to_fit => [100, 100]
+  # version :thumb do
+  #   process :resize_to_fill => [100, 100]
+  # end
+  #
+  # def scale(width, height)
+  #   # do something
+  # end
+
+  ##
+  # Create different versions of your uploaded files
+  #
+  # version :header do
+  #   process :resize_to_fill => [940, 250]
+  #   version :thumb do
+  #     process :resize_to_fill => [230, 85]
+  #   end
+  # end
+  ##
+  # White list of extensions which are allowed to be uploaded:
+  #
+  # def extension_white_list
+  #   %w(jpg jpeg gif png)
+  # end
+
+  ##
+  # Override the filename of the uploaded files
+  #
+  def filename
+    if original_filename
+      original_extname = File.extname(original_filename)
+      "#{model.hero.username}_#{model.id}#{original_extname.blank? ? '.jpg' : original_extname}"
+    end
+  end
+  #
+end

lib/tasks/import.rake

@@ -1,5 +1,7 @@
 desc "import the list of ruby committers"
 task :import => ['ar:drop', 'ar:migrate'] do
   require 'open-uri'
+  require 'fileutils'
+  FileUtils.rm_rf File.join(Padrino.root, 'public', 'images', 'portraits')
   Hero.import open('http://svn.coderepos.org/share/lang/ruby/ruby-committers/ruby-committers.yml')
 end

public/stylesheets/base.css

@@ -1,8 +1,4 @@
 th {
     text-align: right;
     vertical-align: top;
-}
-
-img {
-    height: 100px;
 }