Skip to:
Content

BuddyPress.org

Opened 10 years ago

Last modified 9 years ago

#7195 new enhancement

Throw a notice when activating BP on a site that doesn't meet system requirements

Reported by: boonebgorges Owned by:
Priority: normal Milestone: Awaiting Contributions
Component: Core Version:
Severity: normal Keywords:
Cc:

Description

When BP requires a version of PHP higher than what WP requires, you could have people activating BP on a WP site that can't run BP. We should do the following:

  • make sure our loader file is compatible with the minimum version of PHP supported by WordPress
  • when BP is activated on a WP install that doesn't meet PHP requirements, throw an admin notice and don't load the rest of BP's files

Attachments (1)

7195.01.patch (664 bytes ) - added by DJPaul 10 years ago.

Download all attachments as: .zip

Change History (7)

#1 @DJPaul
10 years ago

  • Milestone Future Release2.8

#2 @djpaul
10 years ago

In 11206:

Update Composer PHP requirement to 5.3+, and shrinkwrap latest NPM packages.

See #7195

#3 @djpaul
10 years ago

In 11207:

Unit Tests: adieu, PHP 5.2.

BP 2.8 drops support for PHP 5.2. See #7195

@DJPaul
10 years ago

#4 @DJPaul
10 years ago

7195.01.patch is really just a proof of concept of showing an admin notice if you're running PHP < 5.3 to start discussion.

I personally think we should intentionally use die to quit processing, so that WordPress falls back and never activates the plugin. See screenshot for a reminder of that behaviour: https://cloudup.com/cL4q1HTjVuo

#5 @DJPaul
10 years ago

I came across a Twitter discussion about this sort of thing, and found another plugin's approach to this: https://plugins.trac.wordpress.org/browser/cart-recovery/trunk/cart-recovery-for-wordpress.php#L40

#6 @DJPaul
9 years ago

  • Milestone 2.8Future Release
Note: See TracTickets for help on using tickets.