Skip to content
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

Fix: Prevent Hijacker From Stealing Combat Bike #879

Merged
merged 1 commit into from
Aug 14, 2022

Conversation

commy2
Copy link
Collaborator

@commy2 commy2 commented Aug 12, 2022

This pull request makes Combat Bikes immune to being hijacked.

Rationale

Logically it makes no sense to steal a Combat Bike with rider on top when you think about it. And it leads to funny bugs, worst of which is trapping an enemy Jarmen Kell.

@commy2 commy2 added Bug Something is not working right Minor Severity: Minor < Major < Critical < Blocker labels Aug 12, 2022
@xezon
Copy link
Collaborator

xezon commented Aug 13, 2022

In this change you do touch GC units. In some other change you advised me to not change GC units. Any guidelines on when and when not to touch?

As for the essence of this change, this looks good to me. Combat Bikes are practically never stolen in regular competitive matches.

@commy2
Copy link
Collaborator Author

commy2 commented Aug 13, 2022

In some other change you advised me to not change GC units.

Huh, where? Always fix them, as they appear in the campaign.

@xezon
Copy link
Collaborator

xezon commented Aug 13, 2022

Ups. I can not recall which one. But it was a recent one.

@commy2
Copy link
Collaborator Author

commy2 commented Aug 13, 2022

All I remember is that I said not to bother with CINE units.

@xezon
Copy link
Collaborator

xezon commented Aug 13, 2022

Ah. That could be yes. Perhaps I am confused.

@xezon xezon merged commit 4d535a7 into main Aug 14, 2022
@xezon xezon deleted the fix-hijacker-combat-bike branch August 14, 2022 12:40
@xezon xezon added GLA Affects GLA faction Controversial Is controversial labels Mar 19, 2023
@commy2 commy2 mentioned this pull request Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is not working right Controversial Is controversial GLA Affects GLA faction Minor Severity: Minor < Major < Critical < Blocker
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Player cannot build new GLA Jarmen Kell if Jarmen on Combat Cycle was hijacked by Hijacker
2 participants