[Trigger] Simple trigger won't work.

[BagOfChips]

I'm making a trigger for a tower, that a second after being attacked, the units will become intoxicated by toxic sludge, and this is what I've made... however it doesn't work. (Tried to make it MUI)

=[ any ideas? Any glaring mistakes?

• Sewage
  • Events
    • Unit - A unit Is attacked
  • Conditions
    • (Unit-type of (Attacking unit)) Equal to Ancient of Wonders
    • (Owner of (Attacked unit)) Equal to Player 10 (Light Blue)
  • Actions
    • -------- Locals --------
    • Custom script: local unit udg_SewageDummy
    • Custom script: local unit udg_SewageTarget
    • Custom script: local location udg_SewagePosCast
    • -------- Set Variables --------
    • Set SewageTarget = (Attacked unit)
    • Set SewagePosCast = (Position of (Attacking unit))
    • Unit - Create 1 Toxic Sewage (Dummy) for (Owner of (Attacking unit)) at SewagePosCast facing (Facing of (Attacking unit)) degrees
    • Set SewageDummy = (Last created unit)
    • -------- Action --------
    • Unit - Add Toxic Sludge (Poison, Sewage) Dummy to SewageDummy
    • Wait 1.00 seconds
    • Unit - Order SewageDummy to Neutral Alchemist - Acid Bomb SewageTarget
    • -------- Leak Clean up --------
    • Custom script: call RemoveLocation(udg_SewagePosCast)
    • -------- Null locals --------
    • Custom script: set udg_SewagePosCast = null
    • Custom script: set udg_SewageDummy = null
    • Custom script: set udg_SewageTarget = null

 

[redmarine]

• Sewage
  • Events
    • Unit - A unit Is attacked
  • Conditions
    • (Unit-type of (Attacking unit)) Equal to Ancient of Wonders
    • (Owner of (Attacked unit)) Equal to Player 10 (Light Blue)
  • Actions
    • -------- Locals --------
    • Custom script: local unit udg_SewageDummy
    • Custom script: local unit udg_SewageTarget
    • Custom script: local location udg_SewagePosCast
    • -------- Set Variables --------
    • Set SewageTarget = (Attacked unit)
    • Set SewagePosCast = (Position of (Attacking unit))
    • Unit - Create 1 Toxic Sewage (Dummy) for (Owner of (Attacking unit)) at SewagePosCast facing (Facing of (Attacking unit)) degrees
    • Set SewageDummy = (Last created unit)
    • -------- Action --------
    • Unit - Add Toxic Sludge (Poison, Sewage) Dummy to SewageDummy
    • Wait 1.00 seconds
    • Unit - Order SewageDummy to Neutral Alchemist - Acid Bomb SewageTarget
    • -------- Leak Clean up --------
    • Custom script: call RemoveLocation(udg_SewagePosCast)
    • -------- Null locals --------
    • Custom script: set udg_SewagePosCast = null
    • Custom script: set udg_SewageDummy = null
    • Custom script: set udg_SewageTarget = null

 

[BagOfChips]

Not sure if the location of the dummy unit really makes a difference here... leme check.

 

[redmarine]

Nvm I was wrong. Make sure the dummy unit has the ability and the amount of mana required.

 

[BagOfChips]

Dummy unit spawns, just doesn't do anything.

Gave the unit 1000 mana init / mana max... and made the spell free to boot. He just won't cast it.

 

[redmarine]

What is the range of the ability being casted perhaps the problem lies there? Also make sure the ability can target enemy structures.

 

[BagOfChips]

What is the range of the ability being casted perhaps the problem lies there? Also make sure the ability can target enemy structures.

I set the range to the correct amount, and I'll check the targets in a bit.

 

[BagOfChips]

Bump, need help!

 

[BagOfChips]

Still nothing? =/

 

[redmarine]

I believe it would be better to do this in Jass. I recommend you to take this sample to the Jass section and ask them for a complete Jass version.

 

[Anachron]

I agree to redmarine, jass makes everything easier.

 

[spiwn]

The problem is that you can make only one GUI global local variable per GUI trigger.
So out of:
Custom script: local unit udg_SewageDummy
Custom script: local unit udg_SewageTarget
Custom script: local location udg_SewagePosCast
Only the first works.
Convert to text, rename the variables to be really local and correct their names when they are used.

 

[Huurka]

Remove the wait.

Add a generic timer to dummy unit, too.