[Aviram1994]

Hello,
I tried to code a Melee Only plugin for TF2,
I wonder why it crashes the server when I change the cvar to 1..?

[kirewv]

PHP Code:

public Action:Event_PlayerSpawn(Handle:event, const String:name[], bool:dontBroadcast)
{
 new MeleeOnlyOn = GetConVarInt(g_meleeonly) 


Are you trying to limit melee per player if so:
you need to devise a way to set the variable (random, by admin choice ... etc.)
make a global

PHP Code:

new bool:MeleeOnly[MAXPLAYERS +1]; 


and then

PHP Code:

public Action:Event_PlayerSpawn(Handle:event, const String:name[], bool:dontBroadcast)
{
MeleeOnly[client] = 0 //make it 1 if you want it on 


But seeing as you have a "shared" CVAR that is server wide you are not making it player specific and you should just
make MeleeOnlyOn global -
then move "MeleeOnlyOn = GetConVarInt(g_meleeonly)"

to OnMapStart or hook the round and put it in round_start.

[kirewv]

on another note why have all the specific remove functions and class checking?

You could just use the following code:

PHP Code:

public Action:Event_PlayerSpawn(Handle:event, const String:name[], bool:dontBroadcast)
{
  if (MeleeOnlyOn == 1)
 {
    RemoveWeaponSlot(client);
 }
}
stock RemoveWeaponSlot(client)
{
 new weaponIndex;
 // Iterate through weapon slots
 for( new i = 0; i < 4; i++ )
 {
  while( ( weaponIndex = GetPlayerWeaponSlot( client, i ) ) != -1 )
  {
   RemovePlayerItem( client, weaponIndex );
  }
 }
} 


[Aviram1994]

I cant use it because,
Some classes have their weapons at 1-2
some classes have weapons in 1,2,4,5
so I cant use loop,
I thought about something like that..

[DJ Tsunami]

Sure you can, you're checking if there is a weapon in that slot anyway. If there's not, then it won't attempt to remove it either. You have to do i < 5 though, not 4.

stock RemoveWeaponSlot(client)
{
  new weaponIndex;
  // Iterate through weapon slots
  for ( new i = 0; i < 5; i++ )
  {
    // Do not remove melee weapon slot
    if ( ( weaponIndex = GetPlayerWeaponSlot( client, i ) ) != -1 && i != 2 )
    {
      RemovePlayerItem( client, weaponIndex );
      RemoveEdict( weaponIndex );
    }
  }
}

Also you can't remove their weapons immediately on spawn, because they get their weapons the frame after that. So you have to make a timer with a 0.1 second interval, and call that stock in that timer callback. I would suggest you just look at the source of TF2 Sudden Death Melee Redux, everything you need is right there.