Update api/routes.py

api/routes.py

@@ -22,7 +22,7 @@ import uuid
 import re
 import subprocess
 from tempfile import NamedTemporaryFile, TemporaryDirectory
-from string import Template  # Added for LaTeX templating
+from string import Template
 
 # Configure logging
 logging.basicConfig(
@@ -569,8 +569,9 @@ async def add_note(
 
 @router.get("/ehr/patients/{patient_id}/pdf", response_class=Response)
 async def generate_patient_pdf(patient_id: str, current_user: dict = Depends(get_current_user)):
-    # Disable logging for this route by setting the logger's level to CRITICAL
-    logger.setLevel(logging.CRITICAL)
+    # Temporarily enable logging to debug LaTeX compilation issues
+    original_log_level = logger.level
+    logger.setLevel(logging.DEBUG)
 
     try:
         if current_user.get('role') not in ['doctor', 'admin']:
@@ -587,42 +588,42 @@ async def generate_patient_pdf(patient_id: str, current_user: dict = Depends(get
         if not patient:
             raise HTTPException(status_code=404, detail="Patient not found")
 
-        # Format rows safely using escape_latex_special_chars
+        # Format rows safely using escape_latex_special_chars with additional safety for None values
         note_rows = " \\\\\n".join(
             "{} & {} & {}".format(
-                escape_latex_special_chars(n.get("date", "")),
-                escape_latex_special_chars(n.get("type", "")),
-                escape_latex_special_chars(n.get("text", ""))
+                escape_latex_special_chars(n.get("date", "") or ""),
+                escape_latex_special_chars(n.get("type", "") or ""),
+                escape_latex_special_chars(n.get("text", "") or "")
             )
             for n in patient.get("notes", [])
         )
         condition_rows = " \\\\\n".join(
             "{} & {} & {} & {} & {}".format(
-                escape_latex_special_chars(c.get("id", "")),
-                escape_latex_special_chars(c.get("code", "")),
-                escape_latex_special_chars(c.get("status", "")),
-                escape_latex_special_chars(c.get("onset_date", "")),
-                escape_latex_special_chars(c.get("verification_status", ""))
+                escape_latex_special_chars(c.get("id", "") or ""),
+                escape_latex_special_chars(c.get("code", "") or ""),
+                escape_latex_special_chars(c.get("status", "") or ""),
+                escape_latex_special_chars(c.get("onset_date", "") or ""),
+                escape_latex_special_chars(c.get("verification_status", "") or "")
             )
             for c in patient.get("conditions", [])
         )
         medication_rows = " \\\\\n".join(
             "{} & {} & {} & {} & {}".format(
-                escape_latex_special_chars(m.get("id", "")),
-                escape_latex_special_chars(m.get("name", "")),
-                escape_latex_special_chars(m.get("status", "")),
-                escape_latex_special_chars(m.get("prescribed_date", "")),
-                escape_latex_special_chars(m.get("dosage", ""))
+                escape_latex_special_chars(m.get("id", "") or ""),
+                escape_latex_special_chars(m.get("name", "") or ""),
+                escape_latex_special_chars(m.get("status", "") or ""),
+                escape_latex_special_chars(m.get("prescribed_date", "") or ""),
+                escape_latex_special_chars(m.get("dosage", "") or "")
             )
             for m in patient.get("medications", [])
         )
         encounter_rows = " \\\\\n".join(
             "{} & {} & {} & {} & {}".format(
-                escape_latex_special_chars(e.get("id", "")),
-                escape_latex_special_chars(e.get("type", "")),
-                escape_latex_special_chars(e.get("status", "")),
-                escape_latex_special_chars(e.get("period", {}).get("start", "")),
-                escape_latex_special_chars(e.get("service_provider", ""))
+                escape_latex_special_chars(e.get("id", "") or ""),
+                escape_latex_special_chars(e.get("type", "") or ""),
+                escape_latex_special_chars(e.get("status", "") or ""),
+                escape_latex_special_chars(e.get("period", {}).get("start", "") or ""),
+                escape_latex_special_chars(e.get("service_provider", "") or "")
             )
             for e in patient.get("encounters", [])
         )
@@ -704,15 +705,15 @@ $encounters
 \end{document}
 """)
 
-        # Set the generated_on date to 03:42 PM CET, May 16, 2025
-        generated_on = datetime.strptime("2025-05-16 15:42:00+01:00", "%Y-%m-%d %H:%M:%S%z").strftime("%A, %B %d, %Y at %I:%M %p")
+        # Set the generated_on date to 03:46 PM CET, May 16, 2025
+        generated_on = datetime.strptime("2025-05-16 15:46:00+01:00", "%Y-%m-%d %H:%M:%S%z").strftime("%A, %B %d, %Y at %I:%M %p")
 
         latex_filled = latex_template.substitute(
             generated_on=generated_on,
-            fhir_id=escape_latex_special_chars(patient.get("fhir_id", "")),
-            full_name=escape_latex_special_chars(patient.get("full_name", "")),
-            gender=escape_latex_special_chars(patient.get("gender", "")),
-            dob=escape_latex_special_chars(patient.get("date_of_birth", "")),
+            fhir_id=escape_latex_special_chars(patient.get("fhir_id", "") or ""),
+            full_name=escape_latex_special_chars(patient.get("full_name", "") or ""),
+            gender=escape_latex_special_chars(patient.get("gender", "") or ""),
+            dob=escape_latex_special_chars(patient.get("date_of_birth", "") or ""),
             age=escape_latex_special_chars(str(calculate_age(patient.get("date_of_birth", "")) or "N/A")),
             address=escape_latex_special_chars(", ".join(filter(None, [
                 patient.get("address", ""),
@@ -721,8 +722,8 @@ $encounters
                 patient.get("postal_code", ""),
                 patient.get("country", "")
             ]))),
-            marital_status=escape_latex_special_chars(patient.get("marital_status", "")),
-            language=escape_latex_special_chars(patient.get("language", "")),
+            marital_status=escape_latex_special_chars(patient.get("marital_status", "") or ""),
+            language=escape_latex_special_chars(patient.get("language", "") or ""),
             notes=note_rows or "No notes available \\\\",
             conditions=condition_rows or "No conditions available \\\\",
             medications=medication_rows or "No medications available \\\\",
@@ -738,20 +739,25 @@ $encounters
                 f.write(latex_filled)
 
             try:
-                subprocess.run(
-                    ["latexmk", "-pdf", "-interaction=nonstopmode", tex_path],
+                # Added -f to force compilation for debugging
+                result = subprocess.run(
+                    ["latexmk", "-pdf", "-interaction=nonstopmode", "-f", tex_path],
                     cwd=tmpdir,
                     check=True,
                     capture_output=True,
                     text=True
                 )
+                logger.debug(f"latexmk stdout: {result.stdout}")
+                logger.debug(f"latexmk stderr: {result.stderr}")
             except subprocess.CalledProcessError as e:
+                logger.error(f"LaTeX compilation failed: stdout={e.stdout}, stderr={e.stderr}")
                 raise HTTPException(
                     status_code=500,
-                    detail=f"LaTeX compilation failed: {e.stderr}"
+                    detail=f"LaTeX compilation failed: stdout={e.stdout}, stderr={e.stderr}"
                 )
 
             if not os.path.exists(pdf_path):
+                logger.error("PDF file was not generated")
                 raise HTTPException(
                     status_code=500,
                     detail="PDF file was not generated"
@@ -768,15 +774,17 @@ $encounters
             return response
 
     except HTTPException as http_error:
+        logger.error(f"HTTP exception in generate_patient_pdf: {str(http_error.detail)}")
         raise http_error
     except Exception as e:
+        logger.error(f"Unexpected error generating PDF: {str(e)}", exc_info=True)
         raise HTTPException(
             status_code=500,
             detail=f"Unexpected error generating PDF: {str(e)}"
         )
     finally:
         # Restore the logger level for other routes
-        logger.setLevel(logging.INFO)
+        logger.setLevel(original_log_level)
 
 @router.post("/signup", status_code=status.HTTP_201_CREATED)
 async def signup(data: SignupForm):