[JASS] Spell Help Please

[Magni The Ironhammer]

function Trig_snapdragonpoison_Conditions takes nothing returns boolean
    if ( not ( GetSpellAbilityId() == 'A000' ) ) then
        return false
    endif
    return true
endfunction
globals
unit sdpTarget
real sdpDamage
endglobals
function Trig_snapdragonpoison_Actions takes nothing returns nothing
local integer i = 7
set sdpTarget = GetSpellTargetUnit()
set sdpDamage = 0.01 * GetUnitAbilityLevelSwapped('A000',GetSpellAbilityUnit())
loop
 exitwhen i == 0
     call UnitDamageTargetBJ(GetSpellAbilityUnit(),sdpTarget,(GetUnitState(GetSpellTargetUnit(),UNIT_STATE_LIFE) * sdpDamage),ATTACK_TYPE_MAGIC,DAMAGE_TYPE_MAGIC)
	 call TriggerSleepAction(1.00)
	 set i = i - 1
endloop
endfunction

//===========================================================================
function InitTrig_snapdragonpoison takes nothing returns nothing
    set gg_trg_snapdragonpoison = CreateTrigger(  )
    call TriggerRegisterAnyUnitEventBJ( gg_trg_snapdragonpoison, EVENT_PLAYER_UNIT_SPELL_EFFECT )
    call TriggerAddCondition( gg_trg_snapdragonpoison, Condition( function Trig_snapdragonpoison_Conditions ) )
    call TriggerAddAction( gg_trg_snapdragonpoison, function Trig_snapdragonpoison_Actions )
endfunction

why does this only work once then stops?

 

[Dr Super Good]

function Trig_snapdragonpoison_Conditions takes nothing returns boolean
    return GetSpellAbilityId() == 'A000' 
endfunction

function Trig_snapdragonpoison_Actions takes nothing returns nothing
    local unit sdpTarget = GetSpellTargetUnit()
    local real sdpDamage = 0.01 * GetUnitAbilityLevelSwapped('A000',GetSpellAbilityUnit())
    local integer i = 7
    loop
        exitwhen i == 0
        call UnitDamageTargetBJ(GetSpellAbilityUnit(),sdpTarget,(GetUnitState(GetSpellTargetUnit(),UNIT_STATE_LIFE) * sdpDamage),ATTACK_TYPE_MAGIC,DAMAGE_TYPE_MAGIC)
        call TriggerSleepAction(1.00)
        set i = i - 1
    endloop
endfunction

function InitTrig_snapdragonpoison takes nothing returns nothing
    set gg_trg_snapdragonpoison = CreateTrigger(  )
    call TriggerRegisterAnyUnitEventBJ( gg_trg_snapdragonpoison, EVENT_PLAYER_UNIT_SPELL_EFFECT )
    call TriggerAddCondition( gg_trg_snapdragonpoison, Condition( function Trig_snapdragonpoison_Conditions ) )
    call TriggerAddAction( gg_trg_snapdragonpoison, function Trig_snapdragonpoison_Actions )
endfunction

I even made it more efficent for you, do note though that this takes into account that you used only normal WE since you did not state you used any other editor.

 

[Preskooldude]

Aye, but considering the

globals
    unit sdpTarget
    real sdpDamage
endglobals

And that he did not say anything about WE crashing, I think you can safely assume that vJASS is enabled.

 

[Magni The Ironhammer]

@Dsg ty mr dsg btw if you know,wat was going wrong?


@Preskooldude yup im using vjass....well its enabled

 

[Dr Super Good]

I have little undestanding of Vjass, I only know standard JASS so thats why I can insure you that my way would work.

 

[PurplePoot]

GetSpellAbilityUnit, as far as I know, doesn't save through waits. Try GetTriggerUnit(), or storing it in a local.

Oh, also, use DSG's current code (with the fix, of course); it's MUI because it has no globals.

 

[Magni The Ironhammer]

Ok

edit:it wasnt meant to be GUI