acts_as_amazon_product 1.1 → 1.2

data/lib/acts_as_amazon_product.rb

@@ -21,16 +21,18 @@ module Netphase
             :asin => 'asin',
             :name => 'name',
             :access_key => ENV['AMAZON_ACCESS_KEY_ID'],
-            :associate_tag => ENV['AMAZON_ASSOCIATE_TAG']
+            :associate_tag => ENV['AMAZON_ASSOCIATE_TAG'],
+            :search_index => 'Books'
           }
           options = defaults.merge options
 
           Amazon::Ecs.options = {:aWS_access_key_id => options[:access_key], :associate_tag => options[:associate_tag] }
 
           write_inheritable_attribute(:amazon_asin, options[:asin])    
-          write_inheritable_attribute(:amazon_name, options[:name])    
+          write_inheritable_attribute(:amazon_name, options[:name])
+          write_inheritable_attribute(:amazon_search_index, options[:search_index])    
           write_inheritable_attribute(:amazon_associate_key, options[:associate_key])
-          class_inheritable_reader :amazon_asin, :amazon_name, :amazon_associate_key
+          class_inheritable_reader :amazon_asin, :amazon_name, :amazon_search_index, :amazon_associate_key
           
           has_one :amazon_product, :as => :amazonable   #, :dependent => :delete
           include Netphase::Acts::Amazonable::InstanceMethods
@@ -51,20 +53,21 @@ module Netphase
           if self.amazon_product.nil?
             asin = (self.respond_to?('amazon_asin')) ? self.send(self.amazon_asin) : nil
             name = (self.respond_to?('amazon_name')) ? self.send(self.amazon_name) : nil
+            search_index = (self.respond_to?('amazon_search_index')) ? self.amazon_search_index : 'Books'
             
             if !asin.nil? && asin.length > 0
-              #puts "Looking up #{asin}"
+              # puts "Looking up #{asin}"
               res = Amazon::Ecs.item_lookup(self.send(self.amazon_asin), :response_group => 'Medium')
               
               self.amazon_product =
                 AmazonProduct.new(:xml => res.doc.to_html, :asin => res.doc.at('asin').inner_html)
               self.amazon_product.save
             elsif !name.nil? && name.length > 0
-              #puts "Searching for #{name}"
-              res = Amazon::Ecs.item_search(self.send(self.amazon_name), :response_group => 'Medium') #, :sort => 'salesrank'
+              # puts "Searching for #{name}"
+              res = Amazon::Ecs.item_search(self.send(self.amazon_name), :search_index => self.amazon_search_index, :response_group => 'Medium') #, :sort => 'salesrank'
               res = res.doc.at('items/item')
               self.amazon_product =
-                AmazonProduct.new(:xml => res.to_html, :asin => res.at('itemattributes/isbn').inner_html)
+                AmazonProduct.new(:xml => res.to_html, :asin => (res.at('itemattributes/isbn').nil? ? res.at('asin').inner_html : res.at('itemattributes/isbn').inner_html))
               self.amazon_product.save
             else
               logger.error "No known attributes to search by"

data/test/acts_as_amazon_product_test.rb

@@ -27,6 +27,11 @@ ActiveRecord::Base.connection.create_table :movies do |t|
   t.column :asin, :string
 end
 
+ActiveRecord::Base.connection.create_table :magazines do |t|
+  t.column :name, :string
+  t.column :asin, :string
+end
+
 ActiveRecord::Base.connection.create_table :amazon_products do |t|  # , :id => false
   t.column :asin, :string
   t.column :xml, :text
@@ -45,6 +50,10 @@ class Movie < ActiveRecord::Base
   acts_as_amazon_product
 end
 
+class Magazine < ActiveRecord::Base
+  acts_as_amazon_product :search_index => 'Magazines'
+end
+
 AmazonProduct.delete_all
 
 class ActAsAmazonProductTest < Test::Unit::TestCase
@@ -55,6 +64,8 @@ class ActAsAmazonProductTest < Test::Unit::TestCase
       @book_ror = Book.create(:title => 'Rails Recipes')
       @book_perl = Book.create(:title => 'Perl')
       @movie_dh = Movie.create(:name=>'Live Free or Die Hard', :asin=>'B000VNMMRA')
+      Magazine.delete_all
+      @mag_lci = Magazine.create(:name => 'La Cucina Italiana')
     end
     
     def test_isbn
@@ -67,9 +78,14 @@ class ActAsAmazonProductTest < Test::Unit::TestCase
       assert_equal("Getting Things Done: The Art of Stress-Free Productivity", @book_gtd.amazon.title)
     end
     
+    def test_magazine
+      assert_not_nil(@mag_lci.amazon)
+      assert_equal("B00009XFML", @mag_lci.amazon.asin)
+    end
+    
     def test_small_image
       assert_not_nil(@book_gtd.amazon)
-      assert_match(/01PBN79FHEL\.jpg/, @book_gtd.amazon.small_image_url)
+      assert_match(/4104N6ME70L\._SL75_\.jpg/, @book_gtd.amazon.small_image_url)
     end
 
     def test_author

data/test/config-example.yml

@@ -1,6 +1,6 @@
 database:  
   adapter: sqlite3
-  dbfile: test.db
+  dbfile: test/test.db
 
 amazon:
   access_key: 1AAAABBBBCCCCDDDDEE2

metadata

@@ -1,61 +1,67 @@
 --- !ruby/object:Gem::Specification 
-rubygems_version: 0.9.0
-specification_version: 1
 name: acts_as_amazon_product
 version: !ruby/object:Gem::Version 
-  version: "1.1"
-date: 2007-11-03 00:00:00 -04:00
-summary: A package for simplifying use of the Amazon/ECS API
-require_paths: 
-- lib
-email: scott@netphase.com
-homepage: 
-rubyforge_project: 
-description: 
-autorequire: acts_as_amazon_product
-default_executable: 
-bindir: bin
-has_rdoc: true
-required_ruby_version: !ruby/object:Gem::Version::Requirement 
-  requirements: 
-  - - ">"
-    - !ruby/object:Gem::Version 
-      version: 0.0.0
-  version: 
+  version: "1.2"
 platform: ruby
-signing_key: 
-cert_chain: 
-post_install_message: 
 authors: 
 - Scott Nedderman
+autorequire: acts_as_amazon_product
+bindir: bin
+cert_chain: []
+
+date: 2008-04-24 00:00:00 -04:00
+default_executable: 
+dependencies: 
+- !ruby/object:Gem::Dependency 
+  name: amazon-ecs
+  version_requirement: 
+  version_requirements: !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        version: 0.5.1
+    version: 
+description: 
+email: scott@netphase.com
+executables: []
+
+extensions: []
+
+extra_rdoc_files: 
+- README
 files: 
 - lib/acts_as_amazon_product.rb
 - lib/amazon_product.rb
 - test/acts_as_amazon_product_test.rb
 - test/config-example.yml
 - test/example.rb
-- test/foo.rb
 - test/test.db
 - README
-test_files: []
-
+has_rdoc: true
+homepage: 
+post_install_message: 
 rdoc_options: []
 
-extra_rdoc_files: 
-- README
-executables: []
-
-extensions: []
-
+require_paths: 
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement 
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      version: "0"
+  version: 
+required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      version: "0"
+  version: 
 requirements: []
 
-dependencies: 
-- !ruby/object:Gem::Dependency 
-  name: amazon-ecs
-  version_requirement: 
-  version_requirements: !ruby/object:Gem::Version::Requirement 
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        version: 0.5.1
-    version: 
+rubyforge_project: 
+rubygems_version: 0.9.5
+signing_key: 
+specification_version: 2
+summary: A package for simplifying use of the Amazon/ECS API
+test_files: []
+

data/test/foo.rb

@@ -1,7 +0,0 @@
-
-def pathmap_partial(offset)
-
-end
-
-@path = "1/2/file"
-pathmap_partial()