Topic: Has anyone tested PHP 7.2?

[spiros]

Has anyone tested PHP 7.2 on SMF?

[Arantor]

No. But I can tell you without any hesitation that it will start vomiting masses of deprecated errors everywhere.

[Sh@mbles]

... I can tell you without any hesitation that it will start vomiting masses of deprecated errors everywhere.

You're not kidding.

Function create_function() is deprecated

by the trillion.

[Arantor]

Yup, every single bbcode definition will cause errors. Probably multiple times per invocation of bbcode parser.

This was fixed in 2.1 but is borderline impossible to properly fix on 2.0 and every 2.0 bbcode.

[Chen Zhen]

You can suppress the deprecated warning for PHP 7.2 specifically.

add to function: error_handler()

Code: [Select]

// Disable PHP 7.2 "Function create_function() is deprecated"  errors from filling the forum error logs
if (defined('E_DEPRECATED') && $error_level == E_DEPRECATED && phpversion() == '7.2.0' && strpos($error_string, 'Function create_function() is deprecated') !== false)
return;

.. this will work fine for PHP 7.2 but the deprecated warning means in an upcoming version of PHP it will be removed altogether where this type of patch will not work.

[vbgamer45]

I like that trick! I did that for SMF 1.1.x and still runs on php 7.1

[Arantor]

You can suppress the deprecated warning for PHP 7.2 specifically.

add to function: error_handler()

Code: [Select]

// Disable PHP 7.2 "Function create_function() is deprecated"  errors from filling the forum error logs
if (defined('E_DEPRECATED') && $error_level == E_DEPRECATED && phpversion() == '7.2.0' && strpos($error_string, 'Function create_function() is deprecated') !== false)
return;

.. this will work fine for PHP 7.2 but the deprecated warning means in an upcoming version of PHP it will be removed altogether where this type of patch will not work.

You really shouldn't do a version comparison like that because when 7.2.1 comes out, it's not going to work... you could turn it into a version_compare() call instead and check for 7.2 or above all at once.

As for SMF 1.1 still working on PHP 7, writing a shim for the mysql_ functions, followed by replacing out all of the preg_replace with /e calls (which don't work in 7), along with all the other changes you're going to need to do, you might as well put the time in upgrading to 2.0 anyway because you're only going to spend more and more time patching up such issues.

[vbgamer45]

I agree the comparison should be changed.. I am still waiting out for a few releases for 7.2 to come before i actually look to move to it for my non smf projects. For more SMF projects I most use SMF 2.0 except for legacy forum hosting where I still have many forums. waiting for 2.1 to make that big jump

[spiros]

Just tested with php 7.2.5, I see no errors on actual pages
https://www.translatum.gr/forum/index.php

on error log plenty of depreciated ones:

Code: [Select]

8192: Function create_function() is deprecated
/public_html/forum/Sources/Subs.php
Line: 1509

[Kindred]

Well, SMF 2.0.15 does not officially support 7.2