Update slug

gistfile1.txt

// change slug on post if title is changed.
add_action( 'save_post', 'icput_post_title_fix' );
function icput_post_title_fix( $post_id ) {

    $url = rwmb_get_value( 'canonical_url', '', $post_id );

    if ( empty( $url ) ) {
        return;
    }

    if ( get_post_type( $post_id ) === 'post' ) {
        $post        = get_post( $post_id );
        $title       = $post->post_title;
        $clean_title = sanitize_title( $title );
        $slug        = $post->post_name;
        if ( $slug !== $clean_title ) {
            $clean_post = array(
                'ID'        => $post_id,
                'post_name' => $clean_title,
            );

            // Update the post.
            wp_update_post( $clean_post );
        }
    }
}

@topher1kenobe What’s the rwmb_get_value() function?

Also do the get_post_type and get_post() lines both make db calls? If so, I’d reverse them.

@shawnhooper https://docs.metabox.io/functions/rwmb-get-value/
could still be the culprit, but it seems straightforward on what it is doing.

Thanks @gilzow

@topher1kenobe is this the only function that’s slow on that server? Is it a large database? Or other things on that same site. Almost wondering if it’s a missing or corrupt index.

rwmb_get_value is from MetaBox and it's a wrapper for get_post_meta. The host sent me some logs that seem to indicate that it's timing out on this file. There are 3400 posts.