Enable player list grouping by position

includes/admin/post-types/meta-boxes/class-sp-meta-box-list-details.php

@@ -22,6 +22,7 @@ class SP_Meta_Box_List_Details {
 		$league_id = sp_get_the_term_id( $post->ID, 'sp_league', 0 );
 		$season_id = sp_get_the_term_id( $post->ID, 'sp_season', 0 );
 		$team_id = get_post_meta( $post->ID, 'sp_team', true );
+		$grouping = get_post_meta( $post->ID, 'sp_grouping', true );
 		$orderby = get_post_meta( $post->ID, 'sp_orderby', true );
 		$order = get_post_meta( $post->ID, 'sp_order', true );
 		?>
@@ -69,6 +70,13 @@ class SP_Meta_Box_List_Details {
 				endif;
 				?>
 			</p>
+			<p><strong><?php _e( 'Grouping', 'sportspress' ); ?></strong></p>
+			<p>
+			<select name="sp_grouping">
+				<option value="0"><?php _e( 'None', 'sportspress' ); ?></option>
+				<option value="position" <?php selected( $grouping, 'position' ); ?>><?php _e( 'Position', 'sportspress' ); ?></option>
+			</select>
+			</p>
 			<p><strong><?php _e( 'Sort by', 'sportspress' ); ?></strong></p>
 			<p>
 			<?php
@@ -82,9 +90,7 @@ class SP_Meta_Box_List_Details {
 				'selected' => $orderby,
 				'values' => 'slug',
 			);
-			if ( ! sp_dropdown_pages( $args ) ):
+			sp_dropdown_pages( $args );
-				sp_post_adder( 'sp_list', __( 'Add New', 'sportspress' ) );
-			endif;
 			?>
 			</p>
 			<p><strong><?php _e( 'Sort Order', 'sportspress' ); ?></strong></p>
@@ -110,6 +116,7 @@ class SP_Meta_Box_List_Details {
 		update_post_meta( $post_id, 'sp_team', sp_array_value( $_POST, 'sp_team', array() ) );
 		wp_set_post_terms( $post_id, sp_array_value( $_POST, 'sp_league', 0 ), 'sp_league' );
 		wp_set_post_terms( $post_id, sp_array_value( $_POST, 'sp_season', 0 ), 'sp_season' );
+		update_post_meta( $post_id, 'sp_grouping', sp_array_value( $_POST, 'sp_grouping', array() ) );
 		update_post_meta( $post_id, 'sp_orderby', sp_array_value( $_POST, 'sp_orderby', array() ) );
 		update_post_meta( $post_id, 'sp_order', sp_array_value( $_POST, 'sp_order', array() ) );
 		sp_update_post_meta_recursive( $post_id, 'sp_player', sp_array_value( $_POST, 'sp_player', array() ) );

templates/player-gallery-thumbnail.php

@@ -0,0 +1,50 @@
+<?php
+/**
+ * Player Gallery Thumbnail
+ *
+ * @author 		ThemeBoy
+ * @package 	SportsPress/Templates
+ * @version     0.8
+ */
+
+if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+
+$defaults = array(
+	'id' => null,
+	'performance' => array(),
+	'icontag' => 'dt',
+	'captiontag' => 'dd',
+	'caption' => null,
+	'size' => 'thumbnail',
+	'link_posts' => get_option( 'sportspress_list_link_players', 'yes' ) == 'yes' ? true : false,
+);
+
+extract( $defaults, EXTR_SKIP );
+
+// Add player number to caption if available
+$player_number = get_post_meta( $id, 'sp_number', true );
+if ( $player_number )
+	$caption = '<strong>' . $player_number . '</strong> ' . $caption;
+
+// Add caption tag if has caption
+if ( $captiontag && $caption )
+	$caption = '<' . $captiontag . ' class="wp-caption-text gallery-caption small-3 columns">' . wptexturize( $caption ) . '</' . $captiontag . '>';
+
+if ( $link_posts )
+	$caption = '<a href="' . get_permalink( $id ) . '">' . $caption . '</a>';
+
+if ( isset( $limit ) && $i >= $limit )
+	continue;
+
+if ( has_post_thumbnail( $id ) )
+	$thumbnail = get_the_post_thumbnail( $id, $size );
+else
+	$thumbnail = '<img width="150" height="150" src="http://www.gravatar.com/avatar/?s=150&d=mm&f=y" class="attachment-thumbnail wp-post-image">';
+
+echo "<{$itemtag} class='gallery-item'>";
+echo "
+	<{$icontag} class='gallery-icon portrait'>"
+		. '<a href="' . get_permalink( $id ) . '">' . $thumbnail . '</a>'
+	. "</{$icontag}>";
+echo $caption;
+echo "</{$itemtag}>";

templates/player-gallery.php

@@ -12,6 +12,7 @@ if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
 $defaults = array(
 	'id' => get_the_ID(),
 	'number' => -1,
+	'grouping' => null,
 	'orderby' => 'default',
 	'order' => 'ASC',
 	'itemtag' => 'dl',
@@ -52,7 +53,14 @@ $labels = $data[0];
 // Remove the first row to leave us with the actual data
 unset( $data[0] );
 
-if ( $orderby != 'default' ):
+if ( ! $grouping || $grouping == 'default' ):
+	$grouping = $list->grouping;
+endif;
+
+if ( $orderby == 'default' ):
+	$orderby = $list->orderby;
+	$order = $list->order;
+else:
 	$list->priorities = array(
 		array(
 			'key' => $orderby,
@@ -85,54 +93,50 @@ if ( apply_filters( 'use_default_gallery_style', true ) )
 	</style>";
 $size_class = sanitize_html_class( $size );
 $gallery_div = "<div id='$selector' class='gallery galleryid-{$id} gallery-columns-{$columns} gallery-size-{$size_class}'>";
-$output = apply_filters( 'gallery_style', $gallery_style . "\n\t\t" . $gallery_div );
+echo apply_filters( 'gallery_style', $gallery_style . "\n\t\t" . $gallery_div );
-
-$i = 0;
 
 if ( is_int( $number ) && $number > 0 )
 	$limit = $number;
 
-foreach( $data as $player_id => $performance ):
+if ( $grouping == 'position' ):
+	$groups = get_terms( 'sp_position' );
+else:
+	$group = new stdClass();
+	$group->term_id = null;
+	$group->name = null;
+	$group->slug = null;
+	$groups = array( $group );
+endif;
+
+foreach ( $groups as $group ):
+	$i = 0;
+
+	if ( ! empty( $group->name ) )
+		echo '<h3 class="sp-list-group-name">' . $group->name . '</h3>';
+
+	foreach( $data as $player_id => $performance ): if ( empty( $group->term_id ) || has_term( $group->term_id, 'sp_position', $player_id ) ):
 
 		$caption = get_the_title( $player_id );
 		$caption = trim( $caption );
 
-	// Add player number to caption if available
+	    sp_get_template( 'player-gallery-thumbnail.php', array(
-	$player_number = get_post_meta( $player_id, 'sp_number', true );
+	    	'id' => $player_id,
-	if ( $player_number )
+	    	'performance' => $performance,
-		$caption = '<strong>' . $player_number . '</strong> ' . $caption;
+	    	'itemtag' => $itemtag,
+	    	'icontag' => $icontag,
+	    	'captiontag' => $captiontag,
+	    	'caption' => $caption,
+	    	'size' => $size,
+	    	'link_posts' => $link_posts,
+	    ) );
 
-	// Add caption tag if has caption
+	endif; endforeach;
-	if ( $captiontag && $caption )
-		$caption = '<' . $captiontag . ' class="wp-caption-text gallery-caption">' . wptexturize( $caption ) . '</' . $captiontag . '>';
 
-	if ( $link_posts )
+	echo '<br style="clear: both;" />';
-		$caption = '<a href="' . get_permalink( $player_id ) . '">' . $caption . '</a>';
-
-	if ( isset( $limit ) && $i >= $limit )
-		continue;
-
-	if ( has_post_thumbnail( $player_id ) ):
-		$thumbnail = get_the_post_thumbnail( $player_id, $size );
-
-		$output .= "<{$itemtag} class='gallery-item'>";
-		$output .= "
-			<{$icontag} class='gallery-icon portrait'>"
-				. '<a href="' . get_permalink( $player_id ) . '">' . $thumbnail . '</a>'
-			. "</{$icontag}>";
-		$output .= $caption;
-		$output .= "</{$itemtag}>";
-		if ( $columns > 0 && ++$i % $columns == 0 )
-			$output .= '<br style="clear: both" />';
-	endif;
 
 endforeach;
 	
-$output .= "
+echo "</div>\n";
-		<br style='clear: both;' />
-	</div>\n";
 
 if ( $show_all_players_link )
-	$output .= '<a class="sp-player-list-link sp-view-all-link" href="' . get_permalink( $id ) . '">' . SP()->text->string('View all players') . '</a>';
+	echo '<a class="sp-player-list-link sp-view-all-link" href="' . get_permalink( $id ) . '">' . SP()->text->string('View all players') . '</a>';
-
-echo apply_filters( 'sportspress_player_gallery',  $output );

templates/player-list.php

@@ -13,6 +13,7 @@ $defaults = array(
 	'id' => get_the_ID(),
 	'number' => -1,
 	'performance' => null,
+	'grouping' => null,
 	'orderby' => 'default',
 	'order' => 'ASC',
 	'show_all_players_link' => false,
@@ -25,9 +26,6 @@ $defaults = array(
 
 extract( $defaults, EXTR_SKIP );
 
-$output = '<div class="sp-table-wrapper">' .
-	'<table class="sp-player-list sp-data-table' . ( $responsive ? ' sp-responsive-table' : '' ) . ( $sortable ? ' sp-sortable-table' : '' ) . ( $paginated ? ' sp-paginated-table' : '' ) . '" data-sp-rows="' . $rows . '">' . '<thead>' . '<tr>';
-
 $list = new SP_Player_List( $id );
 $data = $list->data();
 
@@ -37,7 +35,14 @@ $labels = $data[0];
 // Remove the first row to leave us with the actual data
 unset( $data[0] );
 
-if ( $orderby != 'default' ):
+if ( ! $grouping || $grouping == 'default' ):
+	$grouping = $list->grouping;
+endif;
+
+if ( $orderby == 'default' ):
+	$orderby = $list->orderby;
+	$order = $list->order;
+else:
 	$list->priorities = array(
 		array(
 			'key' => $orderby,
@@ -47,6 +52,25 @@ if ( $orderby != 'default' ):
 	uasort( $data, array( $list, 'sort' ) );
 endif;
 
+if ( $grouping == 'position' ):
+	$groups = get_terms( 'sp_position' );
+else:
+	$group = new stdClass();
+	$group->term_id = null;
+	$group->name = null;
+	$group->slug = null;
+	$groups = array( $group );
+endif;
+
+$output = '';
+
+foreach ( $groups as $group ):
+	if ( ! empty( $group->name ) )
+		$output .= '<h3 class="sp-list-group-name">' . $group->name . '</h3>';
+
+	$output .= '<div class="sp-table-wrapper">' .
+		'<table class="sp-player-list sp-data-table' . ( $responsive ? ' sp-responsive-table' : '' ) . ( $sortable ? ' sp-sortable-table' : '' ) . ( $paginated ? ' sp-paginated-table' : '' ) . '" data-sp-rows="' . $rows . '">' . '<thead>' . '<tr>';
+
 	if ( in_array( $orderby, array( 'number', 'name' ) ) ):
 		$output .= '<th class="data-number">#</th>';
 	else:
@@ -65,7 +89,8 @@ $i = 0;
 	if ( intval( $number ) > 0 )
 		$limit = $number;
 
-foreach( $data as $player_id => $row ):
+	foreach( $data as $player_id => $row ): if ( empty( $group->term_id ) || has_term( $group->term_id, 'sp_position', $player_id ) ):
+
 		if ( isset( $limit ) && $i >= $limit ) continue;
 
 		$name = sp_array_value( $row, 'name', null );
@@ -98,9 +123,10 @@ foreach( $data as $player_id => $row ):
 
 		$i++;
 
-endforeach;
+	endif; endforeach;
 
 	$output .= '</tbody>' . '</table>' . '</div>';
+endforeach;
 
 if ( $show_all_players_link )
 	$output .= '<a class="sp-player-list-link sp-view-all-link" href="' . get_permalink( $id ) . '">' . SP()->text->string('View all players') . '</a>';