What's wrong with this trigger?

[JFAMAP]

Since 2002 i am developing many maps and still i am active. But it is the first time i see something like this.

What is the difference between these 2 triggers? I have turned off all triggers for the testing purpose but what is wrong with this trigger...

Region 27 just a part of the map including tree i am harvesting.

 

[Uncle]

I assume the Entire Map is not technically a "region". Just place a Region over the entire map and use that.

 

[JFAMAP]

I assume the Entire Map is not technically a "region". Just place a Region over the entire map and use that.

Already did. Still not working. I am also tried to use "playable map area" but still not working.

 

[Nichilus]

There is a limit on how many destructibles are picked by the "A destructible within region dies" event on trigger creation. Don't know the exact number, though.

Edit: Just checked, the limit is 64 destructibles

 

[JFAMAP]

There is a limit on how many destructibles are picked by the "A destructible within region dies" event on trigger creation. Don't know the exact number, though.

Edit: Just checked, the limit is 64 destructibles

Wow wait. That makes a difference. My trigger was working well and today was added many trees on the map. That means i have to split the world into many regions and try not to add more than 64 destructibles within the region right?

Thank you by the way. It seems you found the problem.

 

[Dr Super Good]

Wow wait. That makes a difference. My trigger was working well and today was added many trees on the map. That means i have to split the world into many regions and try not to add more than 64 destructibles within the region right?

Or write your own custom script implementation of the event which does not have such a limit.

 

[JFAMAP]

Or write your own custom script implementation of the event which does not have such a limit.

Thank you for your help. I have already added 45 different regions at the event. It seems absurd but it works..

 

[Dr Super Good]

Thank you for your help. I have already added 45 different regions at the event. It seems absurd but it works.

Or instead of...

function TriggerRegisterDestDeathInRegionEvent takes trigger trig,rect r returns nothing
    set bj_destInRegionDiesTrig = trig
    set bj_destInRegionDiesCount = 0
    call EnumDestructablesInRect(r, null, function RegisterDestDeathInRegionEnum)
endfunction

function RegisterDestDeathInRegionEnum takes nothing returns nothing
    set bj_destInRegionDiesCount = bj_destInRegionDiesCount + 1
    if (bj_destInRegionDiesCount <= bj_MAX_DEST_IN_REGION_EVENTS) then
        call TriggerRegisterDeathEvent(bj_destInRegionDiesTrig, GetEnumDestructable())
    endif
endfunction

Use...

function TriggerRegisterDestDeathInRegionEventNoLimit takes trigger trig,rect r returns nothing
    set bj_destInRegionDiesTrig = trig
    call EnumDestructablesInRect(r, null, function RegisterDestDeathInRegionNoLimitEnum)
endfunction

function RegisterDestDeathInRegionNoLimitEnum takes nothing returns nothing
    call TriggerRegisterDeathEvent(bj_destInRegionDiesTrig, GetEnumDestructable())
endfunction

Which has the limit logic removed from it.

 

[JFAMAP]

Or instead of...

function TriggerRegisterDestDeathInRegionEvent takes trigger trig,rect r returns nothing
    set bj_destInRegionDiesTrig = trig
    set bj_destInRegionDiesCount = 0
    call EnumDestructablesInRect(r, null, function RegisterDestDeathInRegionEnum)
endfunction

function RegisterDestDeathInRegionEnum takes nothing returns nothing
    set bj_destInRegionDiesCount = bj_destInRegionDiesCount + 1
    if (bj_destInRegionDiesCount <= bj_MAX_DEST_IN_REGION_EVENTS) then
        call TriggerRegisterDeathEvent(bj_destInRegionDiesTrig, GetEnumDestructable())
    endif
endfunction

Use...

function TriggerRegisterDestDeathInRegionEventNoLimit takes trigger trig,rect r returns nothing
    set bj_destInRegionDiesTrig = trig
    call EnumDestructablesInRect(r, null, function RegisterDestDeathInRegionNoLimitEnum)
endfunction

function RegisterDestDeathInRegionNoLimitEnum takes nothing returns nothing
    call TriggerRegisterDeathEvent(bj_destInRegionDiesTrig, GetEnumDestructable())
endfunction

Which has the limit logic removed from it.

Thank you so much for this but sadly i don't know jass system i don't know how to implement this into my map. Error appears when i try to put it into my map.

 

[Dr Super Good]

Error appears when i try to put it into my map.

That is because you are copying the function that already exists as part of the game to implement the GUI event. I only placed that there for reference so you could see how it was implemented (no black box).

 

[JFAMAP]

That is because you are copying the function that already exists as part of the game to implement the GUI event. I only placed that there for reference so you could see how it was implemented (no black box).

Another words i don't know what to do with this trigger. In the other hand i don't want to force you for anything. I just copied it into my map.
Edit: Somehow still my trigger not working properly. Region includes 33 destructibles but my trigger cannot read some of the trees. I have to go sleep now. Demoralized & tired.

 

[ThompZon]

Another words i don't know what to do with this trigger. In the other hand i don't want to force you for anything. I just copied it into my map.
Edit: Somehow still my trigger not working properly. Region includes 33 destructibles but my trigger cannot read some of the trees. I have to go sleep now. Demoralized & tired.

It's easy to call a GUI trigger from jass, meaning that it's possible to use @Dr Super Good "trick" and have the "main trigger" in GUI. You only have to put a tiny amount of jass in the map.

 

[JFAMAP]

It's easy to call a GUI trigger from jass, meaning that it's possible to use @Dr Super Good "trick" and have the "main trigger" in GUI. You only have to put a tiny amount of jass in the map.

I have created 2 triggers, converted these triggers to custom text, and copied 2 jass triggers within these triggers. Here is the screenshots.
I already did and it gave me error like this. I believe he assumed i know how to work with jass triggers because of my 18 years old map developer carrier but i am not smart to understand this language. And as i said, nobody have to help me. You already helped me a lot. If i will gonna ask more about this Jass trigger, probably you will gonna waste your time with me.

 

[Dr Super Good]

I already did and it gave me error like this.

That is the same error which I explained before.

That is because you are copying the function that already exists as part of the game to implement the GUI event. I only placed that there for reference so you could see how it was implemented (no black box).

You want to copy the other set of functions. You can then call it in an initialization trigger to add the event to another trigger. Turn triggers to custom script to see the trigger variable name they are using, that is the trigger reference you need to pass the function. You can also use this approach to lookup the name of the rect you want to use and pass the function.

 

[JFAMAP]

That is the same error which I explained before.

You want to copy the other set of functions.

No no i think you assumed i have a knowledge about that. I "literally" don't know what to do. I am a bit of unintelligent person.