ci: try to fix CI

- rewrite most logic
- remove GlobalItem
- use a hook for changing StorageHeart.RightClick instead of manual IL edit

.github/workflows/main.yml

@@ -1,30 +0,0 @@
-name: Create Release
-
-on:
-  push:
-    tags:
-      - '*'
-
-jobs:
-  create-release:
-    runs-on: ubuntu-latest
-
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v2
-
-      - name: Create changelog text
-        id: changelog
-        uses: loopwerk/tag-changelog@v1
-        with:
-          token: ${{ secrets.GITHUB_TOKEN }}
-          config_file: .github/workflows/tag_changelog_config.js
-
-      - name: Create release
-        uses: actions/create-release@latest
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-        with:
-          tag_name: ${{ github.ref }}
-          release_name: ${{ github.ref }}
-          body: ${{ steps.changelog.outputs.changes }}

.github/workflows/tag_changelog_config.js

@@ -1,15 +0,0 @@
-module.exports = {
-  types: [
-    { types: ["feat", "feature"], label: "🎉 New Features" },
-    { types: ["fix", "bugfix"], label: "🐛 Bugfixes" },
-    { types: ["improvements", "enhancement"], label: "🔨 Improvements" },
-    { types: ["perf"], label: "🏎️ Performance Improvements" },
-    { types: ["build", "ci"], label: "🏗️ Build System" },
-    { types: ["refactor"], label: "🪚 Refactors" },
-    { types: ["doc", "docs"], label: "📚 Documentation Changes" },
-    { types: ["test", "tests"], label: "🔍 Tests" },
-    { types: ["style"], label: "💅 Code Style Changes" },
-    { types: ["chore"], label: "🧹 Chores" },
-    { types: ["other"], label: "Other Changes" },
-  ],
-};

.woodpecker.yml

@@ -0,0 +1,64 @@
+clone:
+  git:
+    image: woodpeckerci/plugin-git:latest
+    settings:
+      path: MagicStorageVoidBag
+    when:
+      event: tag
+
+pipeline:
+  fetch-deps:
+    image: jackbondpreston/msvb-build-env
+    commands:
+      - steamcmd "+login anonymous" "+workshop_download_item 1281930 2563309347" "+quit"
+      - wget https://github.com/tModLoader/tModLoader/releases/latest/download/tModLoader.zip
+      - unzip -q tModLoader.zip -d tModLoader
+      - rm tModLoader.zip
+      - ln -sf $CI_WORKSPACE/tModLoader/Libraries/Native/Linux/libSDL2-2.0.so.0 $CI_WORKSPACE/tModLoader/Libraries/Native/Linux/libSDL2.so
+      - >
+        echo "<Project ToolsVersion=\"14.0\" xmlns=\"http://schemas.microsoft.com/developer/msbuild/2003\">
+            <Import Project=\"$CI_WORKSPACE/tModLoader/tMLMod.targets\" />
+        </Project>" > tModLoader.targets
+      - |
+        wget https://github.com/steviegt6/tml-patcher/releases/latest/download/TML.Patcher.zip
+        unzip -q TML.Patcher.zip -d TMLPatcher
+      - |
+        rm -f /root/.steam/steamapps/workshop/content/1281930/2563309347/workshop.json
+        VER=$(ls /root/.steam/steamapps/workshop/content/1281930/2563309347 | cut -c 6- | sort | head -n 1)
+        YEAR=$(ls /root/.steam/steamapps/workshop/content/1281930/2563309347 | cut -c -4 | sort | head -n 1)
+        cp /root/.steam/steamapps/workshop/content/1281930/2563309347/$YEAR.$VER/MagicStorage.tmod ./
+      - dotnet TMLPatcher/TML.Patcher.dll extract MagicStorage.tmod
+      - cp MagicStorage/MagicStorage.dll $CI_WORKSPACE/
+    when:
+      event: tag
+
+  build:
+    image: jackbondpreston/msvb-build-env
+    commands:
+      - |
+        echo "Shell: $SHELL"
+        echo "CI_WORKSPACE: $CI_WORKSPACE"
+      - cat tModLoader.targets
+      - cd $CI_WORKSPACE
+      - cd MagicStorageVoidBag
+      - rm -rf .git
+      - dotnet build -c Release || true
+      - cp bin/Release/net6.0/MagicStorageVoidBag.dll .
+      - cd $CI_WORKSPACE
+      - mkdir release
+      - cp ~/.local/share/Terraria/tModLoader/Mods/MagicStorageVoidBag.tmod release/
+      - cp MagicStorageVoidBag/MagicStorageVoidBag.dll release/
+      - ls release
+    when:
+      event: tag
+    
+  gitea-release:
+    image: plugins/gitea-release
+    settings:
+      api_key:
+        from_secret: gitea-api-key
+      base_url: https://git.jackbondpreston.me
+      files: release/*
+    when:
+      event: tag
+

Hooks/GetItemVoidVaultHook.cs

@@ -0,0 +1,62 @@
+using MagicStorage;
+using MagicStorage.Components;
+using MagicStorageVoidBag.Items;
+using System.Linq;
+using Terraria;
+using Terraria.Audio;
+using Terraria.DataStructures;
+using Terraria.GameContent.Achievements;
+using Terraria.ID;
+using Terraria.ModLoader;
+
+namespace MagicStorageVoidBag.Hooks {
+    internal class GetItemVoidVaultHook {
+        private static readonly log4net.ILog Logger = MagicStorageVoidBag.Instance.Logger;
+        public static bool Hook(On.Terraria.Player.orig_GetItem_VoidVault orig, Player player, int plr, Item[] inventory, Item newItem, GetItemSettings settings, Item returnItem) {
+            var i = player.inventory.FirstOrDefault(i => i.type == ModContent.ItemType<MSVoidBag>(), null);
+
+            newItem = newItem.Clone();
+
+            if (i == null) goto original;
+
+            var bag = (MSVoidBag)i.ModItem;
+
+            if (bag.location.X < 0 || bag.location.Y < 0) goto original;
+
+            Tile tile = Main.tile[bag.location.X, bag.location.Y];
+
+            if (!tile.HasTile || tile.TileType != ModContent.TileType<StorageHeart>() || tile.TileFrameX != 0 || tile.TileFrameY != 0) goto original;
+            if (!TileEntity.ByPosition.TryGetValue(bag.location, out TileEntity te)) goto original;
+            if (te.type != ModContent.TileEntityType<TEStorageHeart>()) goto original;
+
+            TEStorageHeart heart = (TEStorageHeart)te;
+
+            heart.TryDeposit(returnItem);
+            heart.ResetCompactStage();
+            StorageGUI.needRefresh = true;
+
+            if (returnItem.stack != newItem.stack) {
+                if (newItem.IsACoin) {
+                    SoundEngine.PlaySound(SoundID.CoinPickup, player.position);
+                } else {
+                    SoundEngine.PlaySound(SoundID.Grab, player.position);
+                }
+
+                if (!settings.NoText) {
+                    PopupText.NewText(PopupTextContext.ItemPickupToVoidContainer, newItem, returnItem.stack, noStack: false, settings.LongText);
+                }
+
+                AchievementsHelper.NotifyItemPickup(player, returnItem);
+            }
+
+            if (returnItem.stack == 0) return true;
+
+            original:
+            if (player.HasItem(ItemID.VoidLens)) {
+                return orig(player, plr, inventory, newItem, settings, returnItem);
+            } else {
+                return false;
+            }
+        }
+    }
+}

Hooks/ItemSpaceForCofveveHook.cs

@@ -0,0 +1,45 @@
+using MagicStorage.Components;
+using MagicStorageVoidBag.Items;
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+using Terraria;
+using Terraria.DataStructures;
+using Terraria.ModLoader;
+using MagicStorage;
+using Terraria.ID;
+
+namespace MagicStorageVoidBag.Hooks {
+    internal class ItemSpaceForCofveveHook {
+        private static readonly log4net.ILog Logger = MagicStorageVoidBag.Instance.Logger;
+
+        // Despite the constant negative press cofveve
+        public static bool Hook(On.Terraria.Player.orig_ItemSpaceForCofveve orig, Player player, Item newItem) {
+            var i = player.inventory.FirstOrDefault(i => i.type == ModContent.ItemType<MSVoidBag>(), null);
+            if (i == null) goto original;
+
+            newItem = newItem.Clone();
+
+            var bag = (MSVoidBag)i.ModItem;
+
+            if (bag.location.X < 0 || bag.location.Y < 0) goto original;
+
+            Tile tile = Main.tile[bag.location.X, bag.location.Y];
+
+            if (!tile.HasTile || tile.TileType != ModContent.TileType<StorageHeart>() || tile.TileFrameX != 0 || tile.TileFrameY != 0) goto original;
+            if (!TileEntity.ByPosition.TryGetValue(bag.location, out TileEntity te)) goto original;
+            if (te.type != ModContent.TileEntityType<TEStorageHeart>()) goto original;
+
+            if (Utility.HeartHasSpaceFor(newItem, (TEStorageHeart)te)) return true;
+
+            original:
+            if (player.HasItem(ItemID.VoidLens)) {
+                return orig(player, newItem);
+            } else {
+                return false;
+            }
+        }
+    }
+}

ILPatches/ILPatch.cs

@@ -0,0 +1,12 @@
+using MonoMod.Cil;
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace MagicStorageVoidBag.ILPatches {
+    internal interface ILPatch {
+        public abstract void Patch(ILContext il);
+    }
+}

ILPatches/PlayerUpdatePatch.cs

@@ -0,0 +1,39 @@
+using MagicStorageVoidBag.Items;
+using Mono.Cecil.Cil;
+using MonoMod.Cil;
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+using Terraria;
+using Terraria.ModLoader;
+
+namespace MagicStorageVoidBag.ILPatches {
+    internal class PlayerUpdatePatch : ILPatch {
+        private static readonly log4net.ILog Logger = MagicStorageVoidBag.Instance.Logger;
+        public void Patch(ILContext il) {
+            if (il == null) {
+                Logger.Error("ILContext null!");
+                return;
+            }
+
+            Logger.Debug("Patching Terraria.Player.Update IL...");
+
+            var c = new ILCursor(il);
+            var setterMethod = typeof(Player).GetProperty(nameof(Player.IsVoidVaultEnabled)).GetSetMethod();
+            if (!c.TryGotoNext(i => i.MatchCallOrCallvirt(setterMethod))) {
+                Logger.Warn("Failed to go to next call or callvirt! :(");
+                return;
+            }
+
+            c.Emit(OpCodes.Ldarg_0);
+            c.Emit(OpCodes.Ldc_I4, ModContent.ItemType<MSVoidBag>());
+            var hasItemMethod = typeof(Player).GetMethod(nameof(Player.HasItem));
+            c.Emit(OpCodes.Call, hasItemMethod);
+            c.Emit(OpCodes.Or);
+
+            Logger.Debug("...Complete!");
+        }
+    }
+}

Items/GlobalItem.cs

@@ -1,38 +0,0 @@
-using MagicStorage.Components;
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Text;
-using System.Threading.Tasks;
-using Terraria;
-using Terraria.DataStructures;
-using Terraria.Localization;
-using Terraria.ModLoader;
-
-namespace MagicStorageVoidBag.Items {
-    internal class GlobalItem : Terraria.ModLoader.GlobalItem {
-        public override bool OnPickup(Item item, Player player) {
-            var i = player.inventory.Where(i => i.type == ModContent.ItemType<MSVoidBag>()).DefaultIfEmpty(null).First();
-            if (i != null && !player.ItemSpace(item).CanTakeItem) {
-                var bag = (MSVoidBag)i.ModItem;
-
-                if (bag.location.X < 0 || bag.location.Y < 0) return false;
-
-                Tile tile = Main.tile[bag.location.X, bag.location.Y];
-
-                if (!tile.HasTile || tile.TileType != ModContent.TileType<StorageHeart>() || tile.TileFrameX != 0 || tile.TileFrameY != 0) return false;
-                if (!TileEntity.ByPosition.TryGetValue(bag.location, out TileEntity te)) return false;
-                if (te.type != ModContent.TileEntityType<TEStorageHeart>()) return false;
-
-                TEStorageHeart heart = (TEStorageHeart)te;
-
-                int oldStack = item.stack;
-                heart.TryDeposit(item);
-                if (item.stack == 0) return true;
-
-            }
-
-            return base.OnPickup(item, player);
-        }
-    }
-}

Items/MSVoidBag.cs

@@ -1,42 +1,21 @@
 using MagicStorage.Items;
 using System;
 using System.Collections.Generic;
+using System.IO;
+using System.Linq;
 using Terraria;
+using Terraria.DataStructures;
 using Terraria.GameContent.Creative;
 using Terraria.ID;
 using Terraria.Localization;
 using Terraria.ModLoader;
+using Terraria.ModLoader.IO;
 
 namespace MagicStorageVoidBag.Items {
     [ExtendsFromMod("MagicStorage")]
     public class MSVoidBag : PortableAccess {
-
-        /* 
-         Code here largely thanks to original MagicStorage source, which can be found at
-         https://github.com/blushiemagic/MagicStorage/
-         License:
-            MIT License
-
-            Copyright(c) 2017 Kaylee Minsuh Kim
-
-            Permission is hereby granted, free of charge, to any person obtaining a copy
-            of this software and associated documentation files(the "Software"), to deal
-            in the Software without restriction, including without limitation the rights
-            to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-            copies of the Software, and to permit persons to whom the Software is
-            furnished to do so, subject to the following conditions:
-
-            The above copyright notice and this permission notice shall be included in all
-            copies or substantial portions of the Software.
-
-            THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-            IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-            FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.IN NO EVENT SHALL THE
-            AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-            LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-            OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
-            SOFTWARE.
-        */
+        [CloneByReference]
+        internal Dictionary<string, Point16> locationsByWorld = new();
 
         public override void SetStaticDefaults() {
             CreativeItemSacrificesCatalog.Instance.SacrificeCountNeededByItemId[Type] = 1;
@@ -51,6 +30,9 @@ namespace MagicStorageVoidBag.Items {
             Item.useAnimation = 28;
             Item.useTime = 28;
             Item.value = Item.sellPrice(gold: 10);
+
+            location = Point16.NegativeOne;
+            locationsByWorld[Main.worldName] = location;
         }
 
         public override void ModifyTooltips(List<TooltipLine> lines) {
@@ -70,5 +52,55 @@ namespace MagicStorageVoidBag.Items {
             recipe.AddIngredient(ItemID.VoidLens);
             recipe.Register();
         }
+
+        public override void UpdateInventory(Player player) {
+            if (!locationsByWorld.ContainsKey(Main.worldName) || location != locationsByWorld[Main.worldName]) {
+                locationsByWorld[Main.worldName] = location;
+                if (Main.netMode == NetmodeID.MultiplayerClient) {
+                    int p = Array.IndexOf(Main.player, player);
+                    int i = Array.IndexOf(player.inventory, Item);
+                    if (p >= 0 && i >= 0) {
+                        NetMessage.SendData(MessageID.SyncEquipment, -1, -1, null, p, i, player.inventory[i].prefix);
+                    }
+                }
+            }
+        }
+
+        public override void SaveData(TagCompound tag) {
+            tag["X"] = location.X;
+            tag["Y"] = location.Y;
+
+            tag["version"] = SAVE_VERSION;
+
+            tag["locations"] = locationsByWorld
+                .Select(kvp => new TagCompound() {
+                    ["world"] = kvp.Key,
+                    ["X"] = kvp.Value.X,
+                    ["Y"] = kvp.Value.Y
+                })
+                .ToList();
+        }
+
+        public override void LoadData(TagCompound tag) {
+            if (tag.GetInt("version") < SAVE_VERSION || tag.GetList<TagCompound>("locations") is not List<TagCompound> locations) {
+                location = new Point16(tag.GetShort("X"), tag.GetShort("Y"));
+                locationsByWorld[Main.worldName] = location;
+            } else {
+                locationsByWorld = locations.ToDictionary(t => t.GetString("world"), t => new Point16(t.GetShort("X"), t.GetShort("Y")));
+                if (locationsByWorld.ContainsKey(Main.worldName)) {
+                    location = locationsByWorld[Main.worldName];
+                }
+            }
+        }
+
+        public override void NetSend(BinaryWriter writer) {
+            writer.Write(location.X);
+            writer.Write(location.Y);
+        }
+
+        public override void NetReceive(BinaryReader reader) {
+            location = new Point16(reader.ReadInt16(), reader.ReadInt16());
+            locationsByWorld[Main.worldName] = location;
+        }
     }
 }

LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2022 Jack Bond-Preston
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

MagicStorage-LICENSE.txt

@@ -0,0 +1,23 @@
+This mod uses various pieces of code from the original MagicStorage mod, the license for which is below:
+
+MIT License
+
+Copyright (c) 2017 Kaylee Minsuh Kim
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

MagicStorageVoidBag.cs

@@ -1,57 +1,29 @@
-using Mono.Cecil.Cil;
-using Mono.Cecil;
-using MonoMod.Cil;
+
 using Terraria.ModLoader;
-using MagicStorageVoidBag.Items;
+using MagicStorageVoidBag.ILPatches;
+using MagicStorageVoidBag.Hooks;
 
 namespace MagicStorageVoidBag {
     public class MagicStorageVoidBag : Mod {
+        public static MagicStorageVoidBag Instance => ModContent.GetInstance<MagicStorageVoidBag>();
+
+        // IL Patches
+        private PlayerUpdatePatch playerUpdatePatch = new();
+
         public override void Load() {
-            IL.MagicStorage.Components.StorageHeart.RightClick += HeartRightClickPatch;
+            IL.Terraria.Player.Update += playerUpdatePatch.Patch;
+
+            On.Terraria.Player.GetItem_VoidVault += GetItemVoidVaultHook.Hook;
+            On.Terraria.Player.ItemSpaceForCofveve += ItemSpaceForCofveveHook.Hook;
         }
 
         public override void Unload() {
+            IL.Terraria.Player.Update -= playerUpdatePatch.Patch;
+
+            On.Terraria.Player.GetItem_VoidVault -= GetItemVoidVaultHook.Hook;
+            On.Terraria.Player.ItemSpaceForCofveve -= ItemSpaceForCofveveHook.Hook;
+
             base.Unload();
         }
-
-        // Patch MagicStorage IL to change the Storage Heart right click handler to also work with
-        //  MSVoidBag.
-        private void HeartRightClickPatch(ILContext il) {
-            if (il == null) {
-                Logger.Error("ILContext null!");
-                return;
-            }
-
-            Logger.Debug("Patching MagicStorage IL...");
-
-            var c = new ILCursor(il);
-
-            if (!c.TryGotoNext(i => i.MatchCallOrCallvirt("Terraria.ModLoader.ModContent", "ItemType"))) {
-                Logger.Warn("IL patching failed! :(");
-            }
-            c = c.GotoPrev().GotoPrev();
-
-            var c2 = c.Clone();
-
-            // copy IL up to call
-            while (c2.Next.OpCode != OpCodes.Call) {
-                if (c2.Next.Operand != null) {
-                    c.Emit(c2.Next.OpCode, c2.Next.Operand);
-                } else {
-                    c.Emit(c2.Next.OpCode);
-                }
-
-                c2 = c2.GotoNext();
-            }
-
-            var method = typeof(ModContent).GetMethod("ItemType").MakeGenericMethod(typeof(MSVoidBag));
-
-            c.Emit(c2.Next.OpCode, method);
-            c2.GotoNext();
-            c.Emit(c2.Next.OpCode, c2.Next.Operand);
-
-            Logger.Debug("...MagicStorage IL patching complete!");
-        }
-
     }
 }

MagicStorageVoidBag.csproj

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <Project Sdk="Microsoft.NET.Sdk">
-  <Import Project="..\tModLoader.targets" />
+  <Import Project="../tModLoader.targets" />
   <PropertyGroup>
     <AssemblyName>MagicStorageVoidBag</AssemblyName>
     <TargetFramework>net6.0</TargetFramework>
@@ -12,10 +12,7 @@
   </ItemGroup>
   <ItemGroup>
     <Reference Include="MagicStorage">
-      <HintPath>..\Mod Libraries\MagicStorage.dll</HintPath>
-    </Reference>
-    <Reference Include="MMHOOK_MagicStorage">
-      <HintPath>lib\MMHOOK_MagicStorage.dll</HintPath>
+      <HintPath>..\MagicStorage\MagicStorage.dll</HintPath>
     </Reference>
   </ItemGroup>
 </Project>

README.md

@@ -1,4 +1,4 @@
-# MagicStorageVoidBag
+# MagicStorage Void Bag
 
 A mod that extends [Magic Storage](https://github.com/blushiemagic/MagicStorage).
 

Utility.cs

@@ -0,0 +1,25 @@
+using MagicStorage;
+using MagicStorage.Components;
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+using Terraria;
+
+namespace MagicStorageVoidBag {
+    internal static class Utility {
+        internal static bool HeartHasSpaceFor(Item newItem, TEStorageHeart heart) {
+            foreach (TEAbstractStorageUnit storageUnit in heart.GetStorageUnits()) {
+                if (!storageUnit.Inactive) {
+                    var unitItems = storageUnit.GetItems();
+
+                    if (!storageUnit.IsFull) return true;
+                    if (storageUnit.HasSpaceInStackFor(newItem)) return true;
+                }
+            }
+
+            return false;
+        }
+    }
+}

build.txt

@@ -1,8 +1,7 @@
 displayName = MagicStorage Void Bag
 author = jack
-version = 0.2
+version = 1.5
 modReferences = MagicStorage
-dllReferences = MMHOOK_MagicStorage
 sortAfter = MagicStorage
 includeSource = true
 homepage = https://github.com/jackbondpreston/MagicStorageVoidBag

description.txt

@@ -1,2 +1,4 @@
 Adds an upgraded void bag that combines functionality of the Void Bag and Portable Remote Storage Access, by putting items into a magic storage system when your inventory is full (and allowing remote access as usual).
 This item is crafted by combining a Void Bag with a Portable Remote Storage Access. To link it with your storage system, right click the Storage Heart with the bag equipped.
+
+Thanks to the original MagicStorage mod for some code and of course the functionality of the original mod!

description_workshop.txt

@@ -1,2 +1,7 @@
 Adds an upgraded void bag that combines functionality of the Void Bag and Portable Remote Storage Access, by putting items into a magic storage system when your inventory is full (and allowing remote access as usual).
 This item is crafted by combining a Void Bag with a Portable Remote Storage Access. To link it with your storage system, right click the Storage Heart with the bag equipped.
+
+GitHub: https://github.com/jackbondpreston/MagicStorageVoidBag
+Discord server if you need support/don't want to report bugs on GitHub: https://discord.gg/Z6V77mbtZQ
+
+Thanks to the original MagicStorage mod for some code and of course the functionality of the original mod!