MantisBT - Zandronum
View Issue Details
0000414Zandronum[All Projects] Bugpublic2011-04-27 12:262018-09-30 20:04
unknownna 
Torr Samaho 
normalminoralways
closedfixed 
98d 
1.0 
0000414: Player turns into a team spectator after a changemap map change if sv_maxplayers is altered
It seems that you can turn into a team spectator if sv_maxplayers is altered before a changemap map change.
1. Start a TDM server with sv_keepteams, sv_noteamselect and sv_noteamswitch set to 1.
2. Connect a client to the server.
3. Join the game.
4. Add 2 bots.
5. "sv_maxplayers 2" in the server console.
6. "changemap map01" in the server console.
I managed to get rid of my team in TeamPossession. I also managed to make 2 hellstones appear on the map after a changemap map change/reset. But I don't know how to reproduce these issues.
No tags attached.
png Screenshot_Doom_20110427_141527.png (31,153) 2011-04-27 12:26
/tracker/file_download.php?file_id=282&type=bug
png

png Screenshot_Doom_20110427_151723.png (31,015) 2011-04-27 13:47
/tracker/file_download.php?file_id=285&type=bug
png

png Screenshot_Doom_20110501_053519.png (34,983) 2011-05-01 03:40
/tracker/file_download.php?file_id=298&type=bug
png

png Screenshot_Doom_20110501_200832.png (34,680) 2011-05-01 18:18
/tracker/file_download.php?file_id=300&type=bug
png
Issue History
2011-04-27 12:26unknownnaNew Issue
2011-04-27 12:26unknownnaFile Added: Screenshot_Doom_20110427_141527.png
2011-04-27 12:26unknownnaDescription Updatedbug_revision_view_page.php?rev_id=769#r769
2011-04-27 12:27unknownnaFile Added: Screenshot_Doom_20110427_131841.png
2011-04-27 12:28unknownnaFile Added: Screenshot_Doom_20110427_131851.png
2011-04-27 13:27unknownnaNote Added: 0001485
2011-04-27 13:27unknownnaStatusnew => confirmed
2011-04-27 13:47unknownnaFile Added: Screenshot_Doom_20110427_151723.png
2011-04-27 13:47unknownnaFile Deleted: Screenshot_Doom_20110427_131841.png
2011-04-27 13:47unknownnaFile Deleted: Screenshot_Doom_20110427_131851.png
2011-04-28 03:04unknownnaNote Edited: 0001485bug_revision_view_page.php?bugnote_id=1485#r773
2011-04-30 23:27Torr SamahoNote Added: 0001517
2011-04-30 23:27Torr SamahoAssigned To => Torr Samaho
2011-04-30 23:27Torr SamahoStatusconfirmed => feedback
2011-04-30 23:33Torr SamahoNote Added: 0001518
2011-04-30 23:37Torr SamahoNote Edited: 0001518bug_revision_view_page.php?rev_id=796
2011-05-01 00:01Torr SamahoNote Deleted: 0001518
2011-05-01 00:42Torr SamahoNote Added: 0001519
2011-05-01 03:38unknownnaNote Added: 0001521
2011-05-01 03:38unknownnaStatusfeedback => assigned
2011-05-01 03:40unknownnaFile Added: Screenshot_Doom_20110501_053519.png
2011-05-01 12:47Torr SamahoNote Added: 0001525
2011-05-01 13:13unknownnaNote Added: 0001528
2011-05-01 13:17unknownnaNote Edited: 0001528bug_revision_view_page.php?bugnote_id=1528#r800
2011-05-01 13:19Torr SamahoStatusassigned => resolved
2011-05-01 13:19Torr SamahoFixed in Version => 1.0
2011-05-01 13:19Torr SamahoResolutionopen => fixed
2011-05-01 16:17Torr SamahoStatusresolved => assigned
2011-05-01 16:20Torr SamahoNote Added: 0001530
2011-05-01 16:22Torr SamahoStatusassigned => feedback
2011-05-01 16:38unknownnaNote Added: 0001531
2011-05-01 16:38unknownnaStatusfeedback => assigned
2011-05-01 16:41unknownnaNote Edited: 0001531bug_revision_view_page.php?bugnote_id=1531#r804
2011-05-01 16:45unknownnaNote Edited: 0001531bug_revision_view_page.php?bugnote_id=1531#r805
2011-05-01 17:33Torr SamahoNote Added: 0001533
2011-05-01 17:37Torr SamahoStatusassigned => feedback
2011-05-01 18:15unknownnaNote Added: 0001535
2011-05-01 18:15unknownnaStatusfeedback => assigned
2011-05-01 18:18unknownnaFile Added: Screenshot_Doom_20110501_200832.png
2011-05-01 18:32unknownnaNote Edited: 0001535bug_revision_view_page.php?bugnote_id=1535#r807
2011-05-01 20:07Torr SamahoNote Added: 0001536
2011-05-01 20:07Torr SamahoNote Added: 0001537
2011-05-01 20:15Torr SamahoNote Deleted: 0001537
2011-05-01 20:16Torr SamahoStatusassigned => feedback
2011-05-02 04:07unknownnaNote Added: 0001545
2011-05-02 04:07unknownnaStatusfeedback => assigned
2011-05-08 18:09Torr SamahoNote Added: 0001571
2011-05-08 18:13Torr SamahoStatusassigned => feedback
2011-05-08 19:00unknownnaNote Added: 0001573
2011-05-08 19:00unknownnaStatusfeedback => assigned
2011-05-08 23:25Torr SamahoNote Added: 0001582
2011-05-08 23:33Torr SamahoNote Edited: 0001582bug_revision_view_page.php?bugnote_id=1582#r829
2011-05-08 23:34Torr SamahoStatusassigned => feedback
2011-05-09 09:40unknownnaNote Added: 0001584
2011-05-09 09:40unknownnaStatusfeedback => assigned
2011-05-09 09:47unknownnaNote Edited: 0001584bug_revision_view_page.php?bugnote_id=1584#r831
2011-05-09 12:29Torr SamahoNote Added: 0001586
2011-05-09 12:34Torr SamahoStatusassigned => feedback
2011-05-09 12:50unknownnaNote Added: 0001587
2011-05-09 12:50unknownnaStatusfeedback => assigned
2011-05-14 15:14Torr SamahoNote Added: 0001602
2011-05-14 15:14Torr SamahoStatusassigned => resolved
2012-06-09 13:22Torr SamahoCategoryGeneral => Bug
2018-09-30 20:04Blzut3Statusresolved => closed

Notes
(0001485)
unknownna   
2011-04-27 13:27   
(edited on: 2011-04-28 03:04)
OK, I finally managed to reproduce the NO TEAM issue.

1. Start a TeamPossession server with sv_keepteams set to 1 and sv_maxplayers set to 0.
2. Connect a client to the server.
3. "join" in the console.
4. "sv_maxplayers 1-32" in the server console.
5. "join" in the console. You'll be put on a team.
6. Turn into a spectator. You'll be put on NO TEAM instead of turning into a spectator.

I forgot to mention that this happens in all team modes.

(0001517)
Torr Samaho   
2011-04-30 23:27   
This should fix the issue described in note 1485.
(0001519)
Torr Samaho   
2011-05-01 00:42   
This should also fix the team spectator after chanegemap map change issue, the player is still forced to spectate though:

If you make sv_maxplayers lower than the number of players in the server what is Skulltag supposed to do? Apparently the admin wants some of the players removed, Skulltag just postpones this to the next map change to not interrupt the current game (actually this is not really done on purpose but a consequence of the game mechanics and IMO a consistent behavior). The only sane way to prevent players from being forced to spectate by this I see would be to forbid the admin to make sv_maxplayers smaller than the number of player currently in the game. Note: If you only considered the part that the spectator is on a team to be a bug, just ignore this paragraph ;-).
(0001521)
unknownna   
2011-05-01 03:38   
It fixed the issues. But if the sv_maxplayers value is raised while in the intermission screen, the players in the queue will be forced to join and this bug will be present.
(0001525)
Torr Samaho   
2011-05-01 12:47   
Since increasing the sv_maxplayers value intentionally makes the players waiting in line join, it's to be expected that they are affected by the same bug as players who manually join during the intermission. So I consider this issue to be resolved, 0000325 still needs to be fixed though of course.
(0001528)
unknownna   
2011-05-01 13:13   
(edited on: 2011-05-01 13:17)
You turn into a team spectator in the intermission screen, and then you turn into a normal spectator again at map start if sv_keepteams is set to 0.

(0001530)
Torr Samaho   
2011-05-01 16:20   
> You turn into a team spectator in the intermission screen, and then you turn into a normal spectator again at map start if sv_keepteams is set to 0.

Can you elaborate how to reproduce this?
(0001531)
unknownna   
2011-05-01 16:38   
(edited on: 2011-05-01 16:45)
1. Start a TDM server with sv_maxplayers set to 0.
2. Connect a client to the server.
3. "join" in the console.
4. "changemap map01" in the server console.
5. "sv_maxplayers 1-32" in the server console.

(0001533)
Torr Samaho   
2011-05-01 17:33   
I see. Letting players join during intermission introduces some timing issues. This should fix the issue, in the sense that even if sv_maxplayers is increased during a map change, the players only join after the map change (it doesn't cover players who try to join during intermission manually yet). Unfortunately, this is another far reaching change and may have an effect during map changes in all game modes that don't use sv_maxlives.
(0001535)
unknownna   
2011-05-01 18:15   
(edited on: 2011-05-01 18:32)
> it doesn't cover players who try to join during intermission manually yet

Indeed, I can still turn into a team spectator if I manually join the game during the intermission screen.

Separate issue: My "switchonpickup" alias forces me to use the fist at map starts in non-coop modes if I hold +attack when leaving the intermission screen. This doesn't happen in 98d. However, this also happens in build 3195.

(0001536)
Torr Samaho   
2011-05-01 20:07   
This should prevent players from joining during intermission in general. Instead they are put in line and join after the map change. So this should also fix 0000325.

PS: The fact that you are possible shown as spectator on a team while the map changes is done should not considered to be a bug. The spectator is put on a team and scheduled to join at the same time, but the join only happens when the map change is complete.

Regarding switchonpickup: This most likely has nothing to do with this report, but likely is related to the weapon sync changes.
(0001545)
unknownna   
2011-05-02 04:07   
> This should prevent players from joining during intermission in general. Instead they are put in line and join after the map change. So this should also fix 0000325.

It fixed the issues. But players aren't put in the queue if they use the "team" command.
(0001571)
Torr Samaho   
2011-05-08 18:09   
Hmm, I checked the code and noticed that Skulltag actively prevents team and changeteam from being used during intermission. I wonder if I should soften this restriction to allow spectators to join the queue during intermission or just keep it as it as and acceped this minor inconsistency.
(0001573)
unknownna   
2011-05-08 19:00   
> I wonder if I should soften this restriction to allow spectators to join the queue during intermission or just keep it as it as and acceped this minor inconsistency.

Would it be possible to only allow spectators to use the commands during the intermission screen? With the new behavior, they will not appear as being on a team until the map actually changes. It's a very trivial issue though. And although they might join the queue during the intermission screen, they are still turned into a regular spectator in the next map if sv_keepteams is set to 0.
(0001582)
Torr Samaho   
2011-05-08 23:25   
(edited on: 2011-05-08 23:33)
Ok, this should allow spectators to join the queue with team/changeteam during intermission.

(0001584)
unknownna   
2011-05-09 09:40   
(edited on: 2011-05-09 09:47)
> Ok, this should allow spectators to join the queue with team/changeteam during intermission.

It fixed the issue, but I'm not put in the queue with the changeteam command. And spectators are not put on a team at map start if sv_keepteams or sv_noteamselect is set to 0. It's not a bug though.

(0001586)
Torr Samaho   
2011-05-09 12:29   
> but I'm not put in the queue with the changeteam command

"changeteam teamname" work fine for me. changeteam without argument is supposed to put you on the next team, i.e. if you are on team one you are put on team two and so forth. In particular, it always puts you on the other team, if there are only two teams.

If you are not on a team, there is no "next" or "other" team, so the command simply does nothing in this case. If you think that this behavior is unexpected, I can add a message that tells the spectator that it can't toggle its team without being on a team.
(0001587)
unknownna   
2011-05-09 12:50   
> "changeteam teamname" work fine for me. changeteam without argument is supposed to put you on the next team, i.e. if you are on team one you are put on team two and so forth. In particular, it always puts you on the other team, if there are only two teams.

I see. I only used "changeteam".

> If you think that this behavior is unexpected, I can add a message that tells the spectator that it can't toggle its team without being on a team.

That'd be nice.
(0001602)
Torr Samaho   
2011-05-14 15:14   
>> If you think that this behavior is unexpected, I can add a message that tells the spectator that it can't toggle its team without being on a team.

> That'd be nice.

Done.