Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Farmland Moisture check showing incorrect results #8

Open
TheBusyBiscuit opened this issue Sep 25, 2020 · 2 comments
Open

Farmland Moisture check showing incorrect results #8

TheBusyBiscuit opened this issue Sep 25, 2020 · 2 comments
Labels
Bug Report A bug that needs to be fixed.

Comments

@TheBusyBiscuit
Copy link
Member

📍 Description (REQUIRED)

The Moisture Checker shows wrong results when using it on wet farmland, it seems to always give off the "yellow indicator", no matter if wet or dry.

This is the farmland:
before

This is what is shown after using a Moisture Checker:
after

As evidently shown, the Moisture Checker has a yellow indicator for both, wet and dry, farmland.

📑 Steps to reproduce the Issue (REQUIRED)

  1. Set up some farmland
  2. Place water next to it
  3. Wait for farmland to get wet
  4. Right click with a Moisture Checker

💡 Expected behavior (REQUIRED)

The Moisture Checker should show different levels of moisture depending on whether the farmland is wet or not.

📜 Server Log

No significant data in the logs

🧭 Environment (REQUIRED)

  • Server Software (Spigot/Paper): Paper (git-204)
  • Minecraft Version: 1.16.3
  • SensibleToolbox Version: DEV - 1 @ b0ab4a7
@TheBusyBiscuit TheBusyBiscuit added Bug Report A bug that needs to be fixed. Hacktoberfest This Issue or Pull Request is part of the Hacktoberfest! labels Sep 25, 2020
@wouterdedroog
Copy link
Contributor

wouterdedroog commented Oct 25, 2020

This issue doesn't seem to happen after watering it with a watering can (/stb give wateringcan). This happens because soil that doesn't get watered with a watering can doesn't have any metadata with the STB_Saturation metadata key. Is this a bug? @TheBusyBiscuit

@TheBusyBiscuit
Copy link
Member Author

@MrWouterNL Well, the saturation can be seen as a water "overflow" thing.
There is dry soil, wet soil and then there is a few levels of "saturated" soil.
It is correct that non-saturated soil has no metadata value assigned to it.
But I think the normal dry and wet variants should still be visually indicated in some form.

@TheBusyBiscuit TheBusyBiscuit removed the Hacktoberfest This Issue or Pull Request is part of the Hacktoberfest! label Oct 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Report A bug that needs to be fixed.
Projects
None yet
Development

No branches or pull requests

2 participants