AoE Ability based off of Hero Stat Not working

[shermellow]

• TaurenChieftan WarStomp Damage
  • Events
    • Unit - A unit Starts the effect of an ability
  • Conditions
    • (Ability being cast) Equal to War Stomp
    • (Unit-type of (Triggering unit)) Equal to Tauren Chieftain
  • Actions
    • Unit Group - Pick every unit in (Units within 250.00 of (Position of (Triggering unit)) matching (((Owner of (Picked unit)) is an enemy of (Owner of (Triggering unit))) Equal to True)) and do (Actions)
      • Loop - Actions
        • Unit - Cause (Triggering unit) to damage (Picked unit), dealing ((Real((Strength of (Triggering unit) (Include bonuses)))) x 1.50) damage of attack type Spells and damage type Magic

why does this not work? I basically copied and pasted it off of several other threads using this same setup, same ability all claiming to have no issues....? Help please. I have been working on this one ability for nearly 5ish hours now... feeling kinda stupid for not being able to figure it out. especially when I downloaded a demo using this same setup that worked and all I did was copy and paste this trigger to my map now it doesn't work... and I didn't change anything...

 

[Doomlord]

It is supposed to be (Matching unit), not (Picked unit) in your Unit Group pick line.

On a side note, you should add this before picking the group to prevent leaks.

• Custom script: set bj_wantDestroyGroup = true

You may also want to store (Triggering unit) to a variable to increase efficiency and avoid the painful drop & scroll in GUI.

Position of (Triggering unit) leaks also. Clean that.

Helpful links:

+http://www.hiveworkshop.com/forums/triggers-scripts-269/things-leak-35124/
+http://www.hiveworkshop.com/forums/...rials-279/things-gui-user-should-know-233242/

 

[shermellow]

It is supposed to be (Matching unit), not (Picked unit) in your Unit Group pick line.

On a side note, you should add this before picking the group to prevent leaks.

• Custom script: set bj_wantDestroyGroup = true

You may also want to store (Triggering unit) to a variable to increase efficiency and avoid the painful drop & scroll in GUI.

Position of (Triggering unit) leaks also. Clean that.

Helpful links:

+http://www.hiveworkshop.com/forums/triggers-scripts-269/things-leak-35124/
+http://www.hiveworkshop.com/forums/...rials-279/things-gui-user-should-know-233242/

I used Matching unit to start with and it also did not work. I know about the leaks ( I will take care of that as soon as I get the spell to function properly).

 

[shermellow]

thanks for the links btw. I've searched for tutorials like those before must have used wrong key words or something.

 

[Doomlord]

That is peculiar. I will investigate this a bit. Expect my answer later.

Don't double post btw. Use this button

to edit your previous post and merge the two of them

 

[13oot-Me]

Use variables for things, it will make it easier to change stuff around if you decide to later. Make sure it's the correct war stomp ability, there are two of them.

• TaurenChieftain WarStomp Damage
• Events
  • Unit - A unit Starts the effect of an ability
• Conditions
  • (Ability being cast) Equal to War Stomp
  • (Unit-type of (Triggering unit)) Equal to Tauren Chieftain
• Actions
  • Set - v_Unit = (Triggering unit)
  • Set - v_wsDamage = ((Strength of (v_Unit) (Include bonuses)) x 1.5)
  • Custom script: set bj_wantDestroyGroup = true
  • Unit Group - Pick every unit in (Units within 250.00 of (Position of (v_Unit)) and do (Actions)
    • Loop - Actions
      • Set - v_TempUnit = (Picked unit)
      • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        • If - Conditions
          • ((Owner of (v_Unit)) is an enemy of (Owner of (v_TempUnit))) Equal to True
        • Then - Actions
          • Unit - Cause (v_Unit) to damage (v_TempUnit), dealing (v_wsDamage) damage of attack type Spells and damage type Magic
        • Else - Actions

 

[shermellow]

Well I told you I felt stupid.... I was using the wrong War Stomp.... Total noob move ...... it works perfectly now haha

 

[Doomlord]

Well, never see that one coming >_> Anyway good thing you fixed it now.

@13oot-Me: You are leaking (Position of (v_Unit)). Just so you know

 

[shermellow]

Yeah i'm just a beginner lol. THANKS for all the help though really appreciate it!

 

[Doomlord]

Yeah i'm just a beginner lol. THANKS for all the help though really appreciate it!

Nah, nobody is born an expert. We are all beginners once

 

[mckill2009]

There's no really need for the v_Unit, GetTriggerUnit is OK...
You leaked location..
Owner of v_Unit >>> Triggering Player

Try Owner of (Triggering Player) is an enemy of (Owner of (MatchingUnit))

 

[13oot-Me]

I never said it was the perfect trigger Your welcome for reminding you about the second war stomp though