Full Version : A Couple Errors
xmlspawner >>XMLPoint's Troubleshooting >>A Couple Errors
<< Prev | Next >>
DrussRob- 01-30-2006
Errors:

CODE
- Error: Scripts\Misc\Notoriety.cs: CS1502: (line 226, column 21) The best over loaded method match for 'Server.Engines.XmlSpawner2.XmlPoints.AreTeamMembers(Ser ver.Mobile, Server.Mobile)' has some invalid arguments - Error: Scripts\Misc\Notoriety.cs: CS1503: (line 226, column 54) Argument '2':
cannot convert from 'Server.Items.Corpse' to 'Server.Mobile'
- Error: Scripts\Misc\Notoriety.cs: CS1502: (line 229, column 25) The best over
loaded method match for 'Server.Engines.XmlSpawner2.XmlPoints.AreChallengers(Ser
ver.Mobile, Server.Mobile)' has some invalid arguments
- Error: Scripts\Misc\Notoriety.cs: CS1503: (line 229, column 58) Argument '2':
cannot convert from 'Server.Items.Corpse' to 'Server.Mobile'

Respective Script Code:

CODE
public static int CorpseNotoriety( Mobile source, Corpse target ) { if ( target.AccessLevel > AccessLevel.Player ) return Notoriety.CanBeAttacked; Body body = (Body) target.Amount;
BaseCreature cretOwner = target.Owner as BaseCreature;
if ( cretOwner != null )
{
Guild sourceGuild = GetGuildFor( source.Guild as Guild, source );
Guild targetGuild = GetGuildFor( target.Guild as Guild, target.Owner );
if ( sourceGuild != null && targetGuild != null )
{
if ( sourceGuild == targetGuild || sourceGuild.IsAlly( targetGuild ) )
return Notoriety.Ally;
else if ( sourceGuild.IsEnemy( targetGuild ) )
return Notoriety.Enemy;
}
if ( CheckHouseFlag( source, target.Owner, target.Location, target.Map ) )
return Notoriety.CanBeAttacked;
int actual = Notoriety.CanBeAttacked;
if ( target.Kills >= 5 || (body.IsMonster && IsSummoned( target.Owner as BaseCreature )) || (target.Owner is BaseCreature && (((BaseCreature)target.Owner).AlwaysMurderer || ((BaseCreature)target.Owner).IsAnimatedDead)) )
actual = Notoriety.Murderer;
if ( DateTime.Now >= (target.TimeOfDeath + Corpse.MonsterLootRightSacrifice) )
return actual;
Party sourceParty = Party.Get( source );
ArrayList list = target.Aggressors;
for ( int i = 0; i < list.Count; ++i )
{
if ( list[i] == source || (sourceParty != null && Party.Get( (Mobile)list[i] ) == sourceParty) )
return actual;
}
return Notoriety.Innocent;
}
else
{
if ( target.Kills >= 5 || (body.IsMonster && IsSummoned( target.Owner as BaseCreature )) || (target.Owner is BaseCreature && (((BaseCreature)target.Owner).AlwaysMurderer || ((BaseCreature)target.Owner).IsAnimatedDead)) )
return Notoriety.Murderer;
if ( target.Criminal )
return Notoriety.Criminal;
Line: 225 /// XmlPoints challenge mod darkstone
Line: 226 if (XmlPoints.AreTeamMembers(source, target))
Line: 227 return Notoriety.Ally;
Line: 228 else
Line: 229 if (XmlPoints.AreChallengers(source, target))
Line: 230 return Notoriety.Enemy;
Line: 231 // end XmlPoints challenge mod
Guild sourceGuild = GetGuildFor( source.Guild as Guild, source );
Guild targetGuild = GetGuildFor( target.Guild as Guild, target.Owner );
if ( sourceGuild != null && targetGuild != null )
{
if ( sourceGuild == targetGuild || sourceGuild.IsAlly( targetGuild ) )
return Notoriety.Ally;
else if ( sourceGuild.IsEnemy( targetGuild ) )
return Notoriety.Enemy;
}
if ( target.Owner != null && target.Owner is BaseCreature && ((BaseCreature)target.Owner).AlwaysAttackable )
return Notoriety.CanBeAttacked;
if ( CheckHouseFlag( source, target.Owner, target.Location, target.Map ) )
return Notoriety.CanBeAttacked;
if ( !body.IsHuman && !body.IsGhost && !IsPet( target.Owner as BaseCreature ) )
return Notoriety.CanBeAttacked;
ArrayList list = target.Aggressors;
for ( int i = 0; i < list.Count; ++i )
{
if ( list[i] == source )
return Notoriety.CanBeAttacked;
}
return Notoriety.Innocent;
}
}

Should target be target.Owner in this case?
ps: I cant seem to get the color code to work inside of the code box.
DrussRob- 01-30-2006
I changed it to target.Owner and it seems to be working.
ArteGordon- 01-30-2006
you put the changes in CorpseNotoriety instead of in MobileNotoriety. They look very similar which is why you got fooled.
DrussRob- 01-30-2006
DOH!
ArteGordon- 01-30-2006
believe me. you are not the first
DrussRob- 01-30-2006
Yeah, but I feel bad for wasting your time on simple stuff I should know.