[The Killer NL]

Description:
Hello Everyone, This plugin basically blocks all the left clicks of your knife.
Could be useful for 35hp knife servers if you only want to have right click...

Install:

• smx file move to csgo/addons/sourcemod/plugins/

Credits:

• Bara
• The Killer [NL]
• SM9

Version 2.2 (Wasn't public) All the normal knives added Version 2.3 Added the newest knives Version 2.4 Changes by SM9 Version 2.5 Fixed issue of Tasers not working. Thanks to SM9 to

Regards Killer.

[Vigge]

Thanks for the plugin! Easy install and works great, got it working right away, plug'n'play!

[butare]

You can replace all your if statements with one if:

	if(StrContains(sWeapon, "knife") != -1 || StrEqual(sWeapon, "weapon_bayonet"))
	{
		buttons &= ~IN_ATTACK;
		return Plugin_Changed;
	}

[Bara]

Originally Posted by butare if(StrContains(sWeapon, "knife") != -1 || StrEqual(sWeapon, "weapon_bayonet")) { buttons &= ~IN_ATTACK; return Plugin_Changed; }

if(StrContains(sWeapon, "knife", false) != -1 || StrContains(sWeapon, "bayonet", false) != -1)

Maybe Valve adds a new bayonet equal knife with just other name.

[butare]

Originally Posted by Bara if(StrContains(sWeapon, "knife", false) != -1 || StrContains(sWeapon, "bayonet", false) != -1) Maybe Valve adds a new bayonet equal knife with just other name.

Or they can add a gun with "bayonet" in name, never knows

[Bara]

Originally Posted by butare Or they can add a gun with "bayonet" in name, never knows

Sure, but my opinion based on Karambit and Talon Knife. Looks basically equal with a few changes.

[butare]

Originally Posted by Bara Sure, but my opinion based on Karambit and Talon Knife. Looks basically equal with a few changes.

Yea but they both have "knife" in their names, so I think they made a mistake when they add bayonet and not add "knife" to it's name. So other versions of knifes (even if it's just another bayonet looking knife) should contain "knife" in their name, as all other knifes do (ofc except bayonet for some reason). And rn it's just an offtopic, so I'll stop right there

[The Killer NL]

I understand what you guys mean but i'll keep it this for now since it's not really a big deal

[shanapu]

Originally Posted by The Killer NL I understand what you guys mean but i'll keep it this for now since it's not really a big deal

It is. In the worst case, your plugin will do fifteen StrEqual checks before it blocks the attack.
Baras example just needs a maximum of two checks. Where IIRC StrContains is less expensive than StrEqual. And when valve adds a new knife the probability is high they will call it somethig like knife (or bayonet), so there will be no need to edit your plugin for a valve update.

But the main reason why I decided to post here is:
With your method to block the attack players can easily lag the server and make it unplayable.

Instead of block the attack on OnPlayerRunCmd I recommend this method:

PHP Code:

void EnableWeaponFire(int client, bool status = true)
{
    if (status) // true = allow shooting
    {
        SDKUnhook(client, SDKHook_PreThink, PreThinkWeapon);
    }
    else // false = suppress shooting
    {
        SDKHook(client, SDKHook_PreThink, PreThinkWeapon);
    
    }
}
 public Action PreThinkWeapon(int client)
{
     int weapon = GetEntPropEnt(client, Prop_Send, "m_hActiveWeapon");
     if (weapon < 0 || !IsValidEdict(weapon) || !IsValidEntity(weapon))
        return Plugin_Continue;
     SetEntPropFloat(weapon, Prop_Send, "m_flNextPrimaryAttack", GetGameTime() + 0.25);  // Primary
     SetEntPropFloat(weapon, Prop_Send, "m_flNextSecondaryAttack", GetGameTime() + 0.25);  // Secondary
     return Plugin_Continue;
} 


[Dr!fter]

The better way is to get the item definition (m_iItemDefinitionIndex or whatever...) convert it to an id with https://sm.alliedmods.net/new-api/cs...emDefIndexToID and comparing that it is within the range of https://github.com/alliedmodders/sou....inc#L144-L159 or one of the 3 other knifes.