Mailing List Archive

Ticket #13406: Patches to fix php count warnings with PHP 7.2
#13406: Patches to fix php count warnings with PHP 7.2
----------------------------------+---------------------------------
Reporter: daraden | Owner: Stuart Auchterlonie
Type: Patch - Bug Fix | Status: new
Priority: minor | Milestone: needs_triage
Component: Plugin - MythWeb | Version: Master Head
Severity: low | Keywords:
Ticket locked: 0 |
----------------------------------+---------------------------------
These patches are to resolve count() warnings when using PHP 7.2. Changes
just check if the object is not null before calling the count function,
otherwise the count variable is set to 0.

link to PHP manual noting the count() behavior as an incompatible change
in PHP 7.2[https://secure.php.net/manual/en/migration72.incompatible.php]

--
Ticket URL: <https://code.mythtv.org/trac/ticket/13406>
MythTV <http://www.mythtv.org>
MythTV Media Center
Re: Ticket #13406: Patches to fix php count warnings with PHP 7.2 [ In reply to ]
#13406: Patches to fix php count warnings with PHP 7.2
------------------------------+-------------------------------------
Reporter: daraden | Owner: Stuart Auchterlonie
Type: Patch - Bug Fix | Status: new
Priority: minor | Milestone: needs_triage
Component: Plugin - MythWeb | Version: Master Head
Severity: low | Resolution:
Keywords: | Ticket locked: 0
------------------------------+-------------------------------------
Changes (by daraden):

* Attachment "mythwebcount.patch" added.

mythweb count warning patch

--
Ticket URL: <https://code.mythtv.org/trac/ticket/13406>
MythTV <http://www.mythtv.org>
MythTV Media Center
Re: Ticket #13406: Patches to fix php count warnings with PHP 7.2 [ In reply to ]
#13406: Patches to fix php count warnings with PHP 7.2
------------------------------+-------------------------------------
Reporter: daraden | Owner: Stuart Auchterlonie
Type: Patch - Bug Fix | Status: new
Priority: minor | Milestone: needs_triage
Component: Plugin - MythWeb | Version: Master Head
Severity: low | Resolution:
Keywords: | Ticket locked: 0
------------------------------+-------------------------------------
Changes (by daraden):

* Attachment "mythcount.patch" added.

mythtv php bindings count patch

--
Ticket URL: <https://code.mythtv.org/trac/ticket/13406>
MythTV <http://www.mythtv.org>
MythTV Media Center
Re: Ticket #13406: Patches to fix php count warnings with PHP 7.2 [ In reply to ]
#13406: Patches to fix php count warnings with PHP 7.2
------------------------------+-------------------------------------
Reporter: daraden | Owner: Stuart Auchterlonie
Type: Patch - Bug Fix | Status: new
Priority: minor | Milestone: needs_triage
Component: Plugin - MythWeb | Version: Master Head
Severity: low | Resolution:
Keywords: | Ticket locked: 0
------------------------------+-------------------------------------

Comment (by Peter Bennett):

See also #13324, #13290

--
Ticket URL: <https://code.mythtv.org/trac/ticket/13406#comment:1>
MythTV <http://www.mythtv.org>
MythTV Media Center
Re: Ticket #13406: Patches to fix php count warnings with PHP 7.2 [ In reply to ]
#13406: Patches to fix php count warnings with PHP 7.2
------------------------------+-------------------------------------
Reporter: daraden | Owner: Stuart Auchterlonie
Type: Patch - Bug Fix | Status: accepted
Priority: minor | Milestone: 30.1
Component: Plugin - MythWeb | Version: Master Head
Severity: low | Resolution:
Keywords: | Ticket locked: 0
------------------------------+-------------------------------------
Changes (by Stuart Auchterlonie):

* status: new => accepted
* milestone: needs_triage => 30.1


Comment:

Needs backporting to fixes/29 and fixes/30

--
Ticket URL: <https://code.mythtv.org/trac/ticket/13406#comment:2>
MythTV <http://www.mythtv.org>
MythTV Media Center
Re: Ticket #13406: Patches to fix php count warnings with PHP 7.2 [ In reply to ]
#13406: Patches to fix php count warnings with PHP 7.2
------------------------------+-------------------------------------
Reporter: daraden | Owner: Stuart Auchterlonie
Type: Patch - Bug Fix | Status: accepted
Priority: minor | Milestone: 30.1
Component: Plugin - MythWeb | Version: Master Head
Severity: low | Resolution:
Keywords: | Ticket locked: 0
------------------------------+-------------------------------------

Comment (by hobbes1069):

Don't know if it's the right way but I "fixed" this on Fedora using the
following:

{{{
iff --git a/modules/tv/tmpl/default/detail.php
b/modules/tv/tmpl/default/detail.php
index f6d1faa9..4975f415 100644
--- a/modules/tv/tmpl/default/detail.php
+++ b/modules/tv/tmpl/default/detail.php
@@ -842,7 +842,7 @@
}
echo ' </ul>';
}
- if (count($program->jobs['queue'])) {
+ if (is_array($program->jobs['queue']) &&
count($program->jobs['queue'])) {
echo t('Queued jobs'), ':',
' <ul class="-queued">';
foreach ($program->jobs['queue'] as $job) {
@@ -856,7 +856,7 @@
}
echo ' </ul>';
}
- if (count($program->jobs['done'])) {
+ if (is_array($program->jobs['done']) &&
count($program->jobs['done'])) {
echo t('Recently completed jobs'), ':',
' <ul class="-done">';
foreach ($program->jobs['done'] as $job) {

}}}

--
Ticket URL: <https://code.mythtv.org/trac/ticket/13406#comment:3>
MythTV <http://www.mythtv.org>
MythTV Media Center
Re: Ticket #13406: Patches to fix php count warnings with PHP 7.2 [ In reply to ]
#13406: Patches to fix php count warnings with PHP 7.2
------------------------------+-------------------------------------
Reporter: daraden | Owner: Stuart Auchterlonie
Type: Patch - Bug Fix | Status: accepted
Priority: minor | Milestone: 30.1
Component: Plugin - MythWeb | Version: Master Head
Severity: low | Resolution:
Keywords: | Ticket locked: 0
------------------------------+-------------------------------------

Comment (by hobbes1069):

Would it be safe to apply the two patches to v30? I would like to fix this
for Fedora / RPM Fusion.

--
Ticket URL: <https://code.mythtv.org/trac/ticket/13406#comment:4>
MythTV <http://www.mythtv.org>
MythTV Media Center
Re: Ticket #13406: Patches to fix php count warnings with PHP 7.2 [ In reply to ]
#13406: Patches to fix php count warnings with PHP 7.2
------------------------------+-------------------------------------
Reporter: daraden | Owner: Stuart Auchterlonie
Type: Patch - Bug Fix | Status: accepted
Priority: minor | Milestone: 30.1
Component: Plugin - MythWeb | Version: Master Head
Severity: low | Resolution:
Keywords: | Ticket locked: 0
------------------------------+-------------------------------------
Changes (by daraden):

* Attachment "mythwebcount.patch" added.

fixed typo in mythwebcount.patch

--
Ticket URL: <https://code.mythtv.org/trac/ticket/13406>
MythTV <http://www.mythtv.org>
MythTV Media Center
Re: Ticket #13406: Patches to fix php count warnings with PHP 7.2 [ In reply to ]
#13406: Patches to fix php count warnings with PHP 7.2
------------------------------+-------------------------------------
Reporter: daraden | Owner: Stuart Auchterlonie
Type: Patch - Bug Fix | Status: accepted
Priority: minor | Milestone: 30.1
Component: Plugin - MythWeb | Version: Master Head
Severity: low | Resolution:
Keywords: | Ticket locked: 0
------------------------------+-------------------------------------
Changes (by daraden):

* Attachment "mythwebcount.patch" removed.

mythweb count warning patch

--
Ticket URL: <https://code.mythtv.org/trac/ticket/13406>
MythTV <http://www.mythtv.org>
MythTV Media Center
Re: Ticket #13406: Patches to fix php count warnings with PHP 7.2 [ In reply to ]
#13406: Patches to fix php count warnings with PHP 7.2
------------------------------+-------------------------------------
Reporter: daraden | Owner: Stuart Auchterlonie
Type: Patch - Bug Fix | Status: accepted
Priority: minor | Milestone: 30.1
Component: Plugin - MythWeb | Version: Master Head
Severity: low | Resolution:
Keywords: | Ticket locked: 0
------------------------------+-------------------------------------

Comment (by daraden):

Should work fine with v30(patches are working on my 29 backend). files the
patches modify are listed as last modified 2 or more years ago.

--
Ticket URL: <https://code.mythtv.org/trac/ticket/13406#comment:5>
MythTV <http://www.mythtv.org>
MythTV Media Center