Merge pull request #2164 from skaut/max-batch-size

Limiting Google API batch requests to 100 individual requests per batch

Author: marekdedic

src/php/class-api-client.php

@@ -170,6 +170,7 @@ public static function async_request( $request, $transform, $rejection_handler =
 			throw new Internal_Exception();
 		}
 
+		self::check_batch_size();
 		$key = wp_rand();
 		// @phan-suppress-next-line PhanPossiblyNonClassMethodCall
 		self::$current_batch->add( $request, $key );
@@ -202,6 +203,7 @@ public static function async_paginated_request(
 		$pagination_helper,
 		$rejection_handler = null
 	) {
+		self::check_batch_size();
 		/**
 		 * Gets one page.
 		 *
@@ -278,6 +280,20 @@ public static function execute( $promises = array() ) {
 		return Utils::all( $promises )->wait();
 	}
 
+	/**
+	 * Google API has an upper limit on batch size of 100 requests. Should a batch be bigger than that, this function executes it and prepares a new batch for the rest of the requests
+	 *
+	 * @return void
+	 */
+	private static function check_batch_size() {
+		if ( count( self::$pending_requests ) < 100 ) {
+			return;
+		}
+
+		self::execute_current_batch();
+		self::initialize_batch();
+	}
+
 	/**
 	 * Executes the current batch request and calls its callbacks
 	 *