From 031e8e3495e9a2d1f409348d69a659006a6de18e Mon Sep 17 00:00:00 2001
From: "Scott E. Graves" <scott.e.graves@protonmail.com>
Date: Fri, 20 Dec 2024 14:49:47 -0600
Subject: [PATCH] refactor

---
 repertory/librepertory/include/app_config.hpp | 31 +++++++++++++++--
 repertory/librepertory/src/app_config.cpp     | 34 -------------------
 2 files changed, 29 insertions(+), 36 deletions(-)

diff --git a/repertory/librepertory/include/app_config.hpp b/repertory/librepertory/include/app_config.hpp
index 7489c4f7..d239f9d8 100644
--- a/repertory/librepertory/include/app_config.hpp
+++ b/repertory/librepertory/include/app_config.hpp
@@ -108,12 +108,39 @@ private:
 private:
   template <typename dest>
   auto get_value(const json &data, const std::string &name, dest &dst,
-                 bool &found) -> bool;
+                 bool &found) -> bool {
+    REPERTORY_USES_FUNCTION_NAME();
+
+    try {
+      if (data.find(name) == data.end()) {
+        found = false;
+        return false;
+      }
+
+      data.at(name).get_to(dst);
+      return true;
+    } catch (const std::exception &ex) {
+      utils::error::raise_error(
+          function_name, ex, fmt::format("failed to get value|name|{}", name));
+    }
+
+    found = false;
+    return false;
+  }
 
   [[nodiscard]] auto load() -> bool;
 
   template <typename dest, typename source>
-  auto set_value(dest &dst, const source &src) -> bool;
+  auto set_value(dest &dst, const source &src) -> bool {
+    if (dst.load() == src) {
+      return false;
+    }
+
+    dst = src;
+    config_changed_ = true;
+    save();
+    return true;
+  }
 
 public:
   [[nodiscard]] auto get_api_auth() const -> std::string { return api_auth_; }
diff --git a/repertory/librepertory/src/app_config.cpp b/repertory/librepertory/src/app_config.cpp
index 391248e6..8013dc6f 100644
--- a/repertory/librepertory/src/app_config.cpp
+++ b/repertory/librepertory/src/app_config.cpp
@@ -866,28 +866,6 @@ auto app_config::get_provider_name(const provider_type &prov) -> std::string {
   return PROVIDER_NAMES.at(static_cast<std::size_t>(prov));
 }
 
-template <typename dest>
-auto app_config::get_value(const json &data, const std::string &name, dest &dst,
-                           bool &found) -> bool {
-  REPERTORY_USES_FUNCTION_NAME();
-
-  try {
-    if (data.find(name) == data.end()) {
-      found = false;
-      return false;
-    }
-
-    data.at(name).get_to(dst);
-    return true;
-  } catch (const std::exception &ex) {
-    utils::error::raise_error(function_name, ex,
-                              fmt::format("failed to get value|name|{}", name));
-  }
-
-  found = false;
-  return false;
-}
-
 auto app_config::get_value_by_name(const std::string &name) const
     -> std::string {
   REPERTORY_USES_FUNCTION_NAME();
@@ -1046,18 +1024,6 @@ void app_config::set_s3_config(s3_config cfg) { set_value(s3_config_, cfg); }
 
 void app_config::set_sia_config(sia_config cfg) { set_value(sia_config_, cfg); }
 
-template <typename dest, typename source>
-auto app_config::set_value(dest &dst, const source &src) -> bool {
-  if (dst.load() == src) {
-    return false;
-  }
-
-  dst = src;
-  config_changed_ = true;
-  save();
-  return true;
-}
-
 auto app_config::set_value_by_name(const std::string &name,
                                    const std::string &value) -> std::string {
   REPERTORY_USES_FUNCTION_NAME();