Changeset 5700 for trunk/bp-members/bp-members-actions.php

Timestamp:

02/11/2012 02:29:55 AM (15 years ago)

Author:

johnjamesjacoby

Message:

Remove unneeded globals and clean up some code in Members component. Remove signup file and move its code into screens and functions. See #3989.

File:

• trunk/bp-members/bp-members-actions.php (modified) (2 diffs)

trunk/bp-members/bp-members-actions.php

@@ -1 +1 @@
 <?php
-/*******************************************************************************
- * Action Functions
+
+/**
+ * BuddyPress Members Actions
  *
  * Action functions are exactly the same as screen functions, however they do not
  * have a template screen associated with them. Usually they will send the user
  * back to the default screen after execution.
+ *
+ * @package BuddyPress
+ * @subpackage MembersActions
  */
 
@@ -154 +158 @@
  */
 function bp_core_get_random_member() {
-        global $bp;
+        if ( ! isset( $_GET['random-member'] ) )
+                return;
 
-        if ( isset( $_GET['random-member'] ) ) {
-                $user = bp_core_get_users( array( 'type' => 'random', 'per_page' => 1 ) );
-                bp_core_redirect( bp_core_get_user_domain( $user['users'][0]->id ) );
-        }
+        $user = bp_core_get_users( array( 'type' => 'random', 'per_page' => 1 ) );
+        bp_core_redirect( bp_core_get_user_domain( $user['users'][0]->id ) );
 }
 add_action( 'bp_actions', 'bp_core_get_random_member' );
+
 ?>