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

Enchantment Extractors can be formed in Custom Crafters and vice versa #321

Closed
Bloo-dev opened this issue Nov 6, 2019 · 2 comments
Closed
Labels
bug Something isn't working

Comments

@Bloo-dev
Copy link
Member

Bloo-dev commented Nov 6, 2019

Describe the bug
Enchantment Extractors aren't based on custom crafters and therefore don't check for a custom crafters existence on creation (the same goes for custom crafters). This makes it possible to create an enchantment extractor inside a custom crafter and vice versa. The resulting machine acts as both Enchantment Extractor and Custom Crafter and displays both top textures.
This does not break any functionality nor does it enable any exploits, it just looks glitchy.

To Reproduce
Steps to reproduce the behavior:

  1. Create a Custom Crafter or Enchantment Extractor
  2. Use the Machine to create an Enchantment Extractor or Custom Crafter
  3. You'll end up with an Enchantment Extractor and Custom Crafter in a single block

Expected behavior
These should not be peaceable in the same block space.

Fix
The modules have to check for each other upon machine creation - or even better neither of them should be craftable if there is already an armor stand in the dropper. This would enable compatibility with all dropper based machines.
Alternatively, Enchantment Extractora could be turned into a Custom Crafters expansion. It feels weird that this machine that is also based on a dropper is not a Custom Crafters upgrade. The general armor stand check as described above might still be a good ideax though.

Screenshots
Screenshots tba.

Environment

  • Minecraft version: 1.14
  • Server version: Spigot 1.14 ; Gamemode 4 Public Server 4

Relates Issues
#76 is caused by the same missing check, just with hoppers instead of droppers as a base block.

@SpecialBuilder32
Copy link
Member

I was thinking we could fix this by looking for an armor_stand with the gm4_no_edit tag already in the dropper when making a new machine. While not the most elegant solution (that might have some conflicts with different sized stands), it would fix all the strange issues with compressors inside custom crafters or enchant extractors inside master crafters ect....

@EpyonProjects
Copy link
Member

Enchantment Extractors can no longer be made within Custom Crafters.

However Custom Crafters can still be made within an Enchantment Extractor. All items used to make the Enchantment Extractor will pop out and the Custom Crafter cannot be used as an Enchantment Extractor. Other than the replacement of the Enchantment Extractor, I would say this is no longer a bug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants