Need help with autocast trigger.

[Archangel_Tidusx]

I have a Knight that has an ability based off the Healing effect that the Potion of healing uses and I need a way to issue the unit to use the ability.

Trigger

• Autocast LifeRestore Knight
  • Events
    • Unit - A unit Is attacked
  • Conditions
    • (Unit-type of (Attacked unit)) Equal to Knight
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • AutocastEnabled[(Player number of (Owner of (Attacked unit)))] Equal to True
      • Then - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (Percentage life of (Attacked unit)) Less than 25.00
          • Then - Actions
            • Unit - Order (Attacked unit) to "use heal abilitiy"
          • Else - Actions
            • Do nothing
      • Else - Actions
        • Do nothing

The ability code is : A00n:Alh1 (Healing (Lesser)) if that helps you to create a custom script action.

 

[Teelo]

What ability did you copy/paste that ability from? That is the one to use under "issue order with no target".

 

[Berb]

He uses the Potion of Healing (lesser) ability. It doesn't have an order ID, which is the problem.

The order ID associated with the ability you're talking about is 852272. You won't be able to do this in GUI, instead, use this custom script where you have the issued order. Copy and paste the following text into a custom script line.

call IssueImmediateOrderById( GetTriggerUnit(), 852272 )

Just tested it out, it works.

 

[Archangel_Tidusx]

I paste that custom script function where my ability trigger says "use heal ability"?

and Teelo, I stated in the first post that the ability came from the Heal (lesser) item ability. The ability code is below the trigger.

Berbanog, if you reply with a yes, I will test this out tomorrow and if it works in my case, I will +rep you.

 

[Berb]

Yea, but you'll have to use the custom script text thing in GUI. I'll show you:

• Untitled Trigger 001
  • Events
  • Conditions
  • Actions
    • Custom script: call IssueImmediateOrderById( GetTriggerUnit(), 852272 )

And yes, where you had "use heal ability".

• Autocast LifeRestore Knight
• Events
  • Unit - A unit Is attacked
• Conditions
  • (Unit-type of (Attacked unit)) Equal to Knight
• Actions
  • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
    • If - Conditions
      • AutocastEnabled[(Player number of (Owner of (Attacked unit)))] Equal to True
    • Then - Actions
      • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        • If - Conditions
          • (Percentage life of (Attacked unit)) Less than 25.00
        • Then - Actions
          • Custom script: call IssueImmediateOrderById( GetTriggerUnit(), 852272 )
        • Else - Actions
          • Do nothing <-- You don't need this.
      • Else - Actions
        • Do nothing <-- Or this. Just remove them.

I will +rep you.

Oh... my... God... Really?!?

 

[Teelo]

and Teelo, I stated in the first post that the ability came from the Heal (lesser) item ability. The ability code is below the trigger.

I know, was just double checking that you didn't actually copy it off priests heal spell or forest trolls heal or something, since its supposed to be autocast, and those are autocast healing spells.

 

[Archangel_Tidusx]

I know, was just double checking that you didn't actually copy it off priests heal spell or forest trolls heal or something, since its supposed to be autocast, and those are autocast healing spells.

Very well.

And two more things berbanog:

The "do nothing" is the correct way to fill in an 'If then else' function. Also I am not good with jass functions, but because the unit I am referring to is actually "attacked unit", shouldnt the custom script be something more like: call IssueImmediateOrderById( GetAttackedUnit(), 852272 ) ?

 

[Justify]

No, "Do Nothing" is useless, why should it be correct? Doing nothing, I don't need a function for this.
GetTriggerUnit ALWAYS works when a unit triggers a trigger (...), and the event is "A unit gets attacked", so GetTriggerUnit returns the unit being attacked.

 

[Berb]

The "do nothing" is the correct way to fill in an 'If then else' function.

This is what the DoNothing function looks like:

function DoNothing takes nothing returns nothing
endfunction

// This is what the function looks like in the API;

There is no reason for this.

shouldnt the custom script be something more like: call IssueImmediateOrderById( GetAttackedUnit(), 852272 ) ?

The function you're referring to is:

function GetAttackedUnitBJ takes nothing returns unit
    return GetTriggerUnit()
endfunction

As you can see, it just returns GetTriggerUnit(). Trust me I know whats best

 

[Archangel_Tidusx]

Thank you for the help. The funds have already been transferred into your account