From 3e8d6590f6bc779cf9765bb916b9d07b6b5a10df Mon Sep 17 00:00:00 2001 From: Daniel Dudzic Date: Fri, 7 Mar 2025 14:37:25 +0100 Subject: [PATCH] Fix linting issues --- modules/ppcp-settings/services.php | 2 +- .../ppcp-settings/src/Service/GatewayRedirectService.php | 9 +++++++-- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/modules/ppcp-settings/services.php b/modules/ppcp-settings/services.php index 64b6aa04f..71ad9a724 100644 --- a/modules/ppcp-settings/services.php +++ b/modules/ppcp-settings/services.php @@ -567,7 +567,7 @@ return array( $container->get( 'settings.data.todos' ) ); }, - 'settings.service.gateway-redirect' => static function (): GatewayRedirectService { + 'settings.service.gateway-redirect' => static function (): GatewayRedirectService { return new GatewayRedirectService(); }, ); diff --git a/modules/ppcp-settings/src/Service/GatewayRedirectService.php b/modules/ppcp-settings/src/Service/GatewayRedirectService.php index c63ee896a..7b043d04f 100644 --- a/modules/ppcp-settings/src/Service/GatewayRedirectService.php +++ b/modules/ppcp-settings/src/Service/GatewayRedirectService.php @@ -85,9 +85,14 @@ class GatewayRedirectService { // Get current URL parameters. // phpcs:disable WordPress.Security.NonceVerification.Recommended - $page = isset( $_GET['page'] ) ? $this->sanitize_get_param( $_GET['page'] ) : ''; - $tab = isset( $_GET['tab'] ) ? $this->sanitize_get_param( $_GET['tab'] ) : ''; + // phpcs:disable WordPress.Security.ValidatedSanitizedInput.InputNotSanitized + // phpcs:disable WordPress.Security.ValidatedSanitizedInput.MissingUnslash + // The sanitize_get_param method handles unslashing and sanitization internally. + $page = isset( $_GET['page'] ) ? $this->sanitize_get_param( $_GET['page'] ) : ''; + $tab = isset( $_GET['tab'] ) ? $this->sanitize_get_param( $_GET['tab'] ) : ''; $section = isset( $_GET['section'] ) ? $this->sanitize_get_param( $_GET['section'] ) : ''; + // phpcs:enable WordPress.Security.ValidatedSanitizedInput.MissingUnslash + // phpcs:enable WordPress.Security.ValidatedSanitizedInput.InputNotSanitized // phpcs:enable WordPress.Security.NonceVerification.Recommended // Check if we're on a WooCommerce settings page and checkout tab.