Actualizar server.js
This commit is contained in:
@@ -232,23 +232,24 @@ app.delete("/statuses/:id", authMiddleware, async (req, res) => {
|
|||||||
const usageCount = parseInt(check.rows[0].count);
|
const usageCount = parseInt(check.rows[0].count);
|
||||||
|
|
||||||
if (usageCount > 0) {
|
if (usageCount > 0) {
|
||||||
|
// Devolvemos 400 Bad Request con un mensaje claro
|
||||||
return res.status(400).json({
|
return res.status(400).json({
|
||||||
ok: false,
|
ok: false,
|
||||||
error: `No se puede borrar: Este estado se usa en ${usageCount} servicios.`
|
error: `No se puede borrar: Este estado se usa en ${usageCount} servicios.`
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
// 2. Si no se usa, borrarlo
|
// 2. Si no se usa, lo borramos tranquilamente
|
||||||
const del = await client.query("DELETE FROM service_statuses WHERE id=$1 AND owner_id=$2", [statusId, accountId]);
|
const del = await client.query("DELETE FROM service_statuses WHERE id=$1 AND owner_id=$2", [statusId, accountId]);
|
||||||
|
|
||||||
if (del.rowCount === 0) {
|
if (del.rowCount === 0) {
|
||||||
return res.status(404).json({ ok: false, error: "Estado no encontrado o no autorizado" });
|
return res.status(404).json({ ok: false, error: "Estado no encontrado" });
|
||||||
}
|
}
|
||||||
|
|
||||||
res.json({ ok: true });
|
res.json({ ok: true });
|
||||||
} catch(e) {
|
} catch(e) {
|
||||||
console.error("Error borrando estado:", e);
|
console.error("Error borrando estado:", e);
|
||||||
res.status(500).json({ ok: false, error: "Error interno" });
|
res.status(500).json({ ok: false, error: "Error interno del servidor" });
|
||||||
} finally {
|
} finally {
|
||||||
client.release();
|
client.release();
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user