Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#4500 closed defect (fixed)

Petra supports disabled templates, but not disabled technologies

Reported by: elexis Owned by: mimo
Priority: Should Have Milestone: Alpha 22
Component: AI Keywords:
Cc: Patch:

Description


Change History (11)

comment:1 by mimo, 7 years ago

Do you have any hint or test case where this is a problem? in principle, petra receives a list of tech it can research and choose the one it will research between them. So disabled techs will just be ignored.

comment:2 by elexis, 7 years ago

Polar sea is the first map using this feature of the player component (which could be used by other maps, mods or gamesettings). Doing a grep for "disabledTemplates" shows support for that, but the disbaledTechnologies part is absent.

comment:3 by mimo, 7 years ago

As i said, the way petra works does not need any such support.

comment:4 by elexis, 7 years ago

ERROR: This entity cannot research gather_lumbering_ironaxes

comment:5 by mimo, 7 years ago

exactly what was needed, usually it's better to start by giving such error messages in the tickets.

comment:7 by elexis, 7 years ago

Tested with the paste and it doesn't make a difference whether a TriggerScript or a JSON file is used.

comment:9 by mimo, 7 years ago

Owner: set to mimo
Resolution: fixed
Status: newclosed

In 19275:

petra support of disabled technologies, fixes #4500
Reviewed By: Sandarac
Differential Revision: https://code.wildfiregames.com/D178

comment:10 by mimo, 7 years ago

Milestone: BacklogAlpha 22

comment:11 by elexis, 7 years ago

thanks for implementing it, those errors were annoying

Note: See TracTickets for help on using tickets.