$group object should be supplied to bp_get_group_name filter in bp-groups-admin.php

[garrett-eclipse]

In BP 2.5 the $group object was supplied to the bp_get_group_name filter as outlined in this ticket;
​https://buddypress.trac.wordpress.org/ticket/6752

I noticed in bp-groups-admin.php there's a reference to this filter but it's missing the $group object;

<?php
$group_name = isset( $group->name ) ? apply_filters( 'bp_get_group_name', $group->name ) : '';

​https://github.com/buddypress/BuddyPress/blob/f8dd8be7fca91a591614b65e3d515ce984efc16c/src/bp-groups/bp-groups-admin.php#L572

Will need to be updated to also supply the $group object.

Thanks

[garrett-eclipse]

I found another instance in bp-groups-template.php;

<?php
/** This filter is documented in bp-groups/bp-groups-template.php */
$name               = apply_filters( 'bp_get_group_name', $current_group_name );

The apply_filters should include $current_group as the group object in this case as follows;

<?php
/** This filter is documented in bp-groups/bp-groups-template.php */
$name               = apply_filters( 'bp_get_group_name', $current_group_name, $current_group );

Cheers

[boonebgorges]

Thanks for the report. Why are we not using the bp_get_group_name() functions in these locations? That's better than redefining the filter everywhere.

[garrett-eclipse]

Agreed, makes more sense to utilize the already filtered function.

[boonebgorges]

In 10775:

Use bp_get_group_name() instead of redefining filter in various places.

Reduces code duplication, and ensures that changes to filter parameters are
automatically applied to every instance of the filter.

Props garrett-eclipse.
See #6752. Fixes #7069.