Clearing element template property (type=String) value leads to left overs in XML

[nikku]

Describe the Bug

When I clear the value of an element template property I expect the value to be removed from the XML if it was otherwise illegal: zeebe:input, zeebe:output, zeebe:property, zeebe:taskHeader, zeebe:taskDefinition:type. Instead what I'm seeing is that the value is kept, leading to illegal BPMN 2.0 XML, i.e.

        <zeebe:ioMapping>
          <zeebe:input target="test" />
          <zeebe:output source="test" />
        </zeebe:ioMapping>

Steps to Reproduce

1. npm run start:cloud-templates
2. Open test_leftover.bpmn
3. clear inputs
4. export
5. Invalid left-overs (cf. test_leftover_after.bpmn

Expected Behavior

Empty inputs result in removed bindings. Validation indicates whether the property is needed.

Environment

• Host (Browser/Node version), if applicable: Any
• OS: Any
• Library version: latest

---

Related to #824 (review).

[smbea]

The expectation is that we also clean up ioMappings/taskHeaders and such elements if there are no values? That is our usual stance on such things, if I'm not mistaken

Example: currently, the zeebe:header empty values aren't persisted but we are left with a <zeebe:taskHeaders /> without any children inside.

[nikku]

That is fine / could be a follow up. But it is non-critical. I.e. Zeebe engine just fine handles these cases.

[nikku]

Closing as not a bug: #824 (comment).