Replacing Grenades With New Grenades in Modding

Created on: December 30, 2015, 06:42:08 am

MJPWGaming
Donator ♥
200 Posts
54 Respect +1

So I have a new grenade type in my map but when you buy them you get 4 of the new and you'll still have the normal standard grenades. When the next round starts, you'll get two standard and none of the alternate. You'll only get it when you buy them off the wall again.

Also aside from this, when I get it from the box it replaces the gun that you have out :(

If someone could help me that would be fantastic!

By the way, just so if there was a coded reply, my standard grenades are "stielhandgranate" and we'll call the new ones "stielhandgranate2"

Reply #1 on: December 30, 2015, 11:08:42 am

If you buy them on the wall, as a normal wall weapon, you need to include your weapon inside the if where it takes the player's weapon (_zombiemode_weapons, weapon_give( weapon, is_upgrade ) ):

Code Snippet

Plaintext

if( isdefined( current_weapon ) )
		{
			if( !( weapon == "fraggrenade" || weapon == "stielhandgranate" || weapon == "molotov" || weapon == "zombie_cymbal_monkey" || weapon == "stielhandgranate2") )
			{
				self TakeWeapon( current_weapon ); 
			}
		}

Then you have to take the player's grenades, checking if the player has them and that you're giving the new grenades to take his normal grenades.
If they appear in the box the code is very similar but in treasure_chest_give_weapon( weapon_string ) and if you're not using the original script for this, you'll have to find where the script for buying wall weapons is.
And for the round part, with the name you gave to grenades is really simple. Go to award_grenades_for_survivors() and you have to add this inside the first if statement:

Code Snippet

Plaintext

new_grenades = "";
if( players[i] hasWeapon( "stielhandgranate2" ) )
    new_grenades = "2";

And every time it says "stielhandgranate" inside this function, put this:

Code Snippet

Plaintext

"stielhandgranate" + new_grenades

Not tested, though.

Last Edit: December 30, 2015, 11:12:04 am by Soy-Yo

Reply #2 on: December 30, 2015, 06:49:36 pm

MJPWGaming
Donator ♥
200 Posts
54 Respect +1

Quote from: Soy-Yo on December 30, 2015, 11:08:42 am

If you buy them on the wall, as a normal wall weapon, you need to include your weapon inside the if where it takes the player's weapon (_zombiemode_weapons, weapon_give( weapon, is_upgrade ) ):
Code Snippet
Plaintext
if( isdefined( current_weapon ) )
		{
			if( !( weapon == "fraggrenade" || weapon == "stielhandgranate" || weapon == "molotov" || weapon == "zombie_cymbal_monkey" || weapon == "stielhandgranate2") )
			{
				self TakeWeapon( current_weapon ); 
			}
		}
Then you have to take the player's grenades, checking if the player has them and that you're giving the new grenades to take his normal grenades.
If they appear in the box the code is very similar but in treasure_chest_give_weapon( weapon_string ) and if you're not using the original script for this, you'll have to find where the script for buying wall weapons is.
And for the round part, with the name you gave to grenades is really simple. Go to award_grenades_for_survivors() and you have to add this inside the first if statement:
Code Snippet
Plaintext
new_grenades = "";
if( players[i] hasWeapon( "stielhandgranate2" ) )
    new_grenades = "2";
And every time it says "stielhandgranate" inside this function, put this:
Code Snippet
Plaintext
"stielhandgranate" + new_grenades
Not tested, though.

Uninitialized variable "weapon"

Reply #3 on: December 30, 2015, 07:27:58 pm

HitmanVere
UGX V.I.P.
3,997 Posts
1,024 Respect +1

Quote from: MJPWGaming on December 30, 2015, 06:49:36 pm

Uninitialized variable "weapon"

Change these:

Code Snippet

Plaintext

			if( !( weapon == "fraggrenade" || weapon == "stielhandgranate" || weapon == "molotov" || weapon == "zombie_cymbal_monkey" || weapon == "stielhandgranate2") )

to be current_weapon instead of weapon

Reply #4 on: December 30, 2015, 07:32:59 pm

BluntStuffy
Community Scripter Elite
1,187 Posts
1,404 Respect +1

Quote from: HitmanVere on December 30, 2015, 07:27:58 pm

Change these:
Code Snippet
Plaintext
			if( !( weapon == "fraggrenade" || weapon == "stielhandgranate" || weapon == "molotov" || weapon == "zombie_cymbal_monkey" || weapon == "stielhandgranate2") )
to be current_weapon instead of weapon

That'll mess things up, this is the function:

Spoiler: click to open...

Code Snippet

Plaintext

weapon_give( weapon, is_upgrade )
{
	primaryWeapons = self GetWeaponsListPrimaries(); 
	current_weapon = undefined; 

	//if is not an upgraded perk purchase
	if( !IsDefined( is_upgrade ) )
	{
		is_upgrade = false;
	}

	// This should never be true for the first time.
	if( primaryWeapons.size >= 2 ) // he has two weapons
	{
		current_weapon = self getCurrentWeapon(); // get his current weapon

		if ( current_weapon == "mine_bouncing_betty" )
		{
			current_weapon = undefined;
		}

		if( isdefined( current_weapon ) )
		{
			if( !( weapon == "fraggrenade" || weapon == "stielhandgranate" || weapon == "molotov" || weapon == "zombie_cymbal_monkey" ) )
			{
				self TakeWeapon( current_weapon ); 
			}
		} 
	}

	if( weapon == "zombie_cymbal_monkey" )
	{
		// PI_CHANGE_BEGIN
		// JMA 051409 sanity check to see if we have the weapon before we remove it	
		has_weapon = self HasWeapon( "molotov" );
		if( isDefined(has_weapon) && has_weapon )
		{
			self TakeWeapon( "molotov" );
		}

		if( isDefined(level.zombie_weapons) && isDefined(level.zombie_weapons["molotov_zombie"]) )
		{		
			has_weapon = self HasWeapon( "molotov_zombie" );
			if( isDefined(has_weapon) && has_weapon )
			{
				self TakeWeapon( "molotov_zombie" );
			}
		}
		// PI_CHANGE_END
 
		self maps\_zombiemode_cymbal_monkey::player_give_cymbal_monkey();
		play_weapon_vo( weapon );
		return;
	}
	if( (weapon == "molotov" || weapon == "molotov_zombie") )
	{
			self TakeWeapon( "zombie_cymbal_monkey" );
	}

	self play_sound_on_ent( "purchase" );
	self GiveWeapon( weapon, 0 ); 
	self GiveMaxAmmo( weapon ); 
	self SwitchToWeapon( weapon );
 
	play_weapon_vo(weapon);
}

The weapon var should be passed on from the function that called it.
Did you put it in the correct function, and not in the function for the box? ( treasure_cheast_give_weapon )

Reply #5 on: December 30, 2015, 07:35:00 pm

HitmanVere
UGX V.I.P.
3,997 Posts
1,024 Respect +1

Quote from: BluntStuffy on December 30, 2015, 07:32:59 pm

That'll mess things up, this is the function:

Spoiler: click to open...

Code Snippet

Plaintext

weapon_give( weapon, is_upgrade )
{
	primaryWeapons = self GetWeaponsListPrimaries(); 
	current_weapon = undefined; 

	//if is not an upgraded perk purchase
	if( !IsDefined( is_upgrade ) )
	{
		is_upgrade = false;
	}

	// This should never be true for the first time.
	if( primaryWeapons.size >= 2 ) // he has two weapons
	{
		current_weapon = self getCurrentWeapon(); // get his current weapon

		if ( current_weapon == "mine_bouncing_betty" )
		{
			current_weapon = undefined;
		}

		if( isdefined( current_weapon ) )
		{
			if( !( weapon == "fraggrenade" || weapon == "stielhandgranate" || weapon == "molotov" || weapon == "zombie_cymbal_monkey" ) )
			{
				self TakeWeapon( current_weapon ); 
			}
		} 
	}

	if( weapon == "zombie_cymbal_monkey" )
	{
		// PI_CHANGE_BEGIN
		// JMA 051409 sanity check to see if we have the weapon before we remove it	
		has_weapon = self HasWeapon( "molotov" );
		if( isDefined(has_weapon) && has_weapon )
		{
			self TakeWeapon( "molotov" );
		}

		if( isDefined(level.zombie_weapons) && isDefined(level.zombie_weapons["molotov_zombie"]) )
		{		
			has_weapon = self HasWeapon( "molotov_zombie" );
			if( isDefined(has_weapon) && has_weapon )
			{
				self TakeWeapon( "molotov_zombie" );
			}
		}
		// PI_CHANGE_END
 
		self maps\_zombiemode_cymbal_monkey::player_give_cymbal_monkey();
		play_weapon_vo( weapon );
		return;
	}
	if( (weapon == "molotov" || weapon == "molotov_zombie") )
	{
			self TakeWeapon( "zombie_cymbal_monkey" );
	}

	self play_sound_on_ent( "purchase" );
	self GiveWeapon( weapon, 0 ); 
	self GiveMaxAmmo( weapon ); 
	self SwitchToWeapon( weapon );
 
	play_weapon_vo(weapon);
}

The weapon var should be passed on from the function that called it.
Did you put it in the correct function, and not in the function for the box? ( treasure_cheast_give_weapon )

Right, I didnt check the actual script, so I went with what was shown in his comment

Yeah, check that the function name is weapon_give( weapon, is_upgrade )

Reply #6 on: December 30, 2015, 07:44:05 pm

BluntStuffy
Community Scripter Elite
1,187 Posts
1,404 Respect +1

Quote from: HitmanVere on December 30, 2015, 07:35:00 pm

Right, I didnt check the actual script, so I went with what was shown in his comment

https://youtu.be/G-2NimrRPAQ

Reply #7 on: December 30, 2015, 09:24:42 pm

HitmanVere
UGX V.I.P.
3,997 Posts
1,024 Respect +1

Quote from: BluntStuffy on December 30, 2015, 07:44:05 pm

https://youtu.be/G-2NimrRPAQ

Reply #8 on: December 30, 2015, 10:08:06 pm

MJPWGaming
Donator ♥
200 Posts
54 Respect +1

Quote from: BluntStuffy on December 30, 2015, 07:32:59 pm

Did you put it in the correct function, and not in the function for the box? ( treasure_cheast_give_weapon )

I dont believe so :( Where in the function would it go?

Reply #9 on: December 30, 2015, 11:34:02 pm

Quote from: MJPWGaming on December 30, 2015, 10:08:06 pm

I dont believe so Where in the function would it go?

You must have put it in other function because weapon is initialized since it's an argument of the function.
The function is the one Blunt copied in the Hidden text.
You only have to find this line inside that function:

Code Snippet

Plaintext

if( !( weapon == "fraggrenade" || weapon == "stielhandgranate" || weapon == "molotov" || weapon == "zombie_cymbal_monkey" ) )

and add:

Code Snippet

Plaintext

|| weapon == "stielhandgranate2"

And you have to do something similar for the box if the grenade appears in the box.

Reply #10 on: January 01, 2016, 05:40:28 am

MJPWGaming
Donator ♥
200 Posts
54 Respect +1

Quote from: Soy-Yo on December 30, 2015, 11:34:02 pm

You must have put it in other function because weapon is initialized since it's an argument of the function.
The function is the one Blunt copied in the Hidden text.
You only have to find this line inside that function:
Code Snippet
Plaintext
if( !( weapon == "fraggrenade" || weapon == "stielhandgranate" || weapon == "molotov" || weapon == "zombie_cymbal_monkey" ) )
and add:
Code Snippet
Plaintext
|| weapon == "stielhandgranate2"
And you have to do something similar for the box if the grenade appears in the box.

Okay now I have uninitialized variable "new_grenades"

Reply #11 on: January 01, 2016, 12:29:24 pm

Quote from: MJPWGaming on January 01, 2016, 05:40:28 am

Okay now I have uninitialized variable "new_grenades"

lol, I'll PM you what you have to change.

	Benevolent Soul who has our eternal gratitude and exclusive access to betas and the donator section of the forum.
UGX V.I.P.
	Has released one or more maps to the UGX-Mods community which have been added to the UGX Map Manager.
BO3 Modtools Alpha	This user has access to the Black Ops 3 Modtools Alpha

	Benevolent Soul who has our eternal gratitude and exclusive access to betas and the donator section of the forum.
UGX V.I.P.
	Has released one or more maps to the UGX-Mods community which have been added to the UGX Map Manager.
BO3 Modtools Alpha	This user has access to the Black Ops 3 Modtools Alpha

	Benevolent Soul who has our eternal gratitude and exclusive access to betas and the donator section of the forum.
UGX V.I.P.
	Has released one or more maps to the UGX-Mods community which have been added to the UGX Map Manager.
BO3 Modtools Alpha	This user has access to the Black Ops 3 Modtools Alpha

	Benevolent Soul who has our eternal gratitude and exclusive access to betas and the donator section of the forum.
	Has shown excellence and experience in the area of custom scripting in the UGX-Mods community.
Oil Rig Beta Access

UGX-Mods Login

Replacing Grenades With New Grenades