throw an error when content doesn't fit to page (too big) #207

package.json

@@ -1,6 +1,6 @@
 {
   "name": "pdfmake",
-  "version": "0.2.0",
+  "version": "0.2.1-fork207B",
   "description": "Client/server side PDF printing in pure JavaScript",
   "main": "src/printer.js",
   "browser": "build/pdfmake.js",
@@ -42,6 +42,15 @@
     "webpack": "^5.42.0",
     "webpack-cli": "^4.7.2"
   },
+  "files": [
+    "build/*.js*",
+    "src",
+    "build-vfs.js",
+    "CHANGELOG.md",
+    "LICENSE",
+    "package.json",
+    "README.md"
+  ],
   "engines": {
     "node": ">=12"
   },

src/layoutBuilder.js

@@ -420,7 +420,13 @@ LayoutBuilder.prototype.processNode = function (node) {
 		}
 
 		if (unbreakable) {
-			self.writer.commitUnbreakableBlock();
+			try {
+				self.writer.commitUnbreakableBlock();
+			} catch (e) {
+				e.node = node;
+				throw e;
+			}
+
 		}
 	});
 

src/pageElementWriter.js

@@ -24,6 +24,9 @@ function fitOnPage(self, addFct) {
 	if (!position) {
 		self.moveToNextPage();
 		position = addFct(self);
+		if (!position) {
+			throw new Error('unable to add it to page');
+		}
 	}
 	return position;
 }
@@ -69,10 +72,9 @@ PageElementWriter.prototype.alignCanvas = function (node) {
 };
 
 PageElementWriter.prototype.addFragment = function (fragment, useBlockXOffset, useBlockYOffset, dontUpdateContextPosition) {
-	if (!this.writer.addFragment(fragment, useBlockXOffset, useBlockYOffset, dontUpdateContextPosition)) {
-		this.moveToNextPage();
-		this.writer.addFragment(fragment, useBlockXOffset, useBlockYOffset, dontUpdateContextPosition);
-	}
+	fitOnPage(this, function (self) {
+		return self.writer.addFragment(fragment, useBlockXOffset, useBlockYOffset, dontUpdateContextPosition);
+	});
 };
 
 PageElementWriter.prototype.moveToNextPage = function (pageOrientation) {

tests/pageElementWriter.js

@@ -159,6 +159,14 @@ describe('PageElementWriter', function () {
 			assert.deepEqual(position, { pageNumber: 2, left: MARGINS.left, top: MARGINS.top, verticalRatio: 0, horizontalRatio: 0, pageOrientation: 'portrait', pageInnerHeight: 1000, pageInnerWidth: 500 });
 		});
 
+		it('should throw an error if not fit in page', function () {
+			var lineHeight = 1800;
+
+			assert.throws(() => {
+				pew.addLine(buildLine(lineHeight));
+			});
+		});
+
 		it('should write into the current page if it\'s a large image and nothing else exists on the page', function () {
 			var position = pew.addImage(buildImage(2000));