#3401 closed defect (fixed)
[PATCH] Structure Tree - Aristeia's egyptian phase-pair technologies cause errors
Reported by: | s0600204 | Owned by: | s0600204 |
---|---|---|---|
Priority: | Should Have | Milestone: | Alpha 19 |
Component: | UI & Simulation | Keywords: | patch |
Cc: | Patch: |
Description
To reproduce:
- Run 0ad with the aristeia mod enabled
- Go to the structure tree (either off the main menu, or by clicking on the emblem top-centre in-game)
- Choose the 'Egyptian' civ from the dropdown
- Notice the error message
The error is caused because the mod's Egyptian civ presents a choice of two possible city-phase techs to the player, and the structree is unable to cope with this.
The attached patch fixes:
- The structree's inability to deal with a phase-pair
- The fact that units dependant on said phase-pair were not appearing where they should or, in fact, at all
Attachments (1)
Change History (4)
by , 9 years ago
Attachment: | structreePhasePairFix.patch added |
---|
comment:1 by , 9 years ago
Milestone: | Backlog → Alpha 19 |
---|
comment:2 by , 9 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:3 by , 9 years ago
Keywords: | review removed |
---|
I just realized my commit message is not really accurate. Anyway, I enhanced the style a bit (trailing space, more curly braces, and early returns).
Thanks for this patch :)
Note:
See TracTickets
for help on using tickets.
In 17073: