Pinging minimap trigger that doesn't work

[terranempire]

So, this trigger doesn't really work, I have no idea why, and I have made multiple tests.

And I also want it to reveal the position of the casting unit for 15 second, but what action does that? (they confuse me)

• Trigger
  • Events
    • Unit - A unit Begins channeling an ability
  • Conditions
    • (Ability being cast) Equal to Flamestrike
  • Actions
    • Unit Group - Pick every unit in (Units in (Playable map area) matching ((Unit-type of (Picked unit)) Equal to Farm)) and do (Actions)
      • Loop - Actions
        • Player Group - Add Player 1 (Red) to PlayerG
        • Player Group - Add (Owner of (Picked unit)) to PlayerG
        • Set AMPoint = (Position of (Casting unit))
        • Cinematic - Ping minimap for PlayerG at AMPoint for 4.00 seconds, using a Simple ping of color (100.00%, 100.00%, 0.00%)
        • Custom script: call RemoveLocation(udg_AMPoint)
        • Set AMPoint = (Target point of ability being cast)
        • Cinematic - Ping minimap for PlayerG at AMPoint for 15.00 seconds, using a Simple ping of color (100.00%, 0.00%, 0.00%)
        • Custom script: call RemoveLocation(udg_AMPoint)
        • Player Group - Remove (Owner of (Picked unit)) from PlayerG

 

[Azeroc]

Can't you just do this since you want to reveal channeling caster to people:

• Channel Reveal
  • Events
    • Unit - A unit Begins channeling an ability
  • Conditions
    • (Ability being cast) Equal to Flame Strike
  • Actions
    • Set TempLoc1 = (Position of (Triggering unit))
    • Cinematic - Ping minimap for (All players) at TempLoc1 for 15.00 seconds, using a Simple ping of color (100.00%, 0.00%, 0.00%)
    • Custom script: call RemoveLocation(udg_TempLoc1)

 

[terranempire]

Can't you just do this since you want to reveal channeling caster to people:

• Channel Reveal
  • Events
    • Unit - A unit Begins channeling an ability
  • Conditions
    • (Ability being cast) Equal to Flame Strike
  • Actions
    • Set TempLoc1 = (Position of (Triggering unit))
    • Cinematic - Ping minimap for (All players) at TempLoc1 for 15.00 seconds, using a Simple ping of color (100.00%, 0.00%, 0.00%)
    • Custom script: call RemoveLocation(udg_TempLoc1)

No, I can't.
I want to reveal the position of the channeling unit, ping its location and ping the point that was targeted with the ability.

 

[Azeroc]

Well that ain't so hard...

• Channel Reveal
  • Events
    • Unit - A unit Begins channeling an ability
  • Conditions
    • (Ability being cast) Equal to Flame Strike
  • Actions
    • Set TempLoc1 = (Position of (Triggering unit))
    • Set TempLoc2 = (Target point of ability being cast)
    • Cinematic - Ping minimap for (All players) at TempLoc1 for 15.00 seconds, using a Simple ping of color (0.00%, 100.00%, 0.00%)
    • Cinematic - Ping minimap for (All players) at TempLoc2 for 15.00 seconds, using a Simple ping of color (100.00%, 0.00%, 0.00%)
    • Custom script: call RemoveLocation(udg_TempLoc1)
    • Custom script: call RemoveLocation(udg_TempLoc2)

 

[terranempire]

Well that ain't so hard...

• Channel Reveal
  • Events
    • Unit - A unit Begins channeling an ability
  • Conditions
    • (Ability being cast) Equal to Flame Strike
  • Actions
    • Set TempLoc1 = (Position of (Triggering unit))
    • Set TempLoc2 = (Target point of ability being cast)
    • Cinematic - Ping minimap for (All players) at TempLoc1 for 15.00 seconds, using a Simple ping of color (0.00%, 100.00%, 0.00%)
    • Cinematic - Ping minimap for (All players) at TempLoc2 for 15.00 seconds, using a Simple ping of color (100.00%, 0.00%, 0.00%)
    • Custom script: call RemoveLocation(udg_TempLoc1)
    • Custom script: call RemoveLocation(udg_TempLoc2)

Okay, firstly, that doesn't help at all.
I know that your trigger will work, but have you looked at the original trigger?
Not to be rude but, that solved absolutely nothing.

The important part is that only players with a certain structure gets his/her minimap pinged, and the location of the casting unit revealed.

 

[terranempire]

Uh, sorry for the double post but this still hasn't been solved, and I would be thankful if somebody could help me solve it.

 

[Azeroc]

How about this? Didn't had time to test this, but I think it could work.

• Channel Reveal
  • Events
    • Unit - A unit Begins channeling an ability
  • Conditions
    • (Ability being cast) Equal to (==) Flame Strike
  • Actions
    • Set TempLoc1 = (Position of (Triggering unit))
    • Set TempLoc2 = (Target point of ability being cast)
      • Do Multiple ActionsFor each (Integer A) from 1 to 12, do (Actions)
        • Loop - Actions
          • Set TempGroup = (Units owned by (Player((Integer A))) of type Farm)
            • Multiple FunctionsIf (All Conditions are True) then do (Then Actions) else do (Else Actions)
              • If - Conditions
                • (TempGroup is empty) Equal to (==) False
              • Then - Actions
                • Cinematic - Ping minimap for (Player group((Player((Integer A))))) at TempLoc1 for 15.00 seconds, using a Simple ping of color (100.00%, 0.00%, 0.00%)
                • Cinematic - Ping minimap for (Player group((Player((Integer A))))) at TempLoc2 for 15.00 seconds, using a Simple ping of color (0.00%, 100.00%, 0.00%)
              • Else - Actions
          • Custom script: call DestroyGroup(udg_TempGroup)
    • Custom script: call RemoveLocation(udg_TempLoc1)
    • Custom script: call RemoveLocation(udg_TempLoc2)

 

[terranempire]

Uh, that one worked, except for one small thing: it pings the locations for players that did have the structure but lost it for some reason or another.

 

[Maker]

You need to change this

• Set TempGroup = (Units owned by (Player((Integer A))) of type Farm)

Use

• Set TempGroup = (Units owned by (Player((Integer A)) matching ((((Matching unit) is alive) Equal to True) and ((Unit-type of (Matching unit)) Equal to Farm)))

 

[terranempire]

Yeah, it works now.
Thanks!

 

[Azeroc]

Ah yea... Forgot that group picks dead unitz too xD