Merge pull request #164 from endlessm/T35536-decouple-alternative

T35536 decouple mid-term option

Author: wnbaum

addons/block_code/block_definition.gd

@@ -0,0 +1,21 @@
+extends Resource
+
+const Types = preload("res://addons/block_code/types/types.gd")
+
+var name: StringName
+var type: Types.BlockType
+var description: String
+var category: String
+
+var label_template: String
+var code_template: String
+var defaults: Dictionary = {}
+
+## Only for blocks of type Types.ENTRY. If non-empty, this block defines a
+## callback that will be connected to the signal with this name.
+var signal_name: String = ""
+
+
+func _init(p_name: StringName, p_type: Types.BlockType):
+	name = p_name
+	type = p_type

addons/block_code/blocks_catalog.gd

@@ -0,0 +1,35 @@
+extends Object
+
+const BlockDefinition = preload("res://addons/block_code/block_definition.gd")
+const Types = preload("res://addons/block_code/types/types.gd")
+
+static var _catalog: Dictionary
+
+
+static func setup():
+	if _catalog:
+		return
+
+	_catalog = {}
+	var block_definition: BlockDefinition = BlockDefinition.new(&"ready_block", Types.BlockType.ENTRY)
+	block_definition.label_template = "On Ready"
+	block_definition.code_template = "func _ready():"
+	block_definition.description = 'Attached blocks will be executed once when the node is "ready"'
+	block_definition.category = "Lifecycle"
+	_catalog[&"ready_block"] = block_definition
+
+	block_definition = BlockDefinition.new(&"print", Types.BlockType.EXECUTE)
+	block_definition.label_template = "print {text: STRING}"
+	block_definition.code_template = "print({text})"
+	block_definition.defaults = {"text": "Hello"}
+	block_definition.description = "Print the text to output"
+	block_definition.category = "Log"
+	_catalog[&"print"] = block_definition
+
+
+static func get_block(block_name: StringName):
+	return _catalog.get(block_name)
+
+
+static func has_block(block_name: StringName):
+	return block_name in _catalog

addons/block_code/examples/pong_game/pong_game.tscn

@@ -161,10 +161,16 @@ func clear_canvas():
 
 
 func load_tree(parent: Node, node: SerializedBlockTreeNode):
-	var _block_scene_path = _block_scenes_by_class[node.serialized_block.block_class]
-	var scene: Block = load(_block_scene_path).instantiate()
+	var scene: Block = Util.instantiate_block(node.name)
+
+	# TODO: Remove once the data/UI decouple is done.
+	if scene == null:
+		var _block_scene_path = _block_scenes_by_class[node.serialized_block.block_class]
+		scene = load(_block_scene_path).instantiate()
 	for prop_pair in node.serialized_block.serialized_props:
 		scene.set(prop_pair[0], prop_pair[1])
+
+	scene.position = node.position
 	scene.resource = node
 	parent.add_child(scene)
 

addons/block_code/ui/block_canvas/block_canvas.gd

@@ -1,10 +1,16 @@
 class_name SerializedBlockTreeNode
 extends Resource
 
-@export var serialized_block: SerializedBlock
+@export var name: StringName
+@export var position: Vector2
 @export var path_child_pairs: Array
 
+# TODO: Remove once the data/UI decouple is done.
+@export var serialized_block: SerializedBlock
+
 
-func _init(p_serialized_block: SerializedBlock = null, p_path_child_pairs: Array = []):
+func _init(p_name: StringName, p_position: Vector2 = Vector2.ZERO, p_serialized_block: SerializedBlock = null, p_path_child_pairs: Array = []):
+	name = p_name
+	position = p_position
 	serialized_block = p_serialized_block
 	path_child_pairs = p_path_child_pairs

addons/block_code/ui/block_canvas/serialized_block_tree_node.gd

@@ -2,14 +2,15 @@
 class_name Block
 extends MarginContainer
 
+const BlocksCatalog = preload("res://addons/block_code/blocks_catalog.gd")
 const InstructionTree = preload("res://addons/block_code/instruction_tree/instruction_tree.gd")
 const Types = preload("res://addons/block_code/types/types.gd")
 
 signal drag_started(block: Block)
 signal modified
 
 ## Name of the block to be referenced by others in search
-@export var block_name: String = ""
+@export var block_name: StringName
 
 ## Label of block (optionally used to draw block labels)
 @export var label: String = ""
@@ -83,18 +84,29 @@ func get_instruction_node() -> InstructionTree.TreeNode:
 func update_resources(undo_redo: EditorUndoRedoManager):
 	if resource == null:
 		var serialized_block = SerializedBlock.new(get_block_class(), get_serialized_props())
-		resource = SerializedBlockTreeNode.new(serialized_block)
+		resource = SerializedBlockTreeNode.new(block_name, position, serialized_block)
 		return
 
+	if resource.position != position:
+		undo_redo.add_undo_property(resource, "position", resource.position)
+		undo_redo.add_do_property(resource, "position", position)
+
 	var serialized_props = get_serialized_props()
+
 	if serialized_props != resource.serialized_block.serialized_props:
 		undo_redo.add_undo_property(resource.serialized_block, "serialized_props", resource.serialized_block.serialized_props)
 		undo_redo.add_do_property(resource.serialized_block, "serialized_props", serialized_props)
 
 
 # Override this method to add more serialized properties
 func get_serialized_props() -> Array:
-	return serialize_props(["block_name", "label", "color", "block_type", "position", "scope"])
+	if not BlocksCatalog.has_block(block_name):
+		return serialize_props(["block_name", "label", "color", "block_type", "position", "scope"])
+
+	# TODO: Remove remaining serialization:
+	# - Derive color from category.
+	# - Handle scope in a different way?
+	return serialize_props(["color", "scope"])
 
 
 func _to_string():

addons/block_code/ui/blocks/block/block.gd

@@ -7,7 +7,7 @@
 size_flags_horizontal = 0
 mouse_filter = 2
 script = ExtResource("1_2hbir")
-block_name = "control_block"
+block_name = &"control_block"
 label = "Control Block"
 color = Color(0.59979, 0.536348, 0.876215, 1)
 bottom_snap_path = NodePath("VBoxContainer/SnapPoint")

addons/block_code/ui/blocks/control_block/control_block.tscn

@@ -30,5 +30,6 @@ func get_entry_statement() -> String:
 
 func get_serialized_props() -> Array:
 	var props := super()
-	props.append_array(serialize_props(["signal_name"]))
+	if not BlocksCatalog.has_block(block_name):
+		props.append_array(serialize_props(["signal_name"]))
 	return props

addons/block_code/ui/blocks/entry_block/entry_block.gd

@@ -6,8 +6,7 @@
 [node name="EntryBlock" instance=ExtResource("1_byjbb")]
 script = ExtResource("2_3ik8h")
 signal_name = ""
-defaults = {}
-block_name = "entry_block"
+block_name = &"entry_block"
 label = "EntryBlock"
 block_type = 1
 

addons/block_code/ui/blocks/entry_block/entry_block.tscn

@@ -18,9 +18,9 @@ corner_radius_bottom_left = 16
 offset_right = 16.0
 offset_bottom = 8.0
 size_flags_horizontal = 0
+mouse_filter = 2
 script = ExtResource("1_0hajy")
-defaults = null
-block_name = "parameter_block"
+block_name = &"parameter_block"
 label = "Param"
 block_type = 3