fix: return model_id instead of error if model is already downloaded (#1524)

vansangpfiev · sangjanai · web-flow · commit fcee871ffbfc · 2024-10-21T14:13:38.000+07:00
* fix: print message if model is already downloaded

* fix: format

---------

Co-authored-by: vansangpfiev &lt;sang@jan.ai&gt;
diff --git a/engine/services/model_service.cc b/engine/services/model_service.cc
@@ -316,7 +316,7 @@ cpp::result<std::string, std::string> ModelService::HandleUrl(
 
   if (model_entry.has_value()) {
     CLI_LOG("Model already downloaded: " << unique_model_id);
-    return cpp::fail("Please delete the model before downloading again");
+    return unique_model_id;
   }
 
   auto local_path{file_manager_utils::GetModelsContainerPath() /

Original file line number	Diff line number	Diff line change
`@@ -316,7 +316,7 @@ cpp::result<std::string, std::string> ModelService::HandleUrl(`
`316`	`316`
`317`	`317`	`if (model_entry.has_value()) {`
`318`	`318`	`CLI_LOG("Model already downloaded: " << unique_model_id);`
`319`		`- return cpp::fail("Please delete the model before downloading again");`
	`319`	`+ return unique_model_id;`
`320`	`320`	`}`
`321`	`321`
`322`	`322`	`auto local_path{file_manager_utils::GetModelsContainerPath() /`