FHEM/mod-AptToDate
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

fix 'Can't call method readFromChild on an undefined value'

Browse Source
This commit is contained in:
Marko Oldenburg 2019-03-24 10:24:00 +01:00
parent bc8f2ac22f
commit 380cea01ed
1 changed files with 31 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

52
42_AptToDate.pm
View File

@ -2,7 +2,7 @@
# #
# Developed with Kate # Developed with Kate
# #
# (c) 2017-2018 Copyright: Marko Oldenburg (leongaultier at gmail dot com) # (c) 2017-2019 Copyright: Marko Oldenburg (leongaultier at gmail dot com)
# All rights reserved # All rights reserved
# #
# Special thanks goes to: # Special thanks goes to:
@ -35,7 +35,7 @@ package main;
use strict; use strict;
use warnings; use warnings;
my $version = "1.4.2"; my $version = "1.4.3";
sub AptToDate_Initialize($) { sub AptToDate_Initialize($) {
@ -499,24 +499,27 @@ sub PollChild($) {
my $hash = shift; my $hash = shift;
my $name = $hash->{NAME}; my $name = $hash->{NAME};
my $subprocess = $hash->{".fhem"}{subprocess};
my $json = $subprocess->readFromChild(); if ( defined($hash->{".fhem"}{subprocess}) ) {
my $subprocess = $hash->{".fhem"}{subprocess};
my $json = $subprocess->readFromChild();
if ( !defined($json) ) { if ( !defined($json) ) {
Log3 $name, 5, "AptToDate ($name) - still waiting (" Log3 $name, 5, "AptToDate ($name) - still waiting ("
. $subprocess->{lasterror} . ")."; . $subprocess->{lasterror} . ").";
InternalTimer( gettimeofday() + POLLINTERVAL, InternalTimer( gettimeofday() + POLLINTERVAL,
"AptToDate::PollChild", $hash, 0 ); "AptToDate::PollChild", $hash, 0 );
return; return;
} }
else { else {
Log3 $name, 4, Log3 $name, 4,
"AptToDate ($name) - got result from asynchronous parsing."; "AptToDate ($name) - got result from asynchronous parsing.";
$subprocess->wait(); $subprocess->wait();
Log3 $name, 4, "AptToDate ($name) - asynchronous finished."; Log3 $name, 4, "AptToDate ($name) - asynchronous finished.";
CleanSubprocess($hash); CleanSubprocess($hash);
PreProcessing( $hash, $json ); PreProcessing( $hash, $json );
}
} }
} }
@ -843,9 +846,16 @@ sub WriteReadings($$) {
readingsBulkUpdateIfChanged( $hash, 'updatesAvailable', readingsBulkUpdateIfChanged( $hash, 'updatesAvailable',
scalar keys %{ $decode_json->{packages} } ) scalar keys %{ $decode_json->{packages} } )
if ( $hash->{".fhem"}{aptget}{cmd} eq 'getUpdateList' ); if ( $hash->{".fhem"}{aptget}{cmd} eq 'getUpdateList' );
readingsBulkUpdateIfChanged( $hash, 'upgradeListAsJSON',
eval { encode_json( $hash->{".fhem"}{aptget}{packages} ) } ) if ( scalar keys%{ $hash->{".fhem"}{aptget}{packages} } > 0 ) {
if ( AttrVal( $name, 'upgradeListReading', 'none' ) ne 'none' ); readingsBulkUpdateIfChanged( $hash, 'upgradeListAsJSON',
eval { encode_json( $hash->{".fhem"}{aptget}{packages} ) } )
if ( AttrVal( $name, 'upgradeListReading', 'none' ) ne 'none' );
}
else { readingsBulkUpdateIfChanged( $hash, 'upgradeListAsJSON', '' )
if ( AttrVal( $name, 'upgradeListReading', 'none' ) ne 'none' );
}
readingsBulkUpdate( $hash, 'toUpgrade', 'successful' ) readingsBulkUpdate( $hash, 'toUpgrade', 'successful' )
if ( $hash->{".fhem"}{aptget}{cmd} eq 'toUpgrade' if ( $hash->{".fhem"}{aptget}{cmd} eq 'toUpgrade'
and not defined( $hash->{".fhem"}{aptget}{'errors'} ) and not defined( $hash->{".fhem"}{aptget}{'errors'} )