WP ページャーが出力記事よりも多く生成されてしまうのを直したい

Question

実現したいこと

タームで絞り込みをかけた際、そのタームが有する記事よりも多くのページが生成されてしまう。
ページャーを、出力されている記事数分だけ生成させたい。

前提

2つの投稿タイプで同一のタクソノミーを利用しており、下記のような構造になっています。

カスタム投稿タイプ
├ NEWS
└ SHOP

共通タクソノミー
items
├ items1
├ items2
├ items3
└ items4

タクソノミーは1つのためtaxonomy.phpのみ用意し、
タームで絞り込んだ時にURLに「news」が含まれていたら NEWS用のテンプレートcontent-news.phpを読み込み、
「shop」が含まれていたら テンプレートcontent-shop.phpを読み込むように条件分岐しています。

起きている問題

ターム一覧ではそれぞれの投稿タイプの記事を取得できるのですが、ページャーが投稿タイプNEWSとSHOPの合算された記事分生成されます。

ターム「items1」で例にすると、
NEWSで3記事、SHOPで10記事を有しているとします。

NEWSアーカイブページ（archive-news.php）
→ item1で絞り込み（taxonomy.php内でcontent-news.phpを読み込み）
→ item1のNEWSアーカイブページ

上記のitem1のNEWSアーカイブページで、
1ページに最大5記事まで出力される場合、本来であればページャーは生成されないはずですが、
3ページ分のページャーが生成されてしまいます。
ちなみに記事は3記事のみが表示され、2ページ目以降は記事はなしでページャーのみが存在する状態です。

恐らく、数的にNEWSとSHOPを分別できずに合わさった分が生成されているのではないかと思います。

該当のソースコード

ページャーの部分（functions.phpとcontent-news.php)、記事出力部分（content-news.php）のコードは以下です。

//functions.php
<?php
function pagination( $pages, $paged, $range = 2, $show_only = false ) {
    $pages = ( int ) $pages;
    $paged = $paged ?: 1;
    $text_first   = "<<";
    $text_before  = "＜";
    $text_next    = "＞";
    $text_last    = ">>";
    if ( $show_only && $pages === 1 ) {
        echo '<div class="pager"><ul class="pager-list"><li><a class="current" href="">1</a></li></ul></div>';
        return;
    }
    if ( $pages === 1 ) return;    // １ページのみで表示設定もない場合
    if ( 1 !== $pages ) {
        //２ページ以上の時
        echo '<div class="pager"><ul class="pager-list">';
        // echo '<div class="pagination"><span class="page_num">Page ', $paged ,' of ', $pages ,'</span>';
        //現在のページ数などを表示する場合はこの上のコメントアウトを外す。
        if ( $paged > $range + 1 ) {
            // 「最初へ」 の表示
            echo '<li class="prev-area"><a href="', get_pagenum_link(1) ,'" class="prev">', $text_first ,'</a></li>';
        }
        if ( $paged > 1 ) {
            // 「前へ」 の表示
            echo '<li class="prev-area"><a href="', get_pagenum_link( $paged - 1 ) ,'" class="prev">', $text_before ,'</a></li>';
        }
        for ( $i = 1; $i <= $pages; $i++ ) {
            if ( $i <= $paged + $range && $i >= $paged - $range ) {
                // $paged +- $range 以内であればページ番号を出力
                if ( $paged === $i ) {
                    echo '<li><a class="current" href="">', $i ,'</a></li>';
                } else {
                    echo '<li><a href="', get_pagenum_link( $i ) ,'">', $i ,'</a></li>';
                }
            }
        }
        if ( $paged < $pages ) {
            // 「次へ」 の表示
            echo '<li class="next-area"><a href="', get_pagenum_link( $paged + 1 ) ,'" class="next">', $text_next ,'</a></li>';
        }
        if ( $paged + $range < $pages ) {
            // 「最後へ」 の表示
            echo '<li class="next-area"><a  href="', get_pagenum_link( $pages ) ,'" class="next">', $text_last ,'</a></li>';
        }
        echo '</ul></div>';
    }
}
?>

//content-news.php(taxonomy.phpでインクルードしている)
　　　　<ul>
            <?php
			$paged = get_query_var('paged') ? get_query_var('paged') : 1 ;
			$term = wp_get_object_terms($post->ID, 'items');
			$args = array( 
				'post_type' => 'news',
				'taxonomy' => 'items',
				'term' => $term[0]->slug,
				'posts_per_page' => 5,
				'paged' => $paged
			);
			$custom_query = new WP_Query( $args );
			if ( $custom_query->have_posts() ) : 
				while ( $custom_query->have_posts() ) : 
					$custom_query->the_post(); 
			?>
			 <!-- ▽ ループ開始 ▽ -->
            <li>
                記事内容
            </li>
            <!-- △ ループ終了 △ -->

			<?php endwhile;
			endif;
			wp_reset_postdata();
			?>
        </ul>
		<!-- ページネーション -->
        <?php
        if ( function_exists( 'pagination' ) ) :
            pagination( $wp_query->max_num_pages, get_query_var( 'paged' ) );
        endif;
        ?>

ちなみに、content-shop.phpも上記のcontent-news.phpと構造はほぼ同じです。
調べてもわからず、お力添えを願います。

Answer 1

$custom_query で、サブループを使って投稿を表示されていますが、ページネーションだけ、$wp_query なのは何故でしょうか？

Comments

[ztj]

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

恥ずかしながらメインループ・サブループについて深く理解していなく、
ページャーもほぼコピペで作成したものでした。
ご指摘いただいたページャー部分の
$wp_query を $custom_query に変更したら正常に生成されるようになりました。

<?php
if ( function_exists( 'pagination' ) ) :
pagination( $custom_query->max_num_pages, get_query_var( 'paged' ) );
endif;
?>

サブループ等について、もっと勉強して参ります。
ご教授いただきありがとうございました！

Answer 2

ここが問題っぽいです

<?php
        if ( function_exists( 'pagination' ) ) :
            pagination( $wp_query->max_num_pages, get_query_var( 'paged' ) );
        endif;

max_num_pagesだと共通テーブルの最大値を取得することになるので、タクソノミーごとの件数を取得するようにしましょう。ここが参考になるかと。

WPのカスタム投稿タイプの件数をトップページで表示する方法

Comments

[ztj]

ご回答ありがとうございます。
やはりページャー部分が問題だったのですね。
今回はページャー部分の
$wp_query を $custom_query に変更したら正常に生成されるようになりました。

ページャーについてもほぼコピペで作成していたので、
カスタマイズできるようリンク先の記事も参考にさせていただきます。
ありがとうございました！