cjw/Oxicloud
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix: improve error messages for file/folder already exists

Browse Source
This commit is contained in:
BillionClaw
2026-03-17 17:05:52 +08:00
parent d1699357d8
commit aebd51788b
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/infrastructure/repositories/pg/file_blob_write_repository.rs
+4 -4
View File
@@ -257,7 +257,7 @@ impl FileWritePort for FileBlobWriteRepository {
{ {
return Err(DomainError::already_exists( return Err(DomainError::already_exists(
"File", "File",
format!("{name} already exists in folder"), format!("'{name}' already exists in this folder"),
)); ));
} }
return Err(DomainError::internal_error( return Err(DomainError::internal_error(
@@ -382,7 +382,7 @@ impl FileWritePort for FileBlobWriteRepository {
{ {
return DomainError::already_exists( return DomainError::already_exists(
"File", "File",
"File with that name already exists in target folder".to_string(), "a file with this name already exists in the target folder",
); );
} }
DomainError::internal_error("FileBlobWrite", format!("copy: {e}")) DomainError::internal_error("FileBlobWrite", format!("copy: {e}"))
@@ -440,7 +440,7 @@ impl FileWritePort for FileBlobWriteRepository {
if let sqlx::Error::Database(ref db_err) = e if let sqlx::Error::Database(ref db_err) = e
&& db_err.code().as_deref() == Some("23505") && db_err.code().as_deref() == Some("23505")
{ {
return DomainError::already_exists("File", format!("{new_name} already exists")); return DomainError::already_exists("File", format!("'{new_name}' already exists"));
} }
DomainError::internal_error("FileBlobWrite", format!("rename: {e}")) DomainError::internal_error("FileBlobWrite", format!("rename: {e}"))
})? })?
@@ -632,7 +632,7 @@ impl FileWritePort for FileBlobWriteRepository {
if db_err.code().as_deref() == Some("23505") { if db_err.code().as_deref() == Some("23505") {
return DomainError::already_exists( return DomainError::already_exists(
"Folder", "Folder",
"A folder with that name already exists in the target".to_string(), "a folder with this name already exists in the target location",
); );
} }
} }