Kaleo Designer adds a <recipients> element to XML document when changing to "View" mode

• Type: Bug
• Status: Closed
• Resolution: Fixed
• Affects Version/s: Master
• Fix Version/s: 7.0.0 DXP FP58, 7.0.0 DXP SP9, 7.0.X, 7.1.0 Beta 1, 7.1.X, 7.2.X, Master
• Component/s: Business Productivity > Kaleo Designer
• Labels:
  • QA-F
  • ee-ts
  • fw_code_review
  • fw_test_needed
  • liferay-dxp-70-sp9
  • liferay-fixpack-de-58-7010
• Branch Version/s:
  7.2.x, 7.1.x, 7.0.x
• Backported to Branch:
  Committed
• Story Points: 1
• Fix Priority: 4
• Git Pull Request: https://github.com/brianchandotcom/liferay-portal-ee/pull/18414

When we add an XML document to the "Source" from Kaleo Designer and then go to the "View" mode, some content can be added to specific nodes, resulting in a different workflow definition than the one first given.

Steps to reproduce

1. Go to Kaleo Designer.
2. Go to the "Source (Kaleo XML)" mode.
3. Replace the content of the editor with the content from single-approver-3.xml.
4. Go back to the "View" mode.
5. Go back to the "Source (Kaleo XML)" mode.
6. Compare the resulting XML document to the original one we got from single-approver-3.xml .

Expected results

• The content we got from the editor would be identical to the one from single-approver-3.xml. At most, some formatting changes would be applied.

Actual results

• Where we had

  <notification>
      <name>Creator Modification Notification</name>
      <template>
          <![CDATA[]]>
      </template>
      <template-language>freemarker</template-language>
      <notification-type>email</notification-type>
      <notification-type>user-notification</notification-type>
      <execution-type>onAssignment</execution-type>
  </notification>
  
  

  now we have this:

      <name>Creator Modification Notification</name>
      <template>
          <![CDATA[]]>
      </template>
      <template-language>freemarker</template-language>
      <notification-type>user-notification</notification-type>
      <recipients>
          <user/>
      </recipients>
      <execution-type>onAssignment</execution-type>
  </notification>
  

  In this new result, the lines below were added:

  <recipients>
      <user/>
  </recipients>
  

  The same happened to the node at line 85.