Added a fix: cfpdf action="addHeader/addFooter" with destination attr…

…ibute throws the correct error (#51)
lucee · Sep 14, 2023 · 0e75732 · 0e75732
1 parent 15bd7da
commit 0e75732
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/source/java/src/org/lucee/extension/pdf/tag/PDF.java b/source/java/src/org/lucee/extension/pdf/tag/PDF.java
@@ -750,6 +750,9 @@ else if (destination != null) {
 	private void doActionAddHeaderFooter(boolean isHeader) throws PageException, IOException, DocumentException {
 		required("pdf", "write", "source", source);
 		if (text == null) throw engine.getExceptionUtil().createApplicationException("when PDF action is [addHeader or addFooter], It requires a attribute [text]");
+
+		if (destination != null && destination.exists() && !overwrite)
+			throw engine.getExceptionUtil().createApplicationException("Destination PDF file [" + destination + "] already exists");
 		// required("pdf", "write", "destination", destination);
 
 		/*
@@ -784,9 +787,6 @@ else if (destination != null) {
 		}
 		PdfStamper stamper = null;
 		try {
-			if (destination != null && destination.exists() && !overwrite)
-				throw engine.getExceptionUtil().createApplicationException("Destination PDF file [" + destination + "] already exists");
-
 			int len = reader.getNumberOfPages();
 			Set<Integer> pageSet = PDFUtil.parsePageDefinition(pages, len);
 			stamper = new PdfStamper(reader, os);