Вывод taxonomy через foreach?

Question

[AlexMark]

<?php $cur_terms = get_the_terms( $post->ID, 'metki' );
foreach( $cur_terms as $cur_term ){
	$term_link = get_term_link( (int)$cur_term->term_id, $cur_term->taxonomy ) ;
    
    // If there was an error, continue to the next term.
    if ( is_wp_error( $term_link ) ) {
        continue;
    }
	echo '<a href="'. esc_url( $term_link ) .'">'. $cur_term->name .'</a> ';
};?>

В общем, оно таксономии выводит как нужно, но если в поста нету тегов, то выбивает ошибку "Invalid argument supplied for foreach() in line 52 - вот она foreach( $cur_terms as $cur_term )"
Как исправить можно...

Answer 1

[Денис Янчевский]

Открыть документацию по функции get_the_terms(), найти фразу "Array of WP_Term objects on success, false if there are no terms or the post does not exist, WP_Error on failure." и добавить проверку:

<?php
$cur_terms = get_the_terms($post->ID, 'metki');
if (is_array($cur_terms) && !is_wp_error($cur_terms)) {
    foreach ($cur_terms as $cur_term) {
        $term_link = get_term_link((int) $cur_term->term_id, $cur_term->taxonomy);

        // If there was an error, continue to the next term.
        if (is_wp_error($term_link)) {
            continue;
        }
        echo '<a href="' . esc_url($term_link) . '">' . $cur_term->name . '</a> ';
    }
}
?>

Comments

[AlexMark]

извините, я пхп не знаю, поэтому я смотрел туда, куда вы говорите, но привести его к такому сам не мог.
Верно я понимаю, что после того, как мы объявили массив, мы проверяем, существует он и не_существуют ли ошибок вп. Если 2 позиции возвращают true, тогда уже идет мой цикл?

[Денис Янчевский]

AlexMark: так я же Вам привел готовый вариант кода. Его можно просто вставить, заменив свой :) Я его не тестировал, но он должен работать.

По поводу как это работает. Да, мы проверяем, что возвращенное функцией get_the_terms значение является массивом и не является экземпляром класса WP_Error. Если всё верно, то выполняем цикл. В случае, если не будет прикрепленных к записи терминов или сама запись не будет существовать, функция get_the_terms вернет значение false, наше условие не выполнится и цикл не запустится.

[AlexMark]

Денис Янчевский: я знаю, я сразу вставил и проверил, просто пояснил, почему я сам этого не сделал, хотя читал документацию)

[Денис Янчевский]

AlexMark: :) Сейчас ещё раз посмотрел на код, вероятно, я перестарался со втором условием "!is_wp_error($cur_terms)" и, думаю, можно обойтись без него. Если функция get_the_terms вернет объект класса WP_Error, то он все равно не пройдет проверку is_array($cur_terms), поэтому её должно быть достаточно :).

Answer 2

[Евгений Вольф]

Думаю как-то так:

if (count($cur_terms)) {
     foreach( $cur_terms as $cur_term )
}

вообще, нужно бы проверить, что в переменной $cur_terms в тот момент когда возникает ошибка, и добавить проверку.

Comments

[AlexMark]

я php не знаю от слова совсем.
но вижу алгоритм в голове, если (count($cur_terms)) равно нулю, то continue;, проскочить foreach. Как реализовать, не знаю. И не знаю, можно ли так...
print_r($cur_terms); это нужно куда-то добавить, чтобы увидеть в чем ошибка?

[Евгений Вольф]

>> print_r($cur_terms); это нужно куда-то добавить, чтобы увидеть в чем ошибка?
Да, это можно добавить сразу над foreach, тогда мы увидим не только ошибку, но и содержимое объекта, который в foreach попадает и вызывает ошибку соответственно.

[AlexMark]

Евгений Вольф: пробовал, оно ничего не выводит.