Fix java.lang.invoke.WrongMethodTypeException:

[timderspieler]

Fixed a WrongMethodTypeException thrown when the chest open animation method has been called.

Original error:
[13:24:00 WARN]: java.lang.invoke.WrongMethodTypeException: cannot convert MethodHandle(int,int,int)BlockPosition to (double,double,double)Object [13:24:00 WARN]: at java.base/java.lang.invoke.MethodHandle.asTypeUncached(MethodHandle.java:884) [13:24:00 WARN]: at java.base/java.lang.invoke.MethodHandle.asType(MethodHandle.java:869) [13:24:00 WARN]: at java.base/java.lang.invoke.Invokers.checkGenericType(Invokers.java:542) [13:24:00 WARN]: at ZombieSurvival-Revamped-1.0.0-all.jar//sv.file14.zs.shaded.xseries.xseries.NMSExtras.chest(NMSExtras.java:561) [13:24:00 WARN]: at ZombieSurvival-Revamped-1.0.0-all.jar//sv.file14.zombiesurvival.util.NMSUtil.playChestAnimation(NMSUtil.java:104) [13:24:00 WARN]: at ZombieSurvival-Revamped-1.0.0-all.jar//sv.file14.zombiesurvival.gameSign.MysteryBoxSign.doEvent(MysteryBoxSign.java:70) [13:24:00 WARN]: at ZombieSurvival-Revamped-1.0.0-all.jar//sv.file14.zombiesurvival.listener.PlayerInteract.onEvent(PlayerInteract.java:84)

[CryptoMorin]

You should make it backwards compatible, it used double for the previous versions.

XSeries/src/main/java/com/cryptomorin/xseries/NMSExtras.java

Lines 213 to 215 in 1fad51f

	blockPosition = lookup.findConstructor(blockPos,
	v(19, MethodType.methodType(void.class, int.class, int.class, int.class)).orElse(
	MethodType.methodType(void.class, double.class, double.class, double.class)));

[timderspieler]

You should make it backwards compatible, it used double for the previous versions.

XSeries/src/main/java/com/cryptomorin/xseries/NMSExtras.java

Lines 213 to 215 in 1fad51f

	blockPosition = lookup.findConstructor(blockPos,
	v(19, MethodType.methodType(void.class, int.class, int.class, int.class)).orElse(
	MethodType.methodType(void.class, double.class, double.class, double.class)));

Done!

[CryptoMorin]

There's another problem now,

Object position = v(19, BLOCK_POSITION.invoke(location.getBlockX(), location.getBlockY(), location.getBlockZ()))
                                   //  ^^^^^ gets called
  .orElse(BLOCK_POSITION.invoke(location.getX(), location.getY(), location.getZ()));
                       // ^^^^^ gets called

Both BLOCK_POSITION invocations are going to be called regardless of which one it's going to pick, so it'd still throw that error. You could use the one with a supplier to prevent it from invoking the unused one:

Object position = v(19, () -> BLOCK_POSITION.invoke(location.getBlockX(), location.getBlockY(), location.getBlockZ()))
  .orElse(() -> BLOCK_POSITION.invoke(location.getX(), location.getY(), location.getZ()));

[timderspieler]

@CryptoMorin that should be it now.

[CryptoMorin]

Thanks!