Issue 335: Repair cost as defined in .ini (#391)

* Fixed issue where repair cost was not fetched from base inis.
* Fixed CargoDrop and Condata that wouldn't compile as a result of a breaking SDK change.
---------

Co-authored-by: oliverpechey <olliezeta9@gmail.com>

CHANGELOG.md

@@ -1,4 +1,8 @@
 # Changelog
+## 4.0.20
+- Fixed issue where repair cost was not fetched from base inis.
+- Fixed CargoDrop and Condata that wouldn't compile as a result of a breaking SDK change.
+
 ## 4.0.19
 - Updated the SonarScan to use version 17 of Java, as 11 is no longer working and causes the pipeline to fail.
 - Updated SonarScan to 5.0.1.3006

plugins/autobuy/Autobuy.cpp

@@ -67,7 +67,15 @@ namespace Plugins::Autobuy
 
 	void handleRepairs(ClientId& client)
 	{
-		auto repairCost = static_cast<uint>(Archetype::GetShip(Players[client].shipArchetype)->fHitPoints * (1 - Players[client].fRelativeHealth) / 3);
+		auto base = Hk::Player::GetCurrentBase(client);
+
+		// If there is no error with finding the base the correct repair cost can be calculated
+		auto baseCost = 0.33;
+		if (base.has_value())
+		{
+			baseCost = BaseDataList_get()->get_base_data(base.value())->get_ship_repair_cost();
+		}
+		auto repairCost = static_cast<uint>((1 - Players[client].fRelativeHealth) * Archetype::GetShip(Players[client].shipArchetype)->fHitPoints * baseCost);
 
 		std::set<short> eqToFix;
 
@@ -530,8 +538,8 @@ namespace Plugins::Autobuy
 		auto config = Serializer::JsonToObject<Config>();
 		global->config = std::make_unique<Config>(config);
 
-		// Get ammo limit
-		for (const auto& iniPath : global->config->iniPaths)
+		// Get ammo limits
+		for (const auto& iniPath : global->config->ammoIniPaths)
 		{
 			INI_Reader ini;
 			if (!ini.open(iniPath.c_str(), false))
@@ -563,7 +571,7 @@ namespace Plugins::Autobuy
 
 					if (valid)
 					{
-						global->ammoLimits[itemname] = itemlimit;
+						global->ammoLimits.insert(std::pair<uint, int>(itemname, itemlimit));
 					}
 				}
 			}
@@ -573,7 +581,7 @@ namespace Plugins::Autobuy
 
 using namespace Plugins::Autobuy;
 
-REFL_AUTO(type(Config), field(nanobot_nickname), field(shield_battery_nickname), field(iniPaths))
+REFL_AUTO(type(Config), field(nanobot_nickname), field(shield_battery_nickname), field(ammoIniPaths))
 
 DefaultDllMainSettings(LoadSettings);
 

plugins/autobuy/Autobuy.h

@@ -40,7 +40,7 @@ namespace Plugins::Autobuy
 		std::string shield_battery_nickname = "ge_s_battery_01";
 
 		// paths to inis which should be read for ammo limits
-		std::vector<std::string> iniPaths 
+		std::vector<std::string> ammoIniPaths 
 		{
 			R"(..\data\equipment\light_equip.ini)",
 			R"(..\data\equipment\select_equip.ini)",

plugins/cargo_drop/CargoDrop.cpp

@@ -85,7 +85,7 @@ namespace Plugins::CargoDrop
 				// Simulate an obj update to stop the ship in space.
 				SSPObjUpdateInfo updateInfo {};
 				snd.lastTimestamp += 1.0;
-				updateInfo.fTimestamp = static_cast<float>(snd.lastTimestamp);
+				updateInfo.dTimestamp = snd.lastTimestamp;
 				updateInfo.cState = 0;
 				updateInfo.fThrottle = 0;
 				updateInfo.vPos = snd.lastPosition;
@@ -201,7 +201,7 @@ namespace Plugins::CargoDrop
 	 */
 	void SPObjUpdate(struct SSPObjUpdateInfo const& ui, ClientId& client)
 	{
-		global->info[client].lastTimestamp = ui.fTimestamp;
+		global->info[client].lastTimestamp = ui.dTimestamp;
 		global->info[client].lastPosition = ui.vPos;
 		global->info[client].lastDirection = ui.vDir;
 	}

plugins/condata/Condata.cpp

@@ -265,7 +265,7 @@ namespace Plugins::ConData
 			return; // ??? 8[
 
 		const mstime timeNow = Hk::Time::GetUnixMiliseconds();
-		const auto timestamp = static_cast<mstime>(ui.fTimestamp * 1000);
+		const auto timestamp = static_cast<mstime>(ui.dTimestamp * 1000);
 
 		auto& con = global->connections[client];