//function for getting old xp of user var oldxpuser; function old_Xp(UID){ return new Promise(function (resolve, reject) { redmine.users(UID,function(err,data) { if (!err){ for (var i=0; i < data.users.length; i++) { if(data.users[i].id == UID){ oldxpuser = parseInt(data.users[i].custom_fields[0].value); console.log("old xp: "+ oldxpuser); } } resolve(oldxpuser); } else{ reject(err); } }) }) } //after old xp get then update will run old_Xp().then(function () { update_Xp(8) }).catch(function(error) { console.log("Error occured ERROR:" + error); }) old_Xp(8); ////////////////////////////////////////////////////////// function update_Xp(UID){ redmine.update_user(UID, updated_XP ,function(err, data) { if (err) console.log(err); console.log('Updated !!'); }); console.log("ME: your promise has been resolved"); } ////////////////////////////////////////////////////////// // console.log(oldxpuser); let new_xp = oldxpuser+50; console.log( parseInt(new_xp)); let updated_XP= { user: { custom_fields: [{id: 1 ,value: new_xp}], //id is important } } ////////////////////////////////////////////////////////// // update_Xp(8); //accepts user ID check_XP();en este código anterior, principalmente creé 2 funciones old_xp y update_xp, así que quiero que se ejecute primero old_xp para poder obtener el valor anterior del usuario y luego se ejecutará update_xp pero no funciona así var oldxpuser muestra un error de NaN o ( solo ejecuta update_xp primero para no poder obtener oldxpuser) ingrese la descripción de la imagen aquí
Veo algunos posibles problemas aquí.
primero Cuando llama a old_XP() primera vez que no pasa un uid , eso no le permite configurar oldxpuser más tarde, ya que la identificación no está definida.
Entonces, la solución es agregar uid 8 allí también.
old_Xp(8).then(function () { update_Xp(8) }) segundo update_Xp usa new_xp que es una variable síncrona, pero esa variable síncrona depende del async oldxpuser , lo que significa que en el momento en que crea la variable new_xp , el oldxpuser aún no está definido (se configurará solo cuando se resuelva la promesa old_XP ).
La solución aquí es generar un objeto updated_XP dinámicamente, como aquí:
redmine.update_user(UID, getUpdatedXp() ,function(err, data) {}); function getUpdatedXp() { const new_xp = oldxpuser + 50; console.log("updatedXP: " + new_xp); return { user: { custom_fields: [{ id: 1, value: new_xp }], } } } //////////////////////////////////////////////////////////Ejemplo:
//function for getting old xp of user var oldxpuser; var usersData = [ { id: 8, custom_fields: [{ value: 8 }] }, { id: 1, custom_fields: [{ value: 1 }] }, ]; var redmine = { users: (uid, callback) => { const users = usersData.filter(user => user.id === uid); setTimeout(() => callback(null, { users }), 100); }, update_user: (uid, newUsersData, callback) => { const userData = usersData.find(user => user.id === uid); userData.custom_fields[0].value = newUsersData.user.custom_fields[0].value; setTimeout(() => callback(null, { userData }), 100); }, }; function old_Xp(UID){ return new Promise(function (resolve, reject) { redmine.users(UID,function(err,data) { if (!err){ for (var i=0; i < data.users.length; i++) { if(data.users[i].id == UID){ oldxpuser = parseInt(data.users[i].custom_fields[0].value); console.log("old xp: "+ oldxpuser); } } resolve(oldxpuser); } else{ reject(err); } }) }) } //after old xp get then update will run old_Xp(8).then(function () { update_Xp(8) }).catch(function(error) { console.log("Error occured ERROR:" + error); }); old_Xp(8); ////////////////////////////////////////////////////////// function update_Xp(UID){ redmine.update_user(UID, getUpdatedXp() ,function(err, data) { if (err) console.log(err); console.log('Updated !!'); }); console.log("ME: your promise has been resolved"); } ////////////////////////////////////////////////////////// // console.log(oldxpuser); let new_xp = oldxpuser+50; console.log( parseInt(new_xp)); let updated_XP= { user: { custom_fields: [{id: 1 ,value: new_xp}], //id is important } } function getUpdatedXp() { const new_xp = oldxpuser + 50; console.log("updatedXP: " + new_xp); return { user: { custom_fields: [{ id: 1, value: new_xp }], } } } //////////////////////////////////////////////////////////