Как исправить ошибку 404 при пагинации?

Question

[Дмитрий Жуков]

Здравствуйте. Сделал произвольный тип записей (files), создал под него отдельный файл архива (archive-files.php). Пагинация появляется, но при переходе 404 ошибка. Вывожу записи в файле архива так:

<div id="content" class="content" role="main">

<?php if( have_posts() ){  $args = array('post_type' => 'files', 'posts_per_archive_page' => 2); query_posts( $args );?>
    <ul class="files-list">
    <?php while( have_posts() ){ the_post(); ?>
        <li>
          Тут вывод произвольных полей и верстка.
        </li>
    <?php }  /* конец while */ ?>
    </ul>

    <div class="navigation">
        <?php the_posts_pagination(); ?>
    </div>

<?php
    wp_reset_query(); } // конец if
else
echo "<h4>Записей нет.</h4>"; ?>

Answer 1

[Игорь Воротнёв]

Никогда не используйте query_posts. Никогда. Сломанная пагинация - это первая проблема, которая всплывает от query_posts. То, что вам нужно (ограничение в 2 поста на странице) делается через хук pre_get_posts.

Comments

[Дмитрий Жуков]

а почему нет? Я ведь с помощью wp_reset_query() аннулирую данные запроса.

[Игорь Воротнёв]

Дмитрий Жуков, потому что ничего вы не аннулируете, вы не понимаете как WP работает внутри. Query_posts заменяет основной запрос (main WP_Query), но делает это после того, как было установлено глобальное состояние движка - в том числе просчитана пагинация на основе как раз основного запроса.

Погуглите почему нельзя использовать query_posts. Даже в официальной документации написано это. И уже лет 5 минимум эту функцию используют только новички, которые учатся по каким-то устаревшим говноурокам, наступают на эти грабли несколько раз и наконец-то учатся делать правильно.

Короткий совет - просто забудьте о том, что существует такая функция.

[Дмитрий Жуков]

Игорь Воротнёв, сделал так:

add_action('pre_get_posts', 'files_pagesize', 1 );
function files_pagesize( $query ) {
    if( is_admin() || ! $query->is_main_query() )
        return;

    if( $query->is_post_type_archive('files') ){
        $query->set( 'posts_per_page', 2 );
    }
}

Заработало.

[Игорь Воротнёв]

Дмитрий Жуков, именно так и должно быть, и просто обязано работать :) можете отмечать ответ, помог ведь?)