IKilledKEnny's JASS Question Theard, I love to give +REP :)

[IKilledKEnny]

function DestroyCirclesGAtions takes nothing returns nothing
call RemoveUnit(GetEnumUnit())
endfunction

function DestroyCirclesGConditions takes nothing returns boolean
if GetPlayerSlotState(GetOwningPlayer(GetFilterUnit()))!=PLAYER_SLOT_STATE_PLAYING then
return true
endif
return false
endfunction

function DestroyCirclesAct takes nothing returns nothing
local boolexpr b=Condition(function DestroyCirclesGConditions)
local group g=GroupEnumUnitsInRect(g,GetWorldBounds(),b)
ForGrouo(g,function DestroyCirclesGAtions)
call DestroyGroup(g)
call DestroyBoolExpr(b)
set b=null
set g=null
endfunction

//===========================================================================
function InitTrig_Destroy_Circles takes nothing returns nothing
set gg_trg_Destroy_Circles = CreateTrigger()
call TriggerRegisterTimerEventSingle(gg_trg_Destroy_Circles,1.00)
call TriggerAddAction(gg_trg_Destroy_Circles,function DestroyCirclesAct)
endfunction

Still disabled. GAtion is Group Action just that I forgot the c . And how could I make the condition simpler?

 

[Doom-Angel]

ForGrouo

AceHart already told u.....
u need to fix that and put in ForGroup not ForGrouo.

and also:

local group g=GroupEnumUnitsInRect(g,GetWorldBounds(),b)

shouldn't u first Create Group and then Group the units inside the group?

 

[Sooda]

function DestroyCirclesGConditions takes nothing returns boolean
if GetPlayerSlotState(GetOwningPlayer(GetFilterUnit()))!=PLAYER_SLOT_STATE_PLAYING then
return true
endif
return false
endfunction

can be:

function DestroyCirclesGConditions takes nothing returns boolean
     return GetPlayerSlotState(GetOwningPlayer(GetFilterUnit())) != PLAYER_SLOT_STATE_PLAYING 
endfunction

 

[IKilledKEnny]

Thanks, both of you, I did what you both said plus few minor corrections in it worked like a charm.

Edit:
Sooda, right of course, I'm used to do that only with spell conditions, heh.

 

[phyrex1an]

Splitted last question: http://www.thehelper.net/forums/showthread.php?t=55139

On a unrelated note, why don't you make a new thread for a new question (and use a descriptive thread title)? It's extreamly hard to know what the current question is and if it has been answered in this thread.

 

[IKilledKEnny]

I keep the first post updated.

 

[phyrex1an]

Just post new threads. With descriptive titles, thank you. I don't like to edit the thread title all the time.

Thread closed, *phyrex1an hopes this is the last 'question thread' ever*