【WordPress】カスタム投稿タイプ内カスタムフィールドに、ユーザー定義関数で生成した乱数を自動入力したいです。

Question

大変初歩的な質問で申し訳ございません。
WordPressのfunctions.php内、自作のフィルターフックにおいて、
2、3日以下の課題に取り組んでいますが、全くクリアされないため、
ご質問を失礼いたします。

前提・実現したいこと

①カスタム投稿にて記事を新規投稿時、
②カスタム投稿画面に設置しているカスタムフィールド
（functions.phpへコードを書いて設置したもの）に、
③生成した乱数を自動入力させておく

発生している問題

カスタム投稿の新規投稿画面に進みますが、
カスタムフィールドが空欄のまま、乱数が自動入力されていません。
※エラーはなく、phpファイルの保存自体は成功し、サイトも正常表示されます。

該当のソースコード

【ソースコード】
/**
 * 乱数パスワード生成（カスタムフィールドに）
 **/
add_filter('wp_insert_post_data','SetPwd', 10, 3);

function SetPwd($data, $postarr){
	$pass = makeRandPwd($length);
	if($data["post_type"] === "goods_info"){
		$value = get_post_meta( get_the_ID(), 'password', true );
		if($value == ""){
			update_post_meta($postid, 'password', $pass);
		}
	}
	return $data;
}

function makeRandPwd($length) {
	$length = 16;
	$str = array_merge(range('a', 'z'), range('0', '9'), range('A', 'Z'));
	$r_str = null;
	for ($i = 0; $i < $length; $i++) { 
	$r_str .= $str[rand(0, count($str))]; 
	}
	return $r_str;
}

試したこと

フィルターフックの設定や、ユーザー定義関数の記述方式など、
書籍（「WordPressユーザーのためのPHP入門」）やWordPress Codexを参照したり、
簡単なコーディングにて、投稿タイプやカスタムフィールドの取得、ユーザー定義関数が
正常に動作することを確認いたしました。

また、上記ソースコードを保存時、エラーメッセージは出ておらず、
サイトも正常に表示されます。

補足情報（FW/ツールのバージョンなど）

• 使用サーバー : Xserver
• PHP version : PHP7.3.16
• WordPress version : WordPress 5.4.1

---

上記ソースコードに至るまで、何回か新規にコーディングし直したり、
Codexを参照し、関数やフィルターフックの使用方法が適正かどうか、
Web検索でのコーディング紹介サイトの参照、書籍での基本的知識の獲得などに基づき、
正規の記述方式を守ったつもりではございますが、
これ以上は、どこが根本的に誤っているのか、主観的に冷静な精査ができかねます為、
恥を忍んでこちらにご質問させていただきました。

主観と致しましては、

add_filter('wp_insert_post_data','SetPwd', 10, 3);

function SetPwd($data, $postarr){
    $pass = makeRandPwd($length);
    if($data["post_type"] === "goods_info"){
        $value = get_post_meta( get_the_ID(), 'password', true );
        if($value == ""){
            update_post_meta($postid, 'password', $pass);
        }
    }
    return $data;
}

こちらの関数の記述内容において、

• $data["post_type"]の記述ミスでカスタム投稿タイプが指定できていない
• 'wp_insert_post_data'のフィルターフックが記述ミスで掛かっていない
• add_filterでの、渡す変数の個数が間違っている
• そもそも変数の記述が間違っている

などの誤りがあるのではないかと考えておりますが、どこがどう間違っているのか、
見当がつきかねる状況です。

第三者の方の視点から、どの箇所に誤りがあるのかご指摘いただけますと幸いです。
お手数ではございますが、以上、何卒宜しくお願いいたします。

Answer 1

結果、以下のコーディングにて解決することができました。

/**
 * 乱数パスワード生成（カスタムフィールドに）
 **/
function makeRandPwd($length) {
	$length = 16;
	$str = array_merge(range('a', 'z'), range('0', '9'), range('A', 'Z'));
	$r_str = null;
	for ($i = 0; $i < $length; $i++) {
		$r_str .= $str[rand(0, count($str))];
	}
	return $r_str;
}

function SetPwd($post_ID, $post, $update){
	$pass = makeRandPwd($legth);
	$post_type = get_post_type( $post_ID );
	if ( "goods_info" != $post_type )
		return;
	if ( isset( $pass ) ) {
		update_post_meta( $post_ID, 'password', sanitize_text_field( $pass ) );
	}
}
add_action('save_post','SetPwd', 10, 3 );

この度貴重なアドバイスをいただきました、kei344様、誠にありがとうございました。
もっと精進してまいりたいと思います。

Answer 2

wp_insert_postで試してみてはいかがでしょう。

【カスタムフィールドの自動更新 | ハックノート】
https://hacknote.jp/archives/19693/

例えばwp_insert_post_data内ではupdate_post_metaは使えません。

【post.php in tags/5.4/src/wp-includes – WordPress Trac】
https://core.trac.wordpress.org/browser/tags/5.4/src/wp-includes/post.php#L4155

Comments

[civcxtBjcyknl]

kei344様
早速のご回答をいただき、ありがとうございます。
wp_insert_post_meta内でメタデータを更新することができない旨、存じ上げませんでした。ありがとうございます。
ご提示いただいたリンク先の情報も踏まえ、wp_insert_postにて、
コーディングを試してみたいと思います。
ありがとうございます。

[civcxtBjcyknl]

この度はお世話になっております。
その後いろいろ調べ、結果フックを「save_post」にて解決でき、
新規投稿時にカスタムフィールド へ乱数を自動入力することができました。
この度は貴重なご意見を頂戴し、また大変お世話になりました事、誠にありがとうございます。
今後ももっとコーディングの知識を深めてまいります。

[kei344]

save_postとwp_insert_postは同じ場所に書かれていて、タイミングも引数も同じだと思いますよ。

【post.php in tags/5.4/src/wp-includes – WordPress Trac】
https://core.trac.wordpress.org/browser/tags/5.4/src/wp-includes/post.php#L4144

[civcxtBjcyknl]

よく確認してみたところ、おっしゃる通りでした。
修正前の「function SetPwd」に根本原因がありました。
いろいろ調べていて、最後にたどり着いたのが「save_post」でしたが、
wp_insert_postと働きは同じところを冷静に判断できておりませんでした。
ご指摘いただき、ありがとうございます。

[kei344]

こっちもか。（タイミングと引数は同じ） あとはclass-wp-customize-manager.phpでも使ってはいるけどタイミングと引数は同じ。

【post.php in tags/5.4/src/wp-includes – WordPress Trac】
https://core.trac.wordpress.org/browser/tags/5.4/src/wp-includes/post.php#L4277

[kei344]

To: civcxtBjcyknlさん
コメントが入れ違いましたが、状況把握しました。

[civcxtBjcyknl]

ありがとうございます。
wp_insert_postにコーディングし直しました。
まだ改良点はあるものの、ご指摘いただきました内容を基に、
まず現段階での課題はクリアできました。
誠にありがとうございます。