Ticket #7224: 7224.01.patch

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

@@ -2298 +2298 @@
                 $types = bp_get_object_terms( $group_id, 'bp_group_type' );
 
                 if ( ! is_wp_error( $types ) ) {
+                        // Filter out currently unregistered group types.
+                        foreach ( $types as $key => $gtype ) {
+                                if ( ! bp_groups_get_group_type_object( $gtype->name ) ) {
+                                        unset( $types[$key] );
+                                }
+                        }
+
                         $types = wp_list_pluck( $types, 'name' );
                         wp_cache_set( $group_id, $types, 'bp_groups_group_type' );
                 }

tests/phpunit/testcases/groups/types.php

@@ -251 +251 @@
 
                 $this->assertEqualSets( array( 'bar' ), $types );
         }
+
+        public function test_groups_get_type_should_not_return_unregistered_types() {
+                $g = $this->factory->group->create( array( 'creator_id' => self::$u1 ) );
+                bp_groups_register_group_type( 'foo' );
+                bp_groups_set_group_type( $g, 'foo' );
+
+                // Directly set a type that hasn't been registered.
+                bp_set_object_terms( $g, 'ugh', 'bp_group_type', true );
+
+                $type = bp_groups_get_group_type( $g, false );
+                $this->assertEquals( array( 'foo' ), $type );
+        }
 }