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 2018/03/29 10:36:12 UTC
svn commit: r1827971 -
/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDAcroForm.java
Author: msahyoun
Date: Thu Mar 29 10:36:12 2018
New Revision: 1827971
URL: http://svn.apache.org/viewvc?rev=1827971&view=rev
Log:
PDFBOX-4157, PDFBOX-4172: adjust resolveNeedsTranslation to only resolve to false if PDFormXObject already has translation
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=1827971&r1=1827970&r2=1827971&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 Thu Mar 29 10:36:12 2018
@@ -730,7 +730,7 @@ public final class PDAcroForm implements
*/
private boolean resolveNeedsTranslation(PDAppearanceStream appearanceStream)
{
- boolean needsTranslation = false;
+ boolean needsTranslation = true;
PDResources resources = appearanceStream.getResources();
if (resources != null && resources.getXObjectNames().iterator().hasNext())
@@ -750,9 +750,9 @@ public final class PDAcroForm implements
PDRectangle bbox = ((PDFormXObject)xObject).getBBox();
float llX = bbox.getLowerLeftX();
float llY = bbox.getLowerLeftY();
- if (llX == 0 && llY == 0)
+ if (Float.compare(llX, 0) != 0 && Float.compare(llY, 0) != 0)
{
- needsTranslation = true;
+ needsTranslation = false;
}
}
}