[mythtv] patch: setsetting -> db
Andy Davidoff
dert at pobox.com
Tue Feb 25 14:50:37 EST 2003
Where do you want the code, then, if not SetSetting()?
In the future, I would expect the settings to be cached and only flush
out to the database during FlushSettings() or maybe ~MythContext().
I have a lot of difficulty with authoring larger patches which implement
these sorts of features all at once, especially when it usually takes me
a day or two to massage the feature into a form that will be accepted.
Please provide some direction on exactly what you want, since saying
"that doesn't exist, feel free to submit a patch" is obviously a waste
of everyone's time.
#if Drew Whittle /* Feb 25, 09:50 */
> On Tue, 2003-02-25 at 05:18, Isaac Richards wrote:
> > I had kind of meant for SetSetting() to only be temporary storage, and not
> > write to the database. The only users of it are the setup program, when it
> > ensures that there's a Theme and a ThemeQt setting present, and mythvideo,
> > where the stuff it stores there is only temp storage, and shouldn't be
> > written to the DB.
>
> This sounds exactly like what I wanted.
>
> I am now sooooo confused it's not funny.
>
> First I get told that the "right" way to save a changed setting is to
> use:
>
> ThemeSelector theme;
> theme.setValue(themename);
> theme.save(db);
>
> Then I find that this isn't available in mythfrontend and I'll have to
> make it available, which then leads to a can of worms because of
> everything else that needs to be included.
>
> Along the way I discover SetSettings and AppearanceSettings.save and
> this seems that is should do what I want, I'm advised that it should
> work but it doesn't.
>
> Then I'm told that it doesn't do what I want, ok fair enough.
#endif /* drew at lug.net.nz */
More information about the mythtv-dev
mailing list