bring back the old behavior of cloneBlock

packages/blocks/src/api/factory.js

@@ -91,21 +91,15 @@ export function createBlock( name, attributes = {}, innerBlocks = [] ) {
 export function cloneBlock( block, mergeAttributes = {}, newInnerBlocks ) {
 	const clientId = uuid();
 
-	// Check for innerBlocks on block before attempting
-	// to map them. There are edge cases where they can
-	// be `undefined`
-	if ( ! newInnerBlocks && block.innerBlocks ) {
-		newInnerBlocks = block.innerBlocks.map( ( innerBlock ) => cloneBlock( innerBlock ) );
-	}
-
 	return {
 		...block,
 		clientId,
 		attributes: {
 			...block.attributes,
 			...mergeAttributes,
 		},
-		innerBlocks: newInnerBlocks,
+		innerBlocks: newInnerBlocks ||
+			block.innerBlocks.map( ( innerBlock ) => cloneBlock( innerBlock ) ),
 	};
 }