From 25272cbaef16c3114523005acd707843145bb762 Mon Sep 17 00:00:00 2001 From: Lemuel Marquez Date: Tue, 18 May 2021 22:40:30 -0500 Subject: [PATCH] archivo helper --- server/controller/Equipo/cargaMasiva.js | 4 +- server/controller/Equipo/equipo.js | 4 +- server/controller/Equipo/equipos.js | 2 +- server/controller/Operador/crear.js | 2 +- server/controller/Operador/login.js | 2 +- server/controller/Operador/operadores.js | 2 +- server/controller/Prestamo/activos.js | 2 +- server/controller/Prestamo/cancelar.js | 4 +- server/controller/Prestamo/historial.js | 2 +- server/controller/Prestamo/historialEquipo.js | 2 +- .../controller/Prestamo/historialUsuario.js | 2 +- server/controller/Prestamo/pedir.js | 12 +++--- server/controller/Prestamo/prestamo.js | 4 +- server/controller/Prestamo/prestamoUsuario.js | 4 +- server/controller/Prestamo/recoger.js | 6 +-- server/controller/Prestamo/regresar.js | 6 +-- server/controller/Prestamo/status.js | 4 +- server/controller/Usuario/crear.js | 16 +------- server/controller/Usuario/login.js | 2 +- server/controller/Usuario/usuario.js | 2 +- server/controller/Usuario/usuarios.js | 2 +- server/helper/helper.js | 37 +++++++++++++++++++ server/helper/validar.js | 19 ++-------- 23 files changed, 76 insertions(+), 66 deletions(-) create mode 100644 server/helper/helper.js diff --git a/server/controller/Equipo/cargaMasiva.js b/server/controller/Equipo/cargaMasiva.js index 420593d..3128760 100644 --- a/server/controller/Equipo/cargaMasiva.js +++ b/server/controller/Equipo/cargaMasiva.js @@ -1,6 +1,6 @@ -const fs = require('fs'); const csv = require('csvtojson'); const { Op } = require('sequelize'); +const { eliminarArchivo } = require('../../helper/helper'); const dbPath = '../../db/tablas'; const Equipo = require(`${dbPath}/Equipo`); const Carrito = require(`${dbPath}/Carrito`); @@ -103,7 +103,7 @@ const cargaMasiva = async (file) => { delete existeEquipo.dataValues.Carrito.dataValues.idTipoCarrito; equiposNuevos.push(existeEquipo); } - fs.unlinkSync(path); + await eliminarArchivo(path); }); return { message: 'Se subio correctamente el archivo csv.', diff --git a/server/controller/Equipo/equipo.js b/server/controller/Equipo/equipo.js index edd8654..2d7a891 100644 --- a/server/controller/Equipo/equipo.js +++ b/server/controller/Equipo/equipo.js @@ -1,4 +1,4 @@ -const validar = require('../../helper/validar'); +const { validarAlfanumerico } = require('../../helper/validar'); const dbPath = '../../db/tablas'; const Carrito = require(`${dbPath}/Carrito`); const Equipo = require(`${dbPath}/Equipo`); @@ -8,7 +8,7 @@ const Status = require(`${dbPath}/Status`); const TipoCarrito = require(`${dbPath}/TipoCarrito`); const equipo = async (body) => { - let numeroSerie = validar.validarAlfanumerico( + let numeroSerie = validarAlfanumerico( body.numeroSerie, 'El numero de serie', 20 diff --git a/server/controller/Equipo/equipos.js b/server/controller/Equipo/equipos.js index 316bdef..3537c40 100644 --- a/server/controller/Equipo/equipos.js +++ b/server/controller/Equipo/equipos.js @@ -8,7 +8,7 @@ const Status = require(`${dbPath}/Status`); const TipoCarrito = require(`${dbPath}/TipoCarrito`); const equipos = async (body) => { - const pagina = validar.validarNumero(body.pagina); + const pagina = validar.validarNumero(body.pagina, true); // filtros return Equipo.findAndCountAll({ diff --git a/server/controller/Operador/crear.js b/server/controller/Operador/crear.js index b2e3513..c4c359b 100644 --- a/server/controller/Operador/crear.js +++ b/server/controller/Operador/crear.js @@ -3,7 +3,7 @@ const validar = require('../../helper/validar'); const Operador = require('../../db/tablas/Operador'); const crear = async (body) => { - const operador = validar.validar(body.operador, 'El operador', 60); + const operador = validar.validarTexto(body.operador, 'El operador', 60); const password = validar.validarAlfanumerico( body.password, 'La contraseña', diff --git a/server/controller/Operador/login.js b/server/controller/Operador/login.js index 688f3fd..b33d08f 100644 --- a/server/controller/Operador/login.js +++ b/server/controller/Operador/login.js @@ -5,7 +5,7 @@ const Operador = require('../../db/tablas/Operador'); const TipoUsuario = require('../../db/tablas/TipoUsuario'); const login = async (body) => { - const operador = validar.validar(body.operador, 'El operador', 60); + const operador = validar.validarTexto(body.operador, 'El operador', 60); const password = validar.validarAlfanumerico( body.password, 'La contraseña', diff --git a/server/controller/Operador/operadores.js b/server/controller/Operador/operadores.js index 90e67c6..e6dc5b6 100644 --- a/server/controller/Operador/operadores.js +++ b/server/controller/Operador/operadores.js @@ -3,7 +3,7 @@ const Operador = require('../../db/tablas/Operador'); const TipoUsuario = require('../../db/tablas/TipoUsuario'); const operadores = async (body) => { - const pagina = validar.validarNumero(body.pagina); + const pagina = validar.validarNumero(body.pagina, true); // filtros return Operador.findAndCountAll({ diff --git a/server/controller/Prestamo/activos.js b/server/controller/Prestamo/activos.js index 09707a5..333e6af 100644 --- a/server/controller/Prestamo/activos.js +++ b/server/controller/Prestamo/activos.js @@ -11,7 +11,7 @@ const Usuario = require(`${dbPath}/Usuario`); const activos = async (body) => { const idModulo = validar.validarId(body.idModulo); - // const pagina = validar.validarNumero(body.pagina); + // const pagina = validar.validarNumero(body.pagina, true); // filtros return Modulo.findOne({ where: { idModulo } }) diff --git a/server/controller/Prestamo/cancelar.js b/server/controller/Prestamo/cancelar.js index 5a23658..16aa136 100644 --- a/server/controller/Prestamo/cancelar.js +++ b/server/controller/Prestamo/cancelar.js @@ -1,10 +1,10 @@ -const validar = require('../../helper/validar'); +const { validarId } = require('../../helper/validar'); const dbPath = '../../db/tablas'; const Prestamo = require(`${dbPath}/Prestamo`); const Equipo = require(`${dbPath}/Equipo`); const cancelar = async (body) => { - const idPrestamo = validar.validarId(body.idPrestamo); + const idPrestamo = validarId(body.idPrestamo); return Prestamo.findOne({ where: { idPrestamo }, diff --git a/server/controller/Prestamo/historial.js b/server/controller/Prestamo/historial.js index 574028d..719103d 100644 --- a/server/controller/Prestamo/historial.js +++ b/server/controller/Prestamo/historial.js @@ -11,7 +11,7 @@ const TipoUsuario = require(`${dbPath}/TipoUsuario`); const Usuario = require(`${dbPath}/Usuario`); const historial = async (body) => { - const pagina = validar.validarNumero(body.pagina); + const pagina = validar.validarNumero(body.pagina, true); // filtros return Prestamo.findAndCountAll({ diff --git a/server/controller/Prestamo/historialEquipo.js b/server/controller/Prestamo/historialEquipo.js index d599784..97cbf07 100644 --- a/server/controller/Prestamo/historialEquipo.js +++ b/server/controller/Prestamo/historialEquipo.js @@ -9,7 +9,7 @@ const Usuario = require(`${dbPath}/Usuario`); const historialEquipo = async (body) => { const idEquipo = validar.validarId(body.idEquipo); - const pagina = validar.validarNumero(body.pagina); + const pagina = validar.validarNumero(body.pagina, true); return Prestamo.findAndCountAll({ include: [ diff --git a/server/controller/Prestamo/historialUsuario.js b/server/controller/Prestamo/historialUsuario.js index 54e9f0d..c7ce0e1 100644 --- a/server/controller/Prestamo/historialUsuario.js +++ b/server/controller/Prestamo/historialUsuario.js @@ -11,7 +11,7 @@ const Usuario = require(`${dbPath}/Usuario`); const historialUsuario = async (body) => { const idUsuario = validar.validarId(body.idUsuario); - const pagina = validar.validarNumero(body.pagina); + const pagina = validar.validarNumero(body.pagina, true); return Prestamo.findAndCountAll({ include: [ diff --git a/server/controller/Prestamo/pedir.js b/server/controller/Prestamo/pedir.js index 1919700..7a359cb 100644 --- a/server/controller/Prestamo/pedir.js +++ b/server/controller/Prestamo/pedir.js @@ -1,5 +1,5 @@ const moment = require('moment'); -const validar = require('../../helper/validar'); +const { validarId } = require('../../helper/validar'); const dbPath = '../../db/tablas'; const Prestamo = require(`${dbPath}/Prestamo`); const Usuario = require(`${dbPath}/Usuario`); @@ -12,12 +12,10 @@ const Modulo = require(`${dbPath}/Modulo`); const pedir = async (body) => { const ahora = moment(); - const idUsuario = validar.validarId(body.idUsuario); - const idTipoCarrito = validar.validarId(body.idTipoCarrito); - const idPrograma = body.idPrograma - ? validar.validarId(body.idPrograma) - : null; - const idModulo = idPrograma ? 1 : validar.validarId(body.idModulo); + const idUsuario = validarId(body.idUsuario); + const idTipoCarrito = validarId(body.idTipoCarrito); + const idPrograma = body.idPrograma ? validarId(body.idPrograma) : null; + const idModulo = idPrograma ? 1 : validarId(body.idModulo); let idEquipo; return TipoCarrito.findOne({ where: { idTipoCarrito } }) diff --git a/server/controller/Prestamo/prestamo.js b/server/controller/Prestamo/prestamo.js index 173d181..751a569 100644 --- a/server/controller/Prestamo/prestamo.js +++ b/server/controller/Prestamo/prestamo.js @@ -1,4 +1,4 @@ -const validar = require('../../helper/validar'); +const { validarId } = require('../../helper/validar'); const dbPath = '../../db/tablas'; const Carrera = require(`${dbPath}/Carrera`); const Carrito = require(`${dbPath}/Carrito`); @@ -12,7 +12,7 @@ const TipoUsuario = require(`${dbPath}/TipoUsuario`); const Usuario = require(`${dbPath}/Usuario`); const prestamo = async (body) => { - const idPrestamo = validar.validarId(body.idPrestamo); + const idPrestamo = validarId(body.idPrestamo); return Prestamo.findOne({ where: { idPrestamo }, diff --git a/server/controller/Prestamo/prestamoUsuario.js b/server/controller/Prestamo/prestamoUsuario.js index 3998852..85f11d7 100644 --- a/server/controller/Prestamo/prestamoUsuario.js +++ b/server/controller/Prestamo/prestamoUsuario.js @@ -1,4 +1,4 @@ -const validar = require('../../helper/validar'); +const { validarId } = require('../../helper/validar'); const dbPath = '../../db/tablas'; const Carrito = require(`${dbPath}/Carrito`); const Equipo = require(`${dbPath}/Equipo`); @@ -7,7 +7,7 @@ const Prestamo = require(`${dbPath}/Prestamo`); const TipoCarrito = require(`${dbPath}/TipoCarrito`); const prestamo = async (body) => { - const idUsuario = validar.validarId(body.idUsuario); + const idUsuario = validarId(body.idUsuario); return Prestamo.findOne({ where: { idUsuario, activo: true }, diff --git a/server/controller/Prestamo/recoger.js b/server/controller/Prestamo/recoger.js index aadff59..476ec9f 100644 --- a/server/controller/Prestamo/recoger.js +++ b/server/controller/Prestamo/recoger.js @@ -1,5 +1,5 @@ const moment = require('moment'); -const validar = require('../../helper/validar'); +const { validarId } = require('../../helper/validar'); const dbPath = '../../db/tablas'; const Carrito = require(`${dbPath}/Carrito`); const Equipo = require(`${dbPath}/Equipo`); @@ -10,8 +10,8 @@ const TipoCarrito = require(`${dbPath}/TipoCarrito`); const recoger = async (body) => { const ahora = moment(); - const idPrestamo = validar.validarId(body.idPrestamo); - const idOperadorEntrega = validar.validarId(body.idOperadorEntrega); + const idPrestamo = validarId(body.idPrestamo); + const idOperadorEntrega = validarId(body.idOperadorEntrega); let idEquipo; return Operador.findOne({ where: { idOperador: idOperadorEntrega } }) diff --git a/server/controller/Prestamo/regresar.js b/server/controller/Prestamo/regresar.js index 37a59b5..34de2c7 100644 --- a/server/controller/Prestamo/regresar.js +++ b/server/controller/Prestamo/regresar.js @@ -1,13 +1,13 @@ const moment = require('moment'); -const validar = require('../../helper/validar'); +const { validarId } = require('../../helper/validar'); const dbPath = '../../db/tablas'; const Prestamo = require(`${dbPath}/Prestamo`); const Equipo = require(`${dbPath}/Equipo`); const Operador = require(`${dbPath}/Operador`); const regresar = async (body) => { - const idPrestamo = validar.validarId(body.idPrestamo); - const idOperadorRegreso = validar.validarId(body.idOperadorRegreso); + const idPrestamo = validarId(body.idPrestamo); + const idOperadorRegreso = validarId(body.idOperadorRegreso); const ahora = moment(); return Operador.findOne({ where: { idOperador: idOperadorRegreso } }) diff --git a/server/controller/Prestamo/status.js b/server/controller/Prestamo/status.js index 5b6775a..080a49d 100644 --- a/server/controller/Prestamo/status.js +++ b/server/controller/Prestamo/status.js @@ -1,12 +1,12 @@ const moment = require('moment'); -const validar = require('../../helper/validar'); +const { validarId } = require('../../helper/validar'); const dbPath = '../../db/tablas'; const Equipo = require(`${dbPath}/Equipo`); const Prestamo = require(`${dbPath}/Prestamo`); const status = async (body) => { const ahora = moment(); - const idPrestamo = validar.validarId(body.idPrestamo); + const idPrestamo = validarId(body.idPrestamo); let idEquipo; return Prestamo.findOne({ diff --git a/server/controller/Usuario/crear.js b/server/controller/Usuario/crear.js index 5ee6415..f92294d 100644 --- a/server/controller/Usuario/crear.js +++ b/server/controller/Usuario/crear.js @@ -3,20 +3,6 @@ const encriptar = require('../../helper/encriptar'); const validar = require('../../helper/validar'); const Usuario = require('../../db/tablas/Usuario'); -const crear = async (body) => { - const usuario = validar.validar(body.usuario, 'El operador', 60); - const password = encriptar.generarPassword(); - - return Usuario.findOne({ where: { usuario } }) - .then((res) => { - if (res) throw new Error('Este usuario ya existe. Trata con otro.'); - return Operador.create({ - operador, - password: encriptar.encriptar(password), - idTipoUsuario: 2, - }); - }) - .then((res) => ({ message: 'Se creo un nuevo usuario correctamente.' })); -}; +const crear = async (body) => {}; module.exports = crear; diff --git a/server/controller/Usuario/login.js b/server/controller/Usuario/login.js index 61df87f..0d80630 100644 --- a/server/controller/Usuario/login.js +++ b/server/controller/Usuario/login.js @@ -6,7 +6,7 @@ const Usuario = require(`${dbPath}/Usuario`); const TipoUsuario = require(`${dbPath}/TipoUsuario`); const login = async (body) => { - const usuario = validar.validar(body.usuario, 'El usuario', 60); + const usuario = validar.validarNumeroCuenta(body.usuario, 'El usuario', 60); const password = validar.validarAlfanumerico( body.password, 'La contraseña', diff --git a/server/controller/Usuario/usuario.js b/server/controller/Usuario/usuario.js index e5a49d5..7a2b884 100644 --- a/server/controller/Usuario/usuario.js +++ b/server/controller/Usuario/usuario.js @@ -15,7 +15,7 @@ const usuario = async (body) => { delete res.dataValues.password; delete res.dataValues.idTipoUsuario; delete res.dataValues.idCarrera; - if (!res.Carrera) delete res.dataValues.Carrera; + if (res.Carrera) delete res.dataValues.Carrera; return res; }); }; diff --git a/server/controller/Usuario/usuarios.js b/server/controller/Usuario/usuarios.js index 9915c69..496d668 100644 --- a/server/controller/Usuario/usuarios.js +++ b/server/controller/Usuario/usuarios.js @@ -5,7 +5,7 @@ const Carrera = require(`${dbPath}/Carrera`); const TipoUsuario = require(`${dbPath}/TipoUsuario`); const usuarios = async (body) => { - const pagina = validar.validarNumero(body.pagina); + const pagina = validar.validarNumero(body.pagina, true); // filtros return Usuario.findAndCountAll({ diff --git a/server/helper/helper.js b/server/helper/helper.js new file mode 100644 index 0000000..f269aa5 --- /dev/null +++ b/server/helper/helper.js @@ -0,0 +1,37 @@ +const fs = require('fs'); +const moment = require('moment'); + +const crearDDMMAAAA = (date) => { + const fechaMoment = moment(date); + let ddmmaaaa = ''; + + if (fechaMoment.isValid()) + ddmmaaaa = `${fechaMoment.date()}/${ + fechaMoment.month() + 1 + }/${fechaMoment.year()}`; + return ddmmaaaa; +}; + +const eliminarArchivo = (path) => { + return new Promise((resolve, reject) => { + fs.unlink(path, (err) => { + if (err) reject(err); + resolve({ message: 'Se elimino el archivo correctamente.' }); + }); + }); +}; + +const crearArchivo = (path, texto) => { + return new Promise((resolve, reject) => { + fs.appendFile(path, texto, (err) => { + if (err) reject(err); + resolve({ message: 'Se creo el archivo correctamente.' }); + }); + }); +}; + +module.exports = { + crearDDMMAAAA, + eliminarArchivo, + crearArchivo, +}; diff --git a/server/helper/validar.js b/server/helper/validar.js index ea07b15..28bd0ea 100644 --- a/server/helper/validar.js +++ b/server/helper/validar.js @@ -103,7 +103,7 @@ const validarFecha = (fecha) => { return fechaMoment; }; -const validarNumero = (numero, length) => { +const validarNumero = (numero, makeNumber = false, length) => { let campo = 'El numero'; noHay(numero, campo); @@ -113,6 +113,7 @@ const validarNumero = (numero, length) => { (length && numero.length > length) ) noValido(campo); + if (makeNumber) return Number(numero); return numero; }; @@ -122,24 +123,13 @@ const validarAccesibilidad = (usuariosValidos = [], idTipoUsuario) => { throw new Error('Este usuario no tiene permitido usar esta linea de la API.'); }; -const isObjectEmpty = (obj) => { +const validarObjetoVacio = (obj) => { for (let key in obj) { if (obj.hasOwnProperty(key)) return false; } return true; }; -const crearDDMMAAAA = (date) => { - const fechaMoment = moment(date); - let ddmmaaaa = ''; - - if (fechaMoment.isValid()) - ddmmaaaa = `${fechaMoment.date()}/${ - fechaMoment.month() + 1 - }/${fechaMoment.year()}`; - return ddmmaaaa; -}; - module.exports = { validar, noValido, @@ -153,6 +143,5 @@ module.exports = { validarFecha, validarNumero, validarAccesibilidad, - isObjectEmpty, - crearDDMMAAAA, + validarObjetoVacio, };