Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#4001 closed defect (fixed)

[PATCH] Show units with researchable techs alongside units with trainable units

Reported by: s0600204 Owned by: sanderd17
Priority: Nice to Have Milestone: Alpha 21
Component: UI & Simulation Keywords: patch
Cc: Patch:

Description

As pointed out by wgoyc on the forums ​here, the structree doesn't like it when a unit has Technologies but not Entities in its ProductionQueue (if both exist, the Technologies are just ignored)

The attached patch remedies that situation, and places units with Technologies in their ProductionQueue (such as wgoyc's Delenda Est mod's version of the Mauryan hero Chanakya) with the units-that-can-train-units down the right-hand side of the structree.

Attachments (2)

chanakyaTechsSupport.patch​ (2.7 KB ) - added by s0600204 8 years ago.
Proposed patch
chanakyaTechsSupport.v2.patch​ (2.7 KB ) - added by s0600204 8 years ago.
Comparison operator tweak as implicitly suggested by fatherbushido on irc

Download all attachments as: .zip

Change History (4)

by s0600204, 8 years ago

Attachment: chanakyaTechsSupport.patch​ added

Proposed patch

by s0600204, 8 years ago

Attachment: chanakyaTechsSupport.v2.patch​ added

Comparison operator tweak as implicitly suggested by fatherbushido on irc

comment:1 by sanderd17, 8 years ago

Owner: set to sanderd17
Resolution: → fixed
Status: new → closed

In 18218:

Add support to the structure tree for units with technologies in their production queue. Patch by s0600204. Fixes #4001

comment:2 by sanderd17, 8 years ago

Keywords: review removed
Note: See TracTickets for help on using tickets.