From 6b072ceb84e37054f12dce7398bfdcadf20b26ba Mon Sep 17 00:00:00 2001
From: Lemuel Marquez <lemuel@iMac-de-Lemuel.local>
Date: Thu, 20 May 2021 11:15:48 -0500
Subject: [PATCH] cargamasiva mejorada

---
 .../Usuario/cargaMasivaPosgrados.js           | 29 ++++++++-----------
 1 file changed, 12 insertions(+), 17 deletions(-)

diff --git a/server/controller/Usuario/cargaMasivaPosgrados.js b/server/controller/Usuario/cargaMasivaPosgrados.js
index b1a4fdd..5378e16 100644
--- a/server/controller/Usuario/cargaMasivaPosgrados.js
+++ b/server/controller/Usuario/cargaMasivaPosgrados.js
@@ -1,5 +1,5 @@
 const csv = require('csvtojson');
-const { Op } = require('sequelize');
+const { validarObjetoVacio } = require('../../helper/validar');
 const { eliminarArchivo } = require('../../helper/helper');
 const dbPath = '../../db/tablas';
 const Usuario = require(`${dbPath}/Usuario`);
@@ -51,14 +51,11 @@ const cargaMasiva = async (file) => {
                 )
               );
           } else {
+            let update = {};
+
             if (!usuario.activo) {
-              await Usuario.update(
-                { activo: true },
-                { where: { idUsuario: usuario.idUsuario } }
-              );
-              alumnosUpdate.push(
-                `El alumno con id ${usuario.idUsuario} y numero de cuenta ${usuario.usuario} fue activado.`
-              );
+              update.activo = true;
+              alumnosUpdate.push(`El alumno ${usuario.usuario} fue activado.`);
             }
             if (usuario.Carrera.carrera != alumnos[i].nomplan) {
               await Carrera.findOne({
@@ -74,12 +71,9 @@ const cargaMasiva = async (file) => {
                   return res;
                 })
                 .then((res) => {
+                  update.idCarrera = res.idCarrera;
                   alumnosUpdate.push(
-                    `Se cambio la carrera del alumno con id ${usuario.idUsuario} y numero de cuenta ${usuario.usuario} de ${usuario.Carrera.carrera} por ${res.carrera}.`
-                  );
-                  return Usuario.update(
-                    { idCarrera: res.idCarrera },
-                    { where: { idUsuario: usuario.idUsuario } }
+                    `Se cambio la carrera del alumno ${usuario.usuario} de ${usuario.Carrera.carrera} por ${res.carrera}.`
                   );
                 });
             }
@@ -88,14 +82,15 @@ const cargaMasiva = async (file) => {
                 `El alumno con id ${usuario.idUsuario} y numero de cuenta ${usuario.usuario} tambien es profesor.`
               );
             if (usuario.idTipoUsuario === 4) {
+              update.idTipoUsuario = 5;
               alumnosUpdate.push(
                 `Se cambio el tipo de usuario alumno con id ${usuario.idUsuario} y numero de cuenta ${usuario.usuario}.`
               );
-              await Usuario.update(
-                { idTipoUsuario: 5 },
-                { where: { idUsuario: usuario.idUsuario } }
-              );
             }
+            if (!validarObjetoVacio)
+              await Usuario.update(update, {
+                where: { idUsuario: usuario.idUsuario },
+              });
           }
         });
       }