[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