[mythtv] MythWeb - recgroup set to null string for new recordings

[Chris Petersen]

> I actually have fixed this by adding recording group support to
> mythweb and am just waiting for my patch to be applied.

And so it has just been.  I also fixed your patch to display "" as 
"Default" so any future saves will have the correct recording group.

-Chris

> On Fri, 22 Oct 2004 18:37:36 -0400, Michael T. Dean
> <mtdean at thirdcontact.com> wrote:
> 
>>MythWeb creates new scheduled recordings with the recgroup set to the
>>null string (not NULL, but "").  Shouldn't it be set to 'Default' (since
>>MythWeb does not provide a capability to specify it)?
>>
>>I just noticed that every record in my recording and record tables had
>>recgroup set to the null string (I always schedule through MythWeb).
>>Unfortunately, this has a side effect of creating a recording group "".
>>When the user selects that recording group on the Watch Recordings
>>screen, a message (to the effect of) "No recordings to display" is shown
>>on top of the list of recordings that had been displayed before changing
>>group, and MythTV doesn't seem to like the group name too well, either.
>>(But I haven't had a chance to investigate the consequences fully.)
>>
>>The MySQL REPLACE statement only uses the column's default value for a
>>column if the column is not specified in the statement (
>>http://dev.mysql.com/doc/mysql/en/REPLACE.html ).  Since the MythWeb
>>REPLACE statements all specify the recgroup, the recgroup is not
>>properly set when creating a new scheduled recording, even though the
>>column has a default value of 'Default'.
>>
>>I won't have a chance to look at it for at least a week, but I want to
>>make sure it actually is a bug before I try to fix it.  :)  If so, I'll
>>probably try to work into the UI way to specify the recgroup.
>>
>>Thanks,
>>Mike
>>
>>_______________________________________________
>>mythtv-dev mailing list
>>mythtv-dev at mythtv.org
>>http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev
>>
>>
>>
>>------------------------------------------------------------------------
>>
>>_______________________________________________
>>mythtv-dev mailing list
>>mythtv-dev at mythtv.org
>>http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev