WordPress.org

Make WordPress Core

Opened 11 months ago

Closed 11 months ago

Last modified 8 months ago

#46729 closed defect (bug) (fixed)

Site Health: use `wp_get_upload_dir` instead of `wp_upload_dir`

Reported by: m_uysl Owned by: SergeyBiryukov
Milestone: 5.2 Priority: normal
Severity: normal Version: 5.2
Component: Site Health Keywords: has-patch site-health
Focuses: Cc:

Description

I think using the lightweight edition of the wp_upload_dir can be more accurate in WP_Debug_Data::debug_data()

Attachments (1)

46729.patch (971 bytes) - added by m_uysl 11 months ago.
this patch also removes an unnecessary call to wp_upload_dir

Download all attachments as: .zip

Change History (6)

@m_uysl
11 months ago

this patch also removes an unnecessary call to wp_upload_dir

#1 @m_uysl
11 months ago

  • Keywords has-patch added

#2 @xkon
11 months ago

  • Keywords site-health added

#3 @SergeyBiryukov
11 months ago

  • Milestone changed from Awaiting Review to 5.2

#4 @SergeyBiryukov
11 months ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 45108:

Site Health: Use wp_get_upload_dir() instead of wp_upload_dir(), remove duplicate call.

Props m_uysl.
Fixes #46729.

#5 @spacedmonkey
8 months ago

  • Component changed from Administration to Site Health
Note: See TracTickets for help on using tickets.