From cfa80c4488087c88b52c95cf83a18c60ab8a333e Mon Sep 17 00:00:00 2001 From: Jason Penilla <11360596+jpenilla@users.noreply.github.com> Date: Sun, 15 Sep 2024 13:36:29 -0700 Subject: [PATCH] Raise priority of collisions LevelMixin to apply after Lithium Lithium replaces some of the height-related methods used by WorldUtil to return fields initialized in their constructor hook. We need to apply afterward to have them return valid data. Given the application-order sensitive nature of the issue, it only presented in-dev. --- .../ca/spottedleaf/moonrise/mixin/collisions/LevelMixin.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/main/java/ca/spottedleaf/moonrise/mixin/collisions/LevelMixin.java b/src/main/java/ca/spottedleaf/moonrise/mixin/collisions/LevelMixin.java index e126c5a..ebaf062 100644 --- a/src/main/java/ca/spottedleaf/moonrise/mixin/collisions/LevelMixin.java +++ b/src/main/java/ca/spottedleaf/moonrise/mixin/collisions/LevelMixin.java @@ -39,7 +39,8 @@ import java.util.ArrayList; import java.util.List; import java.util.Optional; -@Mixin(Level.class) +// Higher priority to apply after Lithium mixin.world.inline_height.WorldMixin +@Mixin(value = Level.class, priority = 1100) abstract class LevelMixin implements CollisionLevel, LevelAccessor, AutoCloseable { @Shadow