Issue upgrading PHP to 8.1
Permalink Browser Info Environment
Hi There,
When attempting to upgrade our concrete v 9.1.3 site from php 7.4 to 8.1 we get this error:
Exception Occurred: C:\xampp\htdocs\packages\dex_superfish\blocks\dex_superfish\nav_item.php:70 Undefined property: Concrete\Package\DexSuperfish\Block\DexSuperfish\NavItem::$cPointerExternalLink (2)
We have had no problems with Dex Superfish v2.1.0 under various Concrete v8 and v9 (to 9.1.3) with php 7.3 and 7.4.
Do you have a suggested fix for running under php 8.1 ?
Thanks & regards,
Jeff.
When attempting to upgrade our concrete v 9.1.3 site from php 7.4 to 8.1 we get this error:
Exception Occurred: C:\xampp\htdocs\packages\dex_superfish\blocks\dex_superfish\nav_item.php:70 Undefined property: Concrete\Package\DexSuperfish\Block\DexSuperfish\NavItem::$cPointerExternalLink (2)
We have had no problems with Dex Superfish v2.1.0 under various Concrete v8 and v9 (to 9.1.3) with php 7.3 and 7.4.
Do you have a suggested fix for running under php 8.1 ?
Thanks & regards,
Jeff.
| Type: | Pre-Sale |
|---|---|
| Status: | In Progress |
Thanks Dex for your reply,
I managed to get the superfish menu to work under PHP 8.1 by making a couple of simple changes to nav_item.php:
Both occurrences of if ( $this->cPointerExternalLink != '' ) {
(lines 70 & 88)
replaced with:
if ( !empty($this->cPointerExternalLink) ) {
I have made these changes in a test environment - so far no other problems found with the menu add on.
Regards,
Jeff.
I managed to get the superfish menu to work under PHP 8.1 by making a couple of simple changes to nav_item.php:
Both occurrences of if ( $this->cPointerExternalLink != '' ) {
(lines 70 & 88)
replaced with:
if ( !empty($this->cPointerExternalLink) ) {
I have made these changes in a test environment - so far no other problems found with the menu add on.
Regards,
Jeff.
I will have to have a look into things to see what is going on. Unfortunately I will be busy for a couple of days so things will have to wait till then but I will get back to you as soon as possible.
Cheers,
Dex