@softwear/latestcollectioncore 1.0.185 → 1.0.187

package/dist/reports.js

@@ -16,7 +16,7 @@ exports.genPDF = void 0;
 const jspdf_1 = require("jspdf");
 const date_fns_1 = require("date-fns");
 const deepCopy_1 = __importDefault(require("./deepCopy"));
-/** Measurement never advances jsPDF sheet count; rendering uses the active canvas page (1-based). */
+/** Measurement uses context `currentPageNumber` (1-based). Rendering uses jsPDF’s `getCurrentPageInfo().pageNumber`, which is already 1-based (first page is 1). */
 function getCurrentPageNumber(doc, context) {
     var _a;
     if (context.mode === 'measurement') {
@@ -507,9 +507,14 @@ addObjectToPDF = function (originX, originY, doc, object, printBuffer, paperSize
     const layoutXtenths = originX + object.x;
     const layoutYtenths = originY + object.y;
     const rawPrintOnlyAtStartOffset = printOnlyAtStartContainersOffset !== null && printOnlyAtStartContainersOffset !== void 0 ? printOnlyAtStartContainersOffset : 0;
-    const effectivePrintOnlyAtStartContainersOffset = containerChain.length === 0 && rawPrintOnlyAtStartOffset > 0 && getCurrentPageNumber(doc, context) === 1 ? rawPrintOnlyAtStartOffset : 0;
+    /** Must consult current page on every read: `addPage` can move to page 2+ mid-invocation while this call stays open. */
+    const effectivePrintOnlyAtStartContainersOffsetNow = () => containerChain.length === 0 &&
+        rawPrintOnlyAtStartOffset > 0 &&
+        getCurrentPageNumber(doc, context) === 1
+        ? rawPrintOnlyAtStartOffset
+        : 0;
     const x = layoutXtenths / 10;
-    const y = (layoutYtenths + effectivePrintOnlyAtStartContainersOffset) / 10;
+    const y = (layoutYtenths + effectivePrintOnlyAtStartContainersOffsetNow()) / 10;
     const width = object.width / 10;
     const height = object.height / 10;
     // Keep track of the lower righthandside of all objects in this container
@@ -530,7 +535,7 @@ addObjectToPDF = function (originX, originY, doc, object, printBuffer, paperSize
         }
         const nrContainers = source.length;
         let originX = layoutXtenths;
-        let originY = layoutYtenths + effectivePrintOnlyAtStartContainersOffset;
+        let originY = layoutYtenths + effectivePrintOnlyAtStartContainersOffsetNow();
         for (let containerIndex = 0; containerIndex < nrContainers; containerIndex++) {
             const newContainerChain = containerChain.filter(function () {
                 return true;
@@ -542,7 +547,8 @@ addObjectToPDF = function (originX, originY, doc, object, printBuffer, paperSize
             if (useOrphanCheck && (object.pageBreak || originY + minHeightMm > paperSize.height - paperSize.footerHeight)) {
                 // We ran out of paper
                 originX = layoutXtenths;
-                originY = layoutYtenths + effectivePrintOnlyAtStartContainersOffset;
+                // Continuation sheets must not reuse the page-1-only overlap bump: addPage hasn't run yet so getCurrentPageNumber is still 1.
+                originY = layoutYtenths;
                 const newOrigin = addPage(originX, originY, doc, layout, rootPrintBuffer, paperSize, options, newContainerChain, context);
                 originX = newOrigin.originX;
                 originY = newOrigin.originY;

package/package.json

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

package/src/reports.ts

@@ -42,7 +42,7 @@ interface RenderContext {
   mode: RenderMode
   /** Total pages from measurement pass; used for field sources like `pageCount` / `page`. */
   pageCount?: number
-  /** During measurement, logical printable page (starts at 1); incremented when {@link addPage} runs. In rendering use {@link getCurrentPageNumber} (jsPDF). */
+  /** During measurement, logical printable page (starts at 1); incremented when {@link addPage} runs. Rendering: see {@link getCurrentPageNumber} (1-based, matches pdf page fields). */
   currentPageNumber?: number
   measureOnly?: boolean // When true, skip drawing to measure bounds (for fillContainer rectangles)
   imageAssets?: Map<string, PdfImageAsset>
@@ -50,7 +50,7 @@ interface RenderContext {
   defaultFontFamily?: string
 }
 
-/** Measurement never advances jsPDF sheet count; rendering uses the active canvas page (1-based). */
+/** Measurement uses context `currentPageNumber` (1-based). Rendering uses jsPDF’s `getCurrentPageInfo().pageNumber`, which is already 1-based (first page is 1). */
 function getCurrentPageNumber(doc: jsPDF, context: RenderContext): number {
   if (context.mode === 'measurement') {
     return context.currentPageNumber ?? 1
@@ -585,11 +585,16 @@ addObjectToPDF = function (
   const layoutXtenths = originX + object.x
   const layoutYtenths = originY + object.y
   const rawPrintOnlyAtStartOffset = printOnlyAtStartContainersOffset ?? 0
-  const effectivePrintOnlyAtStartContainersOffset =
-    containerChain.length === 0 && rawPrintOnlyAtStartOffset > 0 && getCurrentPageNumber(doc, context) === 1 ? rawPrintOnlyAtStartOffset : 0
+  /** Must consult current page on every read: `addPage` can move to page 2+ mid-invocation while this call stays open. */
+  const effectivePrintOnlyAtStartContainersOffsetNow = (): number =>
+    containerChain.length === 0 &&
+    rawPrintOnlyAtStartOffset > 0 &&
+    getCurrentPageNumber(doc, context) === 1
+      ? rawPrintOnlyAtStartOffset
+      : 0
 
   const x = layoutXtenths / 10
-  const y = (layoutYtenths + effectivePrintOnlyAtStartContainersOffset) / 10
+  const y = (layoutYtenths + effectivePrintOnlyAtStartContainersOffsetNow()) / 10
   const width = object.width / 10
   const height = object.height / 10
   // Keep track of the lower righthandside of all objects in this container
@@ -613,7 +618,7 @@ addObjectToPDF = function (
     const nrContainers = source.length
 
     let originX = layoutXtenths
-    let originY = layoutYtenths + effectivePrintOnlyAtStartContainersOffset
+    let originY = layoutYtenths + effectivePrintOnlyAtStartContainersOffsetNow()
 
     for (let containerIndex = 0; containerIndex < nrContainers; containerIndex++) {
       const newContainerChain = containerChain.filter(function () {
@@ -626,7 +631,8 @@ addObjectToPDF = function (
       if (useOrphanCheck && (object.pageBreak || originY + minHeightMm > paperSize.height - paperSize.footerHeight)) {
         // We ran out of paper
         originX = layoutXtenths
-        originY = layoutYtenths + effectivePrintOnlyAtStartContainersOffset
+        // Continuation sheets must not reuse the page-1-only overlap bump: addPage hasn't run yet so getCurrentPageNumber is still 1.
+        originY = layoutYtenths
         const newOrigin = addPage(originX, originY, doc, layout, rootPrintBuffer, paperSize, options, newContainerChain, context)
         originX = newOrigin.originX
         originY = newOrigin.originY