Zandronum Chat on our Discord Server Get the latest version: 3.2
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
0001425Zandronum[All Projects] Bugpublic2013-07-22 18:422014-06-21 01:36
ReporterRusselCS 
Assigned To 
PrioritylowSeverityminorReproducibilityN/A
StatusnewResolutionopen 
PlatformOSOS Version
Product Version1.1.1 
Target VersionFixed in Version 
Summary0001425: Morphed players do not transfer their skin
DescriptionMore accurately, skins on morphed player classes are not kept at all.
Previously, in Skulltag, morphed player classes kept the skin that they previously held upon morphing unless the morphed class had +NOSKIN. Currently, morphed classes lose their skin entirely.
While this may not seem like much of an issue, the class still retains other features the skin had, such as the taunt, jump, pain, landing, grunting, use fail, gibbed, and death sounds.

My primary concern is whether or not this functionality was intended. If it was, then the ticket can be closed here and now. Otherwise, this is just a small issue that may need to be addressed at some point.
Attached Files? file icon morphtest.wad [^] (903 bytes) 2013-07-25 19:58

- Relationships

-  Notes
User avatar (0006753)
Torr Samaho (administrator)
2013-07-22 18:57

 The old behavior was a bug fixed in changeset b5f974e3d6ac6bda5e51e28809fab6921d8e1c42.

I'd say morphed class should also not use any sounds of the skin. How does ZDoom handle this?
User avatar (0006756)
RusselCS (reporter)
2013-07-22 20:47

 I am honestly unsure, until now I never even realized ZDoom had effective skin support.
User avatar (0006793)
Torr Samaho (administrator)
2013-07-25 18:36

 Can you test how ZDoom handles this?
User avatar (0006800)
RusselCS (reporter)
2013-07-25 19:59
edited on: 2013-07-25 20:16

 Dang, I was dumb and thought SKININFO in ZDoom actually worked...let me fix that by using S_SKIN instead.

EDIT: Alright, the old behavior Skulltag held was the correct behavior. Skins should be preserved when morphed.
I will continue experimentation, however, as I am unsure whether this only occurs under certain circumstances such as the morphed class having the same base skin, or whatever.

EDIT 2:As of ZDoom 2.7.1, the morphing is handled in such a way that regardless of the base skin sprites of the destination actor, the morphed class preserves the skin set by the player. In the instance of NOSKIN, this behavior is nullified.
User avatar (0006803)
Edward-san (developer)
2013-07-25 23:03
Quote

Skins should be preserved when morphed.

doesn't this mean "goodbye Zombie Horde"? Just sayin'.
User avatar (0006806)
RusselCS (reporter)
2013-07-26 01:48

 I already mentioned, if +NOSKIN is added to the morphed class in ZDoom, then the class will then just use the base class skin.

Not to mention, this was the functionality of older skulltag versions, which I have stated at least three times now.
User avatar (0009308)
Watermelon (developer)
2014-06-14 17:30

 Does this still happen in 2.0?
User avatar (0009341)
RusselCS (reporter)
2014-06-14 19:05

 Unchanged on Zandronum 2.0-alpha-140112-1124.
User avatar (0009343)
Watermelon (developer)
2014-06-14 21:41

 Now the big important one:
Does it happen in zdoom latest?

If it is fine in zdoom 2.7.1 (most recent version)... check 2.5.0 (to see if it's broken in zandronum)


2.7.1:'http://www.zdoom.org/files/zdoom/2.7/zdoom-2.7.1.zip [^]'
2.5.0:'http://www.zdoom.org/files/zdoom/2.5/zdoom-2.5.0.zip [^]'
User avatar (0009607)
RusselCS (reporter)
2014-06-20 20:47

 I mentioned in an earlier note that in ZDoom 2.7.1 players transfer their skin to morphed classes assuming the morphed class lacks the NOSKIN actor flag.
This behavior is the same in ZDoom 2.5.0.

My apologies for the late response and thank you for your attention.

Issue Community Support
Only registered users can voice their support. Click here to register, or here to log in.
Supporters: No one explicitly supports this issue yet.
Opponents: No one explicitly opposes this issue yet.

- Issue History
Date Modified Username Field Change
2013-07-22 18:42 RusselCS New Issue
2013-07-22 18:57 Torr Samaho Note Added: 0006753
2013-07-22 20:47 RusselCS Note Added: 0006756
2013-07-25 18:36 Torr Samaho Note Added: 0006793
2013-07-25 18:36 Torr Samaho Status new => feedback
2013-07-25 19:58 RusselCS File Added: morphtest.wad
2013-07-25 19:59 RusselCS Note Added: 0006800
2013-07-25 19:59 RusselCS Status feedback => new
2013-07-25 20:11 RusselCS Note Edited: 0006800 View Revisions
2013-07-25 20:16 RusselCS Note Edited: 0006800 View Revisions
2013-07-25 23:03 Edward-san Note Added: 0006803
2013-07-26 01:48 RusselCS Note Added: 0006806
2014-06-14 17:30 Watermelon Note Added: 0009308
2014-06-14 17:30 Watermelon Status new => feedback
2014-06-14 19:05 RusselCS Note Added: 0009341
2014-06-14 19:05 RusselCS Status feedback => new
2014-06-14 21:41 Watermelon Note Added: 0009343
2014-06-14 21:41 Watermelon Status new => feedback
2014-06-20 20:47 RusselCS Note Added: 0009607
2014-06-20 20:47 RusselCS Status feedback => new
2014-06-21 01:36 StrikerMan780 Note Added: 0009608
2014-06-21 01:38 StrikerMan780 Note Deleted: 0009608




Questions or other issues? Contact Us.

Links

Copyright © 2000 - 2025 MantisBT Team
Powered by Mantis Bugtracker