npo_assets 0.4.3 → 0.5.0

data/VERSION

@@ -1 +1 @@
-0.4.3
+0.5.0

data/lib/generators/npo_assets/npo_assets_generator.rb

@@ -0,0 +1,20 @@
+class NpoAssetsGenerator < Rails::Generators::Base
+  include Rails::Generators::Migration  
+  
+  source_root File.expand_path('../templates', __FILE__)
+
+  # Implement the required interface for Rails::Generators::Migration.
+   # taken from http://github.com/rails/rails/blob/master/activerecord/lib/generators/active_record.rb
+   def self.next_migration_number(dirname)
+     if ActiveRecord::Base.timestamped_migrations
+       Time.now.utc.strftime("%Y%m%d%H%M%S")
+     else
+       "%.3d" % (current_migration_number(dirname) + 1)
+     end
+   end
+     
+  def generate_npo_assets
+    migration_template('migration.rb', 'db/migrate/create_assets.rb')
+    copy_file('initializer.rb', 'config/initializers/npo_assets.rb')
+  end
+end

data/{generators → lib/generators}/npo_assets/templates/migration.rb

@@ -2,6 +2,7 @@ class CreateAssets < ActiveRecord::Migration
   def self.up
     create_table :assets do |t|
       t.string :url
+      t.text :original_url
       t.integer :remote_id
     end
   end

data/lib/npo_assets/asset.rb

@@ -75,6 +75,9 @@ module NPO
             vars['errors']['error'].each { |msg| errors.add_to_base(msg) }
             false
           else
+            if self.respond_to?(:original_url=) && attrs[:url]
+              self.original_url = attrs[:url]
+            end
             self.url       = vars['asset']['url']
             self.remote_id = vars['asset']['id']
             true

data/npo_assets.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{npo_assets}
-  s.version = "0.4.3"
+  s.version = "0.5.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Bart Zonneveld"]
-  s.date = %q{2011-07-13}
+  s.date = %q{2011-08-16}
   s.description = %q{Wrapper for media db}
   s.email = %q{loop@superinfinite.com}
   s.extra_rdoc_files = [
@@ -22,9 +22,9 @@ Gem::Specification.new do |s|
     "README.rdoc",
     "Rakefile",
     "VERSION",
-    "generators/npo_assets/npo_assets_generator.rb",
-    "generators/npo_assets/templates/initializer.rb",
-    "generators/npo_assets/templates/migration.rb",
+    "lib/generators/npo_assets/npo_assets_generator.rb",
+    "lib/generators/npo_assets/templates/initializer.rb",
+    "lib/generators/npo_assets/templates/migration.rb",
     "lib/npo_assets.rb",
     "lib/npo_assets/asset.rb",
     "npo_assets.gemspec",

data/spec/npo_assets/asset_spec.rb

@@ -64,7 +64,24 @@ describe NPO::Assets::Asset do
           do_create
           @asset.attributes['remote_id'].should == 1
         end
+        
+        describe "with original url column" do
+          it "should assign the original url to the asset" do
+            @asset = NPO::Assets::Asset.new(:remote_url => "my_remote_image")
+            do_create
+            @asset.attributes['original_url'].should == "my_remote_image"
+          end
+        end
       
+        describe "without original url column" do
+          it "should not assign the original url to the asset" do
+            @asset = NPO::Assets::Asset.new(:remote_url => "my_remote_image")
+            @asset.stub!(:respond_to?).and_return false
+            do_create
+            @asset.attributes['original_url'].should be_nil
+          end
+        end
+        
         it "should save the asset" do
           do_create
           @asset.should_not be_new_record
@@ -263,10 +280,10 @@ describe NPO::Assets::Asset do
       
       it "should save the assets with the correct attributes" do
         do_list
-        NPO::Assets::Asset.first.attributes.should == {"remote_id" => 10, "url" => "000/000/001.jpg", "id" => 1}
-        NPO::Assets::Asset.last.attributes.should ==  {"remote_id" => 20, "url" => "000/000/002.jpg", "id" => 2}
+        NPO::Assets::Asset.first.attributes.should == {"remote_id" => 10, "url" => "000/000/001.jpg", "id" => 1, "original_url"=>nil}
+        NPO::Assets::Asset.last.attributes.should ==  {"remote_id" => 20, "url" => "000/000/002.jpg", "id" => 2, "original_url"=>nil}
       end
-      
+
       it "should return the created objects" do
         do_list.should == NPO::Assets::Asset.all
       end

data/spec/spec_helper.rb

@@ -10,7 +10,7 @@ require 'spec/autorun'
 begin
   require 'sqlite3'
 rescue MissingSourceFile => e
-  puts "Gem sqlite3-ruby could not be found, please install it"
+  puts "Gem sqlite3 could not be found, please install it"
   exit
 end
 
@@ -23,6 +23,7 @@ def setup_db
   ActiveRecord::Schema.define(:version => 1) do
     create_table :assets do |t|
       t.string :url
+      t.text :original_url
       t.integer :remote_id
     end
   end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: npo_assets
 version: !ruby/object:Gem::Version 
-  hash: 9
+  hash: 11
   prerelease: 
   segments: 
   - 0
-  - 4
-  - 3
-  version: 0.4.3
+  - 5
+  - 0
+  version: 0.5.0
 platform: ruby
 authors: 
 - Bart Zonneveld
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-07-13 00:00:00 +02:00
+date: 2011-08-16 00:00:00 +02:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -63,9 +63,9 @@ files:
 - README.rdoc
 - Rakefile
 - VERSION
-- generators/npo_assets/npo_assets_generator.rb
-- generators/npo_assets/templates/initializer.rb
-- generators/npo_assets/templates/migration.rb
+- lib/generators/npo_assets/npo_assets_generator.rb
+- lib/generators/npo_assets/templates/initializer.rb
+- lib/generators/npo_assets/templates/migration.rb
 - lib/npo_assets.rb
 - lib/npo_assets/asset.rb
 - npo_assets.gemspec

data/generators/npo_assets/npo_assets_generator.rb

@@ -1,8 +0,0 @@
-class NpoAssetsGenerator < Rails::Generator::Base
-  def manifest
-    record do |m|
-      m.migration_template('migration.rb', 'db/migrate', :migration_file_name => 'create_assets')
-      m.file('initializer.rb', 'config/initializers/npo_assets.rb')
-    end
-  end
-end