search

Shinoow / AbyssalCraft

Source repo for the AbyssalCraft mod
GNU Lesser General Public License v3.0
81 stars 52 forks source link

[1.12.2] FutureMC's Grindstone produces 100 levels of exp when removing enchantment from AbyssalCraft enchantment book #489

Closed RisingInIris2017 closed 1 year ago

RisingInIris2017 commented 2 years ago

Minecraft Version

1.12.2

Forge Version

14.23.5.2855

Related Mods Version

Forgelin: 1.8.4 Future MC: 0.2.7.3 AbyssalCraft: 1.10.3

How to reproduce

  1. Get an Coralium or Dread Enchantment Book. Other AbyssalCraft's Enchantment Books will not reproduce this issue.
  2. Put them into FutureMC's Grindstone.
  3. Take the output book out, then you will get great numbers of exp points which let you get 100 levels from 0 level.

I have posted the same issue to AbyssalCraft: https://github.com/thedarkcolour/Future-MC/issues/264 Thanks for your help in advance! Looking forward to your reply.

Shinoow commented 2 years ago

You're not meant to obtain enchanted books for those enchantments, and some of the logic in place to also prevent the enchanting table from applying them includes skyrocketing the minimum required enchantability (which is likely what's used by FutureMC in order to calculate how much exp should be given for a specific enchantment).

If FutureMC has some kind of blacklist for this grindstone block, I would add these enchantments to said blacklist if I were you, since I'm most likely not gonna change any of the logic for those enchantments in the near future (they're removed in the 2.0.0 branch).

RisingInIris2017 commented 2 years ago

Got it. I think you are right and I will suggest FutureMC's dev to do so.