MantisBT - Zandronum
View Issue Details
0000415Zandronum[All Projects] Bugpublic2011-04-28 08:462018-09-30 20:02
unknownna 
Torr Samaho 
normalmajoralways
closedfixed 
98d 
1.0 
0000415: Newly connected clients think that other clients use the same "handicap" value
Newly connected clients think that other clients are using their "handicap" value.
1. Start a TDM server.
2. Add a bot.
3. Connect a client to the server with "handicap" set to a value higher than 0.
No tags attached.
png Screenshot_Doom_20110428_104431.png (25,674) 2011-04-28 08:46
/tracker/file_download.php?file_id=286&type=bug
png

png Screenshot_Doom_20110428_123937.png (34,928) 2011-04-28 10:42
/tracker/file_download.php?file_id=287&type=bug
png
Issue History
2011-04-28 08:46unknownnaNew Issue
2011-04-28 08:46unknownnaFile Added: Screenshot_Doom_20110428_104431.png
2011-04-28 08:47unknownnaSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=777#r777
2011-04-28 10:42unknownnaNote Added: 0001493
2011-04-28 10:42unknownnaFile Added: Screenshot_Doom_20110428_123937.png
2011-04-28 10:43unknownnaSeverityminor => major
2011-05-21 19:55Torr SamahoNote Added: 0001715
2011-05-21 19:55Torr SamahoAssigned To => Torr Samaho
2011-05-21 19:55Torr SamahoStatusnew => feedback
2011-05-21 20:35unknownnaNote Added: 0001717
2011-05-21 20:35unknownnaStatusfeedback => assigned
2011-05-21 21:34Torr SamahoNote Added: 0001719
2011-05-21 21:38unknownnaNote Added: 0001720
2011-05-21 21:47unknownnaNote Edited: 0001720bug_revision_view_page.php?bugnote_id=1720#r877
2011-05-21 21:48unknownnaNote Edited: 0001720bug_revision_view_page.php?bugnote_id=1720#r878
2011-05-22 16:24Torr SamahoNote Added: 0001734
2011-05-22 16:24Torr SamahoStatusassigned => feedback
2011-05-22 16:32Torr SamahoNote Edited: 0001734bug_revision_view_page.php?bugnote_id=1734#r882
2011-05-22 16:34Torr SamahoNote Edited: 0001734bug_revision_view_page.php?bugnote_id=1734#r883
2011-05-22 20:21unknownnaNote Added: 0001739
2011-05-22 20:21unknownnaStatusfeedback => assigned
2011-05-22 21:11Torr SamahoStatusassigned => resolved
2011-05-22 21:11Torr SamahoFixed in Version => 1.0
2011-05-22 21:11Torr SamahoResolutionopen => fixed
2012-06-09 13:22Torr SamahoCategoryGeneral => Bug
2018-09-30 20:02Blzut3Statusresolved => closed

Notes
(0001493)
unknownna   
2011-04-28 10:42   
Newly added bots inherit your "handicap" value if you disconnect from the server.

1. Start a standard server.
2. Connect a client to the server with "handicap" set to a value higher than 0.
3. "disconnect" in the console.
4. "addbot" in the server console.
5. "reconnect" in the console.
(0001715)
Torr Samaho   
2011-05-21 19:55   
This should take care of both issue. It also adjusts coopinfo to hide the health and armor of players the client isn't allowed to know.
(0001717)
unknownna   
2011-05-21 20:35   
It fixed the issues. But clients can still see the "handicap" value of other clients by looking at the mugshot.

> It also adjusts coopinfo to hide the health and armor of players the client isn't allowed to know.

0000286: Problem with "see player status" on competitive team games

You might want to hide the ammo as well (sv_infiniteammo 1) to make it consistent with the coop spy view.
(0001719)
Torr Samaho   
2011-05-21 21:34   
> But clients can still see the "handicap" value of other clients by looking at the mugshot.

The handicap value of the other clients is not secret. Or are you referring to the fact that the mugshot should always look like the player has full health if the client doesn't know the exact health?
(0001720)
unknownna   
2011-05-21 21:38   
(edited on: 2011-05-21 21:48)
> Or are you referring to the fact that the mugshot should always look like the player has full health if the client doesn't know the exact health?

This.

> The handicap value of the other clients is not secret.

I see, so the current mugshot behavior isn't broken.

(0001734)
Torr Samaho   
2011-05-22 16:24   
(edited on: 2011-05-22 16:34)
This should always show the full health mugshot for players whose health is secret to the local player. Since the health of those players is not synced to the local client, it's better not to show the mugshot based on the health predicted by the client.

> You might want to hide the ammo as well (sv_infiniteammo 1) to make it consistent with the coop spy view.

The sv_infiniteammo handling is already in: If sv_infiniteammo is 1, instead of the amount a red '-' is drawn. Whether or not the ammo should always be hidden if it is hidden in coop spy should be discussed in 0000286.

(0001739)
unknownna   
2011-05-22 20:21   
> This should always show the full health mugshot for players whose health is secret to the local player.

It fixed the issue.