[mythtv] FIX "do not autoexpire" Flag made persistent

Chris Pinkham cpinkham at bc2va.org
Tue Oct 5 19:36:07 UTC 2004


> As mentioned in a other posting, I found out myself that this discussion
> was already up 2 month ago.

And 2 months before that, and 2 months before that.... :)

> I didn't want to mess up your code im any way and I also not had any
> Intention that I should be added to the CVS. I think I'm to new here to
> already do such things.

I'm not taking it personally, just didn't want to add more confusion to
the mix by changing the meaning of the word "Max."  I'll be happy to apply
a patch if it implements something we can all agree on (ok, that most of
us can agree on) :)

> My sole intention was to add this feature for me as long as not anything
> similar is in Myth. And if someone who know what he is doing also have the
> possibility to ad this feature to his CVS. I did not want to step on anyones
> toe or offend someone, I was only learning to understand Myth and add a feature
> I already missed once ore twice.

Sorry if I came across that way.  I'd like to see the undeletable feature
added, but not by overloading the autoexpire field.

> To your replay:
> Sure a "undeletable" Flag will do it also, but I think as I did it in my
> Fix the so marked recordings should not be counted in the "maxepisodes"
> field or maybe to solve our intention and mine it would be best, when such an
> Episode is flagged as "undeletable" the following Popup will ask if the
> "Maxepisode" counter should be raised also by one - or maybe even a Setupflag
> for this behavior. ;-) Otherwise there would be another place to confuse
> the "normal" customer.

<quoted-out-of-order-paragraph>
> Some of the confusion maybe also raise because of the missleading
> translation from English to German. the German "nicht autom. löschen" means 
> "not delete automatically" which I think is something else than "don't autoexpire"
</quoted-out-of-order-paragraph>

I still think that autoexpire and maxepisodes should be kept separate since
autoexpire is based on disk capacity and maxepisodes is based on a number of
recordings.  The undeletable flag can apply to both. 
I can go along with the "Max Episodes" meaning "Max Deletable Episodes", so
that will mean changing the maxepisodes value won't be necessary, just
possibly some description and translation updates.

> The rest of your assumptions sounds right to me.
> The "undeletable" Flag should it only be possible to reset from the same
> user who set it or from anyone?

Since Myth doesn't support users, it's global so anyone could reset it.

> PS: I apologize in advance for wrong spellings ;-))

Too late, I've been foreevere tramatooriseded. :)  They say that our
brains auto-correct things like spelling errors usually, so I never even
saw them.  I had to go searching when I saw this paragraph.

Thanks for your help in making Myth better and if you can think of a more
accurate translation, patches are welcome for that as well.

-- 

Chris



More information about the mythtv-dev mailing list