Opened 3 years ago
Last modified 16 months ago
#6265 new defect
Formation.js this.offsets is undefined but has no reason
Reported by: | Silier | Owned by: | |
---|---|---|---|
Priority: | Must Have | Milestone: | Backlog |
Component: | Simulation | Keywords: | |
Cc: | Patch: |
Description (last modified by )
ERROR: JavaScript error: simulation/components/Formation.js line 531 this.offsets is undefined
Formation.prototype.MoveMembersIntoFormation@simulation/components/Formation.js:531:18
Formation.prototype.RemoveMembers@simulation/components/Formation.js:400:7
Formation.prototype.OnGlobalOwnershipChanged@simulation/components/Formation.js:960:8
ERROR: Script message handler OnGlobalOwnershipChanged failed
Strange about this complain is existing if above
if (!this.offsets) { this.offsets = this.ComputeFormationOffsets(active, positions); offsetsChanged = true; }
ComputeFormationOffsets
goes like this
... let newOffsets = []; ... return newOffsets;
There is no early return inside ComputeFormationOffsets
which would return undefined
.
There is also no code between those lines which would be able to set this.offsets
to undefined
Attachments (1)
Change History (4)
by , 3 years ago
Attachment: | commands.txt added |
---|
comment:1 by , 3 years ago
Description: | modified (diff) |
---|
comment:2 by , 2 years ago
Milestone: | Alpha 26 → Alpha 27 |
---|
a24 by Yekaterina