mejora código

server/controller/Multa/multar.js

@@ -1,4 +1,5 @@
 const moment = require('moment');
+const { Op } = require('sequelize');
 const validar = require('../../helper/validar');
 const dbPath = '../../db/tablas';
 const Infraccion = require(`${dbPath}/Infraccion`);
@@ -13,11 +14,6 @@ const multar = async (body) => {
     'id operador multa',
     true
   );
-  const idUsuario = validar.validarNumeroEntero(
-    body.idUsuario,
-    'id usuario',
-    true
-  );
   const idInfraccion = validar.validarNumeroEntero(
     body.idInfraccion,
     'id infracción',
@@ -25,7 +21,7 @@ const multar = async (body) => {
   );
   const idPrestamo = validar.validarNumeroEntero(
     body.idPrestamo,
-    'id prestamo',
+    'id préstamo',
     true
   );
   const descripcion = validar.validarAlfanumerico(
@@ -34,8 +30,9 @@ const multar = async (body) => {
     false,
     500
   );
-  const fechaFin = moment();
+  let fechaFin = moment();
   let prestamo = {};
+  let gravedad = '';
 
   return Operador.findOne({ where: { idOperador: idOperadorMulta } })
     .then((res) => {
@@ -44,7 +41,9 @@ const multar = async (body) => {
     })
     .then((res) => {
       if (!res) throw new Error('No existe esta infracción.');
-      if (res.gravedad == '1') fechaFin.add(7, 'd');
+      // Checar si tiene una multa activa, agragar días a multa
+      gravedad = res.gravedad;
+      if (gravedad === '1') fechaFin.add(7, 'd');
       else fechaFin.add(100, 'y');
       return Prestamo.findOne({
         where: { idPrestamo },
@@ -52,39 +51,41 @@ const multar = async (body) => {
       });
     })
     .then((res) => {
-      if (!res) throw new Error('No existe este prestamo.');
+      if (!res) throw new Error('No existe este préstamo.');
       prestamo = res;
-      return Usuario.findOne({ where: { idUsuario } });
-    })
-    .then((res) => {
-      if (!res) throw new Error('No existe este usuario.');
-      return Prestamo.findOne({ where: { idUsuario, activo: true } });
+      return Prestamo.findOne({
+        where: { idUsuario: prestamo.idUsuario, activo: true },
+      });
     })
     .then((res) => {
       if (res)
         throw new Error(
-          'No se puede multar a este usuario ya que tiene un prestamo activo.'
+          'No se puede multar a este usuario ya que tiene un préstamo activo.'
         );
-      return Multa.findOne({ where: { idPrestamo } });
+      return Multa.findOne({
+        where: { idPrestamo, idInfraccion: { [Op.ne]: 1 } },
+      });
     })
     .then((res) => {
       if (res)
-        throw new Error('A este prestamo ya se le ha asignado una multa.');
+        throw new Error(
+          'Este préstamo ya ha sido usado para multar a un usuario.'
+        );
       return Usuario.update(
         { multa: true },
-        { where: { idUsuario: prestamo.Usuario.idUsuario } }
+        { where: { idUsuario: prestamo.idUsuario } }
       );
     })
-    .then((res) =>
+    .then(() =>
       Multa.create({
         descripcion,
-        fechaFin: fechaFin.format(),
+        fechaFin: fechaFin.format('YYYY-MM-DD'),
         idOperadorMulta,
         idInfraccion,
         idPrestamo,
       })
     )
-    .then((res) => ({
+    .then(() => ({
       message: `Se levantó correctamente el reporte de multa.`,
     }));
 };