Saratoga Templates with PHP 7

[Toxic17]

Hi there.

Is anyone running Saratoga Templates with PHP 7? I am thinking of upgrading the PHP from 5.6.29 and was wondering does anyone have php7 running without any severe issues?

any Deprecated commands need changed?

[BigOkie]

My Saratoga website isn't WAN accessible (without firewall modifications as I can access my network from the WAN) but I put 7.0.1 on the Pi I'm running the webserver on and it seems ok. I don't access it a lot, but the only deprecated message I could find was related to Cumulus server side graphs and having to change jpgraphs to 4.0.2 in order for it to work. 3.5.1 won't work on PHP7 as far as I can tell.

[Toxic17]

Thank for the info. Just looked at my jpgraphs they are 4.0.1 but http://jpgraph.net shows php7 was supported from 4.0.0

[saratogaWX]

I'm running PHP Version 7.0.14 on saratoga-weather.org and all the current release (stock) scripts are working fine with no errors. Some 3-rd party scripts may need updates which would be provided by those authors. The USA nws-alerts.php script does need a couple of variable initialization tweaks (to initialize as array() instead of '' (a null string)). PHP 7.1 stops doing auto-recasting of string-to-array and posts warning messages instead.

I'll be testing with PHP 7.1 a bit later when I have it available on my VPS.

[BCJKiwi]

Saratoga templates have worked / are working OK with 7.0.10, 7.0.11, 7.0.12, 7.0.13 and 7.0.14 but not with 7.1.0 because of changes to array handling.

[saratogaWX]

Other than the nws-alerts.php issues with PHP 7.1, did you find any other scripts that were having issues??

[BCJKiwi]

Ken,
Don't actually use NWS Alerts (in New Zealand!!)
The only one I have found so far is the datasummary scripts (creates arrays with [] but could be something else?) by beteljuice.

You can check out the scripts I am using at the link in my sig below BUT the public site is running php 7.0.13. Have various versions on my internal WAMP server for development/testing.

PM or email me if you want any further info

[Toxic17]

Thanks for the responses guys, I've just installed a new package for my NAS (QApache - Apache 2.4.25/ PHP 7.1.0 +PECL) and setup a new Web share.

So far so good, had to change a few paths etc, but will keep an eye out for any issues and let you know what I find.

[BCJKiwi]

Just upgraded the development webserver from Wampserver v3.06 to v3.08 which was required to install PHP v7.1.1
Unfortunately PHP v7.1.1 has the same issue as PHP v7.1.0 with the one webpage mentioned before. Everything else appeared OK.
Pretty sure this relates to the array changes in v7.1.x
Also installed the just released PHP v7.0.15 - it is amazing to see how many bugs they are still fixing in 7.0

v7.0.15 appears OK as the same webpage that does not work in 7.1.x is fine in 7.0.15.

[Toxic17]

Running php 7.1.1 here and so far so good. Also added https to my virtual hosts too!

[BCJKiwi]

Running PHP 7.1.1 here too. Found a fix for the array problem so that page is good now.

Also found this

PHP Deprecated: Methods with the same name as their class will not be constructors in a future version of PHP

when looking at the PHP error logs when sorting out the array changes.
Advised the writer of that script and it has been updated already

[Toxic17]

Running PHP 7.1.2 now. all seems ok.

[saratogaWX]

Me too.. upgraded yesterday to 7.1.2 also for the saratoga main site (and 8 regional networks).

Only one issue found in mesonet-map-common.php that cast a Warning: inside the JSON when a station wasn't reporting Gusts (so the map disappeared).

Changed mesonet-map-common.php

Code: Select all

  if ($GUST <> '-') {
   if (!preg_match('|m/s|',$uomWind)) { // convert wind to meters-per-second

to

Code: Select all

  if ($GUST <> '-' and is_numeric($GUST)) {
   if (!preg_match('|m/s|',$uomWind)) { // convert wind to meters-per-second


and that fixed the issue.

[Toxic17]

Thanks Ken, just amended mine now.

[Toxic17]

just upgraded to 7.1.3. all seems good so far.