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

Posted to commits@pdfbox.apache.org by ms...@apache.org on 2015/09/11 15:14:41 UTC

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

Author: msahyoun
Date: Fri Sep 11 13:14:40 2015
New Revision: 1702456

URL: http://svn.apache.org/r1702456
Log:
PDFBOX-2597: add kids to field cache

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

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDAcroForm.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDAcroForm.java?rev=1702456&r1=1702455&r2=1702456&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDAcroForm.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDAcroForm.java Fri Sep 11 13:14:40 2015
@@ -223,9 +223,8 @@ public final class PDAcroForm implements
         if (cache)
         {
             fieldCache = new HashMap<String, PDField>();
-            // fixme: this code does not cache non-terminal fields or their kids
-            List<PDField> fields = getFields();
-            for (PDField field : fields)
+
+            for (PDField field : getFieldTree())
             {
                 fieldCache.put(field.getFullyQualifiedName(), field);
             }