Wordpressでカスタム投稿タイプに新規投稿した際に、別のカスタム投稿タイプに投稿内容をコピーして保存したい

Question

前提・実現したいこと

Wordpressの管理画面上からAdvanced Custom Fieldsを使用したカスタム投稿タイプ「ABC」に新規投稿した際に、別のカスタム投稿タイプ「DEF」に投稿内容をコピーして保存したいです。

発生している問題・エラーメッセージ

単純なテキストフィールドは上手くいきましたが、タクソノミーのチェックボックスやグループフィールドが登録されません。
また、新規投稿・更新に関わらず全て新規投稿されてしまう状態で、同じタイトルの投稿が投稿タイプ「DEF」に増え続けてしまっています。

該当のソースコード

[ ver1.1 ]

add_action( 'publish_ABC', 'function_copying_posts', 10);
function function_copying_posts(){
  global $post;

  if($post->post_type == 'ABC'){
      $current_postID = $post->ID;

      // 投稿タイプ「ABC」から情報を取得
      $taxonomy_check = $_POST['acf']["field_60860e98beebc"]; //タクソノミーチェックボックス
      $text = $_POST['acf']["field_6074e41c17011"]; //テキストエリア
      $catch_copy = $_POST['acf']["field_6073f00e426ef"]["field_6073f07d426f0"]; //グループ「copy」のサブフィールド1
      $lead_copy = $_POST['acf']["field_6073f00e426ef"]["field_6073f099426f1"]; //グループ「copy」のサブフィールド2

      // 投稿先の指定
      $draft_my_options = array(
          'post_title'    => get_the_title($current_postID),
          'post_content'  => '',
          'post_status'   => 'publish',
          'post_type'      => 'DEF'
      );
      $newpost_id = wp_insert_post($draft_my_options);

      if($newpost_id){
          update_post_meta($newpost_id, 'taxonomy_check', $taxonomy_check);
          update_post_meta($newpost_id, 'text', $text);
          update_post_meta($newpost_id, 'catch_copy', $catch_copy);
          update_post_meta($newpost_id, 'lead_copy', $lead_copy);
} } }

[ ver1.2 ]

add_action('save_post_ABC', 'function_copying_posts', 10, 3);
function function_copying_posts($post_id, $post, $update) {
　if (wp_is_post_revision($post_id)) return; // 自動保存を除外

  remove_action('save_post_ABC', 'function_copying_posts'); // 無限ループ対策

  // 投稿タイプ「ABC」から情報を取得
  $taxonomy_check = get_field('taxonomy_check', $post_id); //タクソノミーチェックボックス
  $check[] = '';
  foreach($taxonomy_check as $term){
    array_push($check, $term->name);
  }
  $text =  get_field('text', $post_id); //テキストエリア
  $copy = get_field('copy', $post_id);
  $catch_copy = $copy['catch_copy']; //グループ「copy」のサブフィールド1
  $lead_copy = $copy['lead_copy']; //グループ「copy」のサブフィールド2

  // 投稿先の指定
  $draft_my_options = array(
      'post_title'    => get_the_title($post_id),
      'post_status'   => get_post_status($post_id),
      'post_type'      => 'DEF'
  );
  $newpost_id = wp_insert_post($draft_my_options);

  update_post_meta($newpost_id, 'text', $text);
  update_post_meta($newpost_id, 'catch_copy', $catch_copy);
  update_post_meta($newpost_id, 'lead_copy', $lead_copy);
  wp_set_object_terms($newpost_id, $check, 'com_check', false);
  
  add_action( 'save_post_ABC', 'function_copying_posts'); // 無限ループ対策
}

試したこと

情報の取得はできていることをデバッグして確認しました。
update_post_metaが上手くいっていないように思いますが、どう直せばよいか分かりません。

Comments

[tabuu]

publish_postのタイミングでは「$_POST」には情報は入っていないと思います。
「$current_postID」を使ってDBから情報を取得できないでしょうか？

ちなみにpublish_postは公開にしたタイミングで実行されます。
公開と非公開を繰り返すたびに実行されますが問題ありませんか？

[harami_]

$current_postIDを使う方法で情報取得できました！難しく考えすぎていたようです。
公開と非公開を繰り返すたびに実行されても問題ないのですが、非公開の新規投稿をした際に実行されないことに気付いたので、post_updatedに変更しました。
新規・更新に関わらず全て新規投稿されてしまう状態は変わらないので、こちらについても良い方法があればご教授お願いいたします。

[tabuu]

save_postアクションは第三引数で新規か更新か確認できます。
追加投稿をするとさらにsave_postがコールされますのでpost_type等で制御し、無限ループにならないように気を付けてください。

https://wpdocs.osdn.jp/%E3%83%97%E3%83%A9%E3%82%B0%E3%82%A4%E3%83%B3_API/%E3%82%A2%E3%82%AF%E3%82%B7%E3%83%A7%E3%83%B3%E3%83%95%E3%83%83%E3%82%AF%E4%B8%80%E8%A6%A7/save_post

[harami_]

ソースもありがとうございます。いただいたアドバイスを元にコード修正を行いました。
ver1.2がそうなのですが、新規登録時に登録先へ「タイトルのみコピーされた下書き状態の投稿」と「投稿時ステータスとタイトルのみコピーされた投稿」の2件が登録されてしまう状態です。更新ボタン再度押せばすべての情報がコピーされた投稿が登録されます。
自動保存は処理から除外しているつもりなのですが、何かおかしいでしょうか。。

[tabuu]

>add_action('save_post_ABC', 'function_copying_posts', 10, 3);
おそらくですが、ACFの登録処理の前に実行されていると思います。
2つ目の引数の10を99等の大きい数字にしてみてください。

>function function_copying_posts($post_id, $post, $update) {
3つ目の引数の$updateがtrueの場合は更新処理なのでスルーしてよいかと思います。

質問文のソースコードはマークダウンで記述いただくと見やすくなって回答も付きやすく
なりますので今後はそのようにお願い致します。

[harami_]

ソースコードの記述の件、以後気を付けます。
引数の数字変更したところ、下書きは作成されなくなりました！ありがとうございます。
自動保存の除外処理の後に if($update) return; を追記してみたのですが、新規投稿時も登録されなくなってしまいました。記述位置が悪いのでしょうか？

[harami_]

$updateでの新規投稿・更新の振り分けは諦め、投稿日時と更新日時の比較で対応することにしました。
ありがとうございました！

Answer 1

丸3日掛かりましたが、実現したいことはできるようになりました。
引数$updateが使えなかったことは今後の課題としてまたの機会に再チャレンジします。

add_action('save_post', 'copy_homes_post', 99, 3);
function copy_homes_post($post_id, $post, $update) {
  if (wp_is_post_revision($post_id)) return; // 自動保存を除外

  if($post->post_type == 'ABC') { // カスタム投稿タイプ「ABC」
    remove_action('save_post', 'copy_homes_post');  // 無限ループ対策
    if($post->post_modified_gmt == $post->post_date_gmt) { // 新規投稿
      …
    } else {  // 更新
      …
    }
　…
  add_action( 'save_post', 'copy_homes_post');  // 関数を改めてフックする。
}}