I am trying to write a plugin that will stop a user from publishing (or updating) a post if a given condition is not met, for example, if the title of the post is already being used in a different post (I am using a custom post type, if that makes a difference).
I wrote a function that gets called through the ‘transition_post_status’ hook to attempt to catch the post before it gets published or updated in the database, so that I can stop that from happening.
When I test this out with a title I know I’ve already used I see the error message when the wp_die() function happens, so I know this function is getting called. However, the post is being created anyways. Is there something I’m missing? Is this even the best way of checking for conditions before letting a post be saved in the database?
This is what I have in the constructor:
add_action( 'transition_post_status', array( $this, 'post_publish_control' ), 10, 3 );
Later in the class:
public function post_publish_control( $new_status, $old_status, $post ) {
// Make sure this function runs on correct post type
if( $post->post_type === 'my_custom_post_type' ) {
// On first-time publish or post update
if( $new_status === 'publish' ) {
// Try to catch an already existing title before post is saved
if( does_post_title_exist( $post->post_title ) ) {
wp_die( 'Title is already being used, try a different one.' );
}
// Continue if conditions are met
else {
return;
}
}
}
}
I am a bit more used to object-oriented programming, so I’m trying to write this plugin as its own class.