[wp-trac] [WordPress Trac] #15432: media_handle_sideload() is not returning right value

WordPress Trac wp-trac at lists.automattic.com
Mon Nov 15 12:32:42 UTC 2010


#15432: media_handle_sideload() is not returning right value
--------------------------+-------------------------------------------------
 Reporter:  acsnaterse    |       Owner:  joostdevalk    
     Type:  defect (bug)  |      Status:  new            
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Media         |     Version:  3.0.1          
 Severity:  normal        |    Keywords:  has-patch      
--------------------------+-------------------------------------------------
Changes (by joostdevalk):

  * keywords:  => has-patch


Comment:

 Attached patch fixes return and makes sure the single function in core
 that uses media_handle_sideload work properly again. Seriously, if you
 write this:

 {{{
  // do the validation and storage stuff
 $id = media_handle_sideload($file_array, $post_id, @$desc);
 $src = $id;
 }}}

 You have to know you're doing something wrong, right? :)

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


More information about the wp-trac mailing list