Changeset 10553 for trunk/src/bp-groups/classes/class-bp-groups-widget.php

Timestamp:

02/08/2016 04:14:59 AM (9 years ago)

Author:

boonebgorges

Message:

Restore template globals after widget loops.

BP template loop functions, like bp_has_members(), are responsible for
populating corresponding template globals. When this happens inside of a
"secondary" loop - such as a sidebar widget - the loop should clean up after
itself by resetting the relevant template global to whatever it previously was
before the secondary loop had run. (This is a rough approximation of WP's
wp_reset_postdata() for BP loops.)

Fixes #5170.

File:

• trunk/src/bp-groups/classes/class-bp-groups-widget.php (modified) (3 diffs)

trunk/src/bp-groups/classes/class-bp-groups-widget.php

@@ -45 +45 @@
      */
     public function widget( $args, $instance ) {
+        global $groups_template;
 
         /**
@@ -100 +101 @@
             'max'             => $max_groups,
         );
+
+        // Back up the global.
+        $old_groups_template = $groups_template;
 
         ?>
@@ -150 +154 @@
         <?php endif; ?>
 
-        <?php echo $after_widget; ?>
-    <?php
+        <?php echo $after_widget;
+
+        // Restore the global.
+        $groups_template = $old_groups_template;
     }