From d67bcd221272a52445ff02d7a2deaedc91cfb74b Mon Sep 17 00:00:00 2001 From: Suraj Shetty Date: Sun, 5 May 2019 12:23:46 +0530 Subject: [PATCH] fix: Define filedata before it is used --- frappe/utils/pdf.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/frappe/utils/pdf.py b/frappe/utils/pdf.py index 8376d32a93..54b90be6ed 100644 --- a/frappe/utils/pdf.py +++ b/frappe/utils/pdf.py @@ -18,12 +18,14 @@ def get_pdf(html, options=None, output=None): "disable-local-file-access": "", }) + filedata = '' + try: # Set filename property to false, so no file is actually created filedata = pdfkit.from_string(html, False, options=options or {}) # https://pythonhosted.org/PyPDF2/PdfFileReader.html - # create in-memory binary streams from filedata and create a PdfFileReader objcet + # create in-memory binary streams from filedata and create a PdfFileReader object reader = PdfFileReader(io.BytesIO(filedata)) except IOError as e: