0001972: SetAmmoCapacity() not executed in ENTER script

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

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

Project

Category

View Status

Date Submitted

Last Update

0001972

Zandronum

[All Projects] Bug

public

2014-10-26 07:48

2014-10-26 09:11

Reporter

Borg

Assigned To

Priority

normal

Severity

major

Reproducibility

always

Status

closed

Resolution

won't fix

Platform

Microsoft

Windows

OS Version

XP/Vista/7

Product Version

1.3

Target Version

Fixed in Version

Summary

0001972: SetAmmoCapacity() not executed in ENTER script

Description

SetAmmoCapacity() is not executed in ENTER script in ACS on map change.
This render MaxAmount bug workaround in ACS useless.

Bug is server side only.

Additional Information

Please revert this code:

// [BB] If the activator is a player, tell the clients about the changed capacity.
-if ( activator->player && NETWORK_GetState() == NETSTATE_SERVER )
+// [BB] Only bother the clients if MaxAmount has actually changed.
+if ( activator->player && NETWORK_GetState() == NETSTATE_SERVER && ( oldMaxAmount != item->MaxAmount ) )
SERVERCOMMANDS_SetPlayerAmmoCapacity( activator->player - players, item );

Attached Files

Relationships

related to

0001868

closed

Torr Samaho

SetAmmoCapacity is not recognized after map is finished

Notes
(0010693) Dusk (developer) 2014-10-26 09:10	The real bug here is 0001868. The change mentioned in the description only breaks a hacky workaround. Nothing to do here.

**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
2014-10-26 07:48	Borg	New Issue
2014-10-26 09:10	Dusk	Note Added: 0010693
2014-10-26 09:10	Dusk	Relationship added	related to 0001868
2014-10-26 09:11	Dusk	Status	new => closed
2014-10-26 09:11	Dusk	Resolution	open => won't fix

Questions or other issues? Contact Us.

Links

Doom and Doom II are the property of id Software.