Ticket #7302: 7302.2.patch

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

@@ -43 +43 @@
  * @return BP_Groups_Group $group The group object.
  */
 function groups_get_group( $group_id ) {
-        // Backward compatibilty.
-        if ( is_array( $group_id ) && isset( $group_id['group_id'] ) ) {
-                $group_id = $group_id['group_id'];
+        /*
+         * Backward compatibilty.
+         * Old-style arguments take the form of an array or a query string.
+         */
+        if ( ! is_numeric( $group_id ) ) {
+                $r = wp_parse_args( $group_id, array(
+                        'group_id'        => false,
+                        'load_users'      => false,
+                        'populate_extras' => false,
+                ) );
+
+                $group_id = $r['group_id'];
         }
 
         $group = new BP_Groups_Group( $group_id );

tests/phpunit/testcases/groups/functions.php

@@ -667 +667 @@
                 $child = groups_get_group( array( 'group_id' => $g3 ) );
                 $this->assertEquals( $g1, $child->parent_id );
         }
+
+        /**
+         * @group groups_get_group
+         * @ticket BP7302
+         */
+        public function test_groups_get_group_accept_integer() {
+                $g1 = $this->factory->group->create();
+                $group = groups_get_group( $g1 );
+
+                $this->assertEquals( $g1, $group->id );
+        }
+
+        /**
+         * @group groups_get_group
+         * @ticket BP7302
+         */
+        public function test_groups_get_group_accept_numeric() {
+                $g1 = $this->factory->group->create();
+                $group = groups_get_group( (string) $g1 );
+
+                $this->assertEquals( $g1, $group->id );
+        }
+
+        /**
+         * @group groups_get_group
+         * @ticket BP7302
+         */
+        public function test_groups_get_group_accept_array() {
+                $g1 = $this->factory->group->create();
+                $group = groups_get_group( array( 'group_id' => $g1 ) );
+
+                $this->assertEquals( $g1, $group->id );
+        }
+
+        /**
+         * @group groups_get_group
+         * @ticket BP7302
+         */
+        public function test_groups_get_group_accept_query_string() {
+                $g1 = $this->factory->group->create();
+                $group = groups_get_group( 'group_id=' . $g1 );
+
+                $this->assertEquals( $g1, $group->id );
+        }
 }