Changeset 13186 for trunk/src/bp-core/classes/class-bp-admin.php

Timestamp:

12/12/2021 03:44:31 PM (3 years ago)

Author:

imath

Message:

Make sure a situation is attached to a BP Email before displaying it

Props dhavalkasvala

Fixes #8165

File:

• trunk/src/bp-core/classes/class-bp-admin.php (modified) (1 diff)

trunk/src/bp-core/classes/class-bp-admin.php

@@ -1136 +1136 @@
 
         // Grab email situations for the current post.
-        $situations = wp_list_pluck( get_the_terms( $post_id, bp_get_email_tax_type() ), 'description' );
-
-        // Output each situation as a list item.
-        echo '<ul><li>';
-        echo implode( '</li><li>', $situations );
-        echo '</li></ul>';
+        $terms           = get_the_terms( $post_id, bp_get_email_tax_type() );
+        $taxonomy_object = get_taxonomy( bp_get_email_tax_type() );
+
+        if ( is_wp_error( $terms ) || ! $terms  ) {
+            printf( '<span aria-hidden="true">&#8212;</span><span class="screen-reader-text">%s</span>', $taxonomy_object->labels->no_terms );
+        } else {
+            $situations = wp_list_pluck( $terms, 'description' );
+
+            // Output each situation as a list item.
+            echo '<ul><li>';
+            echo implode( '</li><li>', $situations );
+            echo '</li></ul>';
+        }
     }