Skip to content

Commit 1ecd7f2

Browse files
committed
Remove default appearance from AcroForm.
DEVSIX-1677
1 parent 83e94be commit 1ecd7f2

16 files changed

+6
-7
lines changed

forms/src/main/java/com/itextpdf/forms/PdfAcroForm.java

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,6 @@ This file is part of the iText (R) project.
5252
import com.itextpdf.kernel.pdf.PdfBoolean;
5353
import com.itextpdf.kernel.pdf.PdfDictionary;
5454
import com.itextpdf.kernel.pdf.PdfDocument;
55-
import com.itextpdf.kernel.pdf.PdfIndirectReference;
5655
import com.itextpdf.kernel.pdf.PdfName;
5756
import com.itextpdf.kernel.pdf.PdfNumber;
5857
import com.itextpdf.kernel.pdf.PdfObject;
@@ -65,6 +64,8 @@ This file is part of the iText (R) project.
6564
import com.itextpdf.kernel.pdf.tagutils.TagReference;
6665
import com.itextpdf.kernel.pdf.tagutils.TagTreePointer;
6766
import com.itextpdf.kernel.pdf.xobject.PdfFormXObject;
67+
import org.slf4j.Logger;
68+
import org.slf4j.LoggerFactory;
6869

6970
import java.util.ArrayList;
7071
import java.util.Arrays;
@@ -74,9 +75,6 @@ This file is part of the iText (R) project.
7475
import java.util.Map;
7576
import java.util.Set;
7677

77-
import org.slf4j.Logger;
78-
import org.slf4j.LoggerFactory;
79-
8078
/**
8179
* This class represents the static form technology AcroForm on a PDF file.
8280
*/
@@ -129,7 +127,6 @@ public class PdfAcroForm extends PdfObjectWrapper<PdfDictionary> {
129127
*/
130128
protected PdfDocument document;
131129

132-
Logger logger = LoggerFactory.getLogger(PdfAcroForm.class);
133130
private static PdfName[] resourceNames = {PdfName.Font, PdfName.XObject, PdfName.ColorSpace, PdfName.Pattern};
134131
private PdfDictionary defaultResources;
135132
private Set<PdfFormField> fieldsForFlattening = new LinkedHashSet<>();
@@ -178,7 +175,6 @@ public static PdfAcroForm getAcroForm(PdfDocument document, boolean createIfNotE
178175
acroForm.makeIndirect(document);
179176
document.getCatalog().put(PdfName.AcroForm, acroForm.getPdfObject());
180177
document.getCatalog().setModified();
181-
acroForm.setDefaultAppearance("/Helv 0 Tf 0 g ");
182178
}
183179
} else {
184180
acroForm = new PdfAcroForm(acroFormDictionary, document);
@@ -839,6 +835,7 @@ public void replaceField(String name, PdfFormField field) {
839835
protected PdfArray getFields() {
840836
PdfArray fields = getPdfObject().getAsArray(PdfName.Fields);
841837
if (fields == null) {
838+
Logger logger = LoggerFactory.getLogger(PdfAcroForm.class);
842839
logger.warn(LogMessageConstant.NO_FIELDS_IN_ACROFORM);
843840
fields = new PdfArray();
844841
getPdfObject().put(PdfName.Fields, fields);
@@ -855,6 +852,7 @@ private Map<String, PdfFormField> iterateFields(PdfArray array, Map<String, PdfF
855852
int index = 1;
856853
for (PdfObject field : array) {
857854
if (field.isFlushed()) {
855+
Logger logger = LoggerFactory.getLogger(PdfAcroForm.class);
858856
logger.warn(LogMessageConstant.FORM_FIELD_WAS_FLUSHED);
859857
continue;
860858
}

forms/src/main/java/com/itextpdf/forms/fields/PdfFormField.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1497,7 +1497,7 @@ public PdfString getDefaultAppearance() {
14971497
}
14981498
}
14991499
}
1500-
//DA is an inherited key, therefore AcroForm shall be checked if there is no parent or no DA in parent.
1500+
// DA is an inherited key, therefore AcroForm shall be checked if there is no parent or no DA in parent.
15011501
if (defaultAppearance == null) {
15021502
defaultAppearance = (PdfString) getAcroFormKey(PdfName.DA, PdfObject.STRING);
15031503
}
@@ -1522,6 +1522,7 @@ public PdfFormField setDefaultAppearance(String defaultAppearance) {
15221522
return this;
15231523
}
15241524

1525+
15251526
/**
15261527
* Gets a code specifying the form of quadding (justification) to be used in displaying the text:
15271528
* 0 Left-justified

0 commit comments

Comments
 (0)