Changeset 5738

Timestamp:

05/13/2015 12:35:20 AM (10 years ago)

Author:

johnjamesjacoby

Message:

Topics: Improve type-casting in topic functions.

Props thebrandonallen, netweb. See #2801.

File:

• trunk/src/includes/topics/functions.php (modified) (9 diffs)

trunk/src/includes/topics/functions.php

@@ -2041 +2041 @@
     $stickies = ( empty( $stickies ) || ! is_array( $stickies ) ) ? array() : $stickies;
 
-    return apply_filters( 'bbp_get_stickies', $stickies, (int) $forum_id );
+    return apply_filters( 'bbp_get_stickies', $stickies, $forum_id );
 }
 
@@ -2418 +2418 @@
     }
 
-    update_post_meta( $topic_id, '_bbp_forum_id', (int) $forum_id );
-
-    return apply_filters( 'bbp_update_topic_forum_id', (int) $forum_id, $topic_id );
+    $forum_id = (int) $forum_id;
+
+    update_post_meta( $topic_id, '_bbp_forum_id', $forum_id );
+
+    return (int) apply_filters( 'bbp_update_topic_forum_id', $forum_id, $topic_id );
 }
 
@@ -2437 +2439 @@
     $topic_id = bbp_get_topic_id( $topic_id );
 
-    update_post_meta( $topic_id, '_bbp_topic_id', (int) $topic_id );
-
-    return apply_filters( 'bbp_update_topic_topic_id', (int) $topic_id );
+    update_post_meta( $topic_id, '_bbp_topic_id', $topic_id );
+
+    return apply_filters( 'bbp_update_topic_topic_id', $topic_id );
 }
 
@@ -2473 +2475 @@
     }
 
-    update_post_meta( $topic_id, '_bbp_reply_count', (int) $reply_count );
-
-    return apply_filters( 'bbp_update_topic_reply_count', (int) $reply_count, $topic_id );
+    $reply_count = (int) $reply_count;
+
+    update_post_meta( $topic_id, '_bbp_reply_count', $reply_count );
+
+    return (int) apply_filters( 'bbp_update_topic_reply_count', $reply_count, $topic_id );
 }
 
@@ -2516 +2520 @@
     }
 
-    update_post_meta( $topic_id, '_bbp_reply_count_hidden', (int) $reply_count );
-
-    return apply_filters( 'bbp_update_topic_reply_count_hidden', (int) $reply_count, $topic_id );
+    $reply_count = (int) $reply_count;
+
+    update_post_meta( $topic_id, '_bbp_reply_count_hidden', $reply_count );
+
+    return (int) apply_filters( 'bbp_update_topic_reply_count_hidden', $reply_count, $topic_id );
 }
 
@@ -2557 +2563 @@
     }
 
+    $active_id = (int) $active_id;
+
     // Update only if published
     if ( bbp_get_public_status_id() === get_post_status( $active_id ) ) {
-        update_post_meta( $topic_id, '_bbp_last_active_id', (int) $active_id );
-    }
-
-    return apply_filters( 'bbp_update_topic_last_active_id', (int) $active_id, $topic_id );
+        update_post_meta( $topic_id, '_bbp_last_active_id', $active_id );
+    }
+
+    return (int) apply_filters( 'bbp_update_topic_last_active_id', $active_id, $topic_id );
 }
 
@@ -2638 +2646 @@
     }
 
+    $reply_id = (int) $reply_id;
+
     // Update if reply is published
     if ( bbp_is_reply_published( $reply_id ) ) {
-        update_post_meta( $topic_id, '_bbp_last_reply_id', (int) $reply_id );
-    }
-
-    return apply_filters( 'bbp_update_topic_last_reply_id', (int) $reply_id, $topic_id );
+        update_post_meta( $topic_id, '_bbp_last_reply_id', $reply_id );
+    }
+
+    return (int) apply_filters( 'bbp_update_topic_last_reply_id', $reply_id, $topic_id );
 }
 
@@ -2678 +2688 @@
 
     // Query the DB to get voices in this topic
-    $voices = $wpdb->get_col( $wpdb->prepare( "SELECT COUNT( DISTINCT post_author ) FROM {$wpdb->posts} WHERE ( post_parent = %d AND post_status = '%s' AND post_type = '%s' ) OR ( ID = %d AND post_type = '%s' );", $topic_id, bbp_get_public_status_id(), bbp_get_reply_post_type(), $topic_id, bbp_get_topic_post_type() ) );
-
-    // If there's an error, make sure we have at least have 1 voice
-    $voices = ( empty( $voices ) || is_wp_error( $voices ) ) ? 1 : $voices[0];
+    $voices = (int) $wpdb->get_var( $wpdb->prepare( "SELECT COUNT( DISTINCT post_author ) FROM {$wpdb->posts} WHERE ( post_parent = %d AND post_status = '%s' AND post_type = '%s' ) OR ( ID = %d AND post_type = '%s' );", $topic_id, bbp_get_public_status_id(), bbp_get_reply_post_type(), $topic_id, bbp_get_topic_post_type() ) );
 
     // Update the voice count for this topic id
-    update_post_meta( $topic_id, '_bbp_voice_count', (int) $voices );
-
-    return apply_filters( 'bbp_update_topic_voice_count', (int) $voices, $topic_id );
+    update_post_meta( $topic_id, '_bbp_voice_count', $voices );
+
+    return (int) apply_filters( 'bbp_update_topic_voice_count', $voices, $topic_id );
 }
 
@@ -2722 +2729 @@
     $anonymous_replies = (int) $wpdb->get_var( $wpdb->prepare( "SELECT COUNT( ID ) FROM {$wpdb->posts} WHERE ( post_parent = %d AND post_status = '%s' AND post_type = '%s' AND post_author = 0 ) OR ( ID = %d AND post_type = '%s' AND post_author = 0 );", $topic_id, bbp_get_public_status_id(), bbp_get_reply_post_type(), $topic_id, bbp_get_topic_post_type() ) );
 
-    update_post_meta( $topic_id, '_bbp_anonymous_reply_count', (int) $anonymous_replies );
-
-    return apply_filters( 'bbp_update_topic_anonymous_reply_count', (int) $anonymous_replies, $topic_id );
+    update_post_meta( $topic_id, '_bbp_anonymous_reply_count', $anonymous_replies );
+
+    return (int) apply_filters( 'bbp_update_topic_anonymous_reply_count', $anonymous_replies, $topic_id );
 }