[app][fix] propagate debug render arg close #60

ulb-sachsen-anhalt · Nov 6, 2024 · 42f895a · 42f895a
1 parent 00136e7
commit 42f895a
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 8 deletions.
diff --git a/src/main/java/de/ulb/digital/derivans/config/DerivansConfiguration.java b/src/main/java/de/ulb/digital/derivans/config/DerivansConfiguration.java
@@ -87,7 +87,7 @@ public DerivansConfiguration(DerivansParameter params) throws DigitalDerivansExc
 		}
 
 		// determine if debug render for PDF required
-		if (params.getDebugPdfRender()) {
+		if (Boolean.TRUE.equals(params.isDebugPdfRender())) {
 			this.debugPdfRender = Boolean.TRUE;
 		}
 

diff --git a/src/main/java/de/ulb/digital/derivans/config/DerivansParameter.java b/src/main/java/de/ulb/digital/derivans/config/DerivansParameter.java
@@ -109,11 +109,8 @@ public void setNamePDF(String namePDF) {
 		this.namePDF = namePDF;
 	}
 
-	public Boolean getDebugPdfRender() {
-		if (debugPdfRender == null) {
-			debugPdfRender = Boolean.FALSE;
-		}
-		return debugPdfRender;
+	public Boolean isDebugPdfRender() {
+		return debugPdfRender != null;
 	}
 
 	public void setDebugPdfRender(boolean isRequired) {

diff --git a/src/main/java/de/ulb/digital/derivans/derivate/pdf/IText5Processor.java b/src/main/java/de/ulb/digital/derivans/derivate/pdf/IText5Processor.java
@@ -326,8 +326,8 @@ private PDFElement render(BaseFont font, int pageHeight, PdfContentByte cb, ICha
 		if (this.debugRender) {
 			LOGGER.trace("put '{}' at {}x{} (fontsize:{})", text, x, v, fontSize);
 		}
-		// propably hide text layer font
-		if (this.renderModus.equalsIgnoreCase("invisible")) {
+		// hide text layer if configured and *not* in debugMode
+		if ((!this.debugRender) && this.renderModus.equalsIgnoreCase("invisible")) {
 			cb.setTextRenderingMode(PdfContentByte.TEXT_RENDER_MODE_INVISIBLE);
 		}
 		// rendering depends on text orientation