From e848c4b3e67a107f52642d8ec60d173288c5887f Mon Sep 17 00:00:00 2001
From: Brian Miyaji <brian@themeboy.com>
Date: Sat, 2 Apr 2016 03:56:53 +1100
Subject: [PATCH] Add visibility icons to config page

---
 assets/css/admin.css                          |  8 +++++
 .../class-sp-meta-box-performance-details.php | 21 ++++++++++++
 .../class-sp-meta-box-statistic-details.php   |  2 +-
 includes/admin/views/html-admin-config.php    | 33 +++++++++++++++++--
 4 files changed, 61 insertions(+), 3 deletions(-)

diff --git a/assets/css/admin.css b/assets/css/admin.css
index c14e3047..823db4e9 100644
--- a/assets/css/admin.css
+++ b/assets/css/admin.css
@@ -155,6 +155,14 @@
 	text-align: right;
 }
 
+.sp-admin-config-table .sp-visibility-icon {
+	color: #ddd;
+}
+
+.sp-admin-config-table .sp-visibility-icon-visible {
+	color: #555;
+}
+
 .sportspress table.form-table .sp-color-option {
 	max-width: 225px;
 }
diff --git a/includes/admin/post-types/meta-boxes/class-sp-meta-box-performance-details.php b/includes/admin/post-types/meta-boxes/class-sp-meta-box-performance-details.php
index a7464234..fd4c36d7 100644
--- a/includes/admin/post-types/meta-boxes/class-sp-meta-box-performance-details.php
+++ b/includes/admin/post-types/meta-boxes/class-sp-meta-box-performance-details.php
@@ -29,6 +29,11 @@ class SP_Meta_Box_Performance_Details extends SP_Meta_Box_Config {
 		} else {
 			$readonly = false;
 		}
+
+		// Options
+		$visibility_options = apply_filters( 'sportspress_statistic_visibility_options', array( 'sp_event', 'sp_player' ) );
+		
+		// Post Meta
 		$section = get_post_meta( $post->ID, 'sp_section', true );
 		if ( '' === $section ) {
 			$section = -1;
@@ -37,6 +42,10 @@ class SP_Meta_Box_Performance_Details extends SP_Meta_Box_Config {
 		if ( '' === $format ) {
 			$format = 'number';
 		}
+		$visibility = get_post_meta( $post->ID, 'sp_visibility', true );
+		if ( ! is_array( $visibility ) ) {
+			$visibility = $visibility_options;
+		}
 		?>
 		<p><strong><?php _e( 'Variable', 'sportspress' ); ?></strong></p>
 		<p>
@@ -65,6 +74,17 @@ class SP_Meta_Box_Performance_Details extends SP_Meta_Box_Config {
 				?>
 			</select>
 		</p>
+		<p><strong><?php _e( 'Visibility', 'sportspress' ); ?></strong></p>
+		<ul class="categorychecklist form-no-clear">
+			<?php foreach ( $visibility_options as $option ) { $object = get_post_type_object( $option ); ?>
+			<li>
+				<label class="selectit">
+					<input name="sp_visibility[]" id="sp_visibility_<?php echo $option; ?>" type="checkbox" value="<?php echo $option; ?>" <?php checked( in_array( $option, $visibility ) ); ?>>
+					<?php echo $object->labels->singular_name; ?>
+				</label>
+			</li>
+			<?php } ?>
+		</ul>
 		<?php
 	}
 
@@ -75,5 +95,6 @@ class SP_Meta_Box_Performance_Details extends SP_Meta_Box_Config {
 		self::delete_duplicate( $_POST );
 		update_post_meta( $post_id, 'sp_section', (int) sp_array_value( $_POST, 'sp_section', -1 ) );
 		update_post_meta( $post_id, 'sp_format', sp_array_value( $_POST, 'sp_format', 'number' ) );
+		update_post_meta( $post_id, 'sp_visibility', (array) sp_array_value( $_POST, 'sp_visibility', array() ) );
 	}
 }
\ No newline at end of file
diff --git a/includes/admin/post-types/meta-boxes/class-sp-meta-box-statistic-details.php b/includes/admin/post-types/meta-boxes/class-sp-meta-box-statistic-details.php
index 21963045..d8d5a9e4 100644
--- a/includes/admin/post-types/meta-boxes/class-sp-meta-box-statistic-details.php
+++ b/includes/admin/post-types/meta-boxes/class-sp-meta-box-statistic-details.php
@@ -28,7 +28,7 @@ class SP_Meta_Box_Statistic_Details extends SP_Meta_Box_Config {
 		$visibility = get_post_meta( $post->ID, 'sp_visibility', true );
 
 		// Options
-		$visibility_options = apply_filters( 'sportspress_statistic_visibility_options', array( 'sp_event', 'sp_player', 'sp_list' ) );
+		$visibility_options = apply_filters( 'sportspress_statistic_visibility_options', array( 'sp_event', 'sp_player' ) );
 
 		// Defaults
 		if ( '' === $precision ) $precision = 0;
diff --git a/includes/admin/views/html-admin-config.php b/includes/admin/views/html-admin-config.php
index 83ec3fb8..32666906 100644
--- a/includes/admin/views/html-admin-config.php
+++ b/includes/admin/views/html-admin-config.php
@@ -1,3 +1,12 @@
+<?php
+$visibility_options = apply_filters( 'sportspress_statistic_visibility_options', array( 'sp_event', 'sp_player' ) );
+$visibility_labels = array();
+foreach ( $visibility_options as $option ) {
+	$object = get_post_type_object( $option );
+	$visibility_labels[ $option ] = $object->labels->singular_name;
+}
+?>
+
 <div class="wrap sportspress sportspress-config-wrap">
 	<h2>
 		<?php _e( 'Configure', 'sportspress' ); ?>
@@ -152,7 +161,7 @@
 					<p class="description"><?php _e( 'Used for events.', 'sportspress' ); ?></p>
 				</th>
 			    <td class="forminp">
-					<legend class="screen-reader-text"><span><?php _e( 'Event Results', 'sportspress' ) ?></span></legend>
+					<legend class="screen-reader-text"><span><?php _e( 'Player Performance', 'sportspress' ) ?></span></legend>
 					<form>
 						<?php wp_nonce_field( 'sp-save-primary-performance', 'sp-primary-performance-nonce', false ); ?>
 						<table class="widefat sp-admin-config-table">
@@ -164,6 +173,7 @@
 									<th scope="col"><?php _e( 'Variable', 'sportspress' ); ?></th>
 									<th scope="col"><?php _e( 'Category', 'sportspress' ); ?></th>
 									<th scope="col"><?php _e( 'Format', 'sportspress' ); ?></th>
+									<th scope="col"><?php _e( 'Visibility', 'sportspress' ); ?></th>
 									<th scope="col"><?php _e( 'Description', 'sportspress' ); ?></th>
 									<th scope="col" class="edit"></th>
 								</tr>
@@ -185,6 +195,10 @@
 								</tr>
 							</tfoot>
 							<?php if ( $data ): $i = 0; foreach ( $data as $row ): ?>
+								<?php
+								$visibility = get_post_meta( $row->ID, 'sp_visibility', true );
+								if ( ! is_array( $visibility ) ) $visibility = $visibility_options;
+								?>
 								<tr<?php if ( $i % 2 == 0 ) echo ' class="alternate"'; ?>>
 									<td class="radio"><input type="radio" class="sp-primary-performance-option" id="sportspress_primary_performance_<?php echo $row->post_name; ?>" name="sportspress_primary_performance" value="<?php echo $row->post_name; ?>" <?php checked( $selection, $row->post_name ); ?>></td>
 									<td class="icon">
@@ -201,6 +215,11 @@
 									<td><code><?php echo $row->post_name; ?></code></td>
 									<td><?php echo sp_get_post_section( $row->ID ); ?></td>
 									<td><?php echo sp_get_post_format( $row->ID ); ?></td>
+									<td>
+										<?php foreach ( $visibility_options as $option ) { ?>
+											<span class="sp-icon-<?php echo $option; ?> sp-tip sp-visibility-icon<?php if ( in_array( $option, $visibility ) ) { ?> sp-visibility-icon-visible<?php } ?>" title="<?php echo sp_array_value( $visibility_labels, $option, $option ); ?>"></span>
+										<?php } ?>
+									</td>
 									<td><p class="description"><?php echo $row->post_excerpt; ?></p></td>
 									<td class="edit"><a class="button" href="<?php echo get_edit_post_link( $row->ID ); ?>"><?php _e( 'Edit', 'sportspress' ); ?></s></td>
 								</tr>
@@ -348,22 +367,32 @@
 								<th scope="col"><?php _e( 'Equation', 'sportspress' ); ?></th>
 								<th scope="col"><?php _e( 'Decimal Places', 'sportspress' ); ?></th>
 								<th scope="col"><?php _e( 'Category', 'sportspress' ); ?></th>
+								<th scope="col"><?php _e( 'Visibility', 'sportspress' ); ?></th>
 								<th scope="col"><?php _e( 'Description', 'sportspress' ); ?></th>
 								<th scope="col" class="edit"></th>
 							</tr>
 						</thead>
 						<?php if ( $data ): $i = 0; foreach ( $data as $row ): ?>
+							<?php
+							$visibility = get_post_meta( $row->ID, 'sp_visibility', true );
+							if ( ! is_array( $visibility ) ) $visibility = $visibility_options;
+							?>
 							<tr<?php if ( $i % 2 == 0 ) echo ' class="alternate"'; ?>>
 								<td class="row-title"><?php echo $row->post_title; ?></td>
 								<td><?php echo sp_get_post_equation( $row->ID ); ?></td>
 								<td><?php echo sp_get_post_precision( $row->ID ); ?></td>
 								<td><?php echo sp_get_post_section( $row->ID ); ?></td>
+								<td>
+									<?php foreach ( $visibility_options as $option ) { ?>
+										<span class="sp-icon-<?php echo $option; ?> sp-tip sp-visibility-icon<?php if ( in_array( $option, $visibility ) ) { ?> sp-visibility-icon-visible<?php } ?>" title="<?php echo sp_array_value( $visibility_labels, $option, $option ); ?>"></span>
+									<?php } ?>
+								</td>
 								<td><p class="description"><?php echo $row->post_excerpt; ?></p></td>
 								<td class="edit"><a class="button" href="<?php echo get_edit_post_link( $row->ID ); ?>"><?php _e( 'Edit', 'sportspress' ); ?></s></td>
 							</tr>
 						<?php $i++; endforeach; else: ?>
 							<tr class="alternate">
-								<td colspan="6"><?php _e( 'No results found.', 'sportspress' ); ?></td>
+								<td colspan="7"><?php _e( 'No results found.', 'sportspress' ); ?></td>
 							</tr>
 						<?php endif; ?>
 					</table>