[Patch] Item - Reinforced Net

[tyrael]

This is used for quest Death from Above

SpellAuras.cpp

                   case 46361:                                     // Reinforced Net                    
                   {
                       if (Unit* caster = GetCaster())
                          target->GetMotionMaster()->MoveFall();
                   }
                       break;
                   case 46699:                             // Requires No Ammo

Respect

[Unkle Nuke]

Such a simple fix as this should be easy to review. I hope it gets quickly accepted.

[Schmoozerd]

Not entirely sure if this is correct:

1) Stunning/ Rooting while they are netted missing?

2) what happens when the net wears off, do they return to their normal position (hight)?

[tyrael]

1. they are not stunned , and they should not be stunned. They fall from the sky and get rooted...

2. When the net wears of they fly back to the original position. But if u aggro them while they are down, they will get in combat and start fighting.

I don't remember very well if they fly back to the original height after net expires... I can't really test for the moment but if anyone can pls feel free and report back this little info.

[Unkle Nuke]

I just did that quest on retail a few months ago, around the time of client 4.2.0.

The net is cast and they do fall to the ground snared. Once the net expires, they fly back up to their nearest waypoint, returning to their original path as they circle above. If the enter combat before flying away, they remain on or near the ground until combat ends. If they are still alive at the end of combat, they will fly back up to their original path in the air as stated previously.

[tyrael]

well with this patch they react as u pointed unklenuke .. the only thing that I don't remember is the flying back...

I will try to get my hand on my computer and test it...

[Unkle Nuke]

Sorry, I should have gone into more detail...

If you don't kill them, they do return immediately to their previous position, hovering in the air at about the same height as the scaffold platform where the dwarf stands. If they wander from the spot where they landed when snared by the net, such as being kited by the player, that's when they will fly straight back to their last waypoint. For those that were already hovering by the platform this will be the spot in the air where they were before the net snares them. If they were snared or get aggro as they were flying in, but haven't arrived in front of the platform (like when the following waves arrive from the west to replace those killed), and are not killed in combat, they return to the nearest waypoint of their flight path and continue on until they end up hovering in front of the platform.

I hope that makes sense. 8)

[tyrael]

well I understood it from the beggining...

I tested, and the result is this:

- mob doesn't get back to his original position if u don't attack him ONLY after u aggro and evade from combat he will leave back to his original place, otherwise as I said before, he will just stand there.

also we don't need if (Unit* caster = GetCaster()) in the code..it works the same without this part

[Unkle Nuke]

The Flamespitter remaining on the ground does not seem correct. The Flamespitter will remain on the ground if you do not attack or evade only because you have remained within aggro radius, but did not have enough aggro for the Flamespitter to attack you, so it stays there waiting for your attack.

If you snare them with a net, but then evade and do not engage in combat, they should also evade and then fly back into the air, once aggro has reset to zero.

Was your test done on retail or another private server?

[tyrael]

hmm... I will have to do one more test to check this part

If you snare them with a net, but then evade and do not engage in combat, they should also evade and then fly back into the air, once aggro has reset to zero.

my test was on a private server

[Unkle Nuke]

I did the quest on official retail, client patch 4.2.0. You can't really trust that a private server will be 100% like retail.

[tyrael]

pff..I guess here we have a language barrier...

I totally agree with ya and I just said that I need to check that part to see if it is on private server, I mean with this patch, as u described in your reply

[tyrael]

well I retested my patch with your info unklenuke and I can add that the mobs do not fly back if I do not engage them in battle.

I will look to see what I can do, but if anyone else can help out with this small problem pls feel free to post it.

[Unkle Nuke]

It's been done with many other quests, so would it be necessary to script the Flamespitters with EventAI to have them behave correctly?

Or is it a matter of using timers and resetting some flags?

[tyrael]

well I am not 2 very familiar with acid system... but if it's possible to make a check for those creatures to get back to their original place when the net on them fade, than that's the rest of the fix

[Unkle Nuke]

Maybe ask X-Savior his opinion on it? He'd probably know better than anyone if the Flamespitters need to be scripted with ACID or ScriptDev2, if at all.

[Schmoozerd]

well, default behaviour (returning to fly) should likely be handled as part of the aura-unapply event in core.

Also I think this core change also requires the additional rooting.

EventAI will be required for normal spells and other behaviour - but most likely not related to this not yet complete patch