[wp-trac] [WordPress Trac] #35443: Allow selective plugins to load for wp-activate.php

[WordPress Trac]

#35443: Allow selective plugins to load for wp-activate.php
------------------------------------+------------------------------
 Reporter:  pbearne                 |       Owner:
     Type:  enhancement             |      Status:  new
 Priority:  normal                  |   Milestone:  Awaiting Review
Component:  Login and Registration  |     Version:  trunk
 Severity:  normal                  |  Resolution:
 Keywords:                          |     Focuses:
------------------------------------+------------------------------

Comment (by pbearne):

 Replying to [comment:1 ericlewis]:
 > Hi @pbearne, thanks for opening this ticket.
 >
 > Active plugins are
 [https://github.com/WordPress/WordPress/blob/a51129c2a6565750e5cb14512d1aff768f174e92
 /wp-includes/load.php#L555 retrieved via a site option]. Thereby, the list
 of active plugins could be modified with the
 >
 [https://github.com/WordPress/WordPress/blob/a51129c2a6565750e5cb14512d1aff768f174e92
 /wp-includes/option.php#L123-L136 option_active_plugins] filter.
 >
 > We should get in contact with the s2member authors and make them aware
 of these tickets and get them involved :) Happy to consider changes where
 core can do better by plugins, but some of these changes should happen
 downstream (in the plugin).

 Already talking to s2members team (or trying to)

 the problem is that on
 [https://github.com/WordPress/WordPress/blob/a51129c2a6565750e5cb14512d1aff768f174e92
 /wp-includes/load.php#L563 line 593] an empty array to returned

 the patch that s2members is install is this

 {{{#!php
 <?php
 // Modified for full plugin compatiblity.
     //if ( empty( $active_plugins ) || wp_installing() )
     if ( empty( $active_plugins ) || ( wp_installing() && !preg_match
 ("/\/wp-activate\.php/", $_SERVER["REQUEST_URI"]) ) )
     return $plugins;
 }}}

 I added the filter so that by default we didn't load any plugins but
 rather had to select them as felt that is was better to load only what is
 need while installing / activating a site

 Thoughts?

 Paul

--
Ticket URL: <https://core.trac.wordpress.org/ticket/35443#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform