Zandronum Chat on our Discord Server Get the latest version: 3.1
Source Code
My View | View Issues | Change Log | Roadmap | Zandronum Issue Support Ranking | Rules | My Account
  

View Issue Details [ Jump to Notes ] [ Issue History ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0004160Zandronum[All Projects] Bugpublic2023-08-31 18:102024-03-23 20:58
ReporterKaapeli47 
Assigned ToKaminsky 
PrioritynormalSeverityminorReproducibilityalways
StatusresolvedResolutionfixed 
PlatformMicrosoftOSWindowsOS VersionXP/Vista/7
Product Version3.1 
Target Version3.2Fixed in Version3.2 
Summary0004160: Skin death sound does not play if the player respawns before death sound
DescriptionPlayer skin death sound does not play if the player leaves the corpse by respawning before A_PlayerDeath or A_XDeath happens. Instead, the default death sound plays, such as Doomguy's death scream in Doom.

This is most noticeable in classic Doom deathmatch servers with a player skin pack and Compat_InstantRespawn set to true.
Steps To ReproduceLoad a skin that uses one or more death sounds. Start a classic Doom deathmatch game with Compat_InstantRespawn set to true. Get fragged and try to respawn as soon as you can, at least before the death sound plays on your corpse. It should play Doomguy's death scream instead of the skin's.
Additional InformationIssue found in Euroboros classic Doom 2 deathmatch servers, such as [EB] Greenwar Deathmatch and [EB] Exec & Brit Deathmatch, which use IDMSkinpack_v1.36.pk3 as an optional file.

Temporary solution/workaround: Set Compat_InstantRespawn to false, and set SV_RespawnDelayTime to 0.3 seconds. This is enough time for the death sound to execute before the player respawns, so the skin's death sound actually plays. However, while it is hardly noticeable, the respawning isn't in fact instant anymore. This setting is now in use in the aforementioned Euroboros classic deathmatch servers after this issue was found.
Attached Files

- Relationships

-  Notes
User avatar (0023418)
Kaminsky (developer)
2024-03-17 17:59

 I created a new merge request to address this issue:'https://foss.heptapod.net/zandronum/zandronum-stable/-/merge_requests/68 [^]'
User avatar (0023419)
Kaminsky (developer)
2024-03-17 21:47

 This got pushed in:'https://foss.heptapod.net/zandronum/zandronum-stable/-/commit/75a8bf29c784ec8466d594767b0d72a6d053135d [^]'
User avatar (0023475)
Ru5tK1ng (updater)
2024-03-23 20:58

 Kaapeli tested out build 240317 that included this fix and reported the issue was resolved.

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
2023-08-31 18:10 Kaapeli47 New Issue
2024-03-17 17:59 Kaminsky Note Added: 0023418
2024-03-17 17:59 Kaminsky Assigned To => Kaminsky
2024-03-17 17:59 Kaminsky Status new => needs review
2024-03-17 17:59 Kaminsky Product Version 4.0-beta => 3.1
2024-03-17 17:59 Kaminsky Target Version => 3.2
2024-03-17 21:47 Kaminsky Note Added: 0023419
2024-03-17 21:47 Kaminsky Status needs review => needs testing
2024-03-23 20:58 Ru5tK1ng Note Added: 0023475
2024-03-23 20:58 Ru5tK1ng Status needs testing => resolved
2024-03-23 20:58 Ru5tK1ng Resolution open => fixed
2024-03-23 20:58 Ru5tK1ng Fixed in Version => 3.2




Questions or other issues? Contact Us.

Links

Copyright © 2000 - 2024 MantisBT Team
Powered by Mantis Bugtracker