[xZk]

Details
Survivors can pick up weapons or items while incapped, by pressing the E key (IN_USE)

Cvars:

PHP Code:

// 0:Disable, 1:Enable plugin
// -
// Default: "1"
// Minimum: "0.000000"
// Maximum: "1.000000"
l4d_incapped_pickup "1"

// Use pickup distance limit (player_use_radius = 96) 
// -
// Default: "96.0"
// Minimum: "1.000000"
l4d_incapped_pickup_distance "96.0"

// 0:Disable pickup scavenge items, 1:Allow pickup gascans, 2: Allow pickup colas, 3:All
// -
// Default: "0"
// Minimum: "0.000000"
// Maximum: "3.000000"
l4d_incapped_pickup_scavenge "0"

// 0:Disable, 1:Allow pickup carryable items (gnome, propanetank, etc)
// -
// Default: "0"
// Minimum: "0.000000"
// Maximum: "1.000000"
l4d_incapped_pickup_carryable "0" 


Special Thanks

• Silvers: for being a good and great example to start writing plugins myself
• Dragokas: for your suggestions and helping me understand the great importance of correct validations

pd: I would have like to thank you before, but my English was disgusting and I hate to use the translator always :p

Rant:

-put l4d2_incapped_pickup.txt in /addons/sourcemod/gamedata/ folder

Changelog

Spoiler

2021-12-03 (v1.4.0)
* Changed methods to find pickup
* Added new cvar: l4d_incapped_pickup_carryable

2020-07-22 (v1.3.0)
* Added visibility filter to pickup
* Added new cvar for disabling to pickup scavenge items 

2020-03-01 (v1.2.0)
* Fixed "IsPlayerCapped" method for L4D1

2020-02-12 (v1.1.0)
* Fixed steal another player's weapon
* Can pickup weapon melee when thrown to the ground by [L4D2] Throwable Melee Weapons  plugin

2020-01-11 (v1.0)
* Initial release.

[KRUTIK]

You have a line 87, a link to the site, is this how it should be?

[Silvers]

Originally Posted by KRUTIK You have a line 87, a link to the site, is this how it should be?

Yes? Credit for source and info.

[KRUTIK]

I checked on my server, there are no errors yet.

[MasterMind420]

stock void SetActiveWeapon(int client, int weapon)
{
SetEntPropEnt(client, Prop_Send, "m_hActiveWeapon", weapon);
ChangeEdictState(client, FindDataMapInfo(client, "m_hActiveWeapon"));
}

Although you do not use this function in your code, I would advise highly against ever using this method to switch a players weapon, i've personally seen bugs where the weapon u switch too can't be seen in your hands nor does the clients hud that shows your items properly update...say after using a molotov and forcefully changing to a primary or secondary the icon for the molotov stays in your inventory even though u no longer can select it or have it...instead use clientcommand/fakeclientcommand to set the weapon bug free by changing to its slot.

[xZk]

ok thanks for the tip, i'm testing this fakeclientcommand method in the Incapped Weapons plugin.

[xZk]

Update:

2020-03-01 (v1.2)
* Fixed "IsPlayerCapped" method for L4D1

thanks TiTz for his report

[Marttt]

Noticed a bug when scavenge items cvar is enable, if you press E next to a scavenge gascan it spawns another one next to it, but in the red color.

After messing with the code a bit I figured out that when pressing "E", sometimes it fires "GetItemOnFloor" for the "weapon_scavenge_item_spawn" entities [invisible], instead of the "weapon_gascan" and it makes another gascan spawn.

So to fix it I added another check on "IsValidItemPickup"

Here is the code in case the author wants to incorporate into the plugin, or someone else has the same problem. Seems to fix this bug.

Spoiler

PHP Code:

bool IsValidItemPickup(int item){

    if(IsValidWeapon(item)){
        if (IsWeaponScavengeItemSpawn(item))
            return false;
        if(IsWeaponGascan(item) && (g_cvarScavengeItem.IntValue & 1)){
            return true;
        }else if(IsWeaponColaBottles(item) && (g_cvarScavengeItem.IntValue & 2)){
            return true;
        }else if(IsWeaponGascan(item) || IsWeaponColaBottles(item)){
            return false;
        }
        return true;

    }
    return false;
}

stock bool IsWeaponScavengeItemSpawn(int weapon){
    if(IsValidWeapon(weapon)){
        char class_name[64];
        GetEntityClassname(weapon, class_name, sizeof(class_name));
        return (strcmp(class_name, "weapon_scavenge_item_spawn") == 0);
    }
    return false;
} 


Note: I will/allow delete the post in case this has been fixed.

[mixjay]

One more fix - restrict pickup chainsaw, when you incapped:

stock bool IsWeaponChainsaw(int weapon){
	if(IsValidWeapon(weapon)){
		char class_name[64];
		GetEntityClassname(weapon, class_name, sizeof(class_name));
		return (strcmp(class_name, "weapon_chainsaw") == 0);
	}
	return false;
}

[sorallll]

A better way to get items that are about to be picked up