Store player loadouts (BuyMenu presets) in the Userdata directory

Fix metagame loadout paths

Author: MaximDude

CHANGELOG.md

@@ -625,7 +625,7 @@ This can be accessed via the new Lua (R/W) `SettingsMan` property `AIUpdateInter
 
 - Swapped to SDL2 for window management and input handling.
 
-- `Settings.ini` is now stored in the `Userdata` directory instead of `Base.rte`.
+- `Settings.ini` and player loadouts (BuyMenu presets) are now stored in the `Userdata` directory instead of `Base.rte`.
 
 - Lua scripts are now run in a more efficient way. As part of this change, `PieSlice` scripts need to be reloaded like `MovableObject` scripts (i.e. using `pieSlice:ReloadScripts()`, in order for their changes to be reflected in-game.  
 	`PresetMan:ReloadAllScripts()` will reload `PieSlice` preset scripts, like it does for `MovableObject`s.

Managers/SettingsMan.cpp

@@ -72,7 +72,7 @@ namespace RTE {
 	int SettingsMan::Initialize() {
 		if (const char *settingsTempPath = std::getenv("CCCP_SETTINGSPATH")) { m_SettingsPath = std::string(settingsTempPath); }
 
-		Reader settingsReader(m_SettingsPath, false, nullptr, true);
+		Reader settingsReader(m_SettingsPath, false, nullptr, true, true);
 
 		if (!settingsReader.ReaderOK()) {
 			Writer settingsWriter(m_SettingsPath);

Menus/BuyMenuGUI.cpp

@@ -405,7 +405,7 @@ bool BuyMenuGUI::LoadAllLoadoutsFromFile()
     if (m_MetaPlayer != Players::NoPlayer)
     {
         // Start loading any additional stuff from the custom user file
-        std::snprintf(loadoutPath, sizeof(loadoutPath), "%s/%s - LoadoutsMP%d.ini", c_UserConquestSavesModuleName.c_str(), g_MetaMan.GetGameName().c_str(), m_MetaPlayer + 1);
+        std::snprintf(loadoutPath, sizeof(loadoutPath), "%s/%s - LoadoutsMP%d.ini", (System::GetUserdataDirectory() + c_UserConquestSavesModuleName).c_str(), g_MetaMan.GetGameName().c_str(), m_MetaPlayer + 1);
 
         if (!System::PathExistsCaseSensitive(loadoutPath))
         {
@@ -416,12 +416,12 @@ bool BuyMenuGUI::LoadAllLoadoutsFromFile()
     // Not a metagame player, just a regular scenario player
     else
 	{
-        std::snprintf(loadoutPath, sizeof(loadoutPath), "Base.rte/LoadoutsP%d.ini", m_pController->GetPlayer() + 1);
+        std::snprintf(loadoutPath, sizeof(loadoutPath), "%sLoadoutsP%d.ini", System::GetUserdataDirectory().c_str(), m_pController->GetPlayer() + 1);
 
 	}
 
     // Open the file
-    Reader loadoutFile(loadoutPath, false, 0, true);
+    Reader loadoutFile(loadoutPath, false, nullptr, true, true);
 
     // Read any and all loadout presets from file
     while (loadoutFile.ReaderOK() && loadoutFile.NextProperty())
@@ -518,12 +518,12 @@ bool BuyMenuGUI::SaveAllLoadoutsToFile()
         // Since the players of a new game are likely to have different techs and therefore different default loadouts
         // So we should start fresh with new loadouts loaded from tech defaults for each player
         if (g_MetaMan.GetGameName() == DEFAULTGAMENAME)
-            std::snprintf(loadoutPath, sizeof(loadoutPath), "%s/%s - LoadoutsMP%d.ini", c_UserConquestSavesModuleName.c_str(), AUTOSAVENAME, m_MetaPlayer + 1);
+            std::snprintf(loadoutPath, sizeof(loadoutPath), "%s/%s - LoadoutsMP%d.ini", (System::GetUserdataDirectory() + c_UserConquestSavesModuleName).c_str(), AUTOSAVENAME, m_MetaPlayer + 1);
         else
-            std::snprintf(loadoutPath, sizeof(loadoutPath), "%s/%s - LoadoutsMP%d.ini", c_UserConquestSavesModuleName.c_str(), g_MetaMan.GetGameName().c_str(), m_MetaPlayer + 1);
+            std::snprintf(loadoutPath, sizeof(loadoutPath), "%s/%s - LoadoutsMP%d.ini", (System::GetUserdataDirectory() + c_UserConquestSavesModuleName).c_str(), g_MetaMan.GetGameName().c_str(), m_MetaPlayer + 1);
     }
     else
-        std::snprintf(loadoutPath, sizeof(loadoutPath), "Base.rte/LoadoutsP%d.ini", m_pController->GetPlayer() + 1);
+        std::snprintf(loadoutPath, sizeof(loadoutPath), "%sLoadoutsP%d.ini", System::GetUserdataDirectory().c_str(), m_pController->GetPlayer() + 1);
 
     // Open the file
     Writer loadoutFile(loadoutPath, false);