Opened 9 years ago
Last modified 8 years ago
#7195 new enhancement
Throw a notice when activating BP on a site that doesn't meet system requirements
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Awaiting Contributions | Priority: | normal |
Severity: | normal | Version: | |
Component: | Core | 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)
Change History (7)
#4
@
8 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
@
8 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
In 11206: