busk-ruby-readability 1.2.8 → 1.2.9

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    busk-ruby-readability (1.2.8)
+    busk-ruby-readability (1.2.9)
 
 GEM
   remote: http://rubygems.org/

data/lib/readability.rb

@@ -110,7 +110,12 @@ module Readability
 
     def google_videos
       uri = URI.parse(@base_uri + @request)
-      video_id = CGI::parse(uri.query)['docid'].first
+
+      if uri.fragment
+        video_id = CGI::parse(uri.fragment)['docid'].first
+      else
+        video_id = CGI::parse(uri.query)['docid'].first
+      end
       Nokogiri::HTML.fragment <<-HTML
         <div>
           <embed id=VideoPlayback src=http://video.google.com/googleplayer.swf?docid=#{video_id}&hl=en&fs=true style=width:400px;height:326px allowFullScreen=true allowScriptAccess=always type=application/x-shockwave-flash>

data/ruby-readability.gemspec

@@ -2,7 +2,7 @@ Gem::Specification.new do |s|
   s.authors = ["Fabio Mont Alegre", "Rodrigo Flores"]
   s.email = "it-team@busk.com"
   s.homepage = "http://github.com/busk/ruby-readability"
-  s.version = "1.2.8"
+  s.version = "1.2.9"
   s.name = "busk-ruby-readability"
   s.summary = "A rewrite of original ruby-readability"
 

data/spec/readability_spec.rb

@@ -269,8 +269,18 @@ describe Readability do
     end
 
     describe "Google Videos" do
-      it 'should extract the videos' do
-        url = "http://video.google.com/videoplay?docid=-4176721009838609904&hl=en#"
+      it 'should extract the videos from the anchor parameter' do
+        url = "http://video.google.com/videoplay?docid=-4176721009838609904&hl=en#docid=-3818636184384512295"
+        uri = URI.parse(url)
+        response = Net::HTTP.get_response(uri)
+        @parsed_page = Readability::Document.new(Nokogiri::HTML(response.body), uri.host, uri.request_uri + "#" + uri.fragment)
+        @content = @parsed_page.content
+
+        @content.should == " <div>\n <embed id=\"VideoPlayback\" src=\"http://video.google.com/googleplayer.swf?docid=-3818636184384512295&amp;hl=en&amp;fs=true\" style=\"width:400px;height:326px\" allowfullscreen=\"true\" allowscriptaccess=\"always\" type=\"application/x-shockwave-flash\"></embed>\n</div>"
+      end
+
+      it 'should extract the videos from the anchor parameter' do
+        url = "http://video.google.com/videoplay?docid=-4176721009838609904&hl=en"
         uri = URI.parse(url)
         response = Net::HTTP.get_response(uri)
         @parsed_page = Readability::Document.new(Nokogiri::HTML(response.body), uri.host, uri.request_uri)
@@ -278,6 +288,7 @@ describe Readability do
 
         @content.should == " <div>\n <embed id=\"VideoPlayback\" src=\"http://video.google.com/googleplayer.swf?docid=-4176721009838609904&amp;hl=en&amp;fs=true\" style=\"width:400px;height:326px\" allowfullscreen=\"true\" allowscriptaccess=\"always\" type=\"application/x-shockwave-flash\"></embed>\n</div>"
       end
+
     end
 
 

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: busk-ruby-readability
 version: !ruby/object:Gem::Version 
-  hash: 15
+  hash: 13
   prerelease: false
   segments: 
   - 1
   - 2
-  - 8
-  version: 1.2.8
+  - 9
+  version: 1.2.9
 platform: ruby
 authors: 
 - Fabio Mont Alegre