[wp-trac] [WordPress Trac] #23880: Minimum PHP version in Plugins

WordPress Trac noreply at wordpress.org
Fri Mar 29 15:26:49 UTC 2013


#23880: Minimum PHP version in Plugins
-----------------------------+------------------------------
 Reporter:  TJNowell         |       Owner:
     Type:  feature request  |      Status:  new
 Priority:  normal           |   Milestone:  Awaiting Review
Component:  Plugins          |     Version:
 Severity:  normal           |  Resolution:
 Keywords:                   |
-----------------------------+------------------------------

Comment (by F J Kaiser):

 Replying to [comment:20 MikeSchinkel]:
 > Maybe I'm missing something but your plugin can resolve this issue on
 it's own.  Have your main file `version_compare()` for PHP 5.3 before
 `require()`ing any files that contain namespaces and if PHP 5.2.x then
 tell the user and [http://grinninggecko.com/activating-deactivating-
 wordpress-plugins-programmatically/ deactivate the plugin]. Does this not
 work for some reason I have not considered?

 Sure this ''does work''. But you have to do it either on runtime (which
 most people already do) or during the {{{register_activation_hook()}}}
 callback, which is the opposite of ideal. "Better" developers will use the
 hook and cook their own soup while not that good devs have all different
 sorts of failing components or performance leaks. As core cares about the
 inclusion of plugins and the whole surroundings, it ''IMHO'' should as
 well care about those parts. It simply is '''core territory''' and not
 ''plugin material''. WP core maintains an array of active plugins as well
 as outputting {{{admin_notices}}}. And this checks would be needed exactly
 there.

 And yes, tickets are in some cases also here to discuss something. In this
 case (1) should we implement this and in case (2) ''how'' should we
 implement it.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/23880#comment:22>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list