Opened 8 years ago
Last modified 8 years ago
#4251 closed defect
[PATCH] Paired techs don't work with the multiple commands handling — at Version 5
Reported by: | Itms | Owned by: | Imarok |
---|---|---|---|
Priority: | Release Blocker | Milestone: | Alpha 21 |
Component: | UI & Simulation | Keywords: | patch |
Cc: | Patch: |
Description (last modified by )
Reported here: https://wildfiregames.com/forum/index.php?/topic/21145-latest-commit-breaks-paired-techs/
In 0ad without mods, there is the paired techs for walls and Seleucids have also paired techs in fortress, if you want to test that.
Change History (7)
comment:1 by , 8 years ago
Priority: | Should Have → Must Have |
---|
comment:2 by , 8 years ago
Keywords: | patch review added |
---|---|
Summary: | Paired techs don't work with the multiple commands handling → [PATCH] Paired techs don't work with the multiple commands handling |
Fixed initial bug: http://pastebin.com/1TbwFYad (github mirror is not updated, and I'm not sure how to produce an svn patch)
comment:4 by , 8 years ago
fix paired techs appear multiple (see second post in forum thread): http://pastebin.com/nzVKLHBJ (this also contains the additional two fixes)
comment:5 by , 8 years ago
Description: | modified (diff) |
---|---|
Priority: | Must Have → Release Blocker |
by , 8 years ago
Attachment: | 4251_fix.2.patch added |
---|
Same as the paste, but replaced JSON.stringify with uneval and change warn
to error
Note:
See TracTickets
for help on using tickets.
A second non-trivial bug found by fatherbushido: