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 2020/07/07 17:40:42 UTC
svn commit: r1879603 -
/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDSignatureField.java
Author: tilman
Date: Tue Jul 7 17:40:42 2020
New Revision: 1879603
URL: http://svn.apache.org/viewvc?rev=1879603&view=rev
Log:
PDFBOX-4907: avoid reusing all type of field names, not just signature ones
Modified:
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDSignatureField.java
Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDSignatureField.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDSignatureField.java?rev=1879603&r1=1879602&r2=1879603&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDSignatureField.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDSignatureField.java Tue Jul 7 17:40:42 2020
@@ -76,16 +76,13 @@ public class PDSignatureField extends PD
private String generatePartialName()
{
String fieldName = "Signature";
- Set<String> sigNames = new HashSet<String>();
+ Set<String> nameSet = new HashSet<String>();
for (PDField field : getAcroForm().getFieldTree())
{
- if(field instanceof PDSignatureField)
- {
- sigNames.add(field.getPartialName());
- }
+ nameSet.add(field.getPartialName());
}
int i = 1;
- while(sigNames.contains(fieldName+i))
+ while (nameSet.contains(fieldName + i))
{
++i;
}