Wordpressのfuctions.phpでupdate_post_metaを動かしたい

Question

実現したいこと

お世話になります。
下記のようにしてカスタム投稿で更新した時だけ、カスタムフィールドを書き換えたいのですが、うまく動いていないようです。

発生している問題・分からないこと

そもそも$post->IDが取れてなさそうなのですが、どこかが間違っているのかわからず。
ご教示いただければ幸いです。

該当のソースコード

 add_action( 'publish_to_publish', function( $post ) {
    $post_type = $post->post_type;
    $post_id = $post->ID;
    update_post_meta( $post_id, 'data', $post_type );
 } );

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

検証のため、いろいろそぎ落としてここまで内容減らしましたが、やはり動かないのでそもそも書き方が間違っているのかと思いましたが、'publish_to_publish'の場合は、$postの値が想定したものと違うものが入っているのでしょうか。

補足

特になし

Comments

[hiroki-o]

実際に動かしたわけではなく、机上調査ですが、
https://developer.wordpress.org/reference/functions/wp_transition_post_status/
を読む限り、publish_to_publishはサポートされていないと思います。
do_action( "{$old_status}_to_{$new_status}", $post );に対応しているのは、「draft_to_publish」「publish_to_trash」「pending_to_draft」のようです。
(読み間違いでしたら、ごめんなさい)

Answer 1

そもそもフックの仕様なのかと思い、下記のサイトを参考にして作り直したら上書きされずに保存できました。
https://qiita.com/_nakashimamura/items/7e0a6de2ad913239ef4e

Answer 2

FreeBSD 13.2、PHP 8.3.1、WordPress 6.4.3で確認しました。

コメント欄にpublish_to_publishは対応していないと書きましたが、実際に動かしてみると、対応していました。
$post->IDも正常に取れています。

 add_action( 'publish_to_publish', function( $post ) {
    $post_type = $post->post_type;
    $post_id = $post->ID;

    // 以下、変更
    if ($post_type == 'カスタム投稿のpost_type') {
        update_post_meta($post_id, 'カスタムフィールドの名前', 'カスタムフィールドの値');
    }
 } );

Comments

[igarashi1020]

何度か試してもダメで以下のようにして検証してみました。
全く関係ない記事を更新すると正常に動作しました。
どうも動作自体はしているものの、そのあとに書き換えた後に該当のカスタムフィールドの値上書きされているのではないかと思います。

add_action( 'publish_to_publish', function( $post ) {
$post_type = $post->post_type;
update_post_meta('2434165', 'data', $post_type);
} );

なので順番の問題なのだろうと思い、下記のようにしたのですが、今度は更新時に「更新に失敗しました。」と出てしまい動きませんでした；；save_postを使えとか言われそうですが、こっちも別の不具合が出るのでまた別の質問にします。
function test( $new_status, $old_status, $post ){
$post_type = $post->post_type;
$post_id = $post->ID;
update_post_meta('2434165', 'htmlarea', $new_status);
}
add_action('publish_to_publish','test',10,3);

[hiroki-o]

後者は検証していませんが、前者についていうと、通常の投稿/カスタム投稿のいずれでも更新時に呼ばれます。
通常の投稿の場合、$post->post_typeは「post」です。なので、私が提示したソースのように場合分けしないと上書きされます。

[igarashi1020]

回答ありがとうございます。試してみましたが解決できませんでした。
場分けというのは、if ($post_type == 'カスタム投稿のpost_type') {の部分だと思ういますが、
入れてみても上書きされてしまうようです。
私の環境だけですかね。。。

[hiroki-o]

デバッグ出力してみて、更新のタイミングで$postに意図した値が入っているかを確認してください。
(ご存じかもしれませんが、念のため↓)
wp-config.phpを変更
define( 'WP_DEBUG', false );
↓
define('WP_DEBUG', true);
define('WP_DEBUG_DISPLAY', false);
define('WP_DEBUG_LOG', true);

update_post_meta呼び出し前
error_log($post_type);
error_log($post_id);

wp-content/debug.logに出力されます。

[igarashi1020]

回答ありがとうございます。試してみましたが解決できませんでした。
２回ずつ同じ値が入ってました。
下記のページを拝見したところsave_postも２回実行されるようなので同じ仕様なのかと思います。
https://zenn.dev/vanshi/scraps/01d101551ac07c

そもそもフックの仕様なのかと思い、下記のサイトを参考にして作り直したら上書きされずに保存できました。
https://qiita.com/_nakashimamura/items/7e0a6de2ad913239ef4e