[mythtv] [PATCH] Souces Setup in ./setup

[Matt Zimmerman]

On Sun, Jul 06, 2003 at 11:24:28AM -0700, Michael Greb wrote:

> --- Matt Zimmerman <mdz at debian.org> wrote:
> > How did he end up with unnamed sources?  Perhaps a check should be added to
> > disallow creating them this way in the first place, as it makes diagnosis
> > difficult.
> 
> It was an enter at the wrong moment, perhaps when typing zip.  I had the same
> problem when setting up MythTV on my first go at it.  I think your method has
> merit I am more at home with SQL then C++ though, hence my route.

It may be time that some validation were added to the settings code.  In
most cases, settings are selected from a (static or dynamic) list, and it is
impossible to enter something invalid, but there are now more and more text
entry boxes, and most of them should not be left blank, or should be unique.

I've added constraints to the database so that videosource names must be
unique and not null.  They could, of course, still end up as the empty
string and other nonsensical things.  I think a good approach would be to
fill the 'name' box with a default name (source1, source2, source3, etc.),
but this probably deserves a bit more thought.

-- 
 - mdz