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

[New Feature] Allow HarvesterUnit= to properly support more than 1 entry #176

Open
Rampastring opened this issue May 13, 2021 · 2 comments · May be fixed by #1122
Open

[New Feature] Allow HarvesterUnit= to properly support more than 1 entry #176

Rampastring opened this issue May 13, 2021 · 2 comments · May be fixed by #1122
Labels
enhancement Enhancement to original behaviour. new-feature New feature or request.
Milestone

Comments

@Rampastring
Copy link
Member

Description:

HarvesterUnit= is a list, but the AI only fetches the first item when figuring out whether it needs to build harvesters (and which harvester to build). This makes it impossible to properly support different harvesters for different factions so that the AI would also utilize the correct harvester for each faction.

Additional Files:

Fixed in CnCNet ts-patches: https://github.com/CnCNet/ts-patches/blob/master/src/mods/buildconst_harvesterunit_baseunit.asm

@Rampastring Rampastring added the new-feature New feature or request. label May 13, 2021
@Rampastring Rampastring added this to the TS-Patches milestone May 13, 2021
@CCHyper
Copy link
Contributor

CCHyper commented May 13, 2021

I would say this is more of a modding bug-fix than a new feature. The programmers expected only one entry in the RULES.INI, so correcting the code to search the vector array just like Red Alert 2 does is the best approach.

@ZivDero
Copy link
Member

ZivDero commented Oct 6, 2024

Resolved in #1073

@ZivDero ZivDero closed this as completed Oct 6, 2024
@ZivDero ZivDero reopened this Oct 28, 2024
@ZivDero ZivDero linked a pull request Oct 28, 2024 that will close this issue
@CCHyper CCHyper removed their assignment Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement to original behaviour. new-feature New feature or request.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants