Add an extra filter to fix the memory caching bug in Google Sheets (#594)

* Add an extra filter to fix the memory caching bug

* Fix the psalm erro

* Fix psalm again

Author: ingeniumed

inc/Integrations/Google/Sheets/GoogleSheetsIntegration.php

@@ -11,6 +11,7 @@
 class GoogleSheetsIntegration {
 	public static function init(): void {
 		add_action( 'init', [ __CLASS__, 'register_blocks' ], 10, 0 );
+		add_filter( 'remote_data_blocks_request_details', [ __CLASS__, 'enhance_request_details' ], 10, 3 );
 	}
 
 	public static function register_blocks(): void {
@@ -186,4 +187,28 @@ public static function get_block_registration_snippets( array $data_source_confi
 
 		return $snippets;
 	}
+
+	/**
+	 * Due to the fact that we are using the same query for both the get and list queries, and
+	 * only filtering out the results based on the input variables, the in-memory cache will not
+	 * work as expected. This enhances the request details to include the input variables, so that
+	 * the in-memory cache will be able to differenciate each request.
+	 *
+	 * @param array<string, mixed> $request_details The request details.
+	 * @param string $_query The query being executed.
+	 * @param array<string, mixed> $input_variables The input variables for the current request.
+	 * @return array<string, array{
+	 *   method: string,
+	 *   options: array<string, mixed>,
+	 *   origin: string,
+	 *   uri: string,
+	 * }>
+	 */
+	public static function enhance_request_details( array $request_details, string $_query, array $input_variables ): array {
+		if ( isset( $request_details['origin'] ) && 'https://sheets.googleapis.com' === $request_details['origin'] && ! empty( $input_variables ) ) {
+			$request_details['input_variables'] = $input_variables;
+		}
+
+		return $request_details;
+	}
 }