NodeStatus::IDLE when action is preempting.

[Timple]

BehaviorTree.ROS/include/behaviortree_ros/bt_action_node.h

Line 103 in 81f49f0

setStatus(NodeStatus::IDLE);

Often actionclients can stop at once if they are asked to (like calculating fibonacci numbers).

But somethings it takes time to cleanup (PREEMPTING state). Like writing large files, getting to a safe state.

I would say the NodeStatus should only be idle when the state is PREEMPTED, not PREEMPTING.

source

[facontidavide]

I think you are correct. I am about to refactor this, and I will take this suggestion into account

[Timple]

Did you already start on the refactor?
Otherwise I'll make a pull-request only implementing this change.