Opened 19 years ago

Closed 17 years ago

#4990 closed enhancement (wontfix)

Basic Podcast/Vodcast Support added to MythNews

Reported by: james@… Owned by: Isaac Richards
Priority: major Milestone: unknown
Component: mythnews Version: 0.21-fixes
Severity: medium Keywords: mythnews podcast vodcast
Cc: Ticket locked: no

Description

With 0.21 I discovered that mythnews could pull down media files. Yay I thought, a way to use it to add podcasting support. Unfortunately I discovered that it only downloaded the files to a temp file which got written over the next time a file was requested.

So I have gone through and very roughly added support for permanent downloads. It still needs a lot of work, including the ability to delete the files once they have been watched, and it is not the most optimal way of storing the files but it's a start.

Essentially what happens now, is this:

  1. A check is made to see if there is an existing directory under /MythNews/files/ for the Article Site. If it does not exist then it is created.
  1. A check is then made to see if the requested file exists within this directory (I know it's redundant if the directory has just been created, still very rough). If the file exists then it is played, if not then it is requested from the url and then stored and played.

Attachments (2)

mythnews_podcast_patch.patch​ (3.7 KB ) - added by james@… 18 years ago.
Updated version of the patch, ignore all previous
mythnews_patch.patch​ (3.8 KB ) - added by james@… 18 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 by james@…, 19 years ago

Sigh I knew things were going too well.

Turns out that the above patch actually breaks youtube downloads. Will work on a fix

by james@…, 18 years ago

Attachment: mythnews_podcast_patch.patch​ added

Updated version of the patch, ignore all previous

by james@…, 18 years ago

Attachment: mythnews_patch.patch​ added

comment:2 by laga, 17 years ago

Status: new → infoneeded_new

Your diff patches files which do not exist anymore in current trunk. I assume the diff was against 0.21.

Are you still interested in working on this? Note that some of this functionality may be provided by #5589 (although I'm not sure if this will make it in time for 0.22) or current MythNews.

comment:3 by stuartm, 17 years ago

Resolution: → wontfix
Status: infoneeded_new → closed

I think this is better implemented as a separate plugin, see 'MythTube' in #5589. Mythnews is best left for purely text feeds, since it's never going to do both the job of online media browser and news aggregator very well, those two things require different things from the UI.

Note: See TracTickets for help on using tickets.