[ISSUE-9451] Loadout Management

Pressing F2 to rename the loadout now grabs the right name
Renaming the loadout by pressing F2 would grab a table address instead
of the name

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

Author: FenikSRT4

spec/System/TestLoadouts_spec.lua

@@ -632,6 +632,21 @@ describe("TestLoadouts", function()
 		end)
 	end)
 
+	describe("BuildSetListControl", function()
+		it("passes the loadout title through the F2 rename shortcut", function()
+			build:NewLoadout("Second")
+			local loadoutList = new("BuildSetListControl", nil, { 0, 0, 380, 200 }, build)
+			local renameName
+			loadoutList.RenameLoadout = function(_, name)
+				renameName = name
+			end
+
+			loadoutList:OnSelKeyDown(2, build.loadoutsList[2], "F2")
+
+			assert.are.equals("Second", renameName)
+		end)
+	end)
+
 	describe("BuildSetService", function()
 		local buildSetService
 		before_each(function()

src/Classes/BuildSetListControl.lua

@@ -198,7 +198,7 @@ end
 
 function BuildSetListClass:OnSelKeyDown(index, spec, key)
 	if key == "F2" then
-		self:RenameLoadout(spec)
+		self:RenameLoadout(spec.title)
 	end
 end
 
@@ -220,7 +220,6 @@ function BuildSetListClass:BasicLoadoutPopup(options)
 	local specNameLookup = self.buildSetService:SpecNameLookup()
 	controls.edit = new("EditControl", nil, { 0, 40, 350, 20 },
 		options.defaultName or "Default", nil, nil, 100, function(buf)
-			print("Buf changed:", buf, specNameLookup[buf])
 			controls.save.enabled = specNameLookup[buf] == nil and buf:match("%S")
 		end)