Topic: WU-History php error

[tmabell]

Since I began using php8 I have found a few scripts that don't like it much.  The WU-History-inc.php is one of them.  I'm using Ken's version 3.4g dated 27 April 2020. 

Here is my page:  https://mymishawakaweather.com/History.php  The Daily & Weekly data populate correctly.  The problem can be seen by choosing "Month" or Year". 

This is the error: 

Code: [Select]

Fatal error: Uncaught TypeError: mktime(): Argument #4 ($month) must be of type ?int, string given in C:\www\WU-History-inc.php:1433 Stack trace: #0 C:\www\WU-History-inc.php(1433): mktime(0, 0, 0, '3-', 1, '2020') #1 C:\www\WU-History-inc.php(1018): AddDate('3-', '1', '2020', 0) #2 C:\www\History.php(218): include('C:\\www\\WU-Histo...') #3 {main} thrown in C:\www\WU-History-inc.php on line 1433
I have turned on error reporting which adds five lines at the top of the script.

[saratogaWX]

That was an easy fix.

I changed

Code: [Select]

$pmo = substr($data, 5, 2);
to

Code: [Select]

$pmo = substr($data, 5, 2);
$pmo = str_replace('-','',$pmo);

I updated https://github.com/ktrue/WU-History distribution with WU-History-inc.php V3.4h - 21-Mar-2021 with that fix.

Thanks for reporting it!

[tmabell]

Thanks for the quick fix Ken!  It's working great now. 

[tmabell]

This morning another glitch popped up:

Code: [Select]

Fatal error: Uncaught DivisionByZeroError: Division by zero in C:\www\WU-History-inc.php:1300 Stack trace: #0 C:\www\WU-History-inc.php(689): wDirAvg(Array) #1 C:\www\History.php(218): include('C:\\www\\WU-Histo...') #2 {main} thrown in C:\www\WU-History-inc.php on line 1300
This is on the Daily display which worked fine yesterday?

[saratogaWX]

I think you can fix it by changing 1299 in your WU-History-inc.php from

Code: [Select]

return round($tot / $num,0);
to

Code: [Select]

if($num>0) {
  return round($tot / $num,0);
} else {
  return(0);
}


[tmabell]

I didn't catch it in time to test that change because by 12:30 PM local time, the problem had gone away on its own.     I did change the code so time will tell.

Thank you for checking into this again.