[mythtv-users] Merged PHP 8 changes

[glen]

On Tue, 2021-12-14 at 09:47 +0000, Nigel Jewell wrote:
> 
> On 14/12/2021 08:31, glen wrote:
> > On Mon, 2021-12-13 at 15:36 -0500, Peter Bennett wrote:
> > > Thank you to Nigel Jewell for the changes.
> > > 
> > > I tested them on PHP 7.4 and PHP 8.0, on mythtv master as well as
> > > mythtv
> > > v31.
> > > 
> > > The changes have been committed to master. If nobody has any
> > > issues I
> > > will merge them into fixes/31.
> > > 
> > > Peter
> > > 
> > > _______________________________________________
> > > mythtv-users mailing list
> > > mythtv-users at mythtv.org
> > > http://email.mg.glenb.net/c/eJxNjk0OwiAUhE9TdpLyTxcs3HiPBzxaEkBT0MTbW42JJrOZb2aSiS4kiEqS7LhWViLXMWirUxBSSG0kWxaD1if0TAEPduY0QMUyyXkt2DxtOMjmgCcfIDCTlqQ5LCwFVBaYYSgiakGK28a4TeI88cuhkvvotD7HNh70uq8HqpBLhfYNc0vXN_w0TveOeye7-7fHg9_-Bc3lQPE
> > > http://email.mg.glenb.net/c/eJxFjTkOwjAUBU8Td1jeYxcuaKjgDMjLd2LhBEh-QNyeSBRIrxrN6GWfSshakeqF0VaBMDkZa0qSSirTK-5cDzYWiFwHkSwTNIUJWqfY0GCOdAYko2dgZIkcWJCh59lmEQ0vIrrkss7BkuZHxEcnj5047XvXW6XTB0d80fsy7OQSaqvzcD3XFa-A9bkBIpDF_6zDtsKy7q__6AvN4zyn
> > > MythTV Forums:
> > > http://email.mg.glenb.net/c/eJxFjDsKAyEUAE-zdpGnPp9aWKTJPfzuBvYT1A3k9llIEZhmGJjsUw1ZI3t6SdpikZQTWapJoUIyKJwzxcZaotBBJguSp7CVdUKY17JHvpfBFm_ACGW0yFk5wBRtBXIBwJGteG3Z6pcxXn1S90k-LurRzo1vn7GMNz_azJr_ye3spfVr_29fZucywA
> > made the changes to mythweb and switched out to php8 current pkg on
> > arch linux. and i get the following error when setting a recording:
> > 
> > '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".
> 
> http://email.mg.glenb.net/c/eJwlzTsPwiAUBeBfUzYJXB62A4OLm5txB-6lNOnDFKrx34sxOcN3znLQxeTRaDY5DAKNhUDeU5TSnIXWkMDI6KUQaQAIafDU8-gXmjstxpnWwFeqLDsAShiRKNgeMA3SpGStkgYkKoXAZpdrfZZOXTq4toxTzUfgcVtauX1qvj8aloY3haaplINKg-nZ7n57fZ2OQntpz__Kt338Avb5PNc
> 
> Nige.
> 
> _______________________________________________
> mythtv-users mailing list
> mythtv-users at mythtv.org
> http://email.mg.glenb.net/c/eJxNjk0OwiAQhU9TdhKgQ0sXLNx4jynMtE2gmoIm3l40Jpq8zfv5khd9YIwWxObjrKIdzEyIFLS2owIwbKwOqJXiyZiZJyQnA2ZKHagl0T7LnapYPQwxOEs0Oka2PDV8Au5BE4B2IYjk11pvXX_uzKUpbaUWmZ91rQ95PZYWZdxSxv1bbjtf3-FncboXOoo4_L9tD378CwnZQQw
> http://email.mg.glenb.net/c/eJxFjbEOwiAURb-mbBKgYGFgcHHSb2ge8GiJtGr7qvHvbeJgcqeTc3KTjxmS0az4FEQyRxUQAKOUphNaq6yMjCCFyE6pkB2g5REmrI0WQ8U58BmJjb61xonkbEQjAbFzKnYWgg1aByNzYtWPRI-mPTXqvO9dboVPHxrpxe_LsJMrlFrmob-UlXqk8tyQCNnif9ZhW3FZ99d_9AUtKT0g
> MythTV Forums: http://email.mg.glenb.net/c/eJxFjMsKwyAQAL8m3iq7ZjXx4KGX_seqa1LIoxhT6N830ENhLsPA5JAKZ0vqGXKEbJ2JwiwJ0Q5AZIqxmBgBijcmFs8y6sSrLB3BtMgW9SZNzcERoBcAVxyPGMHmzAQevR3jQL2oJcytvY6uv3fmcVH2eq56_bS5vfVeJ1XDT27nIfW49v_2Bd-rM6I

Yes, thank you very much. I made the change in the file and it works as
it did under php 7.4. 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mythtv.org/pipermail/mythtv-users/attachments/20211214/ada620e4/attachment.htm>