#3534 closed defect (bug) (fixed)
Incorrect position of bp_after_group_plugin_template hook
| Reported by: | Jonnyauk | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 1.5 |
| Component: | Templates | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: |
Description
I've noticed an inconsistency in the bp_after_group_plugin_template hook position in:
bp-themes/bp-default/groups/single/plugins.php
In this file:
bp_before_group_plugin_template -> inside loopbp_after_group_plugin_template -> outside loop
With the 'before' hook being inside the loop, the 'after' hook should be inside too, as per the structure of:
bp-themes/bp-default/groups/single/home.php
bp_before_group_home_content -> inside loopbp_after_group_home_content -> inside loop
Attachments (1)
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Good catch. These should indeed be in the same place. I happen to think they make more sense outside the loop than inside, but given that in most places they are already inside, we should probably leave them there.