Core Data: Check for nullity in experimental fetch methods

Adds a nullity check and enhances a JSDoc comment to acknowledge the nullable presence of a property in an object we test.
WordPress · Mar 16, 2022 · eec7411 · eec7411
1 parent 19d45c3
commit eec7411
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/packages/core-data/src/fetch/__experimental-fetch-link-suggestions.js b/packages/core-data/src/fetch/__experimental-fetch-link-suggestions.js
@@ -170,7 +170,7 @@ const fetchLinkSuggestions = async (
 			.slice( 0, perPage )
 			.map(
 				/**
-				 * @param {{ id: number, url:string, title?:string, subtype?: string, type?: string }} result
+				 * @param {{ id: number, meta?: object, url:string, title?:string, subtype?: string, type?: string }} result
 				 */
 				( result ) => {
 					return {

diff --git a/packages/core-data/src/fetch/__experimental-fetch-url-data.js b/packages/core-data/src/fetch/__experimental-fetch-url-data.js
@@ -60,6 +60,7 @@ const fetchUrlData = async ( url, options = {} ) => {
 	const protocol = getProtocol( url );
 
 	if (
+		! protocol ||
 		! isValidProtocol( protocol ) ||
 		! protocol.startsWith( 'http' ) ||
 		! /^https?:\/\/[^\/\s]/i.test( url )