Packages[core-data]: Accept numeric values for the "include" property…

… in getQueryParts (#40570) * Packages[core-data]: Change "include" type in getQueryParts Convert the value of the parameter include into a string when it's a number. * Add some units test
WordPress · Apr 29, 2022 · fa23944 · fa23944
1 parent 0b19b54
commit fa23944
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/packages/core-data/src/queried-data/get-query-parts.js b/packages/core-data/src/queried-data/get-query-parts.js
@@ -80,6 +80,9 @@ export function getQueryParts( query ) {
 
 				// Two requests with different include values cannot have same results.
 				if ( key === 'include' ) {
+					if ( typeof value === 'number' ) {
+						value = value.toString();
+					}
 					parts.include = (
 						getNormalizedCommaSeparable( value ) ?? []
 					).map( Number );

diff --git a/packages/core-data/src/queried-data/test/get-query-parts.js b/packages/core-data/src/queried-data/test/get-query-parts.js
@@ -18,8 +18,12 @@ describe( 'getQueryParts', () => {
 	} );
 
 	it( 'parses out `include` ID filtering', () => {
+		const first = getQueryParts( { include: '1' } );
+		const second = getQueryParts( { include: 1 } );
 		const parts = getQueryParts( { include: [ 1 ] } );
 
+		expect( first ).toEqual( second );
+		expect( second ).toEqual( parts );
 		expect( parts ).toEqual( {
 			context: 'default',
 			page: 1,