Правильный ли код определения новых товаров?

Question

[Snowtime]

Ребят, прошу совета.
Мне нужно сделать, чтобы на 5 последних новинках во всех категориях был соответствующий лэйбл. Правильно ли я сделал, что запихнул код определения новых товаров в functions.php таким кодом:

$result = array (); 
$args = array ( 
'post_type' => 'product', 
'posts_per_page' => '5', 
'orderby' => 'date', 
'order' => 'DESC' 
); 
$custom_query = new WP_Query( $args ); 
$temp_query = $wp_query; 
$wp_query = NULL; 
$wp_query = $custom_query; 
if ( $custom_query->have_posts() ) { 
while ( $custom_query->have_posts() ) { 
$custom_query->the_post(); ?> 
<?php $result[] = get_the_ID(); ?> 
<?php } ?> 
<?php } 
wp_reset_postdata(); 
$wp_query = NULL; 
$wp_query = $temp_query; 
wp_reset_query();
и после этого в content-product.php сделал проверку, вызвав глобальную переменную:
<?php global $result; ?> 
<?php if(array_search(get_the_ID(), $result) !== false) { 
echo "Новинка"; 
} ?>
Если нет, то скажите, как правильно.

Comments

[Антон Дробышев]

Правильно ли я делаю? А код работает? Если нет, то неправильно. В этот файл надо хуки пихать.

[Snowtime]

Если оборачивать запрос в функцию, то при создании каждой карточки товара, будет производиться обращение к БД. Я ведь произвожу проверку в цикле. А мой вариант, с global, производить только одно обращение к БД. Потому я и не оборачивал в функцию

Answer 1

[Вова Дружаев]

php открываю, php закрываю... Зачем?
И почему не обернуто в функцию.
В целом - все не верно. Это делается просто через css. Выбираешь 5 первых товаров и вешаешь стиль

Comments

[Snowtime]

Код работает. Я просто хотел узнать правильность написания.

[Вова Дружаев]

Код работает и выполняет лишнюю работу - читай нагрузку. Все это можно сделать средствами стилей