[vJASS] Cancel Casting Ability

[Mr_Bean]

Hi,

I am having trouble with cancelling an ability from being cast. I want it to not be cast when the unit has a specific buff. My code is like this:

private function EvaluateCaster takes nothing returns boolean
    if IsUnitInGroup(GetTriggerUnit(), units) and GetSpellAbilityId() == SPELL_ID then
        call IssueImmediateOrder(GetTriggerUnit(), "stop")
        call DisplayTextToPlayer(GetTriggerPlayer(), 0, 0, "|cffffcc00Hero already has the buff!|r")
        return false
    else
        return true
    endif
endfunction

private function onInit takes nothing returns nothing
    local trigger t = CreateTrigger()
    
    // Register cast to detect if caster already has buff.
    set t = CreateTrigger()
    call TriggerRegisterAnyUnitEventBJ(t, EVENT_PLAYER_UNIT_SPELL_CAST)
    call TriggerAddCondition(t, Condition(function EvaluateCaster))
    
    // Register spell cast.
    call RegisterSpellEffectEvent(SPELL_ID, function SpellActions)
    
    set t = null
endfunction

What happens: It displays the message "Hero already has the buff!", but the function SpellActions still gets executed!

What's wrong? Thanks,

Mr_Bean

 

[Troll-Brain]

Try pause unit/order stop/unpause unit.

 

[Mr_Bean]

Thanks, that works. What's up with having to Pause/Stop/Unpause?

 

[Troll-Brain]

Pause forbid the issued order on event, and then you can give an other order.
However it depends the type of issued order and the type of order you want to give (point, immediate, target), i mean most of times just give the order stop on order event will work.

 

[Mr_Bean]

Oh okay, makes sense. Do I need to pause/stop/unpause or can I just pause/unpause?

 

[Troll-Brain]

Well, you can try, but i think that the spell order would just be re-issued without the order stop.
Also instead of stop you could use the hidden order "851973", which is the "stun" order.
I'm tired to say why this one is better, use the search