Changeset 6719 for trunk/src/includes/users/options.php

Timestamp:

09/27/2017 09:14:36 AM (7 years ago)

Author:

johnjamesjacoby

Message:

Common: remove redundant calls to bbp_number_not_negative().

This change decreases the filter priority of out-of-range prevention from 10 to 8, and also adds filters to pre-formatted values in lieu of hard-coding this function into each individual count getter. This allows plugins to unplug this check more easily, and makes sure each value is only checked once for each call instead of occasionally twice in a row.

File:

• trunk/src/includes/users/options.php (modified) (3 diffs)

trunk/src/includes/users/options.php

@@ -206 +206 @@
         }
 
-        $count  = bbp_number_not_negative( get_user_option( '_bbp_topic_count', $user_id ) );
+        $count  = get_user_option( '_bbp_topic_count', $user_id );
         $filter = ( true === $integer )
             ? 'bbp_get_user_topic_count_int'
@@ -246 +246 @@
         }
 
-        $count  = bbp_number_not_negative( get_user_option( '_bbp_reply_count', $user_id ) );
+        $count  = get_user_option( '_bbp_reply_count', $user_id );
         $filter = ( true === $integer )
             ? 'bbp_get_user_reply_count_int'
@@ -287 +287 @@
         $topics  = bbp_get_user_topic_count( $user_id, true );
         $replies = bbp_get_user_reply_count( $user_id, true );
-        $count   = bbp_number_not_negative( $topics + $replies );
+        $count   = $topics + $replies;
         $filter  = ( true === $integer )
             ? 'bbp_get_user_post_count_int'