[buddypress-trac] [BuddyPress] #2945: Don't hard-code the 'buddypress' plugin folder
buddypress-trac at lists.automattic.com
buddypress-trac at lists.automattic.com
Tue Aug 21 07:14:27 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 cnorris23):
Replying to [comment:25 foxly]:
My original reply to this was incredibly long and snarky, so in the
interest of the community, I've chosen to only respond directly to the
statement directed towards me.
> 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.
You're right. They are completely different things. However, if you'd like
to call me out, please do so directly, rather than passive aggressively
saying "some users," and also make sure that you have thoroughly
read/interpreted what was said. To quote myself "According to Determining
Plugin and Content Directories, the way BP currently handles this is
minimal, but sufficient." As said in the statement, BuddyPress was doing
things minimally correct, as it wasn't hard coding the root plugin
directory, or the wp-content directory locations. However, I said
minimally correct, because to fully comply with, and use, the WP API,
plugins should be using plugins_url(), plugin_dir_url(), and
plugin_dir_path() (or at least mimicking their functionality), which
allows these paths and URLs to be dynamically created using only the
{{{__FILE__}}} constant as a reference. The changes proposed by this
ticket, and the changes already committed, do nothing different than what
is already being done by the majority of actively maintained plugins in
the WP plugin directory (reasonable guess), and what is purported by the
WP API itself. This includes such plugins as Akismet, bbPress, and BP
Media. How did Akismet and BP Media, who both have unit tests available,
account for the possibility of users changing the plugin folder name?
Simple. They didn't, nor should they.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/2945#comment:26>
BuddyPress <http://buddypress.org/>
BuddyPress
More information about the buddypress-trac
mailing list