Another Question..

[Exide]

Does this work:

function Trig_Crabs_Killed_Conditions takes nothing returns boolean
    return GetUnitTypeId(GetDyingUnit()) == 'n003'
    return Trig_Crabs_Killed_Func006C()
endfunction

Does the trigger run both conditions or just the first one?

 

[~GaLs~]

if i am not mistaken...it is the first 1...and ignoring the 2nd 1

 

[Chocobo]

if i am not mistaken...it is the first 1...and ignoring the 2nd 1

True. The return acts as a Skip Remaining Action.

 

[Rheias]

Do the action "Skip Remaining Actions" in GUI, convert to Jass, learn something new.

Anyway, yeah, when the code sees the word "return" it will return whatever after it and stop running the function.

What you need is:

function Trig_Crabs_Killed_Conditions takes nothing returns boolean
    return GetUnitTypeId(GetDyingUnit()) == 'n003' and Trig_Crabs_Killed_Func006C()
endfunction

 

[Exide]

I just realized why this isn't working.

function Trig_Crabs_Killed_Func006C takes nothing returns boolean
    local integer id = GetPlayerId(GetOwningPlayer(GetKillingUnit()))
    return id<5
endfunction

Used to say: GetTriggerUnit, when the event was 'Unit - a Unit Dies'.
My bad. It's working now, thanks.

 

[substance]

GetTriggerUnit() is the dying unit from that EVENT.

 

[Exide]

Yea, but if you read my code you'll see that I replaced TriggerUnit with KillingUnit..