2

Quiero crear un utilitario para imprimir desde NodeJS.

Tratando de sincronizarlo llegué al código que muestro más adelante, pero la corrida indica:

callback is not a function

Es como si, a pesar de usarla con await, pretendiera evaluar el callback.

En algún momento separé ambas funciones (create y toFile) pero me daba el mismo error.

¿Tengo que pasarlas por el promisify a juro?

¿Estoy haciendo algo mal?

Por cierto: a pesar de dar el error, la rutina funciona (escribe el pdf con el html enviado como contenido), el problema es que no llega al cliente, que se queda esperando la respuesta.

// Módulo de Impresión para Node
var   impr;
async function imprime(texto,res) {
  let hora    = new Date();
  let nbRep   = "Asist_"+hora.getTime();
  let retorno = nbRep;

  if(!impr) {
    impr = require("./rogImpr.json");
    impr.pdf = require('html-pdf');
  }

  try {
    let salida = await impr.pdf.create(
      impr.cabecera + (typeof texto === "string" ? texto : texto.texto) + impr.pie,
      impr.prms
    ).toFile('./'+nbRep+'.pdf');
    retorno = salida.fileName;
  } catch (err) {
    console.log(err);
    retorno = { fallo: true, msj: err.message }
  } finally {
    res.send(retorno)        
  }
}

exports.imprime = imprime;

2 respuestas 2

1

Debes tomar en cuenta que la librería html-pdf tiene mucho tiempo sin ser actualizada (2 años a la fecha de esta respuesta). Por lo tanto te sugiero que busques alternativas, ya que esta librería presenta una vulnerabilidad severa.

No es necesario usar promisify, a menos que desees que dicha utilidad se encargue de convertir una función que usa el antiguo sistema de callback por ti.

Te mostraré la forma de hacer lo que necesitas usando tu propia función envoltorio que devuelve una Promesa.

En primer lugar debemos entender lo que sucede cuando se llama al método create(), para ello podemos ver el código de dicho método y darnos cuenta que devuelve un objeto.

Luego, este objeto tiene varios métodos (como podemos ver en su código), entre los cuales se encuentra el método toFile(), que usa una función callback.

Nuestro envoltorio usará el objeto devuelto por la función create y convertirá la respuesta del método toFile en una Promesa. Podríamos escribirlo algo parecido a lo siguiente:

const pdfFile = (objPDF, path) => {
  return new Promise((resolve, reject) => {
    objPDF.toFile(path, (err, result) => {
      if(err) {
        console.log(err.message);
        reject(err);
      }
      console.log(result); // { filename: 'ruta/nombre_archivo.pdf' }
      resolve(result);
    });
  });
}

Esto sería la función que usaríamos para obtener el resultado del método toFile usando async/await, ya que la misma devuelve una Promesa.

Una forma de implementarlo en tu módulo:

// Módulo de Impresión para Node

let impr; // no uses var, es preferible let o const

// declaramos nuestra función envoltorio
const pdfFile = (PDF, path) => {
  return new Promise((resolve, reject) => {
    PDF.toFile(path, (err, result) => {
      if(err) {
        return reject(err);
      }
      return resolve(result);
    });
  });
}

async function imprime(texto,res) {
  let hora    = new Date();
  let nbRep   = `Asist_${hora.getTime()}`; // es mejor usar string literals en vez de concatenar strings, es más intuitivo
  let retorno = nbRep;

  if(!impr) {
    impr = require("./rogImpr.json");
    impr.pdf = require('html-pdf');
  }

  // instanciamos el objeto PDF
  let pdfObj = await impr.pdf.create(
    impr.cabecera + (typeof texto === "string" ? texto : texto.texto) + impr.pie,
    impr.prms
  );

  // trabajamos con la Promesa
  try {
    let salida = await pdfFile(pdfObj, `./${nbRep}.pdf`); // devuelve una Promesa y podemos usar await
    retorno = salida.fileName;
  } catch (err) {
    console.log(err);
    retorno = { fallo: true, msj: err.message }
  } finally {
    res.send(retorno)        
  }
}

exports.imprime = imprime;

Espero que esto aclare tus dudas.

0

Te falta el Callback que va en el metodo toFile en esa linea:

).toFile('./'+nbRep+'.pdf');

En la documentación está un ejemplo:

pdf.create(html, options).toFile('./businesscard.pdf', function(err, res) {
  if (err) return console.log(err);
  console.log(res); // { filename: '/app/businesscard.pdf' }
});

Documentacion de html-pdf

1
  • Ok: ¿Entonces el await va contra el create pero no contra el toFile?
    – Rafa Gomez
    Commented el 5 dic. 2019 a las 19:18

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.