Fix adoption report photos and section order

backend/src/reports/adoptionReport.ts

@@ -2,15 +2,15 @@ import fs from 'fs';
 import PDFDocument from 'pdfkit';
 import QRCode from 'qrcode';
 
-import type { BirdRow, FlockNoteRow, VetVisitRow, WeightRow, WorkspaceRow } from '../types.js';
+import type { BirdRow, FlockNoteRow, VetVisitRow, WeightRow } from '../types.js';
 
 type AdoptionReportInput = {
   bird: BirdRow;
   weights: WeightRow[];
   vetVisits: VetVisitRow[];
   notes: FlockNoteRow[];
-  workspace: WorkspaceRow;
   transferCode: string;
+  birdPhotoBuffer?: Buffer | null;
   assets: {
     logoPath: string;
     wordmarkPath: string;
@@ -256,8 +256,8 @@ export const renderAdoptionReportPdf = async ({
   weights,
   vetVisits,
   notes,
-  workspace,
   transferCode,
+  birdPhotoBuffer = null,
   assets,
   printFriendly = false,
 }: AdoptionReportInput) => {
@@ -275,7 +275,7 @@ export const renderAdoptionReportPdf = async ({
   const logoPath = fs.existsSync(assets.logoPath) ? assets.logoPath : null;
   const wordmarkPath = fs.existsSync(assets.wordmarkPath) ? assets.wordmarkPath : logoPath;
   const defaultPhotoPath = fs.existsSync(assets.defaultBirdPhotoPath) ? assets.defaultBirdPhotoPath : null;
-  const photoBuffer = dataUrlToBuffer(bird.photo_data_url);
+  const photoBuffer = birdPhotoBuffer ?? dataUrlToBuffer(bird.photo_data_url);
   const contentWidth = page.width - page.margin * 2;
   const headerY = page.margin;
   const headerHeight = 136;
@@ -346,21 +346,6 @@ export const renderAdoptionReportPdf = async ({
   );
   y += 72;
 
-  y = drawSectionTitle(doc, 'Weight Graph', y);
-  drawSimpleWeightChart(doc, weights, bird.chart_color, page.margin, y, contentWidth, 120);
-  y += 140;
-
-  y = drawSectionTitle(doc, 'Weight History', y);
-  y = drawTable(
-    doc,
-    ['Date', 'Weight', 'Notes'],
-    weights.length ? weights.map((entry) => [formatDate(entry.recorded_on), formatWeight(entry.weight_grams), entry.notes || '']) : [['No weights recorded.', '', '']],
-    page.margin,
-    y,
-    [95, 70, contentWidth - 165],
-    24,
-  );
-
   if (y > 610) {
     doc.addPage();
     y = page.margin;
@@ -388,6 +373,25 @@ export const renderAdoptionReportPdf = async ({
     28,
   );
 
+  if (y > 575) {
+    doc.addPage();
+    y = page.margin;
+  }
+  y = drawSectionTitle(doc, 'Weight Graph', y);
+  drawSimpleWeightChart(doc, weights, bird.chart_color, page.margin, y, contentWidth, 120);
+  y += 140;
+
+  y = drawSectionTitle(doc, 'Weight History', y);
+  y = drawTable(
+    doc,
+    ['Date', 'Weight', 'Notes'],
+    weights.length ? weights.map((entry) => [formatDate(entry.recorded_on), formatWeight(entry.weight_grams), entry.notes || '']) : [['No weights recorded.', '', '']],
+    page.margin,
+    y,
+    [95, 70, contentWidth - 165],
+    24,
+  );
+
   if (notes.length) {
     if (y > 635) {
       doc.addPage();