[wp-trac] [WordPress Trac] #39522: Avoidable PHP Warning thrown in _wp_handle_upload

[WordPress Trac]

#39522: Avoidable PHP Warning thrown in _wp_handle_upload
-------------------------------------+------------------------------
 Reporter:  dglingren                |       Owner:
     Type:  defect (bug)             |      Status:  new
 Priority:  normal                   |   Milestone:  Awaiting Review
Component:  Media                    |     Version:  trunk
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-testing  |     Focuses:
-------------------------------------+------------------------------
Changes (by achbed):

 * keywords:   => has-patch needs-testing


Comment:

 The attached patch reverses the order of the tests in order to catch
 issues with missing/unreadable files before checking the file length. I've
 also added a "is_readable" check to the existence check to catch
 permission issues.  A second set of eyes (and someone who can run the unit
 tests) would be greatly appreciated.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/39522#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform