Replacing Item Same Slot

[defskull]

How to actually replace (basically, I remove the item and create a new one) an item in the exact same slot after it was created ?

Situation:
I used an ability (via Item) and when it detects, it changes that item (Item A) to new item (Item B) and I want Item B to be moved in the exact location as Item A position in the inventory.

Basically I found this function:

• Unit - Order (Triggering unit) to move (Last created item) to inventory slot 1

But what Integer should I put there ?
Is there any function that GetLastRemovedItemSlot or something ?

EDIT:
This system is meant for more than 1 items on a single unit.
Meaning that a unit can possess Item A more than 1.

 

[ic3_water]

call UnitAddItemToSlotById( GetTriggerUnit(), 'texp', 1 )

 

[defskull]

That's just basically... what the GUI function above did.

I need to know what Integer should I put in order for me to replace the item with the same exact inventory slot as Item A.

Example:
You have Item A at slot 4, and when this item is activated, I will remove Item A and create Item B and will move Item B to the same slot as Item A (which in this case, is slot 4).

How do I know which slot is the last removed/etc item ?

 

[ic3_water]

That's just basically... what the GUI function above did.

I need to know what Integer should I put in order for me to replace the item with the same exact inventory slot as Item A.

Example:
You have Item A at slot 4, and when this item is activated, I will remove Item A and create Item B and will move Item B to the same slot as Item A (which in this case, is slot 4).

How do I know which slot is the last removed/etc item ?

GUI doesnt got that function
In gui, you need 2 actions to do that, 1. Create item. 2. Order unit move to A slot

• (Item-type of (Item carried by Unit in slot 1)) Equal to Tome of Experience // Run loop

 

[defskull]

I made the edit.

NOTE: This system allows you to have more than 1 same item.

I already know how to tackle the problem if the item is unique, meaning that you can only have 1 item of that type per unit.

But since it allows more than 1 same item per unit, you can't really use that.

Because if I have 2 Item A (slot 1 | slot 4)
The condition will evaluate slot 1 first before going to slot 4 (therefore the item replaced occurs at slot 1, while I activated it at slot 4)

Therefore, you can't really use that method/approach.

 

[ic3_water]

• (Item being manipulated) Equal to (Item carried by Unit in slot (Integer A))

 

[defskull]

That condition seems to solve the problem, yipeee
How can I miss that small condition...

Anyway, the function in post #2 really not in GUI, because I overlooked it, that function is 2 in 1, right ?
Create and Move, whereas if in GUI, that's a pain ass, 2 lines of actions.

Okay, here's the trigger code if you want to see it:

trigger

• CR Event
  • Events
    • Unit - A unit Uses an item
  • Conditions
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Item-type of (Item being manipulated)) Equal to RingOfHealth
      • Then - Actions
        • For each (Integer LoopingInteger) from 1 to 6, do (Actions)
          • Loop - Actions
            • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
              • If - Conditions
                • (Item being manipulated) Equal to (Item carried by (Triggering unit) in slot LoopingInteger)
              • Then - Actions
                • Item - Remove (Item being manipulated)
                • Custom script: call UnitAddItemToSlotById(GetTriggerUnit(), 'I000', udg_LoopingInteger)
                • Skip remaining actions
              • Else - Actions
      • Else - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (Item-type of (Item being manipulated)) Equal to RingOfMana
          • Then - Actions
            • For each (Integer LoopingInteger) from 1 to 6, do (Actions)
              • Loop - Actions
                • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                  • If - Conditions
                    • (Item being manipulated) Equal to (Item carried by (Triggering unit) in slot LoopingInteger)
                  • Then - Actions
                    • Item - Remove (Item being manipulated)
                    • Custom script: call UnitAddItemToSlotById(GetTriggerUnit(), 'I001', udg_LoopingInteger)
                    • Skip remaining actions
                  • Else - Actions
          • Else - Actions

 

[ic3_water]

That condition seems to solve the problem, yipeee
How can I miss that small condition...

Anyway, the function in post #2 really not in GUI, because I overlooked it, that function is 2 in 1, right ?
Create and Move, whereas if in GUI, that's a pain ass, 2 lines of actions.

Okay, here's the trigger code if you want to see it :

or you can use

• CR Event - EGUI
  • Events
    • Unit - A unit Uses an item
  • Conditions
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Item-type of (Item being manipulated)) Equal to RingOfHealth
      • Then - Actions
        • For each (Integer LoopingInteger) from 1 to 6, do (Actions)
          • Loop - Actions
            • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
              • If - Conditions
                • (Item being manipulated) Equal to (Item carried by (Triggering unit) in slot LoopingInteger)
              • Then - Actions
                • Item - Remove (Item being manipulated)
                • Hero - Give (Triggering unit) a Tome of Experience in slot LoopingInteger
                • Skip remaining actions
              • Else - Actions
      • Else - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (Item-type of (Item being manipulated)) Equal to RingOfMana
          • Then - Actions
            • For each (Integer LoopingInteger) from 1 to 6, do (Actions)
              • Loop - Actions
                • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                  • If - Conditions
                    • (Item being manipulated) Equal to (Item carried by (Triggering unit) in slot LoopingInteger)
                  • Then - Actions
                    • Item - Remove (Item being manipulated)
                    • Hero - Give (Triggering unit) a Tome of Experience in slot LoopingInteger
                    • Skip remaining actions
                  • Else - Actions
          • Else - Actions

 

[defskull]

It seems you're using EGUI (Enhanced GUI) to enable that action, sadly I don't use them.
It is bug-prone.

By using EGUI, you're basically adding a JASS function to that GUI, lol.

 

[ic3_water]

BTW dont use Skip remaining actions

• Skip remaining actions
• Set Unit = No unit

You cant clear the variable if the clear action after it.

 

[defskull]

There are basically no any other actions after that, relax.

It's a 2-way to choose.

Path A -> Loop -> Find Item -> End
Path B -> Loop -> Find Item -> End

This is a simple programming structure on how to end a program.

 

[ic3_water]

There are basically no any other actions after that, relax.

It's a 2-way to choose.

Path A -> Loop -> Find Item -> End
Path B -> Loop -> Find Item -> End

This is a simple programming structure on how to end a program.

GetTriggerUnit() <---- Leak

 

[defskull]

(Triggering unit) never leak before, where do you find this information saying that native is leak ?
If you are using local variable of unit, well, yes it will leak because we have to null it, but this is native function, it won't leak.

 

[ic3_water]

(Triggering unit) never leak before, where do you find this information saying that native is leak ?
If you are using local variable of unit, well, yes it will leak because we have to null it, but this is native function, it won't leak.

I got fool by people in native forum

 

[Hashjie]

I got fool by people in native forum

And where is this "native forum"?

@defskull

I'll try to search with you for usefull natives. Can't gaurantee you that I'll find it though :/

EDIT: when you remove an item from the unit can't you save the slot the item was in?
Hashtables perhaps?

EDIT2: Might this native be of any use?:

native UnitItemInSlot takes unit whichUnit, integer itemSlot returns item

EDIT3: How did you remove the item? with UnitRemoveItem? What about using UnitRemoveItemFromSlot?

EDIT4: lol I overlooked this thread big time, I see you've already solved it. I should really learn to read before posting...

 

[ic3_water]

And where is this "native forum"?

in my country

 

[defskull]

Hashjie >.>"