[ISSUE-9451] Loadout Management

Fix items getting deleted from shared loadouts
If an item belonged to 2 loadouts and 1 of them was deleted, it would
wipe the items without checking to see if another loadout used them

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

Author: FenikSRT4

spec/System/TestLoadouts_spec.lua

@@ -212,6 +212,26 @@ describe("TestLoadouts", function()
 					assert.is_same(2, build.configTab.activeConfigSetId)
 					assert.is_true(build.modFlag)
 				end)
+
+			it("preserves singleton sets shared by the remaining loadout", function()
+				build:NewLoadout("Second")
+				build.itemsTab:DeleteItemSet(2, 2)
+				build.skillsTab:DeleteSkillSet(2, 2)
+				build.configTab:DeleteConfigSet(2, 2)
+				build.itemsTab:SetActiveItemSet(1, true)
+				build.skillsTab:SetActiveSkillSet(1, true)
+				build.configTab:SetActiveConfigSet(1, false, true)
+				build.itemsTab.itemSets[1]["Body Armour"].selItemId = 1
+				build.skillsTab.skillSets[1].socketGroupList = { { gemList = { { nameSpec = "TestGem" } } } }
+				build.configTab.configSets[1].input.testVar = 123
+				build:SyncLoadouts()
+
+				build:DeleteLoadout("Second", "Default")
+
+				assert.are.equals(1, build.itemsTab.itemSets[1]["Body Armour"].selItemId)
+				assert.are.equals("TestGem", build.skillsTab.skillSets[1].socketGroupList[1].gemList[1].nameSpec)
+				assert.are.equals(123, build.configTab.configSets[1].input.testVar)
+			end)
 		end)
 
 		describe("RenameLoadout", function()

src/Modules/Build.lua

@@ -876,15 +876,15 @@ function buildMode:DeleteLoadout(loadoutName, nextLoadoutName)
 	if loadout.specId then
 		t_remove(self.treeTab.specList, loadout.specId)
 	end
-	if loadout.itemSetId then
+	if loadout.itemSetId and #self.itemsTab.itemSetOrderList > 1 then
 		local index = reverseLookup(self.itemsTab.itemSetOrderList, loadout.itemSetId)
 		self.itemsTab:DeleteItemSet(loadout.itemSetId, index)
 	end
-	if loadout.skillSetId then
+	if loadout.skillSetId and #self.skillsTab.skillSetOrderList > 1 then
 		local index = reverseLookup(self.skillsTab.skillSetOrderList, loadout.skillSetId)
 		self.skillsTab:DeleteSkillSet(loadout.skillSetId, index)
 	end
-	if loadout.configSetId then
+	if loadout.configSetId and #self.configTab.configSetOrderList > 1 then
 		local index = reverseLookup(self.configTab.configSetOrderList, loadout.configSetId)
 		self.configTab:DeleteConfigSet(loadout.configSetId, index)
 	end