WordPress.org

Make WordPress Core

Opened 5 weeks ago

Last modified 3 weeks ago

#52840 accepted enhancement

Include filesize information in REST API response for all media types

Reported by: tfrommen Owned by: rachelbaker
Milestone: 5.8 Priority: normal
Severity: normal Version:
Component: REST API Keywords: has-patch needs-unit-tests
Focuses: rest-api Cc:

Description

Requesting the /media REST API endpoint, only audio attachments include filesize information in the response data (in media_details->filesize).

I suggest to check for each item if filesize is present, or else fill it in using the actual file. This would be pretty much in line with what is done in wp_prepare_attachment_for_js.

Making the information available locally, when creating the response data, is much more performant than having to request the file in one or more follow-up requests.

Attachments (2)

52840.diff (827 bytes) - added by tfrommen 5 weeks ago.
52840.1.diff (819 bytes) - added by tfrommen 5 weeks ago.
Formatting

Download all attachments as: .zip

Change History (3)

@tfrommen
5 weeks ago

@tfrommen
5 weeks ago

Formatting

#1 @rachelbaker
3 weeks ago

  • Keywords needs-unit-tests added
  • Milestone changed from Awaiting Review to 5.8
  • Owner set to rachelbaker
  • Status changed from new to accepted
Note: See TracTickets for help on using tickets.