fix: add checks before migrating

Author: Arukuen

src/block-components/alignment/deprecated/index.js

@@ -29,13 +29,19 @@ export const deprecateInnerBlockRowGapAndContainerHeight = {
 		const getAttrName = getAttrNameFunction( attrNameTemplate )
 		const getAttribute = _attrName => attributes[ getAttrName( _attrName ) ]
 
+		const newAttributes = {
+			...attributes,
+		}
+
 		const containerHeight = getAttribute( 'containerHeight' )
 		const innerBlockRowGap = getAttribute( 'innerBlockRowGap' )
 
-		const newAttributes = {
-			...attributes,
-			[ getAttrName( 'containerHeight' ) ]: String( containerHeight ),
-			[ getAttrName( 'innerBlockRowGap' ) ]: String( innerBlockRowGap ),
+		if ( typeof containerHeight === 'number' ) {
+			newAttributes[ getAttrName( 'containerHeight' ) ] = String( containerHeight )
+		}
+
+		if ( typeof innerBlockRowGap === 'number' ) {
+			newAttributes[ getAttrName( 'innerBlockRowGap' ) ] = String( innerBlockRowGap )
 		}
 
 		return newAttributes

src/block-components/columns/deprecated/index.js

@@ -41,15 +41,24 @@ export const deprecateColumnAndRowGap = {
 		const getAttrName = getAttrNameFunction( attrNameTemplate )
 		const getAttribute = _attrName => attributes[ getAttrName( _attrName ) ]
 
+		const newAttributes = {
+			...attributes,
+		}
+
 		const columnSpacing = getAttribute( 'columnSpacing' )
 		const columnGap = getAttribute( 'columnGap' )
 		const rowGap = getAttribute( 'rowGap' )
 
-		const newAttributes = {
-			...attributes,
-			[ getAttrName( 'columnSpacing' ) ]: String( columnSpacing ),
-			[ getAttrName( 'columnGap' ) ]: String( columnGap ),
-			[ getAttrName( 'rowGap' ) ]: String( rowGap ),
+		if ( typeof columnSpacing === 'number' ) {
+			newAttributes[ getAttrName( 'columnSpacing' ) ] = String( columnSpacing )
+		}
+
+		if ( typeof columnGap === 'number' ) {
+			newAttributes[ getAttrName( 'columnGap' ) ] = String( columnGap )
+		}
+
+		if ( typeof rowGap === 'number' ) {
+			newAttributes[ getAttrName( 'rowGap' ) ] = String( rowGap )
 		}
 
 		return newAttributes

src/block-components/helpers/size/deprecated.js

@@ -29,11 +29,14 @@ export const deprecateSizeControlHeight = {
 		const getAttrName = getAttrNameFunction( attrNameTemplate )
 		const getAttribute = _attrName => attributes[ getAttrName( _attrName ) ]
 
-		const height = getAttribute( 'height' )
-
 		const newAttributes = {
 			...attributes,
-			[ getAttrName( 'height' ) ]: String( height ),
+		}
+
+		const height = getAttribute( 'height' )
+
+		if ( typeof height === 'number' ) {
+			newAttributes[ getAttrName( 'height' ) ] = String( height )
 		}
 
 		return newAttributes

src/block-components/typography/deprecated.js

@@ -130,11 +130,14 @@ export const deprecateTypographyFontSize = {
 		const getAttrName = getAttrNameFunction( attrNameTemplate )
 		const getAttribute = _attrName => attributes[ getAttrName( _attrName ) ]
 
-		const fontSize = getAttribute( 'fontSize' )
-
 		const newAttributes = {
 			...attributes,
-			[ getAttrName( 'fontSize' ) ]: String( fontSize ),
+		}
+
+		const fontSize = getAttribute( 'fontSize' )
+
+		if ( typeof fontSize === 'number' ) {
+			newAttributes[ getAttrName( 'fontSize' ) ] = String( fontSize )
 		}
 
 		return newAttributes