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: Grant Faction Inherent Upgrades By Dummy Instead Of By Building, Version 2 #827

Merged
merged 2 commits into from
Aug 5, 2022

Conversation

commy2
Copy link
Collaborator

@commy2 commy2 commented Aug 5, 2022

Same as #539, but without taking another precious slot in StartingUnits.

@commy2 commy2 added the Bug Something is not working right label Aug 5, 2022
@xezon
Copy link
Collaborator

xezon commented Aug 5, 2022

I think this is very nice that we can bundle everything into one item. Well done.

Now for the core rationale of the change, I have not yet understood why it is a problem that a player gets the Faction perks when stealing their Dozer and building their Structures. Could you elaborate?

@commy2
Copy link
Collaborator Author

commy2 commented Aug 5, 2022

I think it is a very obscure mechanic. It is hard to understand why building a Infa War Factory would grant Nationalism. It's not just buildings either. Building a Stealthed Rebel grants Camouflage. Additionally, there are subtile bugs with this. Capturing an Infa War Factory does not grant Nationalism. You have to actually complete a scaffold for it to count. When the Stealthed Rebel comes from a Combat Bike or Rebel Ambush from a Stealth General Command Center, it does not grant Camouflage either. It has to be one trained from a Barracks.

It seems more intuitive to me that a faction would start with upgrades, and what happens with capturing their buildings / Dozers is entirely divorced from that.

@xezon
Copy link
Collaborator

xezon commented Aug 5, 2022

I just tested with Stealth General, built a Combat Bike, evacuated, and the Rebel was stealthed.

@commy2
Copy link
Collaborator Author

commy2 commented Aug 5, 2022

The Stealthed Rebel grants Camouflage, but does itself not need Camouflage to be stealthed.

Repro steps:

  • Do not build a Stealthed Rebel from Barracks.
  • Build a Comabt Bike
  • Capture a Tox Barracks
  • Build Toxin Rebel
  • Build another Combat Bike (optional)
  • Observe how Toxin Rebel is not stealthed
  • Build a Stealthed Rebel from Barracks
  • Observe how Toxin Rebel is now stelthed.

@xezon
Copy link
Collaborator

xezon commented Aug 5, 2022

Ok. Will this change affect veterancy level on unit spawn?

@commy2
Copy link
Collaborator Author

commy2 commented Aug 5, 2022

No.

Copy link
Collaborator

@xezon xezon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@commy2 commy2 merged commit 8f79de3 into main Aug 5, 2022
@commy2 commy2 deleted the fix-free-upgrades2 branch August 5, 2022 16:40
xezon pushed a commit that referenced this pull request Aug 28, 2022
@xezon xezon added the Minor Severity: Minor < Major < Critical < Blocker label Mar 19, 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 Minor Severity: Minor < Major < Critical < Blocker
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants