carmack78/mooege — comment on commit 4f74705be31e9a69781fd3637635417607ecb0b0
File:  line 0
By TeoMan:

Because you are saving the "VisualEquipment", not the full equipment.

The Visualequipment is used to show player in the selection screen and it only has 8 slots avaiable for sending data from client to server.
i you use the internel Equipment class (inside Inventory class) you have all 13 slots avaiable, but you have to export VisualEquipment as a subset.

As suggesed by Egris, it is better to save Inventory together with Equipment outside Toon class, maybe in Player class, and create VisualEquipment array only when requested.
I tried, but I failed in showing equipment in selection screen, because Toon class is created without Player class.
Maybe moving Inventory class inside Toon, so it can be loaded together with toon, but Attributes for items are taken from Player class that is not present yet, Player class is created once the game starts.

Maybe loading visual equipment from DB if Player not present, otherwise get Equipment from Inventory class once loaded.

Well... it depends on mantainers of the main branch... if we use the wrong path, they will not merge the pull request. Better to ask them the correct way to follow.

(sorry for my english... I'm italian)