WP_REST_Posts_Controller::get_items() has too many concerns

[Shelob9]

I've written a bunch of custom endpoints that return collections of posts of one or more post types. I'd like my responses to be the same formatting, body and header as the default post routes provide. This allows for a client designed to consume default routes to be more easily modified to use custom routes.

Achieving this with a subclass of WP_REST_Posts_Controller still requires some cut and paste for collections of posts, since the get_items() method has way too many concerns. I'd like to break out the preparing of response body, as well as response header into separate methods, so that they can be used with different types of requests or ways to create arrays of WP_Post objects, while still providing a consistent response.

[Shelob9]

Initial pass at abstracting response preparation from get_items() method

[rmccue]

Let's refactor this and split it up into multiple methods. :)

[joehoyle]

+1 from me also

[Shelob9]

I'm thinking, as an alternative to my ticket, what about a WP_REST_Post_Query class that took an array of WP_Query ARGS and generated a proper response?