Q&A
条件分岐に使っている $new_status や $old_status や $post->post_type の値は、どのような値になっていますか？
お世話になっております。よくわからないのでお教えいただければ幸いです。
WORDPRESSのカスタム投稿タイプ「luserpage」に新規投稿した場合、該当メールアドレスへ通知いく内容を情報を参考にして現在以下のコードをテーマのfunction.phpに記載をしています。
無事にメールへの送信は出来ているのですが、新規投稿を→一旦下書き→公開にしないとメールが飛ばないようです。
これを、いきなり「公開」にした場合、メールを飛ばすようにしたいのですが、分かる方お教えいただきたく存じます。
お手数をおかけしますがどうぞよろしくお願いいたします。
該当ソースコード
add_action('transition_post_status', function ($new_status, $old_status, $post) {
//「新規投稿」、「承認待ち」、「下書き」、「自動保存」、「予約済」のステータスから「公開済」に変化した時に処理が行われる。
if (($old_status == 'new' || $old_status == 'pending' || $old_status == 'draft' || $old_status == 'auto-draft' || $old_status == 'future') && $new_status == 'publish' && $old_status != 'publish' && $post->post_type == 'luserpage') {
$user_select = get_field('user_select', $post->ID); //カスタムフィールド「user_select」を取得（返り値はUser ID）
$user_emaillist = array(); //空の配列を定義
foreach ($user_select as $user_id) {
$user_data = get_userdata($user_id); //ユーザーIDからユーザー情報を取得
$user_email = $user_data->user_email; //ユーザー情報の中からメールアドレスを取得
array_push($user_emaillist, $user_email); //取得したメールアドレスを$user_emaillistに追加していく
}
$mail_to = implode(',', $user_emaillist); //メールアドレスだけの配列をカンマ区切りの文字列にして$mail_toに代入
// 送信先のメールアドレス
$to = $mail_to;
以下メール設定情報
回答1件
0
WordPress 6.6.2で確認しました。
標準の投稿($post->post_type=='post')で試してみましたが、
| $old_status | $new_status | |
|---|---|---|
| 公開 | auto-draft | publish |
| $old_status | $new_status | |
|---|---|---|
| 下書き保存 | auto-draft | draft |
| 公開 | draft | publish |
となり、$post->post_type=='luserpage'が適切であるなら、提示しているif文は正しいです。
以下の方法でデバッグしてみてください。
wp-config.phpのdefine( 'WP_DEBUG', false );を書き換える。
define( 'WP_DEBUG', true ); define( 'WP_DEBUG_LOG', true ); define( 'WP_DEBUG_DISPLAY', true );
if文の前に追加する。
error_log(print_r($new_status, true)); error_log(print_r($old_status, true)); error_log(print_r($post->post_type, true));
実行すると、wp-content/debug.logに出力されます。その際、$old_statusが「new」となる出力は無視してください。
上記で不明な点があれば、コメントしてください。
投稿2024/09/23 09:01
総合スコア1377
hiroki-oさま
アドバイスいただきありがとうございます。
良く分からないまま、アドバイス通りにやってみたと所、
debug.logには以下のように出力されていました。
繰り返し実行されているようなのでその一部は以下のようになります。
＝＝ここから＝＝
[25-Sep-2024 05:26:31 UTC] PHP Warning: Invalid argument supplied for foreach() in /export/sd202/www/jp/r/e/gmoserver/3/6/sd0205736/hoge/wp-content/themes/twentysixteen-child-umoto/functions.php on line 91
[25-Sep-2024 05:26:32 UTC] PHP Deprecated: 関数 has_cap がバージョン 2.0.0 から<strong>非推奨</strong>になった引数付きで呼び出されました。
ユーザーレベルの使用は推奨されていません。代わりに権限を使ってください。
in /export/sd202/www/jp/r/e/gmoserver/3/6/sd0205736/hoge/wp-includes/functions.php on line 6085
[25-Sep-2024 05:26:32 UTC] publish
[25-Sep-2024 05:26:32 UTC] publish
[25-Sep-2024 05:26:32 UTC] luserpage
[25-Sep-2024 05:24:55 UTC] PHP Deprecated: 関数 has_cap がバージョン 2.0.0 から<strong>非推奨</strong>になった引数付きで呼び出されました。ユーザーレベルの使用は推奨されていません。代わりに権限を使ってください。
＝＝ここまで＝＝
これを見ると91行目のfunction.phpに「無効な引数がforeachに渡されています」ということだと思うのですが91行目の部分は
foreach ($user_select as $user_id) {
$user_data = get_userdata($user_id); //ユーザーIDからユーザー情報を取得
$user_email = $user_data->user_email; //ユーザー情報の中からメールアドレスを取得
array_push($user_emaillist, $user_email); //取得したメールアドレスを$user_emaillistに追加していく
}
になります。
ユーザーレベルの使用は推奨されていません。代わりに権限を使ってください。 の部分が良く分からないのですがもしよろしければアドバイスいただけますと助かります。
どうぞよろしくお願いいたします。
提示されている情報からは、関数has_capが何だかわからないので、答えられません。
ログの切り取り方の問題かもしれませんが、publish/publish/luserpageより前でエラーが出ているのですか？
foreachはif文より後ですよね？
まずは、いきなり公開した時と下書き保存→公開した時で3変数の中身を見て、if文が通るか確認してください。
if文が通るなら、$user_idや$user_emailの中身を見て(方法は3変数と同じ)、いきなり公開した時と下書き保存→公開した時の違いを確認してください。
違いを発見できたら、has_capのエラーを出している関数を絞り込めるかもしれません。(あるいは関係無いかもしれません)
そうやって、デバッグしてみてください。
あなたの回答
tips
プレビュー
