[General] Ivalid argument (integer)

[Rybak]

Heloes Hive Workshop

I've encountered a slight problem with this line in my function:

call UnitAddAbilityBJ ('A016', GetLastCreatedUnit())

the editor says that it's 'Invalid argument (integer)'. I searched some web, nothing seems to be the problem. What's more, when I made same thing in GUI and converted it afterwards, there was no problem in converted trigger. After copying it to my own JASS trigger, despite looking the same, the same error would occur, as it prevously did. To make it anyhow more clear I'll post the rest of the trigger's part.

call CreateNUnitsAtLoc(1, 'H002', GetPlayerId(GetOwningPlayer(caster)), GetUnitLoc(caster), bj_UNIT_FACING) 
    call UnitAddAbilityBJ ('A016', GetLastCreatedUnit())
    call IssueTargetOrderBJ (GetLastCreatedUnit(), "thunderbolt", cel)
    call RemoveUnit (GetLastCreatedUnit())

Any ideas, maybe?

 

[deathismyfriend]

Check to see if they r backwards. Also u shouldn't use bj's if u r coding in jass. Change them over to natives.

Note: to get rid of bjs look for a tutorial on inlining. Sry can't look for u not at pc

 

[Rybak]

Yup, I know about the BJ's. Swapping doesn't work, I get the same error. It is kinda weird though, as, just like I said, the same trigger converted from GUI works.

 

[deathismyfriend]

Try reloading ur world editor. I had a couple odd bugs tht cleared up after restarting the editor.

 

[Rybak]

Tried that too, unfortunately to no avail.

 

[Arhowk]

Sometimes JASShelper can throw unreal arguments if theres an unclosed if-then-else, loop, function, struct, library, etcetc. Make sure all of them are closed. For example

function a takes nothing returns nothing
if true then
call UnitAddAbilityBJ(0,unit)
endfunction

 

[Rybak]

Well, it is inside of a loop, but the loop is closed.
Maybe I'll post the whole if/then/else function

if (LoadIntegerBJ(1, GetPlayerId(GetOwningPlayer(caster)) +1, udg_Tablica_Manipulator) == 1) then
    set efekt = "Abilities\\Spells\\Other\\Incinerate\\FireLordDeathExplode.mdl"
    set mnoznik = 2.4
    set ta = ATTACK_TYPE_MAGIC
    set to = DAMAGE_TYPE_FIRE
    elseif (LoadIntegerBJ(1, GetPlayerId(GetOwningPlayer(caster)) +1, udg_Tablica_Manipulator) == 2) then
    set efekt = "Abilities\\Spells\\Undead\\FrostNova\\FrostNovaTarget.mdl"
    set mnoznik = 1.8
    set ta = ATTACK_TYPE_MAGIC
    set to = DAMAGE_TYPE_COLD
    call CreateNUnitsAtLoc(1, 'h002', GetPlayerId(GetOwningPlayer(caster)), GetUnitLoc(caster), 0.00) 
    call UnitAddAbility( GetLastCreatedUnit(), 'A01L' )
    call IssueTargetOrderBJ (GetLastCreatedUnit(), "frostnova", cel)
    call RemoveUnit (GetLastCreatedUnit())
    elseif (LoadIntegerBJ(1, GetPlayerId(GetOwningPlayer(caster)) +1, udg_Tablica_Manipulator) == 3) then
    set efekt = "Abilities\\Spells\\Orc\\WarStomp\\WarStompCaster.mdl"
    set mnoznik = 1.5
    set ta = ATTACK_TYPE_SIEGE
    set to = DAMAGE_TYPE_NORMAL
    call CreateNUnitsAtLoc(1, 'H002', GetPlayerId(GetOwningPlayer(caster)), GetUnitLoc(caster), 0.00) 
    call UnitAddAbility (GetLastCreatedUnit(), 'A016')
    call IssueTargetOrderBJ (GetLastCreatedUnit(), "thunderbolt", cel)
    call RemoveUnit (GetLastCreatedUnit())
    elseif (LoadIntegerBJ(1, GetPlayerId(GetOwningPlayer(caster)) +1, udg_Tablica_Manipulator) == 4) then
    set efekt = "Abilities\\Spells\\Other\\Monsoon\\MonsoonBoltTarget.mdl"
    set mnoznik = 2.0
    set ta = ATTACK_TYPE_MAGIC
    set to = DAMAGE_TYPE_LIGHTNING
    elseif (LoadIntegerBJ(1, GetPlayerId(GetOwningPlayer(caster)) +1, udg_Tablica_Manipulator) == 5) then
    set efekt = "Abilities\\Spells\\Human\\HolyBolt\\HolyBoltSpecialArt.mdl"
    set mnoznik = 2.5
    set ta = ATTACK_TYPE_MAGIC
    set to = DAMAGE_TYPE_DIVINE
    else
    set efekt = "Abilities\\Spells\\Undead\\DeathCoil\\DeathCoilSpecialArt.mdl"
    set mnoznik = 2.5
    set ta = ATTACK_TYPE_PIERCE
    set to = DAMAGE_TYPE_NORMAL
    endif

mnoznik means % damage for the skill, I also set attack types and special effects taking place when casting certain spell. When the given hashtable value is 2 or 3, then the spell also causes additiona effects. Whithout them, it all works. Also, as I said prvously, the same function in GUI, when converted to JASS, worked perfectly.

 

[deathismyfriend]

can u paste the full trigger ? also u should use a local integer variable for GetPlayerId(GetOwningPlayer(caster)) +1 since u call it so many times. also make a unit variable and do set unit = createunit( stuff in here). locations leak if not removed and since ur using jass use the x y values tht way u dont have to null or destroy them.( no leaks)

 

[Rybak]

Well, after you said all this I looked at the trigger and saw what's the problem - instead of Player data I inserted player number in place, where I say what player will get the dummy.
Thanks anyawy