[mythtv-users] Can't edit Schedules in MythFrontend

[David Hampton]

On Tue, 2021-12-14 at 16:36 +0000, Paul Harrison wrote:
> 
> On 14/12/2021 16:10, David Hampton via mythtv-users wrote:
>  
> > On Tue, 2021-12-14 at 15:55 +0000, Paul Harrison wrote:
> >  
> > > On 14/12/2021 15:30, James Abernathy wrote:
> > >  
> > >  
> > > >  
> > > > 
> > > >  
> > > > On Tue, Dec 14, 2021 at 10:23 AM David Hampton via mythtv-users
> > > > <mythtv-users at mythtv.org> wrote:
> > > >  
> > > >  
> > > > > On Tue, 2021-12-14 at 09:13 -0500, James Abernathy wrote:
> > > > >  > 
> > > > >  > 
> > > > >  > On Tue, Dec 14, 2021 at 8:38 AM James Abernathy
> > > > >  > <jfabernathy at gmail.com> wrote:
> > > > >  > > I just built from source Pre 32 3152 and noticed that in
> > > > > Manage
> > > > >  > > Recording -> Recording Rules I can't select a rule and
> > > > > edit
> > > > > the
> > > > >  > > schedule.  I was wanting to change the end time to end
> > > > > time +30,
> > > > >  > > but hitting enter on that line didn't do anything.  The
> > > > > other
> > > > >  > > options seem to work.  I didn't have this problem with
> > > > > previous P32
> > > > >  > > Master builds.
> > > > >  > > 
> > > > >  > > JIm A
> > > > >  > > 
> > > > >  > 
> > > > >  > 
> > > > >  > This jumped out at me in the console log:
> > > > >  > 
> > > > >  > 2021-12-14 09:08:59.766266 W  HTTPParser: Invalid HTTP
> > > > > request
> > > > > start
> > > > >  > 'CS_' - quitting
> > > > >  > 2021-12-14 09:09:08.747052 E  Container '' is missing
> > > > > child
> > > > >  > 'autoextend' at  
> > > > >  > 2021-12-14 09:09:08.747081 E  SchedOptEditor, theme is
> > > > > missing
> > > > >  > required elements
> > > > >  > 2021-12-14 09:09:16.812169 I  Created ExitPrompter
> > > > >  > 2021-12-14 09:09:17.741474 I  Deleted ExitPrompter
> > > > >  
> > > > >  You need to update your theme to the latest version.
> > > > >  
> > > > >  David
> > > > > 
> > > > Thanks, I was running Functionality 31 when I had the problem.
> > > > I
> > > > switched back to MythCenter 1.8 and it worked.
> > > > 
> > > > So how do I get  Functionality 31 to work in Pre32? Is this
> > > > something
> > > > that will get fixed by the release of v32 or am I stuck with
> > > > the
> > > > default theme?
> > > > 
> > > > Jim A
> > > >  
> > > >  
> > > You are stuck with a any theme that has been updated to support
> > > the new
> > > sports extender. Looks like Functionality has not been updated.
> > > 
> > > I have not looked at how David implemented the theme part but
> > > sometimes
> > > it;'s more appropriate to make new widgets like this optional so
> > > they
> > > throw a warning rather than stop working altogether. It is more
> > > work
> > > for the developer because they have to code for themes that do
> > > and
> > > don't have the new widgets.
> > Didn't know that was an option. Do you have a pointer to an
> > example? I
> > did update every theme in https://github.com/MythTV-themes that is
> > still packaged, and I submitted pull requests for three or four
> > external themes.
> > 
> > David
> > 
> 
> In this example anything that uses UIUtilE is mandatory and the
> screen will fail to load.
> https://github.com/MythTV/mythtv/blob/master/mythtv/programs/mythfrontend/guidegrid.cpp#L542
> Anything that uses UIUtilW is treated as optional so the screen will
> load but some widgets will be missing.
> So "guidegrid" is a mandatory widget and "datetext" is optional in
> the example above..
> If they are optional  the pointer gets set to null so before using
> one of the optional widgets you just have to do a NULL check.
> 
> Like I said I've not looked at any code so not sure if it's an
> appropriate method for what you want. If most of the active themes
> have been updated already  I don't see any need to make the change
> now anyway.

Ah, that explains the code pattern that I duplicated for the new item.
Thanks. Although it does raise a couple of questions about the
scheduler settings in general, that I'll have to investigate next time
I add a setting.

David