1.15.2 Port

CHANGELOG.md

@@ -3,9 +3,6 @@
 **1.15.2 Port**
 * Tweak Steadfast Spell
 
-**1.2.6**
-* Fix Potential Client-Server De-sync Bug
-
 **1.2.5**
 * Fix Casting Table Bug
 

gradle.properties

@@ -10,7 +10,7 @@ org.gradle.jvmargs = -Xmx1G
 	fabric_loader_version = 0.7.8+build.189
 
 # Mod Properties
-	mod_version = 1.2.6
+	mod_version = 1.2.5
 	maven_group = com.thebrokenrail
 	archives_base_name = sorcerycraft
 

src/main/java/com/thebrokenrail/sorcerycraft/client/gui/CastingTableScreen.java

@@ -21,9 +21,9 @@ import net.minecraft.util.math.MathHelper;
 @Environment(EnvType.CLIENT)
 public class CastingTableScreen extends ContainerScreen<CastingTableScreenHandler> {
     private static final Identifier TEXTURE = new Identifier("textures/gui/container/villager2.png");
-    private int selectedIndex = 0;
-    private int indexStartOffset = 0;
-    private boolean scrolling = false;
+    private int selectedIndex;
+    private int indexStartOffset;
+    private boolean scrolling;
 
     public CastingTableScreen(CastingTableScreenHandler container, PlayerInventory inventory, Text title) {
         super(container, inventory, title);
@@ -40,11 +40,6 @@ public class CastingTableScreen extends ContainerScreen<CastingTableScreenHandle
         renderXPCost();
     }
 
-    public void resetIndex() {
-        selectedIndex = 0;
-        indexStartOffset = 0;
-    }
-
     @Override
     protected void drawBackground(float delta, int mouseX, int mouseY) {
         RenderSystem.color4f(1.0F, 1.0F, 1.0F, 1.0F);

src/main/java/com/thebrokenrail/sorcerycraft/gui/CastingTableScreenHandler.java

@@ -109,11 +109,6 @@ public class CastingTableScreenHandler extends Container {
         }
     }
 
-    private void resetIndex() {
-        index = 0;
-        onContentChanged(inventory);
-    }
-
     public void setSpells(PlayerEntity player) {
         if (player.isCreative() ? SorceryCraft.getConfig().limitCastingTable.creative : SorceryCraft.getConfig().limitCastingTable.survival) {
             SpellPlayerEntity spellPlayer = (SpellPlayerEntity) player;
@@ -130,7 +125,8 @@ public class CastingTableScreenHandler extends Container {
         } else {
             spells = SpellRegistry.getSpells();
         }
-        resetIndex();
+        index = 0;
+        onContentChanged(inventory);
     }
 
     public boolean canTakeResult(PlayerEntity playerEntity) {

src/main/java/com/thebrokenrail/sorcerycraft/mixin/MixinClientPlayNetworkHandler.java

@@ -16,7 +16,6 @@ import org.spongepowered.asm.mixin.injection.Inject;
 import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;
 
 @SuppressWarnings("unused")
-@Environment(EnvType.CLIENT)
 @Mixin(ClientPlayNetworkHandler.class)
 public class MixinClientPlayNetworkHandler {
     @Shadow

src/main/java/com/thebrokenrail/sorcerycraft/mixin/MixinClientPlayerEntity.java

@@ -1,28 +0,0 @@
-package com.thebrokenrail.sorcerycraft.mixin;
-
-import com.thebrokenrail.sorcerycraft.client.gui.CastingTableScreen;
-import net.fabricmc.api.EnvType;
-import net.fabricmc.api.Environment;
-import net.minecraft.client.MinecraftClient;
-import net.minecraft.client.network.ClientPlayerEntity;
-import net.minecraft.util.Identifier;
-import org.spongepowered.asm.mixin.Final;
-import org.spongepowered.asm.mixin.Mixin;
-import org.spongepowered.asm.mixin.Shadow;
-
-import java.util.Map;
-
-@SuppressWarnings("unused")
-@Mixin(ClientPlayerEntity.class)
-@Environment(EnvType.CLIENT)
-public class MixinClientPlayerEntity extends MixinPlayerEntity {
-    @Shadow @Final protected MinecraftClient client;
-
-    @Override
-    public void setDiscoveredSpells(Map<Identifier, Integer> spells) {
-        super.setDiscoveredSpells(spells);
-        if (client.currentScreen instanceof CastingTableScreen) {
-            ((CastingTableScreen) client.currentScreen).resetIndex();
-        }
-    }
-}

src/main/java/com/thebrokenrail/sorcerycraft/mixin/MixinServerPlayerEntity.java

@@ -5,8 +5,6 @@ import com.thebrokenrail.sorcerycraft.packet.UpdateKnownSpellsS2CPacket;
 import com.thebrokenrail.sorcerycraft.spell.util.SpellHelper;
 import com.thebrokenrail.sorcerycraft.spell.util.SpellPlayerEntity;
 import com.thebrokenrail.sorcerycraft.spell.util.SpellServerPlayerEntity;
-import net.fabricmc.api.EnvType;
-import net.fabricmc.api.Environment;
 import net.minecraft.nbt.CompoundTag;
 import net.minecraft.server.network.ServerPlayerEntity;
 import net.minecraft.util.Identifier;
@@ -18,7 +16,6 @@ import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;
 import java.util.Map;
 
 @SuppressWarnings("unused")
-@Environment(EnvType.SERVER)
 @Mixin(ServerPlayerEntity.class)
 public abstract class MixinServerPlayerEntity extends MixinPlayerEntity implements SpellServerPlayerEntity {
     @Inject(at = @At("HEAD"), method = "copyFrom")

src/main/java/com/thebrokenrail/sorcerycraft/packet/UpdateKnownSpellsS2CPacket.java

@@ -5,8 +5,6 @@ import com.thebrokenrail.sorcerycraft.gui.CastingTableScreenHandler;
 import com.thebrokenrail.sorcerycraft.spell.util.SpellPlayerEntity;
 import com.thebrokenrail.sorcerycraft.spell.util.SpellHelper;
 import io.netty.buffer.Unpooled;
-import net.fabricmc.api.EnvType;
-import net.fabricmc.api.Environment;
 import net.fabricmc.fabric.api.network.PacketContext;
 import net.minecraft.nbt.CompoundTag;
 import net.minecraft.network.packet.s2c.play.CustomPayloadS2CPacket;
@@ -15,7 +13,6 @@ import net.minecraft.util.Identifier;
 import net.minecraft.util.PacketByteBuf;
 
 public class UpdateKnownSpellsS2CPacket {
-    @Environment(EnvType.CLIENT)
     public static void handle(PacketContext context, PacketByteBuf bytes) {
         CompoundTag tag = bytes.readCompoundTag();
         if (context.getPlayer() != null) {

src/main/resources/sorcerycraft.mixins.json

@@ -6,10 +6,9 @@
         "MixinClientPlayNetworkHandler"
     ],
     "mixins": [
-        "CriterionRegistryHook",
-        "MixinClientPlayerEntity",
         "MixinPlayerEntity",
-        "MixinServerPlayerEntity"
+        "MixinServerPlayerEntity",
+        "CriterionRegistryHook"
     ],
     "injectors": {
         "defaultRequire": 1