From 849d22e71fcbeeefc54ec4492b6f528a1d1681aa Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bal=C3=A1zs=20Sz=C3=BCcs?= Date: Fri, 2 May 2025 15:04:00 +0200 Subject: [PATCH] Added comment to improve future maintainability --- .../software/SPDF/controller/api/security/GetInfoOnPDF.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/main/java/stirling/software/SPDF/controller/api/security/GetInfoOnPDF.java b/src/main/java/stirling/software/SPDF/controller/api/security/GetInfoOnPDF.java index d5b09742b..06f186a85 100644 --- a/src/main/java/stirling/software/SPDF/controller/api/security/GetInfoOnPDF.java +++ b/src/main/java/stirling/software/SPDF/controller/api/security/GetInfoOnPDF.java @@ -619,6 +619,9 @@ public class GetInfoOnPDF { private void setNodePermissions(PDDocument pdfBoxDoc, ObjectNode permissionsNode) { AccessPermission ap = pdfBoxDoc.getCurrentAccessPermission(); + // Note: The boolean values are inverted because the PDF permission system uses a "prevent" naming scheme. + // For example, canAssembleDocument() returns false when document assembly is prevented. + // This inversion ensures the code's boolean values match the PDF's permission model. permissionsNode.put("Document Assembly", getPermissionState(!ap.canAssembleDocument())); permissionsNode.put("Extracting Content", getPermissionState(!ap.canExtractContent())); permissionsNode.put(