Updated deps. Followed some simple vanilla changes.
Rewrote the Item Group implementation for 1.19.4 (will need a small change for 1.20).
Removed ItemGroupAccessor in favor of ItemGroups.getGroups()
I've tested Creative inventory and Survival crafting and block breaking, so I think the major features are intact. In some cases where I'm not certain exactly why something is the way it is I've tried to preserve existing behavior. The following items should probably be reviewed.
I'm not sure about changing HammerData.group to be an Identifier. This might affect an API and/or Magna. However, the String name of ItemGroups is only for display as far as I can tell.
I'm also not certain how to replace the functionality of ExtendedHammerItem.isIn(ItemGroup) which overrode a removed vanilla method and may have been the only reason for HammerData.group and HammerData.cachedGroup to exist.
This requires the updated Magna here: https://github.com/Draylar/magna/pull/28
ItemGroupAccessorin favor ofItemGroups.getGroups()I've tested Creative inventory and Survival crafting and block breaking, so I think the major features are intact. In some cases where I'm not certain exactly why something is the way it is I've tried to preserve existing behavior. The following items should probably be reviewed.
I'm not sure about changing
HammerData.groupto be an Identifier. This might affect an API and/or Magna. However, the String name of ItemGroups is only for display as far as I can tell.I'm also not certain how to replace the functionality of
ExtendedHammerItem.isIn(ItemGroup)which overrode a removed vanilla method and may have been the only reason forHammerData.groupandHammerData.cachedGroupto exist.