RemoteCrafting optimisation

Mods/0-SCore/Features/RemoteCrafting/Harmony/DropBoxToContainers.cs

@@ -17,13 +17,11 @@ public static bool Prefix(XUiC_LootContainer __instance) {
 
                 if (__instance.localTileEntity == null)
                 {
-                    Log.Out("[DropBox] OnClose: localTileEntity is null, skipping.");
                     return true;
                 }
                 var composite = __instance.localTileEntity as TEFeatureStorage;
                 if (composite == null)
                 {
-                    Log.Out("[DropBox] OnClose: localTileEntity is not TEFeatureStorage, skipping.");
                     return true;
                 }
 
@@ -34,7 +32,6 @@ public static bool Prefix(XUiC_LootContainer __instance) {
                                          StringParsers.TryParseBool(dropBoxStr, out var dropBoxBool) && dropBoxBool;
                 if (block is not BlockDropBoxContainer && !hasDropBoxProperty)
                 {
-                    Log.Out($"[DropBox] OnClose: block at {pos} ({block.GetBlockName()}) is not a drop box, skipping.");
                     return true;
                 }
 
@@ -44,29 +41,26 @@ public static bool Prefix(XUiC_LootContainer __instance) {
                     distance = StringParsers.ParseFloat(strDistance);
 
                 var primaryPlayer = __instance.xui.playerUI.entityPlayer;
-                Log.Out($"[DropBox] OnClose: distributing from {block.GetBlockName()} at {pos}, player={primaryPlayer?.EntityName ?? "null"}, distance={distance}, IsServer={SingletonMonoBehaviour<ConnectionManager>.Instance.IsServer}");
+                if (primaryPlayer == null)
+                    return true;
 
                 if (__instance.localTileEntity.items == null)
                 {
-                    Log.Out("[DropBox] OnClose: items array is null, skipping.");
                     return true;
                 }
 
                 var items = __instance.localTileEntity.items;
+                var tileEntities = RemoteCraftingUtils.GetTileEntities(primaryPlayer, distance, false);
                 for (var i = 0; i < items.Length; i++)
                 {
                     if (items[i].IsEmpty()) continue;
-                    var itemName = items[i].itemValue?.ItemClass?.GetItemName() ?? "unknown";
-                    Log.Out($"[DropBox] OnClose: trying to distribute slot {i}: {itemName} x{items[i].count}");
 
-                    if (RemoteCraftingUtils.AddToNearbyContainer(primaryPlayer, items[i], distance))
+                    if (RemoteCraftingUtils.AddToNearbyContainer(primaryPlayer, items[i], tileEntities))
                     {
-                        Log.Out($"[DropBox] OnClose: distributed {itemName} successfully, clearing slot {i}.");
                         items[i] = ItemStack.Empty.Clone();
                         continue;
                     }
 
-                    Log.Out($"[DropBox] OnClose: no container accepted {itemName}, leaving in slot {i}.");
                     __instance.localTileEntity.UpdateSlot(i, items[i]);
                 }