E.D.T (v1.0)

[Moderator]

19:35, 28th Jan 2013
Maker: We already have an approved GUI tree detection system
Link

 

[Wrda]

Don't use this way of destroying a unit group

• Custom script: set bj_wantDestroyGroup = true

if you have another via variable:

• Set FS_Group = (Units within FS_AoE[FS_Level] of FS_Loc matching (((Matching unit) belongs to an enemy of (Owner of FS_Caster)) Equal to True))
• Unit Group - Pick every unit in FS_Group and do (Actions)
  • Loop - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • ((Picked unit) is Magic Immune) Equal to False
        • ((Picked unit) is A flying unit) Equal to False
        • ((Picked unit) is alive) Equal to True
      • Then - Actions
        • -------- Punishing the terrestrial unit in the affected area for this spell: --------
        • Unit - Cause FS_Caster to damage (Picked unit), dealing FS_Damage[FS_Level] damage of attack type FS_Type_Attack and damage type FS_Type_Damage
      • Else - Actions
        • -------- In case the conditions above are false, in other terms, the unit in this area is immune the magic, it is flying or it is dead, it is not necessary to punish these. --------

Because you still leak the group in variable, you have 2 ways:

• Custom script: set bj_wantDestroyGroup = true
• Unit Group - Pick every unit in "units in range matching condition" and do (Actions)

or

• Set FS_Group = (Units within FS_AoE[FS_Level] of FS_Loc matching (((Matching unit) belongs to an enemy of (Owner of FS_Caster)) Equal to True))
• Unit Group - Pick every unit in FS_Group and do (Actions)
• Custom script: call DestroyGroup(udg_FS_Group)

And maybe you could create a variable and store "picked unit", so the trigger won't be full of function calls

 

[Losam]

Don't use this way of destroying a unit group

• Custom script: set bj_wantDestroyGroup = true

if you have another via variable:

• Set FS_Group = (Units within FS_AoE[FS_Level] of FS_Loc matching (((Matching unit) belongs to an enemy of (Owner of FS_Caster)) Equal to True))
• Unit Group - Pick every unit in FS_Group and do (Actions)
  • Loop - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • ((Picked unit) is Magic Immune) Equal to False
        • ((Picked unit) is A flying unit) Equal to False
        • ((Picked unit) is alive) Equal to True
      • Then - Actions
        • -------- Punishing the terrestrial unit in the affected area for this spell: --------
        • Unit - Cause FS_Caster to damage (Picked unit), dealing FS_Damage[FS_Level] damage of attack type FS_Type_Attack and damage type FS_Type_Damage
      • Else - Actions
        • -------- In case the conditions above are false, in other terms, the unit in this area is immune the magic, it is flying or it is dead, it is not necessary to punish these. --------

Because you still leak the group in variable, you have 2 ways:

• Custom script: set bj_wantDestroyGroup = true
• Unit Group - Pick every unit in "units in range matching condition" and do (Actions)

or

• Set FS_Group = (Units within FS_AoE[FS_Level] of FS_Loc matching (((Matching unit) belongs to an enemy of (Owner of FS_Caster)) Equal to True))
• Unit Group - Pick every unit in FS_Group and do (Actions)
• Custom script: call DestroyGroup(udg_FS_Group)

And maybe you could create a variable and store "picked unit", so the trigger won't be full of function calls

My dear,
Do not get me wrong, but the codes used are correct. Still, thanks for commenting'll check this with a moderator.

 

[Chaosy]

Losam is right you may use it that way.
this is what the group is in jass

call ForGroupBJ( udg_group, function Trig_Untitled_Trigger_001_Func001A )

as you see bellow it should work just fine to set it to true

function ForGroupBJ takes group whichGroup, code callback returns nothing
    // If the user wants the group destroyed, remember that fact and clear
    // the flag, in case it is used again in the callback.
    local boolean wantDestroy = bj_wantDestroyGroup
    set bj_wantDestroyGroup = false

    call ForGroup(whichGroup, callback)

    // If the user wants the group destroyed, do so now.
    if (wantDestroy) then
        call DestroyGroup(whichGroup)
    endif
endfunction

 

[Wrda]

Ugh, didn't know that far. Thanks for telling me.

 

[Almia]

You dont need a dummy to destroy trees,all you have to do is create a peasant,hide it and make it invulnerable(or perhaps add locust ability)

Special effect models or string paths should be added to the configurable trigger.

If you are going to cache the group,you have to destroy it yourself.Not using bj_wantDestroyGroup

 

[doomhammer99]

This is how we Detect trees.

View this Losam

• Custom script: set udg_FS_Dummy = CreateUnit(Player(15), 'hpea', 0, 0, 0)
• Custom script: call UnitAddAbility(udg_FS_Dummy, 'Aloc')
• Custom script: call ShowUnit(udg_FS_Dummy, false)

Cache this "Order"

• Custom script: set udg_FS_Order = "harvest"

 

[Almia]

@doomhammer
its ok,he even write it in right

 

[doomhammer99]

@Almia - Oh okie.