-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Weapon shouldn't check warheads in IsValidAgainst #20369
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree
This PR may expose some incorrectly defined yaml and introduce regressions. I tested out s bunch of RA units and it seems they are defined correctly.
then we need some tester on CNC and D2K to see if there is yaml that hardcode warhead target for targetable |
Note: PR changed and add can target attacker itself option. It is due to there is a bug for repair vehicle repair itself when its projectile use LaserZap and target damage with speard = 0c20. I don't know if it is by design but we'd better give an option to allow that |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm otherwise.
Warheads should not block weapon from firing, since we already has InvalidTarget&ValidTarget in weapon. In this case, considering warheads block fire will be very confusing to user.
With this, we can also get more pref when weapon check fire.
Closes #16023