MantisBT - Zandronum |
View Issue Details |
|
ID | Project | Category | View Status | Date Submitted | Last Update |
0001262 | Zandronum | [All Projects] Bug | public | 2013-01-28 00:10 | 2018-09-30 20:25 |
|
Reporter | Dusk | |
Assigned To | Dusk | |
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | |
Platform | | OS | | OS Version | |
Product Version | | |
Target Version | 1.1 | Fixed in Version | 1.1 | |
|
Summary | 0001262: Freeze hue stays after joining spectators |
Description | If the player dies by freezing and spectates before respawning, the blue freeze hue will stay indefinitely. If the player then respawns, the hue is cleared.
Happens both in offline and online. |
Steps To Reproduce | - Load up example wad
- Move to the blue pad, which triggers an ice rocket trap
- Once dead to the ice rocket, join spectators |
Additional Information | |
Tags | No tags attached. |
Relationships | |
Attached Files | FreezeHueTest.wad.zip (1,847) 2013-01-28 00:10 /tracker/file_download.php?file_id=927&type=bug |
|
Issue History |
Date Modified | Username | Field | Change |
2013-01-28 00:10 | Dusk | New Issue | |
2013-01-28 00:10 | Dusk | File Added: FreezeHueTest.wad.zip | |
2013-01-28 00:10 | Dusk | Status | new => confirmed |
2013-01-28 13:54 | Dusk | Assigned To | => Dusk |
2013-01-28 13:54 | Dusk | Status | confirmed => assigned |
2013-01-28 14:24 | Dusk | Note Added: 0005843 | |
2013-01-28 14:24 | Dusk | Status | assigned => needs review |
2013-01-28 20:42 | Torr Samaho | Note Added: 0005847 | |
2013-01-28 20:43 | Torr Samaho | Status | needs review => feedback |
2013-01-29 06:15 | Torr Samaho | Note Added: 0005854 | |
2013-01-29 11:28 | Dusk | Note Added: 0005857 | |
2013-01-29 11:28 | Dusk | Status | feedback => assigned |
2013-01-29 11:28 | Dusk | Status | assigned => needs review |
2013-01-30 06:18 | Torr Samaho | Note Added: 0005866 | |
2013-01-30 06:20 | Torr Samaho | Status | needs review => needs testing |
2013-01-30 06:20 | Torr Samaho | Target Version | => 1.1 |
2013-01-30 12:01 | Dusk | Note Added: 0005869 | |
2013-04-05 17:52 | Arco | Note Added: 0006232 | |
2013-04-05 19:33 | Dusk | Status | needs testing => resolved |
2013-04-05 19:33 | Dusk | Fixed in Version | => 1.1 |
2013-04-05 19:33 | Dusk | Resolution | open => fixed |
2018-09-30 20:25 | Blzut3 | Status | resolved => closed |
Notes |
|
(0005843)
|
Dusk
|
2013-01-28 14:24
|
|
|
|
|
Your patch should work, but it may be better to reset DamageType in PLAYER_SetDefaultSpectatorValues. That's where all the other special conditions (like the green and red screen tints) are reset and that function should cover all possible ways how a player can turn into a spectator. |
|
|
|
BTW: I can make these kind of tiny adjustments on my own and still make the commits in your name, if you'd like me to. |
|
|
(0005857)
|
Dusk
|
2013-01-29 11:28
|
|
|
|
|
Thanks! The code is now perfect. Pulled.
Just a small remark for future collapsing: It would be nice if the log message only contained what actually happened in the end and not the intermediate steps (like Moved the damagetype clearing to player_setdefaultspectatorvalues). After the collapsing this information lost its meaning. |
|
|
(0005869)
|
Dusk
|
2013-01-30 12:01
|
|
Quote Just a small remark for future collapsing: It would be nice if the log message only contained what actually happened in the end and not the intermediate steps
I know - I just happened to already had pushed the changes and only then noticed on Bitbucket that it contained the other log message too - whoops. |
|
|
(0006232)
|
Arco
|
2013-04-05 17:52
|
|
|