[mythtv] Ticket #4599: Myth archive/cleanup script.
Mark Buechler
mark.buechler at gmail.com
Mon Mar 31 16:27:47 UTC 2008
Hi
On Tue, Mar 11, 2008 at 8:16 PM, Mark Buechler <mark.buechler at gmail.com>
wrote:
> Hi
>
>
> On Sun, Mar 9, 2008 at 3:07 PM, MythTV <mythtv at cvs.mythtv.org> wrote:
>
> > #4599: Myth archive/cleanup script.
> >
> > -----------------------------------------------------+----------------------
> > Reporter: Mark Buechler <Mark.Buechler at gmail.com> | Owner:
> > kormoc
> > Type: enhancement | Status:
> > infoneeded
> > Priority: minor | Milestone:
> > unknown
> > Component: contrib | Version:
> > unknown
> > Severity: medium | Resolution:
> > Mlocked: 0 |
> >
> > -----------------------------------------------------+----------------------
> >
> > Comment(by kormoc):
> >
> > I should be a tad more clear
> >
> > MythTV::get_recording_dirs should be used rather then getStorageGroups,
> > as
> > the bindings don't really return all the info you need, they should be
> > updated so everyone benifits
> >
>
How would you suggest I do this? Currently get_recording_dirs() returns an
array and my function returns a hash. If I modify the bindings version, I
will be breaking the API which dependent scripts may use already. I could
create another method like get_recording_groups(), I suppose. I think, for
the same of future features, if a change is to be made to
get_recording_dirs(), it should return a hashref of all columns in the
table.
>
> > Also, getRecordings should use the recordings class rather then psudo
> > rolling your own, unless there's a specific reason why that's not
> > acceptable?
> >
>
The Recording class didn't contain all that I needed. I can likely extend it
to fit though.
>
> > --
> > Ticket URL: <http://svn.mythtv.org/trac/ticket/4599#comment:4>
> > MythTV <http://svn.mythtv.org/trac>
> > MythTV
> >
>
> I'll take a look at those items in the next few days.
>
> - Mark.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mythtv.org/pipermail/mythtv-dev/attachments/20080331/ca54055c/attachment.htm
More information about the mythtv-dev
mailing list