Changeset 3810 for trunk/bp-themes/bp-default/members/members-loop.php

Timestamp:

01/22/2011 10:13:25 AM (14 years ago)

Author:

johnjamesjacoby

Message:

First pass at normalizing bp-default template files. This includes:

A ton of code cleanup
Using new functions found in BP 1.3
Template header documentation
Improve pagination style and position on root component templates

File:

• trunk/bp-themes/bp-default/members/members-loop.php (modified) (5 diffs)

trunk/bp-themes/bp-default/members/members-loop.php

@@ -1 @@
-<?php /* Querystring is set via AJAX in _inc/ajax.php - bp_dtheme_object_filter() */ ?>
+<?php
 
-<?php do_action( 'bp_before_members_loop' ) ?>
+/**
+ * BuddyPress - Members Loop
+ *
+ * Querystring is set via AJAX in _inc/ajax.php - bp_dtheme_object_filter()
+ *
+ * @package BuddyPress
+ * @subpackage bp-default
+ */
+
+?>
+
+<?php do_action( 'bp_before_members_loop' ); ?>
 
 <?php if ( bp_has_members( bp_ajax_querystring( 'members' ) ) ) : ?>
@@ -8 +19 @@
 
         <div class="pag-count" id="member-dir-count-top">
-            <?php bp_members_pagination_count() ?>
+
+            <?php bp_members_pagination_count(); ?>
+
         </div>
 
         <div class="pagination-links" id="member-dir-pag-top">
-            <?php bp_members_pagination_links() ?>
+
+            <?php bp_members_pagination_links(); ?>
+
         </div>
 
     </div>
 
-    <?php do_action( 'bp_before_directory_members_list' ) ?>
+    <?php do_action( 'bp_before_directory_members_list' ); ?>
 
     <ul id="members-list" class="item-list" role="main">
+
     <?php while ( bp_members() ) : bp_the_member(); ?>
 
         <li>
             <div class="item-avatar">
-                <a href="<?php bp_member_permalink() ?>"><?php bp_member_avatar() ?></a>
+                <a href="<?php bp_member_permalink(); ?>"><?php bp_member_avatar(); ?></a>
             </div>
 
             <div class="item">
                 <div class="item-title">
-                    <a href="<?php bp_member_permalink() ?>"><?php bp_member_name() ?></a>
+                    <a href="<?php bp_member_permalink(); ?>"><?php bp_member_name(); ?></a>
 
                     <?php if ( bp_get_member_latest_update() ) : ?>
 
-                        <span class="update"> - <?php bp_member_latest_update( 'length=10' ) ?></span>
+                        <span class="update"> - <?php bp_member_latest_update( 'length=10' ); ?></span>
 
                     <?php endif; ?>
@@ -39 +55 @@
                 </div>
 
-                <div class="item-meta"><span class="activity"><?php bp_member_last_active() ?></span></div>
+                <div class="item-meta"><span class="activity"><?php bp_member_last_active(); ?></span></div>
 
-                <?php do_action( 'bp_directory_members_item' ) ?>
+                <?php do_action( 'bp_directory_members_item' ); ?>
 
                 <?php
@@ -64 +80 @@
 
     <?php endwhile; ?>
+
     </ul>
 
-    <?php do_action( 'bp_after_directory_members_list' ) ?>
+    <?php do_action( 'bp_after_directory_members_list' ); ?>
 
-    <?php bp_member_hidden_fields() ?>
+    <?php bp_member_hidden_fields(); ?>
 
     <div id="pag-bottom" class="pagination">
 
         <div class="pag-count" id="member-dir-count-bottom">
-            <?php bp_members_pagination_count() ?>
+
+            <?php bp_members_pagination_count(); ?>
+
         </div>
 
         <div class="pagination-links" id="member-dir-pag-bottom">
-            <?php bp_members_pagination_links() ?>
+
+            <?php bp_members_pagination_links(); ?>
+
         </div>
 
@@ -85 +106 @@
 
     <div id="message" class="info">
-        <p><?php _e( "Sorry, no members were found.", 'buddypress' ) ?></p>
+        <p><?php _e( "Sorry, no members were found.", 'buddypress' ); ?></p>
     </div>
 
 <?php endif; ?>
 
-<?php do_action( 'bp_after_members_loop' ) ?>
+<?php do_action( 'bp_after_members_loop' ); ?>