Fixed an issue where thing positions were modified even when the new position was the same as the old one. Fixes #644

This commit is contained in:
biwa 2021-10-25 00:32:10 +02:00 committed by spherallic
parent 22dfd69bbf
commit b48806838d

View file

@ -416,6 +416,8 @@ namespace CodeImp.DoomBuilder.Map
// This moves the thing // This moves the thing
// NOTE: This does not update sector! (call DetermineSector) // NOTE: This does not update sector! (call DetermineSector)
public void Move(Vector3D newpos) public void Move(Vector3D newpos)
{
if (newpos != pos)
{ {
BeforePropsChange(); BeforePropsChange();
@ -425,31 +427,31 @@ namespace CodeImp.DoomBuilder.Map
if (SRB2Type != General.Map.Config.Start3DModeThingType) if (SRB2Type != General.Map.Config.Start3DModeThingType)
General.Map.IsChanged = true; General.Map.IsChanged = true;
} }
}
// This moves the thing // This moves the thing
// NOTE: This does not update sector! (call DetermineSector) // NOTE: This does not update sector! (call DetermineSector)
public void Move(Vector2D newpos) public void Move(Vector2D newpos)
{
Vector3D p = new Vector3D(newpos.x, newpos.y, pos.z);
if (p != pos)
{ {
BeforePropsChange(); BeforePropsChange();
// Change position // Change position
this.pos = new Vector3D(newpos.x, newpos.y, pos.z); this.pos = p;
if (SRB2Type != General.Map.Config.Start3DModeThingType) if (SRB2Type != General.Map.Config.Start3DModeThingType)
General.Map.IsChanged = true; General.Map.IsChanged = true;
} }
}
// This moves the thing // This moves the thing
// NOTE: This does not update sector! (call DetermineSector) // NOTE: This does not update sector! (call DetermineSector)
public void Move(float x, float y, float zoffset) public void Move(float x, float y, float zoffset)
{ {
BeforePropsChange(); Move(new Vector3D(x, y, zoffset));
// Change position
this.pos = new Vector3D(x, y, zoffset);
if(SRB2Type != General.Map.Config.Start3DModeThingType)
General.Map.IsChanged = true;
} }
// This rotates the thing // This rotates the thing