MrSmite
MrSmite
> [593721f](https://github.com/TrinityCore/TrinityCore/commit/593721ff1fc7bb20fa2667625f5d3848a0786c73) Seems still valid. Thanks for keeping my bug reports refreshed :smile:
Update: Still exists in 42f1a0babfaacf64df31a934288979f8edfec940, will try to look into it Note: Holy crap, this is an old report.
Update: _PetAI::CanAttack()_ checks _HasBreakableByDamageCrowdControlAura()_ and returns _IsCommandAttack()_ so the issue seems to be outside the AI.
Github sent a notification that this had a new message... but the comment listed in the e-mail is not here
I don't think tracking this is really necessary: ``` C++ currentTargetNotAffectedByCCEffects = true; ``` All you have to do is ignore the CC when the player clicks attack, for example:...
> A special variable is needed to determine if target was already affected by CC or CC was applied while pet was already attacking. No, not really (see below) >...
@Stryg The check for *IsCommandAttack()* is designed specifically to have the pet break existing CC when told to. When I first rewrote the AI, the CC check was done by...
> [835e291](https://github.com/TrinityCore/TrinityCore/commit/835e2918319ed20109992086311a2c146140937f) Seems still valid but only for targets trapped in freezing trap, not for other crowd controls. That's interesting. I wonder if freezing trap is not returning **_HasBreakableByDamageCrowdControlAura_** properly....
I think the problem is that some of those Horde NPCs were marked neutral to stop them from attacking the player. Technically they should still show the hostile (red) name...
Hmm... well, Samantha has *faction=104* (Thunder Bluff) which has *hostileMask=2* (Alliance players and pets) but both the enemy / friend factions are blank. Vlagga has *faction=1760* (Dark Portal Defender -...