Opened 2 years ago

Closed 2 years ago

#6408 closed defect (fixed)

Cheat code behaviour change for technology research time

Reported by: Langbart Owned by: Freagarach
Priority: Must Have Milestone: Alpha 26
Component: Simulation Keywords: regression
Cc: Patch: ​Phab:D4409

Description

While testing the latest GIT version [26136], I noticed that the cheat code gift from the gods no longer reduces the research time for researched technology to zero.

reproduce

  • Start a game
  • Enter gift from the gods
  • Click the civic center and select Cartography
  • Research time is not zero

bisect

Doing git bisect, the behaviour change starts with changeset [26015].

The ticket can be closed if the change was intentional.

Change History (2)

comment:1 by Freagarach, 2 years ago

Component: UI – In-game → Simulation
Keywords: regression added
Owner: set to Freagarach
Patch: → Phab:D4409
Priority: If Time Permits → Must Have

comment:2 by Freagarach, 2 years ago

Resolution: → fixed
Status: new → closed

In 26192:

Fix modifications to unspecified techCostMultipliers.

r26015 / rP26015 removed the values for the tech cost multipliers from the templates, since the component uses those default values, but failed to account for modifications thereof.
Those weren't applied since the template value didn't exist.

This changes that to iterating over all resources.

Reported by: @Langbart
Differential revision: ​https://code.wildfiregames.com/D4409
Tested by: @Langbart
Fixes: #6408

Note: See TracTickets for help on using tickets.