From 6a345d172158b7d9df6daebfcc796f686227284d Mon Sep 17 00:00:00 2001
From: Eduardo Mathias <ems19@inf.ufpr.br>
Date: Tue, 11 Jul 2023 10:29:29 -0300
Subject: [PATCH] [FIX] Create and Update problems
---
src/libs/models/publication.js | 7 +++++++
src/libs/models/user.js | 20 ++++++++++++++++----
src/libs/routes_v1/user.js | 10 ++++++++--
3 files changed, 31 insertions(+), 6 deletions(-)
diff --git a/src/libs/models/publication.js b/src/libs/models/publication.js
index 5b324e72..b750f95e 100644
--- a/src/libs/models/publication.js
+++ b/src/libs/models/publication.js
@@ -14,6 +14,10 @@ var Publication = db.define("Publication",{
allowNull:false,
validate: {
notNull: { msg: "O campo categoria é obrigatória e aceita apenas os valores 'Artigo', 'Tese', 'Dissertação', 'Relatório', 'Periódico."},
+ isIn:{
+ args: [["Artigo", "Tese", "Dissertação", "Relatório", "Periódico"]],
+ msg: "O campo categoria é obrigatória e aceita apenas os valores 'Artigo', 'Tese', 'Dissertação', 'Relatório', 'Periódico'."
+ }
}
},
title:{
@@ -37,6 +41,9 @@ var Publication = db.define("Publication",{
allowNull:false,
validate: {
notNull: { msg: "O campo origem é obrigatória e aceita apenas os valores 'Baixar', 'Acessar'."},
+ isIn:{
+ args: [["Baixar", "Acessar"]],
+ msg: "O campo origem é obrigatória e aceita apenas os valores 'Baixar', 'Acessar'."}
}
},
link:{
diff --git a/src/libs/models/user.js b/src/libs/models/user.js
index 044b36b9..405909eb 100644
--- a/src/libs/models/user.js
+++ b/src/libs/models/user.js
@@ -18,6 +18,7 @@ var User = db.define("User",{
unique: true,
validate: {
notNull: { msg: "O campo Email é obrigatório." },
+ notEmpty: { msg: "O campo Email é obrigatório." }
}
},
password:{
@@ -36,6 +37,7 @@ var User = db.define("User",{
allowNull: false,
validate: {
notNull: { msg: "O campo Nome é obrigatório." },
+ notEmpty: { msg: "O campo Nome é obrigatório." }
}
},
nickname:{
@@ -43,6 +45,7 @@ var User = db.define("User",{
allowNull: false,
validate: {
notNull: { msg: "O campo Apelido é obrigatório." },
+ notEmpty: { msg: "O campo Apelido é obrigatório." }
}
},
cpf:{
@@ -51,6 +54,7 @@ var User = db.define("User",{
unique: true,
validate: {
notNull: { msg: "O campo CPF é obrigatório." },
+ notEmpty: { msg: "O campo CPF é obrigatório." }
}
},
cep:{
@@ -58,6 +62,7 @@ var User = db.define("User",{
allowNull: false,
validate: {
notNull: { msg: "O campo CEP é obrigatório." },
+ notEmpty: { msg: "O campo CEP é obrigatório." }
}
},
schooling:{
@@ -65,6 +70,7 @@ var User = db.define("User",{
allowNull: false,
validate: {
notNull: { msg: "O campo Formação é obrigatório." },
+ notEmpty: { msg: "O campo Formação é obrigatório." }
}
},
course:{
@@ -84,6 +90,7 @@ var User = db.define("User",{
allowNull: false,
validate: {
notNull: { msg: "O campo Segmento é obrigatório." },
+ notEmpty: { msg: "O campo Segmento é obrigatório." }
}
},
role:{
@@ -91,6 +98,7 @@ var User = db.define("User",{
allowNull: false,
validate: {
notNull: { msg: "O campo Função é obrigatório." },
+ notEmpty: { msg: "O campo Função é obrigatório." }
}
},
institution_name:{
@@ -98,6 +106,7 @@ var User = db.define("User",{
allowNull: false,
validate: {
notNull: { msg: "O campo Instituição em que trabalha ou estuda é obrigatório." },
+ notEmpty: { msg: "O campo Instituição em que trabalha ou estuda é obrigatório." }
}
},
state:{
@@ -105,6 +114,7 @@ var User = db.define("User",{
allowNull: false,
validate: {
notNull: { msg: "O campo Estado é obrigatório." },
+ notEmpty: { msg: "O campo Estado é obrigatório." }
}
},
city:{
@@ -112,6 +122,7 @@ var User = db.define("User",{
allowNull: false,
validate: {
notNull: { msg: "O campo Cidade é obrigatório." },
+ notEmpty: { msg: "O campo Cidade é obrigatório." }
}
},
receive_email:{
@@ -126,6 +137,7 @@ var User = db.define("User",{
allowNull:false,
validate: {
notNull: { msg: "O campo origem é obrigatória e aceita apenas os valores 'LDE', 'SimCAQ' e 'MAPFOR'."},
+ notEmpty: { msg: "O campo origem é obrigatória e aceita apenas os valores 'LDE', 'SimCAQ' e 'MAPFOR'."}
}
},
verified:{
@@ -169,10 +181,10 @@ User.generateObjectId = function(){
}
const setSaltAndPassword = user => {
- // if (user.changed('password')) {
- user.salt = User.generateSalt()
- user.hashed_password = User.encryptPassword(user.password, user.salt).toString('hex');
- // }
+ if (user.changed('password')) {
+ user.salt = User.generateSalt()
+ user.hashed_password = User.encryptPassword(user.password, user.salt).toString('hex');
+ }
}
const setObjectId = user => {
diff --git a/src/libs/routes_v1/user.js b/src/libs/routes_v1/user.js
index 2f92370b..d35d47f4 100644
--- a/src/libs/routes_v1/user.js
+++ b/src/libs/routes_v1/user.js
@@ -108,6 +108,10 @@ userApp.get('/me', passport.authenticate('bearer', { session: false }), (req, re
next();
}, response('user'));
+function isEmpty(str) {
+ return (!str || str.length === 0 );
+}
+
userApp.get('/:id', (req, res, next) => {
User.findByPk(req.params.id).then((user) => {
if (!user) {
@@ -136,7 +140,8 @@ userApp.post('/', async (req, res, next) => {
if (typeof req.body.password === 'undefined' || !req.body.password) {
res.statusCode = 400;
return res.json({ errors: ["O campo senha é obrigatório"] });
- } else {
+ }
+ else {
let user = await User.create({
id: 0,
email: req.body.email,
@@ -240,7 +245,8 @@ userApp.put('/:id', passport.authenticate('bearer', { session: false }), async (
user.citesegment = req.body.citesegment || user.citesegment;
user.citerole = req.body.citerole || user.citerole;
- if ((req.body.password != "") && (req.body.newpassword != "")) {
+ if((!isEmpty(req.body.password)) && (!isEmpty(req.body.newpassword))){
+ console.log("ENTREI INFELIZMENTE")
if (req.body.password != req.body.newpassword) {
if (user.checkPassword(user, req.body.password)) {
await user.update({password:req.body.newpassword});
--
GitLab