Ability to disable calls to wp_add_trashed_suffix_to_post_name_for_trashed_posts()

[johnbillion]

When performing operations where large numbers of posts are saved to the database, one of the bottlenecks is wp_add_trashed_suffix_to_post_name_for_trashed_posts() which is called inside wp_insert_post().

Every time a post is inserted or updated this function is called. It performs a get_posts() call that gets progressively slower the more posts there are. During a large import, it might not be desirable for this trashed post slug handling to occur so there should be a way to disable this for performance reasons (via a filter).

[johnbillion]

Patch that introduces a add_trashed_suffix_to_trashed_posts filter that can be used to disable this functionality.

[davidbaumwald]

@johnbillion Can this be reviewed and possibly merged by tomorrow for version 5.3 Beta 1? The patch is still clean, and seems to be a sensible solution.

[davidbaumwald]

With version 5.3 Beta 1 landing shortly, this is being moved to Future Release. If any committer can review any merge in time for the beta in time, feel free to move this back into 5.3.

[davidbaumwald]

@johnbillion Is this still on your plate for 5.4?

[johnbillion]

In 47203:

Posts, Post Types: Introduce a filter which allows the addition of __trashed suffixes on post slugs to be skipped.

Fixes #46433