Как сделать универсальную функцию для вывода записей?

Question

[Расул Гитинов]

Есть функция, которая выводит все записи из данного типа:

$args = array(
	'post_type'      => 'works',
	'numberposts'    => -1
);
$lastposts = get_posts( $args );
foreach( $lastposts as $post ){ setup_postdata($post);
	get_template_part( 'works-item' );
}
wp_reset_postdata();

Как доработать эту функцию так, чтобы на странице категории автоматически выводились записи из этой категории?

Comments

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

1. Не используйте -1 в numberposts
2. Используйте дополнительные параметры для управления кешированием и подсчетом количества найденных постов (это все для оптимизации запроса)

Подробнее по п.1 и 2 смотрите тут.

3. По самому вопросу use case не очень понятен. Можете пример более детально озвучить?

[Расул Гитинов]

Игорь Воротнёв, Например есть страница, где выводятся все работы (портфолио) и на этой странице есть список категорий, по которым можно пройти (брендинг, сайты и т.д) и если с этой страницы перейти в одну из этих категорий, то исходя из моего кода там также будут выводиться все записи, в независимости от выбранной категории

[Max Medar]

Игорь Воротнёв, какой годный коммент и полезнейшая ссылка! Пошел переписывать код в некоторых проектах.

Спасибо!

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

Max Medar, На здоровье :) Это библия для WP-разработчиков. Еще стоит потратить время на курение https://vip.wordpress.com/documentation/

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

Расул Гитинов, все равно ничего не понял)

[Расул Гитинов]

Игорь Воротнёв, Есть страница с портфолио. Щас для теста я туда закинул 3 категории:

• Брендинг (3 записи)
  
• Mobile (3 записи)
  
• Сайты (5 записей)
  

В общем получается 11 записей.

Захожу на страницу "Портфолио" и вижу все записи (11 штук) и например я хочу посмотреть записи именно из категории "Mobile" например, в которой 3 записи. Вот я и хотел сделать так, чтобы эта же функция мне выдавала 3 записи, когда я перехожу на страницу категории "Mobile", но при этом 11, если я на странице всех записей (портфолио).

Просто я не хотел делить страницу архива записей и страницу таксономии на два отдельных файла, потому что у них одинаковый дизайн. Поэтому хотел сделать функцию, которая будет сама определять сколько и каких записей выдавать.

Ниже Max Medar дал код, который почти делает то, что нужно. Он выводит нужные записи если я перехожу в категории, но на общей странице (портфолио) почему-то перестают выводиться записи вообще.

Вот скрины:

Общая страница
Страница категории

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

Расул Гитинов,

Вот скрины:

Привет из Украины :)

[Расул Гитинов]

Игорь Воротнёв, Упс)
Первый

Второй

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

Расул Гитинов, вот, теперь понял) посмотрел код Max Medar и сразу увидел проблему. В его коде предполагается, что термин какой-то есть всегда, то есть не отработан вариант когда его нет - get_queried_object() в этом случае возвращает не термин, а post type. Его значение подставляется в tax_query, и по нему ничего не находит (поскольку значение нерелевантное). Пофиксить это очень просто:

<?php
// Получаем текущий запрошенный объект
// @see https://codex.wordpress.org/Function_Reference/get_queried_object
$term = get_queried_object();

// Устанавливаем по умолчанию неактивный tax_query
$tax_query = false;

// Если это термин таксономии, формируем tax_query
if ( $term instanceof WP_Term ) {
    $tax_query = array(
		array(
			'taxonomy' => 'category', // или вашу таксономию
			'field' => 'slug',
			'terms' => $term->slug,
		),
	);
}

// Запускаем запрос, tax_query будет выполнен только если он реально установлен (то есть, запрошен термин таксономии)
$_posts  = new WP_Query( array(
	'post_type' => 'works',
	'posts_per_page' => 10,
	'tax_query' => $tax_query,
) );

// А тут уже цикл вывода 
...

[Расул Гитинов]

Игорь Воротнёв, Я скорее всего ошибся где-то, вот такой код в итоге получил:

$term = get_queried_object();
$tax_query = false;
if ( $term instanceof WP_Term ) {
	$tax_query = array(
		array(
			'taxonomy' => 'works-cat',
			'field' => 'slug',
			'terms' => $term->slug,
		),
	);
}
$_posts  = new WP_Query( array(
	'post_type' => 'works',
	'posts_per_page' => 50,
	'tax_query' => $tax_query,
) );
foreach ($_posts as $post) : setup_postdata($post);
	get_template_part( 'works-item' );
endforeach;
wp_reset_postdata();

выводится очень много постов, даже больше чем есть в базе

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

Расул Гитинов, WP_Query возвращает объект WP_Query, значит цикл должен быть стандартный while ( have_posts() ). Это get_posts() возвращает массив постов, там foreach нужен.

Замените цикл на:

while ( $_posts->have_posts() ) : $_posts->the_post();
	get_template_part( 'works-item' );
endwhile;
wp_reset_postdata();

[Расул Гитинов]

Игорь Воротнёв, Спасибо большое! Выручили)

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

Расул Гитинов, на здоровье) закинул результирующий код в виде ответа, отмечайте. И ответ Max Medar тоже можете, он основу заложил.

[Расул Гитинов]

Игорь Воротнёв, Ага)

Answer 1

[Max Medar]

Примерно так (писал по памяти, так что возможны ошибки):

$term      = get_queried_object();
$term_slug = $term->slug;
$_posts    = new WP_Query( array(
	'post_type'      => 'works',
	'posts_per_page' => 10,
	'tax_query'      => array(
		array(
			'taxonomy' => 'category', // или вашу таксономию
			'field'    => 'slug',
			'terms'    => $term_slug,
		),
	),
) );
if ( $_posts->have_posts() ) :
	while ( $_posts->have_posts() ) :
		$_posts->the_post();
                get_template_part( 'works-item' );
	endwhile;
endif;
wp_reset_postdata();

Comments

[Расул Гитинов]

А как это сделать не через WP_Query, а через get_posts?

[Max Medar]

Расул Гитинов, как-то так:

$term      = get_queried_object();
$term_slug = $term->slug;
$_posts    = get_posts( array(
	'post_type'      => 'works',
	'posts_per_page' => 10,
	'tax_query'      => array(
		array(
			'taxonomy' => 'category', // или вашу таксономию
			'field'    => 'slug',
			'terms'    => $term_slug,
		),
	),
) );
foreach ($_posts as $post) : setup_postdata($post);
        get_template_part( 'works-item' );
endforeach;
wp_reset_postdata();

[Расул Гитинов]

Max Medar, Заработало. Но только теперь выводятся записи в категориях, а в общей странице где должны выводиться все записи ничего не выводится

[Max Medar]

Расул Гитинов, конечно ничего не выведется, если вы этот код вставили и для своей общей страницы.

[Расул Гитинов]

Код лежит в файле "archive-works.php" и вот такой код для вывода там:

$term      = get_queried_object();
$term_slug = $term->slug;
$_posts    = get_posts( array(
	'post_type'      => 'works',
	'posts_per_page' => 100,
	'tax_query'      => array(
		array(
			'taxonomy' => 'works-cat',
			'field'    => 'slug',
			'terms'    => $term_slug,
		),
	),
));
foreach ($_posts as $post) : setup_postdata($post);
	get_template_part( 'works-item' );
endforeach;
wp_reset_postdata();

[Max Medar]

Расул Гитинов, а какой файл подгружается для вывода общей страницы, где должны выводиться все записи?

[Расул Гитинов]

Max Medar, Он и есть (archive-works.php), а записи из таксономий в этой странице taxonomy-works-cat.php, но файл я оставил пустым и просто подключил к нему код из archive-works.php с помощью get_template, чтобы не дублировать код

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

Расул Гитинов, Max Medar, ребята, не используйте get_posts. Внутри он все равно запускает WP_Query, только с отключенными фильтрами и некоторыми другими defaults. Эта функция полезная, но в редких случаях, в основном стоит юзать WP_Query с четкими параметрами под каждую конкретную задачу.

Answer 2

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

По результатам обсуждения в комментах и корректировки кода от Max Medar :

$tax_query = false;

$term = get_queried_object();
if ( $term instanceof WP_Term ) {
    $tax_query = array(
        array(
            'taxonomy' => 'works-cat',
            'field' => 'slug',
            'terms' => $term->slug,
        ),
    );
}

$_posts  = new WP_Query( array(
    'post_type' => 'works',
    'posts_per_page' => 50,
    'tax_query' => $tax_query,
) );

while ( $_posts->have_posts() ) : $_posts->the_post();
	get_template_part( 'works-item' );
endwhile;
wp_reset_postdata();

Comments

[Max Medar]

instanceof

Ох, вот что бывает, когда начинаешь делать под wp без нормальных знаний основного языка программирования ( в нашем случае - php ). Спасибо уже второй раз!

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

Max Medar, на здоровье) а матчасть (php) таки надо учить)

Answer 3

[Сергей Герасимов]

По умолчанию в функцию передаётся 'category' => 0, следовательно добавьте в вашу функцию необязательный параметр $category = 0, и установите значение в ваш массив аргументов:
$args['category'] = $category;

Comments

[Расул Гитинов]

Не вышло почему-то

[Сергей Герасимов]

Расул Гитинов, а id категории получили?

$category = get_queried_object();
$args = array(
	'post_type'      => 'works',
	'numberposts'    => -1, 
        'category' => $category->term_id, 
);
$lastposts = get_posts( $args );
foreach( $lastposts as $post ){ setup_postdata($post);
	get_template_part( 'works-item' );
}
wp_reset_postdata();

[Расул Гитинов]

Сергей Герасимов, Вставил код. Выводятся все записи на общей странице, а если перейти на страницу категории, то записи вообще не выводятся. По моему она только с wp_query работает, а у меня вывод через get_posts

[Сергей Герасимов]

Расул Гитинов, а вы уверены что вы именно в том файле указываете?
Возможно в категориях выводить другой шаблон...
Ну или проверьте что возвращает функция get_queried_object()

[Расул Гитинов]

Сергей Герасимов, Да, у меня и для категорий тот же файл вывода, поэтому одна функция. Думал чтоб она автоматически определяла где как выводить)