Action RPG Inventory System

[Pirate]

@Fred.Horgan;480825

First off, great asset. Inventory is incredibly well commented and organised.

I’ve been playing around with adding some additional functionality (on equip/unequip effects) for our game and discovered that UnequipItem is not reliably called. It is called when an item is dragged from the character into the backpack but not when it is unequipped via right click. It also doesn’t run the unequip function when it is placed into a container via right click, again it works fine if dragged.

Good find, I can see that having the simpler logic path being an issue for people looking for a place to tag onto Unequip I will make the changes and post a solution ASAP. The fix will be included in the 1.4 update.

@Fred.Horgan;480825

The second doesn’t seem to be quite so straightforward, the issue is in UseInventoryItem when a container is open. It skips the ItemType checking and just moves the item directly into the open container rather than checking to see if it is Equipment and instead run the UseEquipmentItem function. For now I’ve just disabled placing from character to container and instead it runs the default UseEquipmentItem to place it back into the backpack whether a container is open or not.

I didn’t see this on the list of upcoming fixes so I thought it would be best to mention it

This is by design, when right clicking items when a container is open it overrides the normal logic and will swap the item between the inventory and container. If you look at most MMO’s imagine having your bank storage open. When you right click to use an item it moves the items to and from a container. For example in WoW your backpack is open and your Bank is open. If you right click on a potion it won’t consume the potion it will move it to the Bank or Backpack (depending on where it is located) same goes for equipment. Please feel free to change this logic if it is not what you want in your game