Print warning on raw packet sending

src/main/java/de/florianmichael/viafabricplus/protocoltranslator/impl/provider/viaversion/ViaFabricPlusPickItemProvider.java

@@ -0,0 +1,39 @@
+/*
+ * This file is part of ViaFabricPlus - https://github.com/FlorianMichael/ViaFabricPlus
+ * Copyright (C) 2021-2024 FlorianMichael/EnZaXD <florian.michael07@gmail.com> and RK_01/RaphiMC
+ * Copyright (C) 2023-2024 contributors
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+package de.florianmichael.viafabricplus.protocoltranslator.impl.provider.viaversion;
+
+import com.viaversion.viaversion.api.Via;
+import com.viaversion.viaversion.api.connection.UserConnection;
+import com.viaversion.viaversion.api.minecraft.BlockPosition;
+import com.viaversion.viaversion.protocols.v1_21_2to1_21_4.provider.PickItemProvider;
+
+public class ViaFabricPlusPickItemProvider extends PickItemProvider {
+
+    @Override
+    public void pickItemFromBlock(UserConnection connection, BlockPosition blockPosition, boolean includeData) {
+        Via.getPlatform().getLogger().severe("Tried to remap >=1.21.4 PICK_ITEM_FROM_BLOCK packet which is impossible without breaking the content! Find the cause and fix it!");
+    }
+
+    @Override
+    public void pickItemFromEntity(UserConnection connection, int entityId, boolean includeData) {
+        Via.getPlatform().getLogger().severe("Tried to remap >=1.21.4 PICK_ITEM_FROM_ENTITY packet which is impossible without breaking the content! Find the cause and fix it!");
+    }
+
+}

src/main/java/de/florianmichael/viafabricplus/protocoltranslator/impl/viaversion/ViaFabricPlusVLLoader.java

@@ -26,6 +26,7 @@
 import com.viaversion.viaversion.protocols.v1_12_2to1_13.provider.PlayerLookTargetProvider;
 import com.viaversion.viaversion.protocols.v1_15_2to1_16.provider.PlayerAbilitiesProvider;
 import com.viaversion.viaversion.protocols.v1_18_2to1_19.provider.AckSequenceProvider;
+import com.viaversion.viaversion.protocols.v1_21_2to1_21_4.provider.PickItemProvider;
 import com.viaversion.viaversion.protocols.v1_8to1_9.provider.HandItemProvider;
 import de.florianmichael.viafabricplus.event.PostViaVersionLoadCallback;
 import de.florianmichael.viafabricplus.protocoltranslator.impl.provider.viabedrock.ViaFabricPlusNettyPipelineProvider;
@@ -56,6 +57,7 @@ public void load() {
         providers.use(PlayerAbilitiesProvider.class, new ViaFabricPlusPlayerAbilitiesProvider());
         providers.use(SignableCommandArgumentsProvider.class, new ViaFabricPlusCommandArgumentsProvider());
         providers.use(AckSequenceProvider.class, new ViaFabricPlusAckSequenceProvider());
+        providers.use(PickItemProvider.class, new ViaFabricPlusPickItemProvider());
 
         providers.use(OldAuthProvider.class, new ViaFabricPlusOldAuthProvider());
         providers.use(ClassicWorldHeightProvider.class, new ViaFabricPlusClassicWorldHeightProvider());