bp_get_the_profile_field_ids should check that groups exist

[shawfactor]

The bp_get_the_profile_field_ids function should incorporate a check that $profile_template->groups before the foreach loop. At present when you use it programmatically it can generate an error in some contexts

this would be an improvement:

if (!empty($profile_template->groups)){

foreach ( $profile_template->groups as $group ) {

if ( ! empty( $group->fields ) ) {

$field_ids = array_merge( $field_ids, wp_list_pluck( $group->fields, 'id' ) );

}

}

}

Prevent potential errors in bp_get_the_profile_field_ids().

Trac ticket: ​https://buddypress.trac.wordpress.org/ticket/8789

I love the unit tests. 👯‍♀️

[imath]

In 13528:

Prevent potential errors in bp_get_the_profile_field_ids()

In case this function is used inside an xProfile group loop having no attached profile fields, instead of generating a notice error, let's simply return an empty string.

Props shawfactor

Fixes #8789
Closes ​https://github.com/buddypress/buddypress/pull/140