https://github.com/peacefulcraft-network/Tarje/blob/master/src/net/peacefulcraft/tarje/Tarje.java#L141

Tries to close all open shops, but there are not configured shops to check for inventory views on which causes an error.

Right now we just .toString() Material values. We should add some logic to make those messages look a bit nicer.

• .toLowerCase()
• .replaceAll("-", " ")
• add an 's' for plurality when necessary.

This would probably be best done with a utility method.

It looks like the path might be missing a /.

It's still possible to trick the plugin into caring about containers it shouldn't by renaming, but these transactions are rejected by the underlying shop. The listener will still cancel events it should not, but because of where the views are tracked, there is not a good way to prevent the interact cancels. Adding an activeViews getter to the Shop classes would work, but is convoluted.

Proposed Solution

• Migrate InventoryView tracking to a ViewManager' that stores all active views, indexed by a ViewIndex`.
• Create ViewIndex which takes the Player and a differentiating String for storing / retrieving / removing from ViewManager.

[For /tarjeadmin reload] we should save which inventories were open and for what players, then attempt to re-open the player to those inventories after the reload completes.

/buy menu doesn't let users remove the category items, but it lets them put things in it. The inventory is static across the server so it sort of works like a communal enderchest. It should not let anyone transfer items into it.

Right now the order in which the shops get placed into the Index Shop is arbitrary. There should be a way to specify in a shops .yml config file what slot it should take in the index shop.

At the time of initial development, I could not found a reliable way to determine when a user was moving an item into the sell inventory. The idea was to do the purchasable check when they moved them item into the inventory so we could just cancel the event there rather than waiting until they closed the inventory and having to move it back.

The following should be considered a valid configuration and slot 1 would be left empty. Currently, the config processor loads the item in slot 0, doesn't see a slot one, and stops. The items in slots 2 and 3 are never loaded.

'0':
  item: CHICKEN
  buy: 40.0
  sell: 10.0
'2':
  item: RABBIT
  buy: 40.0
  sell: 10.0
'3':
  item: MUTTON
  buy: 40.0
  sell: 10.0

/buy [material] [quantity].

/sell hand Sells item(s) in main hand if sellable.

Integrate with LuckPerms to allow for permission purchasing so we can roll this out on Trench and Minigames to replace the Skript-powered GUIShop we have right now.

Any integer prices (IE 4 rather than 4.0) don't get read in and the related actions are disabled (buy/sell) as a result. The config loader should be able to read in double or integer values.

• Java 11 Build works
• Update pipeline
• Application testing