blackboard/blackboard_plan.cpp

@@ -11,8 +11,6 @@
 
 #include "blackboard_plan.h"
 
-#include "../util/limbo_utility.h"
-
 bool BlackboardPlan::_set(const StringName &p_name, const Variant &p_value) {
 	String name_str = p_name;
 
@@ -36,7 +34,6 @@ bool BlackboardPlan::_set(const StringName &p_name, const Variant &p_value) {
 		} else {
 			parent_scope_mapping[mapped_var_name] = value;
 		}
-		notify_property_list_changed();
 		return true;
 	}
 
@@ -72,7 +69,7 @@ bool BlackboardPlan::_get(const StringName &p_name, Variant &r_ret) const {
 	// * Editor
 	if (var_map.has(p_name)) {
 		if (has_mapping(p_name)) {
-			r_ret = "Mapped to " + LimboUtility::get_singleton()->decorate_var(parent_scope_mapping[p_name]);
+			r_ret = "Mapped to $" + parent_scope_mapping[p_name];
 		} else {
 			r_ret = var_map[p_name].get_value();
 		}
@@ -181,8 +178,8 @@ void BlackboardPlan::set_base_plan(const Ref<BlackboardPlan> &p_base) {
 	notify_property_list_changed();
 }
 
-void BlackboardPlan::set_parent_scope_plan_provider(const Callable &p_parent_scope_plan_provider) {
-	parent_scope_plan_provider = p_parent_scope_plan_provider;
+void BlackboardPlan::set_parent_scope_plan(const Ref<BlackboardPlan> &p_plan) {
+	parent_scope_plan = p_plan;
 	notify_property_list_changed();
 }
 
@@ -425,8 +422,8 @@ void BlackboardPlan::_bind_methods() {
 
 	ClassDB::bind_method(D_METHOD("set_base_plan", "blackboard_plan"), &BlackboardPlan::set_base_plan);
 	ClassDB::bind_method(D_METHOD("get_base_plan"), &BlackboardPlan::get_base_plan);
-	ClassDB::bind_method(D_METHOD("set_parent_scope_plan_provider", "callable"), &BlackboardPlan::set_parent_scope_plan_provider);
-	ClassDB::bind_method(D_METHOD("get_parent_scope_plan_provider"), &BlackboardPlan::get_parent_scope_plan_provider);
+	ClassDB::bind_method(D_METHOD("set_parent_scope_plan", "blackboard_plan"), &BlackboardPlan::set_parent_scope_plan);
+	ClassDB::bind_method(D_METHOD("get_parent_scope_plan"), &BlackboardPlan::get_parent_scope_plan);
 	ClassDB::bind_method(D_METHOD("is_mapping_enabled"), &BlackboardPlan::is_mapping_enabled);
 	ClassDB::bind_method(D_METHOD("is_derived"), &BlackboardPlan::is_derived);
 	ClassDB::bind_method(D_METHOD("sync_with_base_plan"), &BlackboardPlan::sync_with_base_plan);

blackboard/blackboard_plan.h

@@ -39,9 +39,9 @@ private:
 	// Mapping between variables in this plan and their parent scope names.
 	// Used for linking variables to their parent scope counterparts upon Blackboard creation/population.
 	HashMap<StringName, StringName> parent_scope_mapping;
-	// Fetcher function for the parent scope plan. Funtion should return a Ref<BlackboardPlan>.
-	// Used in the inspector. When set, mapping feature becomes available.
-	Callable parent_scope_plan_provider;
+	// BlackboardPlan that will be used in parent scope Blackboard creation at runtime.
+	// Used to provide hints in the inspector. When set, mapping is enabled in the inspector.
+	Ref<BlackboardPlan> parent_scope_plan;
 
 	// If true, NodePath variables will be prefetched, so that the vars will contain node pointers instead (upon BB creation/population).
 	bool prefetch_nodepath_vars = true;
@@ -59,10 +59,9 @@ public:
 	void set_base_plan(const Ref<BlackboardPlan> &p_base);
 	Ref<BlackboardPlan> get_base_plan() const { return base; }
 
-	void set_parent_scope_plan_provider(const Callable &p_parent_scope_plan_provider);
-	Callable get_parent_scope_plan_provider() const { return parent_scope_plan_provider; }
-
-	bool is_mapping_enabled() const { return parent_scope_plan_provider.is_valid() && (parent_scope_plan_provider.call() != Ref<BlackboardPlan>()); }
+	void set_parent_scope_plan(const Ref<BlackboardPlan> &p_plan);
+	Ref<BlackboardPlan> get_parent_scope_plan() const { return parent_scope_plan; }
+	bool is_mapping_enabled() const { return parent_scope_plan.is_valid(); }
 	bool has_mapping(const StringName &p_name) const;
 
 	void set_prefetch_nodepath_vars(bool p_enable);

bt/bt_state.cpp

@@ -32,10 +32,10 @@ void BTState::set_behavior_tree(const Ref<BehaviorTree> &p_tree) {
 			p_tree->connect(LW_NAME(plan_changed), callable_mp(this, &BTState::_update_blackboard_plan));
 		}
 		behavior_tree = p_tree;
+		_update_blackboard_plan();
 	} else {
 		behavior_tree = p_tree;
 	}
-	_update_blackboard_plan();
 }
 
 void BTState::_update_blackboard_plan() {

bt/tasks/decorators/bt_new_scope.cpp

@@ -35,7 +35,7 @@ void BTNewScope::_set_parent_scope_plan_from_bt() {
 	ERR_FAIL_NULL(get_blackboard_plan());
 	Ref<BehaviorTree> bt = get_root()->editor_get_behavior_tree();
 	ERR_FAIL_NULL(bt);
-	get_blackboard_plan()->set_parent_scope_plan_provider(callable_mp(bt.ptr(), &BehaviorTree::get_blackboard_plan));
+	get_blackboard_plan()->set_parent_scope_plan(bt->get_blackboard_plan());
 }
 #endif // TOOLS_ENABLED
 

editor/editor_property_variable_name.cpp

@@ -54,22 +54,15 @@ void EditorPropertyVariableName::_show_variables_popup() {
 	variables_popup->popup(rect);
 }
 
-void EditorPropertyVariableName::_name_changed(const String &p_new_name, bool p_changing) {
-	emit_changed(get_edited_property(), p_new_name, StringName(), p_changing);
+void EditorPropertyVariableName::_name_changed(const String &p_new_name) {
+	emit_changed(get_edited_property(), p_new_name);
 	_update_status();
 }
 
-void EditorPropertyVariableName::_name_submitted() {
-	_name_changed(name_edit->get_text(), false);
-	if (name_edit->has_focus()) {
-		name_edit->release_focus();
-	}
-}
-
 void EditorPropertyVariableName::_variable_selected(int p_id) {
 	String var_name = plan->get_var_by_index(p_id).first;
 	name_edit->set_text(var_name);
-	_name_submitted();
+	_name_changed(var_name);
 }
 
 void EditorPropertyVariableName::_update_status() {
@@ -150,9 +143,7 @@ void EditorPropertyVariableName::setup(const Ref<BlackboardPlan> &p_plan, bool p
 void EditorPropertyVariableName::_notification(int p_what) {
 	switch (p_what) {
 		case NOTIFICATION_READY: {
-			name_edit->connect(LW_NAME(text_changed), callable_mp(this, &EditorPropertyVariableName::_name_changed).bind(true));
-			name_edit->connect(LW_NAME(text_submitted), callable_mp(this, &EditorPropertyVariableName::_name_submitted).unbind(1));
-			name_edit->connect(LW_NAME(focus_exited), callable_mp(this, &EditorPropertyVariableName::_name_submitted));
+			name_edit->connect(LW_NAME(text_changed), callable_mp(this, &EditorPropertyVariableName::_name_changed));
 			variables_popup->connect(LW_NAME(id_pressed), callable_mp(this, &EditorPropertyVariableName::_variable_selected));
 			drop_btn->connect(LW_NAME(pressed), callable_mp(this, &EditorPropertyVariableName::_show_variables_popup));
 			status_btn->connect(LW_NAME(pressed), callable_mp(this, &EditorPropertyVariableName::_status_pressed));
@@ -250,12 +241,7 @@ bool EditorInspectorPluginVariableName::_parse_property(Object *p_object, const
 			expected_hint = variable.get_hint();
 			expected_hint_string = variable.get_hint_string();
 		}
-		if (plan->get_parent_scope_plan_provider().is_valid()) {
-			Ref<BlackboardPlan> parent_plan = plan->get_parent_scope_plan_provider().call();
-			if (parent_plan.is_valid()) {
-				plan = parent_plan;
-			}
-		}
+		plan = plan->get_parent_scope_plan();
 		ERR_FAIL_NULL_V(plan, false);
 	} else {
 		plan = plan_getter.call();

editor/editor_property_variable_name.h

@@ -55,8 +55,7 @@ private:
 	PopupMenu *variables_popup;
 
 	void _show_variables_popup();
-	void _name_changed(const String &p_new_name, bool p_changing);
-	void _name_submitted();
+	void _name_changed(const String &p_new_name);
 	void _variable_selected(int p_id);
 	void _update_status();
 

hsm/limbo_state.cpp

@@ -22,31 +22,12 @@
 
 void LimboState::set_blackboard_plan(const Ref<BlackboardPlan> &p_plan) {
 	blackboard_plan = p_plan;
-
-	if (Engine::get_singleton()->is_editor_hint() && blackboard_plan.is_valid()) {
-		blackboard_plan->set_parent_scope_plan_provider(callable_mp(this, &LimboState::_get_parent_scope_plan));
-	}
-
 	_update_blackboard_plan();
 }
 
 void LimboState::_update_blackboard_plan() {
 }
 
-Ref<BlackboardPlan> LimboState::_get_parent_scope_plan() const {
-	BlackboardPlan *parent_plan = nullptr;
-	const LimboState *state = this;
-	while (state->get_parent() && IS_CLASS(state->get_parent(), LimboState)) {
-		state = Object::cast_to<LimboState>(state->get_parent());
-		ERR_FAIL_NULL_V(state, parent_plan);
-		if (state->blackboard_plan.is_valid()) {
-			parent_plan = state->blackboard_plan.ptr();
-			break;
-		}
-	}
-	return parent_plan;
-}
-
 LimboState *LimboState::get_root() const {
 	const Node *state = this;
 	while (state->get_parent() && IS_CLASS(state->get_parent(), LimboState)) {

hsm/limbo_state.h

@@ -38,8 +38,6 @@ private:
 	HashMap<StringName, Callable> handlers;
 	Callable guard_callable;
 
-	Ref<BlackboardPlan> _get_parent_scope_plan() const;
-
 protected:
 	friend LimboHSM;
 

util/limbo_string_names.cpp

@@ -85,7 +85,6 @@ LimboStringNames::LimboStringNames() {
 	exited = SN("exited");
 	favorite_tasks_changed = SN("favorite_tasks_changed");
 	Favorites = SN("Favorites");
-	focus_exited = SN("focus_exited");
 	font = SN("font");
 	font_color = SN("font_color");
 	font_size = SN("font_size");

util/limbo_string_names.h

@@ -101,7 +101,6 @@ public:
 	StringName exited;
 	StringName favorite_tasks_changed;
 	StringName Favorites;
-	StringName focus_exited;
 	StringName font_color;
 	StringName font_size;
 	StringName font;