[ISSUE-9451] Loadout Management

Importing a shared item list fix
When importing a shared item list it now uses CreateItemSet so it
doesn't double add the set anymore

Co-authored-by: LocalIdentity <localidentity2@gmail.com>

Author: FenikSRT4

spec/System/TestItemsTab_spec.lua

@@ -196,6 +196,17 @@ describe("TestItemsTab", function()
 		end)
 	end)
 
+	describe("ItemSetListControl", function()
+		it("adds an imported shared item set to the build once", function()
+			local itemSetList = new("ItemSetListControl", nil, { 0, 0, 300, 200 }, build.itemsTab)
+
+			itemSetList:ReceiveDrag("SharedItemList", { title = "Shared Set", slots = {} })
+
+			assert.are.equals(2, #build.itemsTab.itemSetOrderList)
+			assert.are.equals("Shared Set", build.itemsTab.itemSets[build.itemsTab.itemSetOrderList[2]].title)
+		end)
+	end)
+
 	describe("ItemSetService", function()
 		local itemSetService
 		before_each(function()

src/Classes/ItemSetListControl.lua

@@ -109,7 +109,7 @@ end
 
 function ItemSetListClass:ReceiveDrag(type, value, source)
 	if type == "SharedItemList" then
-		local itemSet = self.itemsTab:NewItemSet()
+		local itemSet = self.itemsTab:CreateItemSet()
 		itemSet.title = value.title
 		for slotName, item in pairs(value.slots) do
 			local newItem = new("Item", item.raw)