From c20f8c64a81217318b7aa6581577d73ef8a4bd6b Mon Sep 17 00:00:00 2001
From: Brian Miyaji <brian@themeboy.com>
Date: Tue, 6 Jun 2017 16:28:24 +1000
Subject: [PATCH] Add team gallery shortcode to editor button with sorting
 option

---
 includes/class-sp-ajax.php           | 64 ++++++++++++++++++++++++++++
 includes/sp-formatting-functions.php | 11 +++++
 templates/team-gallery.php           | 14 +-----
 3 files changed, 77 insertions(+), 12 deletions(-)

diff --git a/includes/class-sp-ajax.php b/includes/class-sp-ajax.php
index 95c7788f..c1bfe4d5 100644
--- a/includes/class-sp-ajax.php
+++ b/includes/class-sp-ajax.php
@@ -31,6 +31,7 @@ class SP_AJAX {
 			'event_list_shortcode' => false,
 			'event_blocks_shortcode' => false,
 			'team_standings_shortcode' => false,
+			'team_gallery_shortcode' => false,
 			'player_details_shortcode' => false,
 			'player_statistics_shortcode' => false,
 			'player_list_shortcode' => false,
@@ -682,6 +683,64 @@ class SP_AJAX {
 		die();
 	}
 
+	/**
+	 * AJAX team_gallery shortcode
+	 */
+	public function team_gallery_shortcode() {
+		?>
+		<div class="wrap sp-thickbox-content" id="sp-thickbox-team_gallery">
+			<p>
+				<label>
+					<?php _e( 'Title:', 'sportspress' ); ?>
+					<input class="regular-text" type="text" name="title">
+				</label>
+			</p>
+			<p>
+				<label>
+					<?php printf( __( 'Select %s:', 'sportspress' ), __( 'League Table', 'sportspress' ) ); ?>
+					<?php
+					$args = array(
+						'post_type' => 'sp_table',
+						'name' => 'id',
+						'values' => 'ID',
+					);
+					sp_dropdown_pages( $args );
+					?>
+				</label>
+			</p>
+			<p>
+				<label>
+					<?php _e( 'Number of teams to show:', 'sportspress' ); ?>
+					<input type="text" size="3" name="number" id="number" value="5">
+				</label>
+			</p>
+			<p>
+				<label>
+					<?php _e( 'Order by', 'sportspress' ); ?>:
+					<select name="orderby">
+						<option value="default"><?php _e( 'Rank', 'sportspress' ); ?></option>
+						<option value="name"><?php _e( 'Alphabetical', 'sportspress' ); ?></option>
+						<option value="rand"><?php _e( 'Random', 'sportspress' ); ?></option>
+					</select>
+				</label>
+			</p>
+			<p>
+				<label>
+					<input type="checkbox" name="show_full_table_link" id="show_full_table_link">
+					<?php _e( 'Display link to view full table', 'sportspress' ); ?>
+				</label>
+			</p>
+			<?php do_action( 'sportspress_ajax_shortcode_form', 'league-table' ); ?>
+			<p class="submit">
+				<input type="button" class="button-primary" value="<?php _e( 'Insert Shortcode', 'sportspress' ); ?>" onclick="insertSportsPress('team_gallery');" />
+				<a class="button-secondary" onclick="tb_remove();" title="<?php _e( 'Cancel', 'sportspress' ); ?>"><?php _e( 'Cancel', 'sportspress' ); ?></a>
+			</p>
+		</div>
+		<?php
+		self::scripts();
+		die();
+	}
+
 	/**
 	 * AJAX player_details shortcode
 	 */
@@ -977,6 +1036,11 @@ class SP_AJAX {
                     args.columns = $div.find('[name="columns[]"]:checked').map(function() { return this.value; }).get().join(',');
                     args.show_team_logo = $div.find('[name=show_team_logo]:checked').length;
                     args.show_full_table_link = $div.find('[name=show_full_table_link]:checked').length;
+                } else if ( 'team_gallery' == type ) {
+                    args.title = $div.find('[name=title]').val();
+                    args.number = $div.find('[name=number]').val();
+                    args.orderby = $div.find('[name=orderby]').val();
+                    args.show_full_table_link = $div.find('[name=show_full_table_link]:checked').length;
                 } else if ( 'player_list' == type ) {
                     args.title = $div.find('[name=title]').val();
                     args.number = $div.find('[name=number]').val();
diff --git a/includes/sp-formatting-functions.php b/includes/sp-formatting-functions.php
index 95420db1..c60d0e9f 100644
--- a/includes/sp-formatting-functions.php
+++ b/includes/sp-formatting-functions.php
@@ -96,6 +96,17 @@ function sp_sort_random() {
 	return mt_rand( 0, 1 );
 }
 
+/**
+ * Sort array by name field.
+ *
+ * @access public
+ * @param array $array
+ * @return bool
+ */
+function sp_sort_by_name( $a, $b ) {
+	return strcmp( $a['name'], $b['name'] );
+}
+
 /**
  * let_to_num function.
  *
diff --git a/templates/team-gallery.php b/templates/team-gallery.php
index 5811d590..8fa4deae 100644
--- a/templates/team-gallery.php
+++ b/templates/team-gallery.php
@@ -15,7 +15,6 @@ $defaults = array(
 	'title' => false,
 	'number' => -1,
 	'orderby' => 'default',
-	'order' => 'ASC',
 	'itemtag' => 'dl',
 	'icontag' => 'dt',
 	'captiontag' => 'dd',
@@ -57,19 +56,10 @@ $data = $table->data();
 // Remove the first row to leave us with the actual data
 unset( $data[0] );
 
-if ( $orderby == 'default' ):
-	$orderby = $table->orderby;
-	$order = $table->order;
+if ( $orderby == 'name' ):
+	uasort( $data, 'sp_sort_by_name' );
 elseif ( $orderby == 'rand' ):
 	uasort( $data, 'sp_sort_random' );
-else:
-	$table->priorities = array(
-		array(
-			'key' => $orderby,
-			'order' => $order,
-		),
-	);
-	uasort( $data, array( $table, 'sort' ) );
 endif;
 
 if ( $title )