Changeset 11087 for trunk/tests/phpunit/testcases/groups/class-bp-groups-group.php

Timestamp:

09/13/2016 04:05:07 AM (10 years ago)

Author:

boonebgorges

Message:

Groups: Improve query efficiency for 'admins' and 'mods' properties of group objects.

Previously, the 'admins' and 'mods' property of BP_Groups_Group
objects were only populated when setting the 'populate_extras' flag.
Even then, the query used to populate these properties was uncached,
and required a join against a global table.

This changeset reworks the way that the 'admins' and 'mods' properties
are accessed and set. The properties are now marked protected, and
are accessible by magic __get(). When requested, the cache for the
both properties is set by a single pair of queries: one to fetch
membership data from the BP table, and one to get user objects from
WordPress. The BP table query is cached, and neither query takes place
if the property is never accessed.

This moves us a step closer to eliminating the populate_extras flag
on BP_Groups_Group objects.

See #5451.

File:

• trunk/tests/phpunit/testcases/groups/class-bp-groups-group.php (modified) (1 diff)

trunk/tests/phpunit/testcases/groups/class-bp-groups-group.php

@@ -1243 +1243 @@
 
     /**
+     * @ticket BP5451
+     */
+    public function test_admins_property() {
+        $user_1 = $this->factory->user->create_and_get();
+        $g = $this->factory->group->create( array(
+            'creator_id' => $user_1->ID,
+        ) );
+
+        $group = new BP_Groups_Group( $g );
+
+        $expected_admin_props = array(
+            'user_id' => $user_1->ID,
+            'user_login' => $user_1->user_login,
+            'user_email' => $user_1->user_email,
+            'user_nicename' => $user_1->user_nicename,
+            'is_admin' => 1,
+            'is_mod' => 0,
+        );
+
+        $found_admin = $group->admins[0];
+        foreach ( $expected_admin_props as $prop => $value ) {
+            $this->assertEquals( $value, $found_admin->{$prop} );
+        }
+    }
+
+    /**
+     * @ticket BP5451
+     */
+    public function test_mods_property() {
+        $users = $this->factory->user->create_many( 2 );
+        $user_1 = new WP_User( $users[0] );
+        $user_2 = new WP_User( $users[1] );
+
+        $g = $this->factory->group->create( array(
+            'creator_id' => $user_1->ID,
+        ) );
+
+        $this->add_user_to_group( $user_2->ID, $g, array( 'is_mod' => 1 ) );
+
+        $group = new BP_Groups_Group( $g );
+
+        $expected_mod_props = array(
+            'user_id' => $user_2->ID,
+            'user_login' => $user_2->user_login,
+            'user_email' => $user_2->user_email,
+            'user_nicename' => $user_2->user_nicename,
+            'is_admin' => 0,
+            'is_mod' => 1,
+        );
+
+        $found_mod = $group->mods[0];
+        foreach ( $expected_mod_props as $prop => $value ) {
+            $this->assertEquals( $value, $found_mod->{$prop} );
+        }
+    }
+
+    /**
      * @group group_types
      */