特定のカスタムフィールドをタクソノミー別に表示させたいです

Question

前提・実現したいこと

お世話になります。
特定のカスタムフィールドにある全てのタクソノミーをタクソノミー別に、タクソノミ名とそれに該当する記事の画像フィールドの画像とタイトルを全て表示させたいです。

商品カテゴリ１(ターム)
　ー画像フィールド + 記事タイトル　１－１
　ー画像フィールド + 記事タイトル　１－２

商品カテゴリ２(ターム)
　ー画像フィールド + 記事タイトル　２－１
　ー画像フィールド + 記事タイトル　２－２
　ー画像フィールド + 記事タイトル　２－３

商品カテゴリ３（ターム）
　ー画像フィールド + 記事タイトル　３－１
　ー画像フィールド + 記事タイトル　３－２
　ー画像フィールド + 記事タイトル　３－３
　ー画像フィールド + 記事タイトル　３－４
　ー画像フィールド + 記事タイトル　３－５

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

下記のコードにてタクソノミ名は全て表示することができたのですが、画像フィールドの画像が表示されず、タイトルが通常投稿のタイトルが表示されてしまいました。

画像を表示してタームの記事タイトルを出したいです。
どうぞご指摘のほどよろしくお願い致します。

投稿タイプスラッグ：menu_post
タクソノミースラッグ：bread
カスタムフィールドの画像フィールド名：menu-photo
画像の返り値：ID

<?php
$terms = get_terms('bread');
foreach( $terms as $term )  :
  echo '<h2><span>' . $term->name . '</span></h2><ul>';
	query_posts('showposts=5&cat=' . $term->term_ID);
		while(have_posts()) :
		the_post();?>
		<li>
			<?php $image = get_field('bread','menu_spot_'.$term_id); if( !empty($image) ): ?>
				<img src="<?php echo $image['url']; ?>" alt="<?php echo $image['alt']; ?>" />
			<?php endif; ?>
			<?php the_title(); ?>
		</li>
    <?php 
    endwhile;
  wp_reset_query();
  echo '</ul>';
endforeach;
?>

試したこと

画像の表示で下記も試しましたが表示できませんでした。

<?php
  $image = get_field('menu-photo');
  $size = 'full'; 
  if( $image ) {
    echo wp_get_attachment_image( $image, $size );
  }
?>

Answer 1

query_postsはプラグインやテーマで使われることを想定しない関数です

query_posts

一般的な投稿の取得には、WP_Query または get_posts を使ってください。
query_posts の代わりに次のようにすることを強く推奨します：

1. アクション pre_get_posts を用いる。
2. pre_get_posts にフックした関数の中で is_main_query() を使ってメインクエリーであるかチェックする。
3. メインクエリーの場合のみ、それを変更する。

get_field('bread','menu_spot_'.$term_id)
↓
get_field('bread','menu_spot_'.$term->term_id)

なのでは？

Comments

[matsun]

お世話になります。
pre_get_postsについて分からなかったので、調べてfunctions.phpに試してみたのですが、うまく動かずエラーが出てしまいました。
度々申し訳ないのですがご助言頂けませんでしょうか。
```
<?php
function term_pre_get_posts( $query ) {
/* 管理画面,メインクエリに干渉しないために必須 */
if ( is_admin() || ! $query->is_main_query() ){
return;
}

if ( $query->is_post_type_archive('menu_post')) {
$query->set( 'post_type', 'menu_post' );
$tax_query[] = array(
'taxonomy' => 'bread',
'field' => 'term_id',
'operator' => 'IN');
$query->set( 'tax_query', array('relation' => 'AND' , $tax_query) );
}
}

add_action( 'pre_get_posts', 'term_pre_get_posts' );
?>
```

※エラーメッセージ
```
Error: Call to a member function is_main_query() on null in C:\Users\bette\Local Sites\sample\app\public\wp-content\themes\pizza\functions.php on line 98
Call Stack
# Time Memory Function Location
1 0.0005 390728 {main}( ) ...\index.php:0
2 0.0012 390928 require( 'C:\Users\bette\Local Sites\sample\app\public\wp-blog-header.php' ) ...\index.php:17
3 0.1960 6737456 require_once( 'C:\Users\bette\Local Sites\sample\app\public\wp-includes\template-loader.php' ) ...\wp-blog-header.php:19
4 0.2007 6759968 include( 'C:\Users\bette\Local Sites\sample\app\public\wp-content\themes\pizza\page-menu.php' ) ...\template-loader.php:78
5 0.2090 6795688 apply_filters( ) ...\page-menu.php:5
6 0.2090 6795984 WP_Hook->apply_filters( ) ...\plugin.php:208
7 0.2092 6797504 do_shortcode( ) ...\class-wp-hook.php:286
8 0.2092 6799216 preg_replace_callback ( ) ...\shortcodes.php:199
9 0.2092 6799696 do_shortcode_tag( ) ...\shortcodes.php:199
10 0.2092 6800080 my_php_Include( ) ...\shortcodes.php:325
11 0.2097 6817784 include( 'C:\Users\bette\Local Sites\sample\app\public\wp-content\themes\pizza\menu-archive.php' ) ...\functions.php:22
12 0.2113 6834224 term_pre_get_posts( ) ...\menu-archive.php:8
```

[KazuhiroHatano]

引用の注意文はサブループでメインクエリを上書きするのをやめましょう
メインクエリを上書きする必要がある場合でも
query_postsではなくadd_action('pre_get_posts',...);を使いましょうということです

質問のケースではget_postsを使うか
WP_queryオブジェクトを作ってループを書くかで対応すべきです

https://wpdocs.osdn.jp/関数リファレンス/WP_Query

質問のコードについては
今一度変数の中身を細かくvar_dumpするなりして
想定通りの値が得られているか確認したほうが良いかと思います

[matsun]

自分がいかにループやメインクエリについて理解してなかったかを身をもって知りました。
基本的なところから勉強し直してようやく解決できました。

ご指摘いただいた事で気付くことができました。
ありがとうございました！