Hi! Okay well I don’t know how your code looked before that was just to show you how to retrieve the name rather than just the id 🙂
Your code seems a bit excessive and unsemantic. Is there any reason it couldn’t look like this?
<?php if (get_field('sub_specialty')) { //Sub speciality ?>
<strong>Sub specialty.:</strong>
<ul>
<?php $terms = get_field('sub_specialty'); ?>
<?php foreach ($terms as $term){ ?>
<li><a href="<?php echo $term->slug; ?>" name="<?php $term->slug; ?>" alt="<?php $term->name; ?>"><?php echo $term->name; ?></a></li>
<?php } ?>
</ul>
<?php } ?>
There’s no unnecessary functions running, the else-statement was removed, the if statement is changed to be simpler, name was corrected and the html is semantically correct since it’s now an unordered list instead of links with an inline break. 🙂
Happy coding
