[Fuck For Fun]

PHP Code:

L 07/22/2019 - 12:08:31: [CSTRIKE] Non-player entity 0 out of range
L 07/22/2019 - 12:08:31: [AMXX] Displaying debug trace (plugin "jailbreak_lastrequest.amxx", version "v1.0")
L 07/22/2019 - 12:08:31: [AMXX] Run time error 10: native error (native "cs_set_weapon_ammo")
L 07/22/2019 - 12:08:31: [AMXX]    [0] jailbreak_lastrequest.sma::taskGiveWeapon (line 1258) 


Lines 1258 Colored Red:

public taskGiveWeapon(iTaskId)
{
	if (!is_user_alive(ID_GIVEWEAPON))
		return;

	if (ID_GIVEWEAPON == g_rgLR[LR_GUARD])
	{
		if (is_user_alive(g_rgLR[LR_PRISONER]))
			StripWeapons(g_rgLR[LR_PRISONER]);
	}
	else
	{
		if (is_user_alive(g_rgLR[LR_GUARD]))
			StripWeapons(g_rgLR[LR_GUARD]);
	}

	new pWeapon = give_item(ID_GIVEWEAPON, g_pszNameLrWeapon);

	if (pWeapon != -1)
		cs_set_weapon_ammo(pWeapon, 1);
}

[<VeCo>]

pWeapon is 0.
Make sure you give a valid weapon entity name to a valid player index.

It would make more sense to check if pWeapon > 0

[Fuck For Fun]

Originally Posted by <VeCo> pWeapon is 0. Make sure you give a valid weapon entity name to a valid player index. It would make more sense to check if pWeapon > 0

So change from 1 to 0? Is this the test I was supposed to do?

PHP Code:

cs_set_weapon_ammo (pWeapon, 0); 


Or should I do a test if valid equals to 0 then make it do RELOAD?

[<VeCo>]

What exactly are ID_GIVEWEAPON and g_pszNameLrWeapon?

[Fuck For Fun]

Originally Posted by <VeCo> What exactly are ID_GIVEWEAPON and g_pszNameLrWeapon?

#define ID_GIVEWEAPON (iTaskId - TASK_GIVEWEAPON)

new g_pszNameLrWeapon[32]

	switch (g_rgLR[LR_GAME])
	{
		case GAME_S4S:
		{
			new pszNameWeapon[32], szName[32];

			if (g_iWeaponId != 0)
				get_weaponname(g_iWeaponId, pszNameWeapon, charsmax(pszNameWeapon));
			
			get_user_name(pPrisoner, szName, charsmax(szName));
			copy(g_pszNameLrWeapon, charsmax(g_pszNameLrWeapon), pszNameWeapon);

			if (g_iWeaponId == CSW_DEAGLE)
			{
				set_hudmessage(0, 200, 0, -1.0, 0.5, 1, 1.0, 3.0, 0.2, 0.2, -1);
				show_hudmessage(pGuard, "Prisoner %s will get Deagle in 3 seconds!!!", szName);
			}
			
			set_task(3.0, "taskGiveWeapon", pPrisoner+TASK_GIVEWEAPON);
		}

[E1_531G]

pWeapon is an ent. The correct check will be this:

pWeapon > 0

or

pev_valid(pWeapon)

[Fuck For Fun]

Originally Posted by E1_531G pWeapon is an ent. The correct check will be this: or

Something like that:

public taskGiveWeapon(iTaskId)
{
	if (!is_user_alive(ID_GIVEWEAPON))
		return;

	if (ID_GIVEWEAPON == g_rgLR[LR_GUARD])
	{
		if (is_user_alive(g_rgLR[LR_PRISONER]))
			StripWeapons(g_rgLR[LR_PRISONER]);
	}
	else
	{
		if (is_user_alive(g_rgLR[LR_GUARD]))
			StripWeapons(g_rgLR[LR_GUARD]);
	}

	new pWeapon = give_item(ID_GIVEWEAPON, g_pszNameLrWeapon);
		
	if (!pev_valid(pWeapon)) 
		return;
		
	//if (pWeapon != -1)
	cs_set_weapon_ammo(pWeapon, 1);
}

[thEsp]

pWeapon is used before it's declared, wtf?

[Fuck For Fun]

Solved

	new pWeapon = give_item(ID_GIVEWEAPON, g_pszNameLrWeapon);

	if (pev_valid(pWeapon))
		cs_set_weapon_ammo(pWeapon, 1);