Changeset 6107

Timestamp:

06/18/2012 12:43:03 PM (13 years ago)

Author:

boonebgorges

Message:

Fixes random group fetching in groups_action_redirect_to_random_group()

At some point, groups_get_group() was changed in such a way that the 'type'
and 'per_page' arguments were removed. This changeset reworks the way that
the group action function fetches a random group, to work around these changes.

Fixes #4269

Location:

trunk/bp-groups

Files:

• bp-groups-actions.php (modified) (1 diff)
• bp-groups-classes.php (modified) (1 diff)

trunk/bp-groups/bp-groups-actions.php

@@ -291 +291 @@
 }
 
+/**
+ * Catches requests for a random group page (example.com/groups/?random-group) and redirects
+ */
 function groups_action_redirect_to_random_group() {
 
     if ( bp_is_groups_component() && isset( $_GET['random-group'] ) ) {
-        $group = groups_get_groups( array( 'type' => 'random', 'per_page' => 1 ) );
+        $group = BP_Groups_Group::get_random( 1, 1 );
 
         bp_core_redirect( bp_get_root_domain() . '/' . bp_get_groups_root_slug() . '/' . $group['groups'][0]->slug . '/' );

trunk/bp-groups/bp-groups-classes.php

@@ -556 +556 @@
         global $wpdb, $bp;
 
+        $pag_sql = $hidden_sql = $search_sql = $exclude_sql = '';
+
         if ( !empty( $limit ) && !empty( $page ) )
             $pag_sql = $wpdb->prepare( " LIMIT %d, %d", intval( ( $page - 1 ) * $limit), intval( $limit ) );