Некорректно отрабатывает pre get posts. Как решить?

Question

[Вадим Свердлик]

У меня в в functiions.php такой код,но на страницах категорий выводится 5 постов а не 2, 5 указано в админке. Если закоментировать

if ( is_admin() || ! $query->is_main_query() ) {
        return;
    }

то тогда на старнице категорий выводится по 2 записи как и нужно,но тогда на главнй странице выводит не то количество. Подскажите в чем может быть проблема

function custom_posts_per_page( $query ) {

    if ( is_admin() || ! $query->is_main_query() ) {
        return;
    }


    if ( $query->is_search() ) {
        $query->set( 'posts_per_page', 8 );
    }

    if ( $query->is_category() ) {
        if ( wp_is_mobile() ) {
            $query->set( 'posts_per_page', 2 );
        } else {
            $query->set( 'posts_per_page', 2 );
        }

    }//endif

}//function

add_action( 'pre_get_posts', 'custom_posts_per_page' );

Answer 1

[WP Panda]

Наводящий вопрос, тему сами делали?

Comments

[Вадим Свердлик]

Абссолютно сами,с нуля

[WP Panda]

Вадим Свердлик, второй наводящий вопрос, а как вы выводите записи в категории?

[Вадим Свердлик]

WP Panda,

$args=array(
                          'category__in' => $cat_ID,
                          'orderby'=>data,
                          'paged' => $paged,
                          'caller_get_posts'=>1);

                          $my_query = new wp_query($args);

                        $page = (get_query_var('paged')) ? get_query_var('paged') : 1;
                        query_posts("paged=$page&cat=1");

                          if( $my_query->have_posts() ) {
                          while ($my_query->have_posts()) {
                          $my_query->the_post(); ?>

                          <?php $cat2 = get_query_var('cat');
                           $category = get_category ($cat2);?>

                            <div class="category-result-post">
                                <div class="category-news-post-img">
                                    <a href="<?php echo get_permalink($post->ID); ?>">
                                        <?php if(get_the_post_thumbnail_url($post->ID)) echo get_the_post_thumbnail($post->ID); else echo ' <img alt="#" src="http://blogprogram.ru/wp-content/uploads/2016/05/noimg.png">';?>
                                    </a>
                                </div>
                                <div class="category-result-detail">
                                    <div class="category-result-detail-body">
                                        <p class="date-news"><?php the_time( 'd.m.Y'); ?></p>
                                        <div class="category-result-post-bottom">
                                            <p class="posts-p"><a href="<?php echo get_permalink($post->ID); ?>" class="news-post-text"><?php echo $post->post_title; ?></a></p>
                                        </div>
                                    </div>
                                </div>
                            </div>

                              <?php  }
                               } else {
                              echo('<p>Извините, нет записей.</p>');
                          }

                         wp_reset_query();
                         ?>
<!--                            --><?php //the_posts_pagination();?>
                        </div>

[WP Panda]

Вадим Свердлик,

Вот почти добрались, до сути))

Вопрос номер 3, зачем вы выводите их так, а не обычным циклом? для чего вы переопределяете стандартный запрос?

[Вадим Свердлик]

WP Panda, изначально не планировал использовать pre-get posts, и я так понимаю,что мне ж нужно будет передавать параметры..той же пагинации

[WP Panda]

pre get posts не работает в данном случае потому, что у вас есть там проверка на основной цикл, а у вас цикл произвольный, когда вы ее убираете, оно конечно начинает работать везде))

Вы немного не понимает, что вы делаете, не надо ничего усложнять, пробегитесь по докам про разработку тем, хоть по диагонали.

Вот, то, что у вас есть оно мудреное, не нужное и вообще в отдельных местах странное,

Для категории вам достаточно такого кода, и все будет работать, включая pre get posts

if ( have_posts() ):
	while ( have_posts() ) :
		the_post(); ?>

        <div class="category-result-post">
            <div class="category-news-post-img">
                <a href="<?php the_permalink(); ?>">
					<?php if ( get_the_post_thumbnail_url() ) {
						echo get_the_post_thumbnail();
					} else {
						echo ' <img alt="#" src="http://blogprogram.ru/wp-content/uploads/2016/05/noimg.png">';
					} ?>
                </a>
            </div>
            <div class="category-result-detail">
                <div class="category-result-detail-body">
                    <p class="date-news"><?php the_time( 'd.m.Y' ); ?></p>
                    <div class="category-result-post-bottom">
                        <p class="posts-p"><a href="<?php the_permalink(); ?>"
                                              class="news-post-text"><?php the_title(); ?></a></p>
                    </div>
                </div>
            </div>
        </div>

	<?php
	endwhile;
else:
	echo( '<p>Извините, нет записей.</p>' );
endif;

[Вадим Свердлик]

WP Panda, спасибо ,я уже как раз это только что попробовал,оно работает

[WP Panda]

Вадим Свердлик,
Вооот, наводящие вопросы они как раз для этого, что бы не дать решение, а вы сами к нему пришли)

[Вадим Свердлик]

WP Panda, понимаю, спасибо большое

[WP Panda]

Игорь, email

Answer 2

[Игорь]

Так сделайте это

Если закоментировать

if ( is_admin() || ! $query->is_main_query() ) {
return;
}

то тогда на старнице категорий выводится по 2 записи как и нужно

Создайте дополнительное условие для главной страницы

if ( $query->is_home() ) {
  $query->set( 'posts_per_page', 5 );
}

но тогда на главной странице выводит не то количество

Условие тут избыточно

if ( wp_is_mobile() ) {
  $query->set( 'posts_per_page', 2 );
} else {
  $query->set( 'posts_per_page', 2 );
}

и его можно заменить на
$query->set( 'posts_per_page', 2 );

Откройте раздел "редактирование шаблонов" или что-то подобное, Там можно сразу перемещаться на страницы документации по названиям функций.

Comments

[WP Panda]

is_admin()

Этот код проверяет, админ ли вы

Вам тоже не мешает почитать доки )))

[Игорь]

WP Panda, не спорю) я исправился)

[Вадим Свердлик]

Игорь, не понимаю,что мне даст редактор, у меня не работает почти вся функция

if ( $query->is_search() ) {
$query->set( 'posts_per_page', 8 );
}

if ( $query->is_category() ) {
if ( wp_is_mobile() ) {
$query->set( 'posts_per_page', 2 );
} else {
$query->set( 'posts_per_page', 2 );
}

вот это все не работает,но если комментирую if ( is_admin() || ! $query->is_main_query() ) {
return;
}
то работает,но тогда соответсвенно на главной странице выводится не то количество

[Игорь]

Вадим Свердлик, Редактор вам даст понимание какие стандартные функции и что делают. Это облегчит понимание

[Вадим Свердлик]

Игорь, спасибо за совет

[Игорь]

Вадим Свердлик, Я дополнил ответ, посмотрите