0

Hola me inicio en este mundo y tengo un arreglo json (lista) con el cuan necesito actualizar una coleccion llanada User a traves de metodo findOneAndUpdate de mongo, pero el codigo siemrre genera error y honestamente no se en que falla:

async function actualizarProceso(lista) {

  const arregloDePromesas = []

  lista.foreach((elemento) => {


    var promesa = User.findOneAndUpdate({
      "_id": elemento._id
    }, {
      $set: {
        "pasarela": elemento.previo.pasarela,
        "fecha_fin": elemento.previo.fecha_fin,
        "dispositivos": elemento.previo.dispositivos,
        "estado_token": elemento.previo.estado_token
      }
    }).exec()

    arregloDePromesas.push(promesa)
  })

  //Cuando ya tienes todas las promesas listas las ejecutas asi:

  return Promise.all(arregloDePromesas)
    .then(datos => {
      resolve(datos)
    })
    .catch(err => console.log(err))
}

el error es: ReferenceError: resole is not defined, lo que si es siempre entra a reject por o que siemrpe da error, imagino porque la funcion est mal hecha.

disculpen mis errores, gracias de antemano.

2
  • puedes poner el log del error?
    – hawks
    Commented el 30 abr. 2020 a las 14:40
  • gracias @hawks por tu tiempo
    – rrodo1310
    Commented el 30 abr. 2020 a las 14:44

2 respuestas 2

0

Primero, con una función async/await no tienes resolve ni reject porque cualquier datos que devuelvas con return automaticamente se resolverá como una promesa. Y para errores puedes usar return Promise.reject(err);

Segundo, como haces el return de Promise.all(arregloDePromesas) no hace falta que añadas el then y catch ahí. Lo puedes añadir en la función que llama actualizarProceso.

Osea así.

async function actualizarProceso(lista) {

  const arregloDePromesas = []

  lista.foreach((elemento) => {

    var promesa = User.findOneAndUpdate({
      "_id": elemento._id
    }, {
      $set: {
        "pasarela": elemento.previo.pasarela,
        "fecha_fin": elemento.previo.fecha_fin,
        "dispositivos": elemento.previo.dispositivos,
        "estado_token": elemento.previo.estado_token
      }
    }).exec()

    arregloDePromesas.push(promesa)
  })

  //Cuando ya tienes todas las promesas listas las ejecutas asi:
  return Promise.all(arregloDePromesas);
}

function miFuncion() {
  const lista = [] // lo que sea
  actualizarProceso(lista)
    .then(datos => {
      // aqui tratas la response como quieras
      console.log(datos)
    })
    .catch(err => console.log(err))
} 
1
  • 1
    Gracias @haks, disculpa mi ignoracia en el tema, estoy iniciando....gracias por tu tiempo...
    – rrodo1310
    Commented el 30 abr. 2020 a las 15:23
0

el error era de estructura aqui, el codigo que me funciona por si alguien lo necesita:

function actualizarProceso(lista) {

        const arregloDePromesas = []

        lista.map((elemento) =>{


              var promesa = User.findOneAndUpdate( { "_id" : elemento._id },
              { $set: {  "pasarela" : elemento.previo.pasarela, "fecha_fin" : elemento.previo.fecha_fin, "dispositivos" : elemento.previo.dispositivos, "estado_token" : elemento.previo.estado_token    }}
              ).exec()

              arregloDePromesas.push(promesa)
       })

       //Cuando ya tienes todas las promesas listas las ejecutas asi:

       return Promise.all(arregloDePromesas).then(datos=>{ return(datos)})
        .catch(err=>console.log(err))

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.