2

What I'm trying to achieve: get n terms from a custom taxonomy ordered by the newest post in each term. This will be used to display the newest m posts under each term.

Is there a better way of doing this than getting an array of all the terms, and looping through to find the newest post in each? This approach doesn't feel very efficient or scalable.

Update

Here's my working code so far - can this be improved on?

$taxonomy = 'industry'; //taxonomy name
$terms_to_display = 5; //number of terms to display
$posts_per_term = 4; // number of posts per term to display
$terms = get_terms( array('taxonomy' => $taxonomy, 'hide_empty' => true, 'fields' => 'ids' ) );
$news_items = [];

foreach ( $terms as $term ) {

    $args = array (
        'post_type'         => 'news',
        'posts_per_page'    => $posts_per_term,
        'orderby'           => 'date',
        'order'             => 'DESC',
        'tax_query' => array (
            array (
                'taxonomy' => $taxonomy,
                'field'    => 'id',
                'terms'    => array($term),
            ),
        ),
    );

    $news = new WP_Query( $args );

    if ( $news->post_count == $posts_per_term ) { // ignore terms with not enough posts
        $news_items[$term] = get_the_date("U", $news->posts[0]->ID); //get date of newest post in this term
    }

    wp_reset_query();

}

arsort ( $news_items ); //sort descending, keeping keys

$term_ids = array_keys ( array_slice($news_items, 0, $terms_to_display, true) ); //take 'n' newest and return an array of keys (term ids)

foreach ( $term_ids as $term_id ) {

    $term = get_term ( $term_id, $taxonomy );

    echo "<h2>" . $term->name . "</h2>";

    $args = array (
        'post_type'         => 'news',
        'posts_per_page'    => $posts_per_term,
        'orderby'           => 'date',
        'order'             => 'DESC',
        'tax_query'         => array (
                array (
                    'taxonomy' => $taxonomy,
                    'field'    => 'id',
                    'terms'    => array($term_id),
                ),
        ),
    );

    $news = new WP_Query( $args );

    if ( $news->have_posts() ) {    

        echo "<ul>";

        while ( $news->have_posts() ) {
            $news->the_post();

                echo "<li>" . get_the_title() . "</li>";

        }

        echo "</ul>";
    }

    wp_reset_query();

}
4
  • You have multiple typos in your code arra y, "<h2">. What editor/IDE are you using for coding? It should underline those things.
    – djboris
    Apr 10, 2019 at 12:24
  • @dboris Needless to say, this isn't my actual code - I copied/pasted and then "tidied" up making it suitable for WPSE. Apr 10, 2019 at 12:30
  • I understand. It was just a friendly comment in case this is your actual code.
    – djboris
    Apr 10, 2019 at 12:38
  • No probs - thanks for looking. Apr 10, 2019 at 12:45

1 Answer 1

1

Basically, what you are doing right now is this:

  1. Get all the terms from industry taxonomy
  2. For each term do a WP_Query to get the newest posts, iterate through them to sort the terms, creating a new array with term ids
  3. Sort and slice the new array
  4. For each element in new array of terms ids do again a WP_Query to get the newest posts, and iterate through them

That's a lot of WP_Queryies. Let's try to lower that number.

We can do it like this:

  1. Get all the terms from industry taxonomy
  2. For each term do a WP_Query to get the newest posts, iterate through them to sort the terms, creating a new array
  3. New array will be keyed with a date in U format, and contain both the term object and the array with post objects, so we won't have to query them again.

Here is my suggested code:

$taxonomy = 'industry'; //taxonomy name
$terms_to_display = 5; //number of terms to display
$posts_per_term = 4; // number of posts per term to display
$terms = get_terms( array('taxonomy' => $taxonomy, 'hide_empty' => true ) );
$news_items = [];

foreach ( $terms as $term ) {

    $args = array (
        'post_type'         => 'news',
        'posts_per_page'    => $posts_per_term,
        'orderby'           => 'date',
        'order'             => 'DESC',
        'tax_query' => array (
            array (
                'taxonomy' => $taxonomy,
                'field'    => 'id',
                'terms'    => array( $term->term_id ),
            ),
        ),
    );

    $news_query = new WP_Query( $args );

    // ignore terms with not enough posts
    if ( $news_query->post_count < $posts_per_term ) {
        continue;
    }

    $news_posts = $news_query->get_posts();

    // get date of newest post in this term
    $newest_post_date = get_the_date( "U", $news_posts[0]->ID );

    $news_items[$newest_post_date] = array(
        'term' => $term,
        'news' => $news_posts,
    );
}

krsort( $news_items ); // sort descending by keys

$news_items = array_slice( $news_items, 0, $terms_to_display );

wp_reset_query();

I have made a few more changes as well:
1. $terms is now array of the WP_Term objects, not just ids
2. Made the $news_query->post_count < $posts_per_term check slightly more readable

And how to use:

foreach ( $news_items as $item ) {

    echo "<h2>" . $item['term']->name . "</h2>";

    echo "<ul>";

    foreach ( $item['news'] as $news_post ) {
        setup_postdata( $news_post );

        echo "<li>" . get_the_title() . "</li>";
    }

    echo "</ul>";
}

wp_reset_query();

This way, we have lowered the number of WP_Queryies to the number of non-empty terms in the industry taxonomy, and made the resulting array much more useful.

4
  • Thanks - I thought I'd benchmark this against my original query and surprisingly the page takes slightly longer to load and uses more database queries (according to the Query Monitor plugin). Apr 16, 2019 at 12:16
  • @mistertaylor Thanks for the feedback. That surprises me as well! Your solution uses this formula 1[get_terms] + count($terms)[WP_Query] + ($terms_to_display * 2[get_term][WP_Query]). Mine uses this fomula 1[get_terms] + count($terms)[WP_Query]. So, in theory, at least 10 queries less than yours. I wonder where do the extra queries come from. Can you post more information on the queries?
    – djboris
    Apr 16, 2019 at 13:19
  • Using the original script Query Monitor reports a page generation time of 2.534s, a database query time of 0.1434s, 263 DB queries and 6 duplicate queries. Using your script, it reports a page generation time of 3.3698s, a db query time of 0.9739s, 277 db queries and 13 duplicate queries. Apr 25, 2019 at 15:02
  • Each post uses get_the_image() and with a fallback to display the image attached to the first selected term using wp_get_attachment_image_src, or a default placeholder image if not. Everything else is just done with standard "in-the-loop" functions like the_permalink or the_title. Apr 25, 2019 at 15:08

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.