Как заменить конструкцию query_posts на get_posts?

Question

[ligisayan]

Всем привет! На сайте **wordpress** есть конструкция с `query_posts`

query_posts(
                   array( 'post_type' => APP_POST_TYPE, 
                             'meta_query' => array( 
                                                          array( 'key' => 'cp_category', 
                                                                    'value' => 'Не определено', 
                                                                    'compare' => '='
                                                                   )
                                                               ), '
                            ignore_sticky_posts' => 1
                          )
                       );
    get_template_part( 'loop', 'ad_listing' ); 
    wp_reset_query();

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

$categories = 
    get_posts(
                   array( 'post_type' => APP_POST_TYPE, 
                             'meta_query' => array( 
                                                          array( 'key' => 'cp_category', 
                                                                    'value' => 'Не определено', 
                                                                    'compare' => '='
                                                                   )
                                                               ), '
                            ignore_sticky_posts' => 1
                          )
                       );
    foreach( $categories as $category ) {
    get_template_part( 'loop', 'ad_listing' ); 
    }
    wp_reset_postdata();
?>

Как исправить и решить проблему с пагинацией?

Answer 1

[ligisayan]

Пагинация заработает при выполнении такого условия:

For getting the current pagination number on a static front page (Page template) you have to use the 'page' query variable: Ссылка на источник

if ( is_front_page() ) {
        $paged = (get_query_var('page')) ? get_query_var('page') : 1;   
    } else {
        $paged = (get_query_var('paged')) ? get_query_var('paged') : 1; 
    }

Answer 2

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

Во-первых, форматируйте код адекватно, эти длинные строки читать - адски неудобно (особенно отлавливать ваши двойные $$, лишние скобки и тд):

// Параметры удобнее задавать отдельно
$args = array(
    'post_type' => APP_POST_TYPE,
    'ignore_sticky_posts' => true,
    'meta_query' => array( 
        array(
            'key' => 'cp_category', 
            'value' => 'Не определено', 
            'compare' => '=',
        ),
    ), 
);
// Получаем массив постов.
$categories = get_posts( $args );

// Перебираем массив, обязательно as $post, чтобы корректно работала функция setup_postdata()
foreach( $categories as $post ) {
    // Устанавливаем текущую запись в глобальный scope
    setup_postdata( $post );
    // Подключаем шаблон, который уже сможет видеть корректные данные из глобального scope
    get_template_part( 'loop', 'ad_listing' ); 
}
// Очищаем глобальный scope
wp_reset_postdata();

Не тестил, но должно работать.

Comments

[ligisayan]

Игорь Воротнёв спасибо за помощь! Странно, но пока не сработало..
Вывело 5 пустых листингов с категорией "Не определено" (хотя их тысячи и заполненные инфой) А с query_posts выводит нормально
Отформатировал вопрос как вы просили для удобочитаемости

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

ligisayan, query_posts заменяет основной запрос (main query), поэтому стандартный основной цикл работает. Если сравнить ваши 2 варианта, то во втором есть foreach, в первом нету. Это наталкивает на мысль, что в самом template part у вас код с циклом WordPress Loop, в такой конфигурации работать не будет. Покажите код этого файла.

[ligisayan]

Игорь Воротнёв,

global $cp_options;

<div class="post-block-out <?php cp_display_style( 'featured' ); ?>">

	<div class="post-block">

		<div class="post-left">

			<?php if ( $cp_options->ad_images ) cp_ad_loop_thumbnail(); ?>

		</div>

		<div class="<?php cp_display_style( array( 'ad_images', 'ad_class' ) ); ?>">

			<?php appthemes_before_post_title(); ?>

			<h3><a href="<?php the_permalink(); ?>"><?php if ( mb_strlen( get_the_title() ) >= 75 ) echo mb_substr( get_the_title(), 0, 75 ) . '...'; else the_title(); ?></a></h3>

			<div class="clr"></div>

			<?php appthemes_after_post_title(); ?>

			<div class="clr"></div>

			<?php appthemes_before_post_content(); ?>

			<p class="post-desc"><?php echo cp_get_content_preview(); ?></p>

			<?php appthemes_after_post_content(); ?>

			<div class="clr"></div>

		</div>

		<div class="clr"></div>

	</div><!-- /post-block -->

</div><!-- /post-block-out -->

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

ligisayan, что это? o_O

[ligisayan]

Игорь Воротнёв, "Покажите код этого файла."

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

ligisayan, ну, вы же понимаете что в этом коде нет ни одной нативной функции для вывода контент в цикле кроме the_title(), поэтому без кода вот этих самых ваших cp_get_content_preview() сложно о чем-либо судить.

Попробуйте вместо этого темплейта подключить другой, с вот таким кодом:

<article id="post-<?php the_ID(); ?>">
	<header class="entry-header">
		<?php the_title( '<h1 class="entry-title">', '</h1>' ); ?>
	</header>

	<div class="entry-content">
		<?php the_content(); ?>
	</div>
</article>

И покажите HTML, который будет выведен в результате.

[ligisayan]

Игорь Воротнёв, код такой

function cp_get_content_preview() {
	return apply_filters( 'cp_get_content_preview', get_the_excerpt(), 55 );
}

Ту немного другое - есть функция темы вывода объявлений

function cp_output_random_ads_listing() {
	global $wp_query;

	remove_action( 'appthemes_after_endwhile', 'cp_do_pagination' );

	$post_type_url = add_query_arg( array( 'paged' => 2 ), get_post_type_archive_link( APP_POST_TYPE ) );

	// show all random ads but make sure the sticky featured ads don't show up first
	$paged = ( get_query_var( 'paged' ) ) ? get_query_var( 'paged' ) : 1;

	query_posts( array( 'post_type' => APP_POST_TYPE, 'ignore_sticky_posts' => 1, 'post_status' => 'publish', 'paged' => $paged, 'orderby' => 'rand' ) );

	$total_pages = max( 1, absint( $wp_query->max_num_pages ) );

	get_template_part( 'loop', 'ad_listing' );

	if ( $total_pages > 1 ) {
		$random_url = add_query_arg( array( 'sort' => 'random' ), $post_type_url );

		cp_the_view_more_ads_link( $random_url, 'random' );
	}
}

Я заменил на свою и она практически работает, вернее работает пагинация на всех внутренних страницах, кроме главной (в адресной строка показывает переход вида /page/2, но при этом, остается на той же странице - не понимаю почему, ведь если ее назначить не главной, то все отрабатывает).

$paged = ( get_query_var( 'paged' ) ) ? get_query_var( 'paged' ) : 1;
query_posts(
                      array( 'post_type' => APP_POST_TYPE, 
                                'meta_query' => array (
                                 array( 'key' => 'cp_category', 
                                        'value' => 'Не определено', 
                                        'compare' => '=', 
                                        'paged' => $paged 
                                         )
                                 ), 
                                'ignore_sticky_posts' => 1 
                   );
get_template_part( 'loop', 'ad_listing' );
wp_reset_query();

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

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

ligisayan, ууу какая жесть)) давно я такого говнокода не встречал)) Это не к вам относится, а к тому кто эту тему запилил.

Дебажить такой говнокод по воздуху, без xdebug – неа, сорян. Времени нет. Да и Новый Год на носу. пойду лучше оливье накрошу)

А если серьезно – используйте xdebug и потратьте полчасика чтобы пошагово пройтись, step in / step out. Увидите в каком месте слетают глобальные данные и ломается пагинация.

[ligisayan]

Игорь Воротнёв, ок, и на этом спасибо. С наступающим!