Ticket #7477: 7477.2-change-get.diff

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

@@ -708 +708 @@
                 'include'            => false,          // Only include these specific groups (group_ids).
                 'exclude'            => false,          // Do not include these specific groups (group_ids).
                 'parent_id'          => null,           // Get groups that are children of the specified group(s).
+                'slug'               => false,          // Find a group by slug.
                 'search_terms'       => false,          // Limit to groups that match these search terms.
                 'search_columns'     => array(),        // Select which columns to search.
                 'group_type'         => '',             // Array or comma-separated list of group types to limit results to.
@@ -728 +729 @@
                 'user_id'            => $r['user_id'],
                 'include'            => $r['include'],
                 'exclude'            => $r['exclude'],
+                'slug'               => $r['slug'],
                 'parent_id'          => $r['parent_id'],
                 'search_terms'       => $r['search_terms'],
                 'search_columns'     => $r['search_columns'],

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

@@ -876 +876 @@
          *                                            Default: null (no limit).
          *     @type int          $user_id            Optional. If provided, results will be limited to groups
          *                                            of which the specified user is a member. Default: null.
-         *     @type string       $search_terms       Optional. If provided, only groups whose names or descriptions
+         *     @type string       $slug               Optional. If provided, only the group whose slug matches
+         *                                            the provided slug will be returned.
+         *                                            Default: false.
+         *     @type string       $search_terms       Optional. If provided, only groups whose names or descriptions
          *                                            match the search terms will be returned. Allows specifying the
          *                                            wildcard position using a '*' character before or after the
          *                                            string or both. Works in concert with $search_columns.
@@ -938 +941 @@
                         'per_page'           => null,
                         'page'               => null,
                         'user_id'            => 0,
+                        'slug'               => false,
                         'search_terms'       => false,
                         'search_columns'     => array(),
                         'group_type'         => '',
@@ -974 +978 @@
                         $where_conditions['hidden'] = "g.status != 'hidden'";
                 }
 
+                if ( ! empty( $r['slug'] ) ) {
+                        $where_conditions['slug'] = $wpdb->prepare( "g.slug LIKE %s", $r['slug'] );
+                }
+
                 $search = '';
                 if ( isset( $r['search_terms'] ) ) {
                         $search = trim( $r['search_terms'] );

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

@@ -2011 +2011 @@
                 $found = wp_list_pluck( $groups['groups'], 'id' );
                 $this->assertEqualSets( array( $g1, $g4 ), $found );
         }
+
+        /**
+         * @group group_exists
+         */
+        public function test_group_exists_expected_behavior() {
+                $g1 = $this->factory->group->create( array(
+                        'slug' => 'sam',
+                ) );
+                $g2 = $this->factory->group->create( array(
+                        'slug' => 'sam-e',
+                ) );
+
+                $group_id = BP_Groups_Group::group_exists( 'sam' );
+
+                $this->assertSame( $g1, $group_id );
+        }
+
+        /**
+         * @group group_exists
+         */
+        public function test_group_exists_expected_behavior_no_match() {
+                $group_id = BP_Groups_Group::group_exists( 'brandywine-river' );
+
+                $this->assertNull( $group_id );
+        }
 }
 
 /**