Bug fixes

Author: PDowney

CHANGELOG.md

@@ -18,6 +18,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - Added `uninstall.php` to properly clean up plugin options from the database when the plugin is deleted
 - Added section headers ("Performance Optimizations", "Header Cleanup", "Additional Features") to the settings page for better UX
+
+### Code Quality
+
 - Added `phpcompatibility/phpcompatibility-wp` to `composer.json` `require-dev` (required by `phpcs.xml` ruleset)
 - Added `scripts` section to `composer.json` with `lint:php` and `analyze` commands for CI integration
 

enginescript-site-optimizer.php

@@ -71,10 +71,10 @@ function es_optimizer_init_plugin(): void {
 
 	// Initialize admin functionality.
 	es_optimizer_init_admin();
-	
+
 	// Initialize frontend optimizations.
 	es_optimizer_init_frontend_optimizations();
-	
+
 	// Initialize plugin settings link.
 	es_optimizer_init_plugin_links();
 }
@@ -90,7 +90,7 @@ function es_optimizer_activate_plugin(): void {
 	if ( false === get_option( 'es_optimizer_options' ) ) {
 		add_option( 'es_optimizer_options', es_optimizer_get_default_options() );
 	}
-	
+
 	// Clear any cached data.
 	es_optimizer_clear_options_cache();
 }
@@ -104,7 +104,7 @@ function es_optimizer_activate_plugin(): void {
 function es_optimizer_deactivate_plugin(): void {
 	// Clear any cached data on deactivation.
 	es_optimizer_clear_options_cache();
-	
+
 	// Note: We don't delete options on deactivation to preserve user settings.
 	// Options are only deleted on plugin uninstall.
 }
@@ -612,14 +612,14 @@ function es_optimizer_show_rejection_notice( array $rejected_domains, string $co
 	}
 
 	if ( 'preconnect' === $context ) {
-		$message    = sprintf(
+		$message = sprintf(
 			// translators: %s is the list of rejected domain names.
 			esc_html__( 'Some preconnect domains were rejected for security reasons: %s', 'enginescript-site-optimizer' ),
 			$rejected_message
 		);
 		$error_code = 'preconnect_security';
 	} else {
-		$message    = sprintf(
+		$message = sprintf(
 			// translators: %s is the list of rejected domain names.
 			esc_html__( 'Some DNS prefetch domains were rejected for security reasons: %s', 'enginescript-site-optimizer' ),
 			$rejected_message

readme.txt

@@ -58,6 +58,7 @@ No, the plugin has a simple interface where you can toggle features on and off.
 * **MODERNIZATION**: Replaced `isset()` ternaries with null coalescing operator
 * **CODE QUALITY**: Standardized all option-checking to use `empty()` with early-return pattern
 * **CODE QUALITY**: Consolidated duplicate domain validation functions into shared helpers
+* **CODE QUALITY**: Fixed trailing whitespace on blank lines and equals-sign alignment flagged by PHPCS
 * **CODE QUALITY**: Removed unnecessary static caching in preconnect and DNS prefetch output functions
 * **CODE QUALITY**: Replaced deprecated HTML `valign` attribute in settings form
 * **CI**: Updated Node.js from EOL version 16 to LTS version 20