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 2020/12/12 18:34:26 UTC
svn commit: r1884362 - in /pdfbox/branches/2.0/pdfbox/src:
main/java/org/apache/pdfbox/pdmodel/interactive/form/
test/java/org/apache/pdfbox/pdmodel/interactive/form/
Author: msahyoun
Date: Sat Dec 12 18:34:26 2020
New Revision: 1884362
URL: http://svn.apache.org/viewvc?rev=1884362&view=rev
Log:
PDFBOX-5043: avoid StringIndexOutOfBoundsException; skip generation for PMD widgets
Modified:
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/AppearanceGeneratorHelper.java
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PlainText.java
pdfbox/branches/2.0/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/interactive/form/PDAcroFormGenerateAppearancesTest.java
Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/AppearanceGeneratorHelper.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/AppearanceGeneratorHelper.java?rev=1884362&r1=1884361&r2=1884362&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/AppearanceGeneratorHelper.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/AppearanceGeneratorHelper.java Sat Dec 12 18:34:26 2020
@@ -170,6 +170,13 @@ class AppearanceGeneratorHelper {
}
for (PDAnnotationWidget widget : field.getWidgets()) {
+
+ if (widget.getCOSObject().containsKey("PMD"))
+ {
+ LOG.warn("widget of field " + field.getFullyQualifiedName() + " is a PaperMetaData widet, no appearance stream created");
+ continue;
+ }
+
// some fields have the /Da at the widget level if the
// widgets differ in layout.
PDDefaultAppearanceString acroFormAppearance = defaultAppearance;
Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PlainText.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PlainText.java?rev=1884362&r1=1884361&r2=1884362&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PlainText.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PlainText.java Sat Dec 12 18:34:26 2020
@@ -206,7 +206,7 @@ class PlainText
while (true)
{
splitOffset--;
- String substring = word.trim().substring(0, splitOffset);
+ String substring = word.substring(0, splitOffset);
float substringWidth = font.getStringWidth(substring) * scale;
if (substringWidth < width)
{
Modified: pdfbox/branches/2.0/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/interactive/form/PDAcroFormGenerateAppearancesTest.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/interactive/form/PDAcroFormGenerateAppearancesTest.java?rev=1884362&r1=1884361&r2=1884362&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/interactive/form/PDAcroFormGenerateAppearancesTest.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/interactive/form/PDAcroFormGenerateAppearancesTest.java Sat Dec 12 18:34:26 2020
@@ -93,4 +93,36 @@ public class PDAcroFormGenerateAppearanc
IOUtils.closeQuietly(testPdf);
}
}
+
+ /**
+ * PDFBOX-5043 PaperMetaData
+ * @throws IOException
+ */
+ @Test
+ public void test5043PaperMetaData() throws IOException
+ {
+ String sourceUrl = "https://issues.apache.org/jira/secure/attachment/13016992/PDFBOX-3891-5.pdf";
+
+ PDDocument testPdf = null;
+ try
+ {
+ testPdf = PDDocument.load(new URL(sourceUrl).openStream());
+ PDDocumentCatalog catalog = testPdf.getDocumentCatalog();
+ boolean thrown = false;
+ try
+ {
+ catalog.getAcroForm();
+ }
+ catch (Exception e)
+ {
+ thrown = true;
+ }
+ assertFalse("There shall be no exception when getting the AcroForm", thrown);
+ }
+ finally
+ {
+ IOUtils.closeQuietly(testPdf);
+ }
+ }
+
}