Some pets in Plane of Sky

[Secrets]

Originally Posted by drakelord

Well now, this thread got hijacked sideways.

I wouldn't say it got hijacked... maybe brutally mauled now and there.

I think it's nice we can have civil discussion like this. I think there's a bit of existing tension over the matter of what gets into code, and it's nice that we discuss the future of the project... it just so happens that it was in this thread.

[Dibalamin]

To kinda help get back on track:

Trev, I've never been able to run a script on a pet with sub EVENT_SPAWN. I spent DAYS trying to use it to stop enchanter pets from punching by adding the gear to them via a perl script. I eventually had to cave and start the timer in spawn and have the sub EVENT_TIMER add the gear.

Not clean but effective.

I've already blocked the spells in sky that aren't supposed to be there, that isn't a big deal for me & I tried remodeling the mobs just for Sky, big fat failure. From a development standpoint, I think source is the right fix for this issue if live emulation is the target.

[Congdar]

Originally Posted by Dibalamin

Trev, I've never been able to run a script on a pet with sub EVENT_SPAWN...I eventually had to cave and start the timer in spawn and have the sub EVENT_TIMER add the gear.

Not clean but effective.

The second to the last line in Mob::MakePet() is entity_list.AddNPC(npc); and if you change it to entity_list.AddNPC(npc, true, true); and then put your code back into EVEN_SPAWN from EVENT_TIMER it will work.

Originally Posted by Dibalamin

I've already blocked the spells in sky that aren't supposed to be there, that isn't a big deal for me & I tried remodeling the mobs just for Sky, big fat failure. From a development standpoint, I think source is the right fix for this issue if live emulation is the target.

There had to be over 50 non-air pet spells for the magicain that you had to add to the table. could you share that sql?

[nilbog]

Thanks a lot for posting the source fix for this issue Congdar. From everything I remember from the 1st time I went to airplane, till I quit playing live, necro and mage pets were wisps and djinni.

I took a while one day and fixed the pets using scripts.. and I have to say, it felt like a cheap workaround. Using the timer method, it still has a delay till everything looks right. My method involved creating 2 new illusion spells, wisp and djinn.

heres illusion code for skeleton pet, if it might help anyone w/ anything else.

sub EVENT_DEATH{
quest::say("Sorry to have failed you, oh Great One.");
 }

sub EVENT_SPAWN { 
 quest::say("At your service Master.");
 $mobzone = $npc->GetZoneID() ;
 if ($mobzone == 71) {
 quest::settimer(1,0);
}
}

sub EVENT_TIMER {
if($timer eq "1") {
$skel = $entity_list->GetMobByNpcTypeID(614);
if ($skel) {
      $wisp = $skel->CastToNPC();
$npc->CastSpell(8450,$npc);
quest::stoptimer(1);
}
}
}

Thanks again Congdar!

[Dibalamin]

I PM'd you my email Congdar, drop me a line there and I can send you the blocked_spells table...me and queries don't mix.

[cavedude]

Would you mind making that SQL public? I was meaning to do it myself one day but if you've already got it done, then no need to reinvent the wheel.

Going way back to the original code. From a content standpoint it is correct. Before going into SVN, a simple rule can be added to disable it if anybody doesn't want their airplane changed, no big deal. (I can understand that and agree with that.) Now, I do believe you have a technical issue which KLS would know more about. I believe specifying the zoneid may not work once the instancing stuff hits trunk. But, again you'll need to clarify with KLS.

[Dibalamin]

It isn't complete yet, few more spells need blocked, but my eyes started crossing after doing #findspell over and over and over.

Keep in mind, this SQL will block all (I think) forms of teleportation from the plane of sky including gate. Leaving the parachute the only way to leave the plane. As well as coth. Alter plane sky will still function by teleporting you to the first island.

I don't think I blocked Lev spells yet.

http://www.stormrunner.net/download/blockedspells.sql