Ticket #7677: 7677.02.patch

src/bp-groups/classes/class-bp-groups-group.php

@@ -560 +560 @@
                 $mod_ids = BP_Groups_Member::get_group_moderator_ids( $this->id );
                 $mod_ids_plucked = wp_list_pluck( $mod_ids, 'user_id' );
 
-                $admin_mod_users = get_users( array(
-                        'include' => array_merge( $admin_ids_plucked, $mod_ids_plucked ),
-                        'blog_id' => null,
-                ) );
+                $admin_mod_ids = array_merge( $admin_ids_plucked, $mod_ids_plucked );
+                $admin_mod_users = array();
+
+                if ( ! empty( $admin_mod_ids ) ) {
+                        $admin_mod_users = get_users( array(
+                                'include' => $admin_mod_ids,
+                                'blog_id' => null,
+                        ) );
+                }
 
                 $admin_objects = $mod_objects = array();
                 foreach ( $admin_mod_users as $admin_mod_user ) {

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

@@ -1515 +1515 @@
                 $this->assertEqualSets( array( $user_1->ID ), wp_list_pluck( $group->admins, 'user_id' ) );
         }
 
+        /**
+         * @ticket BP7677
+         * @group rw
+         */
+        public function test_demoting_sole_admin() {
+                $user = self::factory()->user->create_and_get();
+                $group = self::factory()->group->create_and_get( array(
+                        'creator_id' => $user->ID,
+                ) );
+                $member = new BP_Groups_Member( $user->ID, $group->id );
+                $member->demote();
+
+                $this->assertEmpty( $group->admins );
+                $this->assertEmpty( $group->mods );
+        }
+
         /**
          * @ticket BP5451
          */