[Trigger] Local Group not detected

[Fakuspakus]

My ability works this way:

• Meat Cannon Copy
  • Events
    • Unit - A unit Begins channeling an ability
  • Conditions
    • (Ability being cast) Equal to Meat Cannon
  • Actions
    • Custom script: local location udg_LocalPoint
    • Custom script: local location udg_LocalPoint2
    • Custom script: local group udg_LocalGroup
    • Set Caster = (Triggering unit)
    • Set LocalPoint = (Target point of ability being cast)
    • Set LocalPoint2 = (Position of Caster)
    • Set LocalGroup = (Units within 300.00 of LocalPoint matching ((((Matching unit) belongs to an ally of (Owner of Caster)) Equal to False) and (((Matching unit) is alive) Equal to True)))
    • Unit - Create 1 Dummy Meat Wagon for (Owner of Caster) at LocalPoint2 facing Default building facing degrees
    • Unit - Add a 1.50 second Generic expiration timer to (Last created unit)
    • Unit - Order (Last created unit) to Attack Ground LocalPoint
    • Unit - Cause Caster to damage Caster, dealing ((Real((Max HP of Caster))) / 6.00) damage of attack type Chaos and damage type Normal
    • Unit Group - Pick every unit in LocalGroup and do (Actions)
      • Loop - Actions
        • Unit - Cause Caster to damage (Picked unit), dealing ((40.00 x (Real((Level of Meat Cannon for Caster)))) + (4.00 x (Real((Strength of Caster (Include bonuses)))))) damage of attack type Spells and damage type Normal
    • Custom script: set udg_Caster = null
    • Custom script: call DestroyGroup(udg_LocalGroup)
    • Custom script: call RemoveLocation(udg_LocalPoint)
    • Custom script: call RemoveLocation(udg_LocalPoint2)

But I want it to be MUI, so caster to be local and to add wait:

• Meat Cannon Copy
  • Events
    • Unit - A unit Begins channeling an ability
  • Conditions
    • (Ability being cast) Equal to Meat Cannon
  • Actions
    • Custom script: local unit udg_Caster
    • Custom script: local location udg_LocalPoint
    • Custom script: local location udg_LocalPoint2
    • Custom script: local group udg_LocalGroup
    • Set Caster = (Triggering unit)
    • Set LocalPoint = (Target point of ability being cast)
    • Set LocalPoint2 = (Position of Caster)
    • Set LocalGroup = (Units within 300.00 of LocalPoint matching ((((Matching unit) belongs to an ally of (Owner of Caster)) Equal to False) and (((Matching unit) is alive) Equal to True)))
    • Unit - Create 1 Dummy Meat Wagon for (Owner of Caster) at LocalPoint2 facing Default building facing degrees
    • Unit - Add a 1.50 second Generic expiration timer to (Last created unit)
    • Unit - Order (Last created unit) to Attack Ground LocalPoint
    • Unit - Cause Caster to damage Caster, dealing ((Real((Max HP of Caster))) / 6.00) damage of attack type Chaos and damage type Normal
    • Wait 1.30 seconds
    • Unit Group - Pick every unit in LocalGroup and do (Actions)
      • Loop - Actions
        • Unit - Cause Caster to damage (Picked unit), dealing ((40.00 x (Real((Level of Meat Cannon for Caster)))) + (4.00 x (Real((Strength of Caster (Include bonuses)))))) damage of attack type Spells and damage type Normal
    • Custom script: set udg_Caster = null
    • Custom script: call DestroyGroup(udg_LocalGroup)
    • Custom script: call RemoveLocation(udg_LocalPoint)
    • Custom script: call RemoveLocation(udg_LocalPoint2)

But above code won't work properly(units in group are not damaged) after adding those 2 lines.

 

[Tasyen]

group loop actions are in another function where locals of the main actions are not readable from.
Therefore no damage source (caster) can be found (the global variable is read).

 

[Fakuspakus]

So is there no way to make it MUI or perhaps theres custom script for that?

 

[BloodSoul]

You'll need an auxiliary local variable in order to reference the caster, and let go of shadowing the caster global variable. Combine both of them, and retrieve the value after the wait action.

Hidden

• Custom script: local unit caster = GetTriggerUnit()
• Custom script: local location udg_LocalPoint
• Custom script: local location udg_LocalPoint2
• Custom script: local group udg_LocalGroup
• Custom script: set udg_Caster = caster
• Set LocalPoint = (Target point of ability being cast)
• Set LocalPoint2 = (Position of Caster)
• Set LocalGroup = (Units within 300.00 of LocalPoint matching ((((Matching unit) belongs to an ally of (Owner of Caster)) Equal to False) and (((Matching unit) is alive) Equal to True)))
• Unit - Create 1 Dummy Meat Wagon for (Owner of Caster) at LocalPoint2 facing Default building facing degrees
• Unit - Add a 1.50 second Generic expiration timer to (Last created unit)
• Unit - Order (Last created unit) to Attack Ground LocalPoint
• Unit - Cause Caster to damage Caster, dealing ((Real((Max HP of Caster))) / 6.00) damage of attack type Chaos and damage type Normal
• Wait 1.30 seconds
• Custom script: set udg_Caster = caster
• Unit Group - Pick every unit in LocalGroup and do (Actions)
  • Loop - Actions
    • Unit - Cause Caster to damage (Picked unit), dealing ((40.00 x (Real((Level of Meat Cannon for Caster)))) + (4.00 x (Real((Strength of Caster (Include bonuses)))))) damage of attack type Spells and damage type Normal
• Custom script: call RemoveLocation(udg_LocalPoint)
• Custom script: call RemoveLocation(udg_LocalPoint2)
• Custom script: call DestroyGroup(udg_LocalGroup)
• Custom script: set caster = null

EDIT: Never mind this post. Tasyen said something below where you'll put way less effort into it, and will still have the same result.

 

[Wrda]

local location udg_LocalPoint and udg_LocalPoint2 are useless, they're not doing anything there, remove the locations before the wait and it's all good

 

[BloodSoul]

local location udg_LocalPoint and udg_LocalPoint2 are useless, they're not doing anything there, remove the locations before the wait and it's all good

Yeah, you're right about that. I honestly overlooked it. Though I wonder if Fakuspakus really wants to add all units near the target point to LocalGroup before the wait.

 

[Fakuspakus]

• Meat Cannon Copy Copy
  • Events
    • Unit - A unit Begins channeling an ability
  • Conditions
    • (Ability being cast) Equal to Meat Cannon
  • Actions
    • Custom script: local unit udg_Caster
    • Custom script: local group udg_LocalGroup
    • Set Caster = (Triggering unit)
    • Set LocalPoint = (Target point of ability being cast)
    • Set LocalPoint2 = (Position of Caster)
    • Set LocalGroup = (Units within 300.00 of LocalPoint matching ((((Matching unit) belongs to an ally of (Owner of Caster)) Equal to False) and (((Matching unit) is alive) Equal to True)))
    • Unit - Create 1 Dummy Meat Wagon for (Owner of Caster) at LocalPoint2 facing Default building facing degrees
    • Unit - Add a 1.50 second Generic expiration timer to (Last created unit)
    • Unit - Order (Last created unit) to Attack Ground LocalPoint
    • Unit - Cause Caster to damage Caster, dealing ((Real((Max HP of Caster))) / 6.00) damage of attack type Chaos and damage type Normal
    • Custom script: call RemoveLocation(udg_LocalPoint)
    • Custom script: call RemoveLocation(udg_LocalPoint2)
    • Wait 1.30 seconds
    • Unit Group - Pick every unit in LocalGroup and do (Actions)
      • Loop - Actions
        • Unit - Cause Caster to damage (Picked unit), dealing ((40.00 x (Real((Level of Meat Cannon for Caster)))) + (4.00 x (Real((Strength of Caster (Include bonuses)))))) damage of attack type Spells and damage type Normal
    • Custom script: set udg_Caster = null
    • Custom script: call DestroyGroup(udg_LocalGroup)

Still doesn't work

 

[Tasyen]

Remove "Custom script: local unit udg_Caster"
after the wait inser
set Caster = Triggering Unit

Triggering unit is safe over waits

 

[Wrda]

after the wait inser
set Caster = Triggering Unit

Or just don't insert any unit variable at all, just use triggering unit, no need extra variables.

 

[Fakuspakus]

But I thought having waits and global castervariable is not MUI?

 

[BloodSoul]

But I thought having waits and global castervariable is not MUI?

and that's why Tasyen told you to re-assign (Triggering Unit) to the global variable after the wait. In fact, you'll be better off with Wrda's suggestion: just use (Triggering Unit).