You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pdfbox.apache.org by ti...@apache.org on 2019/04/11 16:33:52 UTC

svn commit: r1857349 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDSignatureField.java

Author: tilman
Date: Thu Apr 11 16:33:52 2019
New Revision: 1857349

URL: http://svn.apache.org/viewvc?rev=1857349&view=rev
Log:
PDFBOX-3017: crawl whole field tree and not just root

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDSignatureField.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDSignatureField.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDSignatureField.java?rev=1857349&r1=1857348&r2=1857349&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDSignatureField.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDSignatureField.java Thu Apr 11 16:33:52 2019
@@ -74,8 +74,7 @@ public class PDSignatureField extends PD
     {
         String fieldName = "Signature";
         Set<String> sigNames = new HashSet<>();
-        // fixme: this ignores non-terminal fields, so will miss any descendant signatures
-        for (PDField field : getAcroForm().getFields())
+        for (PDField field : getAcroForm().getFieldTree())
         {
             if(field instanceof PDSignatureField)
             {