[wpmu-trac] [WordPress MU Trac] #1154: switch_to_blog and wp_upload_dir, with wpmu 2.8.4a

[WordPress MU Trac]

#1154: switch_to_blog and wp_upload_dir, with wpmu 2.8.4a
------------------------+---------------------------------------------------
 Reporter:  yannux      |       Owner:  somebody
     Type:  defect      |      Status:  new     
 Priority:  high        |   Milestone:  2.8     
Component:  component1  |     Version:  2.8     
 Severity:  major       |    Keywords:          
------------------------+---------------------------------------------------

Comment(by brianlayman):

 The advantage of wp_upload_dir is that you can pass in a post's date and
 get the path used for that particular article.  Admittedly you could build
 that up manually, but it sure is nice when the function that was meant to
 work does.  It makes taking an URL to an uploaded image back to the local
 file path, very clean.

 So, I've been fighting this all night. It seems to me that it boils down
 to the fact that we are trying to make something a constant, that isn't
 constant for most people.

 I guess there could be a user out there that wants all files for all of
 their blogs uploaded to a single directory, but I can't believe that's the
 majority of us.

 So in my mind things are broken for us blog switchers all the way back in
 wp-settings where UPLOADS and BLOGUPLOADDIR are defined.  With those
 defined as constants, there's really nothing that can be done.

 This late at night, the only way I can see to accommodate both camps, is
 to define a pattern string so that the blogid, or even domain, could be
 correctly substituted in as desired, after a switch is done.

 In the mean time, I am brute forcing it in my plugin:
 {{{
 remove_filter( 'upload_dir', 'mu_upload_dir' );
 function nu_upload_dir( $uploads ) {
         global $blog_id;
         $uploads['path'] = WP_CONTENT_DIR .
 "/blogs.dir/{$blog_id}/files/";
         return $uploads;
 }
 add_filter( 'upload_dir', 'nu_upload_dir' );
 }}}


 Tonight's motto, GitRDone.

-- 
Ticket URL: <http://trac.mu.wordpress.org/ticket/1154#comment:4>
WordPress MU Trac <http://mu.wordpress.org/>
WordPress Multiuser