google_books 0.1.0 → 0.2.0

data/CHANGELOG.md

@@ -1,7 +1,7 @@
 CHANGELOG
 =========
 
-v0.0.1
+v0.1.0
 ------
 This is the initial version of google_books
 
@@ -30,3 +30,10 @@ This is the initial version of google_books
         * `:extra_large`
   *   `preview_link` - *String*
   *   `info_link` - *String*
+
+v0.2.0
+------
+*   New Features
+      *   Standardized publisher names in a similar fashion to the [google-book gem](https://rubygems.org/gems/google-book)
+*   Bug Fixes
+      *   The publishedDate field in the Google Books API are strings in the format of YYYY-MM-DD. Sometimes it's just YYYY and other times it's YYYY-MM.  Previously the `published_date` attribute was a Date type, now it is a string and you can convert the date however you want in your applications.

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    google_books (0.1)
+    google_books (0.2.0)
       hashie
       httparty
 

data/README.md

@@ -57,7 +57,10 @@ Book Attributes
 
 *   `authors`: *Array*
 *   `publisher`: *String*
-*   `published_date`: *Date*
+*   `published_date`: *String*
+
+    The `published_date` is represented by the format YYYY-MM-DD with MM and DD being optional
+    
 *   `isbn`: *String*
 *   `isbn_10`: *String*
 *   `page_count`: *FixNum*

data/lib/google_books/api/book.rb

@@ -12,18 +12,16 @@ module GoogleBooks
       end
       
       private
-      
+
       def parse_item(item)
         volume_info = item['volumeInfo']
         @book = Hashie::Mash.new volume_info
-        if @book.subtitle.nil?
-          @title = @book.title
-        else
-          @title = "#{@book.title}: #{titlize(@book.subtitle)}"
-        end
+        @title = build_title(@book)
         @authors = @book.authors || []
-        @publisher = @book.publisher
-        @published_date = @book.publishedDate
+        @publisher = normalize_publisher(@book.publisher)
+
+        @published_date = @book.publishedDate.to_s
+
         @description = @book.description
         @isbn = get_isbn_for_book
         @isbn_10 = get_isbn_for_book(10)
@@ -35,7 +33,27 @@ module GoogleBooks
         @preview_link = @book.previewLink
         @info_link = @book.infoLink
       end
+
+      def build_title(book)
+        return book.title if book.subtitle.nil?
+        "#{book.title}: #{titlize(book.subtitle)}"
+      end
       
+      def normalize_publisher(name)
+        return name if name.blank?
+        
+        name.
+          gsub(/[ ,]+Inc.?$/i, ' Inc.').
+          gsub(/[ ,]+Llc.?$/i, ' LLC.').
+          gsub(/[ ,]+Ltd.?$/i, ' Ltd.').
+          gsub(/Intl( |$)/i) { "International#{$1}" }.
+          gsub(/Pr( |$)/i) { "Press#{$1}" }.
+          gsub(/Pub Group( |$)/i) { "Publishing Group#{$1}" }.
+          gsub(/Pub( |$)/i) { "Publishers#{$1}" }.
+          gsub(/Pubns( |$)/i) { "Publications#{$1}" }.
+          gsub(/Univ( |$)/i) { "University#{$1}" }
+      end
+
       def get_isbn_for_book(type=13)
         return nil if @book.industryIdentifiers.nil?
         isbn = @book.industryIdentifiers.find { |id| id.type == "ISBN_#{type}" }

data/lib/google_books/version.rb

@@ -1,3 +1,3 @@
 module GoogleBooks
-  VERSION = '0.1.0'
+  VERSION = '0.2.0'
 end

data/spec/google_books/api/book_spec.rb

@@ -7,6 +7,10 @@ module GoogleBooks
       
       subject { API.search('isbn:9781935182320').first }
       
+      let(:item) do
+        item = { 'volumeInfo' => {} }
+      end
+      
       it "should be able to handle a nil object passed" do
         lambda { Book.new(nil) }.should_not raise_error
       end
@@ -26,14 +30,81 @@ module GoogleBooks
         subject.authors[1].should eq "Yehuda Katz"
       end
       
-      it "should have a publisher" do
-        subject.publisher.should_not be_nil
-        subject.publisher.should include "Manning"
-      end
-      
-      it "should have a published date in date format" do
-        subject.published_date.should be_a Date
-        subject.published_date.should eq Date.new(2010, 6, 30)
+      describe "publisher" do
+        it "should have a publisher" do
+          subject.publisher.should_not be_nil
+          subject.publisher.should include "Manning"
+        end
+        
+        it "should standardize an Inc to Inc." do
+          item['volumeInfo']['publisher'] = "Publisher Inc"
+          book = Book.new(item)
+          book.publisher.should eq "Publisher Inc."
+        end
+        
+        it "should standardize an Llc to LLC." do
+          item['volumeInfo']['publisher'] = "Publisher Llc"
+          book = Book.new(item)
+          book.publisher.should eq "Publisher LLC."
+        end
+        
+        it "should standardize an Ltd to Ltd." do
+          item['volumeInfo']['publisher'] = "Publisher Ltd"
+          book = Book.new(item)
+          book.publisher.should eq "Publisher Ltd."
+        end
+        
+        it "should replace Intl with International anywhere in the name" do
+          item['volumeInfo']['publisher'] = "Publisher Intl Clearing House"
+          book = Book.new(item)
+          book.publisher.should eq "Publisher International Clearing House"
+        end
+        
+        it "should replace Pr with Press anywhere in the name" do
+          item['volumeInfo']['publisher'] = "Publisher Pr House"
+          book = Book.new(item)
+          book.publisher.should eq "Publisher Press House"
+        end
+        
+        it "should replace Pub with Publishers anywhere in the name" do
+          item['volumeInfo']['publisher'] = "Publisher Pub, Inc"
+          book = Book.new(item)
+          book.publisher.should eq "Publisher Publishers Inc."
+        end
+        
+        it "should replace Pubns with Publications anywhere in the name" do
+          item['volumeInfo']['publisher'] = "Publisher Pubns Inc"
+          book = Book.new(item)
+          book.publisher.should eq "Publisher Publications Inc."
+        end
+        
+        it "should replace Pub Group with Publishing Group anywhere in the name" do
+          item['volumeInfo']['publisher'] = "Publisher Pub Group Inc"
+          book = Book.new(item)
+          book.publisher.should eq "Publisher Publishing Group Inc."
+        end
+        
+        it "should replace Univ with University anywhere in the name" do
+          item['volumeInfo']['publisher'] = "Publisher Univ Pr"
+          book = Book.new(item)
+          book.publisher.should eq "Publisher University Press"
+        end
+      end
+      
+      describe "published_date" do
+        it "should have a published date in sting format" do
+          subject.published_date.should eq "2010-06-30"
+        end
+
+        it "should handle a published date that is only a year" do
+          book = API.search('isbn:9781934356166').first
+          book.published_date.should eq "2009"
+        end
+
+        it "should handle a published date that is only a month and a year" do
+          book = API.search('isbn:9780954344405').first
+          book.published_date.should eq "2003-01"
+        end        
       end
       
       it "should have description (which may be blank)" do
@@ -66,29 +137,31 @@ module GoogleBooks
         subject.ratings_count.should be_a Fixnum
       end
       
-      it "should contain a covers hash" do
-        subject.covers.should be_a Hash
-        subject.covers.keys.should include :thumbnail
-        subject.covers.keys.should include :small
-        subject.covers.keys.should include :medium
-        subject.covers.keys.should include :large
-        subject.covers.keys.should include :extra_large        
-      end
-      
-      it "should not have curls on the cover urls" do
-        subject.covers[:thumbnail].should_not include 'edge=curl'
-        subject.covers[:small].should_not include 'edge=curl'
-        subject.covers[:medium].should_not include 'edge=curl'
-        subject.covers[:large].should_not include 'edge=curl'
-        subject.covers[:extra_large].should_not include 'edge=curl'
-      end
-      
-      it "should have the cover url zoom level" do
-        subject.covers[:thumbnail].should include 'zoom=5'
-        subject.covers[:small].should include 'zoom=1'
-        subject.covers[:medium].should include 'zoom=2'
-        subject.covers[:large].should include 'zoom=3'
-        subject.covers[:extra_large].should include 'zoom=6'
+      describe "covers" do
+        it "should contain a covers hash" do
+          subject.covers.should be_a Hash
+          subject.covers.keys.should include :thumbnail
+          subject.covers.keys.should include :small
+          subject.covers.keys.should include :medium
+          subject.covers.keys.should include :large
+          subject.covers.keys.should include :extra_large        
+        end
+
+        it "should not have curls on the cover urls" do
+          subject.covers[:thumbnail].should_not include 'edge=curl'
+          subject.covers[:small].should_not include 'edge=curl'
+          subject.covers[:medium].should_not include 'edge=curl'
+          subject.covers[:large].should_not include 'edge=curl'
+          subject.covers[:extra_large].should_not include 'edge=curl'
+        end
+
+        it "should have the cover url zoom level" do
+          subject.covers[:thumbnail].should include 'zoom=5'
+          subject.covers[:small].should include 'zoom=1'
+          subject.covers[:medium].should include 'zoom=2'
+          subject.covers[:large].should include 'zoom=3'
+          subject.covers[:extra_large].should include 'zoom=6'
+        end
       end
       
       it "should contains a preview link" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: google_books
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-08-07 00:00:00.000000000Z
+date: 2011-08-29 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
-  requirement: &70123559467780 !ruby/object:Gem::Requirement
+  requirement: &70187585301000 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70123559467780
+  version_requirements: *70187585301000
 - !ruby/object:Gem::Dependency
   name: hashie
-  requirement: &70123559467360 !ruby/object:Gem::Requirement
+  requirement: &70187585300580 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70123559467360
+  version_requirements: *70187585300580
 - !ruby/object:Gem::Dependency
   name: autotest
-  requirement: &70123559466940 !ruby/object:Gem::Requirement
+  requirement: &70187585300160 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70123559466940
+  version_requirements: *70187585300160
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70123559466520 !ruby/object:Gem::Requirement
+  requirement: &70187585299740 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70123559466520
+  version_requirements: *70187585299740
 - !ruby/object:Gem::Dependency
   name: webmock
-  requirement: &70123559466100 !ruby/object:Gem::Requirement
+  requirement: &70187585299320 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70123559466100
+  version_requirements: *70187585299320
 - !ruby/object:Gem::Dependency
   name: vcr
-  requirement: &70123559465680 !ruby/object:Gem::Requirement
+  requirement: &70187585298900 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,7 +76,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70123559465680
+  version_requirements: *70187585298900
 description: A Ruby wrapper that allows you to query the Google Books API. This project
   was inspired by google-book, see the README for more information
 email: