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

Infuser Does Not Accept Valid Inputs #602

Closed
airbreather opened this issue Apr 9, 2015 · 0 comments
Closed

Infuser Does Not Accept Valid Inputs #602

airbreather opened this issue Apr 9, 2015 · 0 comments

Comments

@airbreather
Copy link
Contributor

Tested with both forestry_1.7.10-3.5.3.11.jar and a local gradle build of the latest source at the time of writing (784a47b).

Steps to reproduce:

  • Start a new creative world.
  • Add an Infuser, Pollen Cluster, Crystalline Pollen Cluster, and Short Mead to the player's inventory.
  • Open the Infuser GUI.
  • Try to put Pollen Cluster into an item slot tagged as "(No Effect)"

Expected: the inventory accepts the Pollen Cluster item, and the description changes from "(No Effect)" to "Strong Curative".
Actual: the inventory rejects the Pollen Cluster item.

  • Try to put Crystalline Pollen Cluster into an item slot tagged as "(No Effect)"

Expected: the inventory accepts the Crystalline Pollen Cluster item, and the description changes from "(No Effect)" to "Weak Curative"
Actual: the inventory rejects the Crystalline Pollen Cluster item.

  • Try to put Short Mead into the input item slot with a bottle as the background.

Expected: the inventory accepts the Short Mead item.
Actual: the inventory rejects the Short Mead item.

I've got a local fix for strictly this bug, but after testing it out, there are other related issues. It happens that doing nothing and letting this bug be a thing would be safer than fixing just this bug without any other changes at the same time. I'll submit a pull request once I've tackled all the issues I can find.

airbreather added a commit to airbreather/ForestryMC that referenced this issue Apr 9, 2015
Do not use the Infuser yet; it still has other issues, and items you put in may be lost.
@mezz mezz closed this as completed in 0462e5a Apr 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants