[JASS] "Expected a Name"

[busterkomo]

I always seem to attain the error "Expected a name" by WE as a try to save my map .

function Trig_Upgrade_Item_Func001Func003001 takes nothing returns boolean
    return ( udg_ItemY > 10 )
endfunction

function Trig_Upgrade_Item_Func001C takes nothing returns boolean
return( GetItemTypeId(UnitItemInSlot(udg_UpgradeWeaponHero, 1))==udg_weapons[udg_ItemX])
endfunction

function Trig_Upgrade_Item_Actions takes nothing returns nothing
    if ( Trig_Upgrade_Item_Func001C() ) then
        call DisplayTextToForce( GetForceOfPlayer(GetOwningPlayer(GetTriggerUnit())), ( "|cffffff00You have upgraded your weapon to level " + ( I2S(udg_ItemY) + "|r" ) ) )
        call DestroyEffect(AddSpecialEffectLoc("Abilities\\Spells\\Items\\TomeOfRetraining\\TomeOfRetrainingCaster.mdl",GetUnitLoc(udg_UpgradeWeaponHero)))
        call RemoveItem( UnitItemInSlotBJ(udg_UpgradeWeaponHero, 1) )
        call UnitAddItemById( udg_UpgradeWeaponHero, udg_Items[(udg_ItemX + 1)] )
    else
        if ( Trig_Upgrade_Item_Func001Func003001() ) then
            set udg_ItemY = 1
        endif
        set udg_ItemX = ( udg_ItemX + 1 )
        set udg_ItemY = ( udg_ItemY + 1 )
        call TriggerExecute( gg_trg_Upgrade_Item )
    endif
endfunction

//===========================================================================
function InitTrig_Upgrade_Item takes nothing returns nothing
    set gg_trg_Upgrade_Item = CreateTrigger(  )
    call TriggerAddAction( gg_trg_Upgrade_Item, function Trig_Upgrade_Item_Actions )
endfunction

call UnitAddItemById( udg_UpgradeWeaponHero, udg_Items[(udg_ItemX + 1)] )

Is the line with the error
Note: I know this hasn't been optimized in the least bit, I like to have all my triggers working before optimization.

 

[PurplePoot]

One of those 3 variables has not been initialized, apparently?

And I think you'll find your triggers much easier to get working if they're not hard to read, converted GUI

 

[busterkomo]

One of those 3 variables has not been initialized, apparently?

And I think you'll find your triggers much easier to get working if they're not hard to read, converted GUI

I said I haven't bothered optimizing it....>>>GUI

GetItemTypeId isn't in GUI....
Well, it is but can't be used the way I did.

 

[PurplePoot]

[jass=It compiles fine for me with these globals]globals
unit udg_UpgradeWeaponHero
trigger gg_trg_Upgrade_Item
integer udg_ItemX
integer udg_ItemY
integer array udg_Items
integer array udg_weapons
endglobals[/code]

 

[busterkomo]

All good, The variable was actually "items" not "Items".
God I feel so dumb