It’s probably easiest to just check the post status within your function. Untested: add_action( ‘save_post’, ‘rewrite_post’, 10, 2 ); function rewrite_post( $post_id ) { if ( ‘trash’ != get_post_status( $post_id ) ) { remove_action( ‘save_post’, ‘rewrite_post’ ); $title = preg_replace( ‘/\_/’, ‘ ‘, get_the_title( $post_id ) ); $my_post = array( ); $my_post[‘ID’] = $post_id; $my_post[‘post_title’] … Read more
Following are the issues that were preventing from the meta values to be saved in both of your save functions. 1. Incorrect post_type checking. You are checking post type to be of ‘page’ in your save function where as your meta boxes are being displayed on your custom post type of ‘film’. Your code: if … Read more
So appreciate this is a bit late but I was having the exact same issue, the $update parameter is almost completely useless if you want to check whether it is a new post or not. The way I got around this was to compare the $post->post_date with $post->post_modified. Full code snippet below. add_action( ‘save_post’, ‘save_post_callback’, … Read more
First, you can use this hook to target only one custom type: https://developer.wordpress.org/reference/hooks/save_post_post-post_type/ This hook (and save_post) is called the first time when you click on “new …” and then the hook is called with $update = FALSE. Then to send e-mail only when the object is updated, you can test $update like this: const … Read more
It’s wp_insert_post() vs. wp_update_post() – where update will ultimately also call: return wp_insert_post( $postarr, $wp_error, $fire_after_hooks ); The term “once” implies that it is being fired “afterwards”. /** * Fires once a post has been saved. * * @since 1.5.0 * * @param int $post_ID Post ID. * @param WP_Post $post Post object. * @param … Read more
It’s informative to check out the /wp-admin/post.php file, that contains the edit_post() function that calls wp_update_post(), which is a wp_insert_post() wrapper. Here’s a skeleton for saving the assigned category terms: /** * Saving assigned category terms (skeleton) */ add_action( ‘admin_action_editpost’, function() { add_filter( ‘wp_insert_post_data’, function( $data, $parr ) { add_action( ‘save_post_post’, function( $post_ID, $post ) … Read more
The problem is that the function update_field from ACF works a little bit different from update_post_meta. Can you see it? update_post_meta( $post_id, $meta_key, $meta_value, $prev_value ); update_field( $field_key, $value, $post_id ); If you use update_post_meta you should in my opinion use get_post_meta to get the meta. If you use update_field you should use the_field or … Read more
For NEW post type ‘post’ use draft_to_publish action hook: function fpw_post_info( $post ) { if ( ‘post’ == $post->post_type ) { // echo ‘<pre>’; print_r( $post ); echo ‘<br />’; // $meta = get_post_meta( $post->ID ); print_r( $meta ); echo ‘</pre>’; die(); // your custom code goes here… } } add_action( ‘draft_to_publish’, ‘fpw_post_info’, 10, 1 … Read more
The ‘save_post’ action was added to core in 2.0, and has always been an action. Looking through the current autosave procedures, it doesn’t appear to call the ‘save_post’ action directly at any time. So the short answer is, no. There is no reason, and has never been any reason, to return any value on this … Read more
http://codex.wordpress.org/Function_Reference/register_post_type example: //add filter to ensure the text Book, or book, is displayed when user updates a book add_filter(‘post_updated_messages’, ‘codex_book_updated_messages’); function codex_book_updated_messages( $messages ) { global $post, $post_ID; $messages[‘book’] = array( 0 => ”, // Unused. Messages start at index 1. 1 => sprintf( __(‘Book updated. <a href=”https://wordpress.stackexchange.com/questions/17885/%s”>View book</a>’), esc_url( get_permalink($post_ID) ) ), 2 => … Read more