Opened 8 years ago
Closed 8 years ago
#3630 closed defect (fixed)
[PATCH] This.orderqueue is undefined
Reported by: | Lionkanzen | Owned by: | mimo |
---|---|---|---|
Priority: | Must Have | Milestone: | Alpha 20 |
Component: | UI & Simulation | Keywords: | |
Cc: | Patch: |
Description
I need some help with this warning: WARNING: JavaScript warning: simulation/components/UnitAI.js line 1941 reference to undefined property this.orderQueue[0].data.attackType
WARNING: unexpected PathResult (2475 3 3)
WARNING: unexpected PathResult (2560 3 3)
WARNING: unexpected PathResult (2266 3 3)
WARNING: JavaScript warning: simulation/components/UnitAI.js line 1710 reference to undefined property this.order.data.attackType
Engine exited successfully on 2015-11-20 at 20:15:35 with 4653 message(s), 0 error(s) and 5 warnin
Attachments (3)
Change History (7)
by , 8 years ago
Attachment: | commands.txt added |
---|
by , 8 years ago
Attachment: | commands.2.txt added |
---|
comment:1 by , 8 years ago
Keywords: | simulation removed |
---|---|
Milestone: | Alpha 19 → Alpha 20 |
Summary: | new issue (Named as you want) Simulation → This.orderqueue is undefined |
by , 8 years ago
Attachment: | idle_t3630.patch added |
---|
comment:3 by , 8 years ago
Summary: | This.orderqueue is undefined → [PATCH] This.orderqueue is undefined |
---|
This bug had the same origine as #3466, going to a transitory IDLE state which should not add new orders, so the fix is to move the fix of #3466 towards the top of the idle enter function, before any source of new orders (Guard order here). As this was unnoticed up to now, it means that nobody on svn uses the Guard command, so probably not worth delaying a19, but if we have to redo the packages for any other reason, this fix should be included.
Pushing it to A20 as the release is imminent and wont have this fixed.