Model Importer does not pass the step when encounters TWO Action tags with different IDs

[farshbafdoustar]

Describe the bug
Model importer does ot pass the step of importing nodes that are defined with Action Tag and different IDs. It seems that it assumes the same BT node Action is visited again.

To Reproduce

Instead of having nodes like this:

<Sequence>
      <WaitForDigitalInput server_name="" digital_input_names="" states="{state}" connection_timeout_ms="200"/>
      <WaitForDigitalOutput  server_name="" digital_output_names="" states="{state}" connection_timeout_ms="200"/
</Sequence>

if the tree has nodes like the followings, the importer does not works properly:

<Sequence>
      <Action ID="WaitForDigitalInput" server_name="" digital_input_names="" states="{state}" connection_timeout_ms="200"/>
      <Action ID="WaitForDigitalOutput" server_name="" digital_output_names="" states="{state}" connection_timeout_ms="200"/>
</Sequence>

Expected behavior

importing the tree node model correctly

Screenshots and videos

Desktop (please complete the following information):

• OS: Ubuntu
• Version 20.04.4

Additional context

[facontidavide]

Good point!

I will try to fix this before the next release

[facontidavide]

Fixed