From 495fa542c9575b498fab0a6b4cc1b77904292ea4 Mon Sep 17 00:00:00 2001
From: Roly Rudy Gutierrez Pinto <gproly@gmail.com>
Date: Thu, 13 Jul 2017 12:18:58 -0400
Subject: [PATCH 1/2] HOR-3489

---
 .../engine/methods/cases/casesMenuLoader.php  | 37 ++-----------------
 1 file changed, 3 insertions(+), 34 deletions(-)

diff --git a/workflow/engine/methods/cases/casesMenuLoader.php b/workflow/engine/methods/cases/casesMenuLoader.php
index d45138fe7..d9371d60b 100644
--- a/workflow/engine/methods/cases/casesMenuLoader.php
+++ b/workflow/engine/methods/cases/casesMenuLoader.php
@@ -177,42 +177,11 @@ function getLoadTreeMenuData ()
         }
     }
 
+    //This function generates an xml, so it prevents the output of a badly formed xml 
+    //by cleaning any content prior to this function with ob_clean
+    ob_clean();
     echo $xml->asXML();
     die;
-
-    // Build xml document for all tree nodes
-    /*$xml = '<menu_cases>';
-    $i = 0;
-    foreach ($menuCases as $menu => $aMenuBlock) {
-        if (isset( $aMenuBlock['blockItems'] ) && sizeof( $aMenuBlock['blockItems'] ) > 0) {
-            $urlProperty = "";
-            if ((isset( $aMenuBlock['link'] )) && ($aMenuBlock['link'] != "")) {
-                $urlProperty = "url='" . $aMenuBlock['link'] . "'";
-            }
-            $xml .= '<menu_block blockTitle="' . $aMenuBlock['blockTitle'] . '" id="' . $menu . '" ' . $urlProperty . '>';
-            foreach ($aMenuBlock['blockItems'] as $id => $aMenu) {
-                $i ++;
-                if (isset( $aMenu['cases_count'] ) && $aMenu['cases_count'] !== '') {
-                    $nofifier = "cases_count=\"{$aMenu['cases_count']}\" ";
-                } else {
-                    $nofifier = '';
-                }
-                $xml .= '<option title="' . $aMenu['label'] . '" id="' . $id . '" ' . $nofifier . ' url="' . $aMenu['link'] . '">';
-                $xml .= '</option>';
-            }
-            $xml .= '</menu_block>';
-        } elseif (isset( $aMenuBlock['blockType'] ) && $aMenuBlock['blockType'] == "blockNestedTree") {
-            $xml .= '<menu_block blockTitle="' . $aMenuBlock['blockTitle'] . '" blockNestedTree = "' . $aMenuBlock['loaderurl'] . '" id="' . $menu . '" folderId="0">';
-            $xml .= '</menu_block>';
-        } elseif (isset( $aMenuBlock['blockType'] ) && $aMenuBlock['blockType'] == "blockHeaderNoChild") {
-            $xml .= '<menu_block blockTitle="' . $aMenuBlock['blockTitle'] . '" blockHeaderNoChild="blockHeaderNoChild" url = "' . $aMenuBlock['link'] . '" id="' . $menu . '">';
-            //$xml .= '<option title="" id="" ></option>';
-            $xml .= '</menu_block>';
-        }
-    }
-    $xml .= '</menu_cases>';
-
-    print $xml;*/
 }
 
 /*----------------------------------********---------------------------------*/

From b7d7abc365aed82d33163d1b6ba97c4e7aef7419 Mon Sep 17 00:00:00 2001
From: Roly Rudy Gutierrez Pinto <gproly@gmail.com>
Date: Fri, 14 Jul 2017 08:02:14 -0400
Subject: [PATCH 2/2] HOR-3489 Improvement

---
 workflow/engine/methods/cases/casesMenuLoader.php | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/workflow/engine/methods/cases/casesMenuLoader.php b/workflow/engine/methods/cases/casesMenuLoader.php
index d9371d60b..7f24922c3 100644
--- a/workflow/engine/methods/cases/casesMenuLoader.php
+++ b/workflow/engine/methods/cases/casesMenuLoader.php
@@ -116,6 +116,9 @@ function getLoadTreeMenuData ()
             }
         }
 
+        //This function generates an xml, so it prevents the output of a badly formed xml 
+        //by cleaning any content prior to this function with ob_clean
+        ob_clean();
         echo $xml->asXML();
         die;
     }