flowhella / get-flash-videos

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

Add support for http://focusonthefamily.com/ #365

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
Many of the clips at http://focusonthefamily.com/popups/media_player.aspx work 
fine with get_flash_video's 'generic' method. However, the rtmp ones, such as 
at 
http://focusonthefamily.com/popups/media_player.aspx?Tab=Shows&subcategory=Radio
Theatre , require special support to properly assumble the RTMP URI.

I have written the attached site plugin which works for both the normal videos 
and rtmp ones that I have stumbled on. I include a `git am'-compatible patch 
too.

Original issue reported on code.google.com by ohnobinki on 6 Mar 2012 at 4:48

Attachments:

GoogleCodeExporter commented 8 years ago
Sorry for the slow reply. I've been using the embed URL (e.g. 
http://www.focusonthefamily.com/popups/media_player.aspx?MediaId={97AF3C5E-2786-
40A7-8214-36903B6B0D6B}) but I can't get any programmes to download, as it 
seems I'm triggering this message:

  Found renderPlayer() instead of renderPlayer3(). This function probably has different argument semantics.

There are a few minor problems with the code:
- Mixture of spaces and tabs (e.g. line 13 uses spaces but line 14 uses tabs). 
Please use spaces everywhere.
- No need to return $filename separately when you return a hash reference.
- Please add a test (see the t/urls) file.

Original comment by zakflash...@gmail.com on 9 Apr 2012 at 3:27

GoogleCodeExporter commented 8 years ago
This patch has fixed whitespace, drops the extraneous $filename, and includes 
some URIs for the tests.

I was unable to reproduce your problem of getting renderPlayer() instead of 
renderPlayer3(). Can you give the exact commandline you're using to invoke 
get_flash_videos with?

Original comment by ohnobinki on 10 Apr 2012 at 5:19

Attachments: