diff --git a/api/v1/pregunta.js b/api/v1/pregunta.js index ce17d03..89198da 100644 --- a/api/v1/pregunta.js +++ b/api/v1/pregunta.js @@ -40,9 +40,6 @@ router.get("/", (req, res) => { }); }); -function editarPregunta(req, res, respuestaIA = null) { -} - router.patch("/", function (req, res) { Pregunta.findByPk(req.body.ID, { include: [ @@ -54,6 +51,7 @@ router.patch("/", function (req, res) { ], }) .then((pregunta) => { + // TODO Refactor: Usar early return en vez de anidar tanto if-else if (!pregunta) { res.status(404).send("Pregunta no encontrada"); return; @@ -248,7 +246,7 @@ router.put('/:ID', function (req, res) { // ,{model:ReportePost,include:{model:TipoReporte, as:'tipo',attributes:[]}} ] }, - { model: SuscripcionesPregunta, as: 'suscripciones' }, + { model: SuscripcionesPregunta, as: 'suscripciones'}, { model: Respuesta, as: 'respuestas', include: Post }, // TODO Refactor: Algún ENUM de tipos de reportes ] @@ -271,7 +269,10 @@ router.put('/:ID', function (req, res) { esperarA.push(pre.post.setEliminador(usuarioActual.DNI).then(p => p.save())); // TODO Feature: Test this. - esperarA.push(pre.getSuscripciones().then(suscripciones => Promise.all(suscripciones.map(suscripcion => suscripcion.setPregunta(preguntaReemplazoID).then(s => s.save()))))); + esperarA.push(pre.getSuscripciones().then(suscripciones => Promise.all(suscripciones.map(suscripcion =>{ + suscripcion.preguntaID=preguntaReemplazoID; + return suscripcion.save(); + })))); // TODO Feature: ¿Pasar reportes de otras índoles? // esperarA.push(pre.post.getReportePosts().then(reportes=>Promise.all(reportes.map(reporte => reporte.tipoID==2?/* ! 2 es pregunta repetida */reporte.destroy()))));