From 454582c1c8ef50b6564c67a82c601419777fb2d1 Mon Sep 17 00:00:00 2001
From: Roly Gutierrez <roly.gutierrez@processmaker.com>
Date: Wed, 26 Jan 2022 11:54:39 -0400
Subject: [PATCH] PMCORE-535 When using ampersands in output document the
 __ERROR__ variable is being set

---
 workflow/engine/classes/model/OutputDocument.php | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/workflow/engine/classes/model/OutputDocument.php b/workflow/engine/classes/model/OutputDocument.php
index a55b1098e..eec28861a 100644
--- a/workflow/engine/classes/model/OutputDocument.php
+++ b/workflow/engine/classes/model/OutputDocument.php
@@ -1,6 +1,7 @@
 <?php
 
 use Illuminate\Filesystem\Filesystem;
+use Illuminate\Support\Facades\Log;
 use ProcessMaker\Core\System;
 
 class OutputDocument extends BaseOutputDocument
@@ -961,10 +962,16 @@ class OutputDocument extends BaseOutputDocument
         }
 
         // Fix the HTML using DOMDocument class
+        libxml_use_internal_errors(true);
         $doc = new DOMDocument('1.0', 'UTF-8');
         if ($content != '') {
             $doc->loadHtml($content);
+            foreach (libxml_get_errors() as $error) {
+                $detail = (array) $error;
+                Log::channel(':OutputDocument::generateTcpdf')->warning('DOMDocument::loadHtml', Bootstrap::context($detail));
+            }
         }
+        libxml_clear_errors();
 
         // Add a page and put the HTML fixed
         $pdf->AddPage();