Topic: wxraintrending and wxglobalwarming scripts and php 8 and 8.1

[hcorrin]

has any one got these scripts wxraintrending and wxglobalwarming  working on php 8 or 8.1
when i tried php 8 last night i got a error 500 with wxraintrending and wxglobalwarming  gave a blank screen  have reverted back to 7.4

[saratogaWX]

I'd need to see the source for those pages (with any of the scripts included that are NOT basic Saratoga template scripts).

Please add them to a .zip file and post it, and I'll take a look.

[hcorrin]

Hi Ken
i can only find 2 files which i have attached as a zip
Thanks
Harold

[saratogaWX]

I've found the issue in these two scripts .. oddly enough, the same un-quoted constants in both.

The original code was

Code: [Select]

$uomTemp = $SITE[uomTemp]; // set here in case gizmo is not used
$uomRain = $SITE[uomRain]; // set here in case gizmo is not used
$imagesDir = $SITE[imagesDir];  // set here in case gizmo is not used
with the fix being

Code: [Select]

$uomTemp = $SITE['uomTemp']; // set here in case gizmo is not used
$uomRain = $SITE['uomRain']; // set here in case gizmo is not used
$imagesDir = $SITE['imagesDir'];  // set here in case gizmo is not used

and original of

Code: [Select]

$year = $today[year];
$month = $today[mon];
$day = $today[mday];
and the fix being

Code: [Select]

$year = $today['year'];
$month = $today['mon'];
$day = $today['mday'];

It was the same for wxglobalwarming.php and wxraintrending.php

[hcorrin]

That has sorted these scripts out and run ok

Thanks for that Ken

Harold

[Weerstation]

 

Thanks Guys 

Got my page working again PHP 8.1.10

 

[hcorrin]

Hi
I was finding the wxraintrending.php was not showing the footer
I found by commenting out lines 274 and 275 (yearrn) and lines 278 and 279 (monthrn) allowed the footer to show
this was then giving two lots of units so I removed "  . $uomRain" from lines 295 and 334
it appears to be working ok
have attached my PHP as a txt file

Harold