[AtomicStryker]

Ive updated the Spawn Uncommons Plugin so you dont need to strip the cheats flag and stuff anymore, just use ServerCommand.

PHP Code:

public IsValidClient(client)
{
    if (client == 0)
        return false;

    if (!IsClientConnected(client))
        return false;
    
    //if (IsFakeClient(client))
        //return false;
    
    if (!IsClientInGame(client))
        return false;
    
    if (!IsPlayerAlive(client))
        return false;

    if (!IsValidEntity(client))
        return false;

    return true;
} 


What is this? Its not even being used. Apart from that, there is no point in using IsValidEntity AFTER youve checked for anything else. Youll never need IsClientConnected either.

Also, why is it returning false when the function is no bool:

Code Improvement suggestions:

PHP Code:

        switch (HordeSpawn) 
        {
            case 1:
            {
                new flags = GetCommandFlags("sm_spawnuncommonhorde");
                SetCommandFlags("sm_spawnuncommonhorde", flags & ~ADMFLAG_CHEATS);
                FakeClientCommand(GetAnyClient(), "sm_spawnuncommonhorde riot");
                SetCommandFlags("sm_spawnuncommonhorde", flags);
                isUncommonBileTimeout = true;
                CreateTimer(GetConVarFloat(cvarUncommonBileTimeout), UncommonBileTimeout);
            }

            case 2:
            {
                new flags = GetCommandFlags("sm_spawnuncommonhorde");
                SetCommandFlags("sm_spawnuncommonhorde", flags & ~ADMFLAG_CHEATS);
                FakeClientCommand(GetAnyClient(), "sm_spawnuncommonhorde ceda");
                SetCommandFlags("sm_spawnuncommonhorde", flags);
                isUncommonBileTimeout = true;
                CreateTimer(GetConVarFloat(cvarUncommonBileTimeout), UncommonBileTimeout);
            }

            case 3:
            {
                new flags = GetCommandFlags("sm_spawnuncommonhorde");
                SetCommandFlags("sm_spawnuncommonhorde", flags & ~ADMFLAG_CHEATS);
                FakeClientCommand(GetAnyClient(), "sm_spawnuncommonhorde clown");
                SetCommandFlags("sm_spawnuncommonhorde", flags);
                isUncommonBileTimeout = true;
                CreateTimer(GetConVarFloat(cvarUncommonBileTimeout), UncommonBileTimeout);
            }

            case 4:
            {
                new flags = GetCommandFlags("sm_spawnuncommonhorde");
                SetCommandFlags("sm_spawnuncommonhorde", flags & ~ADMFLAG_CHEATS);
                FakeClientCommand(GetAnyClient(), "sm_spawnuncommonhorde mud");
                SetCommandFlags("sm_spawnuncommonhorde", flags);
                isUncommonBileTimeout = true;
                CreateTimer(GetConVarFloat(cvarUncommonBileTimeout), UncommonBileTimeout);
            }

            case 5:
            {
                new flags = GetCommandFlags("sm_spawnuncommonhorde");
                SetCommandFlags("sm_spawnuncommonhorde", flags & ~ADMFLAG_CHEATS);
                FakeClientCommand(GetAnyClient(), "sm_spawnuncommonhorde roadcrew");
                SetCommandFlags("sm_spawnuncommonhorde", flags);
                isUncommonBileTimeout = true;
                CreateTimer(GetConVarFloat(cvarUncommonBileTimeout), UncommonBileTimeout);
            }

            case 6:
            {
                new flags = GetCommandFlags("sm_spawnuncommonhorde");
                SetCommandFlags("sm_spawnuncommonhorde", flags & ~ADMFLAG_CHEATS);
                FakeClientCommand(GetAnyClient(), "sm_spawnuncommonhorde random");
                SetCommandFlags("sm_spawnuncommonhorde", flags);
                isUncommonBileTimeout = true;
                CreateTimer(GetConVarFloat(cvarUncommonBileTimeout), UncommonBileTimeout);
            }
        } 


Apart from stripping the CHEATS flag, you do the same thing over and over here while there is just one tiny variable changing. Why dont you only change the variable with the case: code, then do the rest after.

[ScaredKid]

Hey is it possible to make all uncommons be part of the whole zombie population on all maps??

[mommafish]

izzt possible to make boomer vomit call ONLY uncommon horde?

[ghosthunterfool]

stop asking "isit possible" questions go wiki and check it out -_-

[mommafish]

but if i put z_common_limit 0 there will be no non-puked mobs either ;( help me

[chu1720]

when Spawn uncommon Infected updated to 1.0.6, it doesn't work......

[Mortiegama]

@Atomic - Thanks for the update, I'll have a new version with some changes out later this afternoon.

@Scaredkid - That's a question to ask Atomic in his Uncommon spawner plugin, not here.

@Mommafish - This plugin only calls the Uncommon, the rest of it is hard coded into the game.

@Chu - I know, there will be a new version out to fix that soon.

[Mortiegama]

The new version has been released to be compatible with Atomic's latest release. Unfortunately there are a few bugs in his plugin which have the following affect:

1) Spawning the uncommon horde is off and will not spawn random and is misspawning other uncommons. Riot will create CEDA, CEDA will create Clown, and Random is not currently working.
2) l4d2_spawn_uncommons_autoshuffle should be set to 0 or the uncommon horde will only contain a few uncommon and none of the types you have selected.

[chu1720]

Originally Posted by Mortiegama The new version has been released to be compatible with Atomic's latest release. Unfortunately there are a few bugs in his plugin which have the following affect: 1) Spawning the uncommon horde is off and will not spawn random and is misspawning other uncommons. Riot will create CEDA, CEDA will create Clown, and Random is not currently working. 2) l4d2_spawn_uncommons_autoshuffle should be set to 0 or the uncommon horde will only contain a few uncommon and none of the types you have selected.

Still doesn't work...

[IronWarrior]

Any chance you can upload 1.1 again please?