Changeset 10704

Timestamp:

02/18/2021 05:54:14 AM (5 years ago)

Author:

dd32

Message:

Support Forums: User Badges: Switch out wp_get_post_terms() for get_the_terms() which is an older preferred variant of the function with caching.

Removes a query per user per thread.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-bbp-user-badges/inc/class-plugin.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-bbp-user-badges/inc/class-plugin.php

@@ -131 +131 @@
 
         foreach ( $types as $type ) {
-            $slugs = wp_get_post_terms( $topic_id, 'topic-' . $type, array( 'fields' => 'slugs' ) );
-            if ( $slugs ) {
+            $terms = get_the_terms( $topic_id, 'topic-' . $type );
+            if ( $terms ) {
                 break;
             }
         }
 
-        if ( ! $slugs || is_wp_error( $slugs ) ) {
+        if ( ! $terms || is_wp_error( $terms ) ) {
             return false;
         }
@@ -143 +143 @@
         return array(
             'type'          => $type,
-            'slug'          => $slugs[0],
+            'slug'          => $terms[0]->slug,
             'user_nicename' => $user->user_nicename,
         );