tuanta/warehouse-management-BE
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(delete): update delete operations to return affected rows for cabinets, rooms, and warehouses

Browse Source
This commit is contained in:
Tran Anh Tuan
2026-05-08 18:16:40 +07:00
parent 4382c53aac
commit 8562d39071
3 changed files with 13 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
internal/services/cabinet_service.go
View File

@@ -173,11 +173,11 @@ func CabinetDelete(c *gin.Context) error {
}
rowsAffected, err := repositories.DeleteCabinet(c.Request.Context(), global.Queries, id)
if err != nil {
if rowsAffected == 0 {
response.NotFoundError(c, http.StatusNotFound, "Cabinet not found")
response.InternalServerError(c, http.StatusInternalServerError, "Failed to delete cabinet")
return nil
}
response.InternalServerError(c, http.StatusInternalServerError, "Failed to delete cabinet")
if rowsAffected == 0 {
response.NotFoundError(c, http.StatusNotFound, "Cabinet not found")
return nil
}

6
internal/services/room_service.go
View File

@@ -175,11 +175,11 @@ func RoomDelete(c *gin.Context) error {
rowsAffected, err := repositories.DeleteRoom(c.Request.Context(), global.Queries, id)
if err != nil {
log.Error().Err(err).Msgf("Failed to delete room with ID: %d", id)
if rowsAffected == 0 {
response.NotFoundError(c, http.StatusNotFound, "Room not found")
response.InternalServerError(c, http.StatusInternalServerError, "Failed to delete room")
return nil
}
response.InternalServerError(c, http.StatusInternalServerError, "Failed to delete room")
if rowsAffected == 0 {
response.NotFoundError(c, http.StatusNotFound, "Room not found")
return nil
}
response.Ok(c, "Delete Success", nil)

7
internal/services/warehouse_service.go
View File

@@ -165,11 +165,12 @@ func WareHouseDelete(c *gin.Context) error {
rowsAffected, err := repositories.DeleteWarehouse(c.Request.Context(), global.Queries, id)
if err != nil {
log.Error().Err(err).Msgf("Failed to delete warehouse with ID: %d", id)
if rowsAffected == 0 {
response.NotFoundError(c, http.StatusNotFound, "Warehouse not found")
response.InternalServerError(c, http.StatusInternalServerError, "Failed to delete warehouse")
return nil
}
response.InternalServerError(c, http.StatusInternalServerError, "Failed to delete warehouse")
if rowsAffected == 0 {
response.NotFoundError(c, http.StatusNotFound, "Warehouse not found")
return nil
}
response.Ok(c, "Delete Success", nil)