Location: backend/src/modules/surgeries/surgeries.service.ts (around lines 66-69)
Problem:
savePhoto() returns a hardcoded placeholder path string instead of actually handling file storage. Any caller relying on this for real photo uploads will silently get a fake path instead of a stored file (or fail later when something tries to fetch the photo from that fake path).
Why it matters:
Looks complete from the API surface but doesn't work, which is worse than an explicit 'not implemented' error — it can fail silently downstream.
Suggested fix:
- Implement actual file persistence using the existing upload/storage service used by other modules (e.g.
src/modules/upload, src/modules/storage), following the same pattern.
Acceptance criteria:
- Uploaded surgery photos are actually stored and retrievable.
- A test covers the upload + retrieval path.
Location:
backend/src/modules/surgeries/surgeries.service.ts(around lines 66-69)Problem:
savePhoto()returns a hardcoded placeholder path string instead of actually handling file storage. Any caller relying on this for real photo uploads will silently get a fake path instead of a stored file (or fail later when something tries to fetch the photo from that fake path).Why it matters:
Looks complete from the API surface but doesn't work, which is worse than an explicit 'not implemented' error — it can fail silently downstream.
Suggested fix:
src/modules/upload,src/modules/storage), following the same pattern.Acceptance criteria: