Page 1 of 4

Saratoga Templates with PHP 7

Posted: Wed 11 Jan 2017 3:37 pm
by ConligWX
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?

Re: Saratoga Templates with PHP 7

Posted: Wed 11 Jan 2017 4:03 pm
by 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.

Re: Saratoga Templates with PHP 7

Posted: Wed 11 Jan 2017 4:11 pm
by ConligWX
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

Re: Saratoga Templates with PHP 7

Posted: Wed 11 Jan 2017 6:46 pm
by 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.

Re: Saratoga Templates with PHP 7

Posted: Wed 11 Jan 2017 7:39 pm
by 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.

Re: Saratoga Templates with PHP 7

Posted: Wed 11 Jan 2017 8:25 pm
by saratogaWX
Other than the nws-alerts.php issues with PHP 7.1, did you find any other scripts that were having issues??

Re: Saratoga Templates with PHP 7

Posted: Thu 12 Jan 2017 2:05 am
by 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 ;)

Re: Saratoga Templates with PHP 7

Posted: Thu 12 Jan 2017 2:04 pm
by ConligWX
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.

Re: Saratoga Templates with PHP 7

Posted: Sun 22 Jan 2017 4:39 am
by 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.

Re: Saratoga Templates with PHP 7

Posted: Wed 25 Jan 2017 2:17 pm
by ConligWX
Running php 7.1.1 here and so far so good. Also added https to my virtual hosts too!

Re: Saratoga Templates with PHP 7

Posted: Wed 25 Jan 2017 7:24 pm
by 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 :)

Re: Saratoga Templates with PHP 7

Posted: Fri 24 Feb 2017 9:19 am
by ConligWX
Running PHP 7.1.2 now. all seems ok.

Re: Saratoga Templates with PHP 7

Posted: Sat 25 Feb 2017 1:03 am
by 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.

Re: Saratoga Templates with PHP 7

Posted: Sat 25 Feb 2017 8:25 am
by ConligWX
Thanks Ken, just amended mine now.

Re: Saratoga Templates with PHP 7

Posted: Sun 02 Apr 2017 9:32 pm
by ConligWX
just upgraded to 7.1.3. all seems good so far.