ParameterInput: Add a separate option data property

Instead of inferring that there is a list of options based on the
current value, ParameterInput expects to be given a list of options,
and get_raw_input and set_raw_input understand that the raw input is a
value from the list.

In addition, change TemplateEditor itself to determine the list of
options for a given parameter. This option data is always from the
parameter_defaults property, regardless of the block's current value.

https://phabricator.endlessm.com/T35564

Author: dylanmccall

addons/block_code/ui/blocks/utilities/parameter_input/parameter_input.gd

@@ -10,7 +10,9 @@ signal modified
 
 @export var variant_type: Variant.Type = TYPE_STRING
 @export var block_type: Types.BlockType = Types.BlockType.VALUE
-var option: bool = false
+@export var option_data: OptionData:
+	set = _set_option_data
+
 var default_value: Variant
 
 @onready var _panel := %Panel
@@ -42,16 +44,13 @@ var default_value: Variant
 
 
 func set_raw_input(raw_input):
-	if option:
-		_panel.visible = false
-		_option_input.clear()
-		var option_data: OptionData = raw_input as OptionData
-		for item in option_data.items:
-			_option_input.add_item(item.capitalize())
-		_option_input.select(option_data.selected)
-
+	if option_data:
+		_update_option_input(raw_input)
 		return
 
+	if raw_input == null:
+		raw_input = default_value
+
 	match variant_type:
 		TYPE_COLOR:
 			_color_input.color = raw_input
@@ -61,19 +60,16 @@ func set_raw_input(raw_input):
 			_x_line_edit.text = ("%.4f" % raw_input.x).rstrip("0").rstrip(".")
 			_y_line_edit.text = ("%.4f" % raw_input.y).rstrip("0").rstrip(".")
 		TYPE_BOOL:
-			_bool_input_option.select(raw_input)
+			_bool_input_option.select(1 if raw_input else 0)
 		TYPE_NIL:
-			_line_edit.text = raw_input
+			_line_edit.text = raw_input if raw_input else ""
 		_:
-			_line_edit.text = "" if raw_input == null else str(raw_input)
+			_line_edit.text = str(raw_input) if raw_input else ""
 
 
 func get_raw_input():
-	if option:
-		var options: Array = []
-		for i in _option_input.item_count:
-			options.append(_option_input.get_item_text(i).to_snake_case())
-		return OptionData.new(options, _option_input.selected)
+	if option_data:
+		return _option_input.get_selected_metadata()
 
 	match variant_type:
 		TYPE_COLOR:
@@ -92,20 +88,32 @@ func get_raw_input():
 			return _line_edit.text
 
 
+func _set_option_data(new_option_data: OptionData) -> void:
+	option_data = new_option_data
+
+	if not is_node_ready():
+		return
+
+	_update_option_input()
+
+
 func _set_placeholder(new_placeholder: String) -> void:
 	placeholder = new_placeholder
 
 	if not is_node_ready():
 		return
 
 	_line_edit.placeholder_text = placeholder
+	_input_switcher.tooltip_text = placeholder
+	_option_input.tooltip_text = placeholder
 
 
 func _ready():
 	var stylebox = _panel.get_theme_stylebox("panel")
 	stylebox.bg_color = Color.WHITE
 
 	_set_placeholder(placeholder)
+	_set_option_data(option_data)
 
 	snap_point.block_type = block_type
 	snap_point.variant_type = variant_type
@@ -132,8 +140,8 @@ func get_string() -> String:
 
 	var input = get_raw_input()
 
-	if option:
-		return _option_input.get_item_text(_option_input.selected).to_snake_case()
+	if option_data:
+		return input
 
 	match variant_type:
 		TYPE_STRING:
@@ -194,7 +202,7 @@ func _on_y_line_edit_focus_exited():
 func _update_visible_input():
 	if snap_point.has_snapped_block():
 		_switch_input(null)
-	elif option:
+	elif option_data:
 		_switch_input(_option_input)
 	else:
 		match variant_type:
@@ -211,6 +219,29 @@ func _update_visible_input():
 func _switch_input(node: Node):
 	for c in _input_switcher.get_children():
 		c.visible = c == node
+	_panel.visible = node not in [_option_input]
+
+
+func _update_option_input(current_value: Variant = null):
+	if not option_data:
+		return
+
+	if current_value == null:
+		current_value = _option_input.get_selected_metadata()
+
+	_option_input.clear()
+
+	var selected_item_index: int = -1
+
+	for item in option_data.items:
+		var item_index = _option_input.item_count
+		var option_label = item.capitalize()
+		_option_input.add_item(option_label)
+		_option_input.set_item_metadata(item_index, item)
+		if item == current_value:
+			selected_item_index = item_index
+
+	_option_input.select(selected_item_index)
 
 
 func _on_color_input_color_changed(color):

addons/block_code/ui/blocks/utilities/parameter_input/parameter_input.tscn

@@ -42,6 +42,7 @@ theme_override_styles/panel = SubResource("StyleBoxFlat_tn6h4")
 [node name="InputSwitcher" type="MarginContainer" parent="."]
 unique_name_in_owner = true
 layout_mode = 2
+tooltip_text = "Parameter"
 mouse_filter = 2
 theme_override_constants/margin_left = 0
 theme_override_constants/margin_top = 0

addons/block_code/ui/blocks/utilities/template_editor/template_editor.gd

@@ -110,8 +110,15 @@ func _append_input_parameter(parameter_format: String, id: int):
 	parameter_input.name = "ParameterInput%d" % id
 	parameter_input.placeholder = parameter["name"]
 	parameter_input.variant_type = parameter["type"]
-	parameter_input.option = parameter["is_option"]
-	parameter_input.default_value = default_value
+
+	if parameter["is_option"] and default_value is OptionData:
+		var option_data := default_value as OptionData
+		parameter_input.option_data = option_data
+		parameter_input.default_value = option_data.items[option_data.selected]
+	elif parameter["is_option"]:
+		push_warning("The block parameter %s in %s appears to be an option, but no option data is provided" % [parameter_format, parent_block])
+	else:
+		parameter_input.default_value = default_value
 
 	parameter_input.modified.connect(modified.emit)