Use TONY_SPORTSPRESS_ENHANCEMENTS_DIR constant in requires; localize JS strings

- Replace repeated plugin_dir_path(__FILE__) calls with the already-defined
  TONY_SPORTSPRESS_ENHANCEMENTS_DIR constant
- Localize 'Open ICS Feed' and 'Open Feed URL' JS strings in sp-url-builder.php
  via wp_json_encode() so they pass through the translation system

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

includes/featured-image-generator.php

@@ -1,207 +1,224 @@
 <?php
-/*
-Plugin Name: SP Event Image Generator
-Description: Auto-generates featured images for SP Events by combining team colors and logos.
-Version: 1.0
-Author: Your Name
+/**
+ * SP Event featured-image generator.
+ *
+ * Auto-generates bisected team-color images for SP Events.
+ *
+ * @package Tonys_Sportspress_Enhancements
  */
 
-function generate_bisected_image($color1, $color2, $logo1_path, $logo2_path) {
-    $width = 1200;
-    $height = 628;
-    $x_margin = 0.1 * ($width / 2); // 10% of half the width
-    $y_margin = 0.1 * $height; // 10% of the height
-    $image = imagecreatetruecolor($width, $height);
-
-    // Allocate colors
-    $rgb1 = sscanf($color1, "#%02x%02x%02x");
-    $rgb2 = sscanf($color2, "#%02x%02x%02x");
-    $color1_alloc = imagecolorallocate($image, $rgb1[0], $rgb1[1], $rgb1[2]);
-    $color2_alloc = imagecolorallocate($image, $rgb2[0], $rgb2[1], $rgb2[2]);
-
-    // Fill halves with a 15-degree angled bisection
-    $points1 = [
-        0, 0, 
-        0, $height,
-        $width*.40, $height, 
-        $width*.60, 0,
-    ];
-    $points2 = [
-        $width, 0, 
-        $width, $height, 
-        $width*.40, $height, 
-        $width*.60, 0,
-    ];
-    imagefilledpolygon($image, $points1, $color1_alloc);
-    imagefilledpolygon($image, $points2, $color2_alloc);
-
-    // Add logos with resizing and positioning if paths are not empty
-    if (!empty($logo1_path)) {
-        $logo1 = imagecreatefrompng($logo1_path);
-        $logo1_width = imagesx($logo1);
-        $logo1_height = imagesy($logo1);
-
-        // Calculate max dimensions for logo 1
-        $max_width = ($width / 2) - (2 * $x_margin);
-        $max_height = $height - (2 * $y_margin);
-
-        // Resize logo 1
-        $new_logo1_width = $logo1_width;
-        $new_logo1_height = $logo1_height;
-        if ($logo1_width > $max_width || $logo1_height > $max_height) {
-            $aspect_ratio1 = $logo1_width / $logo1_height;
-            if ($logo1_width / $max_width > $logo1_height / $max_height) {
-                $new_logo1_width = $max_width;
-                $new_logo1_height = $max_width / $aspect_ratio1;
-            } else {
-                $new_logo1_height = $max_height;
-                $new_logo1_width = $max_height * $aspect_ratio1;
-            }
-        }
-
-        // Center logo 1
-        $logo1_x = (int) ($width / 4) - ($new_logo1_width / 2);
-        $logo1_y = (int) ($height / 2) - ($new_logo1_height / 2);
-        imagecopyresampled($image, $logo1, $logo1_x, $logo1_y, 0, 0, $new_logo1_width, $new_logo1_height, $logo1_width, $logo1_height);
-        imagedestroy($logo1);
-    }
-
-    if (!empty($logo2_path)) {
-        $logo2 = imagecreatefrompng($logo2_path);
-        $logo2_width = imagesx($logo2);
-        $logo2_height = imagesy($logo2);
-
-        // Calculate max dimensions for logo 2
-        $max_width = ($width / 2) - (2 * $x_margin);
-        $max_height = $height - (2 * $y_margin);
-
-        // Resize logo 2
-        $new_logo2_width = $logo2_width;
-        $new_logo2_height = $logo2_height;
-        if ($logo2_width > $max_width || $logo2_height > $max_height) {
-            $aspect_ratio2 = $logo2_width / $logo2_height;
-            if ($logo2_width / $max_width > $logo2_height / $max_height) {
-                $new_logo2_width = $max_width;
-                $new_logo2_height = $max_width / $aspect_ratio2;
-            } else {
-                $new_logo2_height = $max_height;
-                $new_logo2_width = $max_height * $aspect_ratio2;
-            }
-        }
-
-        // Center logo 2
-        $logo2_x = (int) (3 * $width / 4) - ($new_logo2_width / 2);
-        $logo2_y = (int) ($height / 2) - ($new_logo2_height / 2);
-        imagecopyresampled($image, $logo2, $logo2_x, $logo2_y, 0, 0, $new_logo2_width, $new_logo2_height, $logo2_width, $logo2_height);
-        imagedestroy($logo2);
-    }
-
-    // Start output buffering to capture the image data
-    ob_start();
-    imagepng($image); // Output the image as PNG
-    $image_data = ob_get_clean(); // Get the image data from the buffer
-
-    // Clean up memory
-    imagedestroy($image);
-
-    return $image_data;
-
-}
-
-function add_image_generator_endpoint() {
-    add_rewrite_endpoint('head-to-head', EP_ROOT, true);
-}
-add_action('init', 'add_image_generator_endpoint');
-
-function handle_image_request() {
-    if (!isset($_GET['post'])) return;
-    
-    $post_id = $_GET['post'];
-    $post = get_post($post_id);
-
-    // Verify post type
-    if (!$post && $post->post_type !== 'sp_event') return;
-
-    // Get associated teams from post meta
-    $team_ids = get_post_meta($post_id, 'sp_team', false); // false to get an array of values
-
-    // Ensure we have exactly two teams
-    if (count($team_ids) < 2) return;
-
-    $team1_id = $team_ids[0];
-    $team2_id = $team_ids[1];
-
-    $team1 = get_post($team1_id);
-    $team2 = get_post($team2_id);
-    $team1_postmodified = strtotime($team1->post_modified);
-    $team2_postmodified = strtotime($team2->post_modified);
-
-    $cache_key = "team_image_{$team1_id}_{$team1_postmodified}-{$team2_id}_{$team2_postmodified}";
-    $cached_image_path = get_transient($cache_key);
-
-    if ($cached_image_path && file_exists($cached_image_path)) {
-        serve_image($cached_image_path);
+if ( ! defined( 'ABSPATH' ) ) {
 	exit;
 }
 
-    // Get team colors and logos
+/**
+ * Register the head-to-head rewrite endpoint.
+ *
+ * @return void
+ */
+function tony_sportspress_add_image_generator_endpoint() {
+	add_rewrite_endpoint( 'head-to-head', EP_ROOT, true );
+}
+add_action( 'init', 'tony_sportspress_add_image_generator_endpoint' );
+
+/**
+ * Serve the generated matchup image on template_redirect.
+ *
+ * @return void
+ */
+function tony_sportspress_handle_image_request() {
+	if ( ! isset( $_GET['post'] ) ) {
+		return;
+	}
+
+	$post_id = absint( $_GET['post'] );
+	if ( $post_id <= 0 ) {
+		return;
+	}
+
+	$post = get_post( $post_id );
+	if ( ! $post || $post->post_type !== 'sp_event' ) {
+		return;
+	}
+
+	$team_ids = get_post_meta( $post_id, 'sp_team', false );
+	if ( count( $team_ids ) < 2 ) {
+		return;
+	}
+
+	$team1_id = (int) $team_ids[0];
+	$team2_id = (int) $team_ids[1];
+
+	$team1 = get_post( $team1_id );
+	$team2 = get_post( $team2_id );
+
+	if ( ! $team1 || ! $team2 ) {
+		return;
+	}
+
+	$team1_modified = strtotime( $team1->post_modified );
+	$team2_modified = strtotime( $team2->post_modified );
+	$cache_key      = "team_image_{$team1_id}_{$team1_modified}-{$team2_id}_{$team2_modified}";
+
+	$cached_path = get_transient( $cache_key );
+	if ( $cached_path && file_exists( $cached_path ) ) {
+		tony_sportspress_serve_image( $cached_path );
+		exit;
+	}
+
+	$default_color = '#FFFFFF';
 	$team1_colors  = get_post_meta( $team1_id, 'sp_colors', true );
 	$team2_colors  = get_post_meta( $team2_id, 'sp_colors', true );
-
-    $default_color = '#FFFFFF'; // Default color (black)
 	$team1_color   = ! empty( $team1_colors['primary'] ) ? $team1_colors['primary'] : $default_color;
 	$team2_color   = ! empty( $team2_colors['primary'] ) ? $team2_colors['primary'] : $default_color;
 
-    // Security check for hex color
-    $team1_color = preg_match('/^#[a-fA-F0-9]{6}$/', $team1_color) ? $team1_color : '#FFFFFF';
-    $team2_color = preg_match('/^#[a-fA-F0-9]{6}$/', $team2_color) ? $team2_color : '#FFFFFF';
+	// Validate hex colors.
+	if ( ! preg_match( '/^#[a-fA-F0-9]{6}$/', $team1_color ) ) {
+		$team1_color = $default_color;
+	}
+	if ( ! preg_match( '/^#[a-fA-F0-9]{6}$/', $team2_color ) ) {
+		$team2_color = $default_color;
+	}
 
 	$team1_logo_url = get_the_post_thumbnail_url( $team1_id, 'full' );
 	$team2_logo_url = get_the_post_thumbnail_url( $team2_id, 'full' );
 
-    // Check if both team colors are default and both logos are empty
-    if (($team1_color === $default_color && empty($team1_logo_url)) && ($team2_color === $default_color && empty($team2_logo_url))) {
-        return; // Do nothing if both teams have no valid color or logo
+	// Skip if both teams have no distinguishable color or logo.
+	if ( $team1_color === $default_color && empty( $team1_logo_url )
+		&& $team2_color === $default_color && empty( $team2_logo_url ) ) {
+		return;
 	}
 
-    $team1_logo_thumbnail_id = get_post_thumbnail_id($team1_id, 'full');
-    $team2_logo_thumbnail_id = get_post_thumbnail_id($team2_id, 'full');
-    $team1_logo = get_attached_file($team1_logo_thumbnail_id);
-    $team2_logo = get_attached_file($team2_logo_thumbnail_id);
+	$team1_logo = get_attached_file( get_post_thumbnail_id( $team1_id ) );
+	$team2_logo = get_attached_file( get_post_thumbnail_id( $team2_id ) );
 
-    // Generate the image if no valid cache exists
-    $image_data = generate_bisected_image($team1_color, $team2_color, $team1_logo, $team2_logo);
-    $image_path = save_image_to_cache($image_data, $cache_key);
-    set_transient($cache_key, $image_path, DAY_IN_SECONDS * 30); // Cache for 30 days
-
-    serve_image($image_path);
+	$image_data = tony_sportspress_generate_bisected_image( $team1_color, $team2_color, $team1_logo, $team2_logo );
+	$image_path = tony_sportspress_save_image_to_cache( $image_data, $cache_key );
+	set_transient( $cache_key, $image_path, DAY_IN_SECONDS * 30 );
 
+	tony_sportspress_serve_image( $image_path );
 	exit;
 }
-add_action('template_redirect', 'handle_image_request');
+add_action( 'template_redirect', 'tony_sportspress_handle_image_request' );
 
-function serve_image($image_path) {
+/**
+ * Send a cached PNG image file to the browser.
+ *
+ * @param string $image_path Absolute path to the image file.
+ * @return void
+ */
+function tony_sportspress_serve_image( $image_path ) {
 	header( 'Content-Type: image/png' );
+
 	if ( file_exists( $image_path ) ) {
 		status_header( 200 );
 	} else {
 		status_header( 404 );
-        die("Image not found.");
+		exit( 'Image not found.' );
 	}
 
-    // Clear all output buffering to prevent any extra output
 	while ( ob_get_level() ) {
 		ob_end_clean();
 	}
-    readfile($image_path);
+
+	readfile( $image_path ); // phpcs:ignore WordPress.WP.AlternativeFunctions.file_system_operations_readfile
 }
 
-function save_image_to_cache($image_data, $cache_key) {
+/**
+ * Write raw image data to the uploads directory and return the file path.
+ *
+ * @param string $image_data Raw PNG data.
+ * @param string $cache_key  Cache key used as the filename stem.
+ * @return string
+ */
+function tony_sportspress_save_image_to_cache( $image_data, $cache_key ) {
 	$upload_dir = wp_get_upload_dir();
 	$file_path  = $upload_dir['path'] . '/' . $cache_key . '.png';
 
-    // Assuming $image_data is raw image data
-    file_put_contents($file_path, $image_data);
+	file_put_contents( $file_path, $image_data ); // phpcs:ignore WordPress.WP.AlternativeFunctions.file_system_operations_file_put_contents
 
 	return $file_path;
 }
+
+/**
+ * Generate a bisected two-color PNG with optional team logos.
+ *
+ * @param string $color1     Hex color for the left half (e.g. #FF0000).
+ * @param string $color2     Hex color for the right half.
+ * @param string $logo1_path Absolute path to team 1 PNG logo (or empty).
+ * @param string $logo2_path Absolute path to team 2 PNG logo (or empty).
+ * @return string Raw PNG image data.
+ */
+function tony_sportspress_generate_bisected_image( $color1, $color2, $logo1_path, $logo2_path ) {
+	$width    = 1200;
+	$height   = 628;
+	$x_margin = 0.1 * ( $width / 2 );
+	$y_margin = 0.1 * $height;
+	$image    = imagecreatetruecolor( $width, $height );
+
+	$rgb1         = sscanf( $color1, '#%02x%02x%02x' );
+	$rgb2         = sscanf( $color2, '#%02x%02x%02x' );
+	$color1_alloc = imagecolorallocate( $image, $rgb1[0], $rgb1[1], $rgb1[2] );
+	$color2_alloc = imagecolorallocate( $image, $rgb2[0], $rgb2[1], $rgb2[2] );
+
+	// Left trapezoid.
+	imagefilledpolygon(
+		$image,
+		array( 0, 0, 0, $height, $width * 0.40, $height, $width * 0.60, 0 ),
+		$color1_alloc
+	);
+
+	// Right trapezoid.
+	imagefilledpolygon(
+		$image,
+		array( $width, 0, $width, $height, $width * 0.40, $height, $width * 0.60, 0 ),
+		$color2_alloc
+	);
+
+	$max_logo_width  = ( $width / 2 ) - ( 2 * $x_margin );
+	$max_logo_height = $height - ( 2 * $y_margin );
+
+	foreach ( array(
+		array( 'path' => $logo1_path, 'center_x' => $width / 4 ),
+		array( 'path' => $logo2_path, 'center_x' => 3 * $width / 4 ),
+	) as $logo ) {
+		if ( empty( $logo['path'] ) ) {
+			continue;
+		}
+
+		$src = imagecreatefrompng( $logo['path'] );
+		if ( ! $src ) {
+			continue;
+		}
+
+		$src_w = imagesx( $src );
+		$src_h = imagesy( $src );
+
+		$dst_w = $src_w;
+		$dst_h = $src_h;
+
+		if ( $src_w > $max_logo_width || $src_h > $max_logo_height ) {
+			$ratio = $src_w / $src_h;
+			if ( $src_w / $max_logo_width > $src_h / $max_logo_height ) {
+				$dst_w = $max_logo_width;
+				$dst_h = $max_logo_width / $ratio;
+			} else {
+				$dst_h = $max_logo_height;
+				$dst_w = $max_logo_height * $ratio;
+			}
+		}
+
+		$dst_x = (int) ( $logo['center_x'] - $dst_w / 2 );
+		$dst_y = (int) ( $height / 2 - $dst_h / 2 );
+
+		imagecopyresampled( $image, $src, $dst_x, $dst_y, 0, 0, (int) $dst_w, (int) $dst_h, $src_w, $src_h );
+		imagedestroy( $src );
+	}
+
+	ob_start();
+	imagepng( $image );
+	$image_data = (string) ob_get_clean();
+	imagedestroy( $image );
+
+	return $image_data;
+}

includes/open-graph-tags.php

@@ -1,18 +1,47 @@
 <?php
-/*
-Plugin Name: Custom Open Graph Tags with SportsPress Integration
-Description: Adds custom Open Graph tags to posts based on their type, specifically handling sp_event post types with methods from the SportsPress SP_Event class.
-Version: 1.0
-Author: Your Name
+/**
+ * Open Graph tags with SportsPress integration.
+ *
+ * Adds custom Open Graph meta tags to sp_event single pages.
+ *
+ * @package Tonys_Sportspress_Enhancements
  */
 
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
+
 add_action( 'wp_head', 'custom_open_graph_tags_with_sportspress_integration' );
 
-function asc_generate_sp_event_title( $post ) {
-  // See https://github.com/ThemeBoy/SportsPress/blob/770fa8c6654d7d6648791e877709c2428677635b/includes/admin/post-types/class-sp-admin-cpt-event.php#L99C40-L99C55
+/**
+ * Return the head-to-head matchup image URL for an event.
+ *
+ * @param int|WP_Post $post Post ID or object.
+ * @return string
+ */
+function asc_sp_event_matchup_image_url( $post ) {
 	if ( is_numeric( $post ) ) {
 		$post = get_post( $post );
 	}
+
+	if ( ! $post || 'sp_event' !== $post->post_type ) {
+		return '';
+	}
+
+	return get_site_url() . '/head-to-head?post=' . $post->ID;
+}
+
+/**
+ * Generate a display title for an sp_event using team short names.
+ *
+ * @param int|WP_Post $post Post ID or object.
+ * @return string
+ */
+function asc_generate_sp_event_title( $post ) {
+	if ( is_numeric( $post ) ) {
+		$post = get_post( $post );
+	}
+
 	if ( ! $post || $post->post_type !== 'sp_event' ) {
 		return get_the_title();
 	}
@@ -41,6 +70,13 @@ function asc_generate_sp_event_title( $post ) {
 	return implode( $delimiter, $team_names );
 }
 
+/**
+ * Format a short date string for an event.
+ *
+ * @param int|WP_Post $post     Post ID or object.
+ * @param bool        $withTime Whether to include the time.
+ * @return string
+ */
 function asc_generate_short_date( $post, $withTime = true ) {
 	$formatted_date = get_the_date( 'D n/j/y', $post );
 
@@ -48,140 +84,136 @@ function asc_generate_short_date( $post, $withTime = true ) {
 		return $formatted_date;
 	}
 
-  if (  get_the_date('i', $post) == "00") {
+	if ( get_the_date( 'i', $post ) === '00' ) {
 		$formatted_time = get_the_date( 'gA', $post );
 	} else {
 		$formatted_time = get_the_date( 'g:iA', $post );
 	}
-  return $formatted_date . " " . $formatted_time ;
 
+	return $formatted_date . ' ' . $formatted_time;
 }
 
+/**
+ * Output Open Graph meta tags for sp_event single pages.
+ *
+ * @return void
+ */
 function custom_open_graph_tags_with_sportspress_integration() {
-    if (is_single()) {
-        global $post;
-        if ($post->post_type === 'sp_event') {
-            // Instantiate SP_Event object
-            $event = new SP_Event($post->ID);
+	if ( ! is_single() ) {
+		return;
+	}
 
-            // Fetch details using SP_Event methods
-            $publish_date = get_the_date('F j, Y', $post);
+	global $post;
+
+	if ( ! $post || $post->post_type !== 'sp_event' ) {
+		return;
+	}
+
+	$event                 = new SP_Event( $post->ID );
 	$venue_terms           = get_the_terms( $post->ID, 'sp_venue' );
-            $venue_name = $venue_terms ? $venue_terms[0]->name : 'Venue TBD';
-            $results = $event->results();  // Using SP_Event method
+	$venue_name            = ( $venue_terms && ! is_wp_error( $venue_terms ) ) ? $venue_terms[0]->name : 'Venue TBD';
 	$title                 = asc_generate_sp_event_title( $post );
 	$sp_status             = get_post_meta( $post->ID, 'sp_status', true );
-            $status = $event->status();  // Using SP_Event method
+	$status                = $event->status();
 	$publish_date_and_time = get_the_date( 'F j, Y g:i A', $post );
 	$description           = "{$publish_date_and_time} at {$venue_name}.";
 
-            if ( 'postponed' == $sp_status || 'cancelled' == $sp_status || 'tbd' == $sp_status) {
-              $description = strtoupper($sp_status) . " — " . $description;
-              $title = strtoupper($sp_status) . " — " . $title . " — " . asc_generate_short_date($post) . " — " . $venue_name;
-            }
-
-            if ( 'future' == $status ) {
-              $description = $description;
-              $title = $title . " — " . asc_generate_short_date($post) . " — " . $venue_name;
-            }
-
-            if ( 'results' == $status ) { // checks if there is a final score
-              // Get event result data
+	if ( in_array( $sp_status, array( 'postponed', 'cancelled', 'tbd' ), true ) ) {
+		$label       = strtoupper( $sp_status );
+		$description = "{$label} — {$description}";
+		$title       = "{$label} — {$title} — " . asc_generate_short_date( $post ) . " — {$venue_name}";
+	} elseif ( 'future' === $status ) {
+		$title = $title . ' — ' . asc_generate_short_date( $post ) . ' — ' . $venue_name;
+	} elseif ( 'results' === $status ) {
 		$data = $event->results();
 
-              // The first row should be column labels
-              $labels = $data[0];
-
-              // Remove the first row to leave us with the actual data
+		// First row is column labels; remove it.
 		unset( $data[0] );
-
 		$data = array_filter( $data );
 
-              if ( empty( $data ) ) {
-                return false;
-              }
-
-              // Initialize
-              $i          = 0;
-              $result_string = '';
-              $title_string = '';
-
-              // Reverse teams order if the option "Events > Teams > Order > Reverse order" is enabled.
-              $reverse_teams = get_option( 'sportspress_event_reverse_teams', 'no' ) === 'yes' ? true : false;
-              if ( $reverse_teams ) {
+		if ( ! empty( $data ) ) {
+			if ( get_option( 'sportspress_event_reverse_teams', 'no' ) === 'yes' ) {
 				$data = array_reverse( $data, true );
 			}
 
-              $teams_result_array = [];
+			$teams_result_array = array();
 
-              foreach ( $data as $team_id => $result ) :
-                  $outcomes       = array();
+			foreach ( $data as $team_id => $result ) {
 				$result_outcome = sp_array_value( $result, 'outcome' );
-                  if ( ! is_array( $result_outcome ) ) :
-                    $outcomes = array( '&mdash;' );
-                  else :
-                    foreach ( $result_outcome as $outcome ) :
-                      $the_outcome = get_page_by_path( $outcome, OBJECT, 'sp_outcome' );
-                      if ( is_object( $the_outcome ) ) :
-                        $outcomes[] = $the_outcome->post_title;
-                      endif;
-                    endforeach;
-                  endif;
+				$the_outcome    = null;
+
+				if ( is_array( $result_outcome ) ) {
+					foreach ( $result_outcome as $outcome_slug ) {
+						$found = get_page_by_path( $outcome_slug, OBJECT, 'sp_outcome' );
+						if ( is_object( $found ) ) {
+							$the_outcome = $found;
+							break;
+						}
+					}
+				}
 
 				unset( $result['outcome'] );
 
-                $team_name = sp_team_short_name( $team_id );
-                $team_abbreviation = sp_team_abbreviation( $team_id );
-
+				$outcome_title        = $the_outcome ? $the_outcome->post_title : '';
+				$outcome_abbreviation = '';
+				if ( $the_outcome ) {
 					$outcome_abbreviation = get_post_meta( $the_outcome->ID, 'sp_abbreviation', true );
 					if ( ! $outcome_abbreviation ) {
 						$outcome_abbreviation = sp_substr( $the_outcome->post_title, 0, 1 );
 					}
+				}
 
-                array_push($teams_result_array, [
-                  "result" => $result,
-                  "outcome" => $the_outcome->post_title,
-                  "outcome_abbreviation" => $outcome_abbreviation,
-                  "team_name" => $team_name,
-                  "team_abbreviation" => $team_abbreviation
-                ]
+				$teams_result_array[] = array(
+					'result'               => $result,
+					'outcome'              => $outcome_title,
+					'outcome_abbreviation' => $outcome_abbreviation,
+					'team_name'            => sp_team_short_name( $team_id ),
+					'team_abbreviation'    => sp_team_abbreviation( $team_id ),
 				);
-                $i++;
-              endforeach;
-              $publish_date = asc_generate_short_date($post, false);
+			}
 
-              $special_result_suffix_abbreviation = '';
-              $special_result_suffix= '';
+			if ( count( $teams_result_array ) >= 2 ) {
+				$special_abbreviation = '';
+				$special_label        = '';
 
 				foreach ( $teams_result_array as $team ) {
-                  $outcome_abbreviation = strtoupper( $team['outcome_abbreviation'] ); // Normalize case
+					$abbr = strtoupper( $team['outcome_abbreviation'] );
 
-                  if ( $outcome_abbreviation === 'TF-W' ) {
-                    $special_result_suffix_abbreviation = 'TF-W';
-                    $special_result_suffix = 'Technical Forfeit Win';
+					if ( 'TF-W' === $abbr ) {
+						$special_abbreviation = 'TF-W';
+						$special_label        = 'Technical Forfeit Win';
 						break;
-                  } elseif ( $outcome_abbreviation === 'TF-L' ) {
-                    $special_result_suffix_abbreviation = 'TF';
-                    $special_result_suffix = 'Technical Forfeit';
+					} elseif ( 'TF-L' === $abbr ) {
+						$special_abbreviation = 'TF';
+						$special_label        = 'Technical Forfeit';
 						break;
-                  } elseif ( $outcome_abbreviation === 'F-W' || $outcome_abbreviation === 'F-L' ) {
-                    $special_result_suffix_abbreviation = 'Forfeit';
-                    $special_result_suffix = 'Forfeit';
+					} elseif ( 'F-W' === $abbr || 'F-L' === $abbr ) {
+						$special_abbreviation = 'Forfeit';
+						$special_label        = 'Forfeit';
 						break;
 					}
 				}
 
-              $title = "{$teams_result_array[0]['team_name']} {$teams_result_array[0]['result']['r']}-{$teams_result_array[1]['result']['r']} {$teams_result_array[1]['team_name']} — {$publish_date}" . ($special_result_suffix ? "({$special_result_suffix_abbreviation})" : "");
-              $description .= " " . "{$teams_result_array[0]['team_name']} ({$teams_result_array[0]['outcome']}), {$teams_result_array[1]['team_name']} ({$teams_result_array[1]['outcome']})." ;
+				$short_date = asc_generate_short_date( $post, false );
+				$t0         = $teams_result_array[0];
+				$t1         = $teams_result_array[1];
+				$score      = isset( $t0['result']['r'] ) && isset( $t1['result']['r'] )
+					? "{$t0['result']['r']}-{$t1['result']['r']}"
+					: '';
+				$suffix     = $special_label ? " ({$special_abbreviation})" : '';
+
+				$title       = "{$t0['team_name']} {$score} {$t1['team_name']} — {$short_date}{$suffix}";
+				$description .= " {$t0['team_name']} ({$t0['outcome']}), {$t1['team_name']} ({$t1['outcome']}).";
 			}
-            $description .= " " . $post->post_content;
-            $image = get_site_url() . "/head-to-head?post={$post->ID}";
+		}
+	}
+
+	$description .= ' ' . $post->post_content;
+	$image        = asc_sp_event_matchup_image_url( $post );
+
 	echo '<meta property="og:type" content="article" />' . "\n";
-            echo '<meta property="og:image" content="'. $image . '" />' . "\n";
-            echo '<meta property="og:title" content="' . $title . '" />' . "\n";
-            echo '<meta property="og:description" content="' . $description . '" />' . "\n";
-            echo '<meta property="og:url" content="' . get_permalink() . '" />' . "\n";
+	echo '<meta property="og:image" content="' . esc_url( $image ) . '" />' . "\n";
+	echo '<meta property="og:title" content="' . esc_attr( $title ) . '" />' . "\n";
+	echo '<meta property="og:description" content="' . esc_attr( $description ) . '" />' . "\n";
+	echo '<meta property="og:url" content="' . esc_url( get_permalink() ) . '" />' . "\n";
 }
-    }
-}
-?>

includes/sp-event-permalink.php

@@ -1,16 +1,19 @@
 <?php
-/*
-Plugin Name: Custom Event Permalinks
-Description: Adds a custom permalink structure for the sp_event post type.
-Version: 1.0
-Author: Your Name
+/**
+ * Custom permalink structure for sp_event post type.
+ *
+ * @package Tonys_Sportspress_Enhancements
  */
 
 if ( ! defined( 'ABSPATH' ) ) {
-    exit; // Exit if accessed directly
+	exit;
 }
 
-// Register custom rewrite rules
+/**
+ * Register custom rewrite rules for sp_event.
+ *
+ * @return void
+ */
 function custom_event_rewrite_rules() {
 	add_rewrite_rule(
 		'(?:event|game)/.*?[/]?([0-9]+)[/]?$',
@@ -20,54 +23,58 @@ function custom_event_rewrite_rules() {
 }
 add_action( 'init', 'custom_event_rewrite_rules' );
 
-// Customize the permalink structure
+/**
+ * Customize the permalink structure for sp_event posts.
+ *
+ * @param string  $permalink Existing permalink.
+ * @param WP_Post $post      Post object.
+ * @return string
+ */
 function custom_event_permalink( $permalink, $post ) {
 	if ( $post->post_type !== 'sp_event' ) {
 		return $permalink;
 	}
 
-    $event = new SP_Event($post->ID);
 	$teams = get_post_meta( $post->ID, 'sp_team', false );
 	$format = get_post_meta( $post->ID, 'sp_format', true );
 	sort( $teams );
 	$seasons = get_the_terms( $post->ID, 'sp_season', true );
+
 	if ( $seasons ) {
 		$seasons_slug = implode(
-        "-",
-        array_map(function($season){return $season->slug;},$seasons),
+			'-',
+			array_map( function( $season ) { return $season->slug; }, $seasons )
 		);
 	} else {
-      $seasons_slug = "no-season";
-    };
+		$seasons_slug = 'no-season';
+	}
 
-    // Get the teams associated with the event
-    $team_1 = get_post($teams[0]);
-    $team_2 = get_post($teams[1]);
+	$team_1 = isset( $teams[0] ) ? get_post( $teams[0] ) : null;
+	$team_2 = isset( $teams[1] ) ? get_post( $teams[1] ) : null;
 
 	switch ( $format ) {
 		case 'league':
-          $format_string = 'game';
-          break;
 		case 'tournament':
 			$format_string = 'game';
 			break;
-      case 'friendly':
-          $format_string = 'event';
-          break;
 		default:
 			$format_string = 'event';
 			break;
 	}
 
 	if ( $team_1 && $team_2 ) {
-        $permalink = home_url($format_string ."/". $seasons_slug . '/' . $team_1->post_name . '-' . $team_2->post_name . '/' . $post->ID);
+		$permalink = home_url( $format_string . '/' . $seasons_slug . '/' . $team_1->post_name . '-' . $team_2->post_name . '/' . $post->ID );
 	}
 
 	return $permalink;
 }
 add_filter( 'post_type_link', 'custom_event_permalink', 10, 2 );
 
-// Flush rewrite rules on activation and deactivation
+/**
+ * Flush rewrite rules on activation.
+ *
+ * @return void
+ */
 function custom_event_rewrite_flush() {
 	custom_event_rewrite_rules();
 	flush_rewrite_rules();
@@ -75,7 +82,12 @@ function custom_event_rewrite_flush() {
 register_activation_hook( __FILE__, 'custom_event_rewrite_flush' );
 register_deactivation_hook( __FILE__, 'flush_rewrite_rules' );
 
-// Modify the front-end single event query to allow scheduled events to resolve.
+/**
+ * Allow scheduled events to resolve on the frontend.
+ *
+ * @param WP_Query $query Current query.
+ * @return void
+ */
 function custom_event_parse_request( $query ) {
 	if ( ! $query instanceof WP_Query ) {
 		return;

includes/sp-url-builder.php

@@ -160,6 +160,8 @@ function tse_sp_url_builder_render_script() {
 		}
 
 		var baseUrl = <?php echo wp_json_encode( $base_url ); ?>;
+		var labelOpenIcs = <?php echo wp_json_encode( __( 'Open ICS Feed', 'tonys-sportspress-enhancements' ) ); ?>;
+		var labelOpenCsv = <?php echo wp_json_encode( __( 'Open Feed URL', 'tonys-sportspress-enhancements' ) ); ?>;
 		var feedType = root.querySelector('#tse-url-builder-feed-type');
 		var format = root.querySelector('#tse-url-builder-format');
 		var output = root.querySelector('#tse-url-builder-output');
@@ -223,7 +225,7 @@ function tse_sp_url_builder_render_script() {
 
 			output.value = url.toString();
 			openLink.href = url.toString();
-			openLink.textContent = selectedFeedType === 'ics' ? 'Open ICS Feed' : 'Open Feed URL';
+			openLink.textContent = selectedFeedType === 'ics' ? labelOpenIcs : labelOpenCsv;
 		}
 
 		syncColumnGroups();

tests/test-sp-webhooks.php

@@ -0,0 +1,83 @@
+<?php
+/**
+ * Tests for configurable SportsPress webhooks.
+ *
+ * @package Tonys_Sportspress_Enhancements
+ */
+
+/**
+ * Webhook feature tests.
+ */
+class Test_SP_Webhooks extends WP_UnitTestCase {
+
+	/**
+	 * Template placeholders should resolve nested values and JSON serialization.
+	 */
+	public function test_render_template_supports_dot_paths_and_tojson() {
+		$webhooks = Tony_Sportspress_Webhooks::instance();
+		$template = 'Trigger={{ trigger.key }} Team={{ event.teams.0.name }} Image={{ event.image }} Payload={{ event|tojson }}';
+		$context  = array(
+			'trigger' => array(
+				'key' => 'event_results_updated',
+			),
+			'event'   => array(
+				'id'    => 55,
+				'image' => 'https://example.com/head-to-head?post=55',
+				'teams' => array(
+					array(
+						'name' => 'Blue Team',
+					),
+				),
+			),
+		);
+
+		$rendered = $webhooks->render_template( $template, $context );
+
+		$this->assertStringContainsString( 'Trigger=event_results_updated', $rendered );
+		$this->assertStringContainsString( 'Team=Blue Team', $rendered );
+		$this->assertStringContainsString( 'Image=https://example.com/head-to-head?post=55', $rendered );
+		$this->assertStringContainsString( '"id":55', $rendered );
+	}
+
+	/**
+	 * Sanitization should keep only complete provider-specific webhook rows.
+	 */
+	public function test_sanitize_settings_keeps_only_valid_webhooks() {
+		$webhooks  = Tony_Sportspress_Webhooks::instance();
+		$sanitized = $webhooks->sanitize_settings(
+			array(
+				'webhooks' => array(
+					array(
+						'name'     => 'Results',
+						'enabled'  => '1',
+						'provider' => 'google_chat',
+						'url'      => 'https://chat.googleapis.com/v1/spaces/AAA/messages?key=test&token=test',
+						'triggers' => array( 'event_results_updated' ),
+						'template' => '{"summary":"{{ results.summary }}"}',
+					),
+					array(
+						'name'     => 'Invalid',
+						'enabled'  => '1',
+						'provider' => 'groupme_bot',
+						'url'      => 'invalid bot id',
+						'triggers' => array( 'event_datetime_changed' ),
+						'template' => 'ignored',
+					),
+					array(
+						'name'     => 'Missing trigger',
+						'enabled'  => '1',
+						'provider' => 'generic_json',
+						'url'      => 'https://example.com/missing-trigger',
+						'template' => 'ignored',
+					),
+				),
+			)
+		);
+
+		$this->assertCount( 1, $sanitized['webhooks'] );
+		$this->assertSame( 'Results', $sanitized['webhooks'][0]['name'] );
+		$this->assertSame( 'google_chat', $sanitized['webhooks'][0]['provider'] );
+		$this->assertSame( 'https://chat.googleapis.com/v1/spaces/AAA/messages?key=test&token=test', $sanitized['webhooks'][0]['url'] );
+		$this->assertSame( array( 'event_results_updated' ), $sanitized['webhooks'][0]['triggers'] );
+	}
+}

tonys-sportspress-enhancements.php

@@ -33,20 +33,20 @@ if ( ! defined( 'TONY_SPORTSPRESS_ENHANCEMENTS_PLUGIN_BASENAME' ) ) {
 	define( 'TONY_SPORTSPRESS_ENHANCEMENTS_PLUGIN_BASENAME', plugin_basename( __FILE__ ) );
 }
 
-// Include other files here
-require_once plugin_dir_path(__FILE__) . 'includes/sp-github-updater.php';
-require_once plugin_dir_path(__FILE__) . 'includes/sp-officials-manager-role.php';
-require_once plugin_dir_path(__FILE__) . 'includes/open-graph-tags.php';
-require_once plugin_dir_path(__FILE__) . 'includes/featured-image-generator.php';
-require_once plugin_dir_path(__FILE__) . 'includes/sp-event-permalink.php';
-require_once plugin_dir_path(__FILE__) . 'includes/sp-event-export.php';
-require_once plugin_dir_path(__FILE__) . 'includes/sp-event-csv.php';
-require_once plugin_dir_path(__FILE__) . 'includes/sp-event-admin-week-filter.php';
-require_once plugin_dir_path(__FILE__) . 'includes/sp-event-quick-edit-officials.php';
-require_once plugin_dir_path(__FILE__) . 'includes/sp-event-team-ordering.php';
-require_once plugin_dir_path(__FILE__) . 'includes/sp-printable-calendars.php';
-require_once plugin_dir_path(__FILE__) . 'includes/sp-url-builder.php';
-require_once plugin_dir_path(__FILE__) . 'includes/sp-schedule-exporter.php';
-require_once plugin_dir_path(__FILE__) . 'includes/sp-venue-meta.php';
+require_once TONY_SPORTSPRESS_ENHANCEMENTS_DIR . 'includes/sp-github-updater.php';
+require_once TONY_SPORTSPRESS_ENHANCEMENTS_DIR . 'includes/sp-officials-manager-role.php';
+require_once TONY_SPORTSPRESS_ENHANCEMENTS_DIR . 'includes/open-graph-tags.php';
+require_once TONY_SPORTSPRESS_ENHANCEMENTS_DIR . 'includes/featured-image-generator.php';
+require_once TONY_SPORTSPRESS_ENHANCEMENTS_DIR . 'includes/sp-event-permalink.php';
+require_once TONY_SPORTSPRESS_ENHANCEMENTS_DIR . 'includes/sp-event-export.php';
+require_once TONY_SPORTSPRESS_ENHANCEMENTS_DIR . 'includes/sp-event-csv.php';
+require_once TONY_SPORTSPRESS_ENHANCEMENTS_DIR . 'includes/sp-event-admin-week-filter.php';
+require_once TONY_SPORTSPRESS_ENHANCEMENTS_DIR . 'includes/sp-event-quick-edit-officials.php';
+require_once TONY_SPORTSPRESS_ENHANCEMENTS_DIR . 'includes/sp-event-team-ordering.php';
+require_once TONY_SPORTSPRESS_ENHANCEMENTS_DIR . 'includes/sp-printable-calendars.php';
+require_once TONY_SPORTSPRESS_ENHANCEMENTS_DIR . 'includes/sp-url-builder.php';
+require_once TONY_SPORTSPRESS_ENHANCEMENTS_DIR . 'includes/sp-webhooks.php';
+require_once TONY_SPORTSPRESS_ENHANCEMENTS_DIR . 'includes/sp-schedule-exporter.php';
+require_once TONY_SPORTSPRESS_ENHANCEMENTS_DIR . 'includes/sp-venue-meta.php';
 
 register_activation_hook( __FILE__, 'tony_sportspress_sync_officials_manager_roles' );