カスタム投稿タイプのスラッグをランダムにする方法

Question

WordPressでカスタム投稿タイプpotatoのパーマリンクだけランダムにしたいと思うのですが、下記では全ての投稿がランダムになってしまいます。

カスタム投稿タイプの指定でどこが間違っているのでしょうか？

function potato_slug( $data , $postarr ) {

	if ($data['post_type'] == 'potato') { //potatoだけ

		$str = array_merge(range('a', 'z'), range('0', '9'), range('A', 'Z"'));
		$r_str = null;

		for ($i = 0; $i < 12; $i++) {
			$r_str .= $str[rand(0, count($str))];
		}

		if ( ! in_array( $data['post_status'], array( 'trash', 'auto-draft' ) ) ) {
			$data['post_status'] = 'publish';
			$data['post_name'] = $r_str;
		}

	}
	return $data;

}
add_action( 'wp_insert_post_data', 'potato_slug', 99, 2 );

Answer 1

if ($data['post_type'] == 'potato') { //potatoだけ

このif文において、post_typeが'potato'以外の場合も真になっているという質問に置き換えていいでしょうか？
そうであればif文を見る限りは条件分岐に問題はなさそうなので、別の要因があるのではないでしょうか？

Comments

[minminzemkun]

はい、その置き換えで問題ないと思います。
別の要因ですか。なるほど。洗い直してみます。ありがとうございます。

[tabuu]

簡単にデバッグできる状態にあるのであれば、例えば該当のif文の直下に
print $data['post_type'];
exit();
と書いてみて、'potato'以外の場合もif文に入っているかどうかは確認できると思います。

Answer 2

どのようにカスタム投稿タイプを設定しているか分からないのでとりあえずだけど、
add_action ではなく add_filter ではないでしょうか？

Comments

[CHERRY]

wp_insert_post_data ( https://codex.wordpress.org/Plugin_API/Filter_Reference/wp_insert_post_data ) には、add_filter とありますね。

[kei344]

内部でadd_filterを呼んで入るので大丈夫だと思います。

【add_action() | Function | WordPress Developer Resources】
https://developer.wordpress.org/reference/functions/add_action/

[tabuu]

勉強不足でした。ソース読み込みます。