WPIntell

Source evidence

Old PHP versions

BP MPO Activity Filter · support · 2018-04-25T18:39:00+00:00

complaintsentiment
highseverity
1.0relevance
2replies
Evidence linked to opportunitycommercial context

Proof Health

Open evidence

Commercial opportunities need traceable source links before they are treated as build-worthy.

3 / 31 rows with source links

9.7% of this page's analysis has direct source links.

0 build-decision rows missing links

0 rows here require auditable proof before promotion.

28 rows with no attached evidence

0 rows have source counts but still need direct links.

Conversation

support
ucsdmatthew unresolved
The latest version includes the new [] array syntax, which breaks compatibility of old versions of PHP such as php5.3, which though no longer supported by its original developer is still supported and maintained with security patches by long-term-support releases of RedHat. Would it be possible to change that array declaration from [] to Array()? I made that change myself to quickly get running, though I know it will get wiped out with the next release. If not, I would appreciate if you would officially document your minimum PHP version (per https://make.wordpress.org/plugins/2017/08/29/minimum-php-version-requirement/ ), which would provide documentation for the need to allocate the time to upgrade to a current release of PHP. Observed error: PHP Parse error: syntax error, unexpected ‘[‘ in /unixapps/knitprod/wordpress/wp-content/plugins/bp-mpo-activity-filter/bp-mpo-activity-filter-bp-functions.php on line 24 Thank you! -Matthew Hi Matthew – Thanks for the thorough request, and apologies for the unexpected breakage. The thought of the array syntax hadn’t occurred to me at the time of release. In this case, PHP 5.2 support is trivial to implement, so I’ve gone ahead and done it. Version 1.2.2 of the plugin should be available in the plugin repo momentarily. However, I’ve also added a Requires PHP: 5.3 header to this plugin, as a warning that PHP 5.2 support will not be guaranteed going forward. This is in keeping with the policy of BuddyPress itself since version 2.8. https://codex.buddypress.org/getting-started/buddypress-2-8-will-require-php-5-3/ Again, thanks for your patience. Awesome, thanks for making that change! I probably have a half dozen commits on the plugin I wrote for our site with exactly that same change, after deploying from my workstation to the stageing environment 🙂 It does sound like our not-too-distant plan is to move this to hosting that supports a modern PHP, so hopefully we won’t be crying when 5.3 does get dropped.

Comments

2 shown
Boone Gorges 2018-04-25T19:20:00+00:00

Hi Matthew – Thanks for the thorough request, and apologies for the unexpected breakage. The thought of the array syntax hadn’t occurred to me at the time of release. In this case, PHP 5.2 support is trivial to implement, so I’ve gone ahead and done it. Version 1.2.2 of the plugin should be available in the plugin repo momentarily. However, I’ve also added a Requires PHP: 5.3 header to this plugin, as a warning that PHP 5.2 support will not be guaranteed going forward. This is in keeping with the policy of BuddyPress itself since version 2.8. https://codex.buddypress.org/getting-started/buddypress-2-8-will-require-php-5-3/ Again, thanks for your patience.

ucsdmatthew 2018-04-25T21:17:00+00:00

Awesome, thanks for making that change! I probably have a half dozen commits on the plugin I wrote for our site with exactly that same change, after deploying from my workstation to the stageing environment 🙂 It does sound like our not-too-distant plan is to move this to hosting that supports a modern PHP, so hopefully we won’t be crying when 5.3 does get dropped.