Call initialize() on constructor

Tacha-S · Tacha-S · commit e420aeb5d939 · 2025-02-13T10:57:36.000+09:00
Signed-off-by: Tatsuro Sakaguchi &lt;tatsuro.sakaguchi@g.softbank.co.jp&gt;
diff --git a/nav2_behavior_tree/plugins/action/controller_selector_node.cpp b/nav2_behavior_tree/plugins/action/controller_selector_node.cpp
@@ -32,7 +32,7 @@ ControllerSelector::ControllerSelector(
   const BT::NodeConfiguration & conf)
 : BT::SyncActionNode(name, conf)
 {
-  createROSInterfaces();
+  initialize();
 }
 
 void ControllerSelector::initialize()
diff --git a/nav2_behavior_tree/plugins/action/goal_checker_selector_node.cpp b/nav2_behavior_tree/plugins/action/goal_checker_selector_node.cpp
@@ -32,7 +32,7 @@ GoalCheckerSelector::GoalCheckerSelector(
   const BT::NodeConfiguration & conf)
 : BT::SyncActionNode(name, conf)
 {
-  createROSInterfaces();
+  initialize();
 }
 
 void GoalCheckerSelector::initialize()
diff --git a/nav2_behavior_tree/plugins/action/planner_selector_node.cpp b/nav2_behavior_tree/plugins/action/planner_selector_node.cpp
@@ -32,7 +32,7 @@ PlannerSelector::PlannerSelector(
   const BT::NodeConfiguration & conf)
 : BT::SyncActionNode(name, conf)
 {
-  createROSInterfaces();
+  initialize();
 }
 
 void PlannerSelector::initialize()
diff --git a/nav2_behavior_tree/plugins/action/progress_checker_selector_node.cpp b/nav2_behavior_tree/plugins/action/progress_checker_selector_node.cpp
@@ -31,7 +31,7 @@ ProgressCheckerSelector::ProgressCheckerSelector(
   const BT::NodeConfiguration & conf)
 : BT::SyncActionNode(name, conf)
 {
-  createROSInterfaces();
+  initialize();
 }
 
 void ProgressCheckerSelector::initialize()
diff --git a/nav2_behavior_tree/plugins/action/smoother_selector_node.cpp b/nav2_behavior_tree/plugins/action/smoother_selector_node.cpp
@@ -33,7 +33,7 @@ SmootherSelector::SmootherSelector(
   const BT::NodeConfiguration & conf)
 : BT::SyncActionNode(name, conf)
 {
-  createROSInterfaces();
+  initialize();
 }
 
 void SmootherSelector::initialize()
diff --git a/nav2_behavior_tree/plugins/condition/is_battery_charging_condition.cpp b/nav2_behavior_tree/plugins/condition/is_battery_charging_condition.cpp
@@ -26,7 +26,7 @@ IsBatteryChargingCondition::IsBatteryChargingCondition(
   battery_topic_("/battery_status"),
   is_battery_charging_(false)
 {
-  createROSInterfaces();
+  initialize();
 }
 
 void IsBatteryChargingCondition::initialize()
diff --git a/nav2_behavior_tree/plugins/decorator/goal_updater_node.cpp b/nav2_behavior_tree/plugins/decorator/goal_updater_node.cpp
@@ -35,7 +35,7 @@ GoalUpdater::GoalUpdater(
   goal_updater_topic_("goal_update"),
   goals_updater_topic_("goals_update")
 {
-  createROSInterfaces();
+  initialize();
 }
 
 void GoalUpdater::initialize()

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ ControllerSelector::ControllerSelector(`
`32`	`32`	`const BT::NodeConfiguration & conf)`
`33`	`33`	`: BT::SyncActionNode(name, conf)`
`34`	`34`	`{`
`35`		`- createROSInterfaces();`
	`35`	`+ initialize();`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`void ControllerSelector::initialize()`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ GoalCheckerSelector::GoalCheckerSelector(`
`32`	`32`	`const BT::NodeConfiguration & conf)`
`33`	`33`	`: BT::SyncActionNode(name, conf)`
`34`	`34`	`{`
`35`		`- createROSInterfaces();`
	`35`	`+ initialize();`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`void GoalCheckerSelector::initialize()`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ PlannerSelector::PlannerSelector(`
`32`	`32`	`const BT::NodeConfiguration & conf)`
`33`	`33`	`: BT::SyncActionNode(name, conf)`
`34`	`34`	`{`
`35`		`- createROSInterfaces();`
	`35`	`+ initialize();`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`void PlannerSelector::initialize()`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ ProgressCheckerSelector::ProgressCheckerSelector(`
`31`	`31`	`const BT::NodeConfiguration & conf)`
`32`	`32`	`: BT::SyncActionNode(name, conf)`
`33`	`33`	`{`
`34`		`- createROSInterfaces();`
	`34`	`+ initialize();`
`35`	`35`	`}`
`36`	`36`
`37`	`37`	`void ProgressCheckerSelector::initialize()`
Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ SmootherSelector::SmootherSelector(`
`33`	`33`	`const BT::NodeConfiguration & conf)`
`34`	`34`	`: BT::SyncActionNode(name, conf)`
`35`	`35`	`{`
`36`		`- createROSInterfaces();`
	`36`	`+ initialize();`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`void SmootherSelector::initialize()`
Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ IsBatteryChargingCondition::IsBatteryChargingCondition(`
`26`	`26`	`battery_topic_("/battery_status"),`
`27`	`27`	`is_battery_charging_(false)`
`28`	`28`	`{`
`29`		`- createROSInterfaces();`
	`29`	`+ initialize();`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`void IsBatteryChargingCondition::initialize()`
Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ GoalUpdater::GoalUpdater(`
`35`	`35`	`goal_updater_topic_("goal_update"),`
`36`	`36`	`goals_updater_topic_("goals_update")`
`37`	`37`	`{`
`38`		`- createROSInterfaces();`
	`38`	`+ initialize();`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`void GoalUpdater::initialize()`