flowhella / get-flash-videos

Automatically exported from code.google.com/p/get-flash-videos
0 stars 0 forks source link

Hulu duration is confusing for videos longer than 1 hour. #374

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Find a Hulu video with a duration of over 1 hour. "Kojak 2004 pilot"
2.
3.

What is the expected output? What do you see instead?
The duration should be displayed as 1:28:29, alongside other episodes displayed 
as 41:10 and similiar.

The duration is actually displayed as 1:28, making it appear to be either a 
clip or preview, instead of a full episode.

What version of the product are you using? On what operating system?
get_flash_video-1.24
Hulu.pm dated 17-Mar-2012
Fedora release 15 (Lovelock)

Please provide any additional information below.

I don't know perl, so I fixed it like this:

--- Hulu.pm.old   2012-03-17 17:05:32.000000000 -0500
+++ Hulu.pm       2012-03-28 17:29:23.000000000 -0600
@@ -769,7 +769,7 @@
     my $ishplus = $full_description =~ /Hulu Plus subscription required/i;
     my $hpmsg = "h+,preview";

-    if ($full_description =~ /Duration: (\d+:\d+)/) {
+    if ($full_description =~ /Duration: (\d+:\d+:?\d?\d?)/) {
       $duration = $1;
       my $hpsfx = $ishplus ? ",$hpmsg" : "";
       $title .= " ($duration$hpsfx)";

Original issue reported on code.google.com by hdus...@gmail.com on 28 Mar 2012 at 10:31

GoogleCodeExporter commented 8 years ago
Fixed in Gitorious, thanks for reporting.

Original comment by zakflash...@gmail.com on 9 Apr 2012 at 2:52