@softwear/latestcollectioncore 1.0.177 → 1.0.178

package/dist/pdf.d.ts

@@ -54,7 +54,7 @@ export interface ContainerLayoutObject extends BaseLayoutObject {
     minHeightBeforeBreak?: number;
     /** When true, container and children are drawn only once at the end of the report, not on every page */
     printOnlyAtEnd?: boolean;
-    /** When true, container and children are drawn only once before other body containers (first page header block) */
+    /** Only effective on top-level `layout.objects` (PDF engine ignores this flag on nested containers). */
     printOnlyAtStart?: boolean;
 }
 export type LayoutObject = TextLayoutObject | FieldLayoutObject | RectangleLayoutObject | ImageLayoutObject | ContainerLayoutObject;

package/dist/reports.js

@@ -663,22 +663,26 @@ function genPDF(layout, printBuffer, options = {}) {
         }
         drawStaticPartOfPage(measurementDoc, layout, printBuffer, paperSize, options, measurementContext);
         // Draw containers in measurement mode (same order as render)
+        let measurementTopPrintedStartBottomTenths = 0;
         layout.objects
             .filter((object) => object.type == 'container' && object.printOnlyAtStart)
             .forEach((object) => {
-            addObjectToPDF(0, 0, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext);
+            const lh = addObjectToPDF(0, 0, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext);
+            measurementTopPrintedStartBottomTenths = Math.max(measurementTopPrintedStartBottomTenths, lh.y * 10);
         });
         layout.objects
             .filter((object) => object.type == 'container' &&
             !object.printOnlyAtEnd &&
             !object.printOnlyAtStart)
             .forEach((object) => {
-            addObjectToPDF(0, 0, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext);
+            const bump = Math.max(0, measurementTopPrintedStartBottomTenths - object.y);
+            addObjectToPDF(0, bump, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext);
         });
         layout.objects
             .filter((object) => object.type == 'container' && object.printOnlyAtEnd)
             .forEach((object) => {
-            addObjectToPDF(0, 0, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext);
+            const bump = Math.max(0, measurementTopPrintedStartBottomTenths - object.y);
+            addObjectToPDF(0, bump, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext);
         });
         // Get total page count from measurement pass
         const totalPageCount = measurementContext.currentPageCount || 1;
@@ -697,10 +701,12 @@ function genPDF(layout, printBuffer, options = {}) {
             defaultFontFamily: layout.defaultFontFamily,
         };
         drawStaticPartOfPage(doc, layout, printBuffer, paperSize, options, renderContext);
+        let renderTopPrintedStartBottomTenths = 0;
         layout.objects
             .filter((object) => object.type == 'container' && object.printOnlyAtStart)
             .forEach((object) => {
-            addObjectToPDF(0, 0, doc, object, printBuffer, paperSize, layout, options, printBuffer, [], renderContext);
+            const lh = addObjectToPDF(0, 0, doc, object, printBuffer, paperSize, layout, options, printBuffer, [], renderContext);
+            renderTopPrintedStartBottomTenths = Math.max(renderTopPrintedStartBottomTenths, lh.y * 10);
         });
         // Draw containers in rendering mode (body containers after printOnlyAtStart)
         layout.objects
@@ -708,12 +714,14 @@ function genPDF(layout, printBuffer, options = {}) {
             !object.printOnlyAtEnd &&
             !object.printOnlyAtStart)
             .forEach((object) => {
-            addObjectToPDF(0, 0, doc, object, printBuffer, paperSize, layout, options, printBuffer, [], renderContext);
+            const bump = Math.max(0, renderTopPrintedStartBottomTenths - object.y);
+            addObjectToPDF(0, bump, doc, object, printBuffer, paperSize, layout, options, printBuffer, [], renderContext);
         });
         // Draw printOnlyAtEnd containers once at the end (at their layout coordinates on current page)
         const printOnlyAtEndContainers = layout.objects.filter((object) => object.type == 'container' && object.printOnlyAtEnd);
         printOnlyAtEndContainers.forEach((object) => {
-            addObjectToPDF(0, 0, doc, object, printBuffer, paperSize, layout, options, printBuffer, [], renderContext);
+            const bump = Math.max(0, renderTopPrintedStartBottomTenths - object.y);
+            addObjectToPDF(0, bump, doc, object, printBuffer, paperSize, layout, options, printBuffer, [], renderContext);
         });
         return doc;
     });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@softwear/latestcollectioncore",
-  "version": "1.0.177",
+  "version": "1.0.178",
   "description": "Core functions for LatestCollections applications",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/pdf.ts

@@ -61,7 +61,7 @@ export interface ContainerLayoutObject extends BaseLayoutObject {
   minHeightBeforeBreak?: number
   /** When true, container and children are drawn only once at the end of the report, not on every page */
   printOnlyAtEnd?: boolean
-  /** When true, container and children are drawn only once before other body containers (first page header block) */
+  /** Only effective on top-level `layout.objects` (PDF engine ignores this flag on nested containers). */
   printOnlyAtStart?: boolean
 }
 

package/src/reports.ts

@@ -786,10 +786,12 @@ export async function genPDF(layout: Layout, printBuffer: any, options: GenPdfOp
   drawStaticPartOfPage(measurementDoc, layout, printBuffer, paperSize, options, measurementContext)
 
   // Draw containers in measurement mode (same order as render)
+  let measurementTopPrintedStartBottomTenths = 0
   layout.objects
     .filter((object) => object.type == 'container' && (object as ContainerLayoutObject).printOnlyAtStart)
     .forEach((object) => {
-      addObjectToPDF(0, 0, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext)
+      const lh = addObjectToPDF(0, 0, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext)
+      measurementTopPrintedStartBottomTenths = Math.max(measurementTopPrintedStartBottomTenths, lh.y * 10)
     })
   layout.objects
     .filter(
@@ -799,12 +801,14 @@ export async function genPDF(layout: Layout, printBuffer: any, options: GenPdfOp
         !(object as ContainerLayoutObject).printOnlyAtStart
     )
     .forEach((object) => {
-      addObjectToPDF(0, 0, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext)
+      const bump = Math.max(0, measurementTopPrintedStartBottomTenths - object.y)
+      addObjectToPDF(0, bump, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext)
     })
   layout.objects
     .filter((object) => object.type == 'container' && (object as ContainerLayoutObject).printOnlyAtEnd)
     .forEach((object) => {
-      addObjectToPDF(0, 0, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext)
+      const bump = Math.max(0, measurementTopPrintedStartBottomTenths - object.y)
+      addObjectToPDF(0, bump, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext)
     })
 
   // Get total page count from measurement pass
@@ -826,10 +830,12 @@ export async function genPDF(layout: Layout, printBuffer: any, options: GenPdfOp
   }
   drawStaticPartOfPage(doc, layout, printBuffer, paperSize, options, renderContext)
 
+  let renderTopPrintedStartBottomTenths = 0
   layout.objects
     .filter((object) => object.type == 'container' && (object as ContainerLayoutObject).printOnlyAtStart)
     .forEach((object) => {
-      addObjectToPDF(0, 0, doc, object, printBuffer, paperSize, layout, options, printBuffer, [], renderContext)
+      const lh = addObjectToPDF(0, 0, doc, object, printBuffer, paperSize, layout, options, printBuffer, [], renderContext)
+      renderTopPrintedStartBottomTenths = Math.max(renderTopPrintedStartBottomTenths, lh.y * 10)
     })
 
   // Draw containers in rendering mode (body containers after printOnlyAtStart)
@@ -841,7 +847,8 @@ export async function genPDF(layout: Layout, printBuffer: any, options: GenPdfOp
         !(object as ContainerLayoutObject).printOnlyAtStart
     )
     .forEach((object) => {
-      addObjectToPDF(0, 0, doc, object, printBuffer, paperSize, layout, options, printBuffer, [], renderContext)
+      const bump = Math.max(0, renderTopPrintedStartBottomTenths - object.y)
+      addObjectToPDF(0, bump, doc, object, printBuffer, paperSize, layout, options, printBuffer, [], renderContext)
     })
 
   // Draw printOnlyAtEnd containers once at the end (at their layout coordinates on current page)
@@ -849,7 +856,8 @@ export async function genPDF(layout: Layout, printBuffer: any, options: GenPdfOp
     (object) => object.type == 'container' && (object as ContainerLayoutObject).printOnlyAtEnd
   )
   printOnlyAtEndContainers.forEach((object) => {
-    addObjectToPDF(0, 0, doc, object, printBuffer, paperSize, layout, options, printBuffer, [], renderContext)
+    const bump = Math.max(0, renderTopPrintedStartBottomTenths - object.y)
+    addObjectToPDF(0, bump, doc, object, printBuffer, paperSize, layout, options, printBuffer, [], renderContext)
   })
   return doc
 }

package/test/reports.spec.ts

@@ -1,8 +1,15 @@
 import { describe, expect, it } from 'vitest'
 import { genPDF } from '../src/reports'
 import type { Layout } from '../src/pdf'
-// import fs from 'fs'
 
+/** Tiny valid JPEG as data URL — avoids flaky network-dependent image fetch in CI/sandbox. */
+function offlineJpegFixture(): string {
+  const b64 =
+    '/9j/4AAQSkZJRgABAQIAASABIAAD/2wBDAAgGBgcGBQgHBwcJCQgKDBQNDAsLDBkSEw8UHRofHh0aHBwgJC4nICIsIxwcKDcpLDAxNDQ0Hyc5PTgyPC4zNDL/2wBDAQkJCQwLDBgNDRgyIRwhMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjL/wAARCAAKAAoDASIAAhEBAxEB/8QAFQABAQAAAAAAAAAAAAAAAAAAAAb/xAAUEAEAAAAAAAAAAAAAAAAAAAAA/9oADAMBAAIQAxAAAAHhAP/EABQQAQAAAAAAAAAAAAAAAAAAAAD/2gAIAQEAAQkCf//EABQRAQAAAAAAAAAAAAAAAAAAAAD/2gAIAQMBAT8B/8QAFBEBAAAAAAAAAAAAAAAAAAAAAP/aAAgBAgMBPwF//8QAFBEBAAAAAAAAAAAAAAAAAAAAAP/aAAgBAQYPAj//xAAVEAEAAAAAAAAAAAAAAAAAAAAAAP/aAAgBAQBPCQ//Z'
+  return `data:image/jpeg;base64,${b64}`
+}
+
+// URL still used so collectImageUrlsFromLayout resolves the layout object
 const imageUrl = 'https://upload.wikimedia.org/wikipedia/commons/thumb/7/79/Phillips_PM5538.jpg/500px-Phillips_PM5538.jpg'
 
 describe('genPDF', () => {
@@ -59,7 +66,14 @@ describe('genPDF', () => {
       title: 'Minimal PDF',
     }
 
-    const doc = await genPDF(layout, printBuffer)
+    const doc = await genPDF(layout, printBuffer, {
+      loadImage: async () => ({
+        dataUrl: offlineJpegFixture(),
+        format: 'JPEG',
+        width: 500,
+        height: 375,
+      }),
+    })
     const pdf = doc.output('arraybuffer')
     const pdfText = Buffer.from(pdf).toString('latin1')
 
@@ -138,4 +152,61 @@ describe('genPDF', () => {
     expect(iStart).toBeLessThan(iMid)
     expect(iMid).toBeLessThan(iEnd)
   })
+
+  it('stacks top-level body containers below printOnlyAtStart when they share the same layout y', async () => {
+    const text = (name: string, label: string, y = 0, fontSize = 80) =>
+      ({
+        type: 'text' as const,
+        name,
+        x: 20,
+        y,
+        width: 900,
+        height: 400,
+        active: true,
+        text: label,
+        textAlign: 1 as const,
+        fontFamily: 'Helvetica',
+        fontSize,
+        fontStyle: 0,
+      }) as const
+
+    const sharedY = 400
+    const layout: Layout = {
+      name: 'same-y-stack',
+      paperSize: { width: 210, height: 297, footerHeight: 20 },
+      objects: [
+        {
+          type: 'container',
+          x: 0,
+          y: sharedY,
+          width: 1000,
+          height: 900,
+          active: true,
+          source: '',
+          printOnlyAtStart: true,
+          children: [text('h', 'PAGE1_HEADER_BLOCK')],
+        },
+        {
+          type: 'container',
+          x: 0,
+          y: sharedY,
+          width: 1000,
+          height: 500,
+          active: true,
+          source: '',
+          children: [text('b', 'BODY_AFTER_STACK', 0, 32)],
+        },
+      ],
+    }
+
+    const doc = await genPDF(layout, {})
+    const pdfText = Buffer.from(doc.output('arraybuffer')).toString('latin1')
+    expect(pdfText).toContain('(PAGE1_HEADER_BLOCK)')
+    expect(pdfText).toContain('(BODY_AFTER_STACK)')
+    const iH = pdfText.indexOf('(PAGE1_HEADER_BLOCK)')
+    const iB = pdfText.indexOf('(BODY_AFTER_STACK)')
+    expect(iH).toBeGreaterThanOrEqual(0)
+    expect(iB).toBeGreaterThanOrEqual(0)
+    expect(iH).toBeLessThan(iB)
+  })
 })