Made the output filename the responsibility of the website subclass.

[dead/whatever-dl.git] / src / websites / veoh.rb

diff --git a/src/websites/veoh.rb b/src/websites/veoh.rb
index e10d9853d7ccaf45f4ffb823d6e5f13cd2362cec..c961d688e213b9500db2ab20726e966b2d896894 100644 (file)
--- a/src/websites/veoh.rb
+++ b/src/websites/veoh.rb
@@ -33,7 +33,7 @@ class Veoh < Website
   end
 
   
-  def get_video_url(url)
+  def get_video_url()
     # First, figure out the video id from the URL.
     # Then, use the video id to construct the video details URL.
     # Get the video details page, and parse the redirect
@@ -41,23 +41,25 @@ class Veoh < Website
     # id from the redirect, but for now we're going to rely
     # on our HTTP library to follow the redirect for us and
     # save us a step.
-    video_id = parse_video_id(url)
+    video_id = self.parse_video_id()
     details_url = "http://www.veoh.com/rest/video/#{video_id}/details"
     details_data = get_page_data(details_url)
     redirect_url = parse_redirect_url(details_data)
 
-    # Being slightly explicit about what we're doing here...
-    video_url = redirect_url
-    
-    return video_url
+    # We trust our HTTP library to do the right thing here.
+    return redirect_url
   end
 
+
+  def get_video_filename()
+    return (self.parse_video_id() + '.flv')
+  end
   
   protected;
 
-  def parse_video_id(url)
+  def parse_video_id()
     video_id_regex = /[[:alnum:]]+$/
-    matches = video_id_regex.match(url)
+    matches = video_id_regex.match(@url)
     video_id = matches[0] if not matches.nil?
     
     return video_id
@@ -77,7 +79,7 @@ class Veoh < Website
   
   def get_page_data(url)
     uri = URI.parse(url)
-    
+
     response = Net::HTTP.start(uri.host, uri.port) do |http|
       http.get(uri.path)
     end