[RELEASE] iText pdfHtml 5.0.2

Author: iText-CI

CONTRIBUTING.md

@@ -1,6 +1,6 @@
-# Contributing to iText 7 Community
+# Contributing to iText Community
 
-We'd love for you to contribute to our source code and to make **iText 7 Community** even better than it is
+We'd love for you to contribute to our source code and to make **iText Community** even better than it is
 today! Here are the guidelines we'd like you to follow:
 
  - [Question or Problem?](#question)
@@ -15,7 +15,7 @@ today! Here are the guidelines we'd like you to follow:
 
 ## <a name="question">Got a Question or Problem?</a>
 
-If you have questions about how to use **iText 7 Community**, please direct these to [Stack Overflow][stackoverflow].
+If you have questions about how to use **iText Community**, please direct these to [Stack Overflow][stackoverflow].
 
 If you are a customer with a [support agreement][support], you also have direct access to our JIRA and our developers.
 
@@ -55,7 +55,7 @@ chances of your issue being dealt with quickly:
 * **Related Issues** - has a similar issue been reported before?
 * **Suggest a Fix** - if you can't fix the bug yourself, perhaps you can point to what might be
   causing the problem (line of code or commit)
-* **Tag the question** - add the tag `itext7` to your question so we can find it.
+* **Tag the question** - add the tag `itext` to your question so we can find it.
 
 **If you get help, help others. Good karma rulez!**
 
@@ -79,7 +79,7 @@ Before you submit your pull request consider the following guidelines:
 * Build your changes locally to ensure all the tests pass.
 * Push your changes to your GitHub account.
 * Create a pull request in GitHub.
-"Head fork" should be your repository, and the "base fork" should be the iText7 official repository.
+"Head fork" should be your repository, and the "base fork" should be the iText official repository.
 * If we suggest changes then:
   * Make the required updates.
   * Fix up your commits if needed, with an interactive rebase.
@@ -111,7 +111,7 @@ To ensure consistency throughout the source code, keep these rules in mind as yo
 
 We have guidelines on how our git commit messages should be formatted. This leads to **more
 readable messages** that are easy to follow when looking through the **project history**. But also,
-we use the git commit messages to **generate the iText 7 Community change log**.
+we use the git commit messages to **generate the iText Community change log**.
 
 These guidelines were taken from Chris Beams' blog post [How to Write a Git Commit Message][git-commit].
 
@@ -169,7 +169,7 @@ We use the [Stack Exchange][stackoverflow] network for free support and [GitHub]
 [javadocs]: https://itextpdf.com/api
 [pull]: https://github.com/itext/i7j-pdfhtml/pulls
 [sscce]: http://sscce.org/
-[stackoverflow]: https://stackoverflow.com/questions/tagged/itext7
+[stackoverflow]: https://stackoverflow.com/questions/tagged/itext
 [good-questions]: https://stackoverflow.com/help/how-to-ask
 [mcve]: https://stackoverflow.com/help/mcve
 [support]: https://itextpdf.com/support

README.md

@@ -1,4 +1,4 @@
-**[pdfHtml][pdfHtml]** is an [iText 7][itext] add on.
+**[pdfHtml][pdfHtml]** is an [iText][itext] add on.
 
 This add on will allow you to easily convert HTML to PDF or iText objects. 
 
@@ -35,4 +35,4 @@ Contact [sales] for more info.
 [latest]: https://github.com/itext/i7j-pdfhtml/releases/latest
 [sales]: http://itextpdf.com/sales
 [gratis]: https://en.wikipedia.org/wiki/Gratis_versus_libre
-[pdfHtml]: http://itextpdf.com/itext7/pdfHtml
+[pdfHtml]: https://itextpdf.com/products/convert-html-css-to-pdf-pdfhtml

pom.xml

@@ -5,20 +5,20 @@
   <parent>
     <groupId>com.itextpdf</groupId>
     <artifactId>root</artifactId>
-    <version>8.0.1</version>
+    <version>8.0.2</version>
     <relativePath />
   </parent>
 
   <artifactId>html2pdf</artifactId>
-  <version>5.0.1</version>
+  <version>5.0.2</version>
 
   <name>pdfHTML</name>
   <description>pdfHTML is an iText add-on that lets you to parse (X)HTML snippets and the associated CSS and converts
     them to PDF.</description>
 
   <scm>
-    <connection>scm:git:ssh://git@git.itextsupport.com:7999/i7j/html2pdf.git</connection>
-    <url>https://git.itextsupport.com/projects/I7J/repos/html2pdf</url>
+    <connection>git@github.com:itext/i7j-pdfhtml.git</connection>
+    <url>https://github.com/itext/i7j-pdfhtml</url>
   </scm>
 
   <properties>

src/main/java/com/itextpdf/html2pdf/ConverterProperties.java

@@ -22,10 +22,10 @@ This file is part of the iText (R) project.
  */
 package com.itextpdf.html2pdf;
 
+import com.itextpdf.commons.actions.contexts.IMetaInfo;
 import com.itextpdf.html2pdf.attach.ITagWorkerFactory;
 import com.itextpdf.html2pdf.attach.impl.OutlineHandler;
 import com.itextpdf.html2pdf.css.apply.ICssApplierFactory;
-import com.itextpdf.commons.actions.contexts.IMetaInfo;
 import com.itextpdf.layout.font.FontProvider;
 import com.itextpdf.styledxmlparser.css.media.MediaDeviceDescription;
 import com.itextpdf.styledxmlparser.resolver.resource.IResourceRetriever;
@@ -326,8 +326,11 @@ public boolean isCreateAcroForm() {
      * If createAcroForm is set, then when the form is encountered in HTML, AcroForm will be created, otherwise
      * a visually identical, but not functional element will be created. Please bare in mind that the created
      * Acroform may visually differ a bit from the HTML one.
+     * <p>
+     * When enabling this acroform creation, it will disable the immediateFlushing property.
      *
      * @param createAcroForm true if an AcroForm needs to be created
+     *
      * @return the {@link ConverterProperties} instance
      */
     public ConverterProperties setCreateAcroForm(boolean createAcroForm) {

src/main/java/com/itextpdf/html2pdf/actions/data/PdfHtmlProductData.java

@@ -30,7 +30,7 @@ This file is part of the iText (R) project.
  */
 public final class PdfHtmlProductData {
     private static final String PDF_HTML_PUBLIC_PRODUCT_NAME = "pdfHTML";
-    private static final String PDF_HTML_VERSION = "5.0.1";
+    private static final String PDF_HTML_VERSION = "5.0.2";
     private static final int PDF_HTML_COPYRIGHT_SINCE = 2000;
     private static final int PDF_HTML_COPYRIGHT_TO = 2023;
 

src/main/java/com/itextpdf/html2pdf/attach/impl/tags/HtmlTagWorker.java

@@ -31,6 +31,7 @@ This file is part of the iText (R) project.
 import com.itextpdf.html2pdf.attach.util.WaitingInlineElementsHelper;
 import com.itextpdf.html2pdf.css.CssConstants;
 import com.itextpdf.html2pdf.html.AttributeConstants;
+import com.itextpdf.html2pdf.logs.Html2PdfLogMessageConstant;
 import com.itextpdf.io.font.PdfEncodings;
 import com.itextpdf.kernel.pdf.PdfDocument;
 import com.itextpdf.kernel.pdf.PdfString;
@@ -45,15 +46,24 @@ This file is part of the iText (R) project.
 import com.itextpdf.styledxmlparser.node.IElementNode;
 import com.itextpdf.styledxmlparser.node.INode;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 /**
  * TagWorker class for the {@code html} element.
  */
 public class HtmlTagWorker implements ITagWorker {
 
-    /** The iText document instance. */
+    private static final Logger LOGGER = LoggerFactory.getLogger(HtmlTagWorker.class);
+
+    /**
+     * The iText document instance.
+     */
     private Document document;
 
-    /** Helper class for waiting inline elements. */
+    /**
+     * Helper class for waiting inline elements.
+     */
     private WaitingInlineElementsHelper inlineHelper;
 
     /**
@@ -65,14 +75,19 @@ public class HtmlTagWorker implements ITagWorker {
     public HtmlTagWorker(IElementNode element, ProcessorContext context) {
         // TODO DEVSIX-4261 more precise check if a counter was actually added to the document
         boolean immediateFlush =
-                context.isImmediateFlush() && !context.getCssContext().isPagesCounterPresent();
+                context.isImmediateFlush() && !context.getCssContext().isPagesCounterPresent()
+                        && !context.isCreateAcroForm();
+        if (context.isImmediateFlush() && context.isCreateAcroForm()) {
+            LOGGER.info(Html2PdfLogMessageConstant.IMMEDIATE_FLUSH_DISABLED);
+        }
         PdfDocument pdfDocument = context.getPdfDocument();
         document = new HtmlDocument(pdfDocument, pdfDocument.getDefaultPageSize(), immediateFlush);
         document.setRenderer(new HtmlDocumentRenderer(document, immediateFlush));
 
         DefaultHtmlProcessor.setConvertedRootElementProperties(element.getStyles(), context, document);
 
-        inlineHelper = new WaitingInlineElementsHelper(element.getStyles().get(CssConstants.WHITE_SPACE), element.getStyles().get(CssConstants.TEXT_TRANSFORM));
+        inlineHelper = new WaitingInlineElementsHelper(element.getStyles().get(CssConstants.WHITE_SPACE),
+                element.getStyles().get(CssConstants.TEXT_TRANSFORM));
 
         String lang = element.getAttribute(AttributeConstants.LANG);
         if (lang != null) {

src/main/java/com/itextpdf/html2pdf/attach/impl/tags/ImgTagWorker.java

@@ -36,6 +36,8 @@ This file is part of the iText (R) project.
 import com.itextpdf.layout.element.Image;
 import com.itextpdf.layout.properties.ObjectFit;
 import com.itextpdf.styledxmlparser.node.IElementNode;
+import com.itextpdf.svg.element.SvgImage;
+import com.itextpdf.svg.xobject.SvgImageXObject;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -71,6 +73,8 @@ public ImgTagWorker(IElementNode element, ProcessorContext context) {
         if (imageXObject != null) {
             if (imageXObject instanceof PdfImageXObject) {
                 image = new HtmlImage((PdfImageXObject) imageXObject);
+            } else if (imageXObject instanceof SvgImageXObject) {
+                image = new SvgImage((SvgImageXObject) imageXObject);
             } else if (imageXObject instanceof PdfFormXObject) {
                 image = new HtmlImage((PdfFormXObject) imageXObject);
             } else {

src/main/java/com/itextpdf/html2pdf/attach/impl/tags/ObjectTagWorker.java

@@ -31,7 +31,6 @@ This file is part of the iText (R) project.
 import com.itextpdf.html2pdf.util.SvgProcessingUtil;
 import com.itextpdf.commons.utils.FileUtil;
 import com.itextpdf.commons.utils.MessageFormatUtil;
-import com.itextpdf.kernel.pdf.PdfDocument;
 import com.itextpdf.layout.IPropertyContainer;
 import com.itextpdf.layout.element.Image;
 import com.itextpdf.styledxmlparser.node.IElementNode;
@@ -117,17 +116,10 @@ private boolean isSvgImage(String typeAttribute) {
 
     @Override
     public void processEnd(IElementNode element, ProcessorContext context) {
-        if (context.getPdfDocument() != null) {
-            PdfDocument document = context.getPdfDocument();
-            //Create Image object
-
-            if (res != null) {
-                image = processUtil.createImageFromProcessingResult(res, document);
-                AccessiblePropHelper.trySetLangAttribute(image, element);
-            }
-
-        } else {
-            LOGGER.error(Html2PdfLogMessageConstant.PDF_DOCUMENT_NOT_PRESENT);
+        // Create Image object
+        if (res != null) {
+            image = processUtil.createSvgImageFromProcessingResult(res);
+            AccessiblePropHelper.trySetLangAttribute(image, element);
         }
     }
 

src/main/java/com/itextpdf/html2pdf/attach/impl/tags/SvgTagWorker.java

@@ -68,9 +68,9 @@ public SvgTagWorker(IElementNode element, ProcessorContext context) {
 
     @Override
     public void processEnd(IElementNode element, ProcessorContext context) {
-        if (context.getPdfDocument() != null && processingResult != null) {
+        if (processingResult != null) {
             SvgProcessingUtil util = new SvgProcessingUtil(context.getResourceResolver());
-            svgImage = util.createImageFromProcessingResult(processingResult, context.getPdfDocument());
+            svgImage = util.createSvgImageFromProcessingResult(processingResult);
 
             AccessiblePropHelper.trySetLangAttribute(svgImage, element);
             context.endProcessingInlineSvg();

src/main/java/com/itextpdf/html2pdf/css/apply/util/BackgroundApplierUtil.java

@@ -26,8 +26,6 @@ This file is part of the iText (R) project.
 import com.itextpdf.html2pdf.attach.ProcessorContext;
 import com.itextpdf.html2pdf.css.CssConstants;
 import com.itextpdf.commons.utils.MessageFormatUtil;
-import com.itextpdf.kernel.colors.Color;
-import com.itextpdf.kernel.colors.DeviceRgb;
 import com.itextpdf.kernel.colors.gradients.StrategyBasedLinearGradientBuilder;
 import com.itextpdf.kernel.pdf.xobject.PdfFormXObject;
 import com.itextpdf.kernel.pdf.xobject.PdfImageXObject;