Шорткод для пользовательских категорий Вордпресс?

Question

[Spartak]

Доброго времени суток.
Есть пользовательские типы записей (services) + таксономии (рубрики) к ним (catalog)
Сделал шорткод чтобы можно было выводить пользовательские рубрики с изображениями на любой странице.
Изображения к пользовательским рубрикам прикрепил с помощью плагина Advanced Custom Fields

Код работает, но не уверен что все правильно сделано, (я не профессионал) - посмотрите на предмет ошибок, все ли правильно, корректно ?
Так же вопрос, нужен ли здесь сброс цикла или т.п.?
Спасибо.

// Add Shortcode
function service_cat_shortcode() {
	// Get the taxonomy's terms
	$terms = get_terms(
		array(
			'taxonomy'   => 'catalog',
			'hide_empty' => false,
		)
	);

	if ( ! empty( $terms ) && is_array( $terms ) ) {  ?>
		<div class="uk-grid-match  uk-child-width-1-3@m uk-flex-center" uk-grid>

			<?php foreach ( $terms as $term ) : ?>

				<div class="uk-card uk-card-default uk-card-body uk-card-hover">

					<a  href="<?php echo esc_url( get_term_link( $term ) ) ?>">

						<!--  вывод изображения категории (добавленные через поле ACF)  -->
						<?php if($imgcat=get_field("imgcat",$term)) { ?>
							<img src="<?php echo $imgcat;?>"/>
						<?php  } ?>		
					</a>

					<div class="uk-card-footer">
						<h3 class="uk-card-title ">
							<!-- заголовок пользовательской рубрики -->
							<a  href="<?php echo esc_url( get_term_link( $term ) ) ?> ">
								<?php echo $term->name; ?> 
							</a>
						</h3>
					</div>
				</div>		
			<?php  endforeach;  ?>
		</div>
		<?php
		//wp_reset_postdata();
	}
} 
add_shortcode( 'services_cat', 'service_cat_shortcode' );

Answer 1

[WP Panda]

как минимум не верно - то, что результат выводится на печать а не возвращается.

Comments

[Spartak]

так будет верно?
....
ob_start();

// тут код

return ob_get_clean();
}
add_shortcode( 'services_cat', 'service_cat_shortcode' );

[WP Panda]

Можно и так.

[Spartak]

Спасибо, что уделили время.