0002119: Server doesn't inform the client of actor TIDs with ACS Spawn/SpawnSpawn functions

Chat on our Discord Server Get the latest version: 3.2
Source Code

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

Project

Category

View Status

Date Submitted

Last Update

0002119

Zandronum

[All Projects] Bug

public

2015-03-07 16:11

2018-09-30 23:38

Reporter

haxmurderer

Assigned To

Dusk

Priority

normal

Severity

minor

Reproducibility

always

Status

closed

Resolution

fixed

Platform

Microsoft

Windows

OS Version

XP/Vista/7

Product Version

1.3

Target Version

1.4

Fixed in Version

1.4

Summary

0002119: Server doesn't inform the client of actor TIDs with ACS Spawn/SpawnSpawn functions

Description

When spawning an actor with Spawn() or SpawnSpot() in ACS, one of the parameters you can pass is the "newtid" parameter, which automatically assigns the newly spawned actor the given TID.

However, this TID is not sent to the client, and if used in any clientside scripts, ACS functions will behave as if the TID doesn't exist.

A workaround is to immediately call Thing_ChangeTID(newtid, newtid) after using Spawn() or SpawnSpot(). I've confirmed that this does inform the client that the actor has the new TID.

Steps To Reproduce

In a server-side script, call Spawn("Imp", 8, 8, 8, 1).
In a client-side script, call Print(d:GetActorX(1)) in a timer loop.

Try this in multiplayer. The client-side script will always print 0, even as the imp walks around and changes its X coordinate.

To test the workaround, add a Thing_ChangeTID(1, 1) after the Spawn() call on the server-side.

Attached Files

bug2119.wad [^] (3,805 bytes) 2015-03-07 16:30

Relationships

Notes
(0011801) Dusk (developer) 2015-03-09 06:15 edited on: 2015-03-09 10:34	This should fix that Patch here

(0011805) cobalt (updater) 2015-03-09 20:52	Issue addressed by commit d5f445e1ffce: - fixed: Spawn() and friends did not update the spawned actor(s)'s TID to clients (fixes 2119) Committed by Teemu Piippo [Dusk] on Monday 09 March 2015 08:16:40 Changes in files: docs/zandronum-history.txt \| 1 + src/p_acs.cpp \| 4 ++++ 2 files changed, 5 insertions(+), 0 deletions(-)

**Issue Community Support**
This issue is already marked as resolved. If you feel that is not the case, please reopen it and explain why.
Supporters:	No one explicitly supports this issue yet.
Opponents:	No one explicitly opposes this issue yet.

Issue History
Date Modified	Username	Field	Change
2015-03-07 16:11	haxmurderer	New Issue
2015-03-07 16:29	Dusk	Assigned To	=> Dusk
2015-03-07 16:29	Dusk	Status	new => assigned
2015-03-07 16:30	haxmurderer	File Added: bug2119.wad
2015-03-09 06:15	Dusk	Note Added: 0011801
2015-03-09 06:15	Dusk	Status	assigned => needs review
2015-03-09 10:34	Dusk	Note Edited: 0011801	View Revisions
2015-03-09 20:52	cobalt	Status	needs review => needs testing
2015-03-09 20:52	cobalt	Target Version	=> 1.4
2015-03-09 20:52	cobalt	Description Updated	View Revisions
2015-03-09 20:52	cobalt	Steps to Reproduce Updated	View Revisions
2015-03-09 20:52	cobalt	Note Added: 0011805
2015-03-16 01:23	Dusk	Status	needs testing => resolved
2015-03-16 01:23	Dusk	Fixed in Version	=> 1.4
2015-03-16 01:23	Dusk	Resolution	open => fixed
2018-09-30 23:38	Blzut3	Status	resolved => closed

Questions or other issues? Contact Us.

Links

Doom and Doom II are the property of id Software.