Error in code

[rodead]

whats wrong with this trigger :S
it says it is missing act
and canot convert integer to player...

scope Iceclap initializer init

private function rodead takes nothing returns boolean
    return GetSpellAbilityId() == 'A005'
endfunction

private function rodead2 takes nothing returns nothing 
    local integer id = GetPlayerId(GetOwningPlayer(GetTriggerUnit()))
    local unit caster = GetTriggerUnit()
    local real dx = GetUnitX(caster)
    local real dy = GetUnitY(caster)
    local unit dummy = CreateUnit(id, 'h004', dx, dy, 0.00)
    local location loc 
    
    set loc = Location (dx,dy)
    call IssuePointOrder(dummy,"blizzard",GetLocationX(loc),GetLocationY(loc))
    call UnitApplyTimedLife(dummy, 'BTLF', 5.00)
    call RemoveLocation(loc)
    
set caster = null
set loc = null
set dummy = null


endfunction


    

//===========================================================================
private function init takes nothing returns nothing
    local trigger t = CreateTrigger()
    call TriggerRegisterAnyUnitEventBJ(t, EVENT_PLAYER_UNIT_SPELL_EFFECT )
    call TriggerAddAction(t, function rodead2 )
    call TriggerAddCondition(t, Condition(function rodead))
endfunction

endscope

thanks in advanced and +rep

 

[trb92]

>local unit dummy = CreateUnit(id, 'h004', dx, dy, 0.00)
This takes a player first, not an integer. Change

local integer id = GetPlayerId(GetOwningPlayer(GetTriggerUnit()))

to

local player id = GetOwningPlayer(GetTriggerUnit())

And it should work.