[New Feature] Allow BuildRefinery= to properly have multiple items

Vinifera-Developers / Vinifera

Vinifera is a C&C: Tiberian Sun engine extension implementing new logics and fixing bugs.

GNU General Public License v3.0

45 stars 10 forks source link

[New Feature] Allow BuildRefinery= to properly have multiple items #174

Open Rampastring opened 3 years ago

Rampastring commented 3 years ago

Description:

BuildRefinery= accepts multiple entries, but the game only looks at the first one of them. This prevents giving unique refineries to multiple factions with proper support for AI.

Additional Files:

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

CCHyper commented 3 years ago

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 commented 1 month ago

Resolved in #1073