This update brings some things to the table.
Besides rewriting the whole mod in Kotlin, which might seem useless, but honestly since I enjoy programming with Kotlin more than Java it really makes me more prone to actively work on the project and finally add the features I felt were missing from the start.
This update is currently in Beta, I will wait until I get some positive feedback, or at least some downloads without any negative feedback, before changing this to the Release channel.
The main missing feature that was finally added was the option to choose which items to utilize to check where the enchantments can be applied. This is pretty straight forward but unfortunately, due to some limitiations with my proficiency with the widely available config APIs, there is no way to change this setting ingame :(
The other decent feature is that now all the default configs can be overwritten by any other mod or even via resource pack. This is not exactly the actual feature itself but honestly I see no problem letting it be this way. Power to the player.
The actual original point of the this feature was for people to be able to share configs via resource packs but more importantly for modders to be able to ship a single json file in their mod that will add default values to their custom enchantments for example.
I think this should be about it. It felt like more was accomplished but that might be due to the fruitless time I spent trying to bend mod config apis to my will. I'll try again soonishâ„¢.
- Avoid mutating passed enchantments by Devan-Kerman
- Fix possible NullPointerException trying to draw tooltip icons (by Fourmisain)[https://github.com/Bernasss12/BetterEnchantedBooks/pull/34]
By Fourmisain:
- fix 2 NullPointerExceptions when items have unregistered enchantments
- fix sorting by translated name
- fix caching of default colored books
- fix some books being default colored because they use id "id" instead of "minecraft:id"
- add cloth-config2 as a 'depends'
- enable pretty-printing of config
By Fourmisain:
- update minecraft version, dependencies and gradle build
- fix NullPointerException in method17869Tail due to uninitialized ThreadLocal
- refactor NBTUtils: use streams instead of new ArrayLists, sort() should never throw an Exception
- use WeakHashMap for ItemStack caches
- rename Mixin methods
- add a proper Logger
- update README (do not run gradlew idea!)