Monster weapons

[Noxih]

Update!!!There was a couple more bugs with a couple of weapons but I fixed overwrite all old files

[WonderlandADnc]

SOUNDS GREAT. is it a case of add the item., then click it. on yr paperdoll. if not that sounds great/ if so. cool. great gob.

[Noxih]

nope its onequip now

[Unicorn1259]

This is a great script, but there is one problem with it. when I equip one of them & then take it out of my hand it changes me to a male & I am female. I had to kill myself to change back to female.

[Packer898]

Well there are a few problems with these scripts. This will show you how to directly fix the problem of changing sex's.

		public override bool OnEquip( Mobile from )
		{
			if ( from.Mounted == true )
			{
				from.SendLocalizedMessage( 1042561 );
			}
			else if ( from.BodyValue == 0x190 || from.BodyValue == 0x191  )//Change here, you dont need the from.Female = false. This is determined by 0x191 value.
			{
				from.BodyMod = 0x9F;
				from.HueMod = 0x0;
				from.PlaySound( 278 );
			}
			return base.OnEquip(from);
			
		}
		
		public override void OnRemoved( object parent )
		{
			if(parent is Mobile)
			{
				Mobile from = (Mobile)parent;
				
				if ( from.BodyMod == 0x9F )//Change this to from.BodyMod
				{
					from.BodyMod = 0x0;//Change this
					from.HueMod = -1;
					from.PlaySound( 701 );
				}
			}
		}

Now on a side note. You should also include something to hide guildtitles etc...

p.s. You will need to change the BodyMod values to reflect whichever script you are fixing.

[XxSP1DERxX]

Not to piss people off
but please you guys...

if ( from.Mounted == true )

is the same as

if ( from.Mounted )

lol - An if statement returns a bool, so you don't need to evaluate it to a boolean. Make sense?

[Packer898]

SPeaking of which... You need to edit that if statement... As it sits you could equip the item with a mount and it would look really funky...

		public override bool OnEquip( Mobile from )
		{
			if ( from.Mounted )
			{
				from.SendLocalizedMessage( 1042561 );
				return false;//You need to add this in so they cant use while mounted...
			}
			else if ( from.BodyValue == 0x190 || from.BodyValue == 0x191  )//Change here, you dont need the from.Female = false. This is determined by 0x191 value.
			{
				from.BodyMod = 0x9F;
				from.HueMod = 0x0;
				from.PlaySound( 278 );
			}
			return base.OnEquip(from);
			
		}

[Unicorn1259]

Thanks guys I made the proper changes & it works great now

[Packer898]

Np your welcome.

[WonderlandADnc]

you should update all that and repost the files. thanks again.

[Noxih]

thx for the info i will

[Noxih]

All scripts are updated fixed the bug with not being able to switch back to a woman,you can now equip while mounted

[Packer898]

Originally Posted by Noxih

you can now equip while mounted

You dont want to do that. It will cause some weird graphic issues. You need to amke it NOT allow you to equip while mounted...

		public override bool OnEquip( Mobile from )
		{
			if ( from.Mounted )
			{
				from.SendLocalizedMessage( 1042561 );
				return false;//You need to add this in so they cant use while mounted...
			}

[Noxih]

i have tested them all and no graphic issues

[Packer898]

Originally Posted by Noxih

you can now equip while mounted

If you have it so you CAN equip the items while mounted then there is inded some weird graphic issues. People on my shard can hardly even see the person.

If you have it so they CANT then it works properly.

Maybe this is just a grammar misunderstanding =)-

[Noxih]

when you use it does it look fine to you
maybe thats why it looks fine to me if thats the case ill change it back. if anyone else has this problems post plz

[Noxih]

Originally Posted by Packer898

If you have it so you CAN equip the items while mounted then there is inded some weird graphic issues. People on my shard can hardly even see the person.

If you have it so they CANT then it works properly.

Maybe this is just a grammar misunderstanding =)-

dang you right i just logged onto two accounts then looked at it and it looks fine to thew person holding the weapon but not to others lol on ill change it back

[Packer898]

Not to burst your bubble or anything but you might want to test scripts a little further before releasing them on the script submissions forum. I really like your idea here but its obvious it needs(ed) alot of tuning.

Keep chugging away though. =)-

[Noxih]

sorry about that never thouht to look at it at a second person veiw but i fixed it now i changed it back to not being able to equip it while on a mobile thx for the pointer pack

[WonderlandADnc]

great script., thanks for the update, ive been looking into making them so you can click the item. so u can use it. as yr player, and trans to the monster. all i can see is this, any info would be great.

public override void OnDoubleClick( Mobile from ) 
		{ 
			if ( from.FindItemOnLayer( Layer.TwoHanded ) == this ) 
			{ 
				if ( DateTime.Now >= PreviousUse + TimeSpan.FromSeconds(30) )
				{ 
					PreviousUse = DateTime.Now;
					if ( m_Charges == 0 ) 
					{ 
						from.SendLocalizedMessage( 1019073 ); // This item is out of charges. 
						return; 
					} 
          			
					else 
					{ 
						--m_Charges; 
					} 
						switch ( m_StaffEffect ) 
					{ 
						case StaffEffect.Freeze: new FreezeSpell( from, null ).Cast(); break; 
						case StaffEffect.IceStrike: new IceStrikeSpell( from, null ).Cast(); break; 
						case StaffEffect.IceBall: new IceBallSpell( from, null ).Cast(); break; 
					}

[Noxih]

Originally Posted by WonderlandADnc

great script., thanks for the update, ive been looking into making them so you can click the item. so u can use it. as yr player, and trans to the monster. all i can see is this, any info would be great.

public override void OnDoubleClick( Mobile from ) 
		{ 
			if ( from.FindItemOnLayer( Layer.TwoHanded ) == this ) 
			{ 
				if ( DateTime.Now >= PreviousUse + TimeSpan.FromSeconds(30) )
				{ 
					PreviousUse = DateTime.Now;
					if ( m_Charges == 0 ) 
					{ 
						from.SendLocalizedMessage( 1019073 ); // This item is out of charges. 
						return; 
					} 
          			
					else 
					{ 
						--m_Charges; 
					} 
						switch ( m_StaffEffect ) 
					{ 
						case StaffEffect.Freeze: new FreezeSpell( from, null ).Cast(); break; 
						case StaffEffect.IceStrike: new IceStrikeSpell( from, null ).Cast(); break; 
						case StaffEffect.IceBall: new IceBallSpell( from, null ).Cast(); break; 
					}

if your able to get that working it would be a nice addon to these weapons look in script support under help plz thats where i had it ondoubleclick

[Noxih]

Oh packer898 im sorry if i offended you in any what about the graphic flaw you are alot more exp that me

[Packer898]

Originally Posted by Noxih

Oh packer898 im sorry if i offended you in any what about the graphic flaw you are alot more exp that me

You didnt. I was just informing you off the bug. My comments about testing more before releasing was just general advice. Scripts that are properly scripted without bugs are always a good thing. =)-

[WonderlandADnc]

ok well, ive been looking and i coulnt find it in yr old script. but i did find this, i think its more the sort of thing. maybe.

public override void OnDoubleClick( Mobile m ) 
{ 
Item i = m.FindItemOnLayer( base.Layer );
if (i != null) // Did we find an item?
{ 
m.AddToBackpack(i); //put the item in their backpack (unequip it) 
} 
m.AddItem(this); // Equip this item;

}

[Packer898]

Originally Posted by WonderlandADnc

ok well, ive been looking and i coulnt find it in yr old script. but i did find this, i think its more the sort of thing. maybe.

public override void OnDoubleClick( Mobile m ) 
{ 
Item i = m.FindItemOnLayer( base.Layer );
if (i != null) // Did we find an item?
{ 
m.AddToBackpack(i); //put the item in their backpack (unequip it) 
} 
m.AddItem(this); // Equip this item;

}

What is that for? That would just automatically equip it on 2x click. *Shrugs* Might be cool... Only problem with that is if an item is already using that layer...