Changeset 5837

Timestamp:

07/15/2015 05:29:22 PM (10 years ago)

Author:

johnjamesjacoby

Message:

Terms: Update functions for getting terms & term names to be more flexible & reusable by other functions. See #459.

Location:

trunk/src/includes

Files:

• forums/functions.php (modified) (5 diffs)
• topics/functions.php (modified) (1 diff)

trunk/src/includes/forums/functions.php

@@ -129 +129 @@
     $forum_parent_id = $forum_author = 0;
     $forum_title = $forum_content = '';
+    $terms = array( bbp_get_forum_mod_tax_id() => array() );
 
     /** Forum Author **********************************************************/
@@ -237 +238 @@
     if ( ! bbp_check_for_moderation( $anonymous_data, $forum_author, $forum_title, $forum_content ) ) {
         $post_status = bbp_get_pending_status_id();
+    }
+
+    /** Forum Mods ************************************************************/
+
+    if ( bbp_allow_forum_mods() && ! empty( $_POST['bbp_forum_mods'] ) ) {
+
+        // Escape tag input
+        $terms = sanitize_text_field( $_POST['bbp_forum_mods'] );
+
+        // Explode by comma
+        if ( strstr( $terms, ',' ) ) {
+            $terms = explode( ',', $terms );
+        }
+
+        // Add forum_mod ID as main key
+        $terms = array( bbp_get_forum_mod_tax_id() => $terms );
     }
 
@@ -259 +276 @@
         'post_status'    => $post_status,
         'post_type'      => bbp_get_forum_post_type(),
+        'tax_input'      => $terms,
         'comment_status' => 'closed'
     ) );
@@ -495 +513 @@
     }
 
+    /** Forum Mods ************************************************************/
+
+    // Either replace terms
+    if ( bbp_allow_forum_mods() && current_user_can( 'assign_forum_mods' ) && ! empty( $_POST['bbp_forum_mods'] ) ) {
+
+        // Escape tag input
+        $terms = sanitize_text_field( $_POST['bbp_forum_mods'] );
+
+        // Explode by comma
+        if ( strstr( $terms, ',' ) ) {
+            $terms = explode( ',', $terms );
+        }
+
+        // Add forum mod ID as main key
+        $terms = array( bbp_get_forum_mod_tax_id() => $terms );
+
+    // ...or remove them.
+    } elseif ( isset( $_POST['bbp_forum_mods'] ) ) {
+        $terms = array( bbp_get_forum_mod_tax_id() => array() );
+
+    // Existing terms
+    } else {
+        $terms = array( bbp_get_forum_mod_tax_id() => explode( ',', bbp_get_forum_mod_names( $forum_id, ',' ) ) );
+    }
+
     /** Additional Actions (Before Save) **************************************/
 
@@ -1661 +1704 @@
 }
 
+/** Forum Mods ****************************************************************/
+
+/**
+ * Get forum mods for a specific forum ID
+ *
+ * @since bbPress (r5836)
+ *
+ * @param int $forum_id
+ *
+ * @return string
+ */
+function bbp_get_forum_mods( $forum_id = 0 ) {
+    $forum_id   = bbp_get_forum_id( $forum_id );
+    $terms      = (array) get_the_terms( $forum_id, bbp_get_forum_mod_tax_id() );
+    $forum_mods = array_filter( $terms );
+
+    return apply_filters( 'bbp_get_forum_mods', $forum_mods, $forum_id );
+}
+
+/**
+ * Get forum mods for a specific forum ID
+ *
+ * @since bbPress (r4165)
+ *
+ * @param int    $forum_id
+ * @param string $sep
+ *
+ * @return string
+ */
+function bbp_get_forum_mod_names( $forum_id = 0, $sep = ', ' ) {
+    $forum_mods = bbp_get_topic_tags( $forum_id );
+    $pluck      = wp_list_pluck( $forum_mods, 'name' );
+    $terms      = ! empty( $pluck ) ? implode( $sep, $pluck ) : '';
+
+    return apply_filters( 'bbp_get_forum_mod_names', $terms, $forum_id, $sep );
+}
+
 /** Helpers *******************************************************************/
 

trunk/src/includes/topics/functions.php

@@ -3682 +3682 @@
  * Get topic tags for a specific topic ID
  *
+ * @since bbPress (r5836)
+ *
+ * @param int $topic_id
+ *
+ * @return string
+ */
+function bbp_get_topic_tags( $topic_id = 0 ) {
+    $topic_id   = bbp_get_topic_id( $topic_id );
+    $terms      = (array) get_the_terms( $topic_id, bbp_get_topic_tag_tax_id() );
+    $topic_tags = array_filter( $terms );
+
+    return apply_filters( 'bbp_get_topic_tags', $topic_tags, $topic_id );
+}
+
+/**
+ * Get topic tags for a specific topic ID
+ *
  * @since bbPress (r4165)
  *
- * @param int $topic_id
+ * @param int    $topic_id
  * @param string $sep
+ *
  * @return string
  */
 function bbp_get_topic_tag_names( $topic_id = 0, $sep = ', ' ) {
-    $topic_id   = bbp_get_topic_id( $topic_id );
-    $topic_tags = array_filter( (array) get_the_terms( $topic_id, bbp_get_topic_tag_tax_id() ) );
-    $terms      = array();
-    foreach ( $topic_tags as $term ) {
-        $terms[] = $term->name;
-    }
-    $terms = ! empty( $terms ) ? implode( $sep, $terms ) : '';
-
-    return apply_filters( 'bbp_get_topic_tags', $terms, $topic_id );
+    $topic_tags = bbp_get_topic_tags( $topic_id );
+    $pluck      = wp_list_pluck( $topic_tags, 'name' );
+    $terms      = ! empty( $pluck ) ? implode( $sep, $pluck ) : '';
+
+    return apply_filters( 'bbp_get_topic_tag_names', $terms, $topic_id, $sep );
 }