Doubling the same actions

[slaFFik]

In one of my plugins I display additional navigation using bp_before_group_body hook.
Now lets see how BuddyPress loads pages in theme compatibility mode:
1) You open the group page, /groups/single/home.php is loaded (where header and groups inside data logic). This file has this line on line ~27:

<?php do_action( 'bp_before_group_body' ); ?>

2) In case none of if's below it satisfies the current request it loads /groups/single/plugins.php. And inside that file we again have the same hook.

<?php do_action( 'bp_before_group_body' ); ?>

The same for

<?php do_action( 'bp_after_group_body' ); ?>

This creates possible content duplication. Like in my plugin's case.

[boonebgorges]

Confirmed. We changed the way plugins.php worked in bp-legacy; before theme compat, it was loaded from bp_core_load_template(), so that in these cases home.php was never touched. Once it started getting nested in home.php, we never lost the duplicate hooks.

In fact, we also have duplicate markup - #item-body.

I'm pretty sure the solution is as simple as 5166.patch, but I would like a sanity check from another dev that this isn't going to break stuff.

[johnjamesjacoby]

Patch looks good to me. Was a copy/paste job that never saw clean-up after testing the main action.

[boonebgorges]

In 7377:

Remove duplicate markup and actions from bp-legacy groups plugins template

The removed actions and markup are remnants of the original cut-and-paste
from bp-default (where plugins.php was a top-level template) into bp-legacy
(where it is not).

Fixes #5166