Changeset 6517 for trunk/bp-themes/bp-default/groups/single/home.php

Timestamp:

11/13/2012 07:12:07 AM (14 years ago)

Author:

johnjamesjacoby

Message:

Rework template loader logic in bp-default/groups/single/home.php.

• Includes easier to understand and more concise template loader.
• Opens the door to an improved template hierarchy here later.

File:

• trunk/bp-themes/bp-default/groups/single/home.php (modified) (1 diff)

trunk/bp-themes/bp-default/groups/single/home.php

@@ -30 +30 @@
                                 <?php do_action( 'bp_before_group_body' );
 
-                                if ( bp_is_group_admin_page() && bp_group_is_visible() ) :
-                                        locate_template( array( 'groups/single/admin.php' ), true );
+                                /**
+                                 * Does this next bit look familiar? If not, go check out WordPress's
+                                 * /wp-includes/template-loader.php file.
+                                 *
+                                 * @todo A real template hierarchy? Gasp!
+                                 */
 
-                                elseif ( bp_is_group_members() && bp_group_is_visible() ) :
-                                        locate_template( array( 'groups/single/members.php' ), true );
+                                // Group is visible
+                                if ( bp_group_is_visible() ) : 
 
-                                elseif ( bp_is_group_invites() && bp_group_is_visible() ) :
-                                        locate_template( array( 'groups/single/send-invites.php' ), true );
+                                        // Looking at home location
+                                        if ( bp_is_group_home() ) :
 
-                                        elseif ( bp_is_group_forum() && bp_group_is_visible() && bp_is_active( 'forums' ) && bp_forums_is_installed_correctly() ) :
-                                                locate_template( array( 'groups/single/forum.php' ), true );
+                                                // Use custom front if one exists
+                                                $custom_front = locate_template( array( 'groups/single/front.php' ) );
+                                                if     ( ! empty( $custom_front   ) ) : load_template( $custom_front, true );
+                                                
+                                                // Default to activity
+                                                elseif ( bp_is_active( 'activity' ) ) : locate_template( array( 'groups/single/activity.php' ), true );
 
-                                elseif ( bp_is_group_membership_request() ) :
-                                        locate_template( array( 'groups/single/request-membership.php' ), true );
+                                                // Otherwise show members
+                                                elseif ( bp_is_active( 'members'  ) ) : locate_template( array( 'groups/single/members.php'  ), true );
 
-                                elseif ( bp_group_is_visible() && bp_is_active( 'activity' ) ) :
-                                        locate_template( array( 'groups/single/activity.php' ), true );
+                                                endif;
 
-                                elseif ( bp_group_is_visible() ) :
-                                        locate_template( array( 'groups/single/members.php' ), true );
+                                        // Not looking at home
+                                        else :
 
-                                elseif ( !bp_group_is_visible() ) :
+                                                // Group Admin
+                                                if     ( bp_is_group_admin_page() ) : locate_template( array( 'groups/single/admin.php'        ), true );
+
+                                                // Group Activity
+                                                elseif ( bp_is_group_activity()   ) : locate_template( array( 'groups/single/activity.php'     ), true );
+
+                                                // Group Members
+                                                elseif ( bp_is_group_members()    ) : locate_template( array( 'groups/single/members.php'      ), true );
+
+                                                // Group Invitations
+                                                elseif ( bp_is_group_invites()    ) : locate_template( array( 'groups/single/send-invites.php' ), true );
+
+                                                // Old group forums
+                                                elseif ( bp_is_group_forum()      ) : locate_template( array( 'groups/single/forum.php'        ), true );
+
+                                                // Anything else (plugins mostly)
+                                                else                                : locate_template( array( 'groups/single/plugins.php'      ), true );
+
+                                                endif;
+                                        endif;
+
+                                // Group is not visible
+                                elseif ( ! bp_group_is_visible() ) :
+
+                                        // Membership request
+                                        if ( bp_is_group_membership_request() ) :
+                                                locate_template( array( 'groups/single/request-membership' ), true );
+
                                         // The group is not visible, show the status message
+                                        else :
 
-                                        do_action( 'bp_before_group_status_message' ); ?>
+                                                do_action( 'bp_before_group_status_message' ); ?>
 
-                                        <div id="message" class="info">
-                                                <p><?php bp_group_status_message(); ?></p>
-                                        </div>
+                                                <div id="message" class="info">
+                                                        <p><?php bp_group_status_message(); ?></p>
+                                                </div>
 
-                                        <?php do_action( 'bp_after_group_status_message' );
+                                                <?php do_action( 'bp_after_group_status_message' );
 
-                                else :
-                                        // If nothing sticks, just load a group front template if one exists.
-                                        locate_template( array( 'groups/single/front.php' ), true );
-
-                                endif;
+                                        endif;
+                                endif;                  
 
                                 do_action( 'bp_after_group_body' ); ?>