fix drum/crate drops (#5073)

Author: jurrejelle

docs/content/Development/Data-Sync-System/Migrating-From-LDLib-SyncData.md

@@ -69,9 +69,10 @@ Client sync fields **do not** automatically detect changes. When changing a clie
 - `@RequireRerender` -> `@RerenderOnChanged`
 - `@Persisted` -> `@SaveField`
 - `@UpdateListener` -> `@ClientFieldChangeListener` on listener method.
-- `@DropSaved` - Removed, make machines implement `IDropSaveMachine` instead
+- `@DropSaved` - Replaced with overriding the `loadFromItem`/`saveToItem` methods on `MetaMachine`
 - `@ReadOnlyManaged` and `@LazyManaged` See usage docs for instructions on complex sync objects 
 
+
 ### Other changes
 
  - `saveCustomPersistedData` & `loadCustomPersistedData` methods, and serialization of custom data types - See `ValueTransformer<T>` and `ValueTransformers` classes.

src/main/java/com/gregtechceu/gtceu/common/machine/storage/CrateMachine.java

@@ -52,6 +52,12 @@ public CrateMachine(BlockEntityCreationInfo info, Material material, int invento
         this.inventory = attachTrait(new NotifiableItemStackHandler(inventorySize, IO.BOTH));
     }
 
+    @Override
+    public void onLoad() {
+        super.onLoad();
+        inventory.shouldDropInventoryInWorld(!isTaped);
+    }
+
     @Override
     public MachineUIPanelBuilder getPanelBuilder(PosGuiData data, PanelSyncManager syncManager, UISettings settings) {
         return MachineUIPanelBuilder.panelBuilder(this).addTitleBar(false);

src/main/java/com/gregtechceu/gtceu/common/machine/storage/DrumMachine.java

@@ -102,7 +102,9 @@ public void saveToItem(CompoundTag tag, boolean clone) {
 
     @Override
     public void loadFromItem(CompoundTag tag) {
-        if (!tag.contains("Fluid")) {
+        if (tag.contains("Fluid")) {
+            stored = FluidStack.loadFluidStackFromNBT(tag.getCompound("Fluid"));
+        } else {
             stored = FluidStack.EMPTY;
         }
         // "stored" may not be same as cache (due to item's fluid cap). we should update it.