From f02d49f6b4dd8f9f3b70a69786226e3c7dd9df21 Mon Sep 17 00:00:00 2001
From: Brian Miyaji <brian@themeboy.com>
Date: Sat, 31 May 2014 12:56:32 +1000
Subject: [PATCH] Add ability to link teams and limit list

Limit player list to current team players
---
 .../meta-boxes/class-sp-meta-box-list-data.php   |  6 ++++++
 .../class-sp-meta-box-list-details.php           |  2 +-
 .../admin/settings/class-sp-settings-players.php | 10 +++++++++-
 includes/class-sp-player-list.php                | 10 ++++++----
 templates/player-list.php                        | 16 +++++++++++++++-
 5 files changed, 37 insertions(+), 7 deletions(-)

diff --git a/includes/admin/post-types/meta-boxes/class-sp-meta-box-list-data.php b/includes/admin/post-types/meta-boxes/class-sp-meta-box-list-data.php
index e669511f..e0fe44da 100644
--- a/includes/admin/post-types/meta-boxes/class-sp-meta-box-list-data.php
+++ b/includes/admin/post-types/meta-boxes/class-sp-meta-box-list-data.php
@@ -51,6 +51,10 @@ class SP_Meta_Box_List_Data {
 					<tr>
 						<th>#</th>
 						<th><?php _e( 'Player', 'sportspress' ); ?></th>
+						<th><label for="sp_columns_team">
+							<input type="checkbox" name="sp_columns[]" value="team" id="sp_columns_team" <?php checked( ! is_array( $usecolumns ) || in_array( 'team', $usecolumns ) ); ?>>
+							<?php _e( 'Team', 'sportspress' ); ?>
+						</label></th>
 						<?php foreach ( $columns as $key => $label ): ?>
 							<th><label for="sp_columns_<?php echo $key; ?>">
 								<input type="checkbox" name="sp_columns[]" value="<?php echo $key; ?>" id="sp_columns_<?php echo $key; ?>" <?php checked( ! is_array( $usecolumns ) || in_array( $key, $usecolumns ) ); ?>>
@@ -65,6 +69,7 @@ class SP_Meta_Box_List_Data {
 						$i = 0;
 						foreach ( $data as $player_id => $player_stats ):
 							if ( !$player_id ) continue;
+							$team = get_post_meta( $player_id, 'sp_team', true );
 							$div = get_term( $player_id, 'sp_season' );
 							$number = get_post_meta( $player_id, 'sp_number', true );
 
@@ -85,6 +90,7 @@ class SP_Meta_Box_List_Data {
 										<a class="button button-primary sp-save"><?php _e( 'Save', 'sportspress' ); ?></a>
 									</span>
 								</td>
+								<td><?php echo get_the_title( $team ); ?></td>
 								<?php foreach( $columns as $column => $label ):
 									$value = sp_array_value( $player_stats, $column, '' );
 									$placeholder = sp_array_value( sp_array_value( $placeholders, $player_id, array() ), $column, 0 );
diff --git a/includes/admin/post-types/meta-boxes/class-sp-meta-box-list-details.php b/includes/admin/post-types/meta-boxes/class-sp-meta-box-list-details.php
index 89a64a1e..6b272cd0 100644
--- a/includes/admin/post-types/meta-boxes/class-sp-meta-box-list-details.php
+++ b/includes/admin/post-types/meta-boxes/class-sp-meta-box-list-details.php
@@ -102,7 +102,7 @@ class SP_Meta_Box_List_Details {
 			</p>
 			<p><strong><?php _e( 'Players', 'sportspress' ); ?></strong></p>
 			<?php
-			sp_post_checklist( $post->ID, 'sp_player', 'block', 'sp_team' );
+			sp_post_checklist( $post->ID, 'sp_player', 'block', 'sp_current_team' );
 			sp_post_adder( 'sp_player', __( 'Add New', 'sportspress' ) );
 			?>
 		</div>
diff --git a/includes/admin/settings/class-sp-settings-players.php b/includes/admin/settings/class-sp-settings-players.php
index 7e1eae2c..6787805a 100644
--- a/includes/admin/settings/class-sp-settings-players.php
+++ b/includes/admin/settings/class-sp-settings-players.php
@@ -5,7 +5,7 @@
  * @author 		ThemeBoy
  * @category 	Admin
  * @package 	SportsPress/Admin
- * @version     0.8
+ * @version     1.0
  */
 
 if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
@@ -60,6 +60,14 @@ class SP_Settings_Players extends SP_Settings_Page {
 				'type' 		=> 'checkbox',
 			),
 
+			array(
+				'title'     => __( 'Teams', 'sportspress' ),
+				'desc' 		=> __( 'Link teams', 'sportspress' ),
+				'id' 		=> 'sportspress_list_link_teams',
+				'default'	=> 'no',
+				'type' 		=> 'checkbox',
+			),
+
 			array(
 				'title'     => __( 'Pagination', 'sportspress' ),
 				'desc' 		=> __( 'Paginate', 'sportspress' ),
diff --git a/includes/class-sp-player-list.php b/includes/class-sp-player-list.php
index 74c1580c..3ada21d9 100644
--- a/includes/class-sp-player-list.php
+++ b/includes/class-sp-player-list.php
@@ -239,8 +239,6 @@ class SP_Player_List extends SP_Custom_Post {
 			$totals[ $player_id ]['last10'] = $last10;
 		endforeach;
 
-		$columns = array();
-
 		if ( ! empty( $column_groups ) ):
 
 			$args = array(
@@ -351,8 +349,12 @@ class SP_Player_List extends SP_Custom_Post {
 					unset( $columns[ $key ] );
 				endif;
 			endforeach;
-			$labels = array_merge( array( 'name' => SP()->text->string('Player') ), $columns );
-			$merged[0] = $labels;
+
+			$labels = array( 'name' => SP()->text->string('Player') );
+			if ( in_array( 'team', $this->columns ) )
+				$labels['team'] = __( 'Team', 'sportspress' );
+
+			$merged[0] = array_merge( $labels, $columns );
 			return $merged;
 		endif;
 	}
diff --git a/templates/player-list.php b/templates/player-list.php
index 35b73bc1..b3d527e5 100644
--- a/templates/player-list.php
+++ b/templates/player-list.php
@@ -18,6 +18,7 @@ $defaults = array(
 	'order' => 'ASC',
 	'show_all_players_link' => false,
 	'link_posts' => get_option( 'sportspress_list_link_players', 'yes' ) == 'yes' ? true : false,
+	'link_teams' => get_option( 'sportspress_list_link_teams', 'no' ) == 'yes' ? true : false,
 	'sortable' => get_option( 'sportspress_enable_sortable_tables', 'yes' ) == 'yes' ? true : false,
 	'responsive' => get_option( 'sportspress_enable_responsive_tables', 'yes' ) == 'yes' ? true : false,
 	'paginated' => get_option( 'sportspress_list_paginated', 'yes' ) == 'yes' ? true : false,
@@ -121,9 +122,22 @@ foreach ( $groups as $group ):
 		endif;
 
 		$output .= '<td class="data-name">' . $name . '</td>';
+		
+		if ( array_key_exists( 'team', $labels ) ):
+			$teams = get_post_meta( $player_id, 'sp_current_team' );
+			$team_names = array();
+			foreach ( $teams as $team ):
+				$team_name = get_the_title( $team );
+				if ( $link_teams ):
+					$team_name = '<a href="' . get_post_permalink( $team ) . '">' . $team_name . '</a>';
+				endif;
+				$team_names[] = $team_name;
+			endforeach;
+			$output .= '<td class="data-team">' . implode( ', ', $team_names ) . '</td>';
+		endif;
 
 		foreach( $labels as $key => $value ):
-			if ( $key == 'name' )
+			if ( in_array( $key, array( 'name', 'team' ) ) )
 				continue;
 			if ( ! is_array( $columns ) || in_array( $key, $columns ) )
 			$output .= '<td class="data-' . $key . '">' . sp_array_value( $row, $key, '&mdash;' ) . '</td>';