youtubesearchresultscraper 0.0.5 → 0.0.6

data/CHANGELOG

@@ -15,3 +15,6 @@
 
 0.0.5 2007-02-16
   Fix scraping rule for video_count, video_from, video_to
+
+0.0.6 2007-02-16
+  Fix error handling for video_count, video_from, video_to

data/lib/youtube/searchresultscraper.rb

@@ -83,9 +83,9 @@ module Youtube #:nodoc:
 
     # Get search result from youtube by specified keyword.
     def open
-      url = @@youtube_search_base_url + CGI.escape(@keyword)
-      url += "&page=#{@page}" if not @page == nil
-      @html = Kernel.open(url).read
+      @url = @@youtube_search_base_url + CGI.escape(@keyword)
+      @url += "&page=#{@page}" if not @page == nil
+      @html = Kernel.open(@url).read
       replace_document_write_javascript
       @search_result = Hpricot.parse(@html)
     end
@@ -204,25 +204,34 @@ module Youtube #:nodoc:
 
     def scrape_video_count
       video_count = scrape_result_header
-      raise "no video count" unless video_count.sub!(/.+Results \d+-\d+ of( | about )([0-9,]+)/m , '\2')
+      unless video_count.sub!(/.+Results \d+-\d+ of( | about )([0-9,]+)/m , '\2')
+        raise "no video count: " + @url unless is_no_result
+      end
       video_count.gsub!(/,/, '')
       video_count.to_i
     end
 
     def scrape_video_from
       video_from = scrape_result_header
-      raise "no video from" unless video_from.sub!(/.+Results (\d+)/m, '\1')
+      unless video_from.sub!(/.+Results (\d+)/m, '\1')
+        raise "no video from: " + @url unless is_no_result
+      end
       video_from.to_i
     end
 
     def scrape_video_to
       video_to = scrape_result_header
-      raise "no video to" unless video_to.sub!(/.+Results \d+-(\d+)/m, '\1')
+      unless video_to.sub!(/.+Results \d+-(\d+)/m, '\1')
+        raise "no video to: "  + @url unless is_no_result
+      end
       video_to.to_i
     end
 
     def is_no_result
-        @html.include?('No Videos found')
+      if @is_no_result == nil
+        @is_no_result = @html.include?('No Videos found')
+      end
+      @is_no_result
     end
 
     def check_video video

data/lib/youtube/searchresultscraper.rb~

@@ -199,24 +199,25 @@ module Youtube #:nodoc:
     end
 
     def scrape_result_header
-      @search_result.search("div[@id='sectionHeader']/div").inner_html
+      @search_result.search("div[@id='sectionHeader']").inner_html
     end
 
     def scrape_video_count
-      video_count = scrape_result_header.sub(/.+of *(\d+)/m , '\1')
-      raise "no video count" if video_count.empty?
+      video_count = scrape_result_header
+      raise "no video count" unless video_count.sub!(/.+Results \d+-\d+ of( | about )([0-9,]+)/m , '\2')
+      video_count.gsub!(/,/, '')
       video_count.to_i
     end
 
     def scrape_video_from
-      video_from = scrape_result_header.sub(/Results *(\d+)-.+/m, '\1')
-      raise "no video from" if video_from.empty?
+      video_from = scrape_result_header
+      raise "no video from" unless video_from.sub!(/.+Results (\d+)/m, '\1')
       video_from.to_i
     end
 
     def scrape_video_to
-      video_to = scrape_result_header.sub(/Results.+-(\d+) *of.+/m, '\1')
-      raise "no video to" if video_to.empty?
+      video_to = scrape_result_header
+      raise "no video to" unless video_to.sub!(/.+Results \d+-(\d+)/m, '\1')
       video_to.to_i
     end
 

metadata

@@ -3,7 +3,7 @@ rubygems_version: 0.9.0
 specification_version: 1
 name: youtubesearchresultscraper
 version: !ruby/object:Gem::Version 
-  version: 0.0.5
+  version: 0.0.6
 date: 2007-02-16 00:00:00 +09:00
 summary: This gem provide function to scrape html of search result on youtube
 require_paths: