SafetyFinder: Fixed issue with CBS when run mode is NEVER

[do-gamer]

No description provided.

[Pablete1234]

You're solving the wrong part of the issue.

This used to work just fine prior to the fix for CBS casting, meaning the issue is contained within just the statement:
.map(cbs -> !(cbs.getEntityInfo().isEnemy() || (cbs.getHullId() == 0 && safety.getCbsMode() == SafetyInfo.CbsMode.ALLY)))
which can return true in situations it shouldn't

[do-gamer]

You're solving the wrong part of the issue.

This used to work just fine prior to the fix for CBS casting, meaning the issue is contained within just the statement: .map(cbs -> !(cbs.getEntityInfo().isEnemy() || (cbs.getHullId() == 0 && safety.getCbsMode() == SafetyInfo.CbsMode.ALLY))) which can return true in situations it shouldn't

@Pablete1234 Yes, I agree with you. The bug is exactly in this one condition.
I just implemented an early return, because if the run mode is NEVER, then there is nothing to do more in this method, just return false.

Is it make sense?
Else I will update the condition into map()

[Pablete1234]

I would rather have the root of the issue fixed instead of the early-return workaround

[do-gamer]

I would rather have the root of the issue fixed instead of the early-return workaround

Moved into map()

[do-gamer]

@Pablete1234 Please review again when you available. I have made the changes as requested.
Thank you!

[Pablete1234]

Decided to move the code back to a more readable flow, fixed in 87a1692 , thanks for the PR anyways tho