日本語スラッグになってしまう

Question

実現したいこと

ワードプレスのパーマリンク構造は、「投稿名」のままbbpressで投稿されたトピックのスラッグをIDベースにしたい。

前提

bbpressで投稿されたトピックのURLスラッグが日本語になってしまいます。
一般的な投稿ページや固定ページの場合は、以下のようなコードを追加すれば、日本語スラッグをIDベースにできるかと思うのですがbbpressの場合は適用できませんでした

function auto_post_slug( $slug, $post_ID, $post_status, $post_type ) {
if ( preg_match( '/(%[0-9a-f]{2})+/', $slug ) ) {
$slug = utf8_uri_encode( $post_type ) . '-' . $post_ID;
}
return $slug;
}
add_filter( 'wp_unique_post_slug', 'auto_post_slug', 10, 4 );

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

上記のコードを仮に実装した場合、bbpressには適用されないため、新しく投稿されたURLスラッグが全て

/topic/topic-0/のようになります。

試したこと

以下のようなこーどを試してみましたが無理でした。

function auto_topic_slug( $slug, $post_ID, $post_status, $post_type ) {
	if ( 'topic' === $post_type && preg_match( '/(%[0-9a-f]{2})+/', $slug ) ) {
		$slug = utf8_uri_encode( $post_type ) . '-' . $post_ID;
	}
	return $slug;
}
add_filter( 'wp_unique_post_slug', 'auto_topic_slug', 10, 4 );

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

ワードプレス、bbpress最新バージョン

Answer 1

トピック登録時をフックしてslugを設定してみてはいかがでしょうか。

function my_bbp_new_topic_pre_insert($topic_data) {
    $topic_data['post_name'] = my_make_post_name($topic_data['post_type']);
    return $topic_data;
}
add_filter('bbp_new_topic_pre_insert', 'my_bbp_new_topic_pre_insert');

function my_make_post_name($post_type) {
    $name = $post_type.'-'.uniqid();
    global $wpdb;
    $c = $wpdb->get_var("SELECT count(*) FROM $wpdb->posts WHERE post_name = '$name'");
    if ($c == 0) {
        return $name;
    } else {
        my_make_post_name($post_type);
    }
}

Comments

[nekokotk]

わざわざ回答ありがとうございます。質問です

'...';の部分には何を追加すれば良いのでしょうか？

[nekokotk]

function my_bbp_new_topic_pre_insert($topic_data) {
$topic_data['post_name'] = 'topic-' . $topic_data['topic_id'];
return $topic_data;
}
add_filter('bbp_new_topic_pre_insert', 'my_bbp_new_topic_pre_insert');

このような感じでしょうか？これだと理想通りにIDベースになるのですがスラッグが重複するリスクはないですかね・・

[nekokotk]

連投してしまいすみません
GPTにも聞いたところこのような案をいただいたのですが、リスクありますかね・・

function my_bbp_new_topic_pre_insert($topic_data) {
$topic_data['post_name'] = 'topic-' . $topic_data['forum_id'] . '-' . uniqid();
return $topic_data;
}
add_filter('bbp_new_topic_pre_insert', 'my_bbp_new_topic_pre_insert');

[tabuu]

>$topic_data['post_name'] = 'topic-' . $topic_data['topic_id'];
そもそもトピック登録前の処理なので、まだトピックのIDはありません。

uniqid()はトピック登録程度の頻度で重複するとは考え難いですが
仮にsulgが重複した場合はWPが123-2、123-3のように自動で連番を付けます。

$topic_dataからフォーラムのIDは下記のように取得します。
$topic_data['post_parent']

[tabuu]

過去に作成した重複チェックのコードを一部修正して追記しました。
3年くらい前に作成したコードなのでそのまま使わず参考程度にしてください。

※get_posts関数ではゴミ箱の中までは探さないので直接DBを検索しています。

[nekokotk]

回答ありがとうございます。

→そもそもトピック登録前の処理なので、まだトピックのIDはありません。

トピック登録前の処理の場合、どのような問題やリスクがあるのでしょうか？

$topic_data['post_name'] = 'topic-' . $topic_data['topic_id'];のコードの方を試しに貼っていたのですが、投稿するとtopic-2 topic-3 と綺麗に投稿した順に数字が増えていくので問題なければこれでもいいのかなと思っているのですが、これのリスク等あれば教えてください、、(無知ですみません)

ちなみにトピックは完全にユーザーが投稿する形なので、手動でいじる予定はありません。

[tabuu]

>$topic_data['topic_id']
これが常に値を返さないので「topic-」に自動で連番を振っているだけです。
slugが重複するとWordPressが誤動作するので、そのための防衛機能だと思っています。
なのであまりおすすめはできません。

ちなみにPHPのバージョンはいくつでしょうか。
PHP8ではエラーになると思いますので実施する場合は常に下記のようにしておく方がよいと思います。
$topic_data['post_name'] = 'topic-';

[nekokotk]

ご返信ありがとうございます。。
なるほど、PHPバージョンは7.4でした
8.025にしてみたところ一応大丈夫ではあったのですがやはり避けた方が良いですかね、、

あまり複雑なスラッグにはしたくなく、1.2.3と連番で増やしていきたいのですが、連番のような形で安全にする方法はないのでしょうか？
何度も質問ばかりですみません。もし可能であればお手隙に確認していただけたらと思います🙇‍♂️

[tabuu]

トピックに対する付番でしたらuniqid()で十分だと思いますけど。
100%重複しないとは言えませんが仮に重複してもWPが枝番を付けてくれますので。

[nekokotk]

確かにそうですね、回答ありがとうございました🙇‍♂️