Changeset 6302 for trunk/src/includes/forums/template.php

Timestamp:

02/23/2017 11:24:29 PM (7 years ago)

Author:

johnjamesjacoby

Message:

Common: Introduce bbp_number_not_negative() and use it in the following ways:

• Register the relevant meta-data keys for posts & users, so updated values can never be invalid
• Filter return values for existing database values that might be invalid on existing installs
• Use in place of intval() or (int) casts where negative values should not exist

This has the added benefit of introducing the bbp_register_meta hook, for future meta-data registrations (of which bbPress has much of.) We'll concentrate on counts for 2.6, and integrate IDs and timestamps in future releases.

See #3059.

File:

• trunk/src/includes/forums/template.php (modified) (5 diffs)

trunk/src/includes/forums/template.php

@@ -1326 +1326 @@
     function bbp_get_forum_subforum_count( $forum_id = 0, $integer = false ) {
         $forum_id    = bbp_get_forum_id( $forum_id );
-        $forum_count = (int) get_post_meta( $forum_id, '_bbp_forum_subforum_count', true );
-        $filter      = ( true === $integer ) ? 'bbp_get_forum_subforum_count_int' : 'bbp_get_forum_subforum_count';
+        $forum_count = bbp_number_not_negative( get_post_meta( $forum_id, '_bbp_forum_subforum_count', true ) );
+        $filter      = ( true === $integer )
+            ? 'bbp_get_forum_subforum_count_int'
+            : 'bbp_get_forum_subforum_count';
 
         return apply_filters( $filter, $forum_count, $forum_id );
@@ -1363 +1365 @@
         $forum_id = bbp_get_forum_id( $forum_id );
         $meta_key = empty( $total_count ) ? '_bbp_topic_count' : '_bbp_total_topic_count';
-        $topics   = (int) get_post_meta( $forum_id, $meta_key, true );
-        $filter   = ( true === $integer ) ? 'bbp_get_forum_topic_count_int' : 'bbp_get_forum_topic_count';
+        $topics   = bbp_number_not_negative( get_post_meta( $forum_id, $meta_key, true ) );
+        $filter   = ( true === $integer )
+            ? 'bbp_get_forum_topic_count_int'
+            : 'bbp_get_forum_topic_count';
 
         return apply_filters( $filter, $topics, $forum_id );
@@ -1400 +1404 @@
         $forum_id = bbp_get_forum_id( $forum_id );
         $meta_key = empty( $total_count ) ? '_bbp_reply_count' : '_bbp_total_reply_count';
-        $replies  = (int) get_post_meta( $forum_id, $meta_key, true );
-        $filter   = ( true === $integer ) ? 'bbp_get_forum_reply_count_int' : 'bbp_get_forum_reply_count';
+        $replies  = bbp_number_not_negative( get_post_meta( $forum_id, $meta_key, true ) );
+        $filter   = ( true === $integer )
+            ? 'bbp_get_forum_reply_count_int'
+            : 'bbp_get_forum_reply_count';
 
         return apply_filters( $filter, $replies, $forum_id );
@@ -1439 +1445 @@
         $topics   = bbp_get_forum_topic_count( $forum_id, $total_count, true );
         $replies  = bbp_get_forum_reply_count( $forum_id, $total_count, true );
-        $retval   = $replies + $topics;
+        $retval   = bbp_number_not_negative( $replies + $topics );
         $filter   = ( true === $integer )
                 ? 'bbp_get_forum_post_count_int'
@@ -1476 +1482 @@
     function bbp_get_forum_topic_count_hidden( $forum_id = 0, $integer = false ) {
         $forum_id = bbp_get_forum_id( $forum_id );
-        $topics   = (int) get_post_meta( $forum_id, '_bbp_topic_count_hidden', true );
-        $filter   = ( true === $integer ) ? 'bbp_get_forum_topic_count_hidden_int' : 'bbp_get_forum_topic_count_hidden';
+        $topics   = bbp_number_not_negative( get_post_meta( $forum_id, '_bbp_topic_count_hidden', true ) );
+        $filter   = ( true === $integer )
+            ? 'bbp_get_forum_topic_count_hidden_int'
+            : 'bbp_get_forum_topic_count_hidden';
 
         return apply_filters( $filter, $topics, $forum_id );