fix issue BehaviorTree#757 : skipped nodes should not call post-condition ALWAYS

facontidavide · facontidavide · commit c4ad3cd1d2b8 · 2024-02-01T15:04:12.000+01:00
diff --git a/src/tree_node.cpp b/src/tree_node.cpp
@@ -109,11 +109,10 @@ NodeStatus TreeNode::executeTick()
     }
   }
 
-  checkPostConditions(new_status);
-
   // injected post callback
   if(isStatusCompleted(new_status))
   {
+    checkPostConditions(new_status);
     PostTickCallback  callback;
     {
       std::unique_lock lk(_p->callback_injection_mutex);

Original file line number	Diff line number	Diff line change
`@@ -109,11 +109,10 @@ NodeStatus TreeNode::executeTick()`
`109`	`109`	`}`
`110`	`110`	`}`
`111`	`111`
`112`		`- checkPostConditions(new_status);`
`113`		`-`
`114`	`112`	`// injected post callback`
`115`	`113`	`if(isStatusCompleted(new_status))`
`116`	`114`	`{`
	`115`	`+ checkPostConditions(new_status);`
`117`	`116`	`PostTickCallback callback;`
`118`	`117`	`{`
`119`	`118`	`std::unique_lock lk(_p->callback_injection_mutex);`