Block Library: Fix incorrect attributes definitions (#36264)

* Revert "Block Library: Fix incorrect attributes definitions #36140"

The `templateLock` attribute is mixed between boolean and string. Setting the type as string introduces an regression that will remove `false` on save.

The original PR (#36140) tried to fix a validation issue reported in #35902

The schema error is incorrect. The attribute definition is required to have either a `type` or an `enum`.

* Spesify type as string or boolean

* Add unit test

packages/block-library/src/column/block.json

@@ -17,7 +17,7 @@
 			"type": "array"
 		},
 		"templateLock": {
-			"type": "string",
+			"type": [ "string", "boolean" ],
 			"enum": [ "all", "insert", false ]
 		}
 	},

packages/block-library/src/cover/block.json

@@ -67,7 +67,7 @@
 			"type": "array"
 		},
 		"templateLock": {
-			"type": "string",
+			"type": [ "string", "boolean" ],
 			"enum": [ "all", "insert", false ]
 		}
 	},

packages/block-library/src/group/block.json

@@ -12,7 +12,7 @@
 			"default": "div"
 		},
 		"templateLock": {
-			"type": "string",
+			"type": [ "string", "boolean" ],
 			"enum": [ "all", "insert", false ]
 		}
 	},

packages/blocks/src/api/parser/test/get-block-attributes.js

@@ -222,6 +222,22 @@ describe( 'attributes parsing', () => {
 			expect( value ).toBe( 10 );
 		} );
 
+		it( 'should return the comment attribute value when using multiple types', () => {
+			const value = getBlockAttribute(
+				'templateLock',
+				{
+					type: [ 'string', 'boolean' ],
+					enum: [ 'all', 'insert', false ],
+				},
+				'',
+				{
+					templateLock: false,
+				}
+			);
+
+			expect( value ).toBe( false );
+		} );
+
 		it( 'should reject type-invalid value, with default', () => {
 			const value = getBlockAttribute(
 				'number',