Skip to content

Commit 7ec0966

Browse files
committed
Fix server-accelerated refresh event
1 parent 450e557 commit 7ec0966

5 files changed

Lines changed: 7 additions & 33 deletions

File tree

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ ___
5252
- Added a screen-class blacklist to allow disabling buttons on certain modded screens.
5353
- Fixed an issue causing certain items with data to always be sorted to the end.
5454
- Fixed a crash when pressing `Open Editor` key if also bound to `Drop Selected Item`.
55+
- Fixed an issue causing server-accelerated ops to not trigger refresh events.
5556

5657
## 3.102.5 [2026-07-04]
5758

common/src/main/java/dev/terminalmc/clientsort/mixin/accessor/ContainerAccessor.java

Lines changed: 0 additions & 28 deletions
This file was deleted.

common/src/main/java/dev/terminalmc/clientsort/network/handler/PayloadHandler.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@
2020
import dev.terminalmc.clientsort.exception.PayloadHandlerException;
2121
import dev.terminalmc.clientsort.exception.PayloadHandlerException.InconsistentStateException;
2222
import dev.terminalmc.clientsort.exception.PayloadHandlerException.InvalidDataException;
23-
import dev.terminalmc.clientsort.mixin.accessor.ContainerAccessor;
2423
import dev.terminalmc.clientsort.network.handler.validate.PayloadResult;
2524
import dev.terminalmc.clientsort.platform.services.PlatformServices;
2625
import net.minecraft.network.protocol.common.custom.CustomPacketPayload;
@@ -110,9 +109,8 @@ public static void processPayload(
110109
} finally {
111110
if (menu != null) {
112111
menu.resumeRemoteUpdates();
113-
menu.broadcastChanges();
114-
((ContainerAccessor) menu.slots.getFirst().container).clientsort$setChanged();
115-
((ContainerAccessor) menu.slots.getLast().container).clientsort$setChanged();
112+
menu.slotsChanged(menu.slots.getFirst().container);
113+
menu.slotsChanged(menu.slots.getLast().container);
116114
}
117115
if (PlatformServices.getInstance().canSendToPlayer(player, responseType)) {
118116
PlatformServices.getInstance()

common/src/main/java/dev/terminalmc/clientsort/network/handler/TransferHandler.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,8 @@ private static void transfer(
9191

9292
if (srcStack.isEmpty())
9393
continue;
94+
if (!srcSlot.mayPlace(srcStack))
95+
continue;
9496

9597
// Nonempty stack found; work forwards from the start of the
9698
// destination array, looking for a partial stack of the same item
@@ -105,6 +107,8 @@ private static void transfer(
105107
continue;
106108
if (!ItemStack.isSameItemSameComponents(srcStack, dstStack))
107109
continue;
110+
if (!dstSlot.mayPlace(srcStack))
111+
continue;
108112

109113
// Matching partial stack found
110114

common/src/main/resources/clientsort.mixins.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
"minVersion": "${mixinextras_version_min}"
99
},
1010
"mixins": [
11-
"accessor.ContainerAccessor",
1211
"accessor.ServerCommonPacketListenerImplAccessor",
1312
"slot.SlotMixin"
1413
],

0 commit comments

Comments
 (0)