[Solved] A minor problem

[CrueIIntentions]

I have this script:

JASS

function Trig_Max10_Conditions takes nothing returns boolean
    return GetItemType(GetManipulatedItem()) == ITEM_TYPE_CHARGED
endfunction

function Trig_Max10_Actions takes nothing returns nothing
    local integer index = 1
    local item charge = GetManipulatedItem()
    local unit u = GetTriggerUnit()
    loop
        exitwhen index > 6
        if ( GetItemTypeId( UnitItemInSlot(u, index))) == GetItemTypeId(charge) and UnitItemInSlot(u, index) != charge then
            if ( GetItemCharges(charge) + GetItemCharges(UnitItemInSlot(u, index))) <= 6 then
                if ( GetItemCharges(UnitItemInSlot(u, index)) < 6 ) then
                    call SetItemCharges( UnitItemInSlot(u, index), ( GetItemCharges(UnitItemInSlot(u, index)) + GetItemCharges(charge) ) )
                    call RemoveItem( GetManipulatedItem() )
                endif
            endif
        endif
        set index = index + 1
    endloop
endfunction

I had a couple problems testing this. when I picked up multiple items, item slot 1 only carried 1 charge while the others cariried a full 6.

Problem 2 : When item slots are full, I can only put one on the open slot. I dont understand why. I think perhaps because the inventory is full. But I cannot say. Is there any way I can get around this one?

Thanks in advance.

 

[Raven0]

pretty sure item slot indexes start at 0 and end at 5

 

[CrueIIntentions]

okay, that helps me with that part. But I still can only get one stackable item in slot 6 when inventory is full. Any ideas?


+rep for the help ty

 

[Adiktuz]

if you have full inventory, you won't be able to pick up anymore items using that trigger...

and I don't understand this:

When item slots are full, I can only put one on the open slot

-->if slots are full, how can you still have an open slot?

 

[dardas]

what he means, that for example, he got 6 inventory slots,
and he tries picking a item which was gonna be stacked into another,
and it doesnt work, since inventory is full, he has to drop to 5 inventory slots,
then pick up item, then get the item he just dropped, so he got 6 again, and so on for every item he want to add to the stack, i guess using tomes is a bad idea, since every item as a tome would be a nightmare ^^

 

[Adiktuz]

ahhh... basically to do that, you need to make every item have a power-up/tome version so that you can pick them up, then add the real item via triggers...

 

[CrueIIntentions]

How would you create the condition though, in case you didnt have the item?

 

[Adiktuz]

if you don't have the item, when you try to add it, it will drop automatically... just be sure that for the stack trigger, you use the event, A unit acquires an item...

I think you're thinking that what I said was stack up the item directly upon pick-up of the tome, which is false, because what you will do is add the item to the unit when you pick up the tome, so now, if the item is charged, it will fire your stacking trigger, so if you already have that kind of item, it would stack, if not, it will occupy a space (if there is any, else it will drop)

 

[CrueIIntentions]

would this prove true even for purchasing items?

 

[Adiktuz]

yup, as long as all items that you could pick-up or buy are powerups...