From 4c1c22529da73bf80cb95d5602236b9ba6c3f0d5 Mon Sep 17 00:00:00 2001 From: Michael Orlitzky Date: Thu, 1 Apr 2010 18:29:57 -0400 Subject: [PATCH] Modified the default get_video_filename() so that it removes URL parameters. --- src/website.rb | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/src/website.rb b/src/website.rb index 75f5aa8..bfa1b3d 100644 --- a/src/website.rb +++ b/src/website.rb @@ -114,8 +114,15 @@ class Website # of the video URL, but in some cases, subclasses will want # to override this behavior. def get_video_filename() - # Use whatever comes after the final front slash. - return get_video_url().split('/').pop() + # Use whatever comes after the final front slash. + file_and_params = get_video_url().split('/').pop() + + # Unless it contains URL parameters. We don't want those. + return file_and_params unless file_and_params.include?('?') + + # There must be some parameters. Strip them off. + param_start_idx = file_and_params.index('?') + return file_and_params[0...(param_start_idx)] end end -- 2.44.2