[VEN]

by use this event, admin will keep his "team" money

This is not the case because a reward is given on new round, not on round end.

[Yourgod | DarkSider]

Very good & useful Plugin .
+karma .

[VEN]

If i'd set a long restartround delay (e.g. 60 seconds) why would you want to reset my team back? I have the whole minute for a free look. Also i do not see the advantage of the end_round hook. You providing the option to choose a different hooks to perform a reset of an admin's team back but i do not see why would you need to provide that choice at all. You just creating more problems with the default hook method.

I do not see how this could be useful:

if(g_team[id] == CS_TEAM_SPECTATOR) {
    message_begin(MSG_ALL, g_msgTeamInfo)
    write_byte(id)
    write_string("SPECTATOR")
    message_end()
}

The Spec TeamInfo has been already updated properly (see your join_spec function) so this is unnecessary.

And again you do not care of the manual team changes. Let's say the person has been forced to free look by your plugin on non-team-change death. After that the person is choosing another team. The forced free look is gone since the person's team not the Spectators anymore. You should take care of that.

[jim_yang]

yeah, you are absolutely right VEN, thanks. I didn't test it well. I should do the same thing when they manually choose the other team after dead.

[soccdoodcss]

Very nice... should definately be approved, works great.

[VEN]

Isn't it crash the server with your non-alive manual team change fix?

Also what about autoteambalance? You are not cache the new team for this case.

[VEN]

Plugin doesn't work for VGUI menu.

[jim_yang]

is the menu id of "select a class" of VGUI menu a fixed number ?
a little bit weird, when i use VGUI menu, get_menu_id() always return 0 before I call a common menu. any ideas ?

[VEN]

VGUI menu ids are constant. You could test get_menu_id and report to DEVs if it's (still) buggy.

To catch class selection i'd hook post "joincalss" "1"/"2"/"3"/"4" ClientCommand.

[jim_yang]

before i use cs_set_user_team to transfer them to spec, i must make sure that a client doesn't has a "choose a class" or "select a class" menu, or server will crash, because he already joined spec. now i use get_user_menu(), only old style menu return the right id. VGUI menu always return 0. but after i call any other simple menu, it return the right menu id -26 or -27 again.