Undeclared Variable in JASS

[DiggetyBo]

Hey guys,

This probably will just take a second, but I've spent over an hour trouble shooting this JASS compile error. I'm using Anachron's inventory system, and all I want this to do is to disable one of my triggers called "AncientOre" as the item is dropped, then enable it again after Anachron's system completes the drop action.

Have a look:

library CIEvents
endlibrary

module CIEvents
    public method pick takes CustomItem ci returns boolean
        if .checkPickup(ci) and IsUnitType(.carrier,UNIT_TYPE_STRUCTURE)== false and ( GetItemTypeId(ci.getHandle()) != 'I00G' ) then
            call ci.pick(.carrier)
            call .addItem(ci)
            return true
        endif
        
        return false
    endmethod
    
    public method drop takes CustomItem ci returns boolean
        if .hasItem(ci) then
            call DisableTrigger(AncientOre)
            call ci.drop(.carrier)
            return .remItem(ci)
            call EnableTrigger(AncientOre)
        endif
        
        return false
    endmethod
    
    public method pawn takes CustomItem ci returns boolean
        if .hasItem(ci) then
            call ci.pawn(.carrier)
            return true
        endif
        
        return false
    endmethod
    
    public method use takes CustomItem ci returns boolean
        if .hasItem(ci) then
            call ci.use(.carrier)
            return true
        endif
        
        return false
    endmethod
endmodule

The error says that AncientOre is an undeclare variable. But it's a real, living breathing trigger in my map.

I spelled it right and everything. And if spelling is not the issue, where am I suppose to declare this variable? I tried every place under the sun.

thanks for looking

 

[IcemanBo]

After you do this: return .remItem(ci) it won't enable the trigger again.

You directly endlibrary after you create it?

where am I suppose to declare this variable

What variable exactly?

 

[DiggetyBo]

Yeah, that's the confusing part. I didn't think it was a variable at all.

AncientOre is just a trigger in my map, i want to disable it during the drop item process because if its active during this process it will not function correctly.

I've never had to disable a trigger in JASS before, I didn't think it'd be this hard.

Is it necessary to declare a trigger as a variable in this instance?

 

[IcemanBo]

Do you have a global trigger variable named "AncientOre"?

 

[Geshishouhu]

Have you tried changing this call EnableTrigger(AncientOre) to this call EnableTrigger(gg_trg_AncientOre) ? Also, call DisableTrigger(gg_trg_AncientOre)
As IcemanBo pointed out, you cannot enable the trigger after the "return".

 

[DiggetyBo]

Hey, sry for the wait

I tried, closest I got was this

This debugs the static ifs */
library CIEvents
    globals
        private trigger AncientOre
endlibrary

said it was an error, nested global.

but, that's where the other globals are when I looked at the other triggers he wrote. I must be missing something obvious.

 

[Geshishouhu]

Hey, sry for the wait

I tried, closest I got was this

This debugs the static ifs */
library CIEvents
    globals
        private trigger AncientOre
endlibrary

said it was an error, nested global.

but, that's where the other globals are when I looked at the other triggers he wrote. I must be missing something obvious.

Can you refer to my preview post to see if that's the problem?

Edit: you are also missing one line in your global declaration

library CIEvents
    globals
        private trigger AncientOre
    endglobals //you missed this
endlibrary

But I suggest you delete this global declaration first and try my method in the preview post. If that does not solve your problem, then try to declare the global variable.

 

[IcemanBo]

If you declare your trigger private in your library, you can not turn it on/off outside the library.