[mythtv] [mythtv-commits] Ticket #5595: Unable to create recording override (rev 17794?)

[John P Poet]

On Sat, Aug 2, 2008 at 9:23 PM, MythTV <mythtv at cvs.mythtv.org> wrote:
> #5595: Unable to create recording override (rev 17794?)
> ------------------------------+---------------------------------------------
>  Reporter:  gtgj at pacbell.net  |       Owner:  ijr
>     Type:  defect            |      Status:  new
>  Priority:  minor             |   Milestone:  unknown
> Component:  mythtv            |     Version:  unknown
>  Severity:  medium            |     Mlocked:  0
> ------------------------------+---------------------------------------------
>  Recording overrides stopped working for me between rev 17775
>  and 17867 (eg. selecting "Record anyway" on an upcoming program that's
>  not scheduled to record).  An entry does get added to the "record" DB
>  table but many fields are blank, such as "title", "subtitle",
>  "starttime", and "endtime".  Other fields are correct, such as
>  "description".
>
>  As far as I can tell, the fields are not getting saved because
>  of this check in libs/libmythdb/mythstorage.cpp:
>  {{{
>  void SimpleDBStorage::Save(QString _table)
>  {
>     if (user->GetValue() == initval)
>         return;
>  }}}
>  If I remove that check, the fields are filled in and the override
>  works properly.  It looks like this check was slightly different
>  before rev 17794.

Any ETA on a fix for this hitting trunk?  Is it safe for me to make
this change to my local build?

Thanks,

John
-- 
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?