search

flokol120 / Spigot-Item-Chest-Sorter

A Spigot/Paper minecraft plugin to sort your messy items into separate chests | 💯% written in Kotlin
GNU General Public License v3.0
5 stars 6 forks source link

Plugin claims there are no receiver chests when there actually are #1

Closed EchocallingEarth closed 4 years ago

EchocallingEarth commented 4 years ago

When using the plugin the chat is spammed by a Message if a block is in the sender chest that is not in a receiver chest.

Msg

"There are no receivers configured yet. Use the /ics add receiver command"

This message repeats for the number of items in a chest that have no receiver. How to disable or fix this, please.

Once done I'll leave a review as other than this issue this plugin is perfect for my server's needs. But if I left a review now it would be less then favorable and I will not do that ever.

There appears to be no console log to provide but I can provide screenshots etc as needed if it helps.

Thanks in advance.

flokol120 commented 4 years ago

This is definitely a bug, @EchocallingEarth! Thank you for reporting it directly here on GitHub!

The Problem - in your case - was assumingly that you did not define any 'air chest' (which is completely fine and totally not an error on your side). This way the plugin thought you haven't got any receivers whilst you only didn't have any 'air receiver'.

I fixed the issue by moving the if-branch displaying the message and checking beforehand if you have any 'air receivers' or not. Version 1.2.2 should be on the releases page in a minute. Please report back if this commit fixed your issue. If you are missing a feature, feel free to open a feature request.

EchocallingEarth commented 4 years ago

Working perfectly fine and thank you