Definitely fixed YouTube thumbnails sizes

lib/provider/youtube.rb

@@ -6,14 +6,14 @@ class Youtube
                 :duration, :date, :width, :height,
                 :thumbnail_small, :thumbnail_large,
                 :view_count
-  
+
   def initialize(url)
     @video_id = url.gsub(/.*v=([^&]+).*$/i, '\1')
     get_info unless @video_id == url
   end
-  
+
 private
-  
+
   def get_info
     doc = Hpricot(open("http://gdata.youtube.com/feeds/api/videos/#{@video_id}"))
     @provider         = "YouTube"
@@ -23,15 +23,15 @@ def get_info
     @keywords         = doc.search("media:keywords").inner_text
     @duration         = doc.search("yt:duration").first[:seconds].to_i
     @date             = Time.parse(doc.search("published").inner_text, Time.now.utc)
-    @thumbnail_small  = doc.search("media:thumbnail").last[:url]
-    @thumbnail_large  = doc.search("media:thumbnail").first[:url]
+    @thumbnail_small  = doc.search("media:thumbnail").min { |a,b| a[:height].to_i * a[:width].to_i <=> b[:height].to_i * b[:width].to_i }[:url]
+    @thumbnail_large  = doc.search("media:thumbnail").max { |a,b| a[:height].to_i * a[:width].to_i <=> b[:height].to_i * b[:width].to_i }[:url]
     # when your video still has no view, yt:statistics is not returned by Youtube
     # see: https://github.com/thibaudgg/video_info/issues#issue/2
     if doc.search("yt:statistics").first
       @view_count     = doc.search("yt:statistics").first[:viewcount].to_i
     else
-      @view_count     = 0 
+      @view_count     = 0
     end
   end
-  
+
 end

spec/fixtures/vcr_cassettes/vimeo/898029.yml

@@ -13,17 +13,15 @@
       x-powered-by: 
       - PHP/5.3.3
       expires: 
-      - Wed, 12 Jan 2011 20:05:48 GMT
+      - Sat, 16 Apr 2011 23:26:17 GMT
       content-type: 
       - text/xml;charset=UTF-8
       server: 
       - Apache
       date: 
-      - Thu, 13 Jan 2011 08:05:48 GMT
+      - Sun, 17 Apr 2011 11:26:17 GMT
       content-length: 
       - "1462"
-      x-server: 
-      - 10.90.6.240
       cache-control: 
       - no-store, no-cache, must-revalidate
       - post-check=0, pre-check=0
@@ -48,7 +46,7 @@
           <user_portrait_large>http://b.vimeocdn.com/ps/146/987/1469877_100.jpg</user_portrait_large>
           <user_portrait_huge>http://b.vimeocdn.com/ps/146/987/1469877_300.jpg</user_portrait_huge>
           <stats_number_of_likes>8</stats_number_of_likes>
-          <stats_number_of_plays>4159</stats_number_of_plays>
+          <stats_number_of_plays>4244</stats_number_of_plays>
           <stats_number_of_comments>4</stats_number_of_comments>
           <duration>175</duration>
           <width>640</width>

spec/fixtures/vcr_cassettes/youtube/JM9NgvjjVng.yml

@@ -15,15 +15,15 @@
       x-frame-options: 
       - SAMEORIGIN
       expires: 
-      - Thu, 13 Jan 2011 08:05:47 GMT
+      - Sun, 17 Apr 2011 11:26:16 GMT
       last-modified: 
-      - Wed, 12 Jan 2011 21:43:53 GMT
+      - Sat, 16 Apr 2011 15:49:59 GMT
       content-type: 
       - application/atom+xml; charset=UTF-8
       server: 
       - GSE
       date: 
-      - Thu, 13 Jan 2011 08:05:47 GMT
+      - Sun, 17 Apr 2011 11:26:16 GMT
       x-xss-protection: 
       - 1; mode=block
       gdata-version: 
@@ -36,5 +36,5 @@
       - "*"
       transfer-encoding: 
       - chunked
-    body: <?xml version='1.0' encoding='UTF-8'?><entry xmlns='http://www.w3.org/2005/Atom' xmlns:app='http://purl.org/atom/app#' xmlns:media='http://search.yahoo.com/mrss/' xmlns:gd='http://schemas.google.com/g/2005' xmlns:yt='http://gdata.youtube.com/schemas/2007'><id>http://gdata.youtube.com/feeds/api/videos/JM9NgvjjVng</id><published>2008-04-21T15:34:35.000Z</published><updated>2011-01-12T21:43:53.000Z</updated><app:control><yt:state name='restricted' reasonCode='limitedSyndication'>Syndication of this video was restricted by its owner.</yt:state></app:control><category scheme='http://schemas.google.com/g/2005#kind' term='http://gdata.youtube.com/schemas/2007#video'/><category scheme='http://gdata.youtube.com/schemas/2007/categories.cat' term='Music' label='Music'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='amy'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='macdonald'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='rock'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='roll'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='poison'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='prince'/><title type='text'>Amy Macdonald - Poison Prince - 2008 Version</title><content type='text'>See the brand new video to Amy's new single 'Poison Prince' out May 19th 2008...</content><link rel='alternate' type='text/html' href='http://www.youtube.com/watch?v=JM9NgvjjVng&amp;feature=youtube_gdata'/><link rel='http://gdata.youtube.com/schemas/2007#video.responses' type='application/atom+xml' href='http://gdata.youtube.com/feeds/api/videos/JM9NgvjjVng/responses'/><link rel='http://gdata.youtube.com/schemas/2007#video.related' type='application/atom+xml' href='http://gdata.youtube.com/feeds/api/videos/JM9NgvjjVng/related'/><link rel='self' type='application/atom+xml' href='http://gdata.youtube.com/feeds/api/videos/JM9NgvjjVng'/><author><name>amymacdonaldofficial</name><uri>http://gdata.youtube.com/feeds/api/users/amymacdonaldofficial</uri></author><gd:comments><gd:feedLink href='http://gdata.youtube.com/feeds/api/videos/JM9NgvjjVng/comments' countHint='456'/></gd:comments><media:group><media:category label='Music' scheme='http://gdata.youtube.com/schemas/2007/categories.cat'>Music</media:category><media:description type='plain'>See the brand new video to Amy's new single 'Poison Prince' out May 19th 2008...</media:description><media:keywords>amy, macdonald, rock, roll, poison, prince</media:keywords><media:player url='http://www.youtube.com/watch?v=JM9NgvjjVng&amp;feature=youtube_gdata_player'/><media:restriction type='country' relationship='deny'>DE</media:restriction><media:thumbnail url='http://i.ytimg.com/vi/JM9NgvjjVng/2.jpg' height='90' width='120' time='00:01:48.500'/><media:thumbnail url='http://i.ytimg.com/vi/JM9NgvjjVng/1.jpg' height='90' width='120' time='00:00:54.250'/><media:thumbnail url='http://i.ytimg.com/vi/JM9NgvjjVng/3.jpg' height='90' width='120' time='00:02:42.750'/><media:thumbnail url='http://i.ytimg.com/vi/JM9NgvjjVng/0.jpg' height='240' width='320' time='00:01:48.500'/><media:title type='plain'>Amy Macdonald - Poison Prince - 2008 Version</media:title><yt:duration seconds='217'/></media:group><yt:noembed/><gd:rating average='4.880075' max='5' min='1' numRaters='1601' rel='http://schemas.google.com/g/2005#overall'/><yt:statistics favoriteCount='3057' viewCount='1528580'/></entry>
+    body: <?xml version='1.0' encoding='UTF-8'?><entry xmlns='http://www.w3.org/2005/Atom' xmlns:app='http://purl.org/atom/app#' xmlns:media='http://search.yahoo.com/mrss/' xmlns:gd='http://schemas.google.com/g/2005' xmlns:yt='http://gdata.youtube.com/schemas/2007'><id>http://gdata.youtube.com/feeds/api/videos/JM9NgvjjVng</id><published>2008-04-21T15:34:35.000Z</published><updated>2011-04-16T15:49:59.000Z</updated><app:control><yt:state name='restricted' reasonCode='limitedSyndication'>Syndication of this video was restricted by its owner.</yt:state></app:control><category scheme='http://schemas.google.com/g/2005#kind' term='http://gdata.youtube.com/schemas/2007#video'/><category scheme='http://gdata.youtube.com/schemas/2007/categories.cat' term='Music' label='Music'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='amy'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='macdonald'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='rock'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='roll'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='poison'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='prince'/><title type='text'>Amy Macdonald - Poison Prince - 2008 Version</title><content type='text'>See the brand new video to Amy's new single 'Poison Prince' out May 19th 2008...</content><link rel='alternate' type='text/html' href='http://www.youtube.com/watch?v=JM9NgvjjVng&amp;feature=youtube_gdata'/><link rel='http://gdata.youtube.com/schemas/2007#video.responses' type='application/atom+xml' href='http://gdata.youtube.com/feeds/api/videos/JM9NgvjjVng/responses'/><link rel='http://gdata.youtube.com/schemas/2007#video.related' type='application/atom+xml' href='http://gdata.youtube.com/feeds/api/videos/JM9NgvjjVng/related'/><link rel='self' type='application/atom+xml' href='http://gdata.youtube.com/feeds/api/videos/JM9NgvjjVng'/><author><name>amymacdonaldofficial</name><uri>http://gdata.youtube.com/feeds/api/users/amymacdonaldofficial</uri></author><gd:comments><gd:feedLink href='http://gdata.youtube.com/feeds/api/videos/JM9NgvjjVng/comments' countHint='468'/></gd:comments><media:group><media:category label='Music' scheme='http://gdata.youtube.com/schemas/2007/categories.cat'>Music</media:category><media:description type='plain'>See the brand new video to Amy's new single 'Poison Prince' out May 19th 2008...</media:description><media:keywords>amy, macdonald, rock, roll, poison, prince</media:keywords><media:player url='http://www.youtube.com/watch?v=JM9NgvjjVng&amp;feature=youtube_gdata_player'/><media:restriction type='country' relationship='deny'>DE</media:restriction><media:thumbnail url='http://i.ytimg.com/vi/JM9NgvjjVng/0.jpg' height='240' width='320' time='00:01:48.500'/><media:thumbnail url='http://i.ytimg.com/vi/JM9NgvjjVng/1.jpg' height='90' width='120' time='00:00:54.250'/><media:thumbnail url='http://i.ytimg.com/vi/JM9NgvjjVng/2.jpg' height='90' width='120' time='00:01:48.500'/><media:thumbnail url='http://i.ytimg.com/vi/JM9NgvjjVng/3.jpg' height='90' width='120' time='00:02:42.750'/><media:title type='plain'>Amy Macdonald - Poison Prince - 2008 Version</media:title><yt:duration seconds='217'/></media:group><yt:noembed/><gd:rating average='4.882353' max='5' min='1' numRaters='1666' rel='http://schemas.google.com/g/2005#overall'/><yt:statistics favoriteCount='3109' viewCount='1551835'/></entry>
     http_version: "1.1"

spec/fixtures/vcr_cassettes/youtube/mZqGqE0D0n4.yml

@@ -15,15 +15,15 @@
       x-frame-options: 
       - SAMEORIGIN
       expires: 
-      - Thu, 13 Jan 2011 08:05:47 GMT
+      - Sun, 17 Apr 2011 11:26:15 GMT
       last-modified: 
       - Thu, 06 Jan 2011 20:35:33 GMT
       content-type: 
       - application/atom+xml; charset=UTF-8
       server: 
       - GSE
       date: 
-      - Thu, 13 Jan 2011 08:05:47 GMT
+      - Sun, 17 Apr 2011 11:26:15 GMT
       x-xss-protection: 
       - 1; mode=block
       gdata-version: 
@@ -36,5 +36,5 @@
       - "*"
       transfer-encoding: 
       - chunked
-    body: <?xml version='1.0' encoding='UTF-8'?><entry xmlns='http://www.w3.org/2005/Atom' xmlns:media='http://search.yahoo.com/mrss/' xmlns:gd='http://schemas.google.com/g/2005' xmlns:yt='http://gdata.youtube.com/schemas/2007'><id>http://gdata.youtube.com/feeds/api/videos/mZqGqE0D0n4</id><published>2008-04-12T22:25:35.000Z</published><updated>2011-01-06T20:35:33.000Z</updated><category scheme='http://schemas.google.com/g/2005#kind' term='http://gdata.youtube.com/schemas/2007#video'/><category scheme='http://gdata.youtube.com/schemas/2007/categories.cat' term='Music' label='Music'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='cherry'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='bloom'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='king'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='of'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='the'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='knife'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='guitar'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='drum'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='clip'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='rock'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='alternative'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='tremplin'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='Paris-Forum'/><title type='text'>Cherry Bloom - King Of The Knife</title><content type='text'>The first video from the upcoming album Secret Sounds, to download in-stores April 14. Checkout http://www.cherrybloom.net</content><link rel='alternate' type='text/html' href='http://www.youtube.com/watch?v=mZqGqE0D0n4&amp;feature=youtube_gdata'/><link rel='http://gdata.youtube.com/schemas/2007#video.responses' type='application/atom+xml' href='http://gdata.youtube.com/feeds/api/videos/mZqGqE0D0n4/responses'/><link rel='http://gdata.youtube.com/schemas/2007#video.related' type='application/atom+xml' href='http://gdata.youtube.com/feeds/api/videos/mZqGqE0D0n4/related'/><link rel='self' type='application/atom+xml' href='http://gdata.youtube.com/feeds/api/videos/mZqGqE0D0n4'/><author><name>cherrybloomband</name><uri>http://gdata.youtube.com/feeds/api/users/cherrybloomband</uri></author><gd:comments><gd:feedLink href='http://gdata.youtube.com/feeds/api/videos/mZqGqE0D0n4/comments' countHint='13'/></gd:comments><media:group><media:category label='Music' scheme='http://gdata.youtube.com/schemas/2007/categories.cat'>Music</media:category><media:content url='http://www.youtube.com/v/mZqGqE0D0n4?f=videos&amp;app=youtube_gdata' type='application/x-shockwave-flash' medium='video' isDefault='true' expression='full' duration='175' yt:format='5'/><media:content url='rtsp://v1.cache6.c.youtube.com/CiILENy73wIaGQl-0gNNqIaamRMYDSANFEgGUgZ2aWRlb3MM/0/0/0/video.3gp' type='video/3gpp' medium='video' expression='full' duration='175' yt:format='1'/><media:content url='rtsp://v3.cache7.c.youtube.com/CiILENy73wIaGQl-0gNNqIaamRMYESARFEgGUgZ2aWRlb3MM/0/0/0/video.3gp' type='video/3gpp' medium='video' expression='full' duration='175' yt:format='6'/><media:description type='plain'>The first video from the upcoming album Secret Sounds, to download in-stores April 14. Checkout http://www.cherrybloom.net</media:description><media:keywords>cherry, bloom, king, of, the, knife, guitar, drum, clip, rock, alternative, tremplin, Paris-Forum</media:keywords><media:player url='http://www.youtube.com/watch?v=mZqGqE0D0n4&amp;feature=youtube_gdata_player'/><media:thumbnail url='http://i.ytimg.com/vi/mZqGqE0D0n4/2.jpg' height='90' width='120' time='00:01:27.500'/><media:thumbnail url='http://i.ytimg.com/vi/mZqGqE0D0n4/1.jpg' height='90' width='120' time='00:00:43.750'/><media:thumbnail url='http://i.ytimg.com/vi/mZqGqE0D0n4/3.jpg' height='90' width='120' time='00:02:11.250'/><media:thumbnail url='http://i.ytimg.com/vi/mZqGqE0D0n4/0.jpg' height='240' width='320' time='00:01:27.500'/><media:title type='plain'>Cherry Bloom - King Of The Knife</media:title><yt:duration seconds='175'/></media:group><gd:rating average='4.733333' max='5' min='1' numRaters='30' rel='http://schemas.google.com/g/2005#overall'/><yt:statistics favoriteCount='38' viewCount='4152'/></entry>
+    body: <?xml version='1.0' encoding='UTF-8'?><entry xmlns='http://www.w3.org/2005/Atom' xmlns:media='http://search.yahoo.com/mrss/' xmlns:gd='http://schemas.google.com/g/2005' xmlns:yt='http://gdata.youtube.com/schemas/2007'><id>http://gdata.youtube.com/feeds/api/videos/mZqGqE0D0n4</id><published>2008-04-12T22:25:35.000Z</published><updated>2011-01-06T20:35:33.000Z</updated><category scheme='http://schemas.google.com/g/2005#kind' term='http://gdata.youtube.com/schemas/2007#video'/><category scheme='http://gdata.youtube.com/schemas/2007/categories.cat' term='Music' label='Music'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='cherry'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='bloom'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='king'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='of'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='the'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='knife'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='guitar'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='drum'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='clip'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='rock'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='alternative'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='tremplin'/><category scheme='http://gdata.youtube.com/schemas/2007/keywords.cat' term='Paris-Forum'/><title type='text'>Cherry Bloom - King Of The Knife</title><content type='text'>The first video from the upcoming album Secret Sounds, to download in-stores April 14. Checkout http://www.cherrybloom.net</content><link rel='alternate' type='text/html' href='http://www.youtube.com/watch?v=mZqGqE0D0n4&amp;feature=youtube_gdata'/><link rel='http://gdata.youtube.com/schemas/2007#video.responses' type='application/atom+xml' href='http://gdata.youtube.com/feeds/api/videos/mZqGqE0D0n4/responses'/><link rel='http://gdata.youtube.com/schemas/2007#video.related' type='application/atom+xml' href='http://gdata.youtube.com/feeds/api/videos/mZqGqE0D0n4/related'/><link rel='self' type='application/atom+xml' href='http://gdata.youtube.com/feeds/api/videos/mZqGqE0D0n4'/><author><name>cherrybloomband</name><uri>http://gdata.youtube.com/feeds/api/users/cherrybloomband</uri></author><gd:comments><gd:feedLink href='http://gdata.youtube.com/feeds/api/videos/mZqGqE0D0n4/comments' countHint='13'/></gd:comments><media:group><media:category label='Music' scheme='http://gdata.youtube.com/schemas/2007/categories.cat'>Music</media:category><media:content url='http://www.youtube.com/v/mZqGqE0D0n4?f=videos&amp;app=youtube_gdata' type='application/x-shockwave-flash' medium='video' isDefault='true' expression='full' duration='175' yt:format='5'/><media:content url='rtsp://v1.cache6.c.youtube.com/CiILENy73wIaGQl-0gNNqIaamRMYDSANFEgGUgZ2aWRlb3MM/0/0/0/video.3gp' type='video/3gpp' medium='video' expression='full' duration='175' yt:format='1'/><media:content url='rtsp://v3.cache7.c.youtube.com/CiILENy73wIaGQl-0gNNqIaamRMYESARFEgGUgZ2aWRlb3MM/0/0/0/video.3gp' type='video/3gpp' medium='video' expression='full' duration='175' yt:format='6'/><media:description type='plain'>The first video from the upcoming album Secret Sounds, to download in-stores April 14. Checkout http://www.cherrybloom.net</media:description><media:keywords>cherry, bloom, king, of, the, knife, guitar, drum, clip, rock, alternative, tremplin, Paris-Forum</media:keywords><media:player url='http://www.youtube.com/watch?v=mZqGqE0D0n4&amp;feature=youtube_gdata_player'/><media:thumbnail url='http://i.ytimg.com/vi/mZqGqE0D0n4/0.jpg' height='240' width='320' time='00:01:27.500'/><media:thumbnail url='http://i.ytimg.com/vi/mZqGqE0D0n4/1.jpg' height='90' width='120' time='00:00:43.750'/><media:thumbnail url='http://i.ytimg.com/vi/mZqGqE0D0n4/2.jpg' height='90' width='120' time='00:01:27.500'/><media:thumbnail url='http://i.ytimg.com/vi/mZqGqE0D0n4/3.jpg' height='90' width='120' time='00:02:11.250'/><media:title type='plain'>Cherry Bloom - King Of The Knife</media:title><yt:duration seconds='175'/></media:group><gd:rating average='4.733333' max='5' min='1' numRaters='30' rel='http://schemas.google.com/g/2005#overall'/><yt:statistics favoriteCount='40' viewCount='4420'/></entry>
     http_version: "1.1"