[wp-trac] [WordPress Trac] #63104: Undefined constant "BLOGUPLOADDIR" for subdirectory multisites

[WordPress Trac]

#63104: Undefined constant "BLOGUPLOADDIR" for subdirectory multisites
-------------------------------------+------------------------
 Reporter:  danielpw                 |       Owner:  realloc
     Type:  defect (bug)             |      Status:  closed
 Priority:  normal                   |   Milestone:  6.9
Component:  Networks and Sites       |     Version:  3.0
 Severity:  normal                   |  Resolution:  fixed
 Keywords:  has-test-info has-patch  |     Focuses:  multisite
-------------------------------------+------------------------
Changes (by johnjamesjacoby):

 * status:  assigned => closed
 * resolution:   => fixed


Comment:

 In [changeset:"60956" 60956]:
 {{{
 #!CommitTicketReference repository="" revision="60956"
 Networks and Sites: check if the `BLOGUPLOADDIR` constant is defined
 before using it.

 This change prevents a fatal PHP error from occurring when `ms-files.php`
 is accessed without legacy multisite file-handling support being
 configured.

 Instead of the fatal error, these same requests will new return a header
 status `500` along with a `die()` message "500 - Directory not
 configured." This differs from the existing `404` statuses to help
 indicate to site owners this fallback is not working as intended.

 Props danielpw, dd32, jakeparis, johnjamesjacoby, spacedmonkey,
 saurabh.dhariwal, sukhendu2002, realloc.

 Fixes #63104.
 }}}

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