@@ -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" lib/acts_as_amazon_product.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,6 +78,11 @@ 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(/4104N6ME70L\._SL75_\.jpg/, @book_gtd.amazon.small_image_url) test/acts_as_amazon_product_test.rb d797ef9d7fea6f2512ee8db7c701910ca9bcaf91 Scott Nedderman scott@netphase.com http://github.com/netphase/aaap/commit/48a80e7bedbb0d5f24a31b93278419da681207bd 48a80e7bedbb0d5f24a31b93278419da681207bd 2008-04-24T11:47:27-07:00 2008-04-24T11:47:27-07:00 Accepts search_index parameter to work better with Magazines (patch from Parker Morse) 15f76c2eabce2e6e9ceaa210fb60de04c06abc27 Scott Nedderman scott@netphase.com