Adding optional post parameter to get_the_modified_author function

[samirmalpande]

With the current implementation of the get_the_modified_author() it is not possible to use this function with the custom WP_Query loop. It takes the post ID using the get_post() which returns the global post id and there is no way to pass the ID from the custom query.

We want to add the optional post parameter in get_the_modified_author function so that we can use this function in custom loops as well.

By doing this change we also make this function follow the same standard as other "get_the_" functions like get_the_title, get_the_content, get_the_terms etc.

[samirmalpande]

Patch file

[johnbillion]

Thanks for the ticket and the patch @samirmalpande 👍

[SergeyBiryukov]

Should get_the_author() get the same parameter for consistency?

It used to have an $idmode parameter, which was deprecated in [4146] / #2386. Since it only accepted a string, I think we should be able to un-deprecate and use it if an integer post ID or a WP_Post object is passed. There is a precedent for something like this, see [47366] / #48772.

[samirmalpande]

Ideally get_the_author() should also have same parameter. Since we have a precedent to un-deprecate the parameter, we should pass the $post parameter to get_the_author().

Should we create a new ticket for this or include it in the same patch?

[desrosj]

@samirmalpande Could you refresh the current patch?

@SergeyBiryukov what do you think about protecting against edge cases by throwing a __doing_it_wrong() notice if the value is not an integer post ID or WP_Post object?

Since 6.1 beta 1 is due out in less than 12 hours (the cut off for enhancement and feature request tickets), I'm going to punt this one. But if a refreshed patch is added in time and a committer has bandwidth to review, it can be moved back.

[costdev]

This ticket was reviewed in the bug scrub. As the patch still applies cleanly against trunk, I'm removing the needs-refresh keyword. The changes-requested keyword is accurate. Also adding dev-feedback to draw attention to Jonathan's comment above.

https://core.trac.wordpress.org/ticket/55978

[audrasjb]

I added a refreshed PR but I'm re-adding the changes-requested workflow keyword since I forgot to cover other functions like get_the_author()/the_author().

[mukesh27]

Thanks @samirmalpande for the ticket.

This ticket was discussed during the bug scrub. as this ticket still needs more functions covered. 6.2 Beta 1 is being released today, I'll move this to Future Release.

Additional props: @costdev