PHP7 Compatibility issue
-
Hi 🙂
Your plugin have a compatibility issue with last version of php.
You use dynamic variables names like this :
$$all_options['first_order']but it is ambigous like explained here : http://php.net/manual/en/language.variables.variable.php
In order to use variable variables with arrays, you have to resolve an ambiguity problem. That is, if you write $$a[1] then the parser needs to know if you meant to use $a[1] as a variable, or if you wanted $$a as the variable and then the [1] index from that variable. The syntax for resolving this ambiguity is: ${$a[1]} for the first case and ${$a}[1] for the second.
Consequence : sitemap is not displayed and a lot of notices are displayed on frontend and backend settings page.
The solution is to replace the code by :
${$all_options['first_order']}
-
Based on the feedback of @ajoah, this update to line 1774 technically makes the plugin work in PHP7 :
return ${$all_options['first_order']} . ${$all_options['second_order']} . ${$all_options['third_order']} . ${$all_options['fourth_order']} . ${$all_options['fifth_order']} . ${$all_options['first_order']} . $all_options['seventh_order'] . $promote;One non-fatal error still exists :
Methods with the same name as their class will not be constructors in a future version of PHP; WPRealtimeSitemap has a deprecated constructor in wp-realtime-sitemap/wp-realtime-sitemap.php on line 32
- The topic ‘PHP7 Compatibility issue’ is closed to new replies.