Changeset 6619 for trunk/src/includes/users/functions.php

Timestamp:

07/09/2017 04:53:10 PM (8 years ago)

Author:

johnjamesjacoby

Message:

Users: Use bbp_parse_args() in user content loops.

This commit changes the function signatures of several wrappers for bbp_has_ functions away from simply accepting a $user_id to accepting an array of arguments, in a fully backwards compatible way. It also updates the surrounding documentation to more accurately describe what is returned and why.

This allows the arguments used within these functions to be explicitly filtered, passively or aggressively overridden, or bypassed entirely.

File:

• trunk/src/includes/users/functions.php (modified) (2 diffs)

trunk/src/includes/users/functions.php

@@ -484 +484 @@
  *
  * @since 2.0.0 bbPress (r2660)
- *
- * @param int $user_id Optional. User id
- *
- * @return array|bool Results if the user has created topics, otherwise false
- */
-function bbp_get_user_topics_started( $user_id = 0 ) {
-
-    // Validate user
-    $user_id = bbp_get_user_id( $user_id );
-    if ( empty( $user_id ) ) {
-        return false;
-    }
-
-    // Try to get the topics
-    $query = bbp_has_topics( array(
-        'author' => $user_id
-    ) );
+ * @since 2.6.0 bbPress (r6618) Signature changed to accept an array of arguments
+ *
+ * @param array $args    Optional. Arguments to pass into bbp_has_topics()
+ *
+ * @return bool True if user has started topics, otherwise false
+ */
+function bbp_get_user_topics_started( $args = array() ) {
+
+    // Backwards compat for pre-2.6.0
+    if ( is_numeric( $args ) ) {
+        $args = array(
+            'author' => bbp_get_user_id( $args, false, false )
+        );
+    }
+
+    // Default arguments
+    $defaults = array(
+        'author' => bbp_get_displayed_user_id()
+    );
+
+    // Parse arguments
+    $r = bbp_parse_args( $args, $defaults, 'get_user_topics_started' );
+
+    // Get the topics
+    $query   = bbp_has_topics( $r );
+    $user_id = $r['author'];
 
     // Filter & return
-    return apply_filters( 'bbp_get_user_topics_started', $query, $user_id );
+    return apply_filters( 'bbp_get_user_topics_started', $query, $user_id, $r, $args );
 }
 
@@ -510 +519 @@
  *
  * @since 2.2.0 bbPress (r4225)
- *
- * @param int $user_id Optional. User id
- *
- * @return array|bool Results if the user has created topics, otherwise false
- */
-function bbp_get_user_replies_created( $user_id = 0 ) {
-
-    // Validate user
-    $user_id = bbp_get_user_id( $user_id );
-    if ( empty( $user_id ) ) {
-        return false;
-    }
-
-    // Try to get the topics
-    $query = bbp_has_replies( array(
+ * @since 2.6.0 bbPress (r6618) Signature changed to accept an array of arguments
+ *
+ * @param array $args Optional. Arguments to pass into bbp_has_replies()
+ *
+ * @return bool True if user has created replies, otherwise false
+ */
+function bbp_get_user_replies_created( $args = array() ) {
+
+    // Backwards compat for pre-2.6.0
+    if ( is_numeric( $args ) ) {
+        $args = array(
+            'author' => bbp_get_user_id( $args, false, false ),
+            'post_type' => bbp_get_reply_post_type(),
+            'order'     => 'DESC'
+        );
+    }
+
+    // Default arguments
+    $defaults = array(
+        'author'    => bbp_get_displayed_user_id(),
         'post_type' => bbp_get_reply_post_type(),
-        'order'     => 'DESC',
-        'author'    => $user_id
-    ) );
+        'order'     => 'DESC'
+    );
+
+    // Parse arguments
+    $r = bbp_parse_args( $args, $defaults, 'get_user_replies_created' );
+
+    // Get the replies
+    $query   = bbp_has_replies( $r );
+    $user_id = $r['author'];
 
     // Filter & return
-    return apply_filters( 'bbp_get_user_replies_created', $query, $user_id );
+    return apply_filters( 'bbp_get_user_replies_created', $query, $user_id, $r, $args );
 }