[wp-trac] [WordPress Trac] #17814: wp_load_image() attempts to load directories

WordPress Trac wp-trac at lists.automattic.com
Tue Oct 2 15:17:10 UTC 2012


#17814: wp_load_image() attempts to load directories
--------------------------+--------------------
 Reporter:  primehalo     |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  3.5
Component:  Media         |     Version:  3.1.3
 Severity:  minor         |  Resolution:
 Keywords:  has-patch     |
--------------------------+--------------------

Comment (by DH-Shredder):

 Replying to [comment:6 scribu]:
 > Couldn't we just replace the file_exists() check with is_file() instead
 of calling both?

 You're right, looks like we can!
 Went ahead and [attachment:17814.3.diff updated the patch].  Will write up
 a quick unit test for this later today, if no-one else gets to it first.

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


More information about the wp-trac mailing list