[buddypress-trac] [BuddyPress] #942: WP_CONTENT_URL returns blog url instead of root blog url

buddypress-trac at lists.automattic.com buddypress-trac at lists.automattic.com
Tue Aug 25 06:59:56 UTC 2009


#942: WP_CONTENT_URL returns blog url instead of root blog url
-----------------------+----------------------------------------------------
Reporter:  mariusooms  |       Owner:     
    Type:  defect      |      Status:  new
Priority:  major       |   Milestone:  1.1
Keywords:              |  
-----------------------+----------------------------------------------------
 in bp-core-avatars.php the function bp_core_fetch_avatar sets
 WP_CONTENT_URL as part of the $avatar_folder_url variable around line 89.

 The issue is that WP_CONTENT_URL will return a url specific to the
 current_blog, rather than the root blog. WP_CONTENT_DIR does remain
 specific to the root blog.

 When you try to fetch avatars outside of the root blog, the returned url
 will be wrong and actually relative to the current blog.

 The current (ugly) fix 'for me' is the following:

 {{{
 $avatar_folder_url = apply_filters( 'bp_core_avatar_folder_url', 'http://'
 . $current_site->domain . $current_site->path . 'wp-content/blogs.dir/' .
 BP_ROOT_BLOG . '/files/' . $avatar_dir . '/' . $item_id, $item_id,
 $object, $avatar_dir );
 }}}

 The original line is as follows:

 {{{
 $avatar_folder_url = apply_filters( 'bp_core_avatar_folder_url',
 WP_CONTENT_URL . '/blogs.dir/' . BP_ROOT_BLOG . '/files/' . $avatar_dir .
 '/' . $item_id, $item_id, $object, $avatar_dir );
 }}}

 I imagine there is a better way, maybe a new function called BP_ROOT_URL
 or an existing function I am unaware of.

 On a side note, it almost seems to me this is a WPMU bug since I would
 expect that WP_CONTENT_URL would behave like WP_CONTENT_DIR.

-- 
Ticket URL: <http://trac.buddypress.org/ticket/942>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list