Make WordPress Core

Opened 2 years ago

Closed 3 weeks ago

#57900 closed enhancement (maybelater)

Only prime post thumbnails in post REST API controller

Reported by: spacedmonkey's profile spacedmonkey Owned by: spacedmonkey's profile spacedmonkey
Milestone: Priority: normal
Severity: normal Version: 6.1
Component: REST API Keywords: has-patch
Focuses: rest-api, performance Cc:

Description

Follow on from [53499].

Post thumbnails are being primed in the post REST API endpoint, even if the thumnbail is not present the response. Only load prime post thumbnails if present in response.

Change History (5)

This ticket was mentioned in PR #4210 on WordPress/wordpress-develop by @spacedmonkey.


2 years ago
#1

  • Keywords has-patch added

#2 @spacedmonkey
2 years ago

  • Milestone changed from Future Release to 6.3
  • Owner set to spacedmonkey
  • Status changed from new to assigned

#3 @spacedmonkey
23 months ago

  • Milestone changed from 6.3 to Future Release

#4 @flixos90
3 weeks ago

@spacedmonkey There hasn't been any update on this in almost 2 years, but it seems it could be a worthwhile effort.

Were there any concerns back then that made you punt this ticket? Or is it simply a matter of taking the time to implement, review etc?

I'm currently going over inactive tickets, and this one feels like it could be reasonable to move forward.

#5 @spacedmonkey
3 weeks ago

  • Milestone Future Release deleted
  • Resolution set to maybelater
  • Status changed from assigned to closed

I think lazily loading post meta is a better course. I think I am going to change this to maybe later.

Note: See TracTickets for help on using tickets.