DropWeapon() function not working ....

[BALIST0N]

hello everyone ! i dont understand why the dropweapon function doesn't work ....

this is a clientside script

i have this code :

--------------------------------------------------
--code for drop weapon script :

hook.Add( "Think", "Droptheweap", function()
	if input.IsKeyDown( KEY_G ) then
		DropActualGun()
	end
end)

function DropActualGun()
	
	local weap = ply:GetActiveWeapon():GetClass()
	print (weap)       -- print succesfully the weap name
 	ply:DropWeapon(weap)  -- <= actual error line 

end

-------------------------------------------------

and this error :

[ERROR] addons/gunmenu/lua/autorun/client/cl_gunmenu.lua:38: attempt to call method 'DropWeapon' (a nil value)
  1. DropActualGun - addons/gunmenu/lua/autorun/client/cl_gunmenu.lua:38

thanks for incomming responses ! :v:

[Moosicorn]

DropWeapon() is a server only function.

[BALIST0N]

[QUOTE=Moosicorn;50568239]DropWeapon() is a server only function.[/QUOTE]

i have tried to migrate code to server side like this :

cl_gunmenu.lua

--------------------------------------------------
--code for drop weapon script :

include("../server/sv_dropweap.lua")

hook.Add( "Think", "Droptheweap", function()
	if input.IsKeyDown( KEY_G ) then
		DropActualGun()
	end
end)
----------

sv_dropweap.lua

function DropActualGun()
	
	local weap = ply:GetActiveWeapon():GetClass()
	print (weap)
	ply:DropWeapon("weapon_crowbar")

end

but he doesnt work ... :/

weapon_crowbar

[ERROR] addons/gunmenu/lua/autorun/client/../server/sv_dropweap.lua:6: attempt to call method 'DropWeapon' (a nil value)
  1. DropActualGun - addons/gunmenu/lua/autorun/client/../server/sv_dropweap.lua:6

[James]

The client realm and the server realm are separate. You cannot call something on one from the other without going through a library like the net library or the umsg library. You will want to do something like this instead.

[lua]
-- Client
hook.Add( "Think", "Droptheweap", function()
if input.IsKeyDown( KEY_G ) then
net.Start( "dropactualgun" )
net.SendToServer()
end
end)
[/lua]

This will send a message to the server when you think the player wants to drop their weapon. I would advise against using the Think hook for this however.

[lua]
-- Server
util.AddNetworkString( "dropactualgun" )

net.Receive( "dropactualgun", function(ln, pl)
local weap = pl:GetActiveWeapon():GetClass()
print( weap )
pl:DropWeapon(weapon_entity)
end)
[/lua]

This will listen for the message being sent from the client, and when it receives it, calls the 2nd argument as a callback with 2 parameters: The length of the message (not really used for much), and the player who sent the message. We can use the latter to know who wants to drop their weapon, and then do just that.

[BALIST0N]

ehm ... i dont know what that means :

[ERROR] addons/gunmenu/lua/autorun/server/sv_dropweap.lua:7: Tried to use a NULL entity!
  1. DropWeapon - [C]:-1
   2. func - addons/gunmenu/lua/autorun/server/sv_dropweap.lua:7
    3. unknown - lua/includes/extensions/net.lua:32

[James]

[img]http://wiki.garrysmod.com/favicon.ico[/img] [url=http://wiki.garrysmod.com/page/Player/DropWeapon]Player:DropWeapon[/url] takes a weapon (entity) as an argument, not the name.

[BALIST0N]

lmao so the error come again but the weapon is dropped ! thats a very good start !
also, i can drop 1000 weapons per second, i need to find a way to timer the hook
have you got any ideas ?

PS: i have juste remove " :GetClass() " from " local weap = pl:GetActiveWeapon():GetClass() "

[Rubat]

function DropActualGun()
	
	local weap = ply:GetActiveWeapon():GetClass()
	print (weap)
	ply:DropWeapon("weapon_crowbar")

end

Did you guys really miss this very obvious clue: ply is undefined.

[BALIST0N]

[QUOTE=Robotboy655;50568501]

ply is undefined.[/QUOTE]

i'm not sure :) :v::what:

[editline]22nd June 2016[/editline]

i have fix the script bug : caused beceause game try to drop a weapon but player have not weapon :
*
so this is beatifull

util.AddNetworkString( "dropactualgun" )
 
net.Receive( "dropactualgun", function(ln, pl)
local weap = pl:GetActiveWeapon()
	if weap != NULL then 
		pl:DropWeapon(weap)
	end
end

need to find a timer for slow down the drop rate

[MentallyAdvanced]

for the delay do this

net.Receive( "dropactualgun", function(ln, pl)
	timer.Create("dropDelay",2,1, function()
	local weap = pl:GetActiveWeapon()
	pl:DropWeapon(weap)
	end)
end)

and it doesn't drop every weapon you have, only your active one

[James]

[QUOTE=BALIST0N;50568536]i'm not sure :) :v::what:

[editline]22nd June 2016[/editline]

i have fix the script bug : caused beceause game try to drop a weapon but player have not weapon :
*
so this is beatifull

util.AddNetworkString( "dropactualgun" )
 
net.Receive( "dropactualgun", function(ln, pl)
local weap = pl:GetActiveWeapon()
	if weap != NULL then 
		pl:DropWeapon(weap)
	end
end

need to find a timer for slow down the drop rate[/QUOTE]

This is caused by using the think hook. You could add a cooldown per player.

[BALIST0N]

Thanks all of you ! i have learn much better than the gmod wiki

this is the final script :

cl_dropweap.lua (client side)

--------------------------------------------------
--code for drop weapon script :
local droppable = true

hook.Add( "Think", "Droptheweap", function()
	if droppable == true then
		if input.IsKeyDown( KEY_G ) then
			net.Start( "dropactualgun" )
			net.SendToServer() 
			droppable = false --the player cannot drop weapon
			timer.Simple( 0.5, function() droppable = true end ) --unil 0.5 seconds
			
		end
	end
end)
-------------------------------------------------

sv_dropweap.lua (server side)

-- Server
util.AddNetworkString( "dropactualgun" ) 
 
net.Receive( "dropactualgun", function(ln, pl)
	local weap = pl:GetActiveWeapon()  --get the weapon object
	if weap != NULL then  -- if the player have a weapon on his invetory 
		pl:DropWeapon(weap) 
	end
	
	
end)