Changeset 4258 for trunk/bbp-includes/topics/functions.php

Timestamp:

10/19/2012 07:42:49 AM (12 years ago)

Author:

johnjamesjacoby

Message:

Number Formatting:

• Introduce $integer parameter to template-tags to switch filter for integer usage.
• Use absint() where appropriate when getting counts.
• Smarter int type casting.
• Fixes bug where calculations were being done against formatted strings.
• Fixes #1974.

File:

• trunk/bbp-includes/topics/functions.php (modified) (4 diffs)

trunk/bbp-includes/topics/functions.php

@@ -3049 +3049 @@
  * @since bbPress (r3540)
  *
+ * @param int $default Default replies per page (15)
  * @uses get_option() To get the setting
  * @uses apply_filters() To allow the return value to be manipulated
  * @return int
  */
-function bbp_get_topics_per_page() {
-
-    // The default per setting
-    $default = 15;
+function bbp_get_topics_per_page( $default = 15 ) {
 
     // Get database option and cast as integer
-    $per = $retval = (int) get_option( '_bbp_topics_per_page', $default );
+    $retval = get_option( '_bbp_topics_per_page', $default );
 
     // If return val is empty, set it to default
@@ -3066 +3064 @@
 
     // Filter and return
-    return (int) apply_filters( 'bbp_get_topics_per_page', $retval, $per );
+    return absint( apply_filters( 'bbp_get_topics_per_page', $retval, $default ) );
 }
 
@@ -3074 +3072 @@
  * @since bbPress (r3540)
  *
+ * @param int $default Default replies per page (25)
  * @uses get_option() To get the setting
  * @uses apply_filters() To allow the return value to be manipulated
  * @return int
  */
-function bbp_get_topics_per_rss_page() {
-
-    // The default per setting
-    $default = 25;
+function bbp_get_topics_per_rss_page( $default = 25 ) {
 
     // Get database option and cast as integer
-    $per = $retval = (int) get_option( '_bbp_topics_per_rss_page', $default );
+    $retval = get_option( '_bbp_topics_per_rss_page', $default );
 
     // If return val is empty, set it to default
@@ -3091 +3087 @@
 
     // Filter and return
-    return (int) apply_filters( 'bbp_get_topics_per_rss_page', $retval, $per );
+    return absint( apply_filters( 'bbp_get_topics_per_rss_page', $retval, $default ) );
 }