New change

evidence_pack_export.py

@@ -3,6 +3,22 @@ from fpdf import FPDF
 import tempfile
 import os
 
+def _safe_text_for_pdf(s: object) -> str:
+    """
+    Ensure text passed to FPDF only contains characters encodable in latin-1.
+    FPDF (classic) writes using latin-1 encoding by default and will raise
+    an error when encountering characters outside that range (e.g. •).
+    This helper replaces unsupported characters with a best-effort replacement
+    so PDF generation doesn't fail.
+    """
+    if s is None:
+        return ""
+    try:
+        text = str(s)
+    except Exception:
+        text = ""
+    # encode -> decode using latin-1 with replacement to avoid exceptions
+    return text.encode('latin-1', 'replace').decode('latin-1')
 def export_evidence_pack_pdf(data, filename=None):
     """
     Export evidence pack as PDF. Data should include clause, summary, checklist, scenario, metadata.
@@ -19,7 +35,7 @@ def export_evidence_pack_pdf(data, filename=None):
     pdf.cell(0, 8, txt="Clause:", ln=True)
     pdf.set_font("Arial", size=10)
     clause_text = data.get('clause_text', 'No clause information available')
-    pdf.multi_cell(0, 6, clause_text)
+    pdf.multi_cell(0, 6, _safe_text_for_pdf(clause_text))
     pdf.ln(3)
     
     # Summary section  
@@ -27,7 +43,7 @@ def export_evidence_pack_pdf(data, filename=None):
     pdf.cell(0, 8, txt="Summary:", ln=True)
     pdf.set_font("Arial", size=10)
     summary_text = data.get('summary', 'No summary available')
-    pdf.multi_cell(0, 6, summary_text)
+    pdf.multi_cell(0, 6, _safe_text_for_pdf(summary_text))
     pdf.ln(3)
     
     # Checklist section
@@ -37,7 +53,8 @@ def export_evidence_pack_pdf(data, filename=None):
     checklist = data.get('role_checklist', [])
     if checklist:
         for item in checklist:
-            pdf.multi_cell(0, 6, f"• {item}")
+            # use a simple hyphen bullet and sanitize text
+            pdf.multi_cell(0, 6, _safe_text_for_pdf(f"- {item}"))
     else:
         pdf.multi_cell(0, 6, "No checklist items available")
     pdf.ln(3)
@@ -46,27 +63,27 @@ def export_evidence_pack_pdf(data, filename=None):
     pdf.set_font("Arial", "B", size=12)
     pdf.cell(0, 8, txt="Source Information:", ln=True) 
     pdf.set_font("Arial", size=10)
-    pdf.multi_cell(0, 6, f"Source: {data.get('source_title', 'Not specified')}")
-    pdf.multi_cell(0, 6, f"Clause ID: {data.get('clause_id', 'Not assigned')}")
-    pdf.multi_cell(0, 6, f"Date: {data.get('date', 'Not specified')}")
-    pdf.multi_cell(0, 6, f"URL: {data.get('url', 'Not available')}")
+    pdf.multi_cell(0, 6, _safe_text_for_pdf(f"Source: {data.get('source_title', 'Not specified')}"))
+    pdf.multi_cell(0, 6, _safe_text_for_pdf(f"Clause ID: {data.get('clause_id', 'Not assigned')}"))
+    pdf.multi_cell(0, 6, _safe_text_for_pdf(f"Date: {data.get('date', 'Not specified')}"))
+    pdf.multi_cell(0, 6, _safe_text_for_pdf(f"URL: {data.get('url', 'Not available')}"))
     
     # User role and context information
     if data.get('user_role'):
-        pdf.multi_cell(0, 6, f"User Role: {data.get('user_role', '').title()}")
+        pdf.multi_cell(0, 6, _safe_text_for_pdf(f"User Role: {data.get('user_role', '').title()}"))
     if data.get('language_preference'):
-        pdf.multi_cell(0, 6, f"Language: {data.get('language_preference', '').title()}")
+        pdf.multi_cell(0, 6, _safe_text_for_pdf(f"Language: {data.get('language_preference', '').title()}"))
     
     pdf.ln(5)
     scenario = data.get('scenario_analysis',{})
     if scenario:
-        pdf.multi_cell(0, 8, f"Scenario Analysis:")
-        pdf.multi_cell(0, 8, f"Yearly Results: {scenario.get('yearly_results','')}")
-        pdf.multi_cell(0, 8, f"Cumulative Base: {scenario.get('cumulative_base','')}")
-        pdf.multi_cell(0, 8, f"Cumulative Scenario: {scenario.get('cumulative_scenario','')}")
-        pdf.multi_cell(0, 8, f"Optimistic: {scenario.get('optimistic','')}")
-        pdf.multi_cell(0, 8, f"Pessimistic: {scenario.get('pessimistic','')}")
-        pdf.multi_cell(0, 8, f"Driver Breakdown: {scenario.get('driver_breakdown','')}")
+        pdf.multi_cell(0, 8, _safe_text_for_pdf(f"Scenario Analysis:"))
+        pdf.multi_cell(0, 8, _safe_text_for_pdf(f"Yearly Results: {scenario.get('yearly_results','')}"))
+        pdf.multi_cell(0, 8, _safe_text_for_pdf(f"Cumulative Base: {scenario.get('cumulative_base','')}"))
+        pdf.multi_cell(0, 8, _safe_text_for_pdf(f"Cumulative Scenario: {scenario.get('cumulative_scenario','')}"))
+        pdf.multi_cell(0, 8, _safe_text_for_pdf(f"Optimistic: {scenario.get('optimistic','')}"))
+        pdf.multi_cell(0, 8, _safe_text_for_pdf(f"Pessimistic: {scenario.get('pessimistic','')}"))
+        pdf.multi_cell(0, 8, _safe_text_for_pdf(f"Driver Breakdown: {scenario.get('driver_breakdown','')}"))
     if not filename:
         filename = os.path.join(tempfile.gettempdir(), f"evidence_pack_{os.getpid()}.pdf")
     pdf.output(filename)