[buddypress-trac] [BuddyPress] #2945: Don't hard-code the 'buddypress' plugin folder

buddypress-trac at lists.automattic.com buddypress-trac at lists.automattic.com
Mon Aug 20 21:17:23 UTC 2012


#2945: Don't hard-code the 'buddypress' plugin folder
--------------------------+-----------------------------
 Reporter:  cnorris23     |       Owner:
     Type:  defect (bug)  |      Status:  reopened
 Priority:  normal        |   Milestone:  Future Release
Component:  Core          |     Version:  1.6
 Severity:  normal        |  Resolution:
 Keywords:  needs-patch   |
--------------------------+-----------------------------

Comment (by foxly):

 Just a heads-up that allowing users to install Buddypress to "/my-awesome-
 folder-name" instead of "/buddypress" will make unit-testing *much* more
 complicated and potentially cause all sorts of unpredictable problems
 throughout the Wordpress installation.

 If you make this change, the unit test system will have to remap the
 Buddypress slug after loading the DB image, because IIRC Wordpress tracks
 activated plugins by their folder name in the plugins folder. We were
 hoping to avoid this in BP's unit test runner because it means manually
 creating a DB image on every upgrade, then figuring out the SQL commands
 needed to map it to the arbitrary folder name. It could also *double* test
 run time because, realistically, you're going to have to run the entire
 test panel twice ...once for the standard "/buddypress" folder path and
 once  for the "/my-awesome-folder" path.

 Allowing users to rename the Buddypress install folder introduces MASSIVE
 technical problems while offering ZERO benefit in any practical
 application.

 Some users on this ticket seem to be misinterpreting the WP codex article
 to mean "users should be able to rename an individual plugin's install
 folder".

 This is NOT what the codex says.

 The codex says "users should be able to rename the ROOT plugins folder and
 install their WP installation to an arbitrary location on the server".

 They're completely different things.

 Wordpress actually uses a plugin's install folder as a GUID and, amongst
 other things, checks for updates based on it. You can't just go
 arbitrarily renaming your plugin folder or you'll get collisions  ...for
 example, there is already another plugin that uses "/social" as its folder
 name: http://wordpress.org/extend/plugins/social/ ...which is the name the
 user requesting the change has set his BP installation folder to.

 I suspect the next time the author of the "social" plugin pushes an
 update, an update notice will appear on the user's dashboard; and if they
 click on it, Wordpress will replace Buddypress with the real "Social"
 plugin.

 In the interest of not causing EPIC debugging problems, I strongly
 recommend not implementing this change.

 -F

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/2945#comment:25>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list