fix: pdf generation for referral reports

pkg/clinical/presentation/rest/handlers.go

@@ -377,18 +377,20 @@ func (p PresentationHandlersImpl) ListQuestionnaire(c *gin.Context) {
 	c.JSON(http.StatusOK, questionnaire)
 }
 
+// GenerateReferralReport handles incoming HTTP requests to generate referral reports in PDF format.
 func (p PresentationHandlersImpl) GenerateReferralReport(c *gin.Context) {
 	queryParams := c.Request.URL.Query()
 	serviceRequestID := queryParams.Get("servicerequest")
 
-	c.Header("Content-Type", "application/pdf")
-	// c.Header("Content-Disposition", "attachment; filename=referral_report.pdf")
-
-	err := p.usecases.GenerateReferralReportPDF(c.Request.Context(), serviceRequestID)
+	pdfBytes, err := p.usecases.GenerateReferralReportPDF(c.Request.Context(), serviceRequestID)
 	if err != nil {
 		jsonErrorResponse(c, http.StatusBadRequest, err)
 		return
 	}
 
-	c.JSON(http.StatusOK, gin.H{"Status": "Ok"})
+	fileName := "referral_report.pdf"
+
+	c.Header("Content-Type", "application/pdf")
+	c.Header("Content-Disposition", fmt.Sprintf("attachment; filename=%s", fileName))
+	c.Data(http.StatusOK, "application/pdf", pdfBytes)
 }

pkg/clinical/usecases/clinical/referral_report.go

@@ -5,11 +5,11 @@ import (
 	"context"
 	"fmt"
 	"html/template"
-	"log"
 	"strings"
 	"time"
 
 	"github.com/SebastiaanKlippert/go-wkhtmltopdf"
+	"github.com/savannahghi/clinical/pkg/clinical/application/utils"
 )
 
 type Patient struct {
@@ -89,19 +89,21 @@ type TemplateData struct {
 // information, to construct a comprehensive referral report. The report is formatted
 // as a PDF, making it suitable for clinical review, record-keeping, or sharing with
 // other healthcare professionals.
-func (c *UseCasesClinicalImpl) GenerateReferralReportPDF(ctx context.Context, serviceRequestID string) error {
+func (c *UseCasesClinicalImpl) GenerateReferralReportPDF(ctx context.Context, serviceRequestID string) ([]byte, error) {
 	if serviceRequestID == "" {
-		return fmt.Errorf("service request ID cannot be empty")
+		return nil, fmt.Errorf("service request ID cannot be empty")
 	}
 
 	serviceRequest, err := c.infrastructure.FHIR.GetFHIRServiceRequest(ctx, serviceRequestID)
 	if err != nil {
-		return err
+		utils.ReportErrorToSentry(err)
+		return nil, err
 	}
 
 	patient, err := c.infrastructure.FHIR.GetFHIRPatient(ctx, *serviceRequest.Resource.Subject.ID)
 	if err != nil {
-		return err
+		utils.ReportErrorToSentry(err)
+		return nil, err
 	}
 
 	age := time.Since(patient.Resource.BirthDate.AsTime()).Hours() / 24 / 365
@@ -159,16 +161,17 @@ func (c *UseCasesClinicalImpl) GenerateReferralReportPDF(ctx context.Context, se
 
 	tmpl, err := template.ParseFiles("templates/referral_report_template.html")
 	if err != nil {
-		log.Print(err)
-		return err
+		utils.ReportErrorToSentry(err)
+		return nil, err
 	}
 
 	// Fill the template with data
 	var htmlBuffer bytes.Buffer
 
 	err = tmpl.Execute(&htmlBuffer, data)
 	if err != nil {
-		log.Fatalf("Error executing template: %v", err)
+		utils.ReportErrorToSentry(err)
+		return nil, err
 	}
 
 	// Convert template output to string
@@ -177,7 +180,8 @@ func (c *UseCasesClinicalImpl) GenerateReferralReportPDF(ctx context.Context, se
 	// Create a new PDF generator
 	pdfg, err := wkhtmltopdf.NewPDFGenerator()
 	if err != nil {
-		return err
+		utils.ReportErrorToSentry(err)
+		return nil, err
 	}
 
 	// Add one page from an URL, a file, or HTML content
@@ -186,13 +190,11 @@ func (c *UseCasesClinicalImpl) GenerateReferralReportPDF(ctx context.Context, se
 	// Create PDF document in internal buffer
 	err = pdfg.Create()
 	if err != nil {
-		return err
+		utils.ReportErrorToSentry(err)
+		return nil, err
 	}
 
-	err = pdfg.WriteFile("referral_report.pdf")
-	if err != nil {
-		return err
-	}
+	pdfBytes := pdfg.Bytes()
 
-	return nil
+	return pdfBytes, nil
 }

pkg/clinical/usecases/clinical/referral_report_test.go

@@ -86,7 +86,7 @@ func TestUseCasesClinicalImpl_GenerateReferralReportPDF(t *testing.T) {
 				}
 			}
 
-			if err := c.GenerateReferralReportPDF(tt.args.ctx, tt.args.serviceRequestID); (err != nil) != tt.wantErr {
+			if _, err := c.GenerateReferralReportPDF(tt.args.ctx, tt.args.serviceRequestID); (err != nil) != tt.wantErr {
 				t.Errorf("UseCasesClinicalImpl.GenerateReferralReportPDF() error = %v, wantErr %v", err, tt.wantErr)
 			}
 		})

templates/referral_report_template.html

@@ -59,7 +59,7 @@ <h2>Patient details</h2>
         </div>
         {{end}}
 
-        {{if .NextOfKin}}
+        {{if or .NextOfKin.Name .NextOfKin.PhoneNumber .NextOfKin.Relationship}}
         <div class="detail-section">
             <h2>Next of kin details</h2>
             {{if .NextOfKin.Name}}<div class="detail"><strong>Name:</strong> {{.NextOfKin.Name}}</div>{{end}}