Make WordPress Core

Opened 4 years ago

Last modified 11 months ago

#52732 new defect (bug)

'get_posts' has been waiting for so long to have a hook

Reported by: ttodua's profile ttodua Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Query Keywords: reporter-feedback
Focuses: Cc:

Description

I reckon get_posts is the most (frequently) used function across many plugins or scripts. I know there are internal WP_Query hooks available, but that makes things harder to get into.
I think having hook on get_posts will be very useful and easy for people / developers to remind & amend things easier.
patch attached.

Attachments (3)

52732-1.patch (424 bytes) - added by ttodua 4 years ago.
52732.patch (402 bytes) - added by ttodua 11 months ago.
52732-2.patch (402 bytes) - added by ttodua 11 months ago.

Download all attachments as: .zip

Change History (5)

@ttodua
4 years ago

#1 follow-up: @SergeyBiryukov
4 years ago

  • Component changed from General to Query
  • Keywords reporter-feedback added

Hi there, thanks for the patch!

Could you provide a use case that would be easier to achieve with this new filter than with the existing WP_Query filters, e.g. pre_get_posts, posts_results, the_posts, and others?

@ttodua
11 months ago

@ttodua
11 months ago

#2 in reply to: ↑ 1 @ttodua
11 months ago

Replying to SergeyBiryukov:

Hi there, thanks for the patch!

Could you provide a use case that would be easier to achieve with this new filter than with the existing WP_Query filters, e.g. pre_get_posts, posts_results, the_posts, and others?

sorry for late reply.
yes, the use case is not about building query ourselves, but the intention of this PR is to be able to hook into themes/plugins which we don't control, so many of them directly use get_posts method in their implementations and we hadn't have any other way to hook into them.

Note: See TracTickets for help on using tickets.