Changeset 8081

Timestamp:

03/08/2014 12:33:15 PM (11 years ago)

Author:

boonebgorges

Message:

Improve variable naming in bp_get_member_profile_data()

The overuse of the $user_id variable, combined with the ill-advised use of
explode(), was resulting in situations where it was impossible to pass a custom
value of $user_id to the function.

See #5334

File:

• trunk/bp-members/bp-members-template.php (modified) (1 diff)

trunk/bp-members/bp-members-template.php

@@ -665 +665 @@
 
         // Declare local variables
-        $data    = false;
-        $user_id = 0;
+        $data = false;
 
         // Guess at default $user_id
+        $default_user_id = 0;
         if ( !empty( $members_template->member->id ) )
-            $user_id = $members_template->member->id;
+            $default_user_id = $members_template->member->id;
         elseif ( bp_displayed_user_id() )
-            $user_id = bp_displayed_user_id();
+            $default_user_id = bp_displayed_user_id();
 
         $defaults = array(
-            'field'   => false,   // Field name
-            'user_id' => $user_id
+            'field'   => false,
+            'user_id' => $default_user_id,
         );