[buddypress-trac] [BuddyPress Trac] #6388: Groups single item home : improve the way we deal with custom front and activity component

[buddypress-trac]

#6388: Groups single item home : improve the way we deal with custom front and
activity component
-------------------------------------------------+-------------------------
 Reporter:  imath                                |       Owner:
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  2.4
Component:  Component - Groups                   |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch 2nd-opinion needs-testing  |
  needs-docs                                     |
-------------------------------------------------+-------------------------
Changes (by DJPaul):

 * keywords:  has-patch 2nd-opinion => has-patch 2nd-opinion needs-testing
     needs-docs


Comment:

 This ticket is somehow confusing, I've started writing a reply two times
 before I realised I mis-understood the intent. :)

 Moving the custom front logic out of the template into
 `bp_groups_load_group_front_template` etc seems to make sense as a general
 improvement. I haven't looked at it closely, but I would like us to check
 that the function name is consistent (do we do anything similar anywhere
 else?).

 Pinging @mercime or @hnla to look at the proposed template hierarchy
 changes for this template (look at the
 `bp_groups_get_group_front_template` function in the patch) -- again, does
 this make sense / is the proposed file name/conventions consistent/make
 sense, etc. And also a reminder we'd need to update our documentation when
 this goes in.

 :)

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6388#comment:4>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac