Phobos scripts ignore `FireError`

[TaranDahl]

Description

At least it should check for FireError::Illegal.
Now they will select an unattackable unit as target and get stuck.

Phobos Version

Any

Conditions to reproduce

No response

INI code

Steps to reproduce

...

Expected behaviour

Script should ignore the target that is unattackable.

Actual behaviour

The team selects an unattackable unit as target and keeps staring at it folly.

Additional context

No response

Checklist

• The issue happens on the latest official version of Phobos and wasn't fixed yet.
• I agree to elaborate the details if requested and provide thorough testing if the bugfix is implemented.
• I added a very descriptive title to this issue.
• I used the GitHub search and read the issue list to find a similar issue and didn't find it.
• I have attached as much information as possible (screenshots, gifs, videos, debug and exception logs, etc).