記事の投稿時にエラーが出てしまう（実際に投稿はできています）

Question

前提・実現したいこと

WordpressのPHPで、カスタム投稿タイプ「news」の投稿のときに、スラッグをランダムにさせようと思っているのですけれど、デバッグすると下記のPHPエラーが出てしまいました…

投稿直後にリンクする画面が真っ白（またはデバッグモードなら下記エラーのみが表示される）という状況でして、その真っ白なページを閉じてから確認すると、一応投稿はできているのですが、何かと不具合が心配で解決したいです。

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

実際のエラーメッセージがこちらです。下記の「該当のソースコード」を削除すればこのエラーメッセージは表示されません。

Warning: Missing argument 2 for my_post(), 
called in /export/user/c/zjp_95167c/live_192ba2/var/wordpress/wp-includes/class-wp-hook.php 
on line 286 
and defined in /export/user/c/zjp_95167c/live_192ba2/var/wordpress/wp-content/themes/twentyseventeen-child/lib/custom.php 
on line 374 

Fatal error: Uncaught Error: 
Cannot use object of type WP_Post as array in /export/user/c/zjp_95167c/live_192ba2/var/wordpress/wp-content/themes/twentyseventeen-child/lib/custom.php:375 
Stack trace:
 #0 /export/user/c/zjp_95167c/live_192ba2/var/wordpress/wp-includes/class-wp-hook.php(286): 
 my_post(Object(WP_Post))
 #1 /export/user/c/zjp_95167c/live_192ba2/var/wordpress/wp-includes/class-wp-hook.php(310): 
 WP_Hook->apply_filters('', Array)
 #2 /export/user/c/zjp_95167c/live_192ba2/var/wordpress/wp-includes/plugin.php(453): 
 WP_Hook->do_action(Array)
 #3 /export/user/c/zjp_95167c/live_192ba2/var/wordpress/wp-includes/post.php(4140): 
 do_action('draft_to_publis...', Object(WP_Post))
 #4 /export/user/c/zjp_95167c/live_192ba2/var/wordpress/wp-includes/post.php(3588): 
 wp_transition_post_status('publish', 'draft', Object(WP_Post))
 #5 /export/user/c/zjp_95167c/live_192ba2/var/wordpress/wp-includes/post.php(3746): 
 wp_insert_post(Array, false)
 #6 /export/user/c/zjp_95167c/live_192ba2/var/wordpress/wp-admin/includes/post.php(399): 
 wp_update_p in /export/user/c/zjp_95167c/live_192ba2/var/wordpress/wp-content/themes/twentyseventeen-child/lib/custom.php on line 375 

該当のソースコード

こちらが上のエラーメッセージを表示させるソースコードになります。

function my_rand_slug( $data , $postarr ) {
	if ($data['post_type'] == 'news') {
		$str = array_merge(range('a', 'z'), range('0', '9'), range('A', 'Z"'));
		$r_str = null;
		for ($i = 0; $i < 16; $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( 'draft_to_publish', 'my_rand_slug', 99, 2 );

試したこと

「初めての投稿時」にのみランダムにしたいので、フックの種類はdraft_to_publishが正しいかとは思っているのですが、他のフックは試しても違うように思っていますが、それ以外に修正すべき点がわかりません。

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

Wordpressのバージョンは今朝インストールしたので最新で、PHPも最新だと思います。

初心者につき至らない点もあるかとは思いますので、もし何かございましたら遠慮なく仰ってください。
よろしくお願い致します。

Comments

[nukoshi]

職場の方にこちらで質問したら良いだろうと伺ったのですけれど、まだ質問すること自体が力不足だったようですね。率直で初心者には幾分手厳しいご意見ですが真摯に受け止めたいと思います。

[nukoshi]

またご指摘くださった方がいらっしゃるようですので、別の方法を考えて自分なりに努力してから出直したいと思います。ありがとうございました。

[退会済みユーザー]

猿芝居もいい加減やめたまへ

Answer 1

それ以外に修正すべき点がわかりません。

my_post()の使い方についてエラーが出ているのに別の箇所を調べているから、そこを修正すればよいのでは。

Comments

[nukoshi]

失礼致しました。
「発生している問題・エラーメッセージ」にある「my_post()」ですが、「該当のソースコード」の「my_rand_slug()」を書き換えていたので、エラー箇所としてはそれを指しております。
その後いろいろ調べまして、自分なりに考えてみた方法で改めて質問させて頂くことに致しました。

[kei344]

エラー分のまま、引数の数がおかしいだけですね。
当該フックは引数を1つしか持ちません。

【{$old_status}_to_{$new_status} | Hook | WordPress Developer Resources】
https://developer.wordpress.org/reference/hooks/old_status_to_new_status/

[nukoshi]

そもそも
「draft_to_publish」を書いても
$data['post_type']や
$data['post_status']や
$data['post_name']など
は使えないようで、
この３つなどを使えるのは「draft_to_publish」でなく「wp_insert_post_data」だったようです。

そしてご指摘の引数ですが、これは「wp_insert_post_data」のサンプルコードを見て何も考えずに書いていました。「draft_to_publish」では不要とのこと、覚えておきます。ありがとうございます。

しかし、「draft_to_publish」について引数を消してエラーがなくなっても、上の３つが使えないのでやはり意味がありませんでした。

めちゃくちゃな質問に突き合せてしまってすみませんでした。

[kei344]

> めちゃくちゃですね。。
なにがでしょう？
リンク先を見れば引数で受け取った物が配列では無いこともわかると思ったのですが･･･。

[nukoshi]

めちゃくちゃだと思ったのは、上の３つがスラッグ変更という目的に必須であるのに対し、３つが使えない「draft_to_publish」のエラー解決方法を聞いていた。という点です。せっかくエラー解決方法を教えて頂いたのに目的に活かせないためにお詫び致しました。

あと、
>引数で受け取ったものが配列でない
についてですが、仰る意味がわからないレベルです。

Answer 2

質問自体は解決しませんでしがが、とても良い社会勉強になりました。
どうもありがとうございました。

Comments

[退会済みユーザー]

猿芝居もいい加減やめたまへ