Changeset 8480

Timestamp:

06/07/2014 10:09:36 PM (10 years ago)

Author:

r-a-y

Message:

Groups widget: Set the per_page argument in bp_has_groups().

Previously, the per_page argument was not passed to bp_has_groups().
This meant that it wasn't possible to show more than twenty groups for the
groups widget. This commit addresses this.

Commit also switches the querystring to an array when setting up bp_has_groups().

Fixes #5695.

File:

• trunk/src/bp-groups/bp-groups-widgets.php (modified) (1 diff)

trunk/src/bp-groups/bp-groups-widgets.php

@@ -59 +59 @@
         $title = !empty( $instance['link_title'] ) ? '<a href="' . trailingslashit( bp_get_root_domain() . '/' . bp_get_groups_root_slug() ) . '">' . $title . '</a>' : $title;
 
-        echo $before_title . $title . $after_title; ?>
-
-        <?php if ( bp_has_groups( 'user_id=' . $user_id . '&type=' . $instance['group_default'] . '&max=' . $instance['max_groups'] ) ) : ?>
+        echo $before_title . $title . $after_title;
+
+        $group_args = array(
+            'user_id'         => $user_id,
+            'type'            => $instance['group_default'],
+            'per_page'        => $instance['max_groups'],
+            'max'             => $instance['max_groups'],
+        );
+
+        ?>
+
+        <?php if ( bp_has_groups( $group_args ) ) : ?>
             <div class="item-options" id="groups-list-options">
                 <a href="<?php bp_groups_directory_permalink(); ?>" id="newest-groups"<?php if ( $instance['group_default'] == 'newest' ) : ?> class="selected"<?php endif; ?>><?php _e("Newest", 'buddypress') ?></a> |