Как избавиться от пустого цикла без которого ничего не работает?

Question

[godsplane]

Вывожу посты по меткам.

<div class="portfolio-wrapper row j-center container">
  <?php 
$tags = get_the_tags();
foreach ( $tags as $tag ){
  
}
   $lastposts = get_posts( array(
	'numberposts' => -1,
	'tag' => $tag->slug,
	'category'    => 0,
	'orderby'     => 'menu_order',
	'order'       => 'DESC',
	'include'     => array(),
	'exclude'     => array(),
	'meta_key'    => '',
	'meta_value'  =>'',
	'post_type'   => 'portfolio',
	'suppress_filters' => true, // подавление работы фильтров изменения SQL запроса
) );
   foreach( $lastposts as $post ){ setup_postdata($post);
     ?>

  <div class="portfolio-item">
    <h2>
      <? the_title() ?>
    </h2>
    <a class="fancybox" rel="group" href="<?php the_post_thumbnail_url('large')?>">
      <img src="<?php the_post_thumbnail_url('medium')?>" alt="Фотография">
    </a>


  </div>

  <?php 
     
   }
   wp_reset_postdata()
   ?>
</div>
</div>

Без этого ничего не работает

$tags = get_the_tags();
foreach ( $tags as $tag ){
  
}

Но это же пустой цикл, как можно избавится от этой конструкции и почему это работает так?

Answer 1

[Rickkk]

Цикт имеет значение, т.к. заполняет переменную $tag последним элементом из массива $tags. А затем в последующем коде данная переменная используется вот так - 'tag' => $tag->slug,
Попробуйте заменить цикл на выражение
$tag = array_pop($tags);

Comments

[godsplane]

Сработало, спасибо!

[godsplane]

А можно как то избавиться от ошибки

array_pop() expects parameter 1 to be array, boolean given in

Когда выводится на общей странице без метки или придется добавлять все метки к этой странице?
Сами записи и так выводятся все, но это ошибка вылазит все равно.

[godsplane]

Сейчас я просто скрыл это через условие

if(is_singular( 'portfolio' ) ) {
	$tag = array_pop($tags);

}

Но я не уверен, что это верный вариант

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

godsplane, это нормальный вариант, но вам и $tag->slug в get_posts() убирать надо в идеале.

[godsplane]

Игорь Воротнёв, почему?А как иначе?

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

godsplane, ну, потому что параметр tag у WP_Query / get_posts должен быть string, а у вас на этих страницах переменной $tags, и тем более свойства $tag->slug нету. Я бы вот так делал:

// Default args.
$args = [
    'post_type'   => 'portfolio',
    'numberposts' => 100,
    'orderby'     => 'menu_order',
    'order'       => 'DESC',
];

// Tag should be handled on single entry only. 
if ( is_singular( 'portfolio' ) ) {
    $tags = get_the_tags( get_the_ID() );
    // Make sure tag exists.
    if ($tags) {
        // Pop the last tag.
        $tag = array_pop( $tags );
        // Set 'tag' property of WP_Query.
        $args['tag'] = $tag->slug;   
    }
}

$lastposts = get_posts( $args );

В этом случае вы устанавливаете 'tag' в запрос только тогда, когда $tag точно содержит объект WP_Term. В остальных случаях не будет ошибок и WP_Query не будет выполнять лишних телодвижений.

Кроме этого, у вас в параметрах запроса куча лишнего:
- suppress_filters и так по умолчанию установлен в true,
- никогда не используйте в numberposts значение -1, используйте достаточно большое число (например - 100, 500, 1000 - зависит от конкретного случая),
- category, include, exclude, meta_key, meta_value тут лишние

[godsplane]

Игорь Воротнёв, Хорошо, спасибо, приму к сведению