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/09/12 08:37:05 UTC
svn commit: r1881650 -
/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDAcroForm.java
Author: tilman
Date: Sat Sep 12 08:37:05 2020
New Revision: 1881650
URL: http://svn.apache.org/viewvc?rev=1881650&view=rev
Log:
PDFBOX-4892: refactor
Modified:
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDAcroForm.java
Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDAcroForm.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDAcroForm.java?rev=1881650&r1=1881649&r2=1881650&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDAcroForm.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDAcroForm.java Sat Sep 12 08:37:05 2020
@@ -763,8 +763,6 @@ public final class PDAcroForm implements
*/
private boolean resolveNeedsTranslation(PDAppearanceStream appearanceStream)
{
- boolean needsTranslation = true;
-
PDResources resources = appearanceStream.getResources();
if (resources != null && resources.getXObjectNames().iterator().hasNext())
{
@@ -784,7 +782,7 @@ public final class PDAcroForm implements
float llY = bbox.getLowerLeftY();
if (Float.compare(llX, 0) != 0 && Float.compare(llY, 0) != 0)
{
- needsTranslation = false;
+ return false;
}
}
}
@@ -794,9 +792,10 @@ public final class PDAcroForm implements
// as this might only cause a misplacement
}
}
- return needsTranslation;
}
-
+
+ // a field without specific settings typically needs to be translated
+ // to the correct position
return true;
}