OptiFabric icon indicating copy to clipboard operation
OptiFabric copied to clipboard

[MIXIN ISSUE?] Replay Mod + Optifabric 1.20.1 crashing

Open MegaMinerDL opened this issue 1 year ago • 30 comments

Crash does NOT occur when either OptiFabric or Replay Mod is taken out

OptiFine_1.20.1_HD_U_I5 replaymod-1.20.1-2.6.13 optifabric-1.13.25 fabric-loader-0.14.21-1.20.1 Windows 10

MIRRORED ISSUES: https://github.com/Chocohead/OptiFabric/issues/1113 https://github.com/Chocohead/OptiFabric/issues/1125 https://github.com/Chocohead/OptiFabric/issues/1145

MegaMinerDL avatar Jul 15 '23 16:07 MegaMinerDL

Optifabric 1.13.25: Crashes when Puzzles Lib/trading Post are in mods folder. Tried to load it up again and it just freezes on loading screen. Not crashes, freezes.

Goddessa39 avatar Jul 15 '23 23:07 Goddessa39

Its a Mixin issue, ReplayMod has needed support added to Optifabric for a while now. Its just a case of someone adding support.

TheUnknownCod3r avatar Jul 16 '23 10:07 TheUnknownCod3r

Its a Mixin issue, ReplayMod has needed support added to Optifabric for a while now. Its just a case of someone adding support.

It worked just fine on Fabric 1.19 (Replay mod + Optifine + Optifabric)

MegaMinerDL avatar Jul 16 '23 12:07 MegaMinerDL

Its a Mixin issue, ReplayMod has needed support added to Optifabric for a while now. Its just a case of someone adding support.

It worked just fine on Fabric 1.19 (Replay mod + Optifine + Optifabric)

It worked fine on 1.19 because support was updated with the changes made by Minecraft / ReplayMod. theres changes in between versions that broke those support changes, which need updated. Its a common problem with Optifine, which is the main issue here.

TheUnknownCod3r avatar Jul 24 '23 21:07 TheUnknownCod3r

Its a Mixin issue, ReplayMod has needed support added to Optifabric for a while now. Its just a case of someone adding support.

It worked just fine on Fabric 1.19 (Replay mod + Optifine + Optifabric)

It worked fine on 1.19 because support was updated with the changes made by Minecraft / ReplayMod. theres changes in between versions that broke those support changes, which need updated. Its a common problem with Optifine, which is the main issue here.

So which mod is causing the current compatibility issue?

MegaMinerDL avatar Jul 24 '23 22:07 MegaMinerDL

Its a Mixin issue, ReplayMod has needed support added to Optifabric for a while now. Its just a case of someone adding support.

It worked just fine on Fabric 1.19 (Replay mod + Optifine + Optifabric)

It worked fine on 1.19 because support was updated with the changes made by Minecraft / ReplayMod. theres changes in between versions that broke those support changes, which need updated. Its a common problem with Optifine, which is the main issue here.

So which mod is causing the current compatibility issue?

Optifabric, I think. Im not really familiar with the replayMod codebase, but I don't see what changed in the file in question for 1.20, tbh.

This is the file thats causing the issue, if someone wants to look at it. https://github.com/ReplayMod/jGui/blob/master/src/main/java/de/johni0702/minecraft/gui/versions/mixin/MixinKeyboardListener.java

TheUnknownCod3r avatar Jul 24 '23 23:07 TheUnknownCod3r

I just tried to remove that problematic class and changed the mixin JSON file, and ReplayMod & OptiFine 1.20.1 works without big issues

unzip ReplayMod JAR with 7-Zip, go to com/replaymod/lib/de/john0702/minecraft/gui/versions/mixin and delete MixinKeyboardListener.class, go to the root, edit mixins.jgui.json and remove line 18 ("MixinKeyboardListener", ). Re-zip the unzipped replaymod jar file using 7-Zip (as zip, compression level to STORE, change the extension to JAR). Now open optifabric with 7-Zip and edit optifabric.compat.replaymod.newer-mixins.json and delete line 6 and put the modified json file to the JAR file again.

(EDIT - attaching the modified JARs) the source code is unchanged exact same so refer to the originak github repositories (notice GPL 3.0). I am using Fabric 0.14.21 & Fabric API 0.83.0 (or maybe 0.85 i don't remember)

[mods.zip]

bluedragon-cairo avatar Jul 27 '23 03:07 bluedragon-cairo

Interesting, I did these steps exactly and Minecraft turned on (so there's compatibility), however Replay Mod shows no trace of it's existence in the game. I hope official updates solve this - what does MixinKeyboardListener do here?

MegaMinerDL avatar Jul 27 '23 10:07 MegaMinerDL

what. For me, replaymod menu showed up too with optifine, will send you the modified jar files, the source code is unchanged exact same so refer to the originak github repositories (notice GPL 3.0). I am using Fabric 0.14.21 & Fabric API 0.83.0 (or maybe 0.85 i don't remember)

[mods.zip]

bluedragon-cairo avatar Jul 27 '23 10:07 bluedragon-cairo

KakaoTalk_20230727_195017538 KakaoTalk_20230727_195023004

bluedragon-cairo avatar Jul 27 '23 10:07 bluedragon-cairo

Interesting, I did these steps exactly and Minecraft turned on (so there's compatibility), however Replay Mod shows no trace of it's existence in the game. I hope official updates solve this - what does MixinKeyboardListener do here?

From the code I’d assume it logs the keybinds for triggering record, stop etc, but I’m not sure. Removing it should most likely stop it from working altogether

TheUnknownCod3r avatar Jul 27 '23 10:07 TheUnknownCod3r

Play/Pause keybind worked (didn't test other keybinds) and watching the replay with moving/spectating worked also fine, not sure what that does tho

bluedragon-cairo avatar Jul 27 '23 11:07 bluedragon-cairo

Play/Pause keybind worked (didn't test other keybinds) and watching the replay with moving/spectating worked also fine, not sure what that does tho

Huh. Then honestly, I’m not sure. JGui is apparently some sort of gui lib, idk.

TheUnknownCod3r avatar Jul 27 '23 11:07 TheUnknownCod3r

Well, your files worked, maybe I zipped wrong? Anyway thank you this will do, I tested and all keybinds/settings/recording work

MegaMinerDL avatar Jul 27 '23 13:07 MegaMinerDL

Well, your files worked, maybe I zipped wrong? Anyway thank you this will do, I tested and all keybinds/settings/recording work

If it’s working fine, maybe something to ask the dev of replayMod, as it might be possible to just remove it anyway, and fix the incompatibility all in one. I’ve checked back seems the fix is the same inject point across all versions.

TheUnknownCod3r avatar Jul 27 '23 15:07 TheUnknownCod3r

Opened mirror issue on ReplayMod https://github.com/ReplayMod/ReplayMod/issues/883

MegaMinerDL avatar Jul 27 '23 16:07 MegaMinerDL

Interesting, I did these steps exactly and Minecraft turned on (so there's compatibility), however Replay Mod shows no trace of it's existence in the game. I hope official updates solve this - what does MixinKeyboardListener do here?

That's weird because when I did it, it just crashed as usual with the "error code 1" again.

NotCremon avatar Jul 29 '23 14:07 NotCremon

Interesting, I did these steps exactly and Minecraft turned on (so there's compatibility), however Replay Mod shows no trace of it's existence in the game. I hope official updates solve this - what does MixinKeyboardListener do here?

That's weird because when I did it, it just crashed as usual with the "error code 1" again.

Yeah idk, use the files bluedragon sent

MegaMinerDL avatar Jul 29 '23 17:07 MegaMinerDL

what. For me, replaymod menu showed up too with optifine, will send you the modified jar files, the source code is unchanged exact same so refer to the originak github repositories (notice GPL 3.0). I am using Fabric 0.14.21 & Fabric API 0.83.0 (or maybe 0.85 i don't remember)

[mods.zip]

what. For me, replaymod menu showed up too with optifine, will send you the modified jar files, the source code is unchanged exact same so refer to the originak github repositories (notice GPL 3.0). I am using Fabric 0.14.21 & Fabric API 0.83.0 (or maybe 0.85 i don't remember)

[mods.zip]

YO CHECK my issue in my dashboard i need ur help @bluedragon-cairo

TraderTheWolf avatar Sep 02 '23 10:09 TraderTheWolf

I just tried to remove that problematic class and changed the mixin JSON file, and ReplayMod & OptiFine 1.20.1 works without big issues

unzip ReplayMod JAR with 7-Zip, go to com/replaymod/lib/de/john0702/minecraft/gui/versions/mixin and delete MixinKeyboardListener.class, go to the root, edit mixins.jgui.json and remove line 18 ("MixinKeyboardListener", ). Re-zip the unzipped replaymod jar file using 7-Zip (as zip, compression level to STORE, change the extension to JAR). Now open optifabric with 7-Zip and edit optifabric.compat.replaymod.newer-mixins.json and delete line 6 and put the modified json file to the JAR file again.

(EDIT - attaching the modified JARs) the source code is unchanged exact same so refer to the originak github repositories (notice GPL 3.0). I am using Fabric 0.14.21 & Fabric API 0.83.0 (or maybe 0.85 i don't remember)

[mods.zip]

Thank you!! It finally worked!!

Hatbuilder avatar Dec 19 '23 17:12 Hatbuilder

No problem

bluedragon-cairo avatar Dec 29 '23 05:12 bluedragon-cairo

@bluedragon-cairo Hey! Please for version 1.20.4+ and fabric

KJU51003042 avatar Apr 25 '24 09:04 KJU51003042

I haven't played Minecraft recently but will try to make it soon.

bluedragon-cairo avatar Apr 26 '24 11:04 bluedragon-cairo

@bluedragon-cairo pleas make for optifabric-1.14.3

Xleby4ek avatar May 22 '24 15:05 Xleby4ek

I'd prefer using 1.12.2 with ViaForge or 1.17+ with ViaFabric though

bluedragon-cairo avatar May 23 '24 07:05 bluedragon-cairo

I'd prefer using 1.12.2 with ViaForge or 1.17+ with ViaFabric though

You misunderstand me 1.14.3 is a version of Optifabric, not minecraft image

Xleby4ek avatar May 23 '24 07:05 Xleby4ek

mods.zip

1.20.4 Release with the same edits supplied by BlueDragonCairo, should work fine.

TheUnknownCod3r avatar May 23 '24 07:05 TheUnknownCod3r

@TheUnknownCod3r unfortunately, these files didn't work, optifabric said the error was more precisely mixing error

KJU51003042 avatar May 23 '24 14:05 KJU51003042

@TheUnknownCod3r unfortunately, these files didn't work, optifabric said the error was more precisely mixing error

I uploaded the exact version that worked in my local instance, and only applied the fixes which @bluedragon-cairo posted above. If you want to check the fixes, make sure that you deleted your old Optifabric, Optifine, and ReplayMod jars, then paste mine. Also post your log.

image Just a screenshot, showing the mod saving a screenshot on 1.20.4, with Optifine 1.20.4 displayed

TheUnknownCod3r avatar May 23 '24 14:05 TheUnknownCod3r

Rip this issue

MegaMinerDL avatar May 24 '24 13:05 MegaMinerDL