Editor: Validate var name in BlackboardPlanEditor

blackboard/blackboard_plan.cpp

@@ -158,9 +158,16 @@ String BlackboardPlan::get_var_name(const BBVariable &p_var) const {
 	return String();
 }
 
+bool BlackboardPlan::is_valid_var_name(const String &p_name) const {
+	return p_name.is_valid_identifier() && !var_map.has(p_name);
+}
+
 void BlackboardPlan::rename_var(const String &p_name, const String &p_new_name) {
-	ERR_FAIL_COND(p_new_name.is_empty());
-	ERR_FAIL_COND(var_map.has(p_new_name));
+	if (p_name == p_new_name) {
+		return;
+	}
+
+	ERR_FAIL_COND(!is_valid_var_name(p_new_name));
 	ERR_FAIL_COND(!var_map.has(p_name));
 
 	BBVariable var = var_map[p_name];

blackboard/blackboard_plan.h

@@ -59,6 +59,7 @@ public:
 
 	PackedStringArray list_vars() const;
 	String get_var_name(const BBVariable &p_var) const;
+	bool is_valid_var_name(const String &p_name) const;
 	void rename_var(const String &p_name, const String &p_new_name);
 	void move_var(int p_index, int p_new_index);
 

editor/blackboard_plan_editor.cpp

@@ -40,6 +40,10 @@ using namespace godot;
 
 BlackboardPlanEditor *BlackboardPlanEditor::singleton = nullptr;
 
+LineEdit *BlackboardPlanEditor::_get_name_edit(int p_row_index) const {
+	return Object::cast_to<LineEdit>(rows_vbox->get_child(p_row_index)->get_child(0)->get_child(1));
+}
+
 void BlackboardPlanEditor::_add_var() {
 	ERR_FAIL_NULL(plan);
 
@@ -64,7 +68,26 @@ void BlackboardPlanEditor::_trash_var(int p_index) {
 
 void BlackboardPlanEditor::_rename_var(const String &p_new_name, int p_index) {
 	ERR_FAIL_NULL(plan);
-	plan->rename_var(plan->get_var_by_index(p_index).first, p_new_name);
+
+	LineEdit *name_edit = _get_name_edit(p_index);
+	ERR_FAIL_NULL(name_edit);
+
+	bool is_valid_var_name = plan->is_valid_var_name(p_new_name);
+	if (is_valid_var_name) {
+		plan->rename_var(plan->get_var_by_index(p_index).first, p_new_name);
+	}
+
+	if (is_valid_var_name || plan->get_var_by_index(p_index).first == p_new_name) {
+		if (name_edit->has_theme_color_override(LW_NAME(font_color))) {
+			name_edit->remove_theme_color_override(LW_NAME(font_color));
+			name_edit->queue_redraw();
+		}
+	} else {
+		if (!name_edit->has_theme_color_override(LW_NAME(font_color))) {
+			name_edit->add_theme_color_override(LW_NAME(font_color), Color(1, 0, 0));
+			name_edit->queue_redraw();
+		}
+	}
 }
 
 void BlackboardPlanEditor::_change_var_type(Variant::Type p_new_type, int p_index) {

editor/blackboard_plan_editor.h

@@ -24,6 +24,7 @@
 #ifdef LIMBOAI_GDEXTENSION
 #include <godot_cpp/classes/accept_dialog.hpp>
 #include <godot_cpp/classes/editor_inspector_plugin.hpp>
+#include <godot_cpp/classes/line_edit.hpp>
 #include <godot_cpp/classes/panel_container.hpp>
 #include <godot_cpp/classes/popup_menu.hpp>
 #include <godot_cpp/classes/scroll_container.hpp>
@@ -63,6 +64,8 @@ private:
 	PopupMenu *type_menu;
 	PopupMenu *hint_menu;
 
+	LineEdit *_get_name_edit(int p_row_index) const;
+
 	void _add_var();
 	void _trash_var(int p_index);
 	void _rename_var(const String &p_new_name, int p_index);