Skip to:
Content

BuddyPress.org

Ticket #3961: 3961.08.patch

File 3961.08.patch, 21.7 KB (added by dcavins, 8 years ago)

Make sure that the default parent_id everywhere is null. Add tests for null and false values

  • src/bp-core/admin/bp-core-admin-schema.php 

    diff --git src/bp-core/admin/bp-core-admin-schema.php src/bp-core/admin/bp-core-admin-schema.php
index 6f9043a..44506bf 100644
    function bp_core_install_groups() { 
    202202                                slug varchar(200) NOT NULL,
    203203                                description longtext NOT NULL,
    204204                                status varchar(10) NOT NULL DEFAULT 'public',
     205                                parent_id bigint(20) NOT NULL DEFAULT 0,
    205206                                enable_forum tinyint(1) NOT NULL DEFAULT '1',
    206207                                date_created datetime NOT NULL,
    207208                                KEY creator_id (creator_id),
    208                                 KEY status (status)
     209                                KEY status (status),
     210                                KEY parent_id (parent_id)
    209211                        ) {$charset_collate};";
    210212
    211213        $sql[] = "CREATE TABLE {$bp_prefix}bp_groups_members (

  • src/bp-core/bp-core-update.php 

    diff --git src/bp-core/bp-core-update.php src/bp-core/bp-core-update.php
index 278706c..6d327e3 100644
    function bp_update_to_2_7() { 
    518518
    519519        // Update post_titles
    520520        bp_migrate_directory_page_titles();
     521
     522        /*
     523         * Add `parent_id` column to groups table.
     524         * Also handled by `bp_core_install()`.
     525         */
     526        if ( bp_is_active( 'groups' ) ) {
     527                bp_core_install_groups();
     528
     529                // Invalidate all cached group objects.
     530                global $wpdb;
     531                $bp = buddypress();
     532
     533                $group_ids = $wpdb->get_col( "SELECT id FROM {$bp->groups->table_name}" );
     534
     535                foreach ( $group_ids as $group_id ) {
     536                        wp_cache_delete( $group_id, 'bp_groups' );
     537                }
     538        }
    521539}
    522540
    523541/**

  • src/bp-groups/bp-groups-actions.php 

    diff --git src/bp-groups/bp-groups-actions.php src/bp-groups/bp-groups-actions.php
index ea8ae65..c506708 100644
    function groups_action_group_feed() { 
    566566        ) );
    567567}
    568568add_action( 'bp_actions', 'groups_action_group_feed' );
     569
     570/**
     571 * Update orphaned child groups when the parent is deleted.
     572 *
     573 * @since 2.7.0
     574 *
     575 * @param BP_Groups_Group $group Instance of the group item being deleted.
     576 */
     577function bp_groups_update_orphaned_groups_on_group_delete( $group ) {
     578        // Get child groups and set the parent to the deleted parent's parent.
     579        $grandparent_group_id = $group->parent_id;
     580        $child_args = array(
     581                'parent_id'         => $group->id,
     582                'show_hidden'       => true,
     583                'per_page'          => false,
     584                'update_meta_cache' => false,
     585        );
     586        $children = groups_get_groups( $child_args );
     587        $children = $children['groups'];
     588
     589        foreach ( $children as $cgroup ) {
     590                $cgroup->parent_id = $grandparent_group_id;
     591                $cgroup->save();
     592        }
     593}
     594add_action( 'bp_groups_delete_group', 'bp_groups_update_orphaned_groups_on_group_delete', 10, 2 );
     595 No newline at end of file

  • src/bp-groups/bp-groups-functions.php 

    diff --git src/bp-groups/bp-groups-functions.php src/bp-groups/bp-groups-functions.php
index c31b780..ef3ff06 100644
    function groups_get_group( $group_id ) { 
    7777 *     @type string   $slug         The group slug.
    7878 *     @type string   $status       The group's status. Accepts 'public', 'private' or
    7979 *                                  'hidden'. Defaults to 'public'.
     80 *     @type int      $parent_id    The ID of the parent group. Default: 0.
    8081 *     @type int      $enable_forum Optional. Whether the group has a forum enabled.
    8182 *                                  If the legacy forums are enabled for this group
    8283 *                                  or if a bbPress forum is enabled for the group,
    … … function groups_create_group( $args = '' ) { 
    9596                'description'  => '',
    9697                'slug'         => '',
    9798                'status'       => 'public',
     99                'parent_id'    => 0,
    98100                'enable_forum' => 0,
    99101                'date_created' => bp_core_current_time()
    100102        );
    … … function groups_create_group( $args = '' ) { 
    141143        $group->description  = $description;
    142144        $group->slug         = $slug;
    143145        $group->status       = $status;
     146        $group->parent_id    = $parent_id;
    144147        $group->enable_forum = (int) $enable_forum;
    145148        $group->date_created = $date_created;
    146149
    … … function groups_edit_base_group_details( $group_id, $group_name, $group_desc, $n 
    260263 *                                   to the group. 'members', 'mods', or 'admins'.
    261264 * @return bool True on success, false on failure.
    262265 */
    263 function groups_edit_group_settings( $group_id, $enable_forum, $status, $invite_status = false ) {
     266function groups_edit_group_settings( $group_id, $enable_forum, $status, $invite_status = false, $parent_id = false ) {
    264267
    265268        $group = groups_get_group( $group_id );
    266269        $group->enable_forum = $enable_forum;
    … … function groups_edit_group_settings( $group_id, $enable_forum, $status, $invite_ 
    275278        // Now update the status.
    276279        $group->status = $status;
    277280
     281        // Update the parent ID if necessary.
     282        if ( false !== $parent_id ) {
     283                $group->parent_id = $parent_id;
     284        }
     285
    278286        if ( !$group->save() )
    279287                return false;
    280288
    … … function groups_get_total_member_count( $group_id ) { 
    674682 *
    675683 * @since 1.2.0
    676684 * @since 2.6.0 Added `$group_type`, `$group_type__in`, and `$group_type__not_in` parameters.
    677  * @since 2.7.0 Added `$update_admin_cache` parameter.
     685 * @since 2.7.0 Added `$update_admin_cache` and `$parent_id` parameters.
    678686 *
    679687 * @param array|string $args {
    680688 *     Array of arguments. Supports all arguments of
    … … function groups_get_groups( $args = '' ) { 
    694702                'user_id'            => false,          // Pass a user_id to limit to only groups that this user is a member of.
    695703                'include'            => false,          // Only include these specific groups (group_ids).
    696704                'exclude'            => false,          // Do not include these specific groups (group_ids).
     705                'parent_id'          => null,           // Get groups that are children of the specified group(s).
    697706                'search_terms'       => false,          // Limit to groups that match these search terms.
    698707                'group_type'         => '',
    699708                'group_type__in'     => '',
    … … function groups_get_groups( $args = '' ) { 
    714723                'user_id'            => $r['user_id'],
    715724                'include'            => $r['include'],
    716725                'exclude'            => $r['exclude'],
     726                'parent_id'          => $r['parent_id'],
    717727                'search_terms'       => $r['search_terms'],
    718728                'group_type'         => $r['group_type'],
    719729                'group_type__in'     => $r['group_type__in'],

  • src/bp-groups/bp-groups-template.php 

    diff --git src/bp-groups/bp-groups-template.php src/bp-groups/bp-groups-template.php
index f0ac261..f16bee3 100644
    function bp_groups_directory_permalink() { 
    138138 *                                            about groups. Default: true.
    139139 *     @type array|string $exclude            Array or comma-separated list of group IDs. Results will exclude
    140140 *                                            the listed groups. Default: false.
     141 *     @type array|string $parent_id          Array or comma-separated list of group IDs. Results will include only
     142 *                                            child groups of the listed groups. Default: null.
    141143 *     @type bool         $update_meta_cache  Whether to fetch groupmeta for queried groups. Default: true.
    142144 *     @type bool         $update_admin_cache Whether to pre-fetch group admins for queried groups.
    143145 *                                            Defaults to true when on a group directory, where this
    … … function bp_has_groups( $args = '' ) { 
    202204                'meta_query'         => false,
    203205                'include'            => false,
    204206                'exclude'            => false,
     207                'parent_id'          => null,
    205208                'populate_extras'    => true,
    206209                'update_meta_cache'  => true,
    207210                'update_admin_cache' => bp_is_groups_directory() || bp_is_user_groups(),
    … … function bp_has_groups( $args = '' ) { 
    226229                'meta_query'         => $r['meta_query'],
    227230                'include'            => $r['include'],
    228231                'exclude'            => $r['exclude'],
     232                'parent_id'          => $r['parent_id'],
    229233                'populate_extras'    => (bool) $r['populate_extras'],
    230234                'update_meta_cache'  => (bool) $r['update_meta_cache'],
    231235                'update_admin_cache' => (bool) $r['update_admin_cache'],

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

    diff --git src/bp-groups/classes/class-bp-groups-group.php src/bp-groups/classes/class-bp-groups-group.php
index c12afec..1a1798c 100644
    class BP_Groups_Group { 
    6868        public $status;
    6969
    7070        /**
     71         * Parent ID.
     72         *
     73         * ID of parent group, if applicable.
     74         *
     75         * @since 2.7.0
     76         * @var int
     77         */
     78        public $parent_id;
     79
     80        /**
    7181         * Should (legacy) bbPress forums be enabled for this group?
    7282         *
    7383         * @since 1.6.0
    … … class BP_Groups_Group { 
    208218                $this->slug         = $group->slug;
    209219                $this->description  = stripslashes( $group->description );
    210220                $this->status       = $group->status;
     221                $this->parent_id    = (int) $group->parent_id;
    211222                $this->enable_forum = (int) $group->enable_forum;
    212223                $this->date_created = $group->date_created;
    213224        }
    … … class BP_Groups_Group { 
    229240                $this->slug         = apply_filters( 'groups_group_slug_before_save',         $this->slug,         $this->id );
    230241                $this->description  = apply_filters( 'groups_group_description_before_save',  $this->description,  $this->id );
    231242                $this->status       = apply_filters( 'groups_group_status_before_save',       $this->status,       $this->id );
     243                $this->parent_id    = apply_filters( 'groups_group_parent_id_before_save',    $this->parent_id,    $this->id );
    232244                $this->enable_forum = apply_filters( 'groups_group_enable_forum_before_save', $this->enable_forum, $this->id );
    233245                $this->date_created = apply_filters( 'groups_group_date_created_before_save', $this->date_created, $this->id );
    234246
    … … class BP_Groups_Group { 
    271283                                        slug = %s,
    272284                                        description = %s,
    273285                                        status = %s,
     286                                        parent_id = %d,
    274287                                        enable_forum = %d,
    275288                                        date_created = %s
    276289                                WHERE
    … … class BP_Groups_Group { 
    281294                                        $this->slug,
    282295                                        $this->description,
    283296                                        $this->status,
     297                                        $this->parent_id,
    284298                                        $this->enable_forum,
    285299                                        $this->date_created,
    286300                                        $this->id
    … … class BP_Groups_Group { 
    293307                                        slug,
    294308                                        description,
    295309                                        status,
     310                                        parent_id,
    296311                                        enable_forum,
    297312                                        date_created
    298313                                ) VALUES (
    299                                         %d, %s, %s, %s, %s, %d, %s
     314                                        %d, %s, %s, %s, %s, %d, %d, %s
    300315                                )",
    301316                                        $this->creator_id,
    302317                                        $this->name,
    303318                                        $this->slug,
    304319                                        $this->description,
    305320                                        $this->status,
     321                                        $this->parent_id,
    306322                                        $this->enable_forum,
    307323                                        $this->date_created
    308324                        );
    … … class BP_Groups_Group { 
    836852         *
    837853         * @since 1.6.0
    838854         * @since 2.6.0 Added `$group_type`, `$group_type__in`, and `$group_type__not_in` parameters.
    839          * @since 2.7.0 Added `$update_admin_cache` parameter.
     855         * @since 2.7.0 Added `$update_admin_cache` and `$parent_id` parameters.
    840856         *
    841857         * @param array $args {
    842858         *     Array of parameters. All items are optional.
    … … class BP_Groups_Group { 
    863879         *                                            See {@link WP_Meta_Query::queries} for description.
    864880         *     @type array|string $value              Optional. Array or comma-separated list of group IDs. Results
    865881         *                                            will be limited to groups within the list. Default: false.
     882         *     @type array|string $parent_id          Optional. Array or comma-separated list of group IDs. Results
     883         *                                            will be limited to children of the specified groups. Default: null.
    866884         *     @type bool         $populate_extras    Whether to fetch additional information
    867885         *                                            (such as member count) about groups. Default: true.
    868886         *     @type array|string $exclude            Optional. Array or comma-separated list of group IDs.
    … … class BP_Groups_Group { 
    916934                        'group_type__not_in' => '',
    917935                        'meta_query'         => false,
    918936                        'include'            => false,
     937                        'parent_id'          => null,
    919938                        'populate_extras'    => true,
    920939                        'update_meta_cache'  => true,
    921940                        'update_admin_cache' => false,
    … … class BP_Groups_Group { 
    9871006                        $where_conditions['include'] = "g.id IN ({$include})";
    9881007                }
    9891008
     1009                if ( ! is_null( $r['parent_id'] ) ) {
     1010                        // Note that `wp_parse_id_list()` converts `false` to 0.
     1011                        $parent_id        = implode( ',', wp_parse_id_list( $r['parent_id'] ) );
     1012                        $where_conditions['parent_id'] = "g.parent_id IN ({$parent_id})";
     1013                }
     1014
    9901015                if ( ! empty( $r['exclude'] ) ) {
    9911016                        $exclude        = implode( ',', wp_parse_id_list( $r['exclude'] ) );
    9921017                        $where_conditions['exclude'] = "g.id NOT IN ({$exclude})";

  • src/bp-groups/classes/class-bp-groups-template.php 

    diff --git src/bp-groups/classes/class-bp-groups-template.php src/bp-groups/classes/class-bp-groups-template.php
index b902967..9071718 100644
    class BP_Groups_Template { 
    165165                        'slug'               => false,
    166166                        'include'            => false,
    167167                        'exclude'            => false,
     168                        'parent_id'          => null,
    168169                        'search_terms'       => '',
    169170                        'group_type'         => '',
    170171                        'group_type__in'     => '',
    … … class BP_Groups_Template { 
    224225                                'group_type__not_in' => $group_type__not_in,
    225226                                'include'            => $include,
    226227                                'exclude'            => $exclude,
     228                                'parent_id'          => $parent_id,
    227229                                'populate_extras'    => $populate_extras,
    228230                                'update_meta_cache'  => $update_meta_cache,
    229231                                'update_admin_cache' => $update_admin_cache,

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

    diff --git tests/phpunit/testcases/groups/class-bp-groups-group.php tests/phpunit/testcases/groups/class-bp-groups-group.php
index df5e5a3..bb5b0f6 100644
    class BP_Tests_BP_Groups_Group_TestCases extends BP_UnitTestCase { 
    17051705
    17061706                $this->assertEmpty( $groups['groups'] );
    17071707        }
     1708
     1709        /**
     1710         * @group hierarchical_groups
     1711         */
     1712        public function test_get_by_parent_id() {
     1713                $g1 = $this->factory->group->create();
     1714                $g2 = $this->factory->group->create( array(
     1715                        'parent_id' => $g1,
     1716                ) );
     1717                $g3 = $this->factory->group->create( array(
     1718                        'parent_id' => $g2,
     1719                ) );
     1720                $g4 = $this->factory->group->create();
     1721
     1722                $groups = BP_Groups_Group::get( array(
     1723                        'parent_id' => $g1,
     1724                ) );
     1725
     1726                $found = wp_list_pluck( $groups['groups'], 'id' );
     1727                $this->assertEquals( array( $g2 ), $found );
     1728        }
     1729
     1730        /**
     1731         * @group hierarchical_groups
     1732         */
     1733        public function test_get_by_parent_id_ignore_grandparent() {
     1734                $g1 = $this->factory->group->create();
     1735                $g2 = $this->factory->group->create( array(
     1736                        'parent_id' => $g1,
     1737                ) );
     1738                $g3 = $this->factory->group->create( array(
     1739                        'parent_id' => $g2,
     1740                ) );
     1741                $g4 = $this->factory->group->create();
     1742
     1743                $groups = BP_Groups_Group::get( array(
     1744                        'parent_id' => $g2,
     1745                ) );
     1746
     1747                $found = wp_list_pluck( $groups['groups'], 'id' );
     1748                $this->assertEquals( array( $g3 ), $found );
     1749        }
     1750
     1751        /**
     1752         * @group hierarchical_groups
     1753         */
     1754        public function test_get_by_parent_id_array() {
     1755                $g1 = $this->factory->group->create();
     1756                $g2 = $this->factory->group->create( array(
     1757                        'parent_id' => $g1,
     1758                ) );
     1759                $g3 = $this->factory->group->create( array(
     1760                        'parent_id' => $g2,
     1761                ) );
     1762                $g4 = $this->factory->group->create();
     1763
     1764                $groups = BP_Groups_Group::get( array(
     1765                        'parent_id' => array( $g1, $g2 ),
     1766                ) );
     1767
     1768                $found = wp_list_pluck( $groups['groups'], 'id' );
     1769                $this->assertEqualSets( array( $g2, $g3 ), $found );
     1770        }
     1771
     1772        /**
     1773         * @group hierarchical_groups
     1774         */
     1775        public function test_get_by_parent_id_comma_separated_string() {
     1776                $g1 = $this->factory->group->create();
     1777                $g2 = $this->factory->group->create( array(
     1778                        'parent_id' => $g1,
     1779                ) );
     1780                $g3 = $this->factory->group->create( array(
     1781                        'parent_id' => $g2,
     1782                ) );
     1783                $g4 = $this->factory->group->create();
     1784
     1785                $groups = BP_Groups_Group::get( array(
     1786                        'parent_id' => "$g1, $g2",
     1787                ) );
     1788
     1789                $found = wp_list_pluck( $groups['groups'], 'id' );
     1790                $this->assertEqualSets( array( $g2, $g3 ), $found );
     1791        }
     1792
     1793        /**
     1794         * @group hierarchical_groups
     1795         */
     1796        public function test_get_by_parent_id_top_level_groups() {
     1797                $g1 = $this->factory->group->create();
     1798                $g2 = $this->factory->group->create( array(
     1799                        'parent_id' => $g1,
     1800                ) );
     1801                $g3 = $this->factory->group->create( array(
     1802                        'parent_id' => $g2,
     1803                ) );
     1804                $g4 = $this->factory->group->create();
     1805
     1806                $groups = BP_Groups_Group::get( array(
     1807                        'parent_id' => 0,
     1808                ) );
     1809
     1810                $found = wp_list_pluck( $groups['groups'], 'id' );
     1811                $this->assertEqualSets( array( $g1, $g4 ), $found );
     1812        }
     1813
     1814        /**
     1815         * @group hierarchical_groups
     1816         */
     1817        public function test_get_by_parent_id_top_level_groups_using_false() {
     1818                $g1 = $this->factory->group->create();
     1819                $g2 = $this->factory->group->create( array(
     1820                        'parent_id' => $g1,
     1821                ) );
     1822                $g3 = $this->factory->group->create( array(
     1823                        'parent_id' => $g2,
     1824                ) );
     1825                $g4 = $this->factory->group->create();
     1826
     1827                $groups = BP_Groups_Group::get( array(
     1828                        'parent_id' => false,
     1829                ) );
     1830
     1831                $found = wp_list_pluck( $groups['groups'], 'id' );
     1832                $this->assertEqualSets( array( $g1, $g4 ), $found );
     1833        }
    17081834}
    17091835
    17101836/**

  • tests/phpunit/testcases/groups/functions.php 

    diff --git tests/phpunit/testcases/groups/functions.php tests/phpunit/testcases/groups/functions.php
index 8e2f76a..7fc01de 100644
    Bar!'; 
    634634                groups_accept_invite( $u2, $g );
    635635                $this->assertEquals( 0, groups_get_invite_count_for_user( $u2 ) );
    636636        }
     637
     638        /**
     639         * @group hierarchical_groups
     640         */
     641        public function test_update_orphaned_groups_on_group_delete_top_level() {
     642                $g1 = $this->factory->group->create();
     643                $g2 = $this->factory->group->create( array(
     644                        'parent_id' => $g1,
     645                ) );
     646
     647                groups_delete_group( $g1 );
     648
     649                $child = groups_get_group( array( 'group_id' => $g2 ) );
     650                $this->assertEquals( 0, $child->parent_id );
     651        }
     652
     653        /**
     654         * @group hierarchical_groups
     655         */
     656        public function test_update_orphaned_groups_on_group_delete_two_levels() {
     657                $g1 = $this->factory->group->create();
     658                $g2 = $this->factory->group->create( array(
     659                        'parent_id' => $g1,
     660                ) );
     661                $g3 = $this->factory->group->create( array(
     662                        'parent_id' => $g2,
     663                ) );
     664
     665                groups_delete_group( $g2 );
     666
     667                $child = groups_get_group( array( 'group_id' => $g3 ) );
     668                $this->assertEquals( $g1, $child->parent_id );
     669        }
    637670}

  • tests/phpunit/testcases/groups/template.php 

    diff --git tests/phpunit/testcases/groups/template.php tests/phpunit/testcases/groups/template.php
index 68988ea..106b9cb 100644
    class BP_Tests_Groups_Template extends BP_UnitTestCase { 
    120120        }
    121121
    122122        /**
     123         * @group hierarchical_groups
     124         */
     125        public function test_bp_has_groups_parent_id() {
     126                $g1 = $this->factory->group->create();
     127                $g2 = $this->factory->group->create( array(
     128                        'parent_id' => $g1,
     129                ) );
     130                $g3 = $this->factory->group->create( array(
     131                        'parent_id' => $g2,
     132                ) );
     133                $g4 = $this->factory->group->create();
     134
     135                global $groups_template;
     136                bp_has_groups( array(
     137                        'parent_id' => $g1,
     138                ) );
     139
     140                $ids = wp_parse_id_list( wp_list_pluck( $groups_template->groups, 'id' ) );
     141                $this->assertEquals( array( $g2 ), $ids );
     142        }
     143
     144        /**
     145         * @group hierarchical_groups
     146         */
     147        public function test_bp_has_groups_parent_id_array() {
     148                $g1 = $this->factory->group->create();
     149                $g2 = $this->factory->group->create( array(
     150                        'parent_id' => $g1,
     151                ) );
     152                $g3 = $this->factory->group->create( array(
     153                        'parent_id' => $g2,
     154                ) );
     155                $g4 = $this->factory->group->create();
     156
     157                global $groups_template;
     158                bp_has_groups( array(
     159                        'parent_id' => array( $g1, $g2 ),
     160                ) );
     161
     162                $ids = wp_parse_id_list( wp_list_pluck( $groups_template->groups, 'id' ) );
     163                $this->assertEqualSets( array( $g2, $g3 ), $ids );
     164        }
     165
     166        /**
     167         * @group hierarchical_groups
     168         */
     169        public function test_bp_has_groups_parent_id_comma_separated() {
     170                $g1 = $this->factory->group->create();
     171                $g2 = $this->factory->group->create( array(
     172                        'parent_id' => $g1,
     173                ) );
     174                $g3 = $this->factory->group->create( array(
     175                        'parent_id' => $g2,
     176                ) );
     177                $g4 = $this->factory->group->create();
     178
     179                global $groups_template;
     180                bp_has_groups( array(
     181                        'parent_id' => "{$g1},{$g2}",
     182                ) );
     183
     184                $ids = wp_parse_id_list( wp_list_pluck( $groups_template->groups, 'id' ) );
     185                $this->assertEqualSets( array( $g2, $g3 ), $ids );
     186        }
     187
     188        /**
     189         * @group hierarchical_groups
     190         */
     191        public function test_bp_has_groups_parent_id_null() {
     192                $g1 = $this->factory->group->create();
     193                $g2 = $this->factory->group->create( array(
     194                        'parent_id' => $g1,
     195                ) );
     196                $g3 = $this->factory->group->create( array(
     197                        'parent_id' => $g2,
     198                ) );
     199                $g4 = $this->factory->group->create();
     200
     201                global $groups_template;
     202                bp_has_groups( array(
     203                        'parent_id' => null,
     204                ) );
     205
     206                $ids = wp_parse_id_list( wp_list_pluck( $groups_template->groups, 'id' ) );
     207                $this->assertEqualSets( array( $g1, $g2, $g3, $g4 ), $ids );
     208        }
     209
     210        /**
     211         * @group hierarchical_groups
     212         */
     213        public function test_bp_has_groups_parent_id_top_level_groups() {
     214                $g1 = $this->factory->group->create();
     215                $g2 = $this->factory->group->create( array(
     216                        'parent_id' => $g1,
     217                ) );
     218                $g3 = $this->factory->group->create( array(
     219                        'parent_id' => $g2,
     220                ) );
     221                $g4 = $this->factory->group->create();
     222
     223                global $groups_template;
     224                bp_has_groups( array(
     225                        'parent_id' => 0,
     226                ) );
     227
     228                $ids = wp_parse_id_list( wp_list_pluck( $groups_template->groups, 'id' ) );
     229                $this->assertEqualSets( array( $g1, $g4 ), $ids );
     230        }
     231
     232        /**
     233         * @group hierarchical_groups
     234         */
     235        public function test_bp_has_groups_parent_id_top_level_groups_using_false() {
     236                $g1 = $this->factory->group->create();
     237                $g2 = $this->factory->group->create( array(
     238                        'parent_id' => $g1,
     239                ) );
     240                $g3 = $this->factory->group->create( array(
     241                        'parent_id' => $g2,
     242                ) );
     243                $g4 = $this->factory->group->create();
     244
     245                global $groups_template;
     246                bp_has_groups( array(
     247                        'parent_id' => false,
     248                ) );
     249
     250                $ids = wp_parse_id_list( wp_list_pluck( $groups_template->groups, 'id' ) );
     251                $this->assertEqualSets( array( $g1, $g4 ), $ids );
     252        }
     253
     254        /**
    123255         * @group bp_group_has_members
    124256         */
    125257        public function test_bp_group_has_members_vanilla() {