@softwear/latestcollectioncore 1.0.178 → 1.0.180

package/dist/reports.js

@@ -43,9 +43,21 @@ function getProperty(propertyName, object) {
     }
     return property;
 }
-/** Repeat once over the current buffer when the container is only a visual group (no `source` binding). */
+/** Repeat once when the container has no data binding (`source` empty / whitespace-only). */
+function dummyLayoutSource(source) {
+    return source === undefined || String(source).trim() === '';
+}
+/** Like repeatOnOverflow: redraw after a page break so nested overflowing content can proceed. */
+function containerRedrawsAfterContinuationPage(child) {
+    if (child.type !== 'container')
+        return false;
+    const c = child;
+    if (c.repeatOnOverflow)
+        return true;
+    return dummyLayoutSource(c.source);
+}
 function containerLoopSource(container, printBuffer) {
-    if (!container.source || container.source === '')
+    if (dummyLayoutSource(container.source))
         return [printBuffer];
     return getProperty(container.source, printBuffer);
 }
@@ -368,6 +380,8 @@ function drawSimpleObject(x, y, doc, object, printBuffer, width, height, options
     }
 }
 function addPage(originX, originY, doc, layout, rootPrintBuffer, paperSize, options, containerChain, context) {
+    context.pendingRootStartBumpTenths = 0;
+    context.applyRootStartBumpToThisCall = false;
     if (context.mode === 'measurement') {
         // Track page count in measurement mode
         if (context.currentPageCount === undefined) {
@@ -392,7 +406,7 @@ function addPage(originX, originY, doc, layout, rootPrintBuffer, paperSize, opti
         absoluteLowerRightHandSide.y = originY / 10;
         containerChain.forEach((link) => {
             link.object.children.forEach((child) => {
-                if (!child.snapToBottom && (child.type != 'container' || child.repeatOnOverflow)) {
+                if (!child.snapToBottom && (child.type != 'container' || containerRedrawsAfterContinuationPage(child))) {
                     const childLowerRightHandSide = addObjectToPDF(originX, originY, doc, child, link.printBuffer, paperSize, layout, options, rootPrintBuffer, [], context);
                     absoluteLowerRightHandSide.x = Math.max(absoluteLowerRightHandSide.x, childLowerRightHandSide.x);
                     absoluteLowerRightHandSide.y = Math.max(absoluteLowerRightHandSide.y, childLowerRightHandSide.y);
@@ -441,9 +455,15 @@ function drawBottomDwellers(object, originX, originY, absoluteLowerRightHandSide
  * Use originX,originY as origin to displace the object's own x,y coordinates
  */
 addObjectToPDF = function (originX, originY, doc, object, printBuffer, paperSize, layout, options, rootPrintBuffer, containerChain, context) {
-    var _a;
+    var _a, _b;
+    let rootBump = 0;
+    if (context.applyRootStartBumpToThisCall) {
+        rootBump = (_a = context.pendingRootStartBumpTenths) !== null && _a !== void 0 ? _a : 0;
+        context.applyRootStartBumpToThisCall = false;
+        context.pendingRootStartBumpTenths = 0;
+    }
     const x = (originX + object.x) / 10;
-    const y = (originY + object.y) / 10;
+    const y = (originY + object.y + rootBump) / 10;
     const width = object.width / 10;
     const height = object.height / 10;
     // Keep track of the lower righthandside of all objects in this container
@@ -458,7 +478,7 @@ addObjectToPDF = function (originX, originY, doc, object, printBuffer, paperSize
         let originX = x * 10, originY = y * 10;
         const source = containerLoopSource(object, printBuffer);
         if (source === undefined) {
-            (_a = options.onAlert) === null || _a === void 0 ? void 0 : _a.call(options, { header: 'containernotfound', body: object.source, type: 'warning', timeout: 10000 });
+            (_b = options.onAlert) === null || _b === void 0 ? void 0 : _b.call(options, { header: 'containernotfound', body: object.source, type: 'warning', timeout: 10000 });
             return absoluteLowerRightHandSide;
         }
         const nrContainers = source.length;
@@ -675,14 +695,16 @@ function genPDF(layout, printBuffer, options = {}) {
             !object.printOnlyAtEnd &&
             !object.printOnlyAtStart)
             .forEach((object) => {
-            const bump = Math.max(0, measurementTopPrintedStartBottomTenths - object.y);
-            addObjectToPDF(0, bump, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext);
+            measurementContext.applyRootStartBumpToThisCall = true;
+            measurementContext.pendingRootStartBumpTenths = Math.max(0, measurementTopPrintedStartBottomTenths - object.y);
+            addObjectToPDF(0, 0, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext);
         });
         layout.objects
             .filter((object) => object.type == 'container' && object.printOnlyAtEnd)
             .forEach((object) => {
-            const bump = Math.max(0, measurementTopPrintedStartBottomTenths - object.y);
-            addObjectToPDF(0, bump, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext);
+            measurementContext.applyRootStartBumpToThisCall = true;
+            measurementContext.pendingRootStartBumpTenths = Math.max(0, measurementTopPrintedStartBottomTenths - object.y);
+            addObjectToPDF(0, 0, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext);
         });
         // Get total page count from measurement pass
         const totalPageCount = measurementContext.currentPageCount || 1;
@@ -714,14 +736,16 @@ function genPDF(layout, printBuffer, options = {}) {
             !object.printOnlyAtEnd &&
             !object.printOnlyAtStart)
             .forEach((object) => {
-            const bump = Math.max(0, renderTopPrintedStartBottomTenths - object.y);
-            addObjectToPDF(0, bump, doc, object, printBuffer, paperSize, layout, options, printBuffer, [], renderContext);
+            renderContext.applyRootStartBumpToThisCall = true;
+            renderContext.pendingRootStartBumpTenths = Math.max(0, renderTopPrintedStartBottomTenths - object.y);
+            addObjectToPDF(0, 0, 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) => {
-            const bump = Math.max(0, renderTopPrintedStartBottomTenths - object.y);
-            addObjectToPDF(0, bump, doc, object, printBuffer, paperSize, layout, options, printBuffer, [], renderContext);
+            renderContext.applyRootStartBumpToThisCall = true;
+            renderContext.pendingRootStartBumpTenths = Math.max(0, renderTopPrintedStartBottomTenths - object.y);
+            addObjectToPDF(0, 0, doc, object, printBuffer, paperSize, layout, options, printBuffer, [], renderContext);
         });
         return doc;
     });

package/package.json

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

package/src/reports.ts

@@ -46,6 +46,10 @@ interface RenderContext {
   imageAssets?: Map<string, PdfImageAsset>
   /** Layout default font used when object.fontFamily is undefined */
   defaultFontFamily?: string
+  /** Set with {@link pendingRootStartBumpTenths} only for top-level genPDF container draws; consumed on first addObjectToPDF read. */
+  applyRootStartBumpToThisCall?: boolean
+  /** Vertical pad (tenths-mm) so body roots sit below rendered printOnlyAtStart siblings; use with applyRootStartBumpToThisCall only. */
+  pendingRootStartBumpTenths?: number
 }
 
 // We have to declare some functions that will be called by other functions but also have to call those other functions
@@ -92,10 +96,22 @@ function getProperty(propertyName: string, object: any): any {
   return property
 }
 
-/** Repeat once over the current buffer when the container is only a visual group (no `source` binding). */
+/** Repeat once when the container has no data binding (`source` empty / whitespace-only). */
+function dummyLayoutSource(source?: string): boolean {
+  return source === undefined || String(source).trim() === ''
+}
+
+/** Like repeatOnOverflow: redraw after a page break so nested overflowing content can proceed. */
+function containerRedrawsAfterContinuationPage(child: LayoutObject): boolean {
+  if (child.type !== 'container') return false
+  const c = child as ContainerLayoutObject
+  if (c.repeatOnOverflow) return true
+  return dummyLayoutSource(c.source)
+}
+
 function containerLoopSource(container: ContainerLayoutObject, printBuffer: any): any {
-  if (!container.source || container.source === '') return [printBuffer]
-  return getProperty(container.source, printBuffer)
+  if (dummyLayoutSource(container.source)) return [printBuffer]
+  return getProperty(container.source as string, printBuffer)
 }
 
 /** jsPDF font style strings. fontStyle bitmask: 0=normal, 1=bold, 2=italic, 3=bold+italic */
@@ -433,6 +449,8 @@ function addPage(
   containerChain: { object: ContainerLayoutObject; printBuffer: any }[],
   context: RenderContext
 ): { originX: number; originY: number; x: number; y: number } {
+  context.pendingRootStartBumpTenths = 0
+  context.applyRootStartBumpToThisCall = false
   if (context.mode === 'measurement') {
     // Track page count in measurement mode
     if (context.currentPageCount === undefined) {
@@ -456,7 +474,7 @@ function addPage(
     absoluteLowerRightHandSide.y = originY / 10
     containerChain.forEach((link) => {
       link.object.children.forEach((child) => {
-        if (!child.snapToBottom && (child.type != 'container' || child.repeatOnOverflow)) {
+        if (!child.snapToBottom && (child.type != 'container' || containerRedrawsAfterContinuationPage(child))) {
           const childLowerRightHandSide = addObjectToPDF(originX, originY, doc, child, link.printBuffer, paperSize, layout, options, rootPrintBuffer, [], context)
           absoluteLowerRightHandSide.x = Math.max(absoluteLowerRightHandSide.x, childLowerRightHandSide.x)
           absoluteLowerRightHandSide.y = Math.max(absoluteLowerRightHandSide.y, childLowerRightHandSide.y)
@@ -529,8 +547,14 @@ addObjectToPDF = function (
   containerChain: { object: ContainerLayoutObject; printBuffer: any }[],
   context: RenderContext
 ): { x: number; y: number } {
+  let rootBump = 0
+  if (context.applyRootStartBumpToThisCall) {
+    rootBump = context.pendingRootStartBumpTenths ?? 0
+    context.applyRootStartBumpToThisCall = false
+    context.pendingRootStartBumpTenths = 0
+  }
   const x = (originX + object.x) / 10
-  const y = (originY + object.y) / 10
+  const y = (originY + object.y + rootBump) / 10
   const width = object.width / 10
   const height = object.height / 10
   // Keep track of the lower righthandside of all objects in this container
@@ -801,14 +825,16 @@ export async function genPDF(layout: Layout, printBuffer: any, options: GenPdfOp
         !(object as ContainerLayoutObject).printOnlyAtStart
     )
     .forEach((object) => {
-      const bump = Math.max(0, measurementTopPrintedStartBottomTenths - object.y)
-      addObjectToPDF(0, bump, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext)
+      measurementContext.applyRootStartBumpToThisCall = true
+      measurementContext.pendingRootStartBumpTenths = Math.max(0, measurementTopPrintedStartBottomTenths - object.y)
+      addObjectToPDF(0, 0, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext)
     })
   layout.objects
     .filter((object) => object.type == 'container' && (object as ContainerLayoutObject).printOnlyAtEnd)
     .forEach((object) => {
-      const bump = Math.max(0, measurementTopPrintedStartBottomTenths - object.y)
-      addObjectToPDF(0, bump, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext)
+      measurementContext.applyRootStartBumpToThisCall = true
+      measurementContext.pendingRootStartBumpTenths = Math.max(0, measurementTopPrintedStartBottomTenths - object.y)
+      addObjectToPDF(0, 0, measurementDoc, object, printBuffer, paperSize, layout, options, printBuffer, [], measurementContext)
     })
 
   // Get total page count from measurement pass
@@ -847,8 +873,9 @@ export async function genPDF(layout: Layout, printBuffer: any, options: GenPdfOp
         !(object as ContainerLayoutObject).printOnlyAtStart
     )
     .forEach((object) => {
-      const bump = Math.max(0, renderTopPrintedStartBottomTenths - object.y)
-      addObjectToPDF(0, bump, doc, object, printBuffer, paperSize, layout, options, printBuffer, [], renderContext)
+      renderContext.applyRootStartBumpToThisCall = true
+      renderContext.pendingRootStartBumpTenths = Math.max(0, renderTopPrintedStartBottomTenths - object.y)
+      addObjectToPDF(0, 0, doc, object, printBuffer, paperSize, layout, options, printBuffer, [], renderContext)
     })
 
   // Draw printOnlyAtEnd containers once at the end (at their layout coordinates on current page)
@@ -856,8 +883,9 @@ export async function genPDF(layout: Layout, printBuffer: any, options: GenPdfOp
     (object) => object.type == 'container' && (object as ContainerLayoutObject).printOnlyAtEnd
   )
   printOnlyAtEndContainers.forEach((object) => {
-    const bump = Math.max(0, renderTopPrintedStartBottomTenths - object.y)
-    addObjectToPDF(0, bump, doc, object, printBuffer, paperSize, layout, options, printBuffer, [], renderContext)
+    renderContext.applyRootStartBumpToThisCall = true
+    renderContext.pendingRootStartBumpTenths = Math.max(0, renderTopPrintedStartBottomTenths - object.y)
+    addObjectToPDF(0, 0, doc, object, printBuffer, paperSize, layout, options, printBuffer, [], renderContext)
   })
   return doc
 }

package/test/reports.spec.ts

@@ -209,4 +209,58 @@ describe('genPDF', () => {
     expect(iB).toBeGreaterThanOrEqual(0)
     expect(iH).toBeLessThan(iB)
   })
+
+  it('redraws empty-source wrapper so inner vertical repeat spans multiple pages', async () => {
+    const rowField = {
+      type: 'field' as const,
+      name: 'ln',
+      x: 0,
+      y: 0,
+      width: 1600,
+      height: 80,
+      active: true,
+      source: 'line',
+      textAlign: 1 as const,
+      fontFamily: 'Helvetica',
+      fontSize: 36,
+      fontStyle: 0,
+    }
+    const ROWS = Array.from({ length: 40 }, (_, i) => ({ line: `ROW_${i}_MARK` }))
+    const layout: Layout = {
+      name: 'wrapper-overflow',
+      paperSize: { width: 210, height: 297, footerHeight: 20 },
+      objects: [
+        {
+          type: 'container',
+          name: 'wrap',
+          x: 50,
+          y: 100,
+          width: 1900,
+          height: 2800,
+          active: true,
+          source: '',
+          children: [
+            {
+              type: 'container',
+              name: 'rows',
+              x: 0,
+              y: 0,
+              width: 1800,
+              height: 120,
+              active: true,
+              source: 'ROWS',
+              repeatContainer: 'vertical' as const,
+              children: [rowField],
+            },
+          ],
+        },
+      ],
+    }
+
+    const doc = await genPDF(layout, { ROWS })
+    expect(doc.getNumberOfPages()).toBeGreaterThanOrEqual(2)
+    const pdfText = Buffer.from(doc.output('arraybuffer')).toString('latin1')
+    expect(pdfText).toContain('ROW_0_MARK')
+    expect(pdfText).toContain('ROW_35_MARK')
+  })
 })