WP-PageNaviで2ページ目以降の表示がおかしい

Question

WP-PageNaviでページネーションを実装しました。

固定ページ(single-xxx.php)に「カスタム投稿タイプ」から「求人情報」を出力し、「記事一覧ページ」のようにしました。(「Advance Custom Fields」と「Custom Post Type UI」を使っています)

参考にしたサイト
https://macoblog.com/wp-pagenavi_use/

表示数を3にしテストで4記事作成しましたが、2ページ目に飛んでも最初の3記事が表示されてしまいます。（WordPressの表示設定の「1ページに表示する最大投稿数」は1投稿にしてあります）

コードは下記です。（最初のコードに戻しました）

<?php
/*
Template Name:短期の製造のお仕事情報
*/
?>

<?php get_header(); ?>
    
    <div class="inner">

        <div class="test-card-wrapper">
            <?php
                $my_query = new WP_Query(
                    array(
                        'post_type' => 'recruit',
                        'posts_per_page' => 3,
                        'tax_query' => array(
                            array(
                                'taxonomy' => 'short',
                                'field' => 'slug',
                                'terms' => array(
                                    'short-manufacture',
                                ),
                            ),
                        ),
                    )
                );
                // $queryの条件で記事の有無を確認
                if ( $my_query->have_posts() ):

                // $queryの条件で記事を取得するループ
                while ( $my_query->have_posts() ):
                $my_query->the_post();
            ?>

            <div class="test-card">
                <div class="test-border">
                    <a href="<?php the_permalink(); ?>">
                        <div class="test-number"><?php the_field('job-number'); ?></div>
                        <div class="test-ttl test-row"><?php the_title(); ?></div>
                        <div class="test-flex flex-between test-row test-padding">
                            <div class="test-fa"><i class="fas fa-yen-sign"></i> <?php the_field('salary'); ?></div>
                            <div class="test-fa"><i class="fas fa-map-pin"></i> <?php the_field('work-location'); ?></div>
                            <div class="test-fa"><i class="far fa-clock"></i> <?php the_field('working-hours'); ?></div>
                        </div>
                        <div class="test-detail test-padding"><?php the_field('job-detail'); ?></div>
                    </a>
                </div>
            </div>
            
            <?php
                endwhile;
                wp_reset_postdata(); // メインクエリの $post グローバル変数を復元するために使用

                endif;
            ?>
        </div>
        <?php
            $paged = get_query_var('paged') ? get_query_var('paged') : 1;
            $args = array(
            'post_type' => 'recruit',
            'posts_per_page' => 3,
            'paged' => $paged
            );

            $my_query = new WP_Query( $args );

            if(function_exists('wp_pagenavi')){
                wp_pagenavi(array('query'=>$my_query));
            }

            wp_reset_postdata();
        ?>
    </div><!-- inner -->

<?php get_footer(); ?>

コードは下記です。（ご指摘に対し、追記しました）

// カスタム投稿タイプを設定するregister_post_type関数
add_action( 'init', 'my_post_type' );
function my_post_type() {
  register_post_type(
		'recruit',
		array(
			'label' => '求人募集',
			'public' => true,
			'hierarchical' => true,
			'has_archive' => true,
			'menu_position' => 3,
			'supports' => array('title', 'editor', 'author', 'thumbnail', 'excerpt', 'trackbacks', 'custom-fields', 'comments', 'revisions', 'page-attributes')
		)
	);
}

また、ページネーションの表示数を2ページしかない場合は「①②」だけ表示されるようにしたいのですが、WP-PageNaviの設定からでは出来ないのでしょうか？

参考にしたサイト
https://webkaru.net/wordpress/plugin-wp-pagenavi/

自分なりにググり尽くし、合ってるつもりなのですが、どなたかご教示くださいm(_ _)m

よろしくお願いいたします。

Comments

[kei344]

回答が付いた質問の編集は慎重に行ってください。質問文のコードについて回答にて指摘があった場合は「追記」し、元のコードを編集する場合も「直したこと」がわかるようにしてください。後から見た人から見て、現在の回答が「無いコードに対しての指摘」になり、意味がわからなくなります。

Answer 1

$query = new WP_Queryで表示して$my_query = new WP_Queryでページングを処理しているからでは。

Comments

[hebochans]

ご回答ありがとうございます!!
どちらかに統一しても変わらないようです…。。

[kei344]

統一の仕方によります。

ちなみに固定ページにむりやり一覧ページを表示していますか？その場合は、カスタム投稿タイプを作る際にhas_archiveが指定して、通常のループで処理するほうが良いです。

【WordPress » カスタム投稿タイプの設定 | MORILOG】
http://morilog.com/wordpress/post_type/register_post_type/

[hebochans]

固定ページにむりやり一覧ページを表示しています。

また、書き忘れたのですが「Advance Custom Fields」と「Custom Post Type UI」も使っています。

【固定ページ カスタム投稿タイプをループさせる】

でググったのですが、すみません、、何が何だかサッパリわかりません…。。

[kei344]

> 統一の仕方によります。
> has_archiveが指定して、通常のループで処理するほうが良いです。

この二通りの提示をしていて、それについての反応が無いので、「わからない」ことしかわかりません。

[hebochans]

失礼しました！

> 統一の仕方によります。

$query → $my_query にすべて変更・統一しました。（質問本文を編集済みです）

> has_archiveが指定する

添付していただいたURLを参考にfunctions.phpに追記しました。（質問本文に追記済みです）

> 通常のループで処理する

「通常」がわかっていないのですが…

<?php if ( have_posts() ) : ?>
<?php while ( have_posts() ) : ?>
<?php the_post(); ?>
// ループしたい処理を書く
<?php endwhile; ?>
<?php endif; ?>

上記で合っていますでしょうか？

上記で書くと「サイトに重大なエラーがありました。」となってしまいます…。

現在こんな状況です。

[kei344]

変数を同じ名前にしただけで統一されていません。new WP_Queryを1回だけにしましょう。
また、has_archiveを指定するならテンプレートも変わり、new WP_Query自体も不要になるはずです。（同じスラッグの固定ページがあったらアーカイブより優先されてしまう可能性がある）

>上記で合っていますでしょうか？
「通常のループ」についてはそれで問題ないです。
エラーについては書いた場所に問題があるくらいしか、こちらでは判断できません。

[hebochans]

ご回答ありがとうございます。（何度も付き合わせてしまい申し訳ありません…）

質問のコードを再度修正しました。
現在エラーにはなってませんが、何も取得されていない状態です…。（コードがこれで合っていないのは分かっているのですが…）

> new WP_Query自体も不要になるはずです。

具体的にコードのどこを削除すればよろしいでしょうか？

一番下にあたる「下記コード」は固定ページでWP-PageNaviを使うために必要なので消せないのでは？と考えています。

ソース → https://macoblog.com/wp-pagenavi_use/

<?php
$paged = get_query_var('paged') ? get_query_var('paged') : 1;
$args = array(
'post_type' => 'recruit',
'posts_per_page' => 3,
'paged' => $paged
);

$my_query = new WP_Query( $args );

if(function_exists('wp_pagenavi')){
wp_pagenavi(array('query'=>$my_query));
}

wp_reset_postdata();
?>

[kei344]

二つまったく別の方法を提示してしまったので、伝わらないようですね、すみません。

> 何も取得されていない状態です…。
例えばこれに対して「どこが」とか確認のために毎回聞く必要があるので、私にはこれ以上はサポート不能です。

> 一番下にあたる「下記コード」は固定ページでWP-PageNaviを使うために必要なので消せないのでは？と考えています。
そうですか。WP_Queryが何をしているかをもう一度調べてみてください。

【関数リファレンス/WP Query - WordPress Codex 日本語版】
https://wpdocs.osdn.jp/%E9%96%A2%E6%95%B0%E3%83%AA%E3%83%95%E3%82%A1%E3%83%AC%E3%83%B3%E3%82%B9/WP_Query