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 Silier)

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)

commands.txt (1.4 MB ) - added by Silier 3 years ago.
a24 by Yekaterina

Download all attachments as: .zip

Change History (4)

by Silier, 3 years ago

Attachment: commands.txt added

a24 by Yekaterina

comment:1 by Silier, 3 years ago

Description: modified (diff)

comment:2 by Freagarach, 2 years ago

Milestone: Alpha 26Alpha 27

comment:3 by Freagarach, 16 months ago

Milestone: Alpha 27Backlog

Pushing back.

Note: See TracTickets for help on using tickets.