ワードプレスのカスタム投稿でカテゴリ別ページネーションに全投稿数のナビが表示されてしまいます

Question

###前提・実現したいこと
初めて質問させていただきます。
ソースコードは自分で書いたわけではなく、既存のものをコピーした上で、手探りで変えている状態です。
プログラムは素人同然のため、質問の仕方に不足や問題があればご指摘いただければ助かります。
カスタム投稿でカテゴリ別のページネーションを正しい投稿数（該当カテゴリの投稿数）だけ取得したいです。
よろしくお願いします。

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

カスタム投稿で、全記事では、問題なく動作しているのですが、カテゴリ別で表示した時、ページネーション自体は動作しますが、全投稿数分のナビが表示されてしまいます。
全投稿数 20
カテゴリA 投稿数10
1ページの表示数5と設定した場合、全投稿では1｜2｜3｜4になります。
カテゴリAの一覧では1|2になってほしいのですが、1｜2｜3｜4となり、投稿数に満たない3と4はnotfoundになっていまいます。

###該当のソースコード
function.php

function get_pagination($max_pages = '', $post_type = 'post', $range = 2)
{
  $showitems = ($range )+1;
  global $paged;
  if(empty($paged)) 
  {
    $paged = 1;
  }
  if(empty($max_pages))
  {
    $max_pages = 1;
  }

  if(1 != $max_pages)
  {
    // 連番
    echo '<ul class="pager">';
    
    // 前に
    //if($paged > 1 && $showitems < $max_pages) 
    if($paged > 1) 
    {
      echo "<li class='prev'><a href='".get_pagenum_link($paged - 1)."'>前へ</a></li>";
    }
    
    for ($i=1; $i <= $max_pages; $i++)
    {
      if (1 != $max_pages &&( !($i >= $paged+$range+1 || $i <= $paged-$range-1) || $max_pages <= $showitems ))
      {
        echo ($paged == $i)? "<li><a href='".get_pagenum_link($i)."' class='current'>".$i."</a></li>":"<li><a href='".get_pagenum_link($i)."' class='inactive' >".$i."</a></li>";
      }
    }
    
    // 次に
    //if ($paged < $max_pages && $showitems < $max_pages) 
    if ($paged < $max_pages) 
    {
      echo "<li class='next'><a href='".get_pagenum_link($paged + 1)."'>次へ</a></li>";
    }
    
    echo '</ul>';


  }

}

add_action('pre_get_posts', 'my_pre_get_posts');
function my_pre_get_posts($query) {
	if ( is_admin() || ! $query->is_main_query())
        return;
 
    if ( $query->is_post_type_archive('news') ) {
        $query->set( 'post_type', 'news' );
        $query->set( 'posts_per_page', '2' );
    }
    if ( $query->is_tax('newscat') ) {
        $query->set( 'post_type', 'news' );
        $query->set( 'posts_per_page', '2' );
    }

}

taxonomy-newscat.php

<?php echo get_pagination(ceil(count(get_posts( 'post_type='.$post_type.'&numberposts=-1' ))/$num), $post_type); ?>

###補足情報(言語/FW/ツール等のバージョンなど)
WordPress 4.8

Comments

[退会済みユーザー]

参考にされたサイトはこちら( https://www.kabanoki.net/67 )でしょうか。こちらのサイトでは、get_posts関数を利用しなければならないという条件で実装されているようですが、質問者様はget_posts関数を使わなければならない理由はありますか。

[maomao0316]

ありがとうございます。参考にしたのはこちらのサイトではないですが、もともと参考にしたものはこのソースを参考にしていると思います。「get_posts関数を使わなければならない理由」はありません。やりたいことを実装できているサイトを参考にしたまでで、その意味がわかっていないレベルです。。

Answer 1

miz様の質問から推測して、自己解決しました！

<?php echo get_pagination(ceil(count(get_posts( 'post_type='.$post_type.'&numberposts=-1' ))/$num), $post_type); ?>

を、

<?php if (function_exists("get_pagination")) get_pagination($wp_query->max_num_pages); ?>

に変えたら思い通りに動いているようです。
ありがとうございました！