query_postsからWP_Queryへの書き換えについて

Question

WordPressで関連記事を出力するプログラムを書いているのですが、query_postsを使って書いたところquery_posts以降のデザインが崩れてしまいました。

related.phpというPHPファイルを作り、page.phpで呼び出した場所に関連記事を表示するようにしています。

<?php
/*
	ファイル名：page.php
*/
?>
<?php the_content() ?>
<?php get_template_part( 'related' ); ?>
<?php get_sidebar() ?>

よく調べたところquery_postsは非推奨でWP_Queryで書くことを推奨していたので、パラメータは同じまま「query_posts」を「new WP_Query」に書き換えたところエラーは出ませんでしたが結果が変わってしまいました。

<?php
/*
	ファイル名：related.php
*/
?>
<?php
$tags = wp_get_post_tags($post->ID);
$array = array();

foreach($tags as $tag){
	array_push($array, $tag->term_id);
}

$posts = query_posts(     // ←query_postsをnew WP_Queryに書き換え
	array(
		'tax_query' => array( 
			array(
				'taxonomy' => 'post_tag',
				'terms' => $array,
				'include_children' => true,
				'field' => 'term_id',
				'operator' => 'IN'
			),
			'relation' => 'AND'
		)
	)
);
?>

<h3>関連記事</h3>
<div id="related">
<dl>
	<?php
	if(count($posts)){
		foreach($posts as $post){
			$id = $post -> ID;
	?>
	<?php
			the_title( '<dt><a href="' . esc_url( get_permalink() ) . '">', '</a></dt>' );
	?>
	<dd>
	<?php
			echo get_post_meta($id, 'description', true);
	?>
	</dd>
	<?php
		}
	?>
	<?php
	}else{
		echo "関連記事はありません。";
	}
	?>
</dl>
</div>

query_postsでは結果が取得できていたのですが、WP_Queryで書き換えると以下のように何の条件で出力されたのかわからない結果となりました。

<dl>
<dt><a href="http://uwsc.s1007.xrea.com/?p=1">Hello world!</a></dt>
<dd></dd>
<dt><a href="http://uwsc.s1007.xrea.com/?p=1">Hello world!</a></dt>
<dd></dd>
<dt><a href="http://uwsc.s1007.xrea.com/?p=1">Hello world!</a></dt>
<dd></dd>
<dt><a href="http://uwsc.s1007.xrea.com/?p=1">Hello world!</a></dt>
<dd></dd>
</dl>

WP_Queryの使い方をまとめたサイトで見つけたプログラムなど実行してみても「このサイトで重大なエラーが発生しました。」と表示されてしまいます。

WP_Queryの戻り値から生成されたSQLを実行すると0件でヒットしませんでした。

var_dump($posts);

SELECT SQL_CALC_FOUND_ROWS  wp_posts.ID FROM wp_posts  LEFT JOIN wp_term_relationships ON (wp_posts.ID = wp_term_relationships.object_id) WHERE 1=1  AND ( 
  wp_term_relationships.term_taxonomy_id IN (41,42,43)
) AND wp_posts.post_type = 'post' AND (wp_posts.post_status = 'publish' OR wp_posts.post_status = 'private') GROUP BY wp_posts.ID ORDER BY wp_posts.post_date DESC LIMIT 0, 10

回答よろしくお願い致します。

-----▼追加分-----

以下はWP_Queryで書いたときのプログラムです。

<?php
$tags = wp_get_post_tags($post->ID);
$array = array();

foreach($tags as $tag){
	array_push($array, $tag->term_id);
}

$posts = new WP_Query(
    array(
        'tax_query' => array( 
            array(
                'taxonomy' => 'post_tag',
                'terms' => $array,
                'include_children' => true,
                'field' => 'term_id',
                'operator' => 'IN'
            ),
            'relation' => 'AND'
        )
    )
);
?>

Comments

[CHERRY]

WP_Query で、どのように書き換えたのかも記載していただけないでしょうか？

[akitasoran]

返信ありがとうございます。

WP_Queryで調べたところ指定するパラメータがquery_postsと同じもので使えそうだったので、関数名だけの書き換えとなります。

Answer 1

解決済みとのことですが。

こういったメインループとは異なるポストを取得するのは get_posts() を使うのが推奨だったかと思います。

get_posts() で取得した投稿を処理した後、 wp_reset_postdata() とすることでメインループに戻す、といった動作ができます。

Answer 2

回答に頂いたプログラムを実行してみても結果は変わらずHello World!がいくつも出力されるだけでした。

var_dumpで出力されたのかSQLを元に色々試してみたところ、以下のプログラムで関連記事の取得ができました。

回答して頂いたのに申し訳ありませんでした。

<?php
/*
	Template Name: 関連記事
*/
?>
<?php
$tags = wp_get_post_tags($post->ID);
$array = array();

foreach($tags as $tag){
	array_push($array, $tag->term_id);
}

$tag = implode(",", $array);

$sql = 'SELECT SQL_CALC_FOUND_ROWS * FROM wp_posts INNER JOIN wp_term_relationships ON wp_posts.ID = wp_term_relationships.object_id WHERE wp_posts.ID != ' . get_the_ID() . ' AND wp_term_relationships.term_taxonomy_id IN (' . $tag . ') AND wp_posts.post_status = "publish" GROUP BY wp_posts.ID ORDER BY count(post_title) DESC;';

global $wpdb;
$query = $wpdb -> get_results($sql, ARRAY_A);

if(count($query)){
	echo '<h3>関連記事</h3>';
	echo '<dl>';
	foreach($query as $row){
		$link = get_permalink($row["ID"]);
		echo '<dt><a href="' . $link. '">' . $row['post_title'] . '</a></dt>';
		echo '<dd>' . get_post_meta($row["ID"], "description", true) . '</dd>';
	}
	echo '</dl>';	
}
?>

Answer 3

WP_Queryで調べたところ指定するパラメータがquery_postsと同じもので使えそうだったので、関数名だけの書き換えとなります。

なるほど...

query_posts は、テンプレートタグ/query_posts に

注 2: query_posts() はページ内のメインクエリーを書き換え、新しいクエリーのインスタンスと置き換えるために使う関数としては過度に単純化され、問題が発生しやすい方法です。非効率的で（SQL クエリを再実行します）、一部の状況では適切に実行することもできません（特にページング処理）。

とあるように「メインクエリを書き換えています」($post 等が上書きされていました)ので、質問のコードで動いていましたが、WP_Query は、メインクエリを書き換えないので、$post が更新されないので、投稿の数だけ同じ内容の行が複数行表示されます。

関数リファレンス/WP_Query の「使い方」にあるように

<?php

// The Query
$the_query = new WP_Query( $args );

// The Loop
if ( $the_query->have_posts() ) {
	echo '<ul>';
	while ( $the_query->have_posts() ) {
		$the_query->the_post();
		echo '<li>' . get_the_title() . '</li>';
	}
	echo '</ul>';
	/* Restore original Post Data */
	wp_reset_postdata();
} else {
	// no posts found
}

のような感じで、 ループの中で、$the_query->the_post(); を利用して、$post を上書きして処理してループを抜けてから wp_reset_postdata(); で、$post をメインクエリに戻します。

また、foreach を使うのであれば、 テンプレートタグ/get_postsの「用例」のように

foreach ( $myposts as $post ) : setup_postdata( $post ); ?>
	<li>
		<a href="<?php the_permalink(); ?>"><?php the_title(); ?></a>
	</li>
<?php endforeach; 
wp_reset_postdata();?>

ループ内で、setup_postdata( $post ); で、ループのデータで $post を設定して、 ループを抜けてから wp_reset_postdata(); で、$post をメインクエリに戻しておくように記載します。