tiagobento commented on issue #665:
URL: 
https://github.com/apache/incubator-kie-issues/issues/665#issuecomment-1787935746

   Right, right. Good catch. I don't think DMN nor BPMN have this use-case, but 
yeah, indeed a flaw on `xml-parser-ts` that needs to be addressed. We'll have 
to introduce a convention for this, which I'm not a big fan of, but I don't see 
any other way. Text content will have to be given its own attribute (always), 
and we'll need to add special treatment for it on the `build` function of 
`xml-parser-ts` too.
   
   In the example you share, we'll need to generate a type
   
   ```ts
   export type SceSim__rawValueType = {
     "__$$text": string;
     "@_class": string;
   }
   ```
   
   Note that `__$$text` follows the same absurdity as the `__$$element` 
property we already have for resolving substitution groups. 
   
   This is not a very complicated change, but definitely will impact everyone 
using `xml-parser-ts` right now, as it will introduce breaking changes, for 
example, to `typeRef`s on DMN.
   
   @yesamer How urgent is this for 
   - #453 
   
   ?
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to