[mythtv] Ticket #4599: Myth archive/cleanup script.

[Mark Buechler]

On Mon, Mar 31, 2008 at 12:27 PM, Mark Buechler <mark.buechler at gmail.com>
wrote:

> 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.
>
>

Upon further inspection, maybe I can use the StorageGroup class instead.


>
> > >  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/1fb2de71/attachment.htm