[buddypress-trac] [BuddyPress Trac] #7079: BP-legacy: split group single admin.php in separate template files

buddypress-trac noreply at wordpress.org
Tue Aug 30 22:57:55 UTC 2016


#7079: BP-legacy: split group single admin.php in separate template files
-----------------------------------+-----------------------
 Reporter:  Offereins              |       Owner:  hnla
     Type:  enhancement            |      Status:  accepted
 Priority:  normal                 |   Milestone:  2.7
Component:  Templates              |     Version:
 Severity:  normal                 |  Resolution:
 Keywords:  has-patch 2nd-opinion  |
-----------------------------------+-----------------------

Comment (by r-a-y):

 > I've moved that hidden input out to bp-groups-actions.php. Does that
 seem like a reasonable place to you for it to go?

 Oops, sorry. I glanced the last message of the thread without catching up!
 :)

 `04.patch` looks good.

 Maybe prefix the function with `bp_`.  Starting the function with
 `groups_` is an old relic from the v1.0-1.1 days I believe.

 Hook could also go into bp-legacy's `buddypress-functions.php`.  I just
 took a look at what `bp-nouveau` is doing and they are hardcoding the
 hidden input field directly:
 https://github.com/buddypress/next-template-
 packs/blob/afb886bde7f485e475d79a66afd23c9c490547fb/bp-templates/bp-
 nouveau/includes/groups/template-tags.php#L434

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


More information about the buddypress-trac mailing list