Changeset 3810 for trunk/bp-themes/bp-default/groups/groups-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/groups/groups-loop.php (modified) (5 diffs)

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

@@ -1 @@
-<?php /* Querystring is set via AJAX in _inc/ajax.php - bp_dtheme_object_filter() */ ?>
+<?php
 
-<?php do_action( 'bp_before_groups_loop' ) ?>
+/**
+ * BuddyPress - Groups Loop
+ *
+ * Querystring is set via AJAX in _inc/ajax.php - bp_dtheme_object_filter()
+ *
+ * @package BuddyPress
+ * @subpackage bp-default
+ */
+
+?>
+
+<?php do_action( 'bp_before_groups_loop' ); ?>
 
 <?php if ( bp_has_groups( bp_ajax_querystring( 'groups' ) ) ) : ?>
@@ -8 +19 @@
 
         <div class="pag-count" id="group-dir-count-top">
-            <?php bp_groups_pagination_count() ?>
+
+            <?php bp_groups_pagination_count(); ?>
+
         </div>
 
         <div class="pagination-links" id="group-dir-pag-top">
-            <?php bp_groups_pagination_links() ?>
+
+            <?php bp_groups_pagination_links(); ?>
+
         </div>
 
     </div>
 
-    <?php do_action( 'bp_before_directory_groups_list' ) ?>
+    <?php do_action( 'bp_before_directory_groups_list' ); ?>
 
     <ul id="groups-list" class="item-list" role="main">
+
     <?php while ( bp_groups() ) : bp_the_group(); ?>
 
         <li>
             <div class="item-avatar">
-                <a href="<?php bp_group_permalink() ?>"><?php bp_group_avatar( 'type=thumb&width=50&height=50' ) ?></a>
+                <a href="<?php bp_group_permalink(); ?>"><?php bp_group_avatar( 'type=thumb&width=50&height=50' ); ?></a>
             </div>
 
             <div class="item">
-                <div class="item-title"><a href="<?php bp_group_permalink() ?>"><?php bp_group_name() ?></a></div>
-                <div class="item-meta"><span class="activity"><?php printf( __( 'active %s ago', 'buddypress' ), bp_get_group_last_active() ) ?></span></div>
+                <div class="item-title"><a href="<?php bp_group_permalink(); ?>"><?php bp_group_name(); ?></a></div>
+                <div class="item-meta"><span class="activity"><?php printf( __( 'active %s ago', 'buddypress' ), bp_get_group_last_active() ); ?></span></div>
 
-                <div class="item-desc"><?php bp_group_description_excerpt() ?></div>
+                <div class="item-desc"><?php bp_group_description_excerpt(); ?></div>
 
-                <?php do_action( 'bp_directory_groups_item' ) ?>
+                <?php do_action( 'bp_directory_groups_item' ); ?>
 
             </div>
@@ -39 +55 @@
             <div class="action">
 
-                <?php do_action( 'bp_directory_groups_actions' ) ?>
+                <?php do_action( 'bp_directory_groups_actions' ); ?>
 
                 <div class="meta">
 
-                    <?php bp_group_type() ?> / <?php bp_group_member_count() ?>
+                    <?php bp_group_type(); ?> / <?php bp_group_member_count(); ?>
 
                 </div>
@@ -53 +69 @@
 
     <?php endwhile; ?>
+
     </ul>
 
-    <?php do_action( 'bp_after_directory_groups_list' ) ?>
+    <?php do_action( 'bp_after_directory_groups_list' ); ?>
 
     <div id="pag-bottom" class="pagination">
 
         <div class="pag-count" id="group-dir-count-bottom">
-            <?php bp_groups_pagination_count() ?>
+
+            <?php bp_groups_pagination_count(); ?>
+
         </div>
 
         <div class="pagination-links" id="group-dir-pag-bottom">
-            <?php bp_groups_pagination_links() ?>
+
+            <?php bp_groups_pagination_links(); ?>
+
         </div>
 
@@ -72 +93 @@
 
     <div id="message" class="info">
-        <p><?php _e( 'There were no groups found.', 'buddypress' ) ?></p>
+        <p><?php _e( 'There were no groups found.', 'buddypress' ); ?></p>
     </div>
 
 <?php endif; ?>
 
-<?php do_action( 'bp_after_groups_loop' ) ?>
+<?php do_action( 'bp_after_groups_loop' ); ?>