extract common behavior

include/providers/base_provider.hpp

@@ -91,6 +91,10 @@ public:
                                               std::string &api_path) const
       -> api_error override;
 
+  [[nodiscard]] auto get_file_size(const std::string &api_path,
+                                   std::uint64_t &file_size) const
+      -> api_error override;
+
   [[nodiscard]] auto get_filesystem_item(const std::string &api_path,
                                          bool directory,
                                          filesystem_item &fsi) const

include/providers/s3/s3_provider.hpp

@@ -81,10 +81,6 @@ public:
   [[nodiscard]] auto get_file_list(api_file_list &list) const
       -> api_error override;
 
-  [[nodiscard]] auto get_file_size(const std::string &api_path,
-                                   std::uint64_t &file_size) const
-      -> api_error override;
-
   [[nodiscard]] auto get_total_drive_space() const -> std::uint64_t override;
 
   [[nodiscard]] auto get_provider_type() const -> provider_type override {

include/providers/sia/sia_provider.hpp

@@ -69,10 +69,6 @@ public:
   [[nodiscard]] auto get_file_list(api_file_list &list) const
       -> api_error override;
 
-  [[nodiscard]] auto get_file_size(const std::string &api_path,
-                                   std::uint64_t &file_size) const
-      -> api_error override;
-
   [[nodiscard]] auto get_provider_type() const -> provider_type override {
     return provider_type::sia;
   }

src/providers/base_provider.cpp

@@ -146,6 +146,27 @@ auto base_provider::get_api_path_from_source(const std::string &source_path,
   return api_error::item_not_found;
 }
 
+auto base_provider::get_file_size(const std::string &api_path,
+                                  std::uint64_t &file_size) const -> api_error {
+  bool exists{};
+  auto res = is_directory(api_path, exists);
+  if (res != api_error::success) {
+    return res;
+  }
+  if (exists) {
+    return api_error::directory_exists;
+  }
+
+  api_file file{};
+  res = get_file(api_path, file);
+  if (res != api_error::success) {
+    return res;
+  }
+
+  file_size = file.file_size;
+  return api_error::success;
+}
+
 auto base_provider::get_filesystem_item(const std::string &api_path,
                                         bool directory,
                                         filesystem_item &fsi) const

src/providers/s3/s3_provider.cpp

@@ -589,27 +589,6 @@ auto s3_provider::get_file_list(api_file_list &list) const -> api_error {
   return api_error::success;
 }
 
-auto s3_provider::get_file_size(const std::string &api_path,
-                                std::uint64_t &file_size) const -> api_error {
-  bool exists{};
-  auto res = is_directory(api_path, exists);
-  if (res != api_error::success) {
-    return res;
-  }
-  if (exists) {
-    return api_error::directory_exists;
-  }
-
-  api_file file{};
-  res = get_file(api_path, file);
-  if (res != api_error::success) {
-    return res;
-  }
-
-  file_size = file.file_size;
-  return api_error::success;
-}
-
 auto s3_provider::get_object_info(bool directory, const std::string &api_path,
                                   bool &is_encrypted, std::string &object_name,
                                   head_object_result &result) const

src/providers/sia/sia_provider.cpp

@@ -429,27 +429,6 @@ auto sia_provider::get_file_list(api_file_list &list) const -> api_error {
   return get_files_in_dir("");
 }
 
-auto sia_provider::get_file_size(const std::string &api_path,
-                                 std::uint64_t &file_size) const -> api_error {
-  bool exists{};
-  auto res = is_directory(api_path, exists);
-  if (res != api_error::success) {
-    return res;
-  }
-  if (exists) {
-    return api_error::directory_exists;
-  }
-
-  api_file file{};
-  res = get_file(api_path, file);
-  if (res != api_error::success) {
-    return res;
-  }
-
-  file_size = file.file_size;
-  return api_error::success;
-}
-
 auto sia_provider::get_total_drive_space() const -> std::uint64_t {
   try {
     curl::requests::http_get get{};