[mythtv-users] Merged PHP 8 changes

[James Abernathy]

On Tue, Dec 14, 2021 at 3:41 PM glen <glenb at glenb.net> wrote:

> On Tue, 2021-12-14 at 15:36 -0500, James Abernathy wrote:
>
>
>
> On Tue, Dec 14, 2021 at 3:07 PM Nigel Jewell <nigel.jewell at grufty.co.uk>
> wrote:
>
>
> >> 'Fatal error: Uncaught Error: Undefined constant "Math" in
> >> /srv/http/mythweb/modules/tv/classes/Schedule.php:286 Stack trace: #0
> >> /srv/http/mythweb/modules/tv/detail.php(250): Schedule->save() #1
> >> /srv/http/mythweb/modules/tv/handler.php(82):
> >> require_once('/srv/http/mythw...') #2
> >> /srv/http/mythweb/mythweb.php(35): require_once('/srv/http/mythw...')
> >> #3 {main} thrown
> >> in /srv/http/mythweb/modules/tv/classes/Schedule.php on line 286'
> >>
> >> I put it in as issue on the git. Everything else is working.
> >
> > Good spot.  I raised this some time ago for MythWeb, complete with
> > pull request.  I missed the "this will throw an Error in a future
> > version of PHP".
> >
> > https://github.com/MythTV/mythweb/issues/58
> <http://email.mg.glenb.net/c/eJwlzb0OwiAUBeCnaTcbLNALA4OLm5txvxcupUl_TKEa316MyRm-c5YTnI8YtGon5xFMAOCzEIPX1gAGgdgLjLIXZBk1kfAUO48Lz40S48wrdSuXNjl59tIQMWgwQg5RcVQEvQrBAlqQ7exSKc_cyEvTX2vGqaSDOr8ttdw-Jd0fFUvFm6lqyvngXKFNu7vfXl6nI_Oe6_O_dts-fgHsUTy9>
> >
>
> Thank to Peter's quick response - now merged into master.
>
> Nige.
>
>
> I just tried pulling mythweb from master again and I still get an error
> when setting a new recording from the Listing page.
>
> *Fatal error*: Uncaught Error: Undefined constant "Math" in
> /usr/share/mythtv/mythweb/modules/tv/classes/Schedule.php:286 Stack trace:
> #0 /usr/share/mythtv/mythweb/modules/tv/detail.php(250): Schedule->save()
> #1 /usr/share/mythtv/mythweb/modules/tv/handler.php(82):
> require_once('...') #2 /usr/share/mythtv/mythweb/mythweb.php(35):
> require_once('...') #3 {main} thrown in
> */usr/share/mythtv/mythweb/modules/tv/classes/Schedule.php* on line *286*
>
> Maybe I'm doing something wrong.
> Jim A
>
> _______________________________________________
> mythtv-users mailing list
> mythtv-users at mythtv.org
> http://lists.mythtv.org/mailman/listinfo/mythtv-users
> <http://email.mg.glenb.net/c/eJxNjs0OwiAQhJ-m3CTbHwo9cPDie-wC25IAmoImvr3VmGgyl_lmJhlvHaNXk4jWoTZe69ADzE4tRqMHxAGQxwFoCaiIwBFLhzmkboI1hUKyhCY2GxY_-8CaoHegzayQJmavUTMa6kkku7V268ZzN1wOpVhblfnZtvaQ1309UMaYMpZvGAtf3_DTON1r2KvY7b89Hvz2L51SQk4>
> thehttp://wiki.mythtv.org/Mailing_List_etiquette
> <http://email.mg.glenb.net/c/eJxFjc0OgjAQhJ-G3mxW6A8cevDiSZ-BbNstNAIqLBrf3iYeTOY0-eab6ELCqJXILqBto7V0BDBBd63FCIg1YGpq8B2h9h6CTzLgTFOlYJho8XIhFqMD1AmtaaLRljRiJFVEqjUhGU_GismNzI-qOVX1ueSdb1nOHx75Je_rUJor5ikvQ3_JG_fE-bkTM4nV_ajDvtG6ldf_6AtyvD2r>
> MythTV Forums: https://forum.mythtv.org
> <http://email.mg.glenb.net/c/eJxFjDsKAyEUAE-zdpHnXwuLNLnHU5-7gf0EdQO5fRZSBKYZBqbEXLEYzZ4xo_PFORIANpvgHRZAlIBVSUiB0KQEOVWecaN10jCvtCe-02BLLE6ioBSqNSJIo6zwFbRK2jhvrVRsjcsYrz6p-yQfF_Vo58a3z1jGmx9tZi3-5HZ2av3a_9sX-kEztQ>
>
> why don't you just simply look at your source and go to the file and line
> question. /mythweb/modules/tv/classes/Schedule.php and see if the word
> 'Math' appears in line 286. if it does. remove it. that's the fix.
>
>
Agreed, that fixes the problem locally, I'm just surprised that if the fix
has been pushed to master, then I would expect a new clone from mythweb
master branch would have the fix.  Maybe just one of timing of servier syncs
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mythtv.org/pipermail/mythtv-users/attachments/20211214/926912a4/attachment.htm>