1. The first one is again the " | " sign. I found it's necessary to write a line like this: PHWarehouse randomLoot item <Mediarack1:58|59|60|61|:10>
If the ID list is not closed with a " | ", the last item will be ignored (in this exemple, the 61).
If the ID list is not closed with a " | ", the last item will be ignored (in this exemple, the 61).
Ah, yes. I know how to fix that. That's just a mathematical oversight on my part and an easy fix.
Also not a direct bug, but important to notice if another user read this, having the ID list starting with the " | " sign, will create an invisible item that will be spawned inside the chest, and take space of the determined quantity possibility (ending up with a warehouse of 475/500 despite being empty. Happens on small warehouses too).
I will put a fix on this as well.
2. I installed the MrTS_LimitedInventory plugin because I saw compatibility made in previous patches. I tested it in various position in the plugin list, and tested with YEP_Item Core off.
The limit on MrTS plugin works, blocking the inventory from getting new items, but, a warehouse will never put a stop to withdraw.
It causes items to vanish when the player takes them from the warehouse.
The limit on MrTS plugin works, blocking the inventory from getting new items, but, a warehouse will never put a stop to withdraw.
It causes items to vanish when the player takes them from the warehouse.
Hmm...I'll have to see if there's a way for me to make this work, but no guarantees.
Apart from that I'd like to make 2 suggestions if you want to upgrade the plugin. It would be really nice to have an option allowing items to be displayed one by one instead of stacks (might be hard to code),
and it would be great to have an option removing the "All" when All text option is enable, Leaving only the Withdraw text and Deposit text (would remove some sort of redundancy).
and it would be great to have an option removing the "All" when All text option is enable, Leaving only the Withdraw text and Deposit text (would remove some sort of redundancy).
I would like to do these, but again, my RMMV-based JavaScript knowledge is a little limited. I'll see what I can do.




