HomeWildfire Games

Some fixes after the ProductionQueue split.


Some fixes after the ProductionQueue split.

r26000 / rP2600 introduced a serialisation error (#6391).
Also the templates and the code could be improved.

Differential revision: https://code.wildfiregames.com/D4352
Comments by: @Silier, @Stan
Fixes: #6391

Event Timeline

Langbart raised a concern with this commit.Jul 21 2022, 1:13 PM
Langbart added a subscriber: Langbart.

s0600204 wrote:

The issue with the Structure Tree is that r26015 removed default Researcher/TechCostMultiplier values; and whilst r26192 fixed that for the simulation, a fix was not applied to globalscripts which expects there to be values for each possible resource specified within templates.
This is causing other technology costs to be displayed incorrectly within the Structure Tree, such as those of the Mauryan temple and the Gallic forge

This commit now has outstanding concerns.Jul 21 2022, 1:13 PM
Freagarach requested verification of this commit.Aug 11 2022, 6:40 AM
This commit now requires verification by auditors.Aug 11 2022, 6:40 AM
Langbart resigned from this commit.Aug 16 2022, 4:43 PM
This commit no longer requires audit.Aug 16 2022, 4:43 PM