From 6c5ec2d99642ea45481bf5164f29fc87885ee01b Mon Sep 17 00:00:00 2001
From: Dan Paulat <dpaulat@gmail.com>
Date: Thu, 27 Apr 2023 22:50:27 -0500
Subject: [PATCH] Update map provider setting based on API key availability

---
 .../scwx/qt/manager/settings_manager.cpp      | 41 +++++++++++++++++++
 1 file changed, 41 insertions(+)

diff --git a/scwx-qt/source/scwx/qt/manager/settings_manager.cpp b/scwx-qt/source/scwx/qt/manager/settings_manager.cpp
index be161245..6050d483 100644
--- a/scwx-qt/source/scwx/qt/manager/settings_manager.cpp
+++ b/scwx-qt/source/scwx/qt/manager/settings_manager.cpp
@@ -23,6 +23,7 @@ static const auto        logger_    = scwx::util::Logger::Create(logPrefix_);
 static boost::json::value ConvertSettingsToJson();
 static void               GenerateDefaultSettings();
 static bool               LoadSettings(const boost::json::object& settingsJson);
+static void               ValidateSettings();
 
 static bool        initialized_ {false};
 static std::string settingsPath_ {};
@@ -46,6 +47,7 @@ void Initialize()
    initialized_  = true;
 
    ReadSettings(settingsPath_);
+   ValidateSettings();
 }
 
 void ReadSettings(const std::string& settingsPath)
@@ -137,6 +139,45 @@ static bool LoadSettings(const boost::json::object& settingsJson)
    return jsonDirty;
 }
 
+static void ValidateSettings()
+{
+   logger_->debug("Validating settings");
+
+   bool settingsChanged = false;
+
+   auto& generalSettings = general_settings();
+
+   // Validate map provider
+   std::string mapProvider    = generalSettings.map_provider().GetValue();
+   std::string mapboxApiKey   = generalSettings.mapbox_api_key().GetValue();
+   std::string maptilerApiKey = generalSettings.maptiler_api_key().GetValue();
+
+   if (mapProvider == "maptiler" && //
+       mapboxApiKey.size() > 1 &&   //
+       maptilerApiKey == "?")
+   {
+      logger_->info("Setting Map Provider to Mapbox based on API key settings");
+
+      generalSettings.map_provider().SetValue("mapbox");
+      settingsChanged = true;
+   }
+   else if (mapProvider == "mapbox" &&   //
+            maptilerApiKey.size() > 1 && //
+            mapboxApiKey == "?")
+   {
+      logger_->info(
+         "Setting Map Provider to MapTiler based on API key settings");
+
+      generalSettings.map_provider().SetValue("maptiler");
+      settingsChanged = true;
+   }
+
+   if (settingsChanged)
+   {
+      SaveSettings();
+   }
+}
+
 } // namespace SettingsManager
 } // namespace manager
 } // namespace qt