[buddypress-trac] [BuddyPress] #3737: Add missing hooks in blogs/create.php and groups/create.php
buddypress-trac at lists.automattic.com
buddypress-trac at lists.automattic.com
Mon Nov 7 22:10:13 UTC 2011
#3737: Add missing hooks in blogs/create.php and groups/create.php
-------------------------+-----------------------------
Reporter: mercime | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Theme | Version:
Severity: normal | Keywords:
-------------------------+-----------------------------
There are missing hooks in the
[http://buddypress.trac.wordpress.org/browser/tags/1.5.1/bp-themes/bp-
default/groups/create.php bp-default/groups/create.php] and the
[http://buddypress.trac.wordpress.org/browser/tags/1.5.1/bp-themes/bp-
default/blogs/create.php bp-default/blogs/create.php] files which are
available in the other template files of bp-default theme.
So theme authors can hook into the same location in the HTML structure
even if the hooks are named differently and for the sake of consistency,
attached is a proposed patch.
For blogs/index.php you have
{{{
<?php get_header( 'buddypress' ); ?>
'''<?php do_action( 'bp_before_directory_blogs_page' );
?>'''
<div id="content">
<div class="padder">
'''<?php do_action( 'bp_before_directory_blogs' );
?>'''
}}}
while in blogs/create.php
{{{
<?php get_header( 'buddypress' ); ?>
'''<?php do_action( 'bp_before_directory_blogs_content' );
?>'''
<div id="content">
<div class="padder" role="main">
'''// missing hook here - added in patch'''
<?php do_action( 'template_notices' ); ?>
}}}
For groups/index.php
{{{
<?php get_header( 'buddypress' ); ?>
'''<?php do_action( 'bp_before_directory_groups_page' );
?>'''
<div id="content">
<div class="padder">
'''<?php do_action( 'bp_before_directory_groups' );
?>'''
}}}
while in groups/create.php
{{{
<?php get_header( 'buddypress' ); ?>
'''// missing hook here - not added in patch yet'''
<div id="content">
<div class="padder">
'''// missing hook here - added in patch'''
}}}
Same goes for the required closing hooks in the bottom portion of both
files.
For consideration.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/3737>
BuddyPress <http://buddypress.org/>
BuddyPress
More information about the buddypress-trac
mailing list