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

Citizen Asset References in Code #111

Open
jdrueckert opened this issue Nov 8, 2020 · 0 comments
Open

Citizen Asset References in Code #111

jdrueckert opened this issue Nov 8, 2020 · 0 comments

Comments

@jdrueckert
Copy link
Member

When replacing the placeholder gooey models for bad and good citizens, I added the respective assets named [bad|good]Robot and removed the old [bad|good]Citizen assets. While the prefabs could be spawned perfectly well with spawnPrefab [bad|good]Robot, the citizen spawning mechanism in the MetalRenegades cities and the Minimap didn't work. Assumedly it still tried to use the citizen assets instead of the new robot assets.

I found mentions in FactionAlignmentSystem.java:48 and CharacterOverlay.java:62, but adjusting these to the robot asset names didn't help either. So, this seems to be referenced somewhere in the MR code in a way that is not easy to find with grep or similar tools.

As a result, I created #110 and renamed all assets to the citizen variant again which fixed things. But this kind of hard-coded reference should be removed as it complicates any changes in that area very much.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant