[mythtv-commits] Ticket #13260: Mythweb throws php warning on 'recorded programmes' screen under php7.2
MythTV
noreply at mythtv.org
Sun Apr 22 19:29:00 UTC 2018
#13260: Mythweb throws php warning on 'recorded programmes' screen under php7.2
----------------------------------+--------------------------
Reporter: george.poulson@… | Owner: stuarta
Type: Patch - Bug Fix | Status: new
Priority: minor | Milestone: needs_triage
Component: Plugin - MythWeb | Version: v29-fixes
Severity: low | Keywords:
Ticket locked: 0 |
----------------------------------+--------------------------
MythTV version 29-fixes on lubuntu 18.04 (php version 7.2)
The 'recorded programmes' screen of mythweb reports two php warnings at
the top of the screen. All of the content below displays correctly.
The text of the errors is:
{{{
Warning at /usr/share/mythtv/mythweb/modules/tv/recorded.php, line 188:
!!NoTrans: Use of undefined constant disk_size - assumed 'disk_size' (this
will throw an Error in a future version of PHP)!!
Warning at /usr/share/mythtv/mythweb/modules/tv/recorded.php, line 189:
!!NoTrans: Use of undefined constant disk_used - assumed 'disk_used' (this
will throw an Error in a future version of PHP)!!
}}}
The fix is to add single quotes around the terms 'disk_size' and 'disk
used'
{{{
george at tvbox:/usr/share/mythtv/mythweb/modules/tv$ diff recorded.php
recorded.php~
188,189c188,189
< define(disk_size, ($size * 1024));
< define(disk_used, ($used * 1024));
---
> define('disk_size', ($size * 1024));
> define('disk_used', ($used * 1024));
}}}
I have only seen this on the latest [l]ubuntu release with php7.2. Earlier
releases are fine.
george at tvbox:~$ php --version
PHP 7.2.3-1ubuntu1 (cli) (built: Mar 14 2018 22:03:58) ( NTS )
Copyright (c) 1997-2018 The PHP Group
Zend Engine v3.2.0, Copyright (c) 1998-2018 Zend Technologies
with Zend OPcache v7.2.3-1ubuntu1, Copyright (c) 1999-2018, by Zend
Technologies
--
Ticket URL: <https://code.mythtv.org/trac/ticket/13260>
MythTV <http://www.mythtv.org>
MythTV Media Center
More information about the mythtv-commits
mailing list