Improved filtering GET parameters #99

php/classes/class-ssp-stats.php

@@ -258,8 +258,10 @@ public function set_filters() {
 		}
 
 		// Set episode selection for charts
+		// Can be either integer or string 'all'
 		if ( isset( $_GET['episode'] ) ) {
-			$this->episode = sanitize_text_field( $_GET['episode'] );
+			$episode       = sanitize_text_field( $_GET['episode'] );
+			$this->episode = is_numeric( $episode ) ? intval( $episode ) : 'all';
 		}
 
 		// Set filter selection for charts
@@ -285,13 +287,13 @@ public function load_episode_ids () {
 						if( $this->episode_ids ) {
 							$this->episode_ids .= ',';
 						}
-						$this->episode_ids .= $episode->ID;
+						$this->episode_ids .= intval( $episode->ID );
 					}
 				}
 			break;
 
 			case 'episode':
-				if( 'all' != $this->episode ) {
+				if ( 'all' != $this->episode && is_integer( $this->episode ) ) {
 					$this->episode_ids = $this->episode;
 				}
 			break;
@@ -520,7 +522,7 @@ public function stats_page () {
 				}
 			break;
 			case 'episode':
-				if( 'all' != $this->episode ) {
+				if ( 'all' != $this->episode && is_integer( $this->episode ) ) {
 					$episode_name = get_the_title( $this->episode );
 					if( $episode_name ) {
 						$title_tail = sprintf( __( 'for Episode: %s', 'seriously-simple-stats' ), '<u>' . $episode_name . '</u>' );