Update app.py

app.py

@@ -774,9 +774,6 @@ PHYSICAL ACTIVITY/EXERCISE:
 - [Recommended physical activities, duration, frequency, limitations, and progression]
 SYMPTOM MANAGEMENT:
 - [Detailed instructions for managing specific symptoms (e.g., pain, nausea, shortness of breath), non-pharmacological interventions]
-RED FLAGS / WHEN TO SEEK HELP:
-- [Clear, actionable instructions on symptoms requiring immediate medical attention (Emergency - call 911/local emergency, go to ER)]
-- [Instructions on symptoms requiring contact with doctor/clinic (Urgent but not Emergency - call office, visit clinic)]
 ADDITIONAL RECOMMENDATIONS:
 - [Other relevant advice, e.g., rest, monitoring specific vital signs, wound care instructions, mental health support, caregiver tips]
 FOLLOW-UP:
@@ -800,7 +797,6 @@ Instructions:
 3. Prioritize addressing the issues raised in the patient feedback.
 4. Be specific, actionable, and realistic in your recommendations (e.g., specify *what* activities, *when* to take medications and *what" medications and diet to follow).
 5. Ensure the language is clear, empathetic, and easy for a patient or caregiver to understand. Avoid overly technical jargon where possible.
-6. Include clear instructions in the "RED FLAGS / WHEN TO SEEK HELP" section, distinguishing between immediate emergency (e.g., call 911/local emergency) and urgent but non-emergency situations (e.g., call doctor's office).
 7. Do NOT include any introductory phrases (like "Here is the updated plan:", "Based on your feedback...") or concluding sentences outside the plan structure. Provide ONLY the structured content that fits within the template.
 8. If the previous care plan was unavailable or unreadable, create the plan based solely on the patient information and feedback, still following the template.
 9. Ensure the plan is medically sound and reflects standard care principles. If feedback indicates a significant change or potential issue, the ASSESSMENT and subsequent sections should clearly address this and recommend appropriate actions (like contacting their doctor for a re-evaluation, even if it's not an immediate emergency).
@@ -1006,7 +1002,6 @@ def send_whatsapp_doctor(patient_id):
 
 
 # --- Existing routes remain below ---
-
 @app.route('/download_pdf/<patient_id>')
 def download_pdf(patient_id):
     logger.info(f"Download requested for patient ID: {patient_id}")
@@ -1026,20 +1021,29 @@ def download_pdf(patient_id):
         )
 
         # Check if the buffer contains the error message text (simple check)
-        buffer_content = pdf_buffer.getvalue().decode('latin-1', errors='ignore') # Decode to check content
+        # Using latin-1 and ignore for decoding potentially non-UTF8 PDF content
+        buffer_content = pdf_buffer.getvalue().decode('latin-1', errors='ignore')
         if "Error Generating Care Plan PDF" in buffer_content:
             logger.error(f"PDF generation failed for ID {patient_id}, returning error PDF.")
+            # --- FIX START ---
+            # Extract the re.sub call out of the f-string expression
+            safe_name_error = re.sub(r'[^a-zA-Z0-9_\-]', '', patient.name or 'patient').lower()
+            # Use the variable in the f-string
+            download_name_error = f"care_plan_{safe_name_error}_error.pdf"
+            # --- FIX END ---
+
             # Send the error PDF, but indicate it's an error
             return send_file(
                 io.BytesIO(pdf_buffer.getvalue()), # Need to create a new buffer from the value
                 as_attachment=True,
-                download_name=f"care_plan_{re.sub(r'[^a-zA-Z0-9_\-]', '', patient.name or 'patient').lower()}_error.pdf",
+                download_name=download_name_error, # Use the corrected variable here
                 mimetype='application/pdf'
             )
 
 
         pdf_buffer.seek(0) # Reset buffer position after checking content
 
+        # This part was already correct
         safe_name = re.sub(r'[^a-zA-Z0-9_\-]', '', patient.name or 'patient').lower()
         download_name = f"care_plan_{safe_name}_{datetime.now().strftime('%Y%m%d_%H%M%S')}.pdf"
 
@@ -1054,6 +1058,7 @@ def download_pdf(patient_id):
         logger.error(f"PDF Download Error for ID {patient_id}: {str(e)}", exc_info=True)
         # Fallback error response if even generating the basic error PDF fails
         return f"Error generating PDF for download: {str(e)}", 500
+        
 
 
 @app.route('/get_emergency_notifications')