[wp-trac] [WordPress Trac] #13483: fileupload_url / upload_url_path

WordPress Trac wp-trac at lists.automattic.com
Fri May 21 22:23:57 UTC 2010


#13483: fileupload_url / upload_url_path
--------------------------+-------------------------------------------------
 Reporter:  jimisaacs     |       Owner:                                                      
     Type:  defect (bug)  |      Status:  new                                                 
 Priority:  high          |   Milestone:  3.0                                                 
Component:  Multisite     |     Version:  3.0                                                 
 Severity:  major         |    Keywords:  multisite attachments fileupload_url upload_url_path
--------------------------+-------------------------------------------------

Comment(by jimisaacs):

 I just want to see if I have your comment straight:

 upload_url_path path was blank before so leave if blank to test.

 "switch_to_blog() doesn't seem to do anything with $current_blog. Either
 is_main_site() should use $blog_id(), or maybe wp_upload_dir() should not
 set $main_override if switched"

 To me I think $current_blog should change, because why is there a function
 call switch_to_blog anyway?

 I see  method in the WP_Importer class. This method is called set_blog(),
 which actually calls switch_to_blog().

 It seems that this method more or less should be implemented into the
 overall switch_to_blog() function.

 Changing the wp_upload_dir() function I believe is a bandaid. Changing
 switch_to_blog() would make is_main_site() do what it is supposed to.

 Thoughts?

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/13483#comment:10>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list