#2617 closed defect (fixed)
[PATCH] Bug in Aegis/Headquarters.js
| Reported by: | False Vision | Owned by: | mimo |
|---|---|---|---|
| Priority: | Must Have | Milestone: | Alpha 17 |
| Component: | AI | Keywords: | patch |
| Cc: | Patch: |
Description
There are problems with some variables. Defines a variable and used an undefined one.
It seems like a mistake when copy-pasting a lot of code. Have a look to get my point.
Attachments (3)
Change History (8)
by , 10 years ago
| Attachment: | headquarters.js.patch added |
|---|
by , 10 years ago
| Attachment: | plan-transport.js.patch added |
|---|
comment:1 by , 10 years ago
by , 10 years ago
| Attachment: | naval-manager.js.patch added |
|---|
comment:3 by , 10 years ago
Thanks for the patches falsevision. I'll have a look at them tomorrow. But be aware that naval support was never finished inside Aegis, and thus never activated. So neither plan-transport.js nor naval-manager.js were ever used inside Aegis. In addition, Aegis was not updated since months and I'm not sure its developper will resume it now that Petra is much more advanced. So I would advice you to switch to petra for bug hunting :-) That would be really useful as Petra is now our default AI.
comment:5 by , 10 years ago
| Keywords: | review removed |
|---|---|
| Milestone: | Alpha 18 → Alpha 17 |
Note:
See TracTickets
for help on using tickets.
I just found another bug in Aegis/plan-transport.js too.
There are two attachments. please consider both of them.