Ensure that required data is defined before using it

Kaspars Dambis · Kaspars Dambis · commit 8418bdedade5 · 2024-03-26T12:27:56.000+02:00
diff --git a/search/includes/classes/class-search.php b/search/includes/classes/class-search.php
@@ -1188,8 +1188,8 @@ public function ep_handle_failed_request( $request, $response, $query, $type, $c
 			return;
 		}
 
-		if ( ! $is_cli ) {
-			global $wp;
+		global $wp;
+		if ( ! $is_cli && isset( $wp->query_vars ) && isset( $_SERVER['REQUEST_URI'] ) ) {
 			// phpcs:ignore WordPress.Security.ValidatedSanitizedInput.InputNotSanitized,WordPress.Security.ValidatedSanitizedInput.InputNotValidated
 			$request_url_for_logging = esc_url_raw( add_query_arg( $wp->query_vars, home_url( wp_unslash( $_SERVER['REQUEST_URI'] ) ) ) );
 		}

Original file line number	Diff line number	Diff line change
`@@ -1188,8 +1188,8 @@ public function ep_handle_failed_request( $request, $response, $query, $type, $c`
`1188`	`1188`	`return;`
`1189`	`1189`	`}`
`1190`	`1190`
`1191`		`- if ( ! $is_cli ) {`
`1192`		`- global $wp;`
	`1191`	`+ global $wp;`
	`1192`	`+ if ( ! $is_cli && isset( $wp->query_vars ) && isset( $_SERVER['REQUEST_URI'] ) ) {`
`1193`	`1193`	`// phpcs:ignore WordPress.Security.ValidatedSanitizedInput.InputNotSanitized,WordPress.Security.ValidatedSanitizedInput.InputNotValidated`
`1194`	`1194`	`$request_url_for_logging = esc_url_raw( add_query_arg( $wp->query_vars, home_url( wp_unslash( $_SERVER['REQUEST_URI'] ) ) ) );`
`1195`	`1195`	`}`